diff options
| author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-10-29 06:13:39 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-10-30 04:43:30 -0400 |
| commit | 6a32e4f9dd9219261f8856f817e6655114cfec2f (patch) | |
| tree | 964061ded76a2e5c9e2aa9122a3a88956db109c1 /kernel | |
| parent | 14ef37b6d00eb5d06704e45989ba4c21e7be7673 (diff) | |
| download | linux-6a32e4f9dd9219261f8856f817e6655114cfec2f.tar.gz linux-6a32e4f9dd9219261f8856f817e6655114cfec2f.tar.bz2 linux-6a32e4f9dd9219261f8856f817e6655114cfec2f.zip | |
vlan: allow nested vlan_do_receive()
commit 2425717b27eb (net: allow vlan traffic to be received under bond)
broke ARP processing on vlan on top of bonding.
+-------+
eth0 --| bond0 |---bond0.103
eth1 --| |
+-------+
52870.115435: skb_gro_reset_offset <-napi_gro_receive
52870.115435: dev_gro_receive <-napi_gro_receive
52870.115435: napi_skb_finish <-napi_gro_receive
52870.115435: netif_receive_skb <-napi_skb_finish
52870.115435: get_rps_cpu <-netif_receive_skb
52870.115435: __netif_receive_skb <-netif_receive_skb
52870.115436: vlan_do_receive <-__netif_receive_skb
52870.115436: bond_handle_frame <-__netif_receive_skb
52870.115436: vlan_do_receive <-__netif_receive_skb
52870.115436: arp_rcv <-__netif_receive_skb
52870.115436: kfree_skb <-arp_rcv
Packet is dropped in arp_rcv() because its pkt_type was set to
PACKET_OTHERHOST in the first vlan_do_receive() call, since no eth0.103
exists.
We really need to change pkt_type only if no more rx_handler is about to
be called for the packet.
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Reviewed-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'kernel')
0 files changed, 0 insertions, 0 deletions