I'll modify the comments and add a new patch to initialize MapMask to 0. Thanks for the comment.
Thanks, Dun -----Original Message----- From: Ni, Ray <[email protected]> Sent: Wednesday, March 15, 2023 2:36 PM To: Tan, Dun <[email protected]>; [email protected] Cc: Liu, Zhiguang <[email protected]>; Dong, Eric <[email protected]>; Kumar, Rahul R <[email protected]> Subject: RE: [Patch V2 13/14] UefiCpuPkg: Fix IA32 build failure in CpuPageTableLib.inf > + UINT32 PageTableBaseAddressLow : 19; // Page Table Base Address > High 1. Comments say "High". Should be "Low". > + > > - MapMask.Bits.PageTableBaseAddress = 1; > - MapMask.Bits.Present = 1; > - MapMask.Bits.ReadWrite = 1; > + MapMask.Bits.PageTableBaseAddressLow = 1; > + MapMask.Bits.Present = 1; > + MapMask.Bits.ReadWrite = 1; 2. Can you please create a separate patch to initialize MapMask to 0? Missing the initialization doesn't cause functionality issue but looks confusing. > > PageTable = 0; > PageTableBufferSize = 0; > -- > 2.31.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#101221): https://edk2.groups.io/g/devel/message/101221 Mute This Topic: https://groups.io/mt/97469489/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
