From: Eric Sandeen <sandeen@redhat.com>
To: v9fs@lists.linux.dev
Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
ericvh@kernel.org, lucho@ionkov.net, asmadeus@codewreck.org,
linux_oss@crudebyte.com, eadavis@qq.com
Subject: [PATCH V3 0/4] 9p: Convert to the new mount API
Date: Fri, 10 Oct 2025 16:36:15 -0500 [thread overview]
Message-ID: <20251010214222.1347785-1-sandeen@redhat.com> (raw)
This is another updated attempt to convert 9p to the new mount API. 9p
is one of the last conversions needed, possibly because it is one of the
trickier ones!
This has had only light testing. I confirmed that with the diod server,
both of these mount successfully:
# mount -t 9p -o aname=/tmp/9,version=9p2000.L,uname=fsgqa,access=user,trans=tcp 127.0.0.1 /mnt
and
# mount.diod localhost:/tmp/9 /mnt
(The latter passes trans=fd under the covers)
I have not been able to test other transports, or exhaustively test
functionality of all mount options.
Changes from V1 to V2:
Address "make W=1" warnings from kernel test robot, comments from
dhowells, and some kernel-doc comments for changed arguments.
Changes from V2 to V3:
Patch 1:
None
Patch 2:
None
Patch 3:
Change to not re-use v9fs_session_info and p9_client in the
v9fs_context structure. Instead, new structures p9_client_opts and
p9_session_opts are introduced. This avoids confusion about what is
used only for option parsing, and what is used for the actual
mounted instance.
Patch 4:
Allows unknown mount options as prior code did. I noticed that
mount.diod passes a "rwdepth" option by default, which was ignored
previously but breaks mount with strict unknown option rejection.
Adjust variable names in v9fs_parse_param, v9fs_apply_options,
and v9fs_init_fs_context to reflect new context structure and
to clarify difference.
Limit the msize option to INT_MAX as the old code did.
Remove redundant v9fs_set_super function, set_anon_super is enough.
Properly extract v9fs_seesion_info from sb->s_fs_info, not
the fc context, in v9fs_fill_super. (thanks eadavis@qq.com)
Remove unneeded fc arg from v9fs_fill_super.
next reply other threads:[~2025-10-10 21:42 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-10 21:36 Eric Sandeen [this message]
2025-10-10 21:36 ` [PATCH V3 1/4] fs/fs_parse: add back fsparam_u32hex Eric Sandeen
2025-10-10 21:36 ` [PATCH V3 2/4] net/9p: move structures and macros to header files Eric Sandeen
2025-10-10 21:36 ` [PATCH V3 3/4] 9p: create a v9fs_context structure to hold parsed options Eric Sandeen
2025-10-10 21:36 ` [PATCH V3 4/4] 9p: convert to the new mount API Eric Sandeen
2025-10-13 10:26 ` Dominique Martinet
2025-10-13 18:46 ` Eric Sandeen
2025-10-13 19:04 ` Dominique Martinet
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20251010214222.1347785-1-sandeen@redhat.com \
--to=sandeen@redhat.com \
--cc=asmadeus@codewreck.org \
--cc=eadavis@qq.com \
--cc=ericvh@kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux_oss@crudebyte.com \
--cc=lucho@ionkov.net \
--cc=v9fs@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).