[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/2] virtio-9p-device: convert to new virtio_delete_queue
From: |
Christian Schoenebeck |
Subject: |
Re: [PATCH v2 2/2] virtio-9p-device: convert to new virtio_delete_queue |
Date: |
Fri, 17 Jan 2020 14:03:31 +0100 |
On Freitag, 17. Januar 2020 07:09:27 CET address@hidden wrote:
> From: Pan Nengyuan <address@hidden>
>
> Use virtio_delete_queue to make it more clear.
>
> Signed-off-by: Pan Nengyuan <address@hidden>
> ---
> Changes V2 to V1:
> - replace virtio_del_queue to virtio_delete_queue to make it more clear.
> ---
> hw/9pfs/virtio-9p-device.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/9pfs/virtio-9p-device.c b/hw/9pfs/virtio-9p-device.c
> index 910dc5045e..b146387ae2 100644
> --- a/hw/9pfs/virtio-9p-device.c
> +++ b/hw/9pfs/virtio-9p-device.c
> @@ -215,7 +215,7 @@ static void virtio_9p_device_unrealize(DeviceState *dev,
> Error **errp) V9fsVirtioState *v = VIRTIO_9P(dev);
> V9fsState *s = &v->state;
>
> - virtio_del_queue(vdev, 0);
> + virtio_delete_queue(v->vq);
> virtio_cleanup(vdev);
> v9fs_device_unrealize_common(s, errp);
> }
No idea what the deprecation plans of virtio_del_queue() are; no behaviour
change, so this patch is certainly not high priority. On the other hand though
this change most certainly will be required on day.
I can imagine Greg might want to see a more verbose commit message, but that's
beyond my personal detail level, so from my side
Acked-by: Christian Schoenebeck <address@hidden>