Re: 回复: [edk2-devel] [PATCH v3 00/21] Create a SSDT CPU topology generator

["PierreGondois" <pierre.gondois@arm.com>]

Hi Liming,

I created: https://bugzilla.tianocore.org/show_bug.cgi?id=3681

Aside from this, it seems the DynamicTablesPkg is not available when
reporting a bugzilla on a specific package.

Regards,
Pierre


On 10/11/21 02:16, gaoliming wrote:
> Pierre:
>   Seemly, this is a new feature. Can you submit one BZ for it? Then, I will add it into the feature planning. 
>
> Thanks
> Liming
>> -----邮件原件-----
>> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Sami
>> Mujawar
>> 发送时间: 2021年10月9日 0:05
>> 收件人: Pierre.Gondois@arm.com; devel@edk2.groups.io;
>> Alexei.Fedorov@arm.com; nd <nd@arm.com>
>> 主题: Re: [edk2-devel] [PATCH v3 00/21] Create a SSDT CPU topology
>> generator
>>
>> Merged as 30400318a278..769e63999ff5
>>
>> Thanks.
>>
>> Regards,
>>
>> Sami Mujawar
>>
>>
>> On 08/10/2021 03:46 PM, Pierre.Gondois@arm.com wrote:
>>> From: Pierre Gondois <Pierre.Gondois@arm.com>
>>>
>>> v2:
>>> - Update or error handling. [Sami/Pierre]
>>> - Various comments. [Sami]
>>> and
>>> New patches:
>>> DynamicTablesPkg: Remove unnecessary includes
>>> DynamicTablesPkg: Add missing parameter check
>>> DynamicTablesPkg: Add AddSsdtAcpiHeader()
>>> DynamicTablesPkg: Add AmlRdSetEndTagChecksum()
>>> DynamicTablesPkg: Add AmlSetRdListCheckSum()
>>> DynamicTablesPkg: Set EndTag's Checksum if RdList is modified
>>> DynamicTablesPkg: Clear pointer in node creation fcts
>>> DynamicTablesPkg: Update error handling for node creation
>>>
>>> v3:
>>> -Fix typo and error handling. [Sami]
>>>
>>> As part of the DynamicTablesPkg, add a generator creating a
>>> SSDT table representing a CPU topology. Low Power Idle (LPI)
>>> states can also be generated along the CPUs.
>>> This patch-set also adds new functions to generate AML bytecode.
>>>
>>> The changes can be seen at:
>> https://github.com/PierreARM/edk2/tree/1449_Create_ssdt_cpu_topology_g
>> enerator_v3
>>> Pierre Gondois (21):
>>>    DynamicTablesPkg: Remove unnecessary includes
>>>    DynamicTablesPkg: Add missing parameter check
>>>    DynamicTablesPkg: Add AddSsdtAcpiHeader()
>>>    DynamicTablesPkg: Add AmlRdSetEndTagChecksum()
>>>    DynamicTablesPkg: Add AmlSetRdListCheckSum()
>>>    DynamicTablesPkg: Set EndTag's Checksum if RdList is modified
>>>    DynamicTablesPkg: Clear pointer in node creation fcts
>>>    DynamicTablesPkg: Update error handling for node creation
>>>    DynamicTablesPkg: Make AmlNodeGetIntegerValue public
>>>    DynamicTablesPkg: AML Code generation for Register()
>>>    DynamicTablesPkg: AML Code generation for Resource data EndTag
>>>    DynamicTablesPkg: AML code generation for a Package
>>>    DynamicTablesPkg: Helper function to compute package length
>>>    DynamicTablesPkg: AML code generation for a ResourceTemplate
>>>    DynamicTablesPkg: AML code generation for a Method
>>>    DynamicTablesPkg: AML code generation to Return a NameString
>>>    DynamicTablesPkg: AML code generation for a Method returning a NS
>>>    DynamicTablesPkg: AML code generation for a _LPI object
>>>    DynamicTablesPkg: AML code generation to add an _LPI state
>>>    DynamicTablesPkg: Add CM_ARM_LPI_INFO object
>>>    DynamicTablesPkg: SSDT CPU topology and LPI state generator
>>>
>>>   DynamicTablesPkg/DynamicTables.dsc.inc        |    6 +
>>>   DynamicTablesPkg/Include/AcpiTableGenerator.h |    7 +-
>>>   .../Include/ArmNameSpaceObjects.h             |   68 +
>>>   .../Include/Library/AmlLib/AmlLib.h           |  168 ++
>>>   .../Include/Library/TableHelperLib.h          |   35 +
>>>   .../SsdtCmn600Generator.c                     |    2 -
>>>   .../SsdtCpuTopologyGenerator.c                | 1228
>> +++++++++++++++
>>>   .../SsdtCpuTopologyGenerator.h                |  134 ++
>>>   .../SsdtCpuTopologyLibArm.inf                 |   33 +
>>>   .../SsdtSerialPortGenerator.c                 |    3 -
>>>   .../Library/Common/AmlLib/AmlEncoding/Aml.c   |   87 +-
>>>   .../Library/Common/AmlLib/AmlEncoding/Aml.h   |   47 +-
>>>   .../Common/AmlLib/CodeGen/AmlCodeGen.c        | 1350
>> ++++++++++++++++-
>>>   .../AmlLib/CodeGen/AmlResourceDataCodeGen.c   |  210 ++-
>>>   .../AmlLib/CodeGen/AmlResourceDataCodeGen.h   |   92 ++
>>>   .../AmlLib/ResourceData/AmlResourceData.c     |   33 +
>>>   .../AmlLib/ResourceData/AmlResourceData.h     |   21 +
>>>   .../Library/Common/AmlLib/Tree/AmlClone.c     |    2 -
>>>   .../Library/Common/AmlLib/Tree/AmlNode.c      |   18 +-
>>>   .../Library/Common/AmlLib/Tree/AmlNode.h      |   12 +-
>>>   .../Common/AmlLib/Tree/AmlNodeInterface.c     |    7 +
>>>   .../Library/Common/AmlLib/Tree/AmlTree.c      |   72 +-
>>>   .../Library/Common/AmlLib/Utils/AmlUtility.c  |   79 +-
>>>   .../Library/Common/AmlLib/Utils/AmlUtility.h  |   50 +-
>>>   .../SsdtSerialPortFixupLib.c                  |    1 -
>>>   .../ConfigurationManagerObjectParser.c        |   35 +
>>>   .../Common/TableHelperLib/TableHelper.c       |   89 ++
>>>   27 files changed, 3814 insertions(+), 75 deletions(-)
>>>   create mode 100644
>> DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCpuTopologyLibArm/SsdtCpuTo
>> pologyGenerator.c
>>>   create mode 100644
>> DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCpuTopologyLibArm/SsdtCpuTo
>> pologyGenerator.h
>>>   create mode 100644
>> DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCpuTopologyLibArm/SsdtCpuTo
>> pologyLibArm.inf
>> IMPORTANT NOTICE: The contents of this email and any attachments are
>> confidential and may also be privileged. If you are not the intended recipient,
>> please notify the sender immediately and do not disclose the contents to any
>> other person, use it for any purpose, or store or copy the information in any
>> medium. Thank you.
>>
>>
>> 
>>
>
>
>
>