提交 6b839b6c 编写于 作者: H Heiner Kallweit 提交者: David S. Miller

r8169: fix NAPI handling under high load

rtl_rx() and rtl_tx() are called only if the respective bits are set
in the interrupt status register. Under high load NAPI may not be
able to process all data (work_done == budget) and it will schedule
subsequent calls to the poll callback.
rtl_ack_events() however resets the bits in the interrupt status
register, therefore subsequent calls to rtl8169_poll() won't call
rtl_rx() and rtl_tx() - chip interrupts are still disabled.

Fix this by calling rtl_rx() and rtl_tx() independent of the bits
set in the interrupt status register. Both functions will detect
if there's nothing to do for them.

Fixes: da78dbff ("r8169: remove work from irq handler.")
Signed-off-by: NHeiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: NDavid S. Miller <davem@davemloft.net>
上级 2ee653f6
...@@ -6549,17 +6549,15 @@ static int rtl8169_poll(struct napi_struct *napi, int budget) ...@@ -6549,17 +6549,15 @@ static int rtl8169_poll(struct napi_struct *napi, int budget)
struct rtl8169_private *tp = container_of(napi, struct rtl8169_private, napi); struct rtl8169_private *tp = container_of(napi, struct rtl8169_private, napi);
struct net_device *dev = tp->dev; struct net_device *dev = tp->dev;
u16 enable_mask = RTL_EVENT_NAPI | tp->event_slow; u16 enable_mask = RTL_EVENT_NAPI | tp->event_slow;
int work_done= 0; int work_done;
u16 status; u16 status;
status = rtl_get_events(tp); status = rtl_get_events(tp);
rtl_ack_events(tp, status & ~tp->event_slow); rtl_ack_events(tp, status & ~tp->event_slow);
if (status & RTL_EVENT_NAPI_RX) work_done = rtl_rx(dev, tp, (u32) budget);
work_done = rtl_rx(dev, tp, (u32) budget);
if (status & RTL_EVENT_NAPI_TX) rtl_tx(dev, tp);
rtl_tx(dev, tp);
if (status & tp->event_slow) { if (status & tp->event_slow) {
enable_mask &= ~tp->event_slow; enable_mask &= ~tp->event_slow;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册