mbox series

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

Message ID cover.1613615732.git.isaku.yamahata@intel.com (mailing list archive)
Headers show
Series ACPI related fixes to comform the spec better | expand

Message

Isaku Yamahata Feb. 18, 2021, 5:51 a.m. UTC
From: Isaku Yamahata <isaku.yamahata@intel.com>

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

Changes from v4:
- rebased to 1af5629673
- move compat property from 3/10 to 4/10
- use the end of the address for rage maximum of memory region
- code simplification to generate _CRS

Changes from v3:
- rename compat property name to smm-compat and add it to ICH9-LPC and PIIX4_PM
- MMCFG memory region to use dword memory region when possible
- fix max base address of MMCFG memory region
- add more test cases with smm-compat=on and/or piix4

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
  ich9, piix4: add properoty, smm-compat, 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                    |  11 +++-
 hw/acpi/ich9.c                    |   2 +-
 hw/acpi/piix4.c                   |   5 +-
 hw/core/machine.c                 |   5 +-
 hw/i386/acpi-build.c              |  72 +++++++++++++++++++--
 hw/isa/lpc_ich9.c                 |   1 +
 hw/isa/vt82c686.c                 |   2 +-
 include/hw/acpi/acpi.h            |   4 +-
 include/hw/acpi/ich9.h            |   1 +
 scripts/checkpatch.pl             |   4 +-
 tests/data/acpi/pc/DSDT.nohpet    | Bin 0 -> 4923 bytes
 tests/data/acpi/pc/FACP.nosmm     | Bin 0 -> 116 bytes
 tests/data/acpi/q35/DSDT          | Bin 7801 -> 7859 bytes
 tests/data/acpi/q35/DSDT.acpihmat | Bin 9126 -> 9184 bytes
 tests/data/acpi/q35/DSDT.bridge   | Bin 7819 -> 7877 bytes
 tests/data/acpi/q35/DSDT.cphp     | Bin 8265 -> 8323 bytes
 tests/data/acpi/q35/DSDT.dimmpxm  | Bin 9455 -> 9513 bytes
 tests/data/acpi/q35/DSDT.ipmibt   | Bin 7876 -> 7934 bytes
 tests/data/acpi/q35/DSDT.memhp    | Bin 9160 -> 9218 bytes
 tests/data/acpi/q35/DSDT.mmio64   | Bin 8932 -> 8990 bytes
 tests/data/acpi/q35/DSDT.nohpet   | Bin 0 -> 7717 bytes
 tests/data/acpi/q35/DSDT.numamem  | Bin 7807 -> 7865 bytes
 tests/data/acpi/q35/DSDT.tis      | Bin 8407 -> 8465 bytes
 tests/data/acpi/q35/FACP.nosmm    | Bin 0 -> 244 bytes
 tests/qtest/bios-tables-test.c    | 100 ++++++++++++++++++++++++++++++
 25 files changed, 194 insertions(+), 13 deletions(-)
 create mode 100644 tests/data/acpi/pc/DSDT.nohpet
 create mode 100644 tests/data/acpi/pc/FACP.nosmm
 create mode 100644 tests/data/acpi/q35/DSDT.nohpet
 create mode 100644 tests/data/acpi/q35/FACP.nosmm

Comments

Igor Mammedov Feb. 24, 2021, 4:08 p.m. UTC | #1
On Wed, 17 Feb 2021 21:51:08 -0800
isaku.yamahata@gmail.com wrote:

> From: Isaku Yamahata <isaku.yamahata@intel.com>
> 
> Miscellaneous bug fixes related to ACPI to play nice with guest BIOSes/OSes
> by conforming to ACPI spec better.

Michael,
  I finished with review,
  to me series looks ready to be merged

> Changes from v4:
> - rebased to 1af5629673
> - move compat property from 3/10 to 4/10
> - use the end of the address for rage maximum of memory region
> - code simplification to generate _CRS
> 
> Changes from v3:
> - rename compat property name to smm-compat and add it to ICH9-LPC and PIIX4_PM
> - MMCFG memory region to use dword memory region when possible
> - fix max base address of MMCFG memory region
> - add more test cases with smm-compat=on and/or piix4
> 
> 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
>   ich9, piix4: add properoty, smm-compat, 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                    |  11 +++-
>  hw/acpi/ich9.c                    |   2 +-
>  hw/acpi/piix4.c                   |   5 +-
>  hw/core/machine.c                 |   5 +-
>  hw/i386/acpi-build.c              |  72 +++++++++++++++++++--
>  hw/isa/lpc_ich9.c                 |   1 +
>  hw/isa/vt82c686.c                 |   2 +-
>  include/hw/acpi/acpi.h            |   4 +-
>  include/hw/acpi/ich9.h            |   1 +
>  scripts/checkpatch.pl             |   4 +-
>  tests/data/acpi/pc/DSDT.nohpet    | Bin 0 -> 4923 bytes
>  tests/data/acpi/pc/FACP.nosmm     | Bin 0 -> 116 bytes
>  tests/data/acpi/q35/DSDT          | Bin 7801 -> 7859 bytes
>  tests/data/acpi/q35/DSDT.acpihmat | Bin 9126 -> 9184 bytes
>  tests/data/acpi/q35/DSDT.bridge   | Bin 7819 -> 7877 bytes
>  tests/data/acpi/q35/DSDT.cphp     | Bin 8265 -> 8323 bytes
>  tests/data/acpi/q35/DSDT.dimmpxm  | Bin 9455 -> 9513 bytes
>  tests/data/acpi/q35/DSDT.ipmibt   | Bin 7876 -> 7934 bytes
>  tests/data/acpi/q35/DSDT.memhp    | Bin 9160 -> 9218 bytes
>  tests/data/acpi/q35/DSDT.mmio64   | Bin 8932 -> 8990 bytes
>  tests/data/acpi/q35/DSDT.nohpet   | Bin 0 -> 7717 bytes
>  tests/data/acpi/q35/DSDT.numamem  | Bin 7807 -> 7865 bytes
>  tests/data/acpi/q35/DSDT.tis      | Bin 8407 -> 8465 bytes
>  tests/data/acpi/q35/FACP.nosmm    | Bin 0 -> 244 bytes
>  tests/qtest/bios-tables-test.c    | 100 ++++++++++++++++++++++++++++++
>  25 files changed, 194 insertions(+), 13 deletions(-)
>  create mode 100644 tests/data/acpi/pc/DSDT.nohpet
>  create mode 100644 tests/data/acpi/pc/FACP.nosmm
>  create mode 100644 tests/data/acpi/q35/DSDT.nohpet
>  create mode 100644 tests/data/acpi/q35/FACP.nosmm
>