veth: implement ndo_set_rx_headroom
authorPaolo Abeni <pabeni@redhat.com>
Fri, 26 Feb 2016 09:45:41 +0000 (10:45 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 Mar 2016 20:54:31 +0000 (15:54 -0500)
The rx headroom for veth dev is the peer device needed_headroom.
Avoid ping-pong updates setting the private flag IFF_PHONY_HEADROOM.

This avoids skb head reallocation when forwarding from a veth dev
towards a device adding some kind of encapsulation.

When transmitting frames below the MTU size towards a vxlan device,
this gives about 10% performance speed-up when OVS is used to connect
the veth and the vxlan device and a little more when using a
plain Linux bridge.

Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/veth.c

index ba21d072be31c95827833f9e5014f20cfa43bb8b..4f30a6ae50d029f8ce0bb8105490b13489a5f090 100644 (file)
@@ -35,6 +35,7 @@ struct pcpu_vstats {
 struct veth_priv {
        struct net_device __rcu *peer;
        atomic64_t              dropped;
+       unsigned                requested_headroom;
 };
 
 /*
@@ -271,6 +272,29 @@ static int veth_get_iflink(const struct net_device *dev)
        return iflink;
 }
 
+static void veth_set_rx_headroom(struct net_device *dev, int new_hr)
+{
+       struct veth_priv *peer_priv, *priv = netdev_priv(dev);
+       struct net_device *peer;
+
+       if (new_hr < 0)
+               new_hr = 0;
+
+       rcu_read_lock();
+       peer = rcu_dereference(priv->peer);
+       if (unlikely(!peer))
+               goto out;
+
+       peer_priv = netdev_priv(peer);
+       priv->requested_headroom = new_hr;
+       new_hr = max(priv->requested_headroom, peer_priv->requested_headroom);
+       dev->needed_headroom = new_hr;
+       peer->needed_headroom = new_hr;
+
+out:
+       rcu_read_unlock();
+}
+
 static const struct net_device_ops veth_netdev_ops = {
        .ndo_init            = veth_dev_init,
        .ndo_open            = veth_open,
@@ -285,6 +309,7 @@ static const struct net_device_ops veth_netdev_ops = {
 #endif
        .ndo_get_iflink         = veth_get_iflink,
        .ndo_features_check     = passthru_features_check,
+       .ndo_set_rx_headroom    = veth_set_rx_headroom,
 };
 
 #define VETH_FEATURES (NETIF_F_SG | NETIF_F_FRAGLIST | NETIF_F_ALL_TSO |    \
@@ -301,6 +326,7 @@ static void veth_setup(struct net_device *dev)
        dev->priv_flags &= ~IFF_TX_SKB_SHARING;
        dev->priv_flags |= IFF_LIVE_ADDR_CHANGE;
        dev->priv_flags |= IFF_NO_QUEUE;
+       dev->priv_flags |= IFF_PHONY_HEADROOM;
 
        dev->netdev_ops = &veth_netdev_ops;
        dev->ethtool_ops = &veth_ethtool_ops;