Re: [PATCH v2 0/3] nfsdctl: add support for new lockd configuration interface

[Steve Dickson <steved@redhat.com>]

On 1/10/25 8:46 AM, Jeff Layton wrote:
> v2 is just a small update to fix the problems that Scott spotted.
> 
> This patch series adds support for the new lockd configuration interface
> that should fix this RH bug:
> 
>      https://issues.redhat.com/browse/RHEL-71698
> 
> There are some other improvements here too, notably a switch to xlog.
> Only lightly tested, but seems to do the right thing.
> 
> Port handling with lockd still needs more work. Currently that is
> usually configured by rpc.statd. I think we need to convert it to
> use netlink to configure the ports as well, when it's able.
> 
> Signed-off-by: Jeff Layton <jlayton@kernel.org>
I have reluctantly committed these because the
nlm arg still fails (genl_ctrl_resolve() is not finding the family)
but all the other args seem to work and other patches
are dependent on these that do fixes bugs.

The Bakeathon is coming up so hopefully we can test these
patch there.

Committed (tag: nfs-utils-2-8-3-rc7)

steved.


> --- 
> Changes in v2:
> - properly regenerate manpages
> - fix up bogus merge conflict
> - add D_GENERAL xlog messages when nfsdctl starts and exits
> - Link to v1: https://lore.kernel.org/r/20250109-lockd-nl-v1-0-108548ab0b6b@kernel.org
> 
> ---
> Jeff Layton (3):
>        nfsdctl: convert to xlog()
>        nfsdctl: fix the --version option
>        nfsdctl: add necessary bits to configure lockd
> 
>   configure.ac                  |   4 +
>   utils/nfsdctl/lockd_netlink.h |  29 ++++
>   utils/nfsdctl/nfsdctl.8       |  15 +-
>   utils/nfsdctl/nfsdctl.adoc    |   8 +
>   utils/nfsdctl/nfsdctl.c       | 331 ++++++++++++++++++++++++++++++++++--------
>   5 files changed, 321 insertions(+), 66 deletions(-)
> ---
> base-commit: 65f4cc3a6ce1472ee4092c4bbf4b19beb0a8217b
> change-id: 20250109-lockd-nl-6272fa9e8a5d
> 
> Best regards,