Re: [PATCH] Testsuite: Ensure pie is disabled on some tests

[Simon Marchi <simark@simark.ca>]

On 2019-03-06 5:20 a.m., Alan Hayward wrote:
> Recent versions of Ubuntu and Debian default GCC to enable pie.
> 
> In dump.exp, pie will causes addresses to be out of range for IHEX.
> 
> In break-interp.exp, pie is explicitly set for some tests and assumed
> to be disabled for the remainder.
> 
> Ensure pie is disabled for these tests when required.
> 
> gdb/testsuite/ChangeLog:
> 
> 2019-03-06  Alan Hayward  <alan.hayward@arm.com>
> 
> 	* gdb.base/break-interp.exp: Ensure pie is disabled.
> 	* gdb.base/dump.exp: Likewise.

Hi Alan,

The "nopie" flag to gdb_compile has been introduced recently for this, 
could you use that instead?  See commit 6e8b1ab2fd4c ("Fix various tests 
to use -no-pie linker flag when needed").

> ---
>   gdb/testsuite/gdb.base/break-interp.exp | 4 ++++
>   gdb/testsuite/gdb.base/dump.exp         | 4 ++++
>   2 files changed, 8 insertions(+)
> 
> diff --git a/gdb/testsuite/gdb.base/break-interp.exp b/gdb/testsuite/gdb.base/break-interp.exp
> index f85e8a650a..409c31e568 100644
> --- a/gdb/testsuite/gdb.base/break-interp.exp
> +++ b/gdb/testsuite/gdb.base/break-interp.exp
> @@ -627,6 +627,10 @@ foreach ldprelink {NO YES} {
>   			    if {$binpie != "NO"} {
>   				lappend opts {additional_flags=-fPIE}
>   				lappend opts {ldflags=-pie}
> +			    } else {
> +				# Recent Debian/Ubuntu defaults PIE enabled. Ensure it is disabled.
> +				lappend opts {additional_flags=-no-pie}
> +				lappend opts {ldflags=-no-pie}
>   			    }

I believe it won't be necessary here to change CFLAGS (which 
additional_flags does), only ldflags, to make the final executable a 
non-PIE.  Anyway, you'll probably end up using the "nopie" gdb_compile flag.

>   
>   			    set dir ${exec}.d
> diff --git a/gdb/testsuite/gdb.base/dump.exp b/gdb/testsuite/gdb.base/dump.exp
> index 44b0988b80..b2d7ef938a 100644
> --- a/gdb/testsuite/gdb.base/dump.exp
> +++ b/gdb/testsuite/gdb.base/dump.exp
> @@ -36,6 +36,10 @@ if {[istarget "spu*-*-*"]} then {
>       set is64bitonly "yes"
>   }
>   
> +# Ensure -pie is disabled as this breaks IHEX tests.

Can you just precise how it breaks the IHEX tests, as you did in the 
commit message (it causes the addresses to be out of range for that format)?

This patch is ok with those fixed, thanks!

Simon