All of lore.kernel.org
 help / color / mirror / Atom feed
From: Olaf Kirch <okir@suse.de>
To: Neil Brown <neilb@suse.de>
Cc: Takashi Iwai <tiwai@suse.de>,
	Chuck Lever <chuck.lever@oracle.com>,
	nfs@lists.sourceforge.net
Subject: Re: Regression: NFS locking hangs when statd not running.
Date: Fri, 20 Oct 2006 14:41:19 +0200	[thread overview]
Message-ID: <20061020124119.GE27351@suse.de> (raw)
In-Reply-To: <17720.41873.549441.330938@cse.unsw.edu.au>

On Fri, Oct 20, 2006 at 08:23:13PM +1000, Neil Brown wrote:
>  The problem is that there is a case where we don't want the retry.
> 
>  In 2.6.18, if statd isn't running, then a lock attempt returns ENOLCK
>  immediately, which I think is good.
>  In 2.6.19-rc, in the same situation, a lock attempt waits for a major
>  timeout (30 seconds for TCP mounts) and is not interruptible for this
>  whole time (even with '-o intr' mounts).
> 
>  So: what to do?  Should we retry requests when portmap says "no such
>  service".

When lockd tries to do an upcall to statd? Definitely no, I'd say.
Essentially, statd upcalls should be a one-shot affair with minimal
timeout.

>  I think that for requests to a remote service - lockd or nfsd - we do
>  want to retry.  The server might be rebooting and so "no such
>  service" should be treated much like "no reply".

I believe this should depend on the semantics of the parent mount.
Basically, we should copy intr,hard from the NFS mount to the lockd
client we use, and from there to the portmap client. Otherwise
in a HA setup where you have hard mounts, you will suddenly start
seeing IO errors during failover.

The patch looks good, except maybe I'd use a different name, like
RPC_CLNT_BIND_NORETRY or some such.

Olaf
-- 
Olaf Kirch   |  --- o --- Nous sommes du soleil we love when we play
okir@suse.de |    / | \   sol.dhoop.naytheet.ah kin.ir.samse.qurax

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs

  reply	other threads:[~2006-10-20 12:41 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-20 10:23 Regression: NFS locking hangs when statd not running Neil Brown
2006-10-20 12:41 ` Olaf Kirch [this message]
2006-10-20 13:00   ` Chuck Lever
2006-10-24  1:26     ` Neil Brown
2006-10-24  1:06   ` Neil Brown
2006-10-24  1:53     ` Trond Myklebust
2006-10-24  2:17       ` Neil Brown

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=20061020124119.GE27351@suse.de \
    --to=okir@suse.de \
    --cc=chuck.lever@oracle.com \
    --cc=neilb@suse.de \
    --cc=nfs@lists.sourceforge.net \
    --cc=tiwai@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.