Commit 56fd2b2c authored by Alexander Duyck's avatar Alexander Duyck Committed by David S. Miller

macvlan/macvtap: Add support for L2 forwarding offloads with macvtap

This patch reverts earlier commit b13ba1b8 ("macvlan: forbid L2
fowarding offload for macvtap"). The reason for reverting this is because
the original patch no longer fixes what it previously did as the
underlying structure has changed for macvtap. Specifically macvtap
originally pulled packets directly off of the lowerdev. However in commit
6acf54f1 ("macvtap: Add support of packet capture on macvtap device.")
that code was changed and instead macvtap would listen directly on the
macvtap device itself instead of the lower device. As such, the L2
forwarding offload should now be able to provide a performance advantage of
skipping the checks on the lower dev while not introducing any sort of
regression.
Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 8f2e9ca8
...@@ -598,8 +598,6 @@ static const struct header_ops macvlan_hard_header_ops = { ...@@ -598,8 +598,6 @@ static const struct header_ops macvlan_hard_header_ops = {
.cache_update = eth_header_cache_update, .cache_update = eth_header_cache_update,
}; };
static struct rtnl_link_ops macvlan_link_ops;
static int macvlan_open(struct net_device *dev) static int macvlan_open(struct net_device *dev)
{ {
struct macvlan_dev *vlan = netdev_priv(dev); struct macvlan_dev *vlan = netdev_priv(dev);
...@@ -615,8 +613,7 @@ static int macvlan_open(struct net_device *dev) ...@@ -615,8 +613,7 @@ static int macvlan_open(struct net_device *dev)
goto hash_add; goto hash_add;
} }
if (lowerdev->features & NETIF_F_HW_L2FW_DOFFLOAD && if (lowerdev->features & NETIF_F_HW_L2FW_DOFFLOAD) {
dev->rtnl_link_ops == &macvlan_link_ops) {
vlan->fwd_priv = vlan->fwd_priv =
lowerdev->netdev_ops->ndo_dfwd_add_station(lowerdev, dev); lowerdev->netdev_ops->ndo_dfwd_add_station(lowerdev, dev);
......
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