From: Chuck Lever <cel@kernel.org>
To: NeilBrown <neil@brown.name>, Jeff Layton <jlayton@kernel.org>,
Olga Kornievskaia <okorniev@redhat.com>,
Dai Ngo <dai.ngo@oracle.com>, Tom Talpey <tom@talpey.com>
Cc: <linux-nfs@vger.kernel.org>
Subject: [PATCH v2 00/33] Convert server-side NFSv3 XDR to use xdrgen
Date: Thu, 24 Sep 2026 13:09:39 -0400 [thread overview]
Message-ID: <20260924171012.3978-1-cel@kernel.org> (raw)
Replace NFSD's NFSv3 XDR encoders and decoders with code that is
mechanically generated directly from the NFSv3 RPC language
specification. Unlike NFSv2, where there are at least two
incomplete language specifications and much lore, nfs3.x here is
based closely on RFC 1813.
As we convert newer NFS versions to use xdrgen, I expect a reduction
in human coding errors, improved spec compliance, and better memory
safety for those more complex NFS programs.
Changes in v2:
- Include the generated NFSv3 header directly in nfsfh.c and nfsctl.c.
- Remove the unused struct nfsd3_createargs in the MKDIR patch.
- Regenerate the READDIR and READDIRPLUS result decoders.
- Drop the decode-hook stubs the regenerated decoders no longer call.
- Link to v1: https://lore.kernel.org/linux-nfs/20260923132811.244160-1-cel@kernel.org/
Chuck Lever (33):
NFSD: Report a first-batch readdir error in the reply status
Documentation: Add the RPC language description of NFSv3
NFSD: Add infrastructure for generating NFSv3 XDR encoders and
decoders
NFSD: Replace nfs3.h with nfs3xdr_gen.h
NFSD: Replace the nfsd3_createres macro
NFSD: Relocate the NFSv3 XDR storage union into nfs3proc.c
NFSD: Use xdrgen XDR functions for the NFSv3 NULL procedure
NFSD: Use xdrgen XDR functions for NFSv3 GETATTR procedure
NFSD: Use xdrgen XDR functions for NFSv3 SETATTR procedure
NFSD: Use xdrgen XDR functions for the NFSv3 LOOKUP procedure
NFSD: Use xdrgen XDR functions for NFSv3 ACCESS procedure
NFSD: Use xdrgen XDR functions for NFSv3 READLINK procedure
NFSD: Use xdrgen XDR functions for NFSv3 READ procedure
NFSD: Use xdrgen XDR functions for NFSv3 WRITE procedure
NFSD: Use xdrgen XDR functions for NFSv3 CREATE procedure
NFSD: Use xdrgen XDR functions for NFSv3 MKDIR procedure
NFSD: Use xdrgen XDR functions for NFSv3 SYMLINK procedure
NFSD: Use xdrgen XDR functions for NFSv3 MKNOD procedure
NFSD: Use xdrgen XDR functions for the NFSv3 REMOVE procedure
NFSD: Use xdrgen XDR functions for the NFSv3 RMDIR procedure
NFSD: Use xdrgen XDR functions for the NFSv3 RENAME procedure
NFSD: Use xdrgen XDR functions for the NFSv3 LINK procedure
NFSD: Use xdrgen XDR functions for the NFSv3 FSSTAT procedure
NFSD: Use xdrgen XDR functions for the NFSv3 FSINFO procedure
NFSD: Use xdrgen XDR functions for the NFSv3 PATHCONF procedure
NFSD: Use xdrgen XDR functions for the NFSv3 COMMIT procedure
NFSD: Use xdrgen XDR functions for NFSv3 READDIR arguments
NFSD: Use xdrgen XDR functions for NFSv3 READDIRPLUS arguments
NFSD: Refactor NFSv3 directory cookie encoding
NFSD: Refactor NFSv3 directory entry encoding
NFSD: Split struct nfsd3_readdirres
NFSD: Use xdrgen XDR functions for NFSv3 READDIR results
NFSD: Use xdrgen XDR functions for NFSv3 READDIRPLUS results
Documentation/sunrpc/xdr/nfs3.x | 839 ++++++
fs/nfsd/Makefile | 15 +-
fs/nfsd/nfs2acl.c | 41 +-
fs/nfsd/nfs3acl.c | 12 +-
fs/nfsd/nfs3proc.c | 2545 +++++++++++++------
fs/nfsd/nfs3xdr.c | 1369 +++-------
fs/nfsd/nfs3xdr_gen.c | 3808 ++++++++++++++++++++++++++++
fs/nfsd/nfs3xdr_gen.h | 82 +
fs/nfsd/nfsctl.c | 2 +-
fs/nfsd/nfserr.h | 2 +-
fs/nfsd/nfsfh.c | 2 +-
fs/nfsd/vfs.c | 76 +-
fs/nfsd/xdr3.h | 300 +--
include/linux/nfs3.h | 3 -
include/linux/sunrpc/xdrgen/nfs3.h | 1092 ++++++++
15 files changed, 8021 insertions(+), 2167 deletions(-)
create mode 100644 Documentation/sunrpc/xdr/nfs3.x
create mode 100644 fs/nfsd/nfs3xdr_gen.c
create mode 100644 fs/nfsd/nfs3xdr_gen.h
create mode 100644 include/linux/sunrpc/xdrgen/nfs3.h
base-commit: 21c66e2ece97c0ac8139b44051d565b29574f821
--
2.55.0
next reply other threads:[~2026-09-24 17:10 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-24 17:09 Chuck Lever [this message]
2026-09-24 17:09 ` [PATCH v2 01/33] NFSD: Report a first-batch readdir error in the reply status Chuck Lever
2026-09-24 17:09 ` [PATCH v2 02/33] Documentation: Add the RPC language description of NFSv3 Chuck Lever
2026-09-24 17:09 ` [PATCH v2 03/33] NFSD: Add infrastructure for generating NFSv3 XDR encoders and decoders Chuck Lever
2026-09-24 17:09 ` [PATCH v2 04/33] NFSD: Replace nfs3.h with nfs3xdr_gen.h Chuck Lever
2026-09-24 17:09 ` [PATCH v2 05/33] NFSD: Replace the nfsd3_createres macro Chuck Lever
2026-09-24 17:09 ` [PATCH v2 06/33] NFSD: Relocate the NFSv3 XDR storage union into nfs3proc.c Chuck Lever
2026-09-24 17:09 ` [PATCH v2 07/33] NFSD: Use xdrgen XDR functions for the NFSv3 NULL procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 08/33] NFSD: Use xdrgen XDR functions for NFSv3 GETATTR procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 09/33] NFSD: Use xdrgen XDR functions for NFSv3 SETATTR procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 10/33] NFSD: Use xdrgen XDR functions for the NFSv3 LOOKUP procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 11/33] NFSD: Use xdrgen XDR functions for NFSv3 ACCESS procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 12/33] NFSD: Use xdrgen XDR functions for NFSv3 READLINK procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 13/33] NFSD: Use xdrgen XDR functions for NFSv3 READ procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 14/33] NFSD: Use xdrgen XDR functions for NFSv3 WRITE procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 15/33] NFSD: Use xdrgen XDR functions for NFSv3 CREATE procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 16/33] NFSD: Use xdrgen XDR functions for NFSv3 MKDIR procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 17/33] NFSD: Use xdrgen XDR functions for NFSv3 SYMLINK procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 18/33] NFSD: Use xdrgen XDR functions for NFSv3 MKNOD procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 19/33] NFSD: Use xdrgen XDR functions for the NFSv3 REMOVE procedure Chuck Lever
2026-09-24 17:09 ` [PATCH v2 20/33] NFSD: Use xdrgen XDR functions for the NFSv3 RMDIR procedure Chuck Lever
2026-09-24 17:10 ` [PATCH v2 21/33] NFSD: Use xdrgen XDR functions for the NFSv3 RENAME procedure Chuck Lever
2026-09-24 17:10 ` [PATCH v2 22/33] NFSD: Use xdrgen XDR functions for the NFSv3 LINK procedure Chuck Lever
2026-09-24 17:10 ` [PATCH v2 23/33] NFSD: Use xdrgen XDR functions for the NFSv3 FSSTAT procedure Chuck Lever
2026-09-24 17:10 ` [PATCH v2 24/33] NFSD: Use xdrgen XDR functions for the NFSv3 FSINFO procedure Chuck Lever
2026-09-24 17:10 ` [PATCH v2 25/33] NFSD: Use xdrgen XDR functions for the NFSv3 PATHCONF procedure Chuck Lever
2026-09-24 17:10 ` [PATCH v2 26/33] NFSD: Use xdrgen XDR functions for the NFSv3 COMMIT procedure Chuck Lever
2026-09-24 17:10 ` [PATCH v2 27/33] NFSD: Use xdrgen XDR functions for NFSv3 READDIR arguments Chuck Lever
2026-09-24 17:10 ` [PATCH v2 28/33] NFSD: Use xdrgen XDR functions for NFSv3 READDIRPLUS arguments Chuck Lever
2026-09-24 17:10 ` [PATCH v2 29/33] NFSD: Refactor NFSv3 directory cookie encoding Chuck Lever
2026-09-24 17:10 ` [PATCH v2 30/33] NFSD: Refactor NFSv3 directory entry encoding Chuck Lever
2026-09-24 17:10 ` [PATCH v2 31/33] NFSD: Split struct nfsd3_readdirres Chuck Lever
2026-09-24 17:10 ` [PATCH v2 32/33] NFSD: Use xdrgen XDR functions for NFSv3 READDIR results Chuck Lever
2026-09-24 17:10 ` [PATCH v2 33/33] NFSD: Use xdrgen XDR functions for NFSv3 READDIRPLUS results Chuck Lever
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=20260924171012.3978-1-cel@kernel.org \
--to=cel@kernel.org \
--cc=dai.ngo@oracle.com \
--cc=jlayton@kernel.org \
--cc=linux-nfs@vger.kernel.org \
--cc=neil@brown.name \
--cc=okorniev@redhat.com \
--cc=tom@talpey.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