Acked-by: Ray Ni <[email protected]> I will depend on tool owner to review the tool configuration change making sure that the correct section name is chosen for different C compilers.
Thanks, Ray > -----Original Message----- > From: Marvin Häuser <[email protected]> > Sent: Monday, August 9, 2021 5:51 PM > To: [email protected] > Cc: Dong, Eric <[email protected]>; Ni, Ray <[email protected]>; Kumar, > Rahul1 <[email protected]>; Vitaly Cheptsov > <[email protected]> > Subject: [PATCH v2 2/2] UefiCpuPkg/BaseUefiCpuLib: Use toolchain-specific > rodata section name > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3318 > > Correctly define the read-only data sections with the > toolchain-specific section name. This hardens image permission > security and may save image space. > > Cc: Eric Dong <[email protected]> > Cc: Ray Ni <[email protected]> > Cc: Rahul Kumar <[email protected]> > Cc: Vitaly Cheptsov <[email protected]> > Signed-off-by: Marvin Häuser <[email protected]> > --- > UefiCpuPkg/Library/BaseUefiCpuLib/Ia32/InitializeFpu.nasm | 2 +- > UefiCpuPkg/Library/BaseUefiCpuLib/X64/InitializeFpu.nasm | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/UefiCpuPkg/Library/BaseUefiCpuLib/Ia32/InitializeFpu.nasm > b/UefiCpuPkg/Library/BaseUefiCpuLib/Ia32/InitializeFpu.nasm > index 5e27cc325012..cfb8bf4a5ae0 100644 > --- a/UefiCpuPkg/Library/BaseUefiCpuLib/Ia32/InitializeFpu.nasm > +++ b/UefiCpuPkg/Library/BaseUefiCpuLib/Ia32/InitializeFpu.nasm > @@ -6,7 +6,7 @@ > ;* > > > ;------------------------------------------------------------------------------ > > > > - SECTION .rodata > > + SECTION RODATA_SECTION_NAME > > > > ; > > ; Float control word initial value: > > diff --git a/UefiCpuPkg/Library/BaseUefiCpuLib/X64/InitializeFpu.nasm > b/UefiCpuPkg/Library/BaseUefiCpuLib/X64/InitializeFpu.nasm > index 8485b4713548..3c976a21e391 100644 > --- a/UefiCpuPkg/Library/BaseUefiCpuLib/X64/InitializeFpu.nasm > +++ b/UefiCpuPkg/Library/BaseUefiCpuLib/X64/InitializeFpu.nasm > @@ -6,7 +6,7 @@ > ;* > > > ;------------------------------------------------------------------------------ > > > > - SECTION .rodata > > + SECTION RODATA_SECTION_NAME > > ; > > ; Float control word initial value: > > ; all exceptions masked, double-extended-precision, round-to-nearest > > -- > 2.31.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#79001): https://edk2.groups.io/g/devel/message/79001 Mute This Topic: https://groups.io/mt/84764907/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
