diff options
author | Josef Bacik <josef@redhat.com> | 2011-05-13 10:32:11 -0400 |
---|---|---|
committer | Josef Bacik <josef@redhat.com> | 2011-05-23 13:03:14 -0400 |
commit | 026fd317828500524cdc7e5ff9e8e7923abb2868 (patch) | |
tree | d303bffbd895c307f49286cd90186cf58223dde2 /fs/btrfs/ctree.c | |
parent | 589d8ade83f07c0f11c8191c0ca309f34d7a2c14 (diff) |
Btrfs: don't always do readahead
Our readahead is sort of sloppy, and really isn't always needed. For example if
ls is doing a stating ls (which is the default) it's going to stat in non-disk
order, so if say you have a directory with a stupid amount of files, readahead
is going to do nothing but waste time in the case of doing the stat. Taking the
unconditional readahead out made my test go from 57 minutes to 36 minutes. This
means that everywhere we do loop through the tree we want to make sure we do set
path->reada properly, so I went through and found all of the places where we
loop through the path and set reada to 1. Thanks,
Signed-off-by: Josef Bacik <josef@redhat.com>
Diffstat (limited to 'fs/btrfs/ctree.c')
-rw-r--r-- | fs/btrfs/ctree.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c index f7a0a64b868f..f61c16c1481a 100644 --- a/fs/btrfs/ctree.c +++ b/fs/btrfs/ctree.c | |||
@@ -48,8 +48,6 @@ struct btrfs_path *btrfs_alloc_path(void) | |||
48 | { | 48 | { |
49 | struct btrfs_path *path; | 49 | struct btrfs_path *path; |
50 | path = kmem_cache_zalloc(btrfs_path_cachep, GFP_NOFS); | 50 | path = kmem_cache_zalloc(btrfs_path_cachep, GFP_NOFS); |
51 | if (path) | ||
52 | path->reada = 1; | ||
53 | return path; | 51 | return path; |
54 | } | 52 | } |
55 | 53 | ||