From: Steve Dickson <SteveD@redhat.com>
To: Trond Myklebust <trond.myklebust@primarydata.com>
Cc: 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 12:27:13 -0500 [thread overview]
Message-ID: <54639871.3010903@RedHat.com> (raw)
In-Reply-To: <CAHQdGtSwgyrBzc9iOmPLCd+k3Y9c2QeAFJdks4OBrXZfpiatCA@mail.gmail.com>
On 11/12/2014 10:28 AM, Trond Myklebust wrote:
> On Wed, Nov 12, 2014 at 10:10 AM, Steve Dickson <SteveD@redhat.com> wrote:
>>
>>
>> On 11/12/2014 09:31 AM, Trond Myklebust wrote:
>>>> My point is I don't think we need another variable, say
>>>>> DefaultMinVers, that defines the minor version of v4. I'm
>>>>> thinking that's its overkill and adds unnecessary complexity.
>>>>>
>>> I never said we do.
>> Ok... I misunderstood...
>>
>>>
>>> I think we're in agreement mostly; the only point where I see
>>> disagreement is when Defaultvers is unset.
>>> My position is that in that situation, we don't know what starting
>>> point to use for minor version negotiation, and so we should just
>>> default to minor version 0: if the sysadmin want a different default,
>>> then the answer is to set Defaultvers...
>> Gotta... and there is a disagreement... I saying we make the
>> default the highest supported minor version. With the
>> Linux client and server that's v4.2. So when no option is
>> given and Defaultvers is not set, try 4.2, then 4.1 and
>> then 4.0 and finally v3.
>
> Only for Linux 3.11 and newer, and only if they enable CONFIG_NFS_V4_2
> / CONFIG_NFSD_V4_SECURITY_LABEL.
Being dynamic is never easy! ;-)
The server has /proc/fs/nfsd/versions file that defines the
supported versions. Maybe we can come up with something
on the client that would tell what protocols are supported...
>
> Unless we want to have different defaults for older kernels, this sort
> of implies that we're moving in the direction of coupling the
> nfs-utils releases more tightly to the kernel version. I'm neutral to
> that, but I do want to call it out.
>
>> 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"
Clever.. :-)
>
>> One down side of being the aggressive with minor version
>> negotiation is legacy servers (aka AIX). Today we
>> don't negotiate well with those types of servers...
>> Its not our fault, but is a problem...
>
> Is this because they don't implement that part of RFC3530?
The server returns the wrong type of error when a v4
mount is done so the mount fails instead of rolling back
to v3.
steved.
next prev parent reply other threads:[~2014-11-12 17:27 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
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 [this message]
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=54639871.3010903@RedHat.com \
--to=steved@redhat.com \
--cc=bcodding@redhat.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.