Re: [f2fs-dev] [PATCH v3 00/13] f2fs-tools & inject.f2fs: bugfix and new injections

[Chao Yu via Linux-f2fs-devel <linux-f2fs-devel@lists.sourceforge.net>]

On 8/20/2025 8:42 PM, Sheng Yong wrote:
> Hi, all,
> 
> Since we have not yet determined how to check the test result properly,
> I split patchset v2 into two parts on Chao's suggestion. This is the
> first part which contains some fixes/cleanups for f2fs-tools, and adds
> new injections for inject.f2fs.

There is one missed entry in inject.f2fs manual page in 10/13, once you
fixed it, feel free to add:

Reviewed-by: Chao Yu <chao@kernel.org>

Thanks,

> 
> Changes from last version are also updated in individual patch.
> 
> v3: * split original patchset into two parts
>      * update manual
>      * refactor inject sit/nat in journal
>      * fix memleak
> v2: * add some fix and cleanup
>      * remove img.tar.gz from testcases
>      * add testcases for injection
>      * cleanup helpers script and simplify filter.sed and expected.in
>      https://lore.kernel.org/linux-f2fs-devel/20250610123743.667183-1-shengyong1@xiaomi.com/
> v1: https://lore.kernel.org/linux-f2fs-devel/20241029120956.4186731-1-shengyong@oppo.com/
> 
> Sheng Yong (13):
>    fsck.f2fs: do not finish/reset zone if dry-run is true
>    f2fs-tools: add option N to answer no for all questions
>    f2fs-tools: cleanup {nid|segno}_in_journal
>    fsck.f2fs: fix invalidate checkpoint
>    dump.f2fs: print more info
>    f2fs-tools: add and export lookup_sit_in_journal
>    inject.f2fs: fix injecting sit/nat in journal
>    inject.f2fs: fix injection on zoned device
>    inject.f2fs: fix and cleanup parsing numeric options
>    inject.f2fs: add members in inject_cp
>    inject.f2fs: add member `feature' in inject_sb
>    inject.f2fs: add members in inject_node
>    inject.f2fs: add member `filename' in inject_dentry
> 
>   fsck/dump.c         |  15 +-
>   fsck/f2fs.h         |  12 +-
>   fsck/fsck.c         |   2 +-
>   fsck/fsck.h         |   4 +-
>   fsck/inject.c       | 515 ++++++++++++++++++++++++++++++++++++--------
>   fsck/inject.h       |   1 +
>   fsck/main.c         |  14 +-
>   fsck/mount.c        |  61 ++++--
>   include/f2fs_fs.h   |   1 +
>   lib/libf2fs_zoned.c |   6 +-
>   man/dump.f2fs.8     |   3 +
>   man/fsck.f2fs.8     |   3 +
>   man/inject.f2fs.8   |  43 +++-
>   13 files changed, 552 insertions(+), 128 deletions(-)
> 



_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel