From: Mike Snitzer <snitzer@kernel.org>
To: linux-nfs@vger.kernel.org
Cc: Jeff Layton <jlayton@kernel.org>,
Chuck Lever <chuck.lever@oracle.com>,
Trond Myklebust <trondmy@hammerspace.com>,
snitzer@hammerspace.com
Subject: [for-6.11 PATCH 00/29] nfs/nfsd: add support for localio bypass
Date: Fri, 7 Jun 2024 10:26:17 -0400 [thread overview]
Message-ID: <20240607142646.20924-1-snitzer@kernel.org> (raw)
Hi,
This patch series rebases "localio" changes that Hammerspace (and
Primary Data before it) has been carrying since 2014. The reason they
weren't proposed for upstream inclusion until now was the handshake
for whether or not a client and server are local was brittle. Please
see the commit header of "nfs/localio: discontinue network address
based localio setup" (patch 20) for more context.
Aside from rebasing the original changes (patches 1 - 18) from a
5.15.-130-stable kernel, my contribution to this series was to make
the localio handshake more robust. To do so a new LOCALIO protocol
extension has been added to both NFS v3 and v4. It follows the
well-worn pattern established by the ACL protocol extension.
These changes have proven stable against various test scenarios:
1) client and server both on localhost (for both v3 and v4.2)
2) various permutations of client and server support enablement for
both local and remote client and server.
3) client on host, server within a container (for both v3 and v4.2)
I've preserved all established author and Signed-off-by attribution
despite Andy, Peng and Jeff no longer working for Primary Data (or
Hammerspace). I've confirmed with Trond that its best to keep it all
despite those email addresses no longer being active. My Signed-off-by
and that of reviewers and maintainer(s) to follow will build on the
established development provenance.
I also made sure to preserve the original work done by others (rather
than fold changes that I add to this work, to avoid tainting the long
established development and sequence of changes).
My container testing was done in terms of podman managed containers.
I'd appreciate additional review relative to network namespaces.
fs/nfsd/localio.c:nfsd_local_fakerqst_create() in particular is simply
using the client's network namespace with rpc_net_ns(rpc_clnt). I have
an extra patch that updates nfsd_open_local_fh()'s first argument to
be the server's 'struct net' -- but I stopped short of formally
including that change in this series because it hasn't proven needed
(but more exotic hypothetical scenarios could easily expose the need
for it). I can append it to the series as an "RFC PATCH 30/29" as
needed.
All review and comments are welcome!
Thanks,
Mike
Mike Snitzer (11):
nfs/write: fix nfs_initiate_commit to return error from nfs_local_commit
nfs/localio: discontinue network address based localio setup
nfs_common: add NFS v3 LOCALIO protocol extension enablement
nfs: implement v3 client support for NFS_LOCALIO_PROGRAM
nfsd: implement v3 server support for NFS_LOCALIO_PROGRAM
nfs_common: add NFS v4 LOCALIO protocol extension enablement
nfs: implement v4 client support for NFS_LOCALIO_PROGRAM
nfsd: implement v4 server support for NFS_LOCALIO_PROGRAM
nfs/nfsd: switch GETUUID to using {encode,decode}_opaque_fixed
nfs/nfsd: consolidate {encode,decode}_opaque_fixed in nfs_xdr.h
nfs/localio: move managing nfsd_open_local_fh symbol to nfs_common
Peng Tao (3):
sunrpc: add and export rpc_ntop6_addr_noscopeid
nfs: move nfs_stat_to_errno to nfs.h
nfs/flexfiles: check local DS when making DS connections
Trond Myklebust (8):
NFS: Manage boot verifier correctly in the case of localio
NFS: Enable localio for non-pNFS I/O
pnfs/flexfiles: Enable localio for flexfiles I/O
NFS: Add tracepoints for nfs_local_enable and nfs_local_disable
NFS: Don't call filesystem write() routines directly
NFS: Don't call filesystem read() routines directly
NFS: Use completion rather than flush_work() in nfs_local_commit()
NFS: localio writes need to use a normal workqueue
Weston Andros Adamson (7):
nfs: pass nfs_client to nfs_initiate_pgio
nfs: pass nfs_client to nfs_initiate_commit
nfs: pass descriptor thru nfs_initiate_pgio path
sunrpc: handle NULL req->defer in cache_defer_req
sunrpc: export svc_defer
sunrpc: add rpcauth_map_to_svc_cred
nfs/nfsd: add "local io" support
fs/Kconfig | 3 +
fs/nfs/Kconfig | 25 +
fs/nfs/Makefile | 2 +
fs/nfs/blocklayout/blocklayout.c | 6 +-
fs/nfs/client.c | 15 +-
fs/nfs/filelayout/filelayout.c | 19 +-
fs/nfs/flexfilelayout/flexfilelayout.c | 129 +++-
fs/nfs/flexfilelayout/flexfilelayout.h | 2 +
fs/nfs/flexfilelayout/flexfilelayoutdev.c | 6 +
fs/nfs/inode.c | 28 +-
fs/nfs/internal.h | 101 ++-
fs/nfs/localio.c | 814 ++++++++++++++++++++++
fs/nfs/nfs2xdr.c | 69 --
fs/nfs/nfs3_fs.h | 1 +
fs/nfs/nfs3client.c | 25 +
fs/nfs/nfs3proc.c | 3 +
fs/nfs/nfs3xdr.c | 58 ++
fs/nfs/nfs4_fs.h | 2 +
fs/nfs/nfs4client.c | 23 +
fs/nfs/nfs4proc.c | 3 +
fs/nfs/nfs4xdr.c | 65 +-
fs/nfs/nfstrace.h | 61 ++
fs/nfs/pagelist.c | 35 +-
fs/nfs/pnfs.c | 24 +-
fs/nfs/pnfs.h | 6 +-
fs/nfs/pnfs_nfs.c | 5 +-
fs/nfs/write.c | 28 +-
fs/nfs_common/Makefile | 3 +
fs/nfs_common/nfslocalio.c | 68 ++
fs/nfsd/Kconfig | 25 +
fs/nfsd/Makefile | 2 +
fs/nfsd/filecache.c | 2 +-
fs/nfsd/localio.c | 324 +++++++++
fs/nfsd/netns.h | 4 +
fs/nfsd/nfsd.h | 11 +
fs/nfsd/nfssvc.c | 91 ++-
fs/nfsd/trace.h | 3 +-
fs/nfsd/vfs.h | 8 +
fs/nfsd/xdr.h | 6 +
include/linux/nfs.h | 65 ++
include/linux/nfs_fs.h | 2 +
include/linux/nfs_fs_sb.h | 8 +
include/linux/nfs_xdr.h | 31 +-
include/linux/nfslocalio.h | 37 +
include/linux/sunrpc/auth.h | 4 +
include/linux/sunrpc/svc_xprt.h | 1 +
include/uapi/linux/nfs.h | 4 +
net/sunrpc/auth.c | 16 +
net/sunrpc/cache.c | 2 +
net/sunrpc/svc_xprt.c | 4 +-
50 files changed, 2120 insertions(+), 159 deletions(-)
create mode 100644 fs/nfs/localio.c
create mode 100644 fs/nfs_common/nfslocalio.c
create mode 100644 fs/nfsd/localio.c
create mode 100644 include/linux/nfslocalio.h
--
2.44.0
next reply other threads:[~2024-06-07 14:26 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-07 14:26 Mike Snitzer [this message]
2024-06-07 14:26 ` [for-6.11 PATCH 01/29] nfs: pass nfs_client to nfs_initiate_pgio Mike Snitzer
2024-06-10 12:02 ` Jeff Layton
2024-06-07 14:26 ` [for-6.11 PATCH 02/29] nfs: pass nfs_client to nfs_initiate_commit Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 03/29] nfs: pass descriptor thru nfs_initiate_pgio path Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 04/29] sunrpc: handle NULL req->defer in cache_defer_req Mike Snitzer
2024-06-10 12:21 ` Jeff Layton
2024-06-11 1:03 ` NeilBrown
2024-06-11 2:57 ` Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 05/29] sunrpc: export svc_defer Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 06/29] sunrpc: add rpcauth_map_to_svc_cred Mike Snitzer
2024-06-10 12:19 ` Jeff Layton
2024-06-07 14:26 ` [for-6.11 PATCH 07/29] sunrpc: add and export rpc_ntop6_addr_noscopeid Mike Snitzer
2024-06-09 12:36 ` Jeff Layton
2024-06-10 16:33 ` Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 08/29] nfs: move nfs_stat_to_errno to nfs.h Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 09/29] NFS: Manage boot verifier correctly in the case of localio Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 10/29] nfs/nfsd: add "local io" support Mike Snitzer
2024-06-10 12:43 ` Jeff Layton
2024-06-10 16:42 ` Mike Snitzer
2024-06-12 2:25 ` Mike Snitzer
2024-06-12 3:17 ` NeilBrown
2024-06-12 3:41 ` Mike Snitzer
2024-06-12 4:09 ` NeilBrown
2024-06-12 4:48 ` Mike Snitzer
2024-06-12 6:30 ` NeilBrown
2024-06-07 14:26 ` [for-6.11 PATCH 11/29] NFS: Enable localio for non-pNFS I/O Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 12/29] nfs/flexfiles: check local DS when making DS connections Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 13/29] pnfs/flexfiles: Enable localio for flexfiles I/O Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 14/29] NFS: Add tracepoints for nfs_local_enable and nfs_local_disable Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 15/29] NFS: Don't call filesystem write() routines directly Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 16/29] NFS: Don't call filesystem read() " Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 17/29] NFS: Use completion rather than flush_work() in nfs_local_commit() Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 18/29] NFS: localio writes need to use a normal workqueue Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 19/29] nfs/write: fix nfs_initiate_commit to return error from nfs_local_commit Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 20/29] nfs/localio: discontinue network address based localio setup Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 21/29] nfs_common: add NFS v3 LOCALIO protocol extension enablement Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 22/29] nfs: implement v3 client support for NFS_LOCALIO_PROGRAM Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 23/29] nfsd: implement v3 server " Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 24/29] nfs_common: add NFS v4 LOCALIO protocol extension enablement Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 25/29] nfs: implement v4 client support for NFS_LOCALIO_PROGRAM Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 26/29] nfsd: implement v4 server " Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 27/29] nfs/nfsd: switch GETUUID to using {encode,decode}_opaque_fixed Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 28/29] nfs/nfsd: consolidate {encode,decode}_opaque_fixed in nfs_xdr.h Mike Snitzer
2024-06-07 14:26 ` [for-6.11 PATCH 29/29] nfs/localio: move managing nfsd_open_local_fh symbol to nfs_common Mike Snitzer
2024-06-07 18:06 ` [for-6.11 PATCH 30/29] nfs/nfsd: ensure localio server always uses its network namespace Mike Snitzer
2024-06-09 15:44 ` Chuck Lever
2024-06-10 16:50 ` Mike Snitzer
2024-06-10 22:37 ` Mike Snitzer
2024-06-07 18:09 ` [for-6.11 PATCH 00/29] nfs/nfsd: add support for localio bypass Mike Snitzer
2024-06-10 12:47 ` Jeff Layton
2024-06-10 16:47 ` Mike Snitzer
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=20240607142646.20924-1-snitzer@kernel.org \
--to=snitzer@kernel.org \
--cc=chuck.lever@oracle.com \
--cc=jlayton@kernel.org \
--cc=linux-nfs@vger.kernel.org \
--cc=snitzer@hammerspace.com \
--cc=trondmy@hammerspace.com \
/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