OSDN Git Service

r8169: change irq handler to always trigger NAPI polling
authorHeiner Kallweit <hkallweit1@gmail.com>
Sun, 14 Apr 2019 09:48:39 +0000 (11:48 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 14 Apr 2019 20:58:15 +0000 (13:58 -0700)
This check isn't really needed and we can simplify the code and save
some CPU cycles by removing it. Only in case of an error none of these
bits are set, and calling the NAPI callback doesn't hurt in this case.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/r8169.c

index 68caa26..efaea1a 100644 (file)
@@ -6380,10 +6380,8 @@ static irqreturn_t rtl8169_interrupt(int irq, void *dev_instance)
                set_bit(RTL_FLAG_TASK_RESET_PENDING, tp->wk.flags);
        }
 
-       if (status & (RTL_EVENT_NAPI | LinkChg)) {
-               rtl_irq_disable(tp);
-               napi_schedule_irqoff(&tp->napi);
-       }
+       rtl_irq_disable(tp);
+       napi_schedule_irqoff(&tp->napi);
 out:
        rtl_ack_events(tp, status);