first plugged USB device is sometimes not recognized

Bug #8689 reported by Martin Pitt
6
Affects Status Importance Assigned to Milestone
hal (Ubuntu)
Fix Released
Medium
Martin Pitt

Bug Description

Directly after a hal restart, the very first hotplug event is sometimes not
detected correctly. Subsequent hotplugs (usually) work.

Revision history for this message
Matt Zimmerman (mdz) wrote :

You mentioned that this was apparently fixed in hal CVS, right?

Revision history for this message
Martin Pitt (pitti) wrote :

No, unfortunately it is not fixed in CVS. It worked, but that was just bad/good
luck (it's a race condition).

There is a fix at
http://freedesktop.org/pipermail/hal/2004-September/001093.html, but I would
like to hear upstream's opinion about this before I upload it.

Revision history for this message
Martin Pitt (pitti) wrote :

Created an attachment (id=330)
patch to fix this bug

This is taken from
http://freedesktop.org/pipermail/hal/2004-September/001093.html.

It is still unconfirmed by upstream, but I tested this a dozen times, it works
great for me. So I propose to upload it to receive wider testing; reverting the
patch is no problem, if it causes trouble.

I will upload a new hal package together with a few other fixes, but want each
patch approved separately.

Revision history for this message
Martin Pitt (pitti) wrote :

Fixed in:
 hal (0.2.98-1ubuntu5) warty; urgency=low
 .
   * added patch nofail_nocaps: do not exit hald if capabilities cannot be
     installed (which happens on kernels which do not support capabilities),
     since only few features actually depend on additional capabilities
     (currently only the "link" detection of MII ethernet cards)
     (Warty bug #8721)
   * added patch fix_first_hotplug: the first hotplug event was sometimes not
     recognized properly, this patch should fix that. Thanks to Sjoerd Simons
     for finding it. (Warty bug #8689)
   * added patch add_device_nullarg: check whether given_sysfs_path is NULL and
     immediately return in this case; previously, this parameter was copied and
     compared without checking. (Warty bug #8707)

This patch was reviewed by Thom, and tested by Thom, Sjoerd Simons, Alexander
Poslavsky, and myself and worked fine, so I uploaded it according to Matt's new
'Peer review' rules.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.