On 12/01/2016 10:58 PM, NeilBrown wrote: > This is an RFC series. A little voice at the back of my head keeps > telling me that I'm over-engineering, but there isn't really that much > new code, and I think the result has a lot to recommend it. > > But please tell me if I'm wrong. > > - Various daemons (not all) are enhance to accept configuration > information from /etc/nfs.conf > - the conffile reader is enhanced to support include files, and > particularly to be able to include /etc/sysconf/X or /etc/defaults/X > files usefully > - nfs-config.service is removed, because it isn't really needed with > the above. > - documentation for all the above is provided, including a new > nfs.systemd man page which gives the bigger picture. This is some nice work! Its definitely a step in the right direction... IMHO... I especially like the idea of the include statement, I'm very curious to see how that work! If this is finished, I'll start testing and hopefully get it in... soon. steved. > > Thanks, > NeilBrown > > > --- > > NeilBrown (15): > Add man-page describing /etc/nfs.conf > conffile: add bool support > Add /etc/nfs.conf support to rpc.nfsd > Add /etc/nfs.conf support for mountd. > Add /etc/nfs.conf support for statd > Add /etc/nfs.conf support for sm-notify > conffile: free image of config file after parsing > conffile: split loading of file into a separate function. > conffile: add support for include files. > conffile: strip "quotes" from values in conf file. > conffile: ignore setting of empty string > conffile: allow $name expansion of tag values. > statd: allow --no-notify to be passed via environment variable. > systemd: Remove the nfs-config.service > Add nfs.systemd man page > > > configure.ac | 6 - > support/include/conffile.h | 2 > support/nfs/conffile.c | 147 +++++++++++++++++++++--------- > systemd/Makefile.am | 5 + > systemd/README | 22 +++- > systemd/nfs-blkmap.service | 3 - > systemd/nfs-config.service.in | 13 --- > systemd/nfs-idmapd.service | 6 - > systemd/nfs-mountd.service | 6 - > systemd/nfs-server.service | 7 - > systemd/nfs.conf.man | 186 ++++++++++++++++++++++++++++++++++++++ > systemd/nfs.systemd.man | 167 ++++++++++++++++++++++++++++++++++ > systemd/rpc-gssd.service.in | 7 - > systemd/rpc-statd-notify.service | 6 - > systemd/rpc-statd.service | 7 - > systemd/rpc-svcgssd.service | 6 - > utils/mountd/mountd.c | 36 +++++++ > utils/mountd/mountd.man | 34 +++++++ > utils/nfsd/nfsd.c | 36 +++++++ > utils/nfsd/nfsd.man | 49 +++++++++- > utils/statd/sm-notify.c | 11 ++ > utils/statd/sm-notify.man | 27 ++++++ > utils/statd/statd.c | 25 +++++ > utils/statd/statd.man | 40 ++++++++ > 24 files changed, 737 insertions(+), 117 deletions(-) > delete mode 100644 systemd/nfs-config.service.in > create mode 100644 systemd/nfs.conf.man > create mode 100644 systemd/nfs.systemd.man > > -- > Signature > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
