提交 4fae696c 编写于 作者: S Simon Horman 提交者: Greg Kroah-Hartman

ravb: expand rx descriptor data to accommodate hw checksum

[ Upstream commit 12da64300fbc76b875900445f4146c3dc617d43e ]

EtherAVB may provide a checksum of packet data appended to packet data. In
order to allow this checksum to be received by the host descriptor data
needs to be enlarged by 2 bytes to accommodate the checksum.

In the case of MTU-sized packets without a VLAN tag the
checksum were already accommodated by virtue of the space reserved for the
VLAN tag. However, a packet of MTU-size with a  VLAN tag consumed all
packet data space provided by a descriptor leaving no space for the
trailing checksum.

This was not detected by the driver which incorrectly used the last two
bytes of packet data as the checksum and truncate the packet by two bytes.
This resulted all such packets being dropped.

A work around is to disable RX checksum offload
 # ethtool -K eth0 rx off

This patch resolves this problem by increasing the size available for
packet data in RX descriptors by two bytes.

Tested on R-Car E3 (r8a77990) ES1.0 based Ebisu-4D board

v2
* Use sizeof(__sum16) directly rather than adding a driver-local
  #define for the size of the checksum provided by the hw (2 bytes).

Fixes: 4d86d381 ("ravb: RX checksum offload")
Signed-off-by: NSimon Horman <horms+renesas@verge.net.au>
Reviewed-by: NSergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: NDavid S. Miller <davem@davemloft.net>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
上级 5f1a18e0
...@@ -344,7 +344,7 @@ static int ravb_ring_init(struct net_device *ndev, int q) ...@@ -344,7 +344,7 @@ static int ravb_ring_init(struct net_device *ndev, int q)
int i; int i;
priv->rx_buf_sz = (ndev->mtu <= 1492 ? PKT_BUF_SZ : ndev->mtu) + priv->rx_buf_sz = (ndev->mtu <= 1492 ? PKT_BUF_SZ : ndev->mtu) +
ETH_HLEN + VLAN_HLEN; ETH_HLEN + VLAN_HLEN + sizeof(__sum16);
/* Allocate RX and TX skb rings */ /* Allocate RX and TX skb rings */
priv->rx_skb[q] = kcalloc(priv->num_rx_ring[q], priv->rx_skb[q] = kcalloc(priv->num_rx_ring[q],
...@@ -525,13 +525,15 @@ static void ravb_rx_csum(struct sk_buff *skb) ...@@ -525,13 +525,15 @@ static void ravb_rx_csum(struct sk_buff *skb)
{ {
u8 *hw_csum; u8 *hw_csum;
/* The hardware checksum is 2 bytes appended to packet data */ /* The hardware checksum is contained in sizeof(__sum16) (2) bytes
if (unlikely(skb->len < 2)) * appended to packet data
*/
if (unlikely(skb->len < sizeof(__sum16)))
return; return;
hw_csum = skb_tail_pointer(skb) - 2; hw_csum = skb_tail_pointer(skb) - sizeof(__sum16);
skb->csum = csum_unfold((__force __sum16)get_unaligned_le16(hw_csum)); skb->csum = csum_unfold((__force __sum16)get_unaligned_le16(hw_csum));
skb->ip_summed = CHECKSUM_COMPLETE; skb->ip_summed = CHECKSUM_COMPLETE;
skb_trim(skb, skb->len - 2); skb_trim(skb, skb->len - sizeof(__sum16));
} }
/* Packet receive function for Ethernet AVB */ /* Packet receive function for Ethernet AVB */
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册