[PATCH 2/2] tests/threads-execve: force clock_nanosleep syscall
Alyssa Ross
hi at alyssa.is
Wed May 5 01:48:18 UTC 2021
On Wed, May 05, 2021 at 03:21:46AM +0300, Dmitry V. Levin wrote:
> On Tue, May 04, 2021 at 05:57:43PM +0000, Alyssa Ross wrote:
> > Using clock_nanosleep here means that these sleeps won't show up in
> > the test result, which only monitors for nanosleep. But on Musl,
> > clock_nanosleep(CLOCK_REALTIME, 0, ...) will use the nanosleep
> > syscall, so on Musl these extra sleeps would appear in the test output
> > and break the tests.
> > ---
> > As an aside, by default if they fail, threads-execve* tests will be
> > rerun repeatedly for 1.2 hours, in the hope that they will pass. It
>
> There is an environment variable TIMEOUT_DURATION that controls how long
> each test is allowed to run. It's default value is 1500 (seconds), and
> it's implemented using timeout(1). Please make sure your environment
> provides timeout(1), otherwise TIMEOUT_DURATION has no effect.
Yeah, that's what I was talking about, although I got my maths wrong.
1500 seconds is 24 minutes. These tests retry for half of
TIMEOUT_DURATION, so they keep retrying for 12 minutes each. That seems
like an absurdly long time to keep retrying for, especially since
running the test once takes less than a second (for me at least), so by
the time that timeout elapses, they'll have retried hundreds of times.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20210505/805dadb2/attachment.bin>
More information about the Strace-devel
mailing list