lkml.org 
[lkml]   [2020]   [Sep]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 72/94] f2fs: fix indefinite loop scanning for free nid
    Date
    From: Sahitya Tummala <stummala@codeaurora.org>

    [ Upstream commit e2cab031ba7b5003cd12185b3ef38f1a75e3dae8 ]

    If the sbi->ckpt->next_free_nid is not NAT block aligned and if there
    are free nids in that NAT block between the start of the block and
    next_free_nid, then those free nids will not be scanned in scan_nat_page().
    This results into mismatch between nm_i->available_nids and the sum of
    nm_i->free_nid_count of all NAT blocks scanned. And nm_i->available_nids
    will always be greater than the sum of free nids in all the blocks.
    Under this condition, if we use all the currently scanned free nids,
    then it will loop forever in f2fs_alloc_nid() as nm_i->available_nids
    is still not zero but nm_i->free_nid_count of that partially scanned
    NAT block is zero.

    Fix this to align the nm_i->next_scan_nid to the first nid of the
    corresponding NAT block.

    Signed-off-by: Sahitya Tummala <stummala@codeaurora.org>
    Reviewed-by: Chao Yu <yuchao0@huawei.com>
    Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    fs/f2fs/node.c | 3 +++
    1 file changed, 3 insertions(+)

    diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c
    index e7b8e2b35e226..f8006f62c5467 100644
    --- a/fs/f2fs/node.c
    +++ b/fs/f2fs/node.c
    @@ -2011,6 +2011,9 @@ static void __build_free_nids(struct f2fs_sb_info *sbi, bool sync, bool mount)
    if (unlikely(nid >= nm_i->max_nid))
    nid = 0;

    + if (unlikely(nid % NAT_ENTRY_PER_BLOCK))
    + nid = NAT_BLOCK_OFFSET(nid) * NAT_ENTRY_PER_BLOCK;
    +
    /* Enough entries */
    if (nm_i->nid_cnt[FREE_NID_LIST] >= NAT_ENTRY_PER_BLOCK)
    return;
    --
    2.25.1


    \
     
     \ /
      Last update: 2020-09-21 18:40    [W:5.394 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site