Hello, my good man!<br><br>2009/11/20 Oleg Drokin <span dir="ltr"><<a href="mailto:Oleg.Drokin@sun.com">Oleg.Drokin@sun.com</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<br>Yes, I think this does match bug 2969 behavior.<br>
We add entry to dcache without lock (not visible in the trace you provided). Then we do rename, then we do some sort of stat on a renamed<br>
entry and reobtain the lock. Then we do stat on old name, and since lock is on inode - we find the newly reinstantiated lock and declare<br>
old dentry as valid.<br></blockquote><div><br>Hmm.  My first instinct was that there shouldn't be dentries without locks, but it's been sufficiently long that I can't remember all the details of the dentry life cycle.  What you wrote certainly sounds like a plausible explanation.<br>

 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">I wonder if my patch for 20323 would have helped this case<br>
(or have just always returning the lock), though on the other hand this<br>
is inode from mkdir and so might have never go through open path.<br>
Bug 16417 is what landed into 1.8.2 and is a complete rework of<br>
dcache caching logic for dentries and has a better chance of fixing this,<br>
I would say.<br>
If not, it would be great if the lock will start earlier in time, definitely<br>
before rename happens.<br>
<br>
I hope this problem did not ruin your day in the end.<br></blockquote><div><br>No, no, it's fairly minor for us.  I just wanted to report it in case we were the first to experience it.  We will probably upgrade at some point, but we had a less-than-perfect experience with an upgrade this year, so I imagine Stu will wait until it's really necessary.<br>

<br>Thanks for looking into it.  From skimming 16417, I agree that it stands a good chance of fixing it, or at least permutes the system sufficiently that it'd be worth trying to reproduce it again after an upgrade.<br>

<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
And we do miss you. Does your coming with such a question means you are on your way back to us? ;)<br>
</blockquote><div><br>
You all seem to be doing just fine without me. :)  We certainly make intense use of Lustre in our unending quest to find dinosaur blood, and it serves us very well.<br></div><br>Cheers,<br><br>-p<br>
</div></div>