On Fri, Jan 29, 2010 at 04:39:11PM -0500, Trond Myklebust wrote: > On Fri, 2010-01-29 at 16:27 -0500, Christoph Hellwig wrote: > > And not actually touched by your patch, but that is the reason to > > open/close the file if we don't actually do anything with it for an > > async export? > > I must admit that I was wondering about that too. I'm assuming that the > reason is to provide consistent behaviour w.r.t. access checks and > possibly also to ensure that NFSv4 delegations are revoked. Perhaps > Bruce could comment? Do delegations need to be revoked on commit? (And do we care about access checks?) We could do both without the need for an actual open, but I don't know that it matters much either way. --b. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
