e1000e: use disable_hardirq() also for MSIX vectors in e1000_netpoll()
authorKonstantin Khlebnikov <khlebnikov@yandex-team.ru>
Fri, 19 May 2017 07:18:49 +0000 (10:18 +0300)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Tue, 6 Jun 2017 08:06:06 +0000 (01:06 -0700)
Replace disable_irq() which waits for threaded irq handlers with
disable_hardirq() which waits only for hardirq part.

Fixes: 311191297125 ("e1000: use disable_hardirq() for e1000_netpoll()")
Signed-off-by: Konstantin Khlebnikov <khlebnikov@yandex-team.ru>
Acked-by: Cong Wang <xiyou.wangcong@gmail.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/e1000e/netdev.c

index fc1d92ca3ea257c27afed1fc2d32a45bf5485bc5..e1d46c11cb61b1920fac6287d2eda6f115715bf3 100644 (file)
@@ -6743,20 +6743,20 @@ static irqreturn_t e1000_intr_msix(int __always_unused irq, void *data)
 
                vector = 0;
                msix_irq = adapter->msix_entries[vector].vector;
-               disable_irq(msix_irq);
-               e1000_intr_msix_rx(msix_irq, netdev);
+               if (disable_hardirq(msix_irq))
+                       e1000_intr_msix_rx(msix_irq, netdev);
                enable_irq(msix_irq);
 
                vector++;
                msix_irq = adapter->msix_entries[vector].vector;
-               disable_irq(msix_irq);
-               e1000_intr_msix_tx(msix_irq, netdev);
+               if (disable_hardirq(msix_irq))
+                       e1000_intr_msix_tx(msix_irq, netdev);
                enable_irq(msix_irq);
 
                vector++;
                msix_irq = adapter->msix_entries[vector].vector;
-               disable_irq(msix_irq);
-               e1000_msix_other(msix_irq, netdev);
+               if (disable_hardirq(msix_irq))
+                       e1000_msix_other(msix_irq, netdev);
                enable_irq(msix_irq);
        }