> -----Original Message----- > From: Ni, Ray <ray...@intel.com> > Sent: Tuesday, January 26, 2021 10:26 AM > To: Laszlo Ersek <ler...@redhat.com>; Zeng, Star <star.z...@intel.com>; > Kinney, Michael D <michael.d.kin...@intel.com>; devel@edk2.groups.io > Cc: Dong, Eric <eric.d...@intel.com>; Kumar, Rahul1 > <rahul1.ku...@intel.com> > Subject: RE: [edk2-devel] [Patch 1/1] UefiCpuPkg/Library/MpInitLib: Fix AP > VolatileRegisters race condition > > > > CpuMpData = (CPU_MP_DATA *) (Idtr.Base + Idtr.Limit + 1); > > > + ASSERT (CpuMpData != NULL); > > In this case, Idtr.Base, Idtr.Limit both equal to zero and CpuMpData is 1. > ASSERT () cannot trigger assertion here.
Oh, got it. Then the assert may be like below to check Idtr.Base. ASSERT (Idtr.Base != 0); Thanks, Star -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#70753): https://edk2.groups.io/g/devel/message/70753 Mute This Topic: https://groups.io/mt/80035510/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-