public inbox for linux-nfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Steve Dickson <SteveD@redhat.com>
To: linux-nfs@vger.kernel.org
Cc: Neil Brown <neilb@suse.de>
Subject: Re: [nfs-utils PATCH] retry on EPERM from NFSv4 mount attempt
Date: Mon, 30 Nov 2009 08:11:42 -0500	[thread overview]
Message-ID: <4B13C48E.5020009@RedHat.com> (raw)
In-Reply-To: <4B0BEDDB.1010203-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>

Sorry for the delayed response, I took some time off...

On 11/24/2009 09:29 AM, Steve Dickson wrote:
> 
> 
> On 11/23/2009 06:32 PM, Neil Brown wrote:
>>
>> Hi,
>>  I recently packaged nfs-utils 1.2.1 for openSUSE and fairly quickly
>>  got a bug report - "-o nfsvers=3" was needed to mount NFSv3
>>  filesystems.
>>
>>  mount.nfs in 1.2.1 will first try a v4 mount but will fall-back to v3
>>  if it gets ENOENT.  This works fine.
>>  However for kernel prior to 2.6.25, you don't get ENOENT, you get
>>  EPERM.
>>  In that case the fall-back to v3 doesn't happen and you get a failure
>>  to mount.
>>
>>  So I think we need to fall back on EPERM as well.  See below.
> I already posted this patch on the v4 mailing list
> http://linux-nfs.org/pipermail/nfsv4/2009-November/011595.html
> but it got shot down...  at least that's how I interpreted the
> responses... 
> 
> But I do thing we need this, since there are so many server 
> that will simple break if we don't... Agreed?

There appears to be a consensus that we need to do something
but no agreement on what.... 

I believe the options are:
   1) Start servers with '-N 4' when there is no root configured.
      * The problem I see with this is, in the event v4 support is wanted 
        its yet another configuration knob that has to be turned, basically
        making it more difficult for people use v4. 

   2) Change the kernel to return NFS4ERR_SERVERFAULT when there is no
      root configured. 
      * I see this is yet another errno the mounting code has to deal with..
        We are up to two errnos, do we really want to add a third?

Neither one of these solutions deal with legacy servers or move
the "v4 technology" forward... IMHO...   

So what I suggest we do is:
   1) Finish the dynamic pseudo root support asap, since all this
      goes away when that support exists. 

   2) We commit the "roll back to v3 on EPERM errors" and we draw a 
      line in the sand saying if a server does not return one of these
      errors on v4 mounts when there is a not a root configured, the
      server is broken and needs to be fix. Point, these are the only
      two errnos that will cause a v3 roll back.
      * My only concern is how we are going to work with 
        non-linux servers and how much extra network trace will 
        this cause during automount storms... 

But in the end, I think we will have to go with the "roll back on EPERM errors"
patch, because there is simply no way a mount command can be released 
in an enterprise environment that is not backward compatible with
existing servers.

Consensus? 

steved.
 

  parent reply	other threads:[~2009-11-30 13:11 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-23 23:32 [nfs-utils PATCH] retry on EPERM from NFSv4 mount attempt Neil Brown
     [not found] ` <19211.7054.291514.185591-wvvUuzkyo1EYVZTmpyfIwg@public.gmane.org>
2009-11-24 14:29   ` Steve Dickson
     [not found]     ` <4B0BEDDB.1010203-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-11-24 20:56       ` J. Bruce Fields
2009-11-24 21:19         ` Peter Staubach
2009-11-24 21:51         ` Neil Brown
     [not found]           ` <20091125085122.316f4eb3-wvvUuzkyo1EYVZTmpyfIwg@public.gmane.org>
2009-11-24 21:58             ` Peter Staubach
2009-11-24 22:22               ` Neil Brown
     [not found]                 ` <20091125092227.77735d5a-wvvUuzkyo1EYVZTmpyfIwg@public.gmane.org>
2009-11-24 22:29                   ` Peter Staubach
2009-11-24 22:54                     ` J. Bruce Fields
2009-11-24 22:58                   ` Trond Myklebust
2009-11-30 13:11       ` Steve Dickson [this message]
     [not found]         ` <4B13C48E.5020009-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-11-30 16:43           ` Chuck Lever
2009-11-30 17:41             ` Steve Dickson
     [not found]               ` <4B1403CA.8050107-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-11-30 17:52                 ` Chuck Lever
2009-11-30 18:12                   ` Steve Dickson
2009-11-30 18:18           ` J. Bruce Fields
2009-11-30 21:59             ` Neil Brown
     [not found]               ` <20091201085916.7c1bb644-wvvUuzkyo1EYVZTmpyfIwg@public.gmane.org>
2009-11-30 22:13                 ` J. Bruce Fields
2009-12-07 22:27   ` Steve Dickson

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=4B13C48E.5020009@RedHat.com \
    --to=steved@redhat.com \
    --cc=linux-nfs@vger.kernel.org \
    --cc=neilb@suse.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox