net: vrf: Update flags and features settings
authorDavid Ahern <dsa@cumulusnetworks.com>
Tue, 14 Jun 2016 00:14:12 +0000 (17:14 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 15 Jun 2016 21:03:48 +0000 (14:03 -0700)
1. Default VRF devices to not having a qdisc (IFF_NO_QUEUE). Users
   can add one as desired.

2. Disable adding a VLAN to a VRF device.

3. Enable offloads and hardware features similar to other logical
   devices (e.g., dummy, veth)

Change provides a significant boost in TCP stream Tx performance,
from ~2,700 Mbps to ~18,100 Mbps and makes throughput close to the
performance without a VRF (18,500 Mbps). netperf TCP_STREAM benchmark
using qemu with virtio+vhost for the NICs

Signed-off-by: David Ahern <dsa@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/vrf.c

index 0b5b3c258c2b259e27d1d40221ebc4e9e89cb52c..e3fc6d32a289f0071abaf65ea7a9f377b06afe47 100644 (file)
@@ -1099,6 +1099,20 @@ static void vrf_setup(struct net_device *dev)
 
        /* don't allow vrf devices to change network namespaces. */
        dev->features |= NETIF_F_NETNS_LOCAL;
+
+       /* does not make sense for a VLAN to be added to a vrf device */
+       dev->features   |= NETIF_F_VLAN_CHALLENGED;
+
+       /* enable offload features */
+       dev->features   |= NETIF_F_GSO_SOFTWARE;
+       dev->features   |= NETIF_F_RXCSUM | NETIF_F_HW_CSUM;
+       dev->features   |= NETIF_F_SG | NETIF_F_FRAGLIST | NETIF_F_HIGHDMA;
+
+       dev->hw_features = dev->features;
+       dev->hw_enc_features = dev->features;
+
+       /* default to no qdisc; user can add if desired */
+       dev->priv_flags |= IFF_NO_QUEUE;
 }
 
 static int vrf_validate(struct nlattr *tb[], struct nlattr *data[])