On Thu, Apr 02, 2009 at 10:05:53PM +0300, Benny Halevy wrote: > On Apr. 02, 2009, 21:46 +0300, "J. Bruce Fields" <bfields@xxxxxxxxxxxx> wrote: > > On Thu, Apr 02, 2009 at 09:41:33PM +0300, Benny Halevy wrote: > >> It looks like rpc.nfsd's command line syntax is all negative. > >> I.e. all versions are on by default and they can be turned off > >> using the -N option. I can either add an option to enable minorversion > >> or have it enabled by default and add an option to disable it. > > > > I'm not a big fan of that commandline syntax--seems to me it should give > > a range rather than a list of versions not to support. > > > > For now the important thing is that the kernel default minorversion 1 to > > off. We can worry about what rpc.nfsd should do later--I guess a > > commandline option to turn on specific minor versions would work.... > > Maybe allow --nfs-version as well as --no-nfs-version? > > Sounds like a good idea. > How about --nfs-versions (in plural) to correspond to /proc/fs/nfsd/versions > and then pass the whole versions string? I suppose that would work, but it seems a little inconsistent with the current syntax, and with the way unix commandlines works in general. Something a little less idiosyncratic would be easier to document and remember. --b. -- 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
