Thanks you, Hongbo. I just applied this series to the dev-test branch as below, and will keep testing with incoming patches together. Let's see. :)
https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git/log/?h=dev-test On 06/02, Hongbo Li wrote: > In this version, we have finished the issues pointed in v3. > First, I'd like to express my sincere thanks to Jaegeuk and Chao > for reviewing this patch series and providing corrections. I also > appreciate Eric for rebasing the patches onto the latest branch to > ensure forward compatibility. > > The latest patch series has addressed all the issues mentioned in > the previous set. For modified patches, I've re-added Signed-off-by > tags (SOB) and uniformly removed all Reviewed-by tags. > > v4: > - Change is_remount as bool type in patch 2. > - Remove the warning reported by Dan for patch 5. > - Enhance sanity check and fix some coding style suggested by > Jaegeuk in patch 5. > - Change the log info when compression option conflicts in patch 5. > - Fix the issues reported by code-reviewing in patch 5. > - Context modified in patch 7. > > V3: https://lore.kernel.org/all/20250423170926.76007-1-sand...@redhat.com/ > - Rebase onto git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git > dev branch > - Fix up some 0day robot warnings > > (Here is the origianl cover letter:) > > Since many filesystems have done the new mount API conversion, > we introduce the new mount API conversion in f2fs. > > The series can be applied on top of the current mainline tree > and the work is based on the patches from Lukas Czerner (has > done this in ext4[1]). His patch give me a lot of ideas. > > Here is a high level description of the patchset: > > 1. Prepare the f2fs mount parameters required by the new mount > API and use it for parsing, while still using the old API to > get mount options string. Split the parameter parsing and > validation of the parse_options helper into two separate > helpers. > > f2fs: Add fs parameter specifications for mount options > f2fs: move the option parser into handle_mount_opt > > 2. Remove the use of sb/sbi structure of f2fs from all the > parsing code, because with the new mount API the parsing is > going to be done before we even get the super block. In this > part, we introduce f2fs_fs_context to hold the temporary > options when parsing. For the simple options check, it has > to be done during parsing by using f2fs_fs_context structure. > For the check which needs sb/sbi, we do this during super > block filling. > > f2fs: Allow sbi to be NULL in f2fs_printk > f2fs: Add f2fs_fs_context to record the mount options > f2fs: separate the options parsing and options checking > > 3. Switch the f2fs to use the new mount API for mount and > remount. > > f2fs: introduce fs_context_operation structure > f2fs: switch to the new mount api > > [1] https://lore.kernel.org/all/20211021114508.21407-1-lczer...@redhat.com/ > > Hongbo Li (7): > f2fs: Add fs parameter specifications for mount options > f2fs: move the option parser into handle_mount_opt > f2fs: Allow sbi to be NULL in f2fs_printk > f2fs: Add f2fs_fs_context to record the mount options > f2fs: separate the options parsing and options checking > f2fs: introduce fs_context_operation structure > f2fs: switch to the new mount api > > fs/f2fs/super.c | 2108 +++++++++++++++++++++++++++-------------------- > 1 file changed, 1197 insertions(+), 911 deletions(-) > > -- > 2.33.0 _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
