Patchwork [testsuite] : Do not run simulate-thread on alpha*-*-linux*

login
register
mail settings
Submitter Uros Bizjak
Date Nov. 11, 2011, 5:44 p.m.
Message ID <CAFULd4bb7QNNCBvgzDre9=AARKwwZcCNsN5PO0edez7fOgYs_Q@mail.gmail.com>
Download mbox | patch
Permalink /patch/125237/
State New
Headers show

Comments

Uros Bizjak - Nov. 11, 2011, 5:44 p.m.
Hello!

For some reason, single-stepping executable between ldl_l and stl_c
insns in gdb [1] breaks LL/SC chaining, so atomic operations never
finish. This calls for gdb bugreport.

Also taking into account that dejagnu timeout didn't trigger for
unattended testsuite run and considering huge amount of log
information in testsuite log logged during gdb session, this
all-together was quite devastating to disk space...

We can simply claim that gdb on alpha*-*-linux* is unusable for the
purpose of thread-simulate tests.

2011-11-11  Uros Bizjak  <ubizjak@gmail.com>

	* lib/gcc-simulate-thread.exp (simulate-thread): Do not run on
	alpha*-*-linux* targets.

Tested on alphaev68-pc-linux-gnu.

OK for mainline SVN?

[1] GNU gdb (Gentoo 7.3.1 p1) 7.3.1

Uros.
Richard Henderson - Nov. 11, 2011, 10:09 p.m.
On 11/11/2011 09:44 AM, Uros Bizjak wrote:
> 2011-11-11  Uros Bizjak  <ubizjak@gmail.com>
> 
> 	* lib/gcc-simulate-thread.exp (simulate-thread): Do not run on
> 	alpha*-*-linux* targets.

Ok.

r~

Patch

Index: lib/gcc-simulate-thread.exp
===================================================================
--- lib/gcc-simulate-thread.exp	(revision 181284)
+++ lib/gcc-simulate-thread.exp	(working copy)
@@ -22,6 +22,11 @@ 
 # Call 'fail' if a given test printed "FAIL:", otherwise call 'pass'.
 
 proc simulate-thread { args } {
+
+    # ??? Exit immediately if this is alpha*-*-linux* target, single-stepping
+    # executable between ldl_l and stl_c insns in gdb breaks LL/SC chaining.
+    if { [istarget alpha*-*-linux*] } { return }
+
     if { ![isnative] || [is_remote target] } { return }
 
     if { [llength $args] == 1 } {