Re: [Qemu-devel] [PATCH 0/3] input: add support for kbd delays

[Gerd Hoffmann <kraxel@redhat.com>]

On Mo, 2014-06-02 at 06:40 -0400, Dave Mielke wrote:
> [quoted lines by Gerd Hoffmann on 2014/06/02 at 11:29 +0200]
> 
> Hi:
> 
> When using my patch, I'm specifying -k delay=20 (so 20ms).
> 
> >Highest repeat rate ps/2 allows is 30Hz.  Which is 33 ms delay per key
> >event.  We have two delays in there (one after pressing all keys, one
> >after releasing all keys).  So something between 10 and 15 ms should be
> >short enough to make sure the queue doesn't grow.  Lets try that,
> >incremental patch attached.
> 
> Key repeat now stops when it should, and letting go of backspace as soon as all 
> the characters are gone no longer leaves me with a temporarily dead keyboard. 
> There are still problems, though. When I hold the shift key and type a letter, 
> I get a doubled lowercase letter rather than a single uppercase letter. Also, I 
> still get doubled lowercase letters when holding the control key.

Ok, so looks like the guest really needs a delay after every single key
event, grouping all down and all up events doesn't work.  New version of
the patch series goes out in a moment.  That should fix both curses ui
and monitor sendkey command.

cheers,
  Gerd