Commit 02e71127 authored by Hauke Mehrtens's avatar Hauke Mehrtens Committed by David S. Miller

bgmac: omit the fcs

Do not include the frame check sequence when adding the skb to
netif_receive_skb(). This causes problems when this interface was
bridged to a wifi ap and a big package should be forwarded from this
Ethernet driver through a bride to the wifi client.
Signed-off-by: default avatarHauke Mehrtens <hauke@hauke-m.de>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent b2a43191
...@@ -301,12 +301,16 @@ static int bgmac_dma_rx_read(struct bgmac *bgmac, struct bgmac_dma_ring *ring, ...@@ -301,12 +301,16 @@ static int bgmac_dma_rx_read(struct bgmac *bgmac, struct bgmac_dma_ring *ring,
bgmac_err(bgmac, "Found poisoned packet at slot %d, DMA issue!\n", bgmac_err(bgmac, "Found poisoned packet at slot %d, DMA issue!\n",
ring->start); ring->start);
} else { } else {
/* Omit CRC. */
len -= ETH_FCS_LEN;
new_skb = netdev_alloc_skb_ip_align(bgmac->net_dev, len); new_skb = netdev_alloc_skb_ip_align(bgmac->net_dev, len);
if (new_skb) { if (new_skb) {
skb_put(new_skb, len); skb_put(new_skb, len);
skb_copy_from_linear_data_offset(skb, BGMAC_RX_FRAME_OFFSET, skb_copy_from_linear_data_offset(skb, BGMAC_RX_FRAME_OFFSET,
new_skb->data, new_skb->data,
len); len);
skb_checksum_none_assert(skb);
new_skb->protocol = new_skb->protocol =
eth_type_trans(new_skb, bgmac->net_dev); eth_type_trans(new_skb, bgmac->net_dev);
netif_receive_skb(new_skb); netif_receive_skb(new_skb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment