Messages in this thread Patch in this message | | | Date | Thu, 12 May 2005 16:19:24 +0200 | From | Hannes Reinecke <> | Subject | [PATCH] fix error handling in bus_add_device |
| |
Hi Greg,
this patch fixes the error handling in bus_add_device() and device_attach(). Previously it was 'interesting'. And totally confusing to boot.
Please apply.
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 From: Hannes Reinecke <hare@suse.de> Subject: Fix error handling in bus_add_device()
The error handling in bus_add_device() and device_attach() is simply non-existing. This patch updates both function to align with the default driver core convention to return '0' on success and an error code otherwise. Note that '-ENODEV' is not an error for device_attach and driver_probe_device as it is quite possible that no matching device was found.
Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
diff -pur linux-2.6.12-rc4.orig/drivers/base/bus.c linux-2.6.12-rc4/drivers/base/bus.c --- linux-2.6.12-rc4.orig/drivers/base/bus.c 2005-05-06 23:20:31.000000000 -0600 +++ linux-2.6.12-rc4/drivers/base/bus.c 2005-05-12 08:05:02.000000000 -0600 @@ -312,11 +312,11 @@ int device_attach(struct device * dev) { struct bus_type * bus = dev->bus; struct list_head * entry; - int error; + int error = -ENODEV; if (dev->driver) { device_bind_driver(dev); - return 1; + return 0; } if (bus->match) { @@ -325,7 +325,7 @@ int device_attach(struct device * dev) error = driver_probe_device(drv, dev); if (!error) /* success, driver matched */ - return 1; + return 0; if (error != -ENODEV && error != -ENXIO) /* driver matched but the probe failed */ printk(KERN_WARNING @@ -334,7 +334,7 @@ int device_attach(struct device * dev) } } - return 0; + return error; } @@ -460,11 +460,17 @@ int bus_add_device(struct device * dev) down_write(&dev->bus->subsys.rwsem); pr_debug("bus %s: add device %s\n", bus->name, dev->bus_id); list_add_tail(&dev->bus_list, &dev->bus->devices.list); - device_attach(dev); + error = device_attach(dev); up_write(&dev->bus->subsys.rwsem); - device_add_attrs(bus, dev); - sysfs_create_link(&bus->devices.kobj, &dev->kobj, dev->bus_id); - sysfs_create_link(&dev->kobj, &dev->bus->subsys.kset.kobj, "bus"); + if (!error || error == -ENODEV) + error = device_add_attrs(bus, dev); + if (!error) { + sysfs_create_link(&bus->devices.kobj, &dev->kobj, dev->bus_id); + sysfs_create_link(&dev->kobj, &dev->bus->subsys.kset.kobj, "bus"); + } else { + pr_debug("bus %s: attach device %s failed with %d\n", bus->name, dev->bus_id, error); + put_bus(bus); + } } return error; } | |