Commit 15e67f90 authored by Alan Stern's avatar Alan Stern Committed by Greg Kroah-Hartman

USB: leave LPM alone if possible when binding/unbinding interface drivers

commit 6fb650d4 upstream.

When a USB driver is bound to an interface (either through probing or
by claiming it) or is unbound from an interface, the USB core always
disables Link Power Management during the transition and then
re-enables it afterward.  The reason is because the driver might want
to prevent hub-initiated link power transitions, in which case the HCD
would have to recalculate the various LPM parameters.  This
recalculation takes place when LPM is re-enabled and the new
parameters are sent to the device and its parent hub.

However, if the driver does not want to prevent hub-initiated link
power transitions then none of this work is necessary.  The parameters
don't need to be recalculated, and LPM doesn't need to be disabled and
re-enabled.

It turns out that disabling and enabling LPM can be time-consuming,
enough so that it interferes with user programs that want to claim and
release interfaces rapidly via usbfs.  Since the usbfs kernel driver
doesn't set the disable_hub_initiated_lpm flag, we can speed things up
and get the user programs to work by leaving LPM alone whenever the
flag isn't set.

And while we're improving the way disable_hub_initiated_lpm gets used,
let's also fix its kerneldoc.
Signed-off-by: default avatarAlan Stern <stern@rowland.harvard.edu>
Tested-by: default avatarMatthew Giassa <matthew@giassa.net>
CC: Mathias Nyman <mathias.nyman@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e3a037a5
...@@ -284,7 +284,7 @@ static int usb_probe_interface(struct device *dev) ...@@ -284,7 +284,7 @@ static int usb_probe_interface(struct device *dev)
struct usb_device *udev = interface_to_usbdev(intf); struct usb_device *udev = interface_to_usbdev(intf);
const struct usb_device_id *id; const struct usb_device_id *id;
int error = -ENODEV; int error = -ENODEV;
int lpm_disable_error; int lpm_disable_error = -ENODEV;
dev_dbg(dev, "%s\n", __func__); dev_dbg(dev, "%s\n", __func__);
...@@ -336,12 +336,14 @@ static int usb_probe_interface(struct device *dev) ...@@ -336,12 +336,14 @@ static int usb_probe_interface(struct device *dev)
* setting during probe, that should also be fine. usb_set_interface() * setting during probe, that should also be fine. usb_set_interface()
* will attempt to disable LPM, and fail if it can't disable it. * will attempt to disable LPM, and fail if it can't disable it.
*/ */
lpm_disable_error = usb_unlocked_disable_lpm(udev); if (driver->disable_hub_initiated_lpm) {
if (lpm_disable_error && driver->disable_hub_initiated_lpm) { lpm_disable_error = usb_unlocked_disable_lpm(udev);
dev_err(&intf->dev, "%s Failed to disable LPM for driver %s\n.", if (lpm_disable_error) {
__func__, driver->name); dev_err(&intf->dev, "%s Failed to disable LPM for driver %s\n.",
error = lpm_disable_error; __func__, driver->name);
goto err; error = lpm_disable_error;
goto err;
}
} }
/* Carry out a deferred switch to altsetting 0 */ /* Carry out a deferred switch to altsetting 0 */
...@@ -391,7 +393,8 @@ static int usb_unbind_interface(struct device *dev) ...@@ -391,7 +393,8 @@ static int usb_unbind_interface(struct device *dev)
struct usb_interface *intf = to_usb_interface(dev); struct usb_interface *intf = to_usb_interface(dev);
struct usb_host_endpoint *ep, **eps = NULL; struct usb_host_endpoint *ep, **eps = NULL;
struct usb_device *udev; struct usb_device *udev;
int i, j, error, r, lpm_disable_error; int i, j, error, r;
int lpm_disable_error = -ENODEV;
intf->condition = USB_INTERFACE_UNBINDING; intf->condition = USB_INTERFACE_UNBINDING;
...@@ -399,12 +402,13 @@ static int usb_unbind_interface(struct device *dev) ...@@ -399,12 +402,13 @@ static int usb_unbind_interface(struct device *dev)
udev = interface_to_usbdev(intf); udev = interface_to_usbdev(intf);
error = usb_autoresume_device(udev); error = usb_autoresume_device(udev);
/* Hub-initiated LPM policy may change, so attempt to disable LPM until /* If hub-initiated LPM policy may change, attempt to disable LPM until
* the driver is unbound. If LPM isn't disabled, that's fine because it * the driver is unbound. If LPM isn't disabled, that's fine because it
* wouldn't be enabled unless all the bound interfaces supported * wouldn't be enabled unless all the bound interfaces supported
* hub-initiated LPM. * hub-initiated LPM.
*/ */
lpm_disable_error = usb_unlocked_disable_lpm(udev); if (driver->disable_hub_initiated_lpm)
lpm_disable_error = usb_unlocked_disable_lpm(udev);
/* /*
* Terminate all URBs for this interface unless the driver * Terminate all URBs for this interface unless the driver
...@@ -505,7 +509,7 @@ int usb_driver_claim_interface(struct usb_driver *driver, ...@@ -505,7 +509,7 @@ int usb_driver_claim_interface(struct usb_driver *driver,
struct device *dev; struct device *dev;
struct usb_device *udev; struct usb_device *udev;
int retval = 0; int retval = 0;
int lpm_disable_error; int lpm_disable_error = -ENODEV;
if (!iface) if (!iface)
return -ENODEV; return -ENODEV;
...@@ -526,12 +530,14 @@ int usb_driver_claim_interface(struct usb_driver *driver, ...@@ -526,12 +530,14 @@ int usb_driver_claim_interface(struct usb_driver *driver,
iface->condition = USB_INTERFACE_BOUND; iface->condition = USB_INTERFACE_BOUND;
/* Disable LPM until this driver is bound. */ /* See the comment about disabling LPM in usb_probe_interface(). */
lpm_disable_error = usb_unlocked_disable_lpm(udev); if (driver->disable_hub_initiated_lpm) {
if (lpm_disable_error && driver->disable_hub_initiated_lpm) { lpm_disable_error = usb_unlocked_disable_lpm(udev);
dev_err(&iface->dev, "%s Failed to disable LPM for driver %s\n.", if (lpm_disable_error) {
__func__, driver->name); dev_err(&iface->dev, "%s Failed to disable LPM for driver %s\n.",
return -ENOMEM; __func__, driver->name);
return -ENOMEM;
}
} }
/* Claimed interfaces are initially inactive (suspended) and /* Claimed interfaces are initially inactive (suspended) and
......
...@@ -1068,7 +1068,7 @@ struct usbdrv_wrap { ...@@ -1068,7 +1068,7 @@ struct usbdrv_wrap {
* for interfaces bound to this driver. * for interfaces bound to this driver.
* @soft_unbind: if set to 1, the USB core will not kill URBs and disable * @soft_unbind: if set to 1, the USB core will not kill URBs and disable
* endpoints before calling the driver's disconnect method. * endpoints before calling the driver's disconnect method.
* @disable_hub_initiated_lpm: if set to 0, the USB core will not allow hubs * @disable_hub_initiated_lpm: if set to 1, the USB core will not allow hubs
* to initiate lower power link state transitions when an idle timeout * to initiate lower power link state transitions when an idle timeout
* occurs. Device-initiated USB 3.0 link PM will still be allowed. * occurs. Device-initiated USB 3.0 link PM will still be allowed.
* *
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment