Re: [PATCH, testsuite]: Fix timeout in simulate-thread.exp

2011-11-17 Thread Andrew MacLeod
On 11/17/2011 02:33 PM, Mike Stump wrote: The patch also sets timeout to 10 seconds, due to huge amount of produced log and improves fail messages a bit. 10 might be a bit low... (slow targets, simulators, complex remote target setup) but we can go with that for now until someone has a nee

Re: [PATCH, testsuite]: Fix timeout in simulate-thread.exp

2011-11-17 Thread Mike Stump
On Nov 17, 2011, at 1:54 AM, Uros Bizjak wrote: > The problem was, that dejagnu didn't kill gdb in case of timeout (i.e. > broken LL/SC sequences for alpha). > The patch also sets timeout to 10 seconds, due to huge amount of > produced log and improves fail messages a bit. 10 might be a bit low..

Re: [PATCH, testsuite]: Fix timeout in simulate-thread.exp

2011-11-17 Thread Iain Sandoe
Hi Uros, On 17 Nov 2011, at 14:02, Uros Bizjak wrote: On Thu, Nov 17, 2011 at 2:22 PM, Iain Sandoe wrote: On 17 Nov 2011, at 10:49, Alan Modra wrote: On Thu, Nov 17, 2011 at 10:54:23AM +0100, Uros Bizjak wrote: The solution for this problem is to search for "simulate_thread_done" in g

Re: [PATCH, testsuite]: Fix timeout in simulate-thread.exp

2011-11-17 Thread Uros Bizjak
On Thu, Nov 17, 2011 at 2:22 PM, Iain Sandoe wrote: > > On 17 Nov 2011, at 10:49, Alan Modra wrote: > >> On Thu, Nov 17, 2011 at 10:54:23AM +0100, Uros Bizjak wrote: >>> >>> The solution for this problem is to search for "simulate_thread_done" >>> in gdb log. >> >> Thanks!  I like this for another

Re: [PATCH, testsuite]: Fix timeout in simulate-thread.exp

2011-11-17 Thread Iain Sandoe
On 17 Nov 2011, at 10:49, Alan Modra wrote: On Thu, Nov 17, 2011 at 10:54:23AM +0100, Uros Bizjak wrote: The solution for this problem is to search for "simulate_thread_done" in gdb log. Thanks! I like this for another reason too: I've seen these tests fail due to gdb never reporting simul

Re: [PATCH, testsuite]: Fix timeout in simulate-thread.exp

2011-11-17 Thread Alan Modra
On Thu, Nov 17, 2011 at 10:54:23AM +0100, Uros Bizjak wrote: > The solution for this problem is to search for "simulate_thread_done" > in gdb log. Thanks! I like this for another reason too: I've seen these tests fail due to gdb never reporting simulate_thread_main on any instruction in that fun

[PATCH, testsuite]: Fix timeout in simulate-thread.exp

2011-11-17 Thread Uros Bizjak
Hello! The problem was, that dejagnu didn't kill gdb in case of timeout (i.e. broken LL/SC sequences for alpha). The cause for this was in the way remote_expect was called. The script found "simulate_thread_main" to assume that gdb works OK, but unfortunately, this overrode the "timeout" default c