On Fri, 12 Aug 2016 16:32:56 +0100
Stefan Hajnoczi <stefa...@redhat.com> wrote:
> virtqueue_descard() moves vq->last_avail_idx back so the element can be
> popped again. It's necessary to decrement vq->inuse to avoid "leaking"
> the element count.
>
> Signed-off-by: Stefan Hajnoczi <stefa...@redhat.com>
> ---
> hw/virtio/virtio.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/virtio/virtio.c b/hw/virtio/virtio.c
> index 4df8274..00158b6 100644
> --- a/hw/virtio/virtio.c
> +++ b/hw/virtio/virtio.c
> @@ -268,6 +268,7 @@ void virtqueue_discard(VirtQueue *vq, const
> VirtQueueElement *elem,
> unsigned int len)
> {
> vq->last_avail_idx--;
> + vq->inuse--;
> virtqueue_unmap_sg(vq, elem, len);
> }
>
Reviewed-by: Cornelia Huck <cornelia.h...@de.ibm.com>
