Gerd, thanks. I am about to raise the same comments...

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Gerd Hoffmann
Sent: Wednesday, November 3, 2021 3:00 PM
To: Xu, Min M <min.m...@intel.com>
Cc: devel@edk2.groups.io; Brijesh Singh <brijesh.si...@amd.com>; Erdem Aktas 
<erdemak...@google.com>; James Bottomley <j...@linux.ibm.com>; Yao, Jiewen 
<jiewen....@intel.com>; Tom Lendacky <thomas.lenda...@amd.com>; Dong, Eric 
<eric.d...@intel.com>; Ni, Ray <ray...@intel.com>; Kumar, Rahul1 
<rahul1.ku...@intel.com>
Subject: Re: [edk2-devel] [PATCH V3 23/29] UefiCpuPkg: Update AddressEncMask in 
CpuPageTable

  Hi,

> +  gEfiMdeModulePkgTokenSpaceGuid.PcdTdxSharedBitMask                    ## 
> CONSUMES

>    AddressEncMask = PcdGet64 (PcdPteMemoryEncryptionAddressOrMask) & 
> PAGING_1G_ADDRESS_MASK_64;
> +  if (AddressEncMask == 0) {
> +    AddressEncMask = PcdGet64 (PcdTdxSharedBitMask) & 
> + PAGING_1G_ADDRESS_MASK_64;  }

Looks like two PCDs for basically the same thing.
Should we create a common CC PCD here?

take care,
  Gerd








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83906): https://edk2.groups.io/g/devel/message/83906
Mute This Topic: https://groups.io/mt/86739889/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to