From: Chuck Lever <chuck.lever@oracle.com>
To: NeilBrown <neilb@suse.de>
Cc: nfs@lists.sourceforge.net, Trond Myklebust <trond.myklebust@fys.uio.no>
Subject: Re: [PATCH 000 of 2] A couple of possible nfs client patches
Date: Tue, 06 Mar 2007 09:57:44 -0500 [thread overview]
Message-ID: <45ED8168.4050203@oracle.com> (raw)
In-Reply-To: <20070306163710.8128.patches@notabene>
[-- Attachment #1: Type: text/plain, Size: 808 bytes --]
NeilBrown wrote:
> Hi Trond,
> I wonder if you would consider the following two patches.
>
> The first fixes a real bug that is trivial to demonstrate: script
> below. The problem is that the fattr which is extracted from
> READDIRPLUS results does not get a ->time_start set. This can result
> is incorrect attributes being set on files.
>
> The second is really a work-around for a deeper problem (invalidate_inode_pages2
> racing with do_no_page) but has value in itself I think. It extents the combination
> -o nolock,nocto
> to avoid flush and invalidate on locking requests.
This sounds like the behavior enabled by the 'llock' mount option on
Solaris. Is that your intent?
IMO it is a clever idea to enable 'llock' behavior on Linux with the
combination of already existing mount options.
[-- Attachment #2: chuck.lever.vcf --]
[-- Type: text/x-vcard, Size: 265 bytes --]
begin:vcard
fn:Chuck Lever
n:Lever;Chuck
org:Oracle Corporation;Corporate Architecture Linux Projects Group
email;internet:chuck dot lever at nospam oracle dot com
title:Principal Member of Staff
tel;work:+1 248 614 5091
x-mozilla-html:FALSE
version:2.1
end:vcard
[-- Attachment #3: Type: text/plain, Size: 345 bytes --]
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
[-- Attachment #4: Type: text/plain, Size: 140 bytes --]
_______________________________________________
NFS maillist - NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs
prev parent reply other threads:[~2007-03-06 14:59 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-06 5:40 [PATCH 000 of 2] A couple of possible nfs client patches NeilBrown
2007-03-06 5:40 ` [PATCH 001 of 2] Set meaningful value for fattr->time_start in readdirplus results NeilBrown
2007-04-14 22:13 ` Trond Myklebust
2007-04-15 23:35 ` Neil Brown
2007-04-16 15:23 ` Trond Myklebust
2007-03-06 5:40 ` [PATCH 002 of 2] Avoid flush-when-locking when interclient consistency not needed NeilBrown
2007-03-06 14:40 ` Trond Myklebust
2007-03-07 23:03 ` Neil Brown
2007-04-14 22:21 ` Trond Myklebust
2007-04-16 1:49 ` Chuck Lever
2007-04-16 14:12 ` Peter Staubach
2007-04-16 14:41 ` Trond Myklebust
2007-04-16 14:48 ` Peter Staubach
2007-03-06 14:57 ` Chuck Lever [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=45ED8168.4050203@oracle.com \
--to=chuck.lever@oracle.com \
--cc=neilb@suse.de \
--cc=nfs@lists.sourceforge.net \
--cc=trond.myklebust@fys.uio.no \
/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.