Vladislav Yasevich
2017-May-18 13:31 UTC
[PATCH net 3/3] virtio-net: enable TSO/checksum offloads for Q-in-Q vlans
Since virtio does not provide it's own ndo_features_check handler, TSO, and now checksum offload, are disabled for stacked vlans. Re-enable the support and let the host take care of it. This restores/improves Guest-to-Guest performance over Q-in-Q vlans. CC: "Michael S. Tsirkin" <mst at redhat.com> CC: Jason Wang <jasowang at redhat.com> CC: virtualization at lists.linux-foundation.org Signed-off-by: Vladislav Yasevich <vyasevic at redhat.com> --- drivers/net/virtio_net.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index 8324a5e..341fb96 100644 --- a/drivers/net/virtio_net.c +++ b/drivers/net/virtio_net.c @@ -2028,6 +2028,7 @@ static const struct net_device_ops virtnet_netdev = { .ndo_poll_controller = virtnet_netpoll, #endif .ndo_xdp = virtnet_xdp, + .ndo_features_check = passthru_features_check, }; static void virtnet_config_changed_work(struct work_struct *work) -- 2.7.4
Michael S. Tsirkin
2017-May-18 15:06 UTC
[PATCH net 3/3] virtio-net: enable TSO/checksum offloads for Q-in-Q vlans
On Thu, May 18, 2017 at 09:31:05AM -0400, Vladislav Yasevich wrote:> Since virtio does not provide it's own ndo_features_check handler, > TSO, and now checksum offload, are disabled for stacked vlans. > Re-enable the support and let the host take care of it. This > restores/improves Guest-to-Guest performance over Q-in-Q vlans. > > CC: "Michael S. Tsirkin" <mst at redhat.com> > CC: Jason Wang <jasowang at redhat.com> > CC: virtualization at lists.linux-foundation.org > Signed-off-by: Vladislav Yasevich <vyasevic at redhat.com>Acked-by: Michael S. Tsirkin <mst at redhat.com>> --- > drivers/net/virtio_net.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c > index 8324a5e..341fb96 100644 > --- a/drivers/net/virtio_net.c > +++ b/drivers/net/virtio_net.c > @@ -2028,6 +2028,7 @@ static const struct net_device_ops virtnet_netdev = { > .ndo_poll_controller = virtnet_netpoll, > #endif > .ndo_xdp = virtnet_xdp, > + .ndo_features_check = passthru_features_check, > }; > > static void virtnet_config_changed_work(struct work_struct *work) > -- > 2.7.4
Jason Wang
2017-May-19 14:18 UTC
[PATCH net 3/3] virtio-net: enable TSO/checksum offloads for Q-in-Q vlans
On 2017?05?18? 21:31, Vladislav Yasevich wrote:> Since virtio does not provide it's own ndo_features_check handler, > TSO, and now checksum offload, are disabled for stacked vlans. > Re-enable the support and let the host take care of it. This > restores/improves Guest-to-Guest performance over Q-in-Q vlans. > > CC: "Michael S. Tsirkin" <mst at redhat.com> > CC: Jason Wang <jasowang at redhat.com> > CC: virtualization at lists.linux-foundation.org > Signed-off-by: Vladislav Yasevich <vyasevic at redhat.com> > --- > drivers/net/virtio_net.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c > index 8324a5e..341fb96 100644 > --- a/drivers/net/virtio_net.c > +++ b/drivers/net/virtio_net.c > @@ -2028,6 +2028,7 @@ static const struct net_device_ops virtnet_netdev = { > .ndo_poll_controller = virtnet_netpoll, > #endif > .ndo_xdp = virtnet_xdp, > + .ndo_features_check = passthru_features_check, > }; > > static void virtnet_config_changed_work(struct work_struct *work)Acked-by: Jason Wang <jasowang at redhat.com>
Possibly Parallel Threads
- [PATCH net 3/3] virtio-net: enable TSO/checksum offloads for Q-in-Q vlans
- [PATCH v7 net-next 2/4] net: Introduce generic failover module
- [PATCH net-next 1/3] virtio-net: remove unnecessary parameter of virtnet_xdp_xmit()
- [PATCH net-next 1/3] virtio-net: remove unnecessary parameter of virtnet_xdp_xmit()
- [virtio-dev] [RFC PATCH net-next v2 1/2] virtio_net: Introduce VIRTIO_NET_F_BACKUP feature bit