diff mbox series

pc-bios/s390-ccw/Makefile: Use -z noexecstack to silence linker warning

Message ID 20230622130822.396793-1-thuth@redhat.com
State New
Headers show
Series pc-bios/s390-ccw/Makefile: Use -z noexecstack to silence linker warning | expand

Commit Message

Thomas Huth June 22, 2023, 1:08 p.m. UTC
Recent versions of ld complain when linking the s390-ccw bios:

 /usr/bin/ld: warning: start.o: missing .note.GNU-stack section implies
              executable stack
 /usr/bin/ld: NOTE: This behaviour is deprecated and will be removed in
              a future version of the linker

We can silence the warning by telling the linker to mark the stack
as not executable.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 pc-bios/s390-ccw/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Christian Borntraeger June 22, 2023, 1:12 p.m. UTC | #1
Am 22.06.23 um 15:08 schrieb Thomas Huth:
> Recent versions of ld complain when linking the s390-ccw bios:
> 
>   /usr/bin/ld: warning: start.o: missing .note.GNU-stack section implies
>                executable stack
>   /usr/bin/ld: NOTE: This behaviour is deprecated and will be removed in
>                a future version of the linker
> 
> We can silence the warning by telling the linker to mark the stack
> as not executable.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   pc-bios/s390-ccw/Makefile | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/pc-bios/s390-ccw/Makefile b/pc-bios/s390-ccw/Makefile
> index 2e8cc015aa..acfcd1e71a 100644
> --- a/pc-bios/s390-ccw/Makefile
> +++ b/pc-bios/s390-ccw/Makefile
> @@ -55,7 +55,7 @@ config-cc.mak: Makefile
>   	    $(call cc-option,-march=z900,-march=z10)) 3> config-cc.mak
>   -include config-cc.mak
>   
> -LDFLAGS += -Wl,-pie -nostdlib
> +LDFLAGS += -Wl,-pie -nostdlib -z noexecstack
>   
>   build-all: s390-ccw.img s390-netboot.img

In the end this should not matter as the resulting binary is not loaded by an
elf loader so this should be fine
Acked-by: Christian Borntraeger <borntraeger@linux.ibm.com>
diff mbox series

Patch

diff --git a/pc-bios/s390-ccw/Makefile b/pc-bios/s390-ccw/Makefile
index 2e8cc015aa..acfcd1e71a 100644
--- a/pc-bios/s390-ccw/Makefile
+++ b/pc-bios/s390-ccw/Makefile
@@ -55,7 +55,7 @@  config-cc.mak: Makefile
 	    $(call cc-option,-march=z900,-march=z10)) 3> config-cc.mak
 -include config-cc.mak
 
-LDFLAGS += -Wl,-pie -nostdlib
+LDFLAGS += -Wl,-pie -nostdlib -z noexecstack
 
 build-all: s390-ccw.img s390-netboot.img