[Qemu-devel] [PATCH v3 0/8] hw: acpi: RSDP fixes and refactoring

Samuel Ortiz posted 8 patches 5 years, 3 months ago
Test asan passed
Test checkpatch passed
Test docker-quick@centos7 passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20181130130032.11835-1-sameo@linux.intel.com
There is a newer version of this series
include/hw/acpi/acpi-defs.h | 19 ++++-------
include/hw/acpi/aml-build.h |  2 ++
tests/acpi-utils.h          |  4 ++-
hw/acpi/aml-build.c         | 68 +++++++++++++++++++++++++++++++++++++
hw/arm/virt-acpi-build.c    | 40 +++++-----------------
hw/i386/acpi-build.c        | 36 +++++---------------
tests/acpi-utils.c          | 46 ++++++++++++++++++++-----
tests/bios-tables-test.c    | 21 +++++++++---
tests/vmgenid-test.c        |  8 +++--
9 files changed, 157 insertions(+), 87 deletions(-)
[Qemu-devel] [PATCH v3 0/8] hw: acpi: RSDP fixes and refactoring
Posted by Samuel Ortiz 5 years, 3 months ago
This patch serie fixes a couple of RSDP checksum related issues:

- On RSDP rev2, we are not adding the extended checksum and computing
  the checksum on the full table instead of the first 20 bytes.
- On RSDP rev1, we are computing the checksum on 36 bytes instead of 20.
  We're lucky enough that this is only adding zeroes to the checksum.

A guest Linux kernel does not seem to care about RSDP checksums, so
those 2 fixes are mostly for correctness sake.
Any machine type that generates rev2 RSDP will see its RSDP table
modified but since x86 builds RSDP v1 and all ACPI tests only run on
either pc or q35, the ACPI tests tables are not affected by this fix.

The serie also extends the ARM virt ACPI RSDP build routine to support
both RSDP v1 and v2, in order to share this code between x86 and
aarch64. While extending, we also convert the routine to the latest
build_append_foo() API. The new implementation is a closer reflection of
the ACPI spec itself, is endian agnostic and allows for getting rid of the
AcpiRsdpDescriptor structure.

V2 -> V3:
  * Add Reviewed-Bys from Igor and Andrew.
  * Check for valid RSDP and XSDT pointers depending on the revision.
  * Use RSDP initial table len as offset for checksum and pointer
    insertion.
  * Do not explictly pass RSDP revision as the RSDP table parsing test
    utility.

v1 -> v2:
  * Add missing SOBs.
  * Rephrase some commit messages according to the maintainer's input.
  * Replace ACPI spec definitions with hard coded constants.
  * Replace init_rsdp_data() helper with open coded rsdp variable init.
  * Rename acpi_find_[rx]sdt_address to acpi_get_[rx]sdt_address.

Igor Mammedov (2):
  hw: arm: acpi: Fix incorrect checksums in RSDP
  hw: i386: Use correct RSDT length for checksum

Samuel Ortiz (6):
  hw: acpi: The RSDP build API can return void
  hw: arm: Carry RSDP specific data through AcpiRsdpData
  hw: arm: Convert the RSDP build to the buid_append_foo() API
  hw: arm: Support both legacy and current RSDP build
  hw: acpi: Export and share the ARM RSDP build
  hw: acpi: Remove AcpiRsdpDescriptor and fix tests

 include/hw/acpi/acpi-defs.h | 19 ++++-------
 include/hw/acpi/aml-build.h |  2 ++
 tests/acpi-utils.h          |  4 ++-
 hw/acpi/aml-build.c         | 68 +++++++++++++++++++++++++++++++++++++
 hw/arm/virt-acpi-build.c    | 40 +++++-----------------
 hw/i386/acpi-build.c        | 36 +++++---------------
 tests/acpi-utils.c          | 46 ++++++++++++++++++++-----
 tests/bios-tables-test.c    | 21 +++++++++---
 tests/vmgenid-test.c        |  8 +++--
 9 files changed, 157 insertions(+), 87 deletions(-)

-- 
2.19.2