[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/2] ps2: add support of auto-repeat
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/2] ps2: add support of auto-repeat |
Date: |
Thu, 30 May 2013 11:48:46 -0500 |
User-agent: |
Notmuch/0.15.2+77~g661dcf8 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Amos Kong <address@hidden> writes:
> Guest driver sets repeat rate and delay time by KBD_CMD_SET_RATE,
> but ps2 backend doesn't process it and no auto-repeat implementation.
> This patch adds support of auto-repeat feature. The repeated events
> from host are ignored and re-implements ps2's auto-repeat.
>
> Guest ps2 driver sets autorepeat to fastest possible in reset,
> period: 250ms, delay: 33ms
>
> Tested by 'sendkey' monitor command.
> Tested by Linux & Windows guests with SDL, VNC, SPICE, GTK+
>
> referenced: http://www.computer-engineering.org/ps2keyboard/
>
> Signed-off-by: Amos Kong <address@hidden>
> ---
> hw/input/ps2.c | 41 +++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 41 insertions(+)
>
> diff --git a/hw/input/ps2.c b/hw/input/ps2.c
> index 3412079..8adbb4a 100644
> --- a/hw/input/ps2.c
> +++ b/hw/input/ps2.c
> @@ -94,6 +94,10 @@ typedef struct {
> int translate;
> int scancode_set; /* 1=XT, 2=AT, 3=PS/2 */
> int ledstate;
> + int repeat_period; /* typematic period, ms */
> + int repeat_delay; /* typematic delay, ms */
> + int repeat_key; /* keycode to repeat */
> + QEMUTimer *repeat_timer;
This state needs to be migrated, no? I suspect it can/should be done
via a subsection too.
Regards,
Anthony Liguori
> } PS2KbdState;
>
> typedef struct {
> @@ -146,6 +150,15 @@ void ps2_queue(void *opaque, int b)
> s->update_irq(s->update_arg, 1);
> }
>
> +static void repeat_ps2_queue(void *opaque)
> +{
> + PS2KbdState *s = opaque;
> +
> + qemu_mod_timer(s->repeat_timer, qemu_get_clock_ns(vm_clock) +
> + muldiv64(get_ticks_per_sec(), s->repeat_period, 1000));
> + ps2_queue(&s->common, s->repeat_key);
> +}
> +
> /*
> keycode is expressed as follow:
> bit 7 - 0 key pressed, 1 = key released
> @@ -167,7 +180,23 @@ static void ps2_put_keycode(void *opaque, int keycode)
> keycode = ps2_raw_keycode_set3[keycode & 0x7f];
> }
> }
> +
> + /* ignore repeated events from host, re-implement it */
> + if (keycode == s->repeat_key) {
> + return;
> + }
> ps2_queue(&s->common, keycode);
> + qemu_del_timer(s->repeat_timer);
> +
> + /* only auto-repeat press event */
> + if (!(keycode & 0x80)) {
> + s->repeat_key = keycode;
> + /* delay a while before first repeat */
> + qemu_mod_timer(s->repeat_timer, qemu_get_clock_ns(vm_clock) +
> + muldiv64(get_ticks_per_sec(), s->repeat_delay, 1000));
> + } else {
> + s->repeat_key = -1;
> + }
> }
>
> uint32_t ps2_read_data(void *opaque)
> @@ -213,6 +242,11 @@ static void ps2_reset_keyboard(PS2KbdState *s)
>
> void ps2_write_keyboard(void *opaque, int val)
> {
> + /* repeat period/delay table from kernel
> (drivers/input/keyboard/atkbd.c) */
> + const short period[32] = { 33, 37, 42, 46, 50, 54, 58, 63, 67,
> 75,
> + 83, 92, 100, 109, 116, 125, 133, 149, 167, 182, 200, 217,
> 232,
> + 250, 270, 303, 333, 370, 400, 435, 470, 500 };
> + const short delay[4] = { 250, 500, 750, 1000 };
> PS2KbdState *s = (PS2KbdState *)opaque;
>
> switch(s->common.write_cmd) {
> @@ -288,6 +322,10 @@ void ps2_write_keyboard(void *opaque, int val)
> s->common.write_cmd = -1;
> break;
> case KBD_CMD_SET_RATE:
> + /* Bit0-4 specifies the repeat rate */
> + s->repeat_period = period[val & 0x1f];
> + /* Bit5-6 bit specifies the delay time */
> + s->repeat_delay = delay[val >> 5 & 0x3];
> ps2_queue(&s->common, KBD_REPLY_ACK);
> s->common.write_cmd = -1;
> break;
> @@ -536,6 +574,9 @@ static void ps2_kbd_reset(void *opaque)
> s->scan_enabled = 0;
> s->translate = 0;
> s->scancode_set = 0;
> + s->repeat_period = 92;
> + s->repeat_delay = 500;
> + s->repeat_timer = qemu_new_timer_ns(vm_clock, repeat_ps2_queue, s);
> }
>
> static void ps2_mouse_reset(void *opaque)
> --
> 1.8.1.4