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

Dmitry V. Levin ldv at altlinux.org
Wed Jul 10 13:53:01 UTC 2019


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.


-- 
ldv
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20190710/47af1d2f/attachment.bin>


More information about the Strace-devel mailing list