amd-xgbe: Associate Tx SKB with proper ring descriptor
authorLendacky, Thomas <Thomas.Lendacky@amd.com>
Wed, 3 Dec 2014 00:16:54 +0000 (18:16 -0600)
committerDavid S. Miller <davem@davemloft.net>
Tue, 9 Dec 2014 18:15:42 +0000 (13:15 -0500)
The SKB for a Tx packet is associated with an xgbe_ring_data structure
in the xgbe_map_tx_skb function.  However, it is being saved in the
structure after the last structure used when the SKB is mapped.  Use
the last used structure to save the SKB value.

Signed-off-by: Tom Lendacky <thomas.lendacky@amd.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/amd/xgbe/xgbe-desc.c

index 43b7d2e948f77763151232798dca5c7ab7265f0d..b15551bad7fa7e67e68db33ff6d0ce6fb56b3bed 100644 (file)
@@ -480,7 +480,11 @@ static int xgbe_map_tx_skb(struct xgbe_channel *channel, struct sk_buff *skb)
                }
        }
 
-       /* Save the skb address in the last entry */
+       /* Save the skb address in the last entry. We always have some data
+        * that has been mapped so rdata is always advanced past the last
+        * piece of mapped data - use the entry pointed to by cur_index - 1.
+        */
+       rdata = XGBE_GET_DESC_DATA(ring, cur_index - 1);
        rdata->skb = skb;
 
        /* Save the number of descriptor entries used */