Re: [edk2-devel] [PATCH V2] UefiCpuPkg CpuCommFeaturesLib: Fix ASSERT if LMCE is supported

2019-05-23 Thread Dong, Eric
Reviewed-by: Eric Dong > -Original Message- > From: Zeng, Star > Sent: Wednesday, May 22, 2019 6:18 PM > To: devel@edk2.groups.io > Cc: Zeng, Star ; Laszlo Ersek ; > Dong, Eric ; Ni, Ray ; Kumar, > Chandana C ; Li, Kevin Y > > Subject: [PATCH V2] UefiCpuPkg CpuCommFeaturesLib: Fix ASSERT

Re: [edk2-devel] [PATCH V2] UefiCpuPkg CpuCommFeaturesLib: Fix ASSERT if LMCE is supported

2019-05-22 Thread Zeng, Star
l] [PATCH V2] UefiCpuPkg CpuCommFeaturesLib: Fix > ASSERT if LMCE is supported > > On 05/22/19 12:17, Zeng, Star wrote: > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1829 > > > > There will be ASSERT if LMCE is supported as below. > > DXE_ASSERT!: [CpuFeaturesDxe] > &g

Re: [edk2-devel] [PATCH V2] UefiCpuPkg CpuCommFeaturesLib: Fix ASSERT if LMCE is supported

2019-05-22 Thread Laszlo Ersek
On 05/22/19 12:17, Zeng, Star wrote: > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1829 > > There will be ASSERT if LMCE is supported as below. > DXE_ASSERT!: [CpuFeaturesDxe] > XXX\UefiCpuPkg\Library\CpuCommonFeaturesLib\MachineCheck.c (342): > ConfigData != ((void *) 0) > > The cod

[edk2-devel] [PATCH V2] UefiCpuPkg CpuCommFeaturesLib: Fix ASSERT if LMCE is supported

2019-05-22 Thread Zeng, Star
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1829 There will be ASSERT if LMCE is supported as below. DXE_ASSERT!: [CpuFeaturesDxe] XXX\UefiCpuPkg\Library\CpuCommonFeaturesLib\MachineCheck.c (342): ConfigData != ((void *) 0) The code should get Config Data and FeatureControlGetConfigD