Re: [dpdk-dev] [PATCH 20.02 v2] net/virtio-user: do not close tap when disabling queue pairs

[Maxime Coquelin <maxime.coquelin@redhat.com>]

On 11/25/19 9:14 AM, Tiwei Bie wrote:
> Do not close the tap fds when disabling queue pairs, instead,
> we just need to unbind the backend. Otherwise, tap port can be
> destroyed unexpectedly.
> 
> Fixes: e3b434818bbb ("net/virtio-user: support kernel vhost")
> Cc: stable@dpdk.org
> 
> Reported-by: Stephen Hemminger <stephen@networkplumber.org>
> Signed-off-by: Tiwei Bie <tiwei.bie@intel.com>
> ---
> v2:
> - Squash attach_queue/detach_queue into one routine (Stephen);
> - Use initializer instead of memset (Stephen);
> - Drop the cast to void * (Stephen);
> 
>  drivers/net/virtio/virtio_user/vhost_kernel.c | 34 +++++++++++++++----
>  .../net/virtio/virtio_user/vhost_kernel_tap.c | 34 ++++++++++++-------
>  .../net/virtio/virtio_user/vhost_kernel_tap.h |  8 +++++
>  drivers/net/virtio/virtio_user/vhost_user.c   |  4 +++
>  .../net/virtio/virtio_user/virtio_user_dev.c  |  5 ++-
>  .../net/virtio/virtio_user/virtio_user_dev.h  |  1 +
>  6 files changed, 65 insertions(+), 21 deletions(-)
> 

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime