diff mbox series

[PATCHv2] Fix sim/testsuite/bpf due to linker warnings

Message ID 20230104214109.51006-1-guillermo.e.martinez@oracle.com
State Accepted
Headers
Series [PATCHv2] Fix sim/testsuite/bpf due to linker warnings |

Checks

Context Check Description
snail/binutils-gdb-check success Github commit url

Commit Message

Guillermo E. Martinez Jan. 4, 2023, 9:41 p.m. UTC 
  Hello,

This is the patch-v2 to fix the bug:
  https://sourceware.org/bugzilla/show_bug.cgi?id=29954

Changes from v1:
  + Remove LDFLAGS_FOR_TARGET and its assignment.

Please let know your thoughts. I'll really appreciate them!,
guillermo
--

On a bpf-*-* testsuite fails:
	./ld/ld-new: warning: test has a LOAD segment with RWX permissions

Adjusting `--memory-size=10Mb' to the simulator bpf testsuite passes.

Tested on bpf-*-*:

Bug 29954: https://sourceware.org/bugzilla/show_bug.cgi?id=29954

sim/testsuite:
	* bpf/allinsn.exp (SIMFLAGS_FOR_TARGET): Adjust sim flags.
---
 sim/testsuite/bpf/allinsn.exp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Mike Frysinger Jan. 5, 2023, 1:54 a.m. UTC | #1 
thanks, verified it worked for me, tweaked the commit message, and pushed
-mike
diff mbox series

Patch

diff --git a/sim/testsuite/bpf/allinsn.exp b/sim/testsuite/bpf/allinsn.exp
index 98f0346776a..1da0b9756c7 100644
--- a/sim/testsuite/bpf/allinsn.exp
+++ b/sim/testsuite/bpf/allinsn.exp
@@ -6,10 +6,7 @@  sim_init
 set all_machs "bpf"
 
 global SIMFLAGS_FOR_TARGET
-set SIMFLAGS_FOR_TARGET "--memory-size=4Mb"
-
-global LDFLAGS_FOR_TARGET
-set LDFLAGS_FOR_TARGET "-Ttext=0x0"
+set SIMFLAGS_FOR_TARGET "--memory-size=10Mb"
 
 foreach src [lsort [glob -nocomplain $srcdir/$subdir/*.s]] {
     # If we're only testing specific files and this isn't one of them, skip it.