[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] virtio-net: delete also control queue when TX/RX deleted
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH 2/2] virtio-net: delete also control queue when TX/RX deleted |
Date: |
Wed, 1 Jan 2020 18:43:49 -0500 |
On Thu, Dec 26, 2019 at 06:36:49AM +0200, Yuri Benditovich wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=1708480
> If the control queue is not deleted together with TX/RX, it
> later will be ignored in freeing cache resources and hot
> unplug will not be completed.
>
> Signed-off-by: Yuri Benditovich <address@hidden>
> ---
> hw/net/virtio-net.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> index db3d7c38e6..f325440d01 100644
> --- a/hw/net/virtio-net.c
> +++ b/hw/net/virtio-net.c
> @@ -3101,7 +3101,8 @@ static void virtio_net_device_unrealize(DeviceState
> *dev, Error **errp)
> for (i = 0; i < max_queues; i++) {
> virtio_net_del_queue(n, i);
> }
> -
> + /* delete also control vq */
> + virtio_del_queue(vdev, max_queues * 2);
> qemu_announce_timer_del(&n->announce_timer, false);
> g_free(n->vqs);
> qemu_del_nic(n->nic);
Do we need to limit this to when ctrl vq exists?
> --
> 2.17.1
- Re: [PATCH 2/2] virtio-net: delete also control queue when TX/RX deleted,
Michael S. Tsirkin <=