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: sta...@dpdk.org
>
> Reported-by: Stephen Hemminger <step...@networkplumber.org>
> Signed-off-by: Tiwei Bie <tiwei....@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.coque...@redhat.com>
Thanks,
Maxime