[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/8] hw: arm: acpi: Fix incorrect checksums in R
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH 2/8] hw: arm: acpi: Fix incorrect checksums in RSDP |
Date: |
Tue, 27 Nov 2018 15:50:16 +0100 |
On Mon, 26 Nov 2018 17:29:35 +0100
Samuel Ortiz <address@hidden> wrote:
> From: Igor Mammedov <address@hidden>
>
> When RSDP table was introduced (d4bec5d87), we calculated only legacy
> checksum, and that was incorrect as it
> - specified rev=2 and forgot about extended checksum.
> - legacy checksum calculated on full table instead of the 1st 20 bytes
>
> Fix it by adding extended checksum calculation and using correct
> size for legacy checksum.
>
> While at it use explicit constants to specify sub/full tables
> sizes instead of relying on AcpiRsdpDescriptor size and fields offsets.
> The follow up commits will convert this table to build_append_int_noprefix()
> API,
> will use constants anyway and remove unused AcpiRsdpDescriptor structure.
>
> Based on "[PATCH v5 05/24] hw: acpi: Implement XSDT support for RSDP"
> by Samuel Ortiz, who did it right in his impl.
>
> Fixes: d4bec5d87 (hw/arm/virt-acpi-build: Generate RSDP table)
> Signed-off-by: Igor Mammedov <address@hidden>
> CC: Ard Biesheuvel <address@hidden>
> CC: Shannon Zhao <address@hidden>
> Reviewed-by: Samuel Ortiz <address@hidden>
For future reference, if one re-sends someone else patch, one is supposed to
add his/her own SoB as the last one.
> ---
> hw/arm/virt-acpi-build.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
> index fcaa350892..0835900052 100644
> --- a/hw/arm/virt-acpi-build.c
> +++ b/hw/arm/virt-acpi-build.c
> @@ -390,8 +390,13 @@ build_rsdp(GArray *rsdp_table, BIOSLinker *linker,
> unsigned 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 - rsdp_table->data, 20 /* ACPI rev 1.0 RSDP size */,
> (char *)&rsdp->checksum - rsdp_table->data);
> +
> + /* Extended checksum to be filled by Guest linker */
> + bios_linker_loader_add_checksum(linker, ACPI_BUILD_RSDP_FILE,
> + (char *)rsdp - rsdp_table->data, 36 /* ACPI rev 2.0 RSDP size */,
> + (char *)&rsdp->extended_checksum - rsdp_table->data);
> }
>
> static void
- [Qemu-devel] [PATCH 0/8] hw: acpi: RSDP fixes and refactoring, Samuel Ortiz, 2018/11/26
- [Qemu-devel] [PATCH 2/8] hw: arm: acpi: Fix incorrect checksums in RSDP, Samuel Ortiz, 2018/11/26
- Re: [Qemu-devel] [PATCH 2/8] hw: arm: acpi: Fix incorrect checksums in RSDP,
Igor Mammedov <=
- [Qemu-devel] [PATCH 1/8] hw: acpi: The RSDP build API can return void, Samuel Ortiz, 2018/11/26
- [Qemu-devel] [PATCH 3/8] hw: i386: Use correct RSDT length for checksum, Samuel Ortiz, 2018/11/26
- [Qemu-devel] [PATCH 5/8] hw: arm: Convert the RSDP build to the buid_append_foo() API, Samuel Ortiz, 2018/11/26
- [Qemu-devel] [PATCH 4/8] hw: arm: Carry RSDP specific data through AcpiRsdpData, Samuel Ortiz, 2018/11/26