From: Rahul Pathak Date: Sun, 25 Dec 2022 03:22:11 +0000 (+0530) Subject: Makefile: Remove -N ldflag to prevent linker RWX warning X-Git-Tag: accepted/tizen/unified/riscv/20231123.095152~119 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6509127ad620748c90c02b4bd32fa91df165ee82;p=platform%2Fkernel%2Fopensbi.git Makefile: Remove -N ldflag to prevent linker RWX warning -N option coalesce all sections into single LOAD segment which causes data and other sections to have executable permission causing warning with new binutils ld 2.39. New ld emits warning when any segment have all three permissions RWX. ld.bfd: warning: test.elf has a LOAD segment with RWX permissions ld.bfd: warning: fw_dynamic.elf has a LOAD segment with RWX permissions ld.bfd: warning: fw_jump.elf has a LOAD segment with RWX permissions ld.bfd: warning: fw_payload.elf has a LOAD segment with RWX permissions This option was added in below commit - commit: eeab92f2423e ("Makefile: Convert to a more standard format") Removing -N option allows to have text and rodata into one LOAD segment and other sections into separate LOAD segment which prevents RWX permissions on single LOAD segment. Here X == E Current LOAD 0x0000000000000120 0x0000000080000000 0x0000000080000000 0x000000000001d4d0 0x0000000000032ed8 RWE 0x10 -N removed LOAD 0x0000000000001000 0x0000000080000000 0x0000000080000000 0x00000000000198cc 0x00000000000198cc R E 0x1000 LOAD 0x000000000001b000 0x000000008001a000 0x000000008001a000 0x00000000000034d0 0x0000000000018ed8 RW 0x1000 Signed-off-by: Rahul Pathak Reviewed-by: Anup Patel Tested-by: Samuel Holland --- diff --git a/Makefile b/Makefile index 92203c5..b742d6d 100644 --- a/Makefile +++ b/Makefile @@ -369,7 +369,7 @@ ASFLAGS += $(firmware-asflags-y) ARFLAGS = rcs ELFFLAGS += $(USE_LD_FLAG) -ELFFLAGS += -Wl,--build-id=none -Wl,-N +ELFFLAGS += -Wl,--build-id=none ELFFLAGS += $(platform-ldflags-y) ELFFLAGS += $(firmware-ldflags-y)