Hi Laszlo, Thanks.
Regards, Jian From: Laszlo Ersek [mailto:[email protected]] Sent: Tuesday, August 21, 2018 11:45 PM To: Wang, Jian J <[email protected]>; [email protected] Cc: Ni, Ruiyu <[email protected]>; Dong, Eric <[email protected]> Subject: Re: [edk2] [PATCH v2 4/4] UefiCpuPkg/PiSmmCpuDxeSmm: implement non-stop mode for SMM On 08/21/18 05:05, Jian J Wang wrote: >> v2 changes: >> fix GCC build error caused by an unused variable in GuardPagePFHandler() > > Since SMM profile feature has already implemented non-stop mode if #PF > occurred, this patch just makes use of the existing implementation to > accommodate heap guard and NULL pointer detection feature. > > Cc: Eric Dong <[email protected]<mailto:[email protected]>> > Cc: Laszlo Ersek <[email protected]<mailto:[email protected]>> > Cc: Ruiyu Ni <[email protected]<mailto:[email protected]>> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Jian J Wang > <[email protected]<mailto:[email protected]>> > --- > UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/PageTbl.c | 43 ++++++++++++------ > UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/SmiException.nasm | 3 +- > UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfile.c | 58 > +++++++++++++++++++++++- > UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfile.h | 15 ++++++ > UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfileInternal.h | 6 +++ > UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c | 43 ++++++++++++------ > 6 files changed, 137 insertions(+), 31 deletions(-) It seems like the changes are no-ops for platforms that set all these PCDs to zero, so from my side Acked-by: Laszlo Ersek <[email protected]<mailto:[email protected]>> Thanks Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
