Commit 9198aceb authored by Jaegeuk Kim's avatar Jaegeuk Kim

f2fs: check nid == 0 in add_free_nid

It is more obvious that add_free_nid checks whether the free nid is zero or not.
Reviewed-by: default avatarNamjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk.kim@samsung.com>
parent 8680441c
...@@ -1245,6 +1245,10 @@ static int add_free_nid(struct f2fs_nm_info *nm_i, nid_t nid) ...@@ -1245,6 +1245,10 @@ static int add_free_nid(struct f2fs_nm_info *nm_i, nid_t nid)
if (nm_i->fcnt > 2 * MAX_FREE_NIDS) if (nm_i->fcnt > 2 * MAX_FREE_NIDS)
return 0; return 0;
/* 0 nid should not be used */
if (nid == 0)
return 0;
retry: retry:
i = kmem_cache_alloc(free_nid_slab, GFP_NOFS); i = kmem_cache_alloc(free_nid_slab, GFP_NOFS);
if (!i) { if (!i) {
...@@ -1286,10 +1290,6 @@ static int scan_nat_page(struct f2fs_nm_info *nm_i, ...@@ -1286,10 +1290,6 @@ static int scan_nat_page(struct f2fs_nm_info *nm_i,
int fcnt = 0; int fcnt = 0;
int i; int i;
/* 0 nid should not be used */
if (start_nid == 0)
++start_nid;
i = start_nid % NAT_ENTRY_PER_BLOCK; i = start_nid % NAT_ENTRY_PER_BLOCK;
for (; i < NAT_ENTRY_PER_BLOCK; i++, start_nid++) { for (; i < NAT_ENTRY_PER_BLOCK; i++, start_nid++) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment