All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andi Kleen <andi@firstfloor.org>
To: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Andi Kleen <andi@firstfloor.org>, Matthew Wilcox <matthew@wil.cx>,
	Christoph Hellwig <hch@infradead.org>,
	linux-fsdevel@vger.kernel.org
Subject: Re: [RFC] Remove BKL from fs/locks.c
Date: Sun, 30 Dec 2007 20:18:49 +0100	[thread overview]
Message-ID: <20071230191849.GA2623@one.firstfloor.org> (raw)
In-Reply-To: <1199040294.6962.21.camel@heimdal.trondhjem.org>

> The only problem I can see from an NFS perspective is with NFSv2/v3
> locking: unfortunately the protocol provides no way for the server to
> notify that a lock may not be granted after the client has been told to
> block. You would therefore have to bend the protocol rules by simply
> delaying replying to the client until the deadlock timeout occurred
> instead of telling it to block. I'm not sure that all clients would be
> able to cope...

If the delay is short enough (let's say < 2 jiffies) that should be surely no problem? 
If they couldn't deal with that they couldn't deal with a congested network 
either.

Otherwise lockd could just force a 0 timeout.

-Andi

  reply	other threads:[~2007-12-30 19:17 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-30  6:16 [RFC] Remove BKL from fs/locks.c Matthew Wilcox
2007-12-30  9:36 ` Stephen Rothwell
2007-12-30 14:42   ` Matthew Wilcox
2007-12-31  0:47     ` Stephen Rothwell
2007-12-30 13:05 ` Christoph Hellwig
2007-12-30 14:51   ` Matthew Wilcox
2007-12-30 15:17     ` Andi Kleen
2007-12-30 18:44       ` Trond Myklebust
2007-12-30 19:18         ` Andi Kleen [this message]
2007-12-30 19:23           ` Trond Myklebust
2008-01-04 23:41 ` J. Bruce Fields

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=20071230191849.GA2623@one.firstfloor.org \
    --to=andi@firstfloor.org \
    --cc=Trond.Myklebust@netapp.com \
    --cc=hch@infradead.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=matthew@wil.cx \
    /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.