On Fri, Feb 05, 2010 at 11:10:02AM -0500, J. Bruce Fields wrote: > On Thu, Feb 04, 2010 at 05:19:56PM -0500, Steve Dickson wrote: > > On 02/01/2010 02:58 PM, J. Bruce Fields wrote: > > > On Wed, Jan 27, 2010 at 05:26:06PM -0500, J. Bruce Fields wrote: > > >> The current kernel code should not be enabled by default, because it > > >> does not yet attempt to be a conform completely to the rfc; for example, > > >> some required pieces of protocol are missing. > > >> > > >> Therefore the kernel defaults to leaving minorversion1 off. When the > > >> code matures sufficiently, that default will change. > > >> > > >> That kernel default becomes meaningless if nfs-utils always explicitly > > >> turns 4.1 on or off. So, nfs-utils should by default do nothing. > > >> > > >> Provide a --enable-experimental-v41-support option to turn it on > > >> explicitly. The option is intentionally spelled out (and has no short > > >> equivalent), to help ensure that users know what they're getting into. > > Command options like this are so hard to get rid of.... We just can't introduce > > an option one release and then have it go away a few releases down the road. > > That's sure fire way to breaking existing configurations which something that is, > > has been and will continue to be unacceptable... > > OK, fair enough. It shouldn't be a problem to keep it indefinitely, > though. Actually, an even simpler option: instead of adding a new option, just modify the code so that the *absence* of -N4.1 isn't taken to mean "please turn on 4.1". People can always just do their own echo +4.1 >/proc/fs/nfsd/version if that's what they want. --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
