Re: [PATCH RFC nfs-utils v2 00/10] nfsdctl: allow it to handle rpcbind registration and deregistration

[Jeff Layton <jlayton@kernel.org>]

On Mon, 2026-09-14 at 12:00 -0400, Jeff Layton wrote:
> This version fixes some problems that Sashiko flagged in review.
> 
> This patchset adds support for NFSD_A_SERVER_SOCK_USERSPACE_RPCBIND,
> which tells the kernel to not do rpcbind registration and to instead
> allow userland to handle it. After setting listeners, nfsdctl will then
> call out to rpcbind using the normal libtirpc client interfaces.
> 
> This is a companion set to the kernel set I recently posted (and which
> also should have been marked RFC).
> 
> Signed-off-by: Jeff Layton <jlayton@kernel.org>
> ---
> Changes in v2:
> - Fixes for bugs v1 introduced are folded into the patches that
>   introduced them. Three pre-existing bugs found along the way are
>   separate patches at the end, so the set is 10 patches instead of 7.
> - "register nfsd listeners with rpcbind": register only once the
>   listener_set reply has been parsed. On a reply carrying no rpcbind
>   attributes the sweep cleared every NFS and NFSACL entry and put
>   nothing back, leaving a running server advertising nothing. Also
>   abort rather than send a listener_set that lost the ownership flag.
> - "clear rpcbind entries when the serv is torn down" (was "... when the
>   last thread stops"): also sweep on the autostart teardown path.
> - New, pre-existing: fix a heap overflow in autostart_func(). A failed
>   listener set zeroed the pool count before the thread array was sized,
>   but the array is filled from the nfs.conf list, so any [nfsd] threads
>   setting wrote past the allocation.
> - New, pre-existing: reject a listener insertion when the table is full,
>   instead of writing one element past nfsd_sockets[].
> - New, pre-existing: check the nla_put() calls in set_listeners().
> - Link to v1: https://lore.kernel.org/r/20260910-nfsdctl-rpcb-v1-0-4bba1e7a3a43@kernel.org
> 
> ---
> Jeff Layton (10):
>       nfsdctl: sync nfsd_netlink.h with the kernel
>       nfsdctl: probe the policy of any command
>       nfsdctl: share the listener nest parser
>       nfsdctl: add rpcbind registration helpers
>       nfsdctl: register nfsd listeners with rpcbind
>       nfsdctl: clear rpcbind entries when the serv is torn down
>       nfsdctl: add missing break in update_listeners()
>       nfsdctl: fix heap overflow in autostart_func() listener cleanup
>       nfsdctl: reject a listener insertion when the table is full
>       nfsdctl: check the netlink attribute puts in set_listeners()
> 
>  configure.ac                   |   2 +-
>  support/include/nfsd_netlink.h |  18 +++
>  utils/nfsdctl/Makefile.am      |  10 +-
>  utils/nfsdctl/nfsdctl.8        |  16 +++
>  utils/nfsdctl/nfsdctl.adoc     |  17 +++
>  utils/nfsdctl/nfsdctl.c        | 256 +++++++++++++++++++++++++++++++++--------
>  utils/nfsdctl/rpcbind.c        | 251 ++++++++++++++++++++++++++++++++++++++++
>  utils/nfsdctl/rpcbind.h        |  42 +++++++
>  8 files changed, 562 insertions(+), 50 deletions(-)
> ---
> base-commit: 630393e9c73667b3fda6a7507cb5171ec0ec38dc
> change-id: 20260908-nfsdctl-rpcb-adc5a1fc090b
> 
> Best regards,

Forgot to remove the "RFC" from the subject line for this one. Consider
this an "official" submission.

Thanks,
-- 
Jeff Layton <jlayton@kernel.org>