Hi Jeff,
Thank you for this patch.
On 18/08/2022 11:22 am, Ard Biesheuvel wrote:
On Wed, 17 Aug 2022 at 18:14, Jeff Brasen <jbra...@nvidia.com> wrote:
-----Original Message-----
From: Ard Biesheuvel <a...@kernel.org>
Sent: Wednesday, August 17, 2022 1:01 AM
To: Jeff Brasen <jbra...@nvidia.com>
Cc: devel@edk2.groups.io; quic_llind...@quicinc.com;
ardb+tianoc...@kernel.org; sami.muja...@arm.com;
alexei.fedo...@arm.com
Subject: Re: [PATCH] DynamicTablesPkg: Correct cluster index
External email: Use caution opening links or attachments
On Tue, 16 Aug 2022 at 22:46, Jeff Brasen <jbra...@nvidia.com> wrote:
Current code will generate duplicate UID if there are nested clusters
in the topology.
What is a nested cluster?
Better terminology would be nested processor containers, in our case we have
socket/cluster/cpu. I'll update the commit message. Should I also change
CreateAmlCluster to CreateAmlProcessorContainer?
Works for me, but Sami has the final call as he maintains this package.
[SAMI] CreateAmlProcessorContainer is more appropriate. Please change
CreateAmlCluster to CreateAmlProcessorContainer.
Regards,
Sami Mujawar
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92550): https://edk2.groups.io/g/devel/message/92550
Mute This Topic: https://groups.io/mt/93067950/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
