Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-19 Thread Juan Quintela
Alexander Graf wrote: > We can currently buffer up to 16 events in our event queue for input > devices. While it sounds like 16 events backlog for everyone should > be enough, our automated testing tools (OpenQA) manage to easily > type faster than our guests can handle. > > So we

Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-18 Thread Alexander Graf
On 18.04.16 11:21, Gerd Hoffmann wrote: > On Mo, 2016-04-18 at 09:26 +0200, Alexander Graf wrote: >> >> On 18.04.16 08:53, Gerd Hoffmann wrote: >>> Hi, >>> Vnc already uses qemu_input_event_send_key_delay today, so I'm not sure where things fall apart. >>> >>> Well, not everywhere.

Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-18 Thread Gerd Hoffmann
On Mo, 2016-04-18 at 09:26 +0200, Alexander Graf wrote: > > On 18.04.16 08:53, Gerd Hoffmann wrote: > > Hi, > > > >> Vnc already uses qemu_input_event_send_key_delay today, so I'm not sure > >> where things fall apart. > > > > Well, not everywhere. Try the attached patch. > > > > Also

Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-18 Thread Alexander Graf
On 18.04.16 08:53, Gerd Hoffmann wrote: > Hi, > >> Vnc already uses qemu_input_event_send_key_delay today, so I'm not sure >> where things fall apart. > > Well, not everywhere. Try the attached patch. > > Also worth trying: > * use xhci instead of ohci (current slof should handle >

Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-18 Thread Gerd Hoffmann
Hi, > Vnc already uses qemu_input_event_send_key_delay today, so I'm not sure > where things fall apart. Well, not everywhere. Try the attached patch. Also worth trying: * use xhci instead of ohci (current slof should handle kbd-via-xhci fine) * use virtio-keyboard (no slof driver yet

Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-15 Thread Dinar Valeev
On 04/14/2016 06:19 PM, Gerd Hoffmann wrote: > On Do, 2016-04-14 at 17:29 +0200, Alexander Graf wrote: >> On 04/14/2016 05:17 PM, Gerd Hoffmann wrote: >>> On Do, 2016-04-14 at 16:25 +0200, Alexander Graf wrote: We can currently buffer up to 16 events in our event queue for input devices.

Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-14 Thread Gerd Hoffmann
On Do, 2016-04-14 at 17:29 +0200, Alexander Graf wrote: > On 04/14/2016 05:17 PM, Gerd Hoffmann wrote: > > On Do, 2016-04-14 at 16:25 +0200, Alexander Graf wrote: > >> We can currently buffer up to 16 events in our event queue for input > >> devices. While it sounds like 16 events backlog for

Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-14 Thread Alexander Graf
On 04/14/2016 05:17 PM, Gerd Hoffmann wrote: On Do, 2016-04-14 at 16:25 +0200, Alexander Graf wrote: We can currently buffer up to 16 events in our event queue for input devices. While it sounds like 16 events backlog for everyone should be enough, our automated testing tools (OpenQA) manage to

Re: [Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-14 Thread Gerd Hoffmann
On Do, 2016-04-14 at 16:25 +0200, Alexander Graf wrote: > We can currently buffer up to 16 events in our event queue for input > devices. While it sounds like 16 events backlog for everyone should > be enough, our automated testing tools (OpenQA) manage to easily > type faster than our guests can

[Qemu-devel] [PATCH] hid: Extend the event queue size to 1024

2016-04-14 Thread Alexander Graf
We can currently buffer up to 16 events in our event queue for input devices. While it sounds like 16 events backlog for everyone should be enough, our automated testing tools (OpenQA) manage to easily type faster than our guests can handle. So we run into queue overflows and start to drop input