From: Steve Dickson <SteveD@redhat.com>
To: Anna Schumaker <Anna.Schumaker@netapp.com>,
Trond Myklebust <trond.myklebust@primarydata.com>
Cc: Chuck Lever <chuck.lever@oracle.com>,
Benjamin Coddington <bcodding@redhat.com>,
Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: mount default minor version behavior
Date: Wed, 12 Nov 2014 15:32:56 -0500 [thread overview]
Message-ID: <5463C3F8.50004@RedHat.com> (raw)
In-Reply-To: <5463C066.8030205@Netapp.com>
On 11/12/2014 03:17 PM, Anna Schumaker wrote:
> On 11/12/2014 01:41 PM, Trond Myklebust wrote:
>> On Wed, Nov 12, 2014 at 1:10 PM, Steve Dickson <SteveD@redhat.com> wrote:
>>>
>>> On 11/12/2014 10:37 AM, Chuck Lever wrote:
>>>>>> But I do see your point of not having to recompile mount
>>>>>>>> when we want to change the default minor release so
>>>>>>>> how that default is set is the question... Maybe
>>>>>>>> an environment variable??
>>>>>> That's still something that requires a user or sysadmin action, and it
>>>>>> wouldn't really play well with autofs and its ilk. As Marie Antoinette
>>>>>> would say: "Let them edit /etc/nfsmount.conf”
>>>> Fwiw: I thought this was the whole point of nfsmount.conf.
>>>> We should be able to rev nfs-utils while preserving the
>>>> administrator’s locally chosen default settings.
>>>>
>>>> +1 for using /etc/nfsmount.conf for this.
>>>>
>>> The reason the files exists is when we move the default
>>> version from v3 to v4 there would be away move the
>>> default back to v3 for legacy servers. Way
>>> way to move back from the future, if you will ;-)
>>> I never thought we would used it to go forward,
>>> just back...
>>>
>>> The problem with setting defaults in nfsmount.conf
>>> its not scalable especially in very large
>>> installations. I get it that distros can set
>>> it during installation, but that becomes error prone
>>> when different nfs-utils are used with different
>>> kernels. I think we should be more dynamic
>>>
>>> I think we barrow a paradigm from the server. On
>>> server the supported protocols are in /proc/fs/nfsd/verions
>>> that rpc.nfsd reads. We should do the same thing on the
>>> client.
>>>
>>> The kernel will tell mount.nfs where to start the negotiation.
>>>
>>> This will stop mount.nfs for needing to be compiled
>>> on minor version updates, plus it solves the problem
>>> of different kernels having different protocols enabled.
>>>
>>> I think this approach much more dynamic and it
>>> seems to work on the server side...
>>>
>> The NFS client modules are loaded on demand. The kernel will therefore
>> not actually know the capabilities until we attempt the mount.
>>
>>
> NFS v4.0, 4.1, and 4.2 are all part of the same module, though. Is there a way to analyze modules and determine what is compiled in?
Maybe come up with some global bit field could be used?
Each bit signifies a minor version is enabled...
steved.
next prev parent reply other threads:[~2014-11-12 20:32 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-11 14:39 mount default minor version behavior Benjamin Coddington
2014-11-11 15:01 ` Weston Andros Adamson
2014-11-11 15:15 ` Benjamin Coddington
2014-11-11 15:01 ` Trond Myklebust
2014-11-11 15:20 ` Benjamin Coddington
2014-11-11 15:24 ` Trond Myklebust
2014-11-11 15:27 ` Anna Schumaker
2014-11-11 16:27 ` Boaz Harrosh
2014-11-11 19:19 ` Frank Filz
2014-11-11 19:25 ` Steve Dickson
2014-11-11 19:16 ` Steve Dickson
2014-11-11 20:43 ` Trond Myklebust
2014-11-12 13:08 ` Steve Dickson
2014-11-12 14:31 ` Trond Myklebust
2014-11-12 15:10 ` Steve Dickson
2014-11-12 15:28 ` Trond Myklebust
2014-11-12 15:37 ` Chuck Lever
2014-11-12 18:10 ` Steve Dickson
2014-11-12 18:41 ` Trond Myklebust
2014-11-12 20:07 ` J. Bruce Fields
2014-11-12 20:30 ` Steve Dickson
2014-11-12 20:17 ` Anna Schumaker
2014-11-12 20:32 ` Steve Dickson [this message]
2014-11-12 22:42 ` Trond Myklebust
2014-11-13 17:57 ` Steve Dickson
2014-11-13 18:52 ` Trond Myklebust
2014-11-13 19:02 ` J. Bruce Fields
2014-11-13 19:26 ` Trond Myklebust
2014-11-13 19:55 ` J. Bruce Fields
2014-11-13 20:22 ` Trond Myklebust
2014-11-13 20:38 ` J. Bruce Fields
2014-11-13 15:07 ` Chuck Lever
2014-11-13 20:31 ` Steve Dickson
2014-11-12 17:27 ` Steve Dickson
2014-11-11 16:38 ` Chuck Lever
2014-11-17 19:47 ` Benjamin Coddington
2014-11-17 21:02 ` Chuck Lever
2014-11-12 13:09 ` Benjamin Coddington
2014-11-12 14:36 ` Trond Myklebust
2014-11-12 15:01 ` Benjamin Coddington
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5463C3F8.50004@RedHat.com \
--to=steved@redhat.com \
--cc=Anna.Schumaker@netapp.com \
--cc=bcodding@redhat.com \
--cc=chuck.lever@oracle.com \
--cc=linux-nfs@vger.kernel.org \
--cc=trond.myklebust@primarydata.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.