Skip to content

Commit

Permalink
ibmveth: Fix use of ibmveth in a bridge.
Browse files Browse the repository at this point in the history
[ Upstream commit 2ac8af0 ]

The check for src mac address in ibmveth_is_packet_unsupported is wrong.
Commit 6f22754 wanted to shut down messages for loopback packets,
but now suppresses bridged frames, which are accepted by the hypervisor
otherwise bridging won't work at all.

Fixes: 6f22754 ("ibmveth: Detect unsupported packets before sending to the hypervisor")
Signed-off-by: Michal Suchanek <msuchanek@suse.de>
Signed-off-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>
Link: https://lore.kernel.org/r/20201026104221.26570-1-msuchanek@suse.de
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Thomas Bogendoerfer authored and gregkh committed Nov 1, 2020
1 parent d0cf5ae commit fa14e0a
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions drivers/net/ethernet/ibm/ibmveth.c
Expand Up @@ -1031,12 +1031,6 @@ static int ibmveth_is_packet_unsupported(struct sk_buff *skb,
ret = -EOPNOTSUPP;
}

if (!ether_addr_equal(ether_header->h_source, netdev->dev_addr)) {
netdev_dbg(netdev, "source packet MAC address does not match veth device's, dropping packet.\n");
netdev->stats.tx_dropped++;
ret = -EOPNOTSUPP;
}

return ret;
}

Expand Down

0 comments on commit fa14e0a

Please sign in to comment.