[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 05/23] hw: arm: Switch to the AML build RSDP
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH v4 05/23] hw: arm: Switch to the AML build RSDP building routine |
Date: |
Fri, 2 Nov 2018 11:23:47 +0100 |
On Fri, 2 Nov 2018 17:35:06 +0800
Shannon Zhao <address@hidden> wrote:
> On 2018/11/1 18:22, Samuel Ortiz wrote:
> > We make the ARM virt ACPI code use the now shared build_rsdp() API from
> > aml-build.c. By doing so we fix a bug where the ARM implementation was
> > missing adding both the legacy and extended checksums, which was
> > building an invalid RSDP table.
> >
> > Signed-off-by: Samuel Ortiz <address@hidden>
> > ---
> > hw/arm/virt-acpi-build.c | 31 +------------------------------
> > 1 file changed, 1 insertion(+), 30 deletions(-)
> >
> > diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
> > index 0ed132b79b..0a6a88380a 100644
> > --- a/hw/arm/virt-acpi-build.c
> > +++ b/hw/arm/virt-acpi-build.c
> > @@ -35,6 +35,7 @@
> > #include "target/arm/cpu.h"
> > #include "hw/acpi/acpi-defs.h"
> > #include "hw/acpi/acpi.h"
> > +#include "hw/acpi/aml-build.h"
> > #include "hw/nvram/fw_cfg.h"
> > #include "hw/acpi/bios-linker-loader.h"
> > #include "hw/loader.h"
> > @@ -366,36 +367,6 @@ static void acpi_dsdt_add_power_button(Aml *scope)
> > aml_append(scope, dev);
> > }
> >
> > -/* RSDP */
> > -static void
> > -build_rsdp(GArray *rsdp_table, BIOSLinker *linker, unsigned
> > xsdt_tbl_offset)
> Note: here we use xsdt table not rsdt for ARM. After your change which
> assigns xsdt table address to rsdt_physical_address, it doesn't work.
Hi Shannon,
I do not really like how this series organized/split and doing refactoring
backwards but that's another story and I'll do suggestions how to make it
better on per patch review later.
But as far as I see this change should only add legacy checksum over
the current rsdp, which is bug fix and makes table spec compliant.
Could you point out why it doesn't work and what exactly breaks?
> IIRC, we discussed this before that ARM virt uses xsdt while pc/q35 uses
> rsdt. So this patch is not necessary I think.
>
> Thanks,
> Shannon
> > -{
> > - AcpiRsdpDescriptor *rsdp = acpi_data_push(rsdp_table, sizeof *rsdp);
> > - unsigned xsdt_pa_size = sizeof(rsdp->xsdt_physical_address);
> > - unsigned xsdt_pa_offset =
> > - (char *)&rsdp->xsdt_physical_address - rsdp_table->data;
> > -
> > - bios_linker_loader_alloc(linker, ACPI_BUILD_RSDP_FILE, rsdp_table, 16,
> > - true /* fseg memory */);
> > -
> > - memcpy(&rsdp->signature, "RSD PTR ", sizeof(rsdp->signature));
> > - memcpy(rsdp->oem_id, ACPI_BUILD_APPNAME6, sizeof(rsdp->oem_id));
> > - rsdp->length = cpu_to_le32(sizeof(*rsdp));
> > - rsdp->revision = 0x02;
> > -
> > - /* Address to be filled by Guest linker */
> > - bios_linker_loader_add_pointer(linker,
> > - ACPI_BUILD_RSDP_FILE, xsdt_pa_offset, xsdt_pa_size,
> > - ACPI_BUILD_TABLE_FILE, xsdt_tbl_offset);
> > -
> > - /* Checksum to be filled by Guest linker */
> > - bios_linker_loader_add_checksum(linker, ACPI_BUILD_RSDP_FILE,
> > - (char *)rsdp - rsdp_table->data, sizeof *rsdp,
> > - (char *)&rsdp->checksum - rsdp_table->data);
> > -
> > - return rsdp_table;
> > -}
> > -
> > static void
> > build_iort(GArray *table_data, BIOSLinker *linker, VirtMachineState *vms)
> > {
> >
- [Qemu-devel] [PATCH v4 03/23] hw: acpi: Export the RSDP build API, (continued)
- [Qemu-devel] [PATCH v4 01/23] hw: i386: Decouple the ACPI build from the PC machine type, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 04/23] hw: acpi: Implement XSDT support for RSDP, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 05/23] hw: arm: Switch to the AML build RSDP building routine, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 11/23] hw: acpi: Export the MCFG getter, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 08/23] hw: i386: Move PCI host definitions to pci_host.h, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 12/23] hw: acpi: Do not create hotplug method when handler is not defined, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 13/23] hw: i386: Make the hotpluggable memory size property more generic, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 09/23] hw: acpi: Export the PCI host and holes getters, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 10/23] hw: acpi: Export and generalize the PCI host AML API, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 07/23] hw: acpi: Factorize _OSC AML across architectures, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 14/23] hw: i386: Export the i386 ACPI SRAT build method, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 15/23] hw: acpi: Fix memory hotplug AML generation error, Samuel Ortiz, 2018/11/01
- [Qemu-devel] [PATCH v4 06/23] hw: acpi: Generalize AML build routines, Samuel Ortiz, 2018/11/01