f2fs: fix to check return value of f2fs_recover_xattr_data
Should check return value of f2fs_recover_xattr_data in
__f2fs_setxattr rather than doing invalid retry if error happen.
Also just do set_page_dirty in f2fs_recover_xattr_data when
page is changed really.
Fixes: 50a472bbc7 ("f2fs: do not return EFSCORRUPTED, but try to run online repair")
Signed-off-by: Zhiguo Niu <zhiguo.niu@unisoc.com>
Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
This commit is contained in:
+3
-3
@@ -2751,11 +2751,11 @@ recover_xnid:
|
||||
f2fs_update_inode_page(inode);
|
||||
|
||||
/* 3: update and set xattr node page dirty */
|
||||
if (page)
|
||||
if (page) {
|
||||
memcpy(F2FS_NODE(xpage), F2FS_NODE(page),
|
||||
VALID_XATTR_BLOCK_SIZE);
|
||||
|
||||
set_page_dirty(xpage);
|
||||
set_page_dirty(xpage);
|
||||
}
|
||||
f2fs_put_page(xpage, 1);
|
||||
|
||||
return 0;
|
||||
|
||||
Reference in New Issue
Block a user