diff options
| author | Michael Brown | 2015-02-12 01:46:22 +0100 |
|---|---|---|
| committer | Michael Brown | 2015-02-12 01:57:15 +0100 |
| commit | 6fe8f80418c71b4d05b7de362db9f142690d1a57 (patch) | |
| tree | b5ca7b9246439cf44c0ff27fb55317008b6e5c67 /src/Makefile.housekeeping | |
| parent | [rndis] Add rndis_rx_err() (diff) | |
| download | ipxe-6fe8f80418c71b4d05b7de362db9f142690d1a57.tar.gz ipxe-6fe8f80418c71b4d05b7de362db9f142690d1a57.tar.xz ipxe-6fe8f80418c71b4d05b7de362db9f142690d1a57.zip | |
[usb] Handle port status changes received after failing to find a driver
Commit a60f2dd ("[usb] Try multiple USB device configurations")
changed the behaviour of register_usb() such that if no drivers are
found then the device will be closed and the memory used will be
freed.
If a port status change subsequently occurs while the device is still
physically attached, then usb_hotplug() will see this as a new device
having been attached, since there is no device recorded as being
currently attached to the port. This can lead to spurious hotplug
events (or even endless loops of hotplug events, if the process of
opening and closing the device happens to generate a port status
change).
Fix by using a separate flag to indicate that a device is physically
attached (even if we have no corresponding struct usb_device).
Reported-by: Dan Ellis <Dan.Ellis@displaylink.com>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
Diffstat (limited to 'src/Makefile.housekeeping')
0 files changed, 0 insertions, 0 deletions