On Wed, Oct 17, 2012 at 12:28:05AM +0530, raghu.prabhu13@xxxxxxxxx wrote:
> From: Raghavendra D Prabhu <rprabhu@xxxxxxxxxxx>
>
> >From 51daa88ebd8e0d437289f589af29d4b39379ea76, page_sync_readahead coalesces
> async readahead into its readahead window, so another checking for that again is
> not required.
>
> Signed-off-by: Raghavendra D Prabhu <rprabhu@xxxxxxxxxxx>
> ---
> fs/btrfs/relocation.c | 10 ++++------
> mm/filemap.c | 3 +--
> 2 files changed, 5 insertions(+), 8 deletions(-)
>
> diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c
> index 4da0865..6362003 100644
> --- a/fs/btrfs/relocation.c
> +++ b/fs/btrfs/relocation.c
> @@ -2996,12 +2996,10 @@ static int relocate_file_extent_cluster(struct inode *inode,
> ret = -ENOMEM;
> goto out;
> }
> - }
> -
> - if (PageReadahead(page)) {
> - page_cache_async_readahead(inode->i_mapping,
> - ra, NULL, page, index,
> - last_index + 1 - index);
> + } else if (PageReadahead(page)) {
> + page_cache_async_readahead(inode->i_mapping,
> + ra, NULL, page, index,
> + last_index + 1 - index);
That extra indent is not necessary.
Otherwise looks good to me. Thanks!
Reviewed-by: Fengguang Wu <fengguang.wu@xxxxxxxxx>
> }
>
> if (!PageUptodate(page)) {
> diff --git a/mm/filemap.c b/mm/filemap.c
> index 3843445..d703224 100644
> --- a/mm/filemap.c
> +++ b/mm/filemap.c
> @@ -1113,8 +1113,7 @@ find_page:
> page = find_get_page(mapping, index);
> if (unlikely(page == NULL))
> goto no_cached_page;
> - }
> - if (PageReadahead(page)) {
> + } else if (PageReadahead(page)) {
> page_cache_async_readahead(mapping,
> ra, filp, page,
> index, last_index - index);
> --
> 1.7.12.3
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
