| Message ID | 20220323083724.55902-1-sunke32@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v3] generic/066: attr1 is still there after log replay on f2fs | expand |
On 2022/3/23 16:37, Sun Ke via Linux-f2fs-devel wrote: > The test fail on f2fs: > xattr names and values after second fsync log replay: > # file: SCRATCH_MNT/foobar > +user.attr1="val1" > user.attr3="val3" > > attr1 is still there after log replay. f2fs doesn't support fs-op level > transaction functionality. so it have no way to persist all metadata > updates in one transaction. We can use "fastboot" mountoption for this > case, so during last fsync on qwerty, f2fs can trigger a checkpoint > which will persist all metadata updates before fsync(). > > Suggested-by: Chao Yu <chao@kernel.org> > Signed-off-by: Sun Ke <sunke32@huawei.com> Reviewed-by: Chao Yu <chao@kernel.org> Thanks,
diff --git a/tests/generic/066 b/tests/generic/066 index 105a7acd..d8a38655 100755 --- a/tests/generic/066 +++ b/tests/generic/066 @@ -56,6 +56,14 @@ sync $SETFATTR_PROG -x user.attr2 $SCRATCH_MNT/foobar $XFS_IO_PROG -c "fsync" $SCRATCH_MNT/foobar +# f2fs doesn't support fs-op level transaction functionality. so it have no way +# to persist all metadata updates in one transaction. We can use "fastboot" +# mountoption for this case, so during last fsync on qwerty, f2fs can trigger a +# checkpoint which will persist all metadata updates before fsync(). +if [ $FSTYP = "f2fs" ]; then + export MOUNT_OPTIONS="-o fastboot $MOUNT_OPTIONS" +fi + _flakey_drop_and_remount # After the fsync log is replayed, the file should have only 2 xattrs, the ones
The test fail on f2fs: xattr names and values after second fsync log replay: # file: SCRATCH_MNT/foobar +user.attr1="val1" user.attr3="val3" attr1 is still there after log replay. f2fs doesn't support fs-op level transaction functionality. so it have no way to persist all metadata updates in one transaction. We can use "fastboot" mountoption for this case, so during last fsync on qwerty, f2fs can trigger a checkpoint which will persist all metadata updates before fsync(). Suggested-by: Chao Yu <chao@kernel.org> Signed-off-by: Sun Ke <sunke32@huawei.com> --- v3: merge "fastboot" with original mount options tests/generic/066 | 8 ++++++++ 1 file changed, 8 insertions(+)