From: Kay Sievers <kay.sievers@vrfy.org>
To: linux-hotplug@vger.kernel.org
Subject: Re: udevd: serialize device chain event sequence
Date: Thu, 09 Dec 2004 08:18:28 +0000 [thread overview]
Message-ID: <1102580308.10393.14.camel@localhost.localdomain> (raw)
In-Reply-To: <20041208231335.GA6402@vrfy.org>
On Wed, 2004-12-08 at 19:07 -0800, David Brownell wrote:
> On Wednesday 08 December 2004 3:13 pm, Kay Sievers wrote:
> > Any objections against the serialization of the event sequence of a
> > chain of devices. Currently udevd delays only events for the same
> > DEVPATH.
>
> I'm amused ... that's what the original hotplug code achieved.
> Thing is, now serialization would be done in userspace, rather
> than offering new and fun paths to kernel deadlock nirvana! :)
Yeah, the same applies to current userspace code, like the one we have
in HAL :)
> > Example of an "add" event sequence:
> > /block/sda
> > /block/sda/sda1
> >
> > With this change, we make sure, that the udev process handling /block/sda
> > has finished its work (waited for all attributes, created the node) before
> > we fork the udev event for /block/sda/sda1. This way the event for sda1 can
> > be sure, that the node for the main device is already created (may be
> > useful for disk labels).
>
> But the "add" for the underlying hardware (maybe USB) would
> not be serialized.
Right, just the stream of events for the /sys/devices/-device is
serialized, but in another event sequence. The class/block device
carries from kernel 2.6.10+ on the PHYSDEVPATH variable in the
environment, which makes it easy to know what to wait for.
> > The main motivation to do this is the program execution of the dev.d/
> > and hotplug.d/ directory. If we don't wait for the parent event to exit,
> > we can't be sure that the executed scripts are run in the right order.
>
> Could that argument apply to the underlying hardware, too?
Good question. I never thought about that until you asked. :) It sounds
nice to let udevd serialize the complete event stream, based on
PHYSDEVPATH. I will try it today and come back...
Thanks,
Kay
-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://productguide.itmanagersjournal.com/
_______________________________________________
Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel
next prev parent reply other threads:[~2004-12-09 8:18 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-12-08 23:13 udevd: serialize device chain event sequence Kay Sievers
2004-12-09 3:07 ` David Brownell
2004-12-09 8:18 ` Kay Sievers [this message]
2004-12-09 10:06 ` Kay Sievers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1102580308.10393.14.camel@localhost.localdomain \
--to=kay.sievers@vrfy.org \
--cc=linux-hotplug@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.