linux-nfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Steve Dickson <SteveD@redhat.com>
To: Steve Dickson <SteveD@redhat.com>
Cc: "Myklebust, Trond" <Trond.Myklebust@netapp.com>,
	"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH] Get normalized paths for comparing NFS export paths
Date: Sun, 04 Mar 2012 17:58:38 -0500	[thread overview]
Message-ID: <4F53F39E.9050802@RedHat.com> (raw)
In-Reply-To: <4F53ED3C.2020209@RedHat.com>

On 03/04/2012 05:31 PM, Steve Dickson wrote:
>> NFSv4 is the only protocol that has a standard mount path syntax, and
>> that is because the client performs the job of interpreting the path
>> name and translating it into PUTROOTFH followed by a bunch of LOOKUPs.
>> IOW: the standard syntax there is the one imposed by the client.
>>
>> There is nothing in the NFSv2/v3 MOUNT spec that states that a path
>> needs to start with '/'. Nor is there even anything in the spec that
>> states that '/' is required to be used as the directory component
>> separator. The X/OPEN docs state that '/' is recommended for
>> portability, but do not make it a requirement. See
>> http://pubs.opengroup.org/onlinepubs/9629799/chap8.htm#tagcjh_09_02_02_03 
>>
>> IOW: I'm perfectly allowed to set up a 'mountd' server that uses '\' or
>> even something like '|' as a path component separator. This kind of
>> patch would break the client's existing ability to mount from such a
>> server.
> And where does an server like this exist? One that uses '|' as its
> path component separator?? ;-)
> 
> Just to be clear, you are ok with striping the multiple slashes, for
> all protocol versions, but its only kosher to added the leading 
> slash for v4 mounts. Correct?  
After further review... it appears this is only a v4 issue since
v3 mounts with multiple slashes appear in /proc/mounts with the
multiple slashes (unlike v4 mounts)... 

So to restate... this normalizing of the path names (aka striping 
multiple slashes or added the leading slash) only need to occur on 
(successful) v4 mounts... Although the former would not be an 
problem with v2/v3 mounts but its not needed....

steved

  parent reply	other threads:[~2012-03-04 22:58 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-03  1:42 [PATCH] Check for beginning '/' in the mount path Malahal Naineni
2012-02-03 11:15 ` Boaz Harrosh
2012-02-03 12:16   ` NeilBrown
2012-02-03 14:29     ` Malahal Naineni
2012-02-05 11:03       ` Boaz Harrosh
2012-02-06 18:11         ` Malahal Naineni
2012-02-07 20:44         ` [PATCH] Get normalized paths for comparing NFS export paths Malahal Naineni
2012-02-16 18:09           ` Malahal Naineni
2012-03-02 19:10           ` Steve Dickson
2012-03-02 19:27             ` Malahal Naineni
2012-03-02 20:57               ` Steve Dickson
2012-03-02 22:01                 ` Malahal Naineni
2012-03-03 17:39                   ` Steve Dickson
2012-03-03 19:12                     ` Myklebust, Trond
2012-03-04 22:31                       ` Steve Dickson
2012-03-04 22:46                         ` Myklebust, Trond
2012-03-04 23:08                           ` Steve Dickson
2012-03-05  4:46                           ` Malahal Naineni
2012-03-05 12:03                             ` Steve Dickson
2012-03-04 22:58                         ` Steve Dickson [this message]
2012-03-04 23:26                           ` Myklebust, Trond
2012-03-05  0:03                             ` Steve Dickson
2012-03-05  2:04                               ` Myklebust, Trond
2012-03-05  4:53                                 ` Malahal Naineni
2012-03-05 11:55                                 ` Steve Dickson
2012-03-05 14:47                                   ` Malahal Naineni
2012-03-05 15:03                                     ` 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=4F53F39E.9050802@RedHat.com \
    --to=steved@redhat.com \
    --cc=Trond.Myklebust@netapp.com \
    --cc=linux-nfs@vger.kernel.org \
    /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;
as well as URLs for NNTP newsgroup(s).