On Thursday 22 June 2006 1:29 pm, Greg KH wrote:
>
> David, we really should not be caring about what the children of a USB
> device is doing here, as who knows what type of "device" might hang off
> of a struct usb_device.
Should be _only_ interfaces; everything else descends from an interface.
There was previously an invariant that the interfaces were marked
as quiescent unless the interface (a) had a driver, and (b) that
driver was not suspended. Evidently that has been lost. This patch
may be insufficient; ISTR other places relying on that invariant.
And yes, we _should_ care about whether or not any interface is
still active, until the pm core code starts to pay attention to
the driver model tree at all times ... even outside of system-wide
suspend transitions. Today, the pm core code doesn't even use
that tree directly, and all runtime state changes (like selective
suspend with USB) completely bypass that pm tree.
- Dave
> --- gregkh-2.6.orig/drivers/usb/core/usb.c
> +++ gregkh-2.6/drivers/usb/core/usb.c
> @@ -991,6 +991,8 @@ void usb_buffer_unmap_sg (struct usb_dev
>
> static int verify_suspended(struct device *dev, void *unused)
> {
> + if (dev->driver == NULL)
> + return 0;
> return (dev->power.power_state.event == PM_EVENT_ON) ? -EBUSY : 0;
> }
>
>
