diff options
| author | Roman Kagan <rkagan@mail.ru> | 2005-04-13 19:40:17 +0200 |
|---|---|---|
| committer | Greg KH <gregkh@suse.de> | 2005-05-05 08:44:38 +0200 |
| commit | b2d84f078a8be40f5ae3b4d2ac001e2a7f45fe4f (patch) | |
| tree | 173f941991f1b68da820e9926a3b7ebdd3a2c8b9 /drivers | |
| parent | [PATCH] Hotplug: Make dev->bus checking consistent (diff) | |
| download | linux-b2d84f078a8be40f5ae3b4d2ac001e2a7f45fe4f.tar.xz linux-b2d84f078a8be40f5ae3b4d2ac001e2a7f45fe4f.zip | |
[PATCH] drivers/base/bus.c: fix iteration in driver_detach()
With 2.6.11 and 2.6.12-rc2 (and perhaps a few versions before) usb
drivers for multi-interface devices, which do
usb_driver_release_interface() in their disconnect(), make rmmod hang.
It turns out to be due to a bug in drivers/base/bus.c:driver_detach(),
that iterates over the list of attached devices with
list_for_each_safe() under an assumption that device_release_driver()
only releases the current device, while it may also call
device_release_driver() for other devices on the same list.
The following patch fixes it. Please consider applying.
Signed-off-by: Roman Kagan <rkagan@mail.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/base/bus.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/base/bus.c b/drivers/base/bus.c index f4fa27315fb4..2b3902c867da 100644 --- a/drivers/base/bus.c +++ b/drivers/base/bus.c @@ -405,9 +405,8 @@ void device_release_driver(struct device * dev) static void driver_detach(struct device_driver * drv) { - struct list_head * entry, * next; - list_for_each_safe(entry, next, &drv->devices) { - struct device * dev = container_of(entry, struct device, driver_list); + while (!list_empty(&drv->devices)) { + struct device * dev = container_of(drv->devices.next, struct device, driver_list); device_release_driver(dev); } } |