On Mon, 2011-09-12 at 13:31 -0700, Benny Halevy wrote: > On 2011-09-12 07:56, Peng Tao wrote: > >> The layout segments are not really in use while in LAYOUTCOMMIT. > >> We only need to get the stateid right with respect to concurrent layout recalls. > > LAYOUTCOMMIT takes lseg reference to mark them as in use so that > > layoutrecall cannot free them. > > > > And if layoutrecall would have freed layout segments during layoutcommit, > what is your specific concern? That layoutcommit is supposed to return NFS4ERR_BAD_LAYOUT in that case according to section 18.42.3 of RFC5661. I can't find anything in the errata that changes that requirement. -- Trond Myklebust Linux NFS client maintainer NetApp Trond.Myklebust@xxxxxxxxxx www.netapp.com -- 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