]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
scsi: megaraid-sas: request irqs later
authorTomas Henzl <thenzl@redhat.com>
Tue, 1 Nov 2016 16:32:02 +0000 (17:32 +0100)
committerChuck Anderson <chuck.anderson@oracle.com>
Tue, 28 Feb 2017 04:40:11 +0000 (20:40 -0800)
Orabug: 25568930

It is not good when an irq arrives before driver structures are
allocated.

Signed-off-by: Tomas Henzl <thenzl@redhat.com>
Acked-by: Sumit Saxena <sumit.saxena@broadcom.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
(cherry picked from commit 18103efcacee0563d57c3b7af8d849faae62a117)
Signed-off-by: Brian Maly <brian.maly@oracle.com>
drivers/scsi/megaraid/megaraid_sas_base.c

index 09e89b8ab3950c0185e41da93bbe6d7d7b567263..b1c48e59854fbe75e31f47412b17640d707e3bb3 100644 (file)
@@ -5150,11 +5150,6 @@ static int megasas_init_fw(struct megasas_instance *instance)
        tasklet_init(&instance->isr_tasklet, instance->instancet->tasklet,
                (unsigned long)instance);
 
-       if (instance->msix_vectors ?
-               megasas_setup_irqs_msix(instance, 1) :
-               megasas_setup_irqs_ioapic(instance))
-               goto fail_setup_irqs;
-
        instance->ctrl_info = kzalloc(sizeof(struct megasas_ctrl_info),
                                GFP_KERNEL);
        if (instance->ctrl_info == NULL)
@@ -5170,6 +5165,10 @@ static int megasas_init_fw(struct megasas_instance *instance)
        if (instance->instancet->init_adapter(instance))
                goto fail_init_adapter;
 
+       if (instance->msix_vectors ?
+               megasas_setup_irqs_msix(instance, 1) :
+               megasas_setup_irqs_ioapic(instance))
+               goto fail_init_adapter;
 
        instance->instancet->enable_intr(instance);
 
@@ -5309,9 +5308,8 @@ static int megasas_init_fw(struct megasas_instance *instance)
 
 fail_get_pd_list:
        instance->instancet->disable_intr(instance);
-fail_init_adapter:
        megasas_destroy_irqs(instance);
-fail_setup_irqs:
+fail_init_adapter:
        if (instance->msix_vectors)
                pci_disable_msix(instance->pdev);
        instance->msix_vectors = 0;