From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steve Dickson Subject: Re: [PATCH 2/2] Enable v4 mounts when either "nfsvers=4" or "vers=4" option are set (vers-02) Date: Thu, 27 Aug 2009 15:28:25 -0400 Message-ID: <4A96DE59.2030302@RedHat.com> References: <4A9424DB.2040303@RedHat.com> <4A942593.8030101@RedHat.com> <4A943914.9020104@RedHat.com> <7AB7BC01-F9E5-4611-BB4B-2B6E27069631@oracle.com> <4A944645.1020003@RedHat.com> <1251233345.25372.67.camel@heimdal.trondhjem.org> <4A954FBF.3030606@RedHat.com> <23199F1A-EA23-4DE1-AAB8-92D4B508C865@oracle.com> <4A956BF2.6000902@RedHat.com> <4A95760C.9000604@RedHat.com> <1251316764.5226.21.camel@heimdal.trondhjem.org> <4A9694D2.2030205@RedHat.com> <1251395324.5173.21.camel@heimdal.trondhjem.org> <41843262-CA6A-47A3-94C0-69F7711944EC@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: Linux NFS Mailing list , Linux NFSv4 mailing list To: Chuck Lever Return-path: In-Reply-To: <41843262-CA6A-47A3-94C0-69F7711944EC@oracle.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: nfsv4-bounces@linux-nfs.org Errors-To: nfsv4-bounces@linux-nfs.org List-ID: On 08/27/2009 01:58 PM, Chuck Lever wrote: > > On Aug 27, 2009, at 1:48 PM, Trond Myklebust wrote: > >> On Thu, 2009-08-27 at 10:14 -0400, Steve Dickson wrote: >>> I say we go with the proposed patch since its simple, architecturally >>> sound, will not cause problems down the road as long as nfs4 remains >>> a standalone file system and its done! Plus I have a patch waiting >>> in the wings that actually does make v4 the first mount that is >>> tried... making v4 the default version... >> >> I do worry that if, at some point, we do get rid of the nfs4 filesystem >> alias then we may find ourselves in trouble if we have a large base of >> mount programs out there that translate '-t nfs -overs=4' into '-t >> nfs4'. Well at that point, if that every and happens and if the mount command is the only thing that has to change it would a minor miracle... IMHO... >> >> I think that if you want to go down this path, you should somehow ensure >> that the resulting program is capable of passing '-t nfs -overs=4' down >> to the kernel (perhaps a configuration file option?). > > My impression was that this mount command behavior would be controlled > with a kernel version switch in mount.nfs, much the same way that we > already handle switching between text-based and legacy mounting. That > won't help with old nfs-utils on new kernels, though. Right... and I really don't want to adding yet another configuration option for something that should be so simple... steved.