Re: [Lustre-discuss] question about dcache revalidate

2012-01-17 Thread tao.peng
Got it. Thanks. Best, Tao From: Lai Siyao [mailto:laisi...@whamcloud.com] Sent: Tuesday, January 17, 2012 5:44 PM To: Peng, Tao Cc: gr...@whamcloud.com; Lustre-discuss@lists.lustre.org Subject: Re: [Lustre-discuss] question about dcache revalidate On Tue, Jan 17, 2012 at 5:13 PM, mailto:tao.p..

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, but set > LUSTRE_DCACHE_INVALID flag.

Re: [Lustre-discuss] question about dcache revalidate

2012-01-17 Thread tao.peng
> 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_INVALID flag. So in > the race you mentioned, another proces

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_INVALID flag. So in > > the race you mentioned, anothe