[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/2] ps2: preserve repeat state on migration
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/2] ps2: preserve repeat state on migration |
Date: |
Thu, 30 May 2013 11:49:18 -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:
> Use a subsection to migrate repeat state (repate period and first
> delay).
>
> Signed-off-by: Amos Kong <address@hidden>
Ah,
You should fold this into 1/2. Otherwise you break migration during
bisecting.
Regards,
Anthony Liguori
> ---
> hw/input/ps2.c | 22 ++++++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/hw/input/ps2.c b/hw/input/ps2.c
> index 8adbb4a..cdb18e6 100644
> --- a/hw/input/ps2.c
> +++ b/hw/input/ps2.c
> @@ -611,6 +611,13 @@ static const VMStateDescription vmstate_ps2_common = {
> }
> };
>
> +static bool ps2_keyboard_repeatstate_needed(void *opaque)
> +{
> + PS2KbdState *s = opaque;
> +
> + return s->repeat_period || s->repeat_delay;
> +}
> +
> static bool ps2_keyboard_ledstate_needed(void *opaque)
> {
> PS2KbdState *s = opaque;
> @@ -626,6 +633,18 @@ static int ps2_kbd_ledstate_post_load(void *opaque, int
> version_id)
> return 0;
> }
>
> +static const VMStateDescription vmstate_ps2_keyboard_repeatstate = {
> + .name = "ps2kbd/repeatstate",
> + .version_id = 3,
> + .minimum_version_id = 2,
> + .minimum_version_id_old = 2,
> + .fields = (VMStateField[]) {
> + VMSTATE_INT32(repeat_period, PS2KbdState),
> + VMSTATE_INT32(repeat_delay, PS2KbdState),
> + VMSTATE_END_OF_LIST()
> + }
> +};
> +
> static const VMStateDescription vmstate_ps2_keyboard_ledstate = {
> .name = "ps2kbd/ledstate",
> .version_id = 3,
> @@ -665,6 +684,9 @@ static const VMStateDescription vmstate_ps2_keyboard = {
> .vmsd = &vmstate_ps2_keyboard_ledstate,
> .needed = ps2_keyboard_ledstate_needed,
> }, {
> + .vmsd = &vmstate_ps2_keyboard_repeatstate,
> + .needed = ps2_keyboard_repeatstate_needed,
> + }, {
> /* empty */
> }
> }
> --
> 1.8.1.4