[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: |
Amos Kong |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/2] ps2: add support of auto-repeat |
Date: |
Fri, 14 Jun 2013 13:46:41 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, May 31, 2013 at 08:31:17PM +0800, Amos Kong wrote:
> On Thu, May 30, 2013 at 11:48:46AM -0500, Anthony Liguori wrote:
> > Amos Kong <address@hidden> writes:
> > > 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.
>
> It sounds only reasonable for 'sendkey' command. We want to repeat one
> key for 100 times, the key should be continaully repeated in the dest
> vm until it reaches to 100 times.
>
> For implement this, we should also migrate key_timer in ui/input.c,
> then it will send a release event to ps2 queue when the key_timer
> is expired. The bottom patch migrates repeat_timer & repeat_key,
> where should we save key_timer for migration?
Luiz, any suggestion about migrate the key_timer in ui/input.c?
We need to migrate it, then sendkey can continually work in dest vm
until the timer is expired.
Thanks.
--
Amos.
Re: [Qemu-devel] [PATCH v2 1/2] ps2: add support of auto-repeat,
Amos Kong <=