[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] acpi_piix4: fix save/load of PIIX4PMState
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH] acpi_piix4: fix save/load of PIIX4PMState |
Date: |
Mon, 18 Apr 2011 18:26:08 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux) |
Isaku Yamahata <address@hidden> wrote:
> It's vmstate parameter was wrong. This patch fixes it.
>
> Reported-by: Avi Kivity <address@hidden>
> Signed-off-by: Isaku Yamahata <address@hidden>
> ---
> hw/acpi_piix4.c | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/hw/acpi_piix4.c b/hw/acpi_piix4.c
> index 96f5222..3a8fece 100644
> --- a/hw/acpi_piix4.c
> +++ b/hw/acpi_piix4.c
> @@ -214,10 +214,9 @@ static int vmstate_acpi_post_load(void *opaque, int
> version_id)
> { \
> .name = (stringify(_field)), \
> .version_id = 0, \
> - .num = GPE_LEN, \
> .info = &vmstate_info_uint16, \
> .size = sizeof(uint16_t), \
> - .flags = VMS_ARRAY | VMS_POINTER, \
> + .flags = VMS_SINGLE | VMS_POINTER, \
> .offset = vmstate_offset_pointer(_state, _field, uint8_t),
> \
shouldn't last one still be uint16_t?
I guess that on ich9, GPE becomes one array, do you have that code handy
somewhere, just to see what you want to do?
I think that best thing to do at this point is just to revert this whole
patch. We are creating a new type for uint8_t, that becomes a pointer.
We are not sending the length of that array, so we need to add a new
version/subsection when we add ICH9 anyways.
Seeing what you want to do would help me trying to figure out the best
vmstate aproach.
Thanks, Juan.