Re: [platform-testers] new snapshot available: coreutils-8.23.237-eff51

[Pádraig Brady]

On 02/07/15 03:38, Pádraig Brady wrote:
> On 02/07/15 02:55, Assaf Gordon wrote:
>> Hello,
>>
>> few results for coreutils-8.23.245-8bf2af  :


>> On OpenSUSE 13.2 x86_64, "tests/tail-2/assert" fail, log attached.
> 
> Interesting. Reproduced 100% of the time here too.
> BTW that VM gives a different version?
>   $ head -n1 /etc/issue
>   Welcome to openSUSE 12.3 ...

It's not 100% reproducible actually, though once it starts happening
it continues to, no matter how many times the file is replaced and removed.
What seems to be happening is that the IN_ATTRIB event is delivered
for the file on unlink() (the link count decreasing is an attribute change).
However the subsequent open() in recheck() succeeds!
I.E. the kernel dentry (cache) is still accessible through that name.
Adding a xnanosleep(.1) before the open() is enough to make the test succeed.

We could avoid this issue by adding IN_DELETE on the parent directory,
though I'm loath to change this at the moment.

cheers,
Pádraig.

opensuse-3.7.10-iniotify-dentry-race.patch
Description: Text Data