On Tue, 2025-01-14 at 16:09 -0500, Scott Mayhew wrote: > 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. > That sounds good. We can do that in a separate patch. > Reviewed-by: Scott Mayhew <smayhew@xxxxxxxxxx> Thanks! > > --- > > 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> > > > -- Jeff Layton <jlayton@xxxxxxxxxx>
