Re: [edk2-devel] [PATCH v1 0/5] Add ACPI support for Kvmtool

["Laszlo Ersek" <lersek@redhat.com>]

On 06/23/21 16:06, PierreGondois wrote:
> From: Pierre Gondois <Pierre.Gondois@arm.com>
> 
> Kvmtool dynamically generates a device tree describing the platform
> to boot on. Using the patch-sets listed below, the DynamicTables
> framework generates ACPI tables describing a similar platform.
> 
> This patch-set:
>  - adds a ConfigurationManager allowing to generate ACPI tables
>    for Kvmtool
>  - adds the acpiview command line utility to the ArmVirtPkg
>  - update ArmVirtPkg.ci.yaml to add new words and use the
>    DynamicTablesPkg
> 
> This patch sets also set the default platform description format
> to ACPI instead of the device tree (c.f.: PcdForceNoAcpi is set
> to FALSE).
> 
> The changes can be seen at: https://github.com/PierreARM/edk2/tree/1456_Add_ACPI_support_for_Kvmtool_v1
> The results of the CI can be seen at: https://github.com/tianocore/edk2/pull/1753
> 
> This patch-set is dependent over the following patch-sets:
>   [PATCH v1 00/10] Various DynamicTablesPkg modifications 
>   https://edk2.groups.io/g/devel/message/76929
> and:
>   [PATCH v1 00/13] Create a SSDT CPU topology generator 
>   https://edk2.groups.io/g/devel/message/76941
> and:
>   [PATCH v1 0/7] Create a SSDT PCIe generator 
>   https://edk2.groups.io/g/devel/message/76958
> and:
>   [PATCH v1 00/14] Implement a FdtHwInfoParserLib
>   https://edk2.groups.io/g/devel/message/76967
> and:
>   [PATCH v1 0/5] Add DynamicPlatRepoLib
>   https://edk2.groups.io/g/devel/message/76984

Not sure if you want just one BZ for all of these subfeatures, or one BZ
per subfeature, but we definitely need at least one BZ for this series.
Please update the commit messages accordingly.

Thanks
Laszlo

> 
> Pierre Gondois (1):
>   ArmVirtPkg: Add cspell exceptions
> 
> Sami Mujawar (4):
>   ArmVirtPkg: Add DSDT ACPI table for Kvmtool firmware
>   ArmVirtPkg: Add Configuration Manager for Kvmtool firmware
>   ArmVirtPkg: Enable ACPI support for Kvmtool
>   ArmVirtPkg: Enable Acpiview for ArmVirtPkg
> 
>  ArmVirtPkg/ArmVirt.dsc.inc                    |   3 +-
>  ArmVirtPkg/ArmVirtKvmTool.dsc                 |  18 +-
>  ArmVirtPkg/ArmVirtKvmTool.fdf                 |  11 +
>  ArmVirtPkg/ArmVirtPkg.ci.yaml                 |   3 +
>  .../KvmtoolCfgMgrDxe/AslTables/Dsdt.asl       |  19 +
>  .../KvmtoolCfgMgrDxe/ConfigurationManager.c   | 948 ++++++++++++++++++
>  .../KvmtoolCfgMgrDxe/ConfigurationManager.h   |  94 ++
>  .../ConfigurationManagerDxe.inf               |  58 ++
>  8 files changed, 1151 insertions(+), 3 deletions(-)
>  create mode 100644 ArmVirtPkg/KvmtoolCfgMgrDxe/AslTables/Dsdt.asl
>  create mode 100644 ArmVirtPkg/KvmtoolCfgMgrDxe/ConfigurationManager.c
>  create mode 100644 ArmVirtPkg/KvmtoolCfgMgrDxe/ConfigurationManager.h
>  create mode 100644 ArmVirtPkg/KvmtoolCfgMgrDxe/ConfigurationManagerDxe.inf
>