netpoll: netpoll_send_skb() returns transmit status
authorEric Dumazet <edumazet@google.com>
Thu, 7 May 2020 16:32:20 +0000 (09:32 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 8 May 2020 01:11:07 +0000 (18:11 -0700)
Some callers want to know if the packet has been sent or
dropped, to inform upper stacks.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netpoll.h
net/core/netpoll.c

index e466ddf..f47af13 100644 (file)
@@ -63,7 +63,7 @@ int netpoll_setup(struct netpoll *np);
 void __netpoll_cleanup(struct netpoll *np);
 void __netpoll_free(struct netpoll *np);
 void netpoll_cleanup(struct netpoll *np);
-void netpoll_send_skb(struct netpoll *np, struct sk_buff *skb);
+netdev_tx_t netpoll_send_skb(struct netpoll *np, struct sk_buff *skb);
 
 #ifdef CONFIG_NETPOLL
 static inline void *netpoll_poll_lock(struct napi_struct *napi)
index 34cd34f..40d2753 100644 (file)
@@ -305,7 +305,7 @@ static int netpoll_owner_active(struct net_device *dev)
 }
 
 /* call with IRQ disabled */
-static void __netpoll_send_skb(struct netpoll *np, struct sk_buff *skb)
+static netdev_tx_t __netpoll_send_skb(struct netpoll *np, struct sk_buff *skb)
 {
        netdev_tx_t status = NETDEV_TX_BUSY;
        struct net_device *dev;
@@ -320,7 +320,7 @@ static void __netpoll_send_skb(struct netpoll *np, struct sk_buff *skb)
 
        if (!npinfo || !netif_running(dev) || !netif_device_present(dev)) {
                dev_kfree_skb_irq(skb);
-               return;
+               return NET_XMIT_DROP;
        }
 
        /* don't get messages out of order, and no recursion */
@@ -359,15 +359,18 @@ static void __netpoll_send_skb(struct netpoll *np, struct sk_buff *skb)
                skb_queue_tail(&npinfo->txq, skb);
                schedule_delayed_work(&npinfo->tx_work,0);
        }
+       return NETDEV_TX_OK;
 }
 
-void netpoll_send_skb(struct netpoll *np, struct sk_buff *skb)
+netdev_tx_t netpoll_send_skb(struct netpoll *np, struct sk_buff *skb)
 {
        unsigned long flags;
+       netdev_tx_t ret;
 
        local_irq_save(flags);
-       __netpoll_send_skb(np, skb);
+       ret = __netpoll_send_skb(np, skb);
        local_irq_restore(flags);
+       return ret;
 }
 EXPORT_SYMBOL(netpoll_send_skb);