Re: [PATCH 3/3] Add test for fetching TLS from core file

["Maciej W. Rozycki" <macro@mips.com>]

On Thu, 19 Oct 2017, Simon Marchi wrote:

> >  Also `while (1) sleep (10);' perhaps, so that you don't cause a race with
> > the whole process terminating (I think `pthread_join (&threads[0], NULL);'
> > would do too; I'm sure there are other ways as well to prevent from
> > running to exit(2)).  Overall please don't assume any particular thread
> > scheduling characteristics of the underlying OS, and especially on SMP
> > systems effects may be sometimes surprising.
> 
> Just make sure to avoid an infinite loop, in case the inferior is somehow left
> running after executing the test because of some bug somewhere.

 You can't guarantee overall a test program won't hang, loop, etc. due to 
a bug somewhere.  The issue of lingering processes should IMHO be handled 
by assigning software run on the target a new session and then killing the 
process group on completion.  I believe this is actually done already, as 
this is how expect's `spawn' works and I think we also have it handled for 
the remote case in the `gdbserver' test harness (but I'd love to get a 
confirmation here).

 How do you propose anyway, in the absence of an infinite wait/loop, to 
ensure that the process does not terminate before the test harness has 
done its processing?

  Maciej