summaryrefslogtreecommitdiff
path: root/drivers/usb/host/ohci-hub.c
diff options
context:
space:
mode:
authorAlan Stern <stern@rowland.harvard.edu>2014-07-19 00:26:07 +0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2014-07-19 03:33:01 +0400
commitcdb4dd15e62eb984d9461b520d15d00ff2b88d9d (patch)
tree2015b60c2cd3bece77023c27c2e681cc76814732 /drivers/usb/host/ohci-hub.c
parentc6fcb85ea22889527ee44aba42c3e3b479fd2d92 (diff)
downloadlinux-cdb4dd15e62eb984d9461b520d15d00ff2b88d9d.tar.xz
USB: OHCI: make URB completions single-threaded
URBs for a particular endpoint should complete sequentially. That is, we shouldn't call the completion handler for one URB until the handler for the previous URB has returned. When the OHCI watchdog routine is added, there will be two paths for completing URBs: interrupt handler and watchdog routine. Their activities have to be synchronized so that completions don't occur in multiple threads concurrently. For that purpose, this patch creates an ohci_work() routine which will be responsible for calling process_done_list() and finish_unlinks(), the two routines that detect when an URB is complete. Everything will funnel through ohci_work(), and it will be careful not to run in more than one thread at a time. Signed-off-by: Alan Stern <stern@rowland.harvard.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/host/ohci-hub.c')
-rw-r--r--drivers/usb/host/ohci-hub.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/usb/host/ohci-hub.c b/drivers/usb/host/ohci-hub.c
index dccb90edd66e..8991692bcfb8 100644
--- a/drivers/usb/host/ohci-hub.c
+++ b/drivers/usb/host/ohci-hub.c
@@ -40,8 +40,7 @@
(OHCI_CTRL_CLE|OHCI_CTRL_BLE|OHCI_CTRL_PLE|OHCI_CTRL_IE)
static void update_done_list(struct ohci_hcd *);
-static void process_done_list(struct ohci_hcd *);
-static void finish_unlinks (struct ohci_hcd *, u16);
+static void ohci_work(struct ohci_hcd *);
#ifdef CONFIG_PM
static int ohci_rh_suspend (struct ohci_hcd *ohci, int autostop)
@@ -89,8 +88,7 @@ __acquires(ohci->lock)
spin_lock_irq (&ohci->lock);
}
update_done_list(ohci);
- process_done_list(ohci);
- finish_unlinks (ohci, ohci_frame_no(ohci));
+ ohci_work(ohci);
/*
* Some controllers don't handle "global" suspend properly if