Re: [PATCH 0/25] Sysfs cleanups & tagged directory support

From: Tejun Heo
Date: Wed Aug 08 2007 - 04:55:05 EST


Cornelia Huck wrote:
> On Wed, 08 Aug 2007 01:57:07 -0600,
> ebiederm@xxxxxxxxxxxx (Eric W. Biederman) wrote:
>
>>> Got it: It's patch 6, the readdir simplification.
>>>
>>> (The udev on that guest is ancient (063)...)
>> Ok. That is weird.
>
> More weirdness. If I activate another dasd from the repair file
> system, /dev/dasdb is created...
>
> Same if I set the card reader online: /dev/vmrdr-0.0.000c is created as
> expected.
>
>> Does it depend on the order in which the dentries are returned from
>> readdir?
>
> I'd think not.
>
>> Unless I made a really stupid error otherwise the two versions
>> of readdir should have the same semantics.
>
> Yes, your patch looks sane. I have no idea why it breaks stuff...

Are you sure it's patch 6? Patch 17 adds a deadlock in rename path.

--
tejun
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/