From 1cdf0e1d0db332dbdbf9a294060f62d6241b9007 Mon Sep 17 00:00:00 2001
From: Johannes Anderwald <johannes.anderwald@reactos.org>
Date: Mon, 23 Jan 2012 04:22:11 +0000
Subject: [PATCH] [USBEHCI] - Don't complete irps with lock held

svn path=/branches/usb-bringup-trunk/; revision=55091
---
 drivers/usb/usbehci_new/usb_queue.cpp | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/drivers/usb/usbehci_new/usb_queue.cpp b/drivers/usb/usbehci_new/usb_queue.cpp
index a7936b2e714..ed605d1e540 100644
--- a/drivers/usb/usbehci_new/usb_queue.cpp
+++ b/drivers/usb/usbehci_new/usb_queue.cpp
@@ -695,6 +695,7 @@ CUSBQueue::QueueHeadCleanup(
     IUSBRequest * Request;
     BOOLEAN ShouldReleaseWhenDone;
     USBD_STATUS UrbStatus;
+    KIRQL OldLevel;
 
     //
     // sanity checks
@@ -763,11 +764,21 @@ CUSBQueue::QueueHeadCleanup(
         //
         if (Request->GetQueueHead(&NewQueueHead) == STATUS_SUCCESS)
         {
+            //
+            // first acquire request lock
+            //
+            KeAcquireSpinLock(&m_Lock, &OldLevel);
+
             //
             // add to pending list
             //
             InsertTailList(&m_PendingRequestAsyncList, &NewQueueHead->LinkedQueueHeads);
 
+            //
+            // release queue head
+            //
+            KeReleaseSpinLock(&m_Lock, OldLevel);
+
             //
             // Done for now
             //
@@ -859,10 +870,20 @@ CUSBQueue::CompleteAsyncRequests()
         //
         Request = (IUSBRequest*) CurrentQH->Request;
 
+        //
+        // release lock
+        //
+        KeReleaseSpinLock(&m_Lock, OldLevel);
+
         //
         // complete request now
         //
         QueueHeadCleanup(CurrentQH);
+
+        //
+        // first acquire request lock
+        //
+        KeAcquireSpinLock(&m_Lock, &OldLevel);
     }
 
     //