[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH 09/11] hw/arm/virt-acpi-build: don't save VirtGues
From: |
Andrew Jones |
Subject: |
Re: [Qemu-arm] [PATCH 09/11] hw/arm/virt-acpi-build: don't save VirtGuestInfo on AcpiBuildState |
Date: |
Thu, 15 Dec 2016 17:25:03 +0100 |
User-agent: |
Mutt/1.6.0.1 (2016-04-01) |
On Thu, Dec 15, 2016 at 04:08:22PM +0100, Igor Mammedov wrote:
> On Tue, 13 Dec 2016 22:45:20 +0100
> Andrew Jones <address@hidden> wrote:
>
> > We can get to VirtMachineState without the need for saving a pointer
> > on AcpiBuildState. This is the mach-virt equivalent to "acpi: Don't save
> > PcGuestInfo on AcpiBuildState"
> >
> > Signed-off-by: Andrew Jones <address@hidden>
> > ---
> > hw/arm/virt-acpi-build.c | 7 +++----
> > 1 file changed, 3 insertions(+), 4 deletions(-)
> >
> > diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
> > index 7a1efd0dc538..137781646bec 100644
> > --- a/hw/arm/virt-acpi-build.c
> > +++ b/hw/arm/virt-acpi-build.c
> > @@ -706,7 +706,6 @@ struct AcpiBuildState {
> > MemoryRegion *linker_mr;
> > /* Is table patched? */
> > bool patched;
> > - VirtGuestInfo *guest_info;
> > } AcpiBuildState;
> >
> > static
> > @@ -779,6 +778,8 @@ static void acpi_ram_update(MemoryRegion *mr, GArray
> > *data)
> >
> > static void virt_acpi_build_update(void *build_opaque)
> > {
> > + MachineState *machine = MACHINE(qdev_get_machine());
> > + VirtMachineState *vms = VIRT_MACHINE(machine);
> > AcpiBuildState *build_state = build_opaque;
> > AcpiBuildTables tables;
> >
> > @@ -790,8 +791,7 @@ static void virt_acpi_build_update(void *build_opaque)
> >
> > acpi_build_tables_init(&tables);
> >
> > - virt_acpi_build(container_of(build_state->guest_info,
> > - VirtMachineState, acpi_guest_info),
> > &tables);
> > + virt_acpi_build(vms, &tables);
>
> virt_acpi_build(VIRT_MACHINE(qdev_get_machine()), &tables);
>
> or even better do it inside of virt_acpi_build(&tables);
Sounds good.
Thanks,
drew
>
>
> >
> > acpi_ram_update(build_state->table_mr, tables.table_data);
> > acpi_ram_update(build_state->rsdp_mr, tables.rsdp);
> > @@ -841,7 +841,6 @@ void virt_acpi_setup(VirtMachineState *vms)
> > }
> >
> > build_state = g_malloc0(sizeof *build_state);
> > - build_state->guest_info = guest_info;
> >
> > acpi_build_tables_init(&tables);
> > virt_acpi_build(vms, &tables);
>
- Re: [Qemu-arm] [Qemu-devel] [PATCH 07/11] hw/arm/virt: pass VirtMachineState instead of VirtGuestInfo, (continued)
[Qemu-arm] [PATCH 08/11] hw/arm/virt-acpi-build: remove redundant members from VirtGuestInfo, Andrew Jones, 2016/12/13
[Qemu-arm] [PATCH 10/11] hw/arm/virt: remove VirtGuestInfo, Andrew Jones, 2016/12/13
[Qemu-arm] [PATCH 09/11] hw/arm/virt-acpi-build: don't save VirtGuestInfo on AcpiBuildState, Andrew Jones, 2016/12/13
[Qemu-arm] [PATCH 11/11] hw/arm/virt-acpi-build: Don't incorrectly claim architectural timer to be edge-triggered, Andrew Jones, 2016/12/13
Re: [Qemu-arm] [PATCH 00/11] Remove VirtGuestInfo, Michael S. Tsirkin, 2016/12/15