On Fri, 10 Jan 2025, 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@xxxxxxxxxx> I think the read_nfsd_conf call should be moved out of autostart_func and into main (right before the command-line options are parsed). Right now if you enable debugging in nfs.conf, you get the "configuring listeners" and "nfsdctl exiting" messages, but not the "nfsdctl started" message. It's not a big deal though and could be done if additional debug logging is added in the future. Reviewed-by: Scott Mayhew <smayhew@xxxxxxxxxx> > --- > 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@xxxxxxxxxx > > --- > 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, > -- > Jeff Layton <jlayton@xxxxxxxxxx> >
