]> git.proxmox.com Git - mirror_qemu.git/commit
virtio-input: fix eventq batching
authorLadi Prosek <lprosek@redhat.com>
Fri, 24 Mar 2017 14:24:50 +0000 (15:24 +0100)
committerGerd Hoffmann <kraxel@redhat.com>
Mon, 27 Mar 2017 10:14:45 +0000 (12:14 +0200)
commit57094547dfea4ce784923b8abb53ac3ab4e3961a
tree439ffe430c2e91d781e8280bc56aeae8585eaf88
parent0f5a15e40ad1512c70d1323a73c7b073e32e17ee
virtio-input: fix eventq batching

virtio_input_send buffers input events until it sees a SYNC. Then it
either sends or drops the entire batch, depending on whether eventq
has enough space available. The case to avoid here is partial sends
where only part of the batch would get to the guest.

Using virtqueue_get_avail_bytes to check the state of eventq was not
correct. The queue may have a smaller number of larger buffers
available so bytes may be enough but the batch would still not be
possible to send, leading to the "Huh?  No vq elem available" error.

Instead of checking available bytes, this patch optimistically pops
buffers from the queue and puts them back in case it runs out of
space and the batch needs to be dropped.

Signed-off-by: Ladi Prosek <lprosek@redhat.com>
Message-id: 1490365490-4854-3-git-send-email-lprosek@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
hw/input/virtio-input.c
include/hw/virtio/virtio-input.h