[PATCH 0/7] AML Housekeeping

Bernhard Beschow posted 7 patches 1 year, 2 months ago
There is a newer version of this series
hw/acpi/hmat.h                       |  3 ++-
hw/i386/acpi-common.h                |  7 +++++--
include/hw/acpi/acpi_aml_interface.h | 11 +++++++++++
include/hw/acpi/acpi_dev_interface.h |  4 ----
include/hw/acpi/cpu.h                |  6 +++++-
include/hw/i386/pc.h                 |  6 ------
include/hw/isa/isa.h                 |  1 -
hw/acpi/acpi-x86-stub.c              |  7 -------
hw/acpi/cpu.c                        | 12 +++++-------
hw/acpi/hmat.c                       |  1 +
hw/acpi/memory_hotplug.c             |  1 +
hw/acpi/piix4.c                      |  2 --
hw/i2c/smbus_ich9.c                  |  5 +----
hw/i386/acpi-build.c                 |  7 ++-----
hw/i386/acpi-common.c                | 10 ++++------
hw/i386/acpi-microvm.c               |  6 +++---
hw/i386/generic_event_device_x86.c   |  9 ---------
hw/isa/isa-bus.c                     | 10 ----------
hw/isa/lpc_ich9.c                    |  6 +-----
hw/isa/piix3.c                       |  5 +----
monitor/qmp-cmds.c                   |  1 +
21 files changed, 43 insertions(+), 77 deletions(-)
[PATCH 0/7] AML Housekeeping
Posted by Bernhard Beschow 1 year, 2 months ago
This series resolves the AcpiDeviceIfClass::madt_cpu function pointer. It turns
out that it isn't needed and it even frees the ACPI controllers from assigning
it an x86 specific function. This is especially interesting for the PIIX4 PM
which is also used in MIPS only contexts.

Furthermore, the series introduces qbus_build_aml() which then gets
used to resolve isa_build_aml().

Testing done:
* `make check`
* `make check-avocado`
* `qemu-system-x86_64 -M pc -m 2G -cdrom manjaro-kde-21.2.6-220416-linux515.iso`
* `qemu-system-x86_64 -M q35 -m 2G -cdrom \
   manjaro-kde-21.2.6-220416-linux515.iso`

Bernhard Beschow (7):
  hw/acpi: Add missing includes
  hw/acpi/acpi_dev_interface: Remove unused parameter from
    AcpiDeviceIfClass::madt_cpu
  hw/acpi/acpi_dev_interface: Resolve AcpiDeviceIfClass::madt_cpu
  hw/i386/acpi-build: Remove unused attributes
  hw/acpi/acpi_aml_interface: Extract qbus_build_aml()
  hw/i386/acpi-microvm: Reuse qbus_build_aml()
  hw/isa/isa-bus: Remove now unused isa_build_aml()

 hw/acpi/hmat.h                       |  3 ++-
 hw/i386/acpi-common.h                |  7 +++++--
 include/hw/acpi/acpi_aml_interface.h | 11 +++++++++++
 include/hw/acpi/acpi_dev_interface.h |  4 ----
 include/hw/acpi/cpu.h                |  6 +++++-
 include/hw/i386/pc.h                 |  6 ------
 include/hw/isa/isa.h                 |  1 -
 hw/acpi/acpi-x86-stub.c              |  7 -------
 hw/acpi/cpu.c                        | 12 +++++-------
 hw/acpi/hmat.c                       |  1 +
 hw/acpi/memory_hotplug.c             |  1 +
 hw/acpi/piix4.c                      |  2 --
 hw/i2c/smbus_ich9.c                  |  5 +----
 hw/i386/acpi-build.c                 |  7 ++-----
 hw/i386/acpi-common.c                | 10 ++++------
 hw/i386/acpi-microvm.c               |  6 +++---
 hw/i386/generic_event_device_x86.c   |  9 ---------
 hw/isa/isa-bus.c                     | 10 ----------
 hw/isa/lpc_ich9.c                    |  6 +-----
 hw/isa/piix3.c                       |  5 +----
 monitor/qmp-cmds.c                   |  1 +
 21 files changed, 43 insertions(+), 77 deletions(-)

-- 
2.39.0