linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v4 0/7] f2fs: new mount API conversion
@ 2025-06-02  9:02 Hongbo Li
  2025-06-02  9:02 ` [PATCH v4 1/7] f2fs: Add fs parameter specifications for mount options Hongbo Li
                   ` (7 more replies)
  0 siblings, 8 replies; 17+ messages in thread
From: Hongbo Li @ 2025-06-02  9:02 UTC (permalink / raw)
  To: jaegeuk, chao; +Cc: linux-f2fs-devel, sandeen, lihongbo22, linux-fsdevel

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-sandeen@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-lczerner@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


^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2025-07-10  9:25 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-02  9:02 [PATCH v4 0/7] f2fs: new mount API conversion Hongbo Li
2025-06-02  9:02 ` [PATCH v4 1/7] f2fs: Add fs parameter specifications for mount options Hongbo Li
2025-06-02  9:02 ` [PATCH v4 2/7] f2fs: move the option parser into handle_mount_opt Hongbo Li
2025-07-04  3:27   ` Chao Yu
2025-07-07 13:20     ` Hongbo Li
2025-07-09 17:48       ` Jaegeuk Kim
2025-07-10  9:25         ` Hongbo Li
2025-06-02  9:02 ` [PATCH v4 3/7] f2fs: Allow sbi to be NULL in f2fs_printk Hongbo Li
2025-06-02  9:02 ` [PATCH v4 4/7] f2fs: Add f2fs_fs_context to record the mount options Hongbo Li
2025-06-02  9:02 ` [PATCH v4 5/7] f2fs: separate the options parsing and options checking Hongbo Li
2025-07-07 12:17   ` Chao Yu
2025-07-08  2:56     ` Hongbo Li
2025-07-10  6:54       ` Chao Yu
2025-06-02  9:02 ` [PATCH v4 6/7] f2fs: introduce fs_context_operation structure Hongbo Li
2025-06-02  9:02 ` [PATCH v4 7/7] f2fs: switch to the new mount api Hongbo Li
2025-06-02 16:05 ` [PATCH v4 0/7] f2fs: new mount API conversion Jaegeuk Kim
2025-06-03  1:09   ` Hongbo Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).