net: bcmgenet: add support for xmit_more
authorFlorian Fainelli <f.fainelli@gmail.com>
Fri, 13 Mar 2015 19:11:07 +0000 (12:11 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 13 Mar 2015 19:52:31 +0000 (15:52 -0400)
Delay the update of the TDMA producer index unless this is the last SKB
in a batch, or the queue is already stopped. Move the check for whether
the queue should be stopped before the xmit_more check to avoid locking
the transmit queue in case there was a SKB submitted which has xmit_more
set.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/genet/bcmgenet.c

index 53c916ea06a2a666c81faa3b0f1d91a09b9dea92..e74ae628bbb98ef40779629a66f8fbc6d9706848 100644 (file)
@@ -1316,12 +1316,13 @@ static netdev_tx_t bcmgenet_xmit(struct sk_buff *skb, struct net_device *dev)
        ring->prod_index += nr_frags + 1;
        ring->prod_index &= DMA_P_INDEX_MASK;
 
-       bcmgenet_tdma_ring_writel(priv, ring->index,
-                                 ring->prod_index, TDMA_PROD_INDEX);
-
        if (ring->free_bds <= (MAX_SKB_FRAGS + 1))
                netif_tx_stop_queue(txq);
 
+       if (!skb->xmit_more || netif_xmit_stopped(txq))
+               /* Packets are ready, update producer index */
+               bcmgenet_tdma_ring_writel(priv, ring->index,
+                                         ring->prod_index, TDMA_PROD_INDEX);
 out:
        spin_unlock_irqrestore(&ring->lock, flags);