[PATCH v7 4/4] tests: check status qualifier

Paul Chaignon paul.chaignon at gmail.com
Wed Jul 10 13:57:52 UTC 2019


On Wed, Jul 10, 2019 at 04:53:01PM +0300, Dmitry V. Levin wrote:
> On Wed, Jul 10, 2019 at 03:21:21AM +0300, Dmitry V. Levin wrote:
> > On Tue, Jul 09, 2019 at 02:36:41PM +0200, Paul Chaignon wrote:
> > > On Mon, Jul 08, 2019 at 03:16:49PM +0300, Dmitry V. Levin wrote:
> > > > On Sat, Jul 06, 2019 at 09:20:15AM +0200, Paul Chaignon wrote:
> > > 
> > > [...]
> > > 
> > > > btw, why do you need two different kinds of sleep in the child,
> > > > and why do you need two nanosleep invocations in the child at all?
> > > >
> > > > This is -e status=none test, strace doesn't print any of these syscalls,
> > > > all you need is to let the parent enough time to fall into deep sleep
> > > > before invoking execve.
> > > 
> > > You're right.  They're remnants from threads-execve.c.  I've changed
> > > clock_nanosleep() into nanosleep() and removed the second sleep in the
> > > child, in both status-none.c and status-unfinished.c.
> > 
> > If you don't use clock_nanosleep in status-none and status-unfinished,
> > then you don't need -lpthread $(clock_LIBS) to link them.
> 
> Although if there is a pthread_create, it needs -lpthread anyway,
> sorry for confusion.

Yep, I thought that was a typo and kept the -lpthread.

Paul


More information about the Strace-devel mailing list