Staging: hv: vmbus: Don't wait indefinitely for IRQ resouces
authorK. Y. Srinivasan <kys@microsoft.com>
Fri, 15 Jul 2011 20:38:56 +0000 (13:38 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 23 Aug 2011 23:16:11 +0000 (16:16 -0700)
If an attempt is made to load the vmbus driver on a non-Hyper-V platform,
the load operation will hang since we currently wait indefinitely to
retrieve the IRQ information. This is done in the context of an acpi callback
context (which will obviously not happen when this driver is
loaded on a non-Hyper-V platform). This patch fixes the problem.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/hv/vmbus_drv.c

index 1c949f5..67a4f33 100644 (file)
@@ -768,7 +768,7 @@ MODULE_DEVICE_TABLE(pci, microsoft_hv_pci_table);
 
 static int __init hv_acpi_init(void)
 {
-       int ret;
+       int ret, t;
 
        init_completion(&probe_event);
 
@@ -781,16 +781,25 @@ static int __init hv_acpi_init(void)
        if (ret)
                return ret;
 
-       wait_for_completion(&probe_event);
+       t = wait_for_completion_timeout(&probe_event, 5*HZ);
+       if (t == 0) {
+               ret = -ETIMEDOUT;
+               goto cleanup;
+       }
 
        if (irq <= 0) {
-               acpi_bus_unregister_driver(&vmbus_acpi_driver);
-               return -ENODEV;
+               ret = -ENODEV;
+               goto cleanup;
        }
 
        ret = vmbus_bus_init(irq);
        if (ret)
-               acpi_bus_unregister_driver(&vmbus_acpi_driver);
+               goto cleanup;
+
+       return 0;
+
+cleanup:
+       acpi_bus_unregister_driver(&vmbus_acpi_driver);
        return ret;
 }