[v3,00/10] ACPI related fixes to comform the spec better

Message

Isaku Yamahata Feb. 11, 2021, 6:46 a.m. UTC

Miscellaneous bug fixes related to ACPI to play nice with guest BIOSes/OSes
by conforming to ACPI spec better.

Changes from v2:
- improved commit message
- introduced compat property x-smm-compat-5
- _CRS for MMCONFIG resource, read MMCONFIG info from qemu, generate resource
  instead of dynamically reading chipset configuration.

Changes from v1:
- fixed style issue with fixes to checkpatch.pl
- fixed make check breakage
- added ACPI table tests
- update comment message to include acpi table diff

Isaku Yamahata (9):
  checkpatch: don't emit warning on newly created acpi data files
  qtest: update tests/qtest/bios-tables-test-allowed-diff.h
  i386: add properoty, x-smm-compat-5, to keep compatibility of SMM
  acpi/core: always set SCI_EN when SMM isn't supported
  acpi: set fadt.smi_cmd to zero when SMM is not supported
  acpi: add test case for smm unsupported -machine smm=off
  hw/i386: declare ACPI mother board resource for MMCONFIG region
  acpi: add test case for -no-hpet
  qtest/acpi/bios-tables-test: update acpi tables

Sean Christopherson (1):
  i386: acpi: Don't build HPET ACPI entry if HPET is disabled

 hw/acpi/core.c                    |  20 ++++++++-
 hw/acpi/ich9.c                    |   2 +-
 hw/acpi/piix4.c                   |   3 +-
 hw/i386/acpi-build.c              |  72 +++++++++++++++++++++++++++---
 hw/i386/pc_piix.c                 |  10 +++--
 hw/i386/pc_q35.c                  |   1 +
 hw/i386/x86.c                     |  18 ++++++++
 hw/isa/vt82c686.c                 |   2 +-
 include/hw/acpi/acpi.h            |   4 +-
 include/hw/i386/x86.h             |   2 +
 scripts/checkpatch.pl             |   4 +-
 tests/data/acpi/q35/DSDT          | Bin 7801 -> 7892 bytes
 tests/data/acpi/q35/DSDT.acpihmat | Bin 9126 -> 9217 bytes
 tests/data/acpi/q35/DSDT.bridge   | Bin 7819 -> 7910 bytes
 tests/data/acpi/q35/DSDT.cphp     | Bin 8265 -> 8356 bytes
 tests/data/acpi/q35/DSDT.dimmpxm  | Bin 9455 -> 9546 bytes
 tests/data/acpi/q35/DSDT.ipmibt   | Bin 7876 -> 7967 bytes
 tests/data/acpi/q35/DSDT.memhp    | Bin 9160 -> 9251 bytes
 tests/data/acpi/q35/DSDT.mmio64   | Bin 8932 -> 9023 bytes
 tests/data/acpi/q35/DSDT.nohpet   | Bin 0 -> 7750 bytes
 tests/data/acpi/q35/DSDT.nosmm    | Bin 0 -> 7892 bytes
 tests/data/acpi/q35/DSDT.numamem  | Bin 7807 -> 7898 bytes
 tests/data/acpi/q35/DSDT.tis      | Bin 8407 -> 8498 bytes
 tests/data/acpi/q35/FACP.nosmm    | Bin 0 -> 244 bytes
 tests/qtest/bios-tables-test.c    |  24 ++++++++++
 25 files changed, 146 insertions(+), 16 deletions(-)
 create mode 100644 tests/data/acpi/q35/DSDT.nohpet
 create mode 100644 tests/data/acpi/q35/DSDT.nosmm
 create mode 100644 tests/data/acpi/q35/FACP.nosmm