From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wanpeng Li Subject: [PATCH 2/3] f2fs: fix get stale meta pages when build free nids Date: Mon, 9 Mar 2015 11:00:54 +0800 Message-ID: <1425870055-27694-2-git-send-email-wanpeng.li@linux.intel.com> References: <1425870055-27694-1-git-send-email-wanpeng.li@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Wanpeng Li , linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, linux-fsdevel@vger.kernel.org To: Jaegeuk Kim Return-path: In-Reply-To: <1425870055-27694-1-git-send-email-wanpeng.li@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net List-Id: linux-fsdevel.vger.kernel.org The blocks of nat pages to be scanned will be readahead when build free nids. The start blkno will be adjusted to 0 when the intended range over max_nids, then function ra_meta_pages readahead the right blkno, however, function get_current_nat_page which is called in build_free_nids still get the meta pages w/ stale blkno, this patch fix it by adjusting the nid and the blkno which will be readahead in avdance in order to readahead and get the right meta pages. Signed-off-by: Wanpeng Li --- fs/f2fs/checkpoint.c | 3 --- fs/f2fs/node.c | 9 ++++++++- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c index 53bc328..aedb441 100644 --- a/fs/f2fs/checkpoint.c +++ b/fs/f2fs/checkpoint.c @@ -129,9 +129,6 @@ int ra_meta_pages(struct f2fs_sb_info *sbi, block_t start, int nrpages, int type switch (type) { case META_NAT: - if (unlikely(blkno >= - NAT_BLOCK_OFFSET(NM_I(sbi)->max_nid))) - blkno = 0; /* get nat block addr */ fio.blk_addr = current_nat_addr(sbi, blkno * NAT_ENTRY_PER_BLOCK); diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c index 1a7e925..8751375 100644 --- a/fs/f2fs/node.c +++ b/fs/f2fs/node.c @@ -1509,13 +1509,20 @@ static void build_free_nids(struct f2fs_sb_info *sbi) struct f2fs_summary_block *sum = curseg->sum_blk; int i = 0; nid_t nid = nm_i->next_scan_nid; + block_t blkno = NAT_BLOCK_OFFSET(nid); /* Enough entries */ if (nm_i->fcnt > NAT_ENTRY_PER_BLOCK) return; + if (unlikely(blkno >= + NAT_BLOCK_OFFSET(NM_I(sbi)->max_nid))) { + blkno = 0; + nid = 0; + } + /* readahead nat pages to be scanned */ - ra_meta_pages(sbi, NAT_BLOCK_OFFSET(nid), FREE_NID_PAGES, META_NAT); + ra_meta_pages(sbi, blkno, FREE_NID_PAGES, META_NAT); while (1) { struct page *page = get_current_nat_page(sbi, nid); -- 1.9.1 ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/