diff options
| author |   Heiner Kallweit <hkallweit1@gmail.com> | 2019-04-14 11:48:39 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-04-14 13:58:15 -0700 |
| commit | e62b2fd5d3b4c5c958cf88b92f31960750d88dc5 (patch) | |
| tree | 611d0bc630c5b39ee6378d8303c752a2890befee /drivers/net/ethernet/realtek | |
| parent | Merge branch 'r8169-phy-func-ptr-arrays' (diff) | |
| download | linux-dev-e62b2fd5d3b4c5c958cf88b92f31960750d88dc5.tar.xz linux-dev-e62b2fd5d3b4c5c958cf88b92f31960750d88dc5.zip | |
r8169: change irq handler to always trigger NAPI polling
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>
Diffstat (limited to 'drivers/net/ethernet/realtek')
| -rw-r--r-- | drivers/net/ethernet/realtek/r8169.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/ethernet/realtek/r8169.c b/drivers/net/ethernet/realtek/r8169.c index 68caa26f0c3e..efaea1a0ad64 100644 --- a/drivers/net/ethernet/realtek/r8169.c +++ b/drivers/net/ethernet/realtek/r8169.c @@ -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); |