Minki/linux
1
0
Fork 1
mirror of https://github.com/torvalds/linux.git synced 2024-11-22 20:22:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

f2fs: fix to tag gcing flag on page during block migration

Browse Source 
It needs to add missing gcing flag on page during block migration,
in order to garantee migrated data be persisted during checkpoint,
otherwise out-of-order persistency between data and node may cause
data corruption after SPOR.

Similar issue was fixed by commit 2d1fe8a86b ("f2fs: fix to tag
gcing flag on page during file defragment").

Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
This commit is contained in:
Chao Yu 2023-12-10 19:35:42 +08:00 committed by Jaegeuk Kim
parent 87f3afd366
commit 4961acdd65
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
fs/f2fs/compress.c
View File

@ -1036,8 +1036,10 @@ static void set_cluster_dirty(struct compress_ctx *cc)
int i; int i;
for (i = 0; i < cc->cluster_size; i++) for (i = 0; i < cc->cluster_size; i++)
if (cc->rpages[i]) if (cc->rpages[i]) {
set_page_dirty(cc->rpages[i]); set_page_dirty(cc->rpages[i]);
set_page_private_gcing(cc->rpages[i]);
}
} }
static int prepare_compress_overwrite(struct compress_ctx *cc, static int prepare_compress_overwrite(struct compress_ctx *cc,

2
fs/f2fs/file.c
View File

@ -1312,6 +1312,7 @@ static int __clone_blkaddrs(struct inode *src_inode, struct inode *dst_inode,
} }
memcpy_page(pdst, 0, psrc, 0, PAGE_SIZE); memcpy_page(pdst, 0, psrc, 0, PAGE_SIZE);
set_page_dirty(pdst); set_page_dirty(pdst);
set_page_private_gcing(pdst);
f2fs_put_page(pdst, 1); f2fs_put_page(pdst, 1);
f2fs_put_page(psrc, 1); f2fs_put_page(psrc, 1);
@ -4046,6 +4047,7 @@ static int redirty_blocks(struct inode *inode, pgoff_t page_idx, int len)
f2fs_bug_on(F2FS_I_SB(inode), !page); f2fs_bug_on(F2FS_I_SB(inode), !page);
set_page_dirty(page); set_page_dirty(page);
set_page_private_gcing(page);
f2fs_put_page(page, 1); f2fs_put_page(page, 1);
f2fs_put_page(page, 0); f2fs_put_page(page, 0);
} }