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

PierreGondois posted 5 patches 2 years, 10 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
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
[edk2-devel] [PATCH v1 0/5] Add ACPI support for Kvmtool
Posted by PierreGondois 2 years, 10 months ago
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

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

-- 
2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#76990): https://edk2.groups.io/g/devel/message/76990
Mute This Topic: https://groups.io/mt/83738484/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH v1 0/5] Add ACPI support for Kvmtool
Posted by Laszlo Ersek 2 years, 10 months ago
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
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#77056): https://edk2.groups.io/g/devel/message/77056
Mute This Topic: https://groups.io/mt/83738484/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH v1 0/5] Add ACPI support for Kvmtool
Posted by Laszlo Ersek 2 years, 10 months ago
On 06/23/21 16:06, PierreGondois wrote:

> 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

The subject lines of Sami's 4 patches should be updated as follows:

ArmVirtPkg/Kvmtool: Add DSDT ACPI table
ArmVirtPkg/Kvmtool: Add Configuration Manager
ArmVirtPkg/Kvmtool: Enable ACPI support
ArmVirtPkg/Kvmtool: Enable Acpiview

(regarding the last patch, I requested in its subthread that ACPIVIEW be
restricted to kvmtool please.)

Thanks
Laszlo



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#77061): https://edk2.groups.io/g/devel/message/77061
Mute This Topic: https://groups.io/mt/83738484/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-