[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] 答复: [PATCH] vnc: add additional key up event before repeat
From: |
Li, Guang |
Subject: |
[Qemu-devel] 答复: [PATCH] vnc: add additional key up event before repeated key down |
Date: |
Thu, 18 Sep 2014 06:01:26 +0000 |
> -----邮件原件-----
> 发件人: address@hidden
> [mailto:address@hidden 代表 Peter
> Maydell
> 发送时间: 2014年9月18日 0:09
> 收件人: Gerd Hoffmann
> 抄送: Stefano Stabellini; Markus Armbruster; qemu-devel;
> address@hidden; Anthony Liguori; Chun Yan Liu
> 主题: Re: [Qemu-devel] [PATCH] vnc: add additional key up event before
> repeated key down
>
> On 17 September 2014 04:24, Gerd Hoffmann <address@hidden> wrote:
> >> Which brings us to the other half of this: what does our UI layer
> >> specify should be the behaviour for key repeat?
> >> Gerd, can you clarify what the common input layer's expectation is
> >> here? Should UI front ends call qemu_input_event_send_key() with
> >> 'down/down/down/up' or 'down/up/down/up' semantics?
> >
> > It isn't formally specified anywhere. The UIs usually simply pass
> > through the key events they get. IMO it is more useful to go for
> > down/down/down/up. This is how PS/2 works, and this allows the guest
> > to figure whenever it's autorepeat or really multiple key presses.
>
> Makes sense. It would be nice to have a comment somewhere (include file near
> the prototype for the 'deliver key event'
> function?) saying this is the required key-repeat behaviour.
>
> I wouldn't be totally surprised if some of our UIs weren't getting this
> right, but
> I'm not sure I care enough to audit them all :-)
>
> Anyway, I think this reinforces Markus's conclusion that TigerVNC is correct
> and our VNC server implementation is correct and the bug is in whichever PV
> guest is not handling the key-repeat info it gets out of PS/2. (At least I
> assume
> that the Xen config in question is going to send key events via emulated
> PS/2; if
> it's something else I guess that something else could potentially be buggy.)
Confirmed, we surely do right things now to collect make,break keycode and
handle repeat case,
But, seems forgot to handle typematic.
Thanks!
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, (continued)
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Chun Yan Liu, 2014/09/08
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Markus Armbruster, 2014/09/09
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Chun Yan Liu, 2014/09/09
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Anthony Liguori, 2014/09/09
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Chun Yan Liu, 2014/09/10
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Markus Armbruster, 2014/09/17
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Peter Maydell, 2014/09/17
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Markus Armbruster, 2014/09/17
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Gerd Hoffmann, 2014/09/17
- Re: [Qemu-devel] [PATCH] vnc: add additional key up event before repeated key down, Peter Maydell, 2014/09/17
- [Qemu-devel] 答复: [PATCH] vnc: add additional key up event before repeated key down,
Li, Guang <=