Re: [Lustre-discuss] question about dcache revalidate

2012-01-17 Thread Lai Siyao
On Tue, Jan 17, 2012 at 5:13 PM, wrote: > > On Jan 12, 2012, at 3:52 AM, Lai Siyao wrote: > > > > > No, to add a dentry to hash client needs holding LOOKUP lock, but > lustre client unhash (see > > ll_unhash_aliases()) doesn't really remove dentry from hash, b

Re: [Lustre-discuss] question about dcache revalidate

2012-01-17 Thread Lai Siyao
> > > On Jan 12, 2012, at 3:52 AM, Lai Siyao wrote: > > > > > No, to add a dentry to hash client needs holding LOOKUP lock, but > lustre client unhash (see > > ll_unhash_aliases()) doesn't really remove dentry from hash, but set > LUSTRE_DCACHE_INVAL

Re: [Lustre-discuss] question about dcache revalidate

2012-01-12 Thread Lai Siyao
No, to add a dentry to hash client needs holding LOOKUP lock, but lustre client unhash (see ll_unhash_aliases()) doesn't really remove dentry from hash, but set LUSTRE_DCACHE_INVALID flag. So in the race you mentioned, another process may add the dentry but later the lock is canceled soon, so at th