Hi Jiaxin, Thanks for your reminder! I have tested Gerd's patch and it works to me!
Joey Lee On Wed, Nov 15, 2023 at 11:30:07AM +0000, Wu, Jiaxin wrote: > Hi Joey, > > Please check your local code whether has the commit > 170d4ce8e90abb1eff03852940a69c9d17f8afe5 from Gerd, > > Assert in 1478 means you don't have that patch. > > Besides, I'm also porting the change to BaseXApicLib, see patch: > https://edk2.groups.io/g/devel/message/111257 > > Thanks, > Jiaxin > > > -----Original Message----- > > From: joeyli <[email protected]> > > Sent: Wednesday, November 15, 2023 3:35 PM > > To: [email protected]; Zhang, Hongbin1 <[email protected]> > > Cc: Dong, Eric <[email protected]>; Ni, Ray <[email protected]>; Kumar, > > Rahul R <[email protected]>; Gerd Hoffmann <[email protected]>; > > Zeng, Star <[email protected]>; Wu, Jiaxin <[email protected]> > > Subject: Re: [edk2-devel] [PATCH v1] UefiCpuPkg: Get processor extended > > information for SmmCpuServiceProtocol > > > > Hi Hongbin1, > > > > On Mon, May 29, 2023 at 02:39:38PM +0800, Zhang, Hongbin1 via groups.io > > wrote: > > > Some features like RAS need to use processor extended information > > > under smm, So add code to support it > > > > > > Signed-off-by: Hongbin1 Zhang <[email protected]> > > > > I got a ASSERT when booting edk2-stable202308 on a issue machine: > > > > ASSERT /home/joeyli/source_code- > > git/edk2/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c(147 > > 8): (Index != 0) || (LevelType == 0x01) > > > > And, the ASSERT can also be reproduced on edk2 master. After reverted this > > patch, the ASSERT is gone. > > > > I have filed a bug here: > > https://bugzilla.tianocore.org/show_bug.cgi?id=4598 > > > > I have put some tracing information on bugzilla. > > > > Thank a lot! > > Joey Lee > > > > > Cc: Eric Dong <[email protected]> > > > Cc: Ray Ni <[email protected]> > > > Cc: Rahul Kumar <[email protected]> > > > Cc: Gerd Hoffmann <[email protected]> > > > Cc: Star Zeng <[email protected]> > > > Cc: Jiaxin Wu <[email protected]> > > > --- > > > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c > > b/UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c > > > index c0e368ea94..8311c3b9de 100644 > > > --- a/UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c > > > +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c > > > @@ -929,7 +929,7 @@ PiCpuSmmEntry ( > > > gSmmCpuPrivate->Operation[Index] = SmmCpuNone; > > > > > > if (Index < mNumberOfCpus) { > > > - Status = MpServices->GetProcessorInfo (MpServices, Index, > > &gSmmCpuPrivate->ProcessorInfo[Index]); > > > + Status = MpServices->GetProcessorInfo (MpServices, Index | > > CPU_V2_EXTENDED_TOPOLOGY, &gSmmCpuPrivate->ProcessorInfo[Index]); > > > ASSERT_EFI_ERROR (Status); > > > mCpuHotPlugData.ApicId[Index] = gSmmCpuPrivate- > > >ProcessorInfo[Index].ProcessorId; > > > > > > -- > > > 2.37.0.windows.1 > > > > > > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111269): https://edk2.groups.io/g/devel/message/111269 Mute This Topic: https://groups.io/mt/99209786/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
