linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hannes Reinecke <hare@suse.de>
To: linux-hotplug@vger.kernel.org
Subject: Firmware class breaks udev
Date: Tue, 15 Mar 2005 08:25:10 +0000	[thread overview]
Message-ID: <42369BE6.7020807@suse.de> (raw)

Hi Kay,

The current implementation of the firmware class breaks a fundamental
assumption in udev: that the physical device can be initialised fully
prior to executing the next event for that device.

As the firmware is indeed _required_ to initialise the device we wait
until the timeout is triggered and the try to execute the fimware event,
which of course will fail.

If you have a look at the firmware class you'll see exactly what happens:
The class insists on sending their own firmware events during
initialisation. This code is typically executed during device probing,
so the modprobe for this device will still be running when the firmware
event is triggered.

Which is also why you're seeing this only when using the events
themselves, as when executing modprobe directly udev is only started for
 the firmware event, which will succeed as no physical device event is
in the queue.

For now I'll be putting in a quick exit for firmware events (ie not wait
for the devices initialisation to finish) but this is nevertheless _nasty_.

Currently the firmware class is definitely not compliant to the driver
model. So either we should fix the firmware class or extend the driver
model to allow for such beasts; Kay, your kobj hotplug extension might
be a way to go.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke			hare@suse.de
SuSE Linux AG				S390 & zSeries
Maxfeldstraße 5				+49 911 74053 688
90409 Nürnberg				http://www.suse.de


-------------------------------------------------------
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://ads.osdn.com/?ad_ide95&alloc_id\x14396&opÌk
_______________________________________________
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

             reply	other threads:[~2005-03-15  8:25 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-15  8:25 Hannes Reinecke [this message]
2005-03-15 12:17 ` Firmware class breaks udev Kay Sievers
2005-03-15 13:09 ` Hannes Reinecke
2005-03-15 15:06 ` Kay Sievers
2005-03-15 15:20 ` Jon Smirl
2005-03-15 16:07 ` Hannes Reinecke
2005-03-15 16:20 ` Greg KH
2005-03-16  7:27 ` Hannes Reinecke
2005-03-16 15:52 ` Jon Smirl
2005-03-16 20:25 ` Kay Sievers
2005-03-17  6:01 ` Greg KH
2005-03-17  6:03 ` Greg KH

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=42369BE6.7020807@suse.de \
    --to=hare@suse.de \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).