From: Steve Dickson <SteveD@redhat.com>
To: Steve Dickson <SteveD@redhat.com>
Cc: Jeff Layton <jlayton@redhat.com>,
linux-nfs@vger.kernel.org, bfields@fieldses.org, neilb@suse.de
Subject: Re: [PATCH] rpc.nfsd: mount up nfsdfs is it doesn't appear to be mounted yet (try #4)
Date: Thu, 16 Sep 2010 17:32:35 -0400 [thread overview]
Message-ID: <4C928CF3.6030508@RedHat.com> (raw)
In-Reply-To: <4C921580.2050903-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
On 09/16/2010 09:02 AM, Steve Dickson wrote:
> Hey Jeff,
>
> Here is the patch with the L_WARNING change and the
> s/else if/if change I just mentioned...
>
> steved.
>
> Author: Jeff Layton <jlayton@redhat.com>
> Date: Thu Sep 16 08:58:02 2010 -0400
>
> There's a bit of a chicken and egg problem when nfsd is run the first
> time. On Fedora/RHEL at least, /proc/fs/nfsd is mounted up whenever nfsd
> is plugged in via a modprobe.conf "install" directive.
>
> If someone runs rpc.nfsd without plugging in nfsd.ko first,
> /proc/fs/nfsd won't be mounted and rpc.nfsd will end up using the legacy
> nfsctl interface. After that, nfsd will be plugged in and subsequent
> rpc.nfsd invocations will use that instead.
>
> This is a problem as some nfsd command-line options are ignored when the
> legacy interface is used. It'll also be a problem for people who want
> IPv6 enabled servers. The upshot is that we really don't want to use the
> legacy interface unless there is no other option.
>
> To avoid this situation, have rpc.nfsd check to see if the "threads"
> file is already present. If it's not, then make an attempt to mount
> /proc/fs/nfsd. This is a "best-effort" sort of thing, however so we
> just ignore the return code from the mount attempt and fall back to
> using nfsctl() if it fails.
>
> Signed-off-by: Jeff Layton <jlayton@redhat.com>
> Signed-off-by: Steve Dickson <steved@redhat.com>
Committed...
steved.
prev parent reply other threads:[~2010-09-16 21:32 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-31 19:32 [PATCH] rpc.nfsd: mount up nfsdfs is it doesn't appear to be mounted yet (try #2) Jeff Layton
2010-09-01 20:48 ` J. Bruce Fields
2010-09-01 20:56 ` Jeff Layton
2010-09-01 21:31 ` Neil Brown
2010-09-02 0:17 ` Jeff Layton
2010-09-02 1:32 ` J. Bruce Fields
2010-09-02 11:29 ` Steve Dickson
2010-09-02 11:55 ` Jeff Layton
2010-09-02 14:04 ` Chuck Lever
2010-09-02 14:25 ` J. Bruce Fields
2010-09-02 16:41 ` Steve Dickson
2010-09-02 18:49 ` J. Bruce Fields
2010-09-02 14:30 ` Chuck Lever
2010-09-14 13:23 ` Jeff Layton
2010-09-15 20:09 ` Steve Dickson
2010-09-15 22:31 ` Jeff Layton
2010-09-16 11:06 ` Steve Dickson
2010-09-16 11:32 ` Jeff Layton
[not found] ` <20100916073203.2c217bfc-xSBYVWDuneFaJnirhKH9O4GKTjYczspe@public.gmane.org>
2010-09-16 11:43 ` Steve Dickson
2010-09-16 12:30 ` Steve Dickson
2010-09-16 13:02 ` [PATCH] rpc.nfsd: mount up nfsdfs is it doesn't appear to be mounted yet (try #4) Steve Dickson
[not found] ` <4C921580.2050903-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2010-09-16 13:40 ` Jeff Layton
2010-09-16 21:32 ` Steve Dickson [this message]
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=4C928CF3.6030508@RedHat.com \
--to=steved@redhat.com \
--cc=bfields@fieldses.org \
--cc=jlayton@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 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.