tsnep: Fix NAPI polling with budget 0
authorGerhard Engleder <gerhard@engleder-embedded.com>
Fri, 15 Sep 2023 21:01:26 +0000 (23:01 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 18 Sep 2023 09:42:36 +0000 (10:42 +0100)
According to the NAPI documentation networking/napi.rst, Rx specific
APIs like page pool and XDP cannot be used at all when budget is 0.
skb Tx processing should happen regardless of the budget.

Stop NAPI polling after Tx processing and skip Rx processing if budget
is 0.

Signed-off-by: Gerhard Engleder <gerhard@engleder-embedded.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/engleder/tsnep_main.c

index 0cdf0de555ed04024f3f4c819044a55d8c483d1f..8b992dc9bb52b499b5ced0cb24d408e705fb45b0 100644 (file)
@@ -1734,6 +1734,10 @@ static int tsnep_poll(struct napi_struct *napi, int budget)
        if (queue->tx)
                complete = tsnep_tx_poll(queue->tx, budget);
 
+       /* handle case where we are called by netpoll with a budget of 0 */
+       if (unlikely(budget <= 0))
+               return budget;
+
        if (queue->rx) {
                done = queue->rx->xsk_pool ?
                       tsnep_rx_poll_zc(queue->rx, napi, budget) :