diff options
| author | David S. Miller <davem@davemloft.net> | 2020-05-08 04:11:07 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-05-08 04:11:07 +0300 |
| commit | 738fea32af86f5d58f30dfca6645494070c976ef (patch) | |
| tree | 5d1540325ac32853f85625cdb0262934f8244dbc /net/dsa/slave.c | |
| parent | 3a13f98b4c16fb3489bdfd7550fcaa333ee69850 (diff) | |
| parent | ae46f184bc1fb15bf2de47114c29236e61ca4bbc (diff) | |
| download | linux-738fea32af86f5d58f30dfca6645494070c976ef.tar.xz | |
Merge branch 'bonding-report-transmit-status-to-callers'
Eric Dumazet says:
====================
bonding: report transmit status to callers
First patches cleanup netpoll, and make sure it provides tx status to its users.
Last patch changes bonding to not pretend packets were sent without error.
By providing more accurate status, TCP stack can avoid adding more
packets if the slave qdisc is already full.
This came while testing latest horizon feature in sch_fq, with
very low pacing rate flows, but should benefit hosts under stress.
====================
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dsa/slave.c')
| -rw-r--r-- | net/dsa/slave.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/net/dsa/slave.c b/net/dsa/slave.c index dfb4282fc339..61b0de52040a 100644 --- a/net/dsa/slave.c +++ b/net/dsa/slave.c @@ -445,12 +445,11 @@ static inline netdev_tx_t dsa_slave_netpoll_send_skb(struct net_device *dev, #ifdef CONFIG_NET_POLL_CONTROLLER struct dsa_slave_priv *p = netdev_priv(dev); - if (p->netpoll) - netpoll_send_skb(p->netpoll, skb); + return netpoll_send_skb(p->netpoll, skb); #else BUG(); -#endif return NETDEV_TX_OK; +#endif } static void dsa_skb_tx_timestamp(struct dsa_slave_priv *p, |