All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chuck Lever <chuck.lever@oracle.com>
To: Steve Dickson <SteveD@redhat.com>
Cc: Linux NFS Mailing list <linux-nfs@vger.kernel.org>,
	Linux NFSv4 mailing list <nfsv4@linux-nfs.org>
Subject: Re: [PATCH 2/2] Enable v4 mounts when either "nfsvers=4" or "vers=4" option are set (vers-02)
Date: Wed, 26 Aug 2009 15:50:21 -0400	[thread overview]
Message-ID: <B26EBCDF-5836-440E-81D5-5E089158BFE1@oracle.com> (raw)
In-Reply-To: <4A95760C.9000604@RedHat.com>

On Aug 26, 2009, at 1:51 PM, Steve Dickson wrote:
> On 08/26/2009 01:22 PM, Chuck Lever wrote:
>>>> and if your proposed method to handle -t nfs -o vers=4 will make
>>>> it more complicated to get there.
>>> No. I'm proposing a simple shorthand patch that will make mounting  
>>> nfs4
>>> file systems easier in hope of moving the technology forward by  
>>> making
>>> it more accessible... What I believe you are proposing is  
>>> architecture
>>> change to hide the fact nfs and nfs4 are separate file systems...
>>
>> Nope, we're proposing doing the simple method in the kernel instead  
>> of
>> in the mount command.
>>
> My apologize then... I was misinterpreting what you guys were  
> suggesting..
> (email sometimes causes that... :-\ )
>
> I don't think the -o v4 translation will be as easy as a
> "simple method in the kernel" and it surely will not be as simple
> and unintrusive as the patch I'm proposing.... Here is why...
>
> From an Linux  architecture standpoint the mount command *always*
> know what file system its mounting. There not been a precedence set
> where mount, mounts one file system which turns into mounting a
> different file system. Meaning there is no kernel support for
> nfs_get_sb() to all of sudden decide to roll back the system call
> and jump into nfs4_get_sb() (or vice a versa depending on which is the
> default).

Yeah, switching file system types in the mount(2) system call is the  
fly in the ointment.  I'm just wondering if Trond had some thoughts  
about making that more feasible.

> Of course we could set that precedence and quit frank we would have
> to. I'm not totally against that, although other in the kernel  
> community
> might be...
>
> But there is no way that re-architecturing of kernel will as simple
> and straightforward as following the Linux standard architecture of
> having mount, mount the correct file system.

--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com

  reply	other threads:[~2009-08-26 19:50 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-25 17:52 [PATCH 0/2] Enable v4 mounts when either "nfsvers=4" or "vers=4" option are set (vers-02) Steve Dickson
2009-08-25 17:54 ` [PATCH 1/2] " Steve Dickson
     [not found] ` <4A9424DB.2040303-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-08-25 17:55   ` [PATCH 2/2] " Steve Dickson
2009-08-25 18:59     ` Chuck Lever
2009-08-25 19:18       ` Steve Dickson
2009-08-25 19:32         ` Chuck Lever
2009-08-25 20:15           ` Steve Dickson
2009-08-25 20:37             ` Chuck Lever
2009-08-26 12:10               ` Steve Dickson
2009-08-25 20:49             ` Trond Myklebust
2009-08-26 12:28               ` Steve Dickson
2009-08-26 14:20               ` Chuck Lever
2009-08-26 15:07                 ` Steve Dickson
2009-08-26 16:35                   ` Chuck Lever
2009-08-26 17:08                     ` Steve Dickson
2009-08-26 17:22                       ` Chuck Lever
2009-08-26 17:51                         ` Steve Dickson
2009-08-26 19:50                           ` Chuck Lever [this message]
2009-08-26 19:59                             ` Trond Myklebust
2009-08-27 14:14                               ` Steve Dickson
2009-08-27 17:32                                 ` Chuck Lever
2009-08-28  2:55                                   ` Steve Dickson
2009-08-28 16:12                                   ` Christoph Hellwig
2009-08-28 16:35                                     ` Steve Dickson
     [not found]                                       ` <4A980751.7070206-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-08-28 16:41                                         ` Christoph Hellwig
2009-08-28 16:44                                           ` Chuck Lever
2009-08-28 16:53                                           ` Steve Dickson
2009-08-28 16:59                                             ` Christoph Hellwig
2009-08-28 17:19                                               ` Steve Dickson
2009-08-27 17:48                                 ` Trond Myklebust
2009-08-27 17:58                                   ` Chuck Lever
2009-08-27 19:28                                     ` 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=B26EBCDF-5836-440E-81D5-5E089158BFE1@oracle.com \
    --to=chuck.lever@oracle.com \
    --cc=SteveD@redhat.com \
    --cc=linux-nfs@vger.kernel.org \
    --cc=nfsv4@linux-nfs.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 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.