From e80446e0d522a97e59dcb86302970526c2391e98 Mon Sep 17 00:00:00 2001 From: Alan Stern Date: Fri, 4 May 2007 11:54:50 -0400 Subject: [PATCH] --- yaml --- r: 59385 b: refs/heads/master c: 06b84e8adcad8280d76a7c71e772c5cddba96d85 h: refs/heads/master i: 59383: 01e43cfe0ab757885964b5817a210622d480e16d v: v3 --- [refs] | 2 +- trunk/drivers/usb/core/hub.c | 48 +----------------------------------- 2 files changed, 2 insertions(+), 48 deletions(-) diff --git a/[refs] b/[refs] index cba20581d26c..f619e0b80c09 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: f3fd77cd2f4499f3e2ef9a1e6d5e4f4349d556c3 +refs/heads/master: 06b84e8adcad8280d76a7c71e772c5cddba96d85 diff --git a/trunk/drivers/usb/core/hub.c b/trunk/drivers/usb/core/hub.c index 702023075310..9464eb504ae6 100644 --- a/trunk/drivers/usb/core/hub.c +++ b/trunk/drivers/usb/core/hub.c @@ -982,49 +982,6 @@ hub_ioctl(struct usb_interface *intf, unsigned int code, void *user_data) } -/* grab device/port lock, returning index of that port (zero based). - * protects the upstream link used by this device from concurrent - * tree operations like suspend, resume, reset, and disconnect, which - * apply to everything downstream of a given port. - */ -static int locktree(struct usb_device *udev) -{ - int t; - struct usb_device *hdev; - - if (!udev) - return -ENODEV; - - /* root hub is always the first lock in the series */ - hdev = udev->parent; - if (!hdev) { - usb_lock_device(udev); - return 0; - } - - /* on the path from root to us, lock everything from - * top down, dropping parent locks when not needed - */ - t = locktree(hdev); - if (t < 0) - return t; - - /* everything is fail-fast once disconnect - * processing starts - */ - if (udev->state == USB_STATE_NOTATTACHED) { - usb_unlock_device(hdev); - return -ENODEV; - } - - /* when everyone grabs locks top->bottom, - * non-overlapping work may be concurrent - */ - usb_lock_device(udev); - usb_unlock_device(hdev); - return udev->portnum; -} - static void recursively_mark_NOTATTACHED(struct usb_device *udev) { int i; @@ -2594,10 +2551,7 @@ static void hub_events(void) /* Lock the device, then check to see if we were * disconnected while waiting for the lock to succeed. */ - if (locktree(hdev) < 0) { - usb_put_intf(intf); - continue; - } + usb_lock_device(hdev); if (hub != usb_get_intfdata(intf)) goto loop;