Thanks for fixing the issue. For this patch series:
Reviewed-by: Jian J Wang <jian.j.w...@intel.com> > -----Original Message----- > From: Wu, Hao A > Sent: Tuesday, March 06, 2018 9:33 PM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A <hao.a...@intel.com>; Wang, Jian J <jian.j.w...@intel.com>; > Zeng, Star <star.z...@intel.com>; Dong, Eric <eric.d...@intel.com>; Yao, > Jiewen <jiewen....@intel.com>; Ni, Ruiyu <ruiyu...@intel.com> > Subject: [PATCH v2 0/2] Resolve feature conflict between NX and Stack guard > > V2 changes: > > A. Use Hoblib APIs to get the base of stack from Hob. > B. Remove unnecessary local variable used in function > InitializeDxeNxMemoryProtectionPolicy(). > > V1 history: > > If enabled, NX memory protection feature will mark some types of active > memory as NX (non-executable), which includes the first page of the stack. > This will overwrite the attributes of the first page of the stack if the > stack guard feature is also enabled. > > The series will override the attributes setting to the first page of the > stack by adding back the 'EFI_MEMORY_RP' attribute when the stack guard > feature is enabled. > > Cc: Jian J Wang <jian.j.w...@intel.com> > Cc: Star Zeng <star.z...@intel.com> > Cc: Eric Dong <eric.d...@intel.com> > Cc: Jiewen Yao <jiewen....@intel.com> > Cc: Ruiyu Ni <ruiyu...@intel.com> > > Hao Wu (2): > MdeModulePkg/Core: Refine handling NULL detection in NX setting > MdeModulePkg/Core: Fix feature conflict between NX and Stack guard > > MdeModulePkg/Core/Dxe/DxeMain.inf | 4 +- > MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c | 74 > +++++++++++++++++++++++---- > 2 files changed, 67 insertions(+), 11 deletions(-) > > -- > 2.12.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel