Re: [PATCH v3] net/virtio-user: check negotiated features before set

[Tiwei Bie <tiwei.bie@intel.com>]

On Wed, Aug 29, 2018 at 11:55:21AM -0400, eric zhang wrote:
> This patch checks negotiated features to see if necessary to offload
> before set the tap device offload capabilities. It also checks if kernel
> support the TUNSETOFFLOAD operation.

Fixes: 5e97e4202563 ("net/virtio-user: enable offloading")
Cc: stable@dpdk.org

Reviewed-by: Tiwei Bie <tiwei.bie@intel.com>

Thanks!

> 
> Signed-off-by: eric zhang <eric.zhang@windriver.com>
> 
> ---
> v3:
> * make other offloading features depend on CSUM
> * check IFF_VNET_HDR support when handling VHOST_GET_FEATURES
> 
> ---
> v2:
> * don't return failure when failed to set offload to tap
> * check if offloads available when handling VHOST_GET_FEATURES
> ---
>  drivers/net/virtio/virtio_user/vhost_kernel.c     | 18 +++++---
>  drivers/net/virtio/virtio_user/vhost_kernel_tap.c | 56 +++++++++++++++++------
>  drivers/net/virtio/virtio_user/vhost_kernel_tap.h |  2 +-
>  3 files changed, 54 insertions(+), 22 deletions(-)
>