All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Staubach <staubach@redhat.com>
To: Neil Brown <neilb@suse.de>
Cc: Linux NFS Mailing List <nfs@lists.sourceforge.net>,
	Trond Myklebust <trond.myklebust@fys.uio.no>
Subject: Re: Can we flush directory data when the ctime changes?
Date: Tue, 15 Aug 2006 11:26:21 -0400	[thread overview]
Message-ID: <44E1E79D.5080300@redhat.com> (raw)
In-Reply-To: <17633.17452.840813.593342@cse.unsw.edu.au>

Neil Brown wrote:
> On Tuesday August 8, trond.myklebust@fys.uio.no wrote:
>   
>> On Tue, 2006-08-08 at 17:26 +1000, Neil Brown wrote:
>>     
>>> The GETATTR call that the client makes to validate the cache reports
>>> that the mtime hasn't change, *but the ctime has*.
>>>       
>> Growl! If the directory contents change, then the mtime should too. Page
>> 22 of RFC1813:
>>
>>         Mtime is the time when the file data was last modified.  Ctime
>>         is the time when the attributes of the file were last changed.
>>
>> Directory cookies are not file or directory attributes!
>>
>> In addition, NFSv3 has the cookie verifier mechanism for the server to
>> specifically inform the client that the cookies are invalid.
>>     
>
> What would you expect the user experience to be if the server returned 
> NFS3ERR_BAD_COOKIE ?
> I cannot really tell from the nfs client code, but I would guess that
> the only option would be to start again from the beginning of the
> directory, and that would lead to duplicate entries being returned,
> which is exactly the problem are seeing (though the server doesn't
> return BAD_COOKIE, it just starts again from the beginning).
>
>   

The only thing that the client can do is return an error, perhaps something
like EIO.  The real recovery needs to happen in the application, but there
isn't a way to convey the correct information to the application so that it
could do the recovery.

As was noted, transparently starting again at the beginning of the 
directory,
whether in the kernel or not, can lead to bad things happening.

In retrospect, NFS3ERR_BAD_COOKIE wasn't such a good idea.  There just
isn't enough infrastructure in the rest of the system to correctly
support the semantic that it represents.

>> IOW: this really needs to be fixed on the _server_. I'm not happy taking
>> new patches that may cause the client GETATTR calls to skyrocket again.
>>
>>     
>
> I certainly have a lot of sympathy for this position.

I would agree as well.  I think that represents a constraint on the file
systems which can be used on the server.  Although it is not a huge issue,
it sure would be nice to have it fixed.  Perhaps we need to construct the
infrastructure required in order to be able to detect and recover from
directory cookies which change mid-stream.

    Thanx...

       ps

-------------------------------------------------------------------------
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-08-15 15:26 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-08  7:26 Can we flush directory data when the ctime changes? Neil Brown
2006-08-08 13:28 ` Chuck Lever
2006-08-08 14:05 ` Trond Myklebust
2006-08-08 14:27   ` Peter Staubach
2006-08-08 14:35     ` Trond Myklebust
2006-08-15  3:49   ` Neil Brown
2006-08-15 15:26     ` Peter Staubach [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=44E1E79D.5080300@redhat.com \
    --to=staubach@redhat.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.