Re: [PATCH] Change the check for PageReadahead into an else-if
Hi, * On Tue, Oct 16, 2012 at 10:02:44PM -0400, KOSAKI Motohiro kosaki.motoh...@gmail.com wrote: On Tue, Oct 16, 2012 at 10:00 PM, Fengguang Wu fengguang...@intel.com wrote: On Wed, Oct 17, 2012 at 12:28:05AM +0530, raghu.prabh...@gmail.com wrote: From: Raghavendra D Prabhu rpra...@wnohang.net 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 rpra...@wnohang.net --- 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...@intel.com Hi Raghavendra, Indentation breakage is now welcome. Please respin it. Otherwise Acked-by: KOSAKI Motohiro kosaki.motoh...@jp.fujitsu.com Thanks, will do. Regards, -- Raghavendra Prabhu GPG Id : 0xD72BE977 Fingerprint: B93F EBCB 8E05 7039 CD3C A4B8 A616 DCA1 D72B E977 www: wnohang.net pgpci9gnaoBWB.pgp Description: PGP signature
[PATCH] Change the check for PageReadahead into an else-if
From: Raghavendra D Prabhu rpra...@wnohang.net 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 rpra...@wnohang.net --- 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); } 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] Change the check for PageReadahead into an else-if
On Wed, Oct 17, 2012 at 12:28:05AM +0530, raghu.prabh...@gmail.com wrote: From: Raghavendra D Prabhu rpra...@wnohang.net 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 rpra...@wnohang.net --- 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...@intel.com } 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] Change the check for PageReadahead into an else-if
On Tue, Oct 16, 2012 at 10:00 PM, Fengguang Wu fengguang...@intel.com wrote: On Wed, Oct 17, 2012 at 12:28:05AM +0530, raghu.prabh...@gmail.com wrote: From: Raghavendra D Prabhu rpra...@wnohang.net 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 rpra...@wnohang.net --- 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...@intel.com Hi Raghavendra, Indentation breakage is now welcome. Please respin it. Otherwise Acked-by: KOSAKI Motohiro kosaki.motoh...@jp.fujitsu.com -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html