All of lore.kernel.org
 help / color / mirror / Atom feed
From: "J. Bruce Fields" <bfields@fieldses.org>
To: "Myklebust, Trond" <Trond.Myklebust@netapp.com>
Cc: Chuck Lever <chuck.lever@oracle.com>,
	"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH 0/2] [RFC] Maybe avoid gssd upcall timeout
Date: Wed, 15 May 2013 14:15:51 -0400	[thread overview]
Message-ID: <20130515181551.GP16811@fieldses.org> (raw)
In-Reply-To: <1368634683.3568.5.camel@leira.trondhjem.org>

On Wed, May 15, 2013 at 04:18:03PM +0000, Myklebust, Trond wrote:
> On Mon, 2013-05-13 at 12:25 -0400, Chuck Lever wrote:
> > Hi-
> > 
> > Here's a stab at addressing the 15 second wait for some 3.10 sec=sys
> > mounts where the client is not running rpc.gssd.
> > 
> > After reverting the "use krb5i for SETCLIENTID" patch, I've added
> > the AUTH_SYS fallback in the EACCES case in
> > nfs4_discover_server_trunking().  I'm not sure whether we need to
> > supplement what's there now, or replace it.
> > 
> > "case -ENOKEY:" is added so the kernel will recognize that when gssd
> > is changed to return that instead of EACCES in this case.  If the
> > second patch is appled to 3.7 stable and following, it might be a way
> > to address the same regression in older kernels.
> > 
> > I've been focused on another bug this week, so this has seen very
> > light testing only.  Looking for comments.
> 
> I'd like to propose a different approach: we can set up rpc_pipefs files
> clnt/gssd and clnt/krb5

I'm not sure what paths you mean exactly?  Is "clnt" a top-level
rpc-pipefs directory, or a dummy client at the level of other rpc
clients (like rpc_pipefs/nfs/clnt/gssd).

I'm assuming the latter--seems like it might work.

Long-term, it might also be nice to add an interface to allow the kernel
to determine whether gssd is running, without depending on timeouts.  It
could be something very trivial.

--b.

> as "honeypots" that rpc.gssd will connect to,
> but that won't do any upcalls. When gssd connects, we set a
> per-rpc_net_ns variable that tells us 'gssd' is up and running. That
> variable only gets cleared if we see a timeout.
> 
> 
> -- 
> Trond Myklebust
> Linux NFS client maintainer
> 
> NetApp
> Trond.Myklebust@netapp.com
> www.netapp.com
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

      parent reply	other threads:[~2013-05-15 18:15 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-13 16:25 [PATCH 0/2] [RFC] Maybe avoid gssd upcall timeout Chuck Lever
2013-05-13 16:25 ` [PATCH 1/2] NFS: Revert commit 4edaa308 and follow-on fixes Chuck Lever
2013-05-13 16:25 ` [PATCH 2/2] NFS: Fall back to AUTH_SYS for SETCLIENTID (take 2) Chuck Lever
2013-05-15 16:04   ` J. Bruce Fields
2013-05-15 16:23     ` Chuck Lever
2013-05-15 17:16       ` J. Bruce Fields
2013-05-15 17:39         ` J. Bruce Fields
2013-05-15 17:42           ` Chuck Lever
2013-05-15 17:48             ` J. Bruce Fields
2013-05-15 19:28               ` Chuck Lever
2013-05-15 19:52                 ` J. Bruce Fields
2013-05-15 19:55                   ` Myklebust, Trond
2013-05-15 16:18 ` [PATCH 0/2] [RFC] Maybe avoid gssd upcall timeout Myklebust, Trond
2013-05-15 16:22   ` Chuck Lever
2013-05-15 16:24     ` Myklebust, Trond
2013-05-15 16:30       ` Chuck Lever
2013-05-15 16:40         ` Myklebust, Trond
2013-05-15 16:47           ` Chuck Lever
2013-05-15 18:15   ` J. Bruce Fields [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=20130515181551.GP16811@fieldses.org \
    --to=bfields@fieldses.org \
    --cc=Trond.Myklebust@netapp.com \
    --cc=chuck.lever@oracle.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 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.