[PATCH] ext3_readdir: use generic readahead

Linus points out that ext3_readdir's readahead only cuts in when
ext3_readdir() is operating at the very start of the directory.  So for large
directories we end up performing no readahead at all and we suck.

So take it all out and use the core VM's page_cache_readahead().  This means
that ext3 directory reads will use all of readahead's dynamic sizing goop.

Note that we're using the directory's filp->f_ra to hold the readahead state,
but readahead is actually being performed against the underlying blockdev's
address_space.  Fortunately the readahead code is all set up to handle this.

Tested with printk.  It works.  I was struggling to find a real workload which
actually cared.

(The patch also exports page_cache_readahead() to GPL modules)

Cc: "Stephen C. Tweedie" <sct@redhat.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

1 files changed, 24 insertions, 28 deletions

diff --git a/fs/ext3/dir.c b/fs/ext3/dir.c
index 832867aef3dc..773459164bb2 100644
--- a/fs/ext3/dir.c
+++ b/fs/ext3/dir.c
@@ -95,11 +95,10 @@ static int ext3_readdir(struct file * filp,
                          void * dirent, filldir_t filldir)
 {
         int error = 0;
-        unsigned long offset, blk;
-        int i, num, stored;
-        struct buffer_head * bh, * tmp, * bha[16];
-        struct ext3_dir_entry_2 * de;
-        struct super_block * sb;
+        unsigned long offset;
+        int i, stored;
+        struct ext3_dir_entry_2 *de;
+        struct super_block *sb;
         int err;
         struct inode *inode = filp->f_dentry->d_inode;
         int ret = 0;
@@ -124,12 +123,29 @@ static int ext3_readdir(struct file * filp,
         }
 #endif
         stored = 0;
-        bh = NULL;
         offset = filp->f_pos & (sb->s_blocksize - 1);
 
         while (!error && !stored && filp->f_pos < inode->i_size) {
-                blk = (filp->f_pos) >> EXT3_BLOCK_SIZE_BITS(sb);
-                bh = ext3_bread(NULL, inode, blk, 0, &err);
+                unsigned long blk = filp->f_pos >> EXT3_BLOCK_SIZE_BITS(sb);
+                struct buffer_head map_bh;
+                struct buffer_head *bh = NULL;
+
+                map_bh.b_state = 0;
+                err = ext3_get_block_handle(NULL, inode, blk, &map_bh, 0, 0);
+                if (!err) {
+                        page_cache_readahead(sb->s_bdev->bd_inode->i_mapping,
+                                &filp->f_ra,
+                                filp,
+                                map_bh.b_blocknr >>
+                                        (PAGE_CACHE_SHIFT - inode->i_blkbits),
+                                1);
+                        bh = ext3_bread(NULL, inode, blk, 0, &err);
+                }
+
+                /*
+                 * We ignore I/O errors on directories so users have a chance
+                 * of recovering data when there's a bad sector
+                 */
                 if (!bh) {
                         ext3_error (sb, "ext3_readdir",
                                 "directory #%lu contains a hole at offset %lu",
@@ -138,26 +154,6 @@ static int ext3_readdir(struct file * filp,
                         continue;
                 }
 
-                /*
-                 * Do the readahead
-                 */
-                if (!offset) {
-                        for (i = 16 >> (EXT3_BLOCK_SIZE_BITS(sb) - 9), num = 0;
-                             i > 0; i--) {
-                                tmp = ext3_getblk (NULL, inode, ++blk, 0, &err);
-                                if (tmp && !buffer_uptodate(tmp) &&
-                                                !buffer_locked(tmp))
-                                        bha[num++] = tmp;
-                                else
-                                        brelse (tmp);
-                        }
-                        if (num) {
-                                ll_rw_block (READA, num, bha);
-                                for (i = 0; i < num; i++)
-                                        brelse (bha[i]);
-                        }
-                }
-
 revalidate:
                 /* If the dir block has changed since the last call to
                  * readdir(2), then we might be pointing to an invalid