Re: [edk2-devel] [PATCH v6 1/6] MdePkg: Add macro definitions for CET feature for NASM files.

2023-11-23 Thread Ni, Ray
> How about this: unify the macro definitions in a single file, but call > that file: > > MdePkg/Include/X86Cet.inc > I don't have concerns on using one unified file: X86Cet.inc. @Kinney, Michael D, any comments? -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to th

Re: [edk2-devel] [PATCH v6 1/6] MdePkg: Add macro definitions for CET feature for NASM files.

2023-11-22 Thread Laszlo Ersek
On 11/21/23 08:53, Ni, Ray wrote: > You might need to drop Reviewed-by from Laszlo as the patch content is so > different than the reviewed version. > > Reviewed-by: Ray Ni > > > > Thanks, > Ray >> -Original Message- >> From: Sheng, W >> Sent: Tuesday, November 21, 2023 3:03 PM >> To

Re: [edk2-devel] [PATCH v6 1/6] MdePkg: Add macro definitions for CET feature for NASM files.

2023-11-20 Thread Ni, Ray
You might need to drop Reviewed-by from Laszlo as the patch content is so different than the reviewed version. Reviewed-by: Ray Ni Thanks, Ray > -Original Message- > From: Sheng, W > Sent: Tuesday, November 21, 2023 3:03 PM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Ni, Ray ; Lasz

[edk2-devel] [PATCH v6 1/6] MdePkg: Add macro definitions for CET feature for NASM files.

2023-11-20 Thread Sheng Wei
Signed-off-by: Sheng Wei Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Cc: Wu Jiaxin Cc: Tan Dun Reviewed-by: Laszlo Ersek --- MdePkg/Include/Ia32/Cet.inc | 26 ++ MdePkg/Include/X64/Cet.inc | 26 ++ 2 files changed, 52 insertions(+) create mode