[NTOSKRNL]

- Reenable and fix the correct reset code
[HIDCLASS]
- Disable removal of PDOs due to IRP cancellation issues
[USBEHCI][USBOHCI][USBUHCI]
- Disable removal of FDOs due to reinitialization issues

svn path=/trunk/; revision=55827

reactos/drivers/hid/hidclass/pdo.c

@@ -580,7 +580,12 @@ HidClassPDO_PnP(
             //
             // no/op
             //
+#if 0
             Status = STATUS_SUCCESS;
+#else
+            DPRINT1("Denying removal of HID device due to IRP cancellation bugs\n");
+            Status = STATUS_UNSUCCESSFUL;
+#endif
             break;
         }
         default:

reactos/drivers/usb/usbehci/hcd_controller.cpp

@@ -537,6 +537,7 @@ CHCDController::HandlePnp(
         case IRP_MN_QUERY_REMOVE_DEVICE:
         case IRP_MN_QUERY_STOP_DEVICE:
         {
+#if 0
             //
             // sure
             //
@@ -547,6 +548,12 @@ CHCDController::HandlePnp(
             //
             IoSkipCurrentIrpStackLocation(Irp);
             return IoCallDriver(m_NextDeviceObject, Irp);
+#else
+            DPRINT1("Denying controller removal due to reinitialization bugs\n");
+            Irp->IoStatus.Status = STATUS_UNSUCCESSFUL;
+            IoCompleteRequest(Irp, IO_NO_INCREMENT);
+            return STATUS_UNSUCCESSFUL;
+#endif
         }
         case IRP_MN_REMOVE_DEVICE:
         {

reactos/drivers/usb/usbohci/hcd_controller.cpp

@@ -537,6 +537,7 @@ CHCDController::HandlePnp(
         case IRP_MN_QUERY_REMOVE_DEVICE:
         case IRP_MN_QUERY_STOP_DEVICE:
         {
+#if 0
             //
             // sure
             //
@@ -547,6 +548,12 @@ CHCDController::HandlePnp(
             //
             IoSkipCurrentIrpStackLocation(Irp);
             return IoCallDriver(m_NextDeviceObject, Irp);
+#else
+            DPRINT1("Denying controller removal due to reinitialization bugs\n");
+            Irp->IoStatus.Status = STATUS_UNSUCCESSFUL;
+            IoCompleteRequest(Irp, IO_NO_INCREMENT);
+            return STATUS_UNSUCCESSFUL;
+#endif
         }
         case IRP_MN_REMOVE_DEVICE:
         {

reactos/drivers/usb/usbuhci/hcd_controller.cpp

@@ -537,6 +537,7 @@ CHCDController::HandlePnp(
         case IRP_MN_QUERY_REMOVE_DEVICE:
         case IRP_MN_QUERY_STOP_DEVICE:
         {
+#if 0
             //
             // sure
             //
@@ -547,6 +548,12 @@ CHCDController::HandlePnp(
             //
             IoSkipCurrentIrpStackLocation(Irp);
             return IoCallDriver(m_NextDeviceObject, Irp);
+#else
+            DPRINT1("Denying controller removal due to reinitialization bugs\n");
+            Irp->IoStatus.Status = STATUS_UNSUCCESSFUL;
+            IoCompleteRequest(Irp, IO_NO_INCREMENT);
+            return STATUS_UNSUCCESSFUL;
+#endif
         }
         case IRP_MN_REMOVE_DEVICE:
         {

reactos/ntoskrnl/io/pnpmgr/plugplay.c

@@ -555,38 +555,37 @@ IopResetDevice(PPLUGPLAY_CONTROL_RESET_DEVICE_DATA ResetDeviceData)
     ASSERT(DeviceNode->Flags & DNF_ENUMERATED);
     ASSERT(DeviceNode->Flags & DNF_PROCESSED);
 
-#if 0
+    /* Check if there's already a driver loaded for this device */
-    /* Remove the device node */
+    if (DeviceNode->Flags & DNF_ADDED)
-    Status = IopRemoveDevice(DeviceNode);
-    if (NT_SUCCESS(Status))
     {
-        /* Invalidate device relations for the parent to reenumerate the device */
+        /* Remove the device node */
-        Status = IoSynchronousInvalidateDeviceRelations(DeviceNode->Parent->PhysicalDeviceObject, BusRelations);
+        Status = IopRemoveDevice(DeviceNode);
-        DPRINT1("A new driver has been loaded for '%wZ'\n", &DeviceInstance);
+        if (NT_SUCCESS(Status))
-    }
+        {
-#else
+            /* Invalidate device relations for the parent to reenumerate the device */
-    /* FIXME: We might clear some important flags */
+            Status = IoSynchronousInvalidateDeviceRelations(DeviceNode->Parent->PhysicalDeviceObject, BusRelations);
-    DeviceNode->Flags &= ~DNF_DISABLED;
+            DPRINT1("A new driver has been loaded for '%wZ' (FDO above removed)\n", &DeviceNode->InstancePath);
-
+        }
-    /* Load service data from the registry */
+        else
-    Status = IopActionConfigureChildServices(DeviceNode, DeviceNode->Parent);
+        {
-
+            /* A driver has already been loaded for this device */
-    if (NT_SUCCESS(Status))
+            DPRINT1("A reboot is required for the current driver for '%wZ' to be replaced\n", &DeviceNode->InstancePath);
-    {
+        }
-        /* Start the service and begin PnP initialization of the device again */
-        Status = IopActionInitChildServices(DeviceNode, DeviceNode->Parent);
-        DPRINT1("HACK: A new driver has been loaded for '%wZ' WITHOUT removing the old one\n", &DeviceInstance);
-    }
-#endif
-    else if (DeviceNode->Flags & DNF_ADDED)
-    {
-        /* A driver has already been loaded for this device */
-        DPRINT1("A reboot is required for the current driver for '%wZ' to be replaced\n", &DeviceInstance);
     }
     else
     {
-        /* This device needs a driver */
+        /* FIXME: What if the device really is disabled? */
-        DPRINT1("A reboot is required for the new driver for '%wZ' to load\n", &DeviceInstance);
+        DeviceNode->Flags &= ~DNF_DISABLED;
+
+        /* Load service data from the registry */
+        Status = IopActionConfigureChildServices(DeviceNode, DeviceNode->Parent);
+
+        if (NT_SUCCESS(Status))
+        {
+            /* Start the service and begin PnP initialization of the device again */
+            Status = IopActionInitChildServices(DeviceNode, DeviceNode->Parent);
+            DPRINT1("A new driver will be loaded for '%wZ' (no FDO above)\n", &DeviceNode->InstancePath);
+        }
     }
 
     ObDereferenceObject(DeviceObject);