• Eric Dumazet's avatar
    net: do not abort bulk send on BQL status · fe60faa5
    Eric Dumazet authored
    Before calling dev_hard_start_xmit(), upper layers tried
    to cook optimal skb list based on BQL budget.
    
    Problem is that GSO packets can end up comsuming more than
    the BQL budget.
    
    Breaking the loop is not useful, since requeued packets
    are ahead of any packets still in the qdisc.
    
    It is also more expensive, since next TX completion will
    push these packets later, while skbs are not in cpu caches.
    
    It is also a behavior difference with TSO packets, that can
    break the BQL limit by a large amount.
    
    Note that drivers should use __netdev_tx_sent_queue()
    in order to have optimal xmit_more support, and avoid
    useless atomic operations as shown in the following patch.
    Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    fe60faa5
dev.c 242 KB