Re: [patch 00/13] devtmpfs patches

[Arjan van de Ven]

On Mon, 11 May 2009 03:22:53 +0200
Kay Sievers <kay.sievers@xxxxxxxx> wrote:

> On Mon, May 11, 2009 at 02:00, Arjan van de Ven <arjan@xxxxxxxxxxxxx>
> wrote:
> > (and it seems to be irrelevant to the devtmpfs discussion anyway
> > since Eric Biederman has shown that pulling the device numbers out
> > of sysfs is basically free... even though it'd be nice to give that
> > some help to make it a nice index)
> 
> It's not free, and it's racy to just copy over the stuff. You have to
> subscribe to events _before_ you start reading /sys and create nodes,
> to catch the new stuff that comes in in the meantime. You have to
> mount an empty tmpfs, bring up a udevd-like process to subscribe to
> events for new devices and create nodes for them, create the nodes for
> the already existing devices found in /sys. That's all far from
> "free", and has non-trivial dependencies.
> 

How about we do something that is useful for more scenarios instead,
like proposed already? A /proc or similar file that just lists the
type,major,minor and name for the known nodes.
This has none of the race issues you mention (since udev still later
processes the events) but opens a LOT of flexibility. You could use it
in your set up, but I could use it in the Moblin setup as well.
And I suspect many other setups suddenly become possible, just because
there now is a lost of device nodes that are "active". Debugging
drivers, etc etc. Long list.

I sort of get a bad taste in my mouth from this thread to be honest;
it really feels like "this is the code that HAS to be pushed", rather
than working on getting something useful for a wide set of uses... even
if it's not your initial implementation. 

-- 
Arjan van de Ven 	Intel Open Source Technology Centre
For development, discussion and tips for power savings, 
visit http://www.lesswatts.org
--
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/