Re: [RFC] [PATCH] driver core: allow userspace to unbind drivers from devices.

Dmitry Torokhov
2004-11-09T23:39:53Z

Hi Greg,

On Tue, 9 Nov 2004 14:37:29 -0800, Greg KH <greg@kroah.com> wrote:
> Ok, everone's been back and forth about the whole bind/unbind stuff
> lately, so let's just do this a step at a time.
>
> How about the following patch. It adds a "unbind" file to any device
> that is bound to a driver. Writing any value to that file disconnects
> the device from the driver associated with it.
>
> It's small, simple, and it works.
>
> It also can cause bad things to happen if you aren't careful about what
> type of device you are unbinding (some i2c chip devices don't really
> unbind from the driver fully, but that's an i2c issue, and I'm working
> on it.)
>
> Also, unbinding a device from a driver can cause the children devices to
> disappear, depending on the type of driver that is bound to the device.

With the present implementation it is pretty much impossible to do
since unbind grabs bus's rwsem. That means that any driver attempting
to remove children will deadlock. Driver core is not aware of evry bus's
topology issues that's why you need a bus method to do proper locking
and children removal.

>
> As an example, a usb-storage device, that has a scsi-host, and scsi
> devices as children. If you unbind the usb-storage device, the
> scsi-host and devices are all removed from the system (as they should
> be.)
>

What about unbinding USB hub driver? It will hang because you can not
remove children on the same bus. In serio the core takes care of removing
any children before unbinding the driver, but again, this is bus-specific
implementation. The bus knows how to handle this.

I also have issue with doing it in steps - it will cause every device have
3 or 4 method-attributes - unbind, bind, rescan, [reconnect]. They all
implement very similar action - control link between device and driver.
I do not see the reason for splitting them apart and it will be a waste
of resources to have all of them as well.

--
Dmitry