Re: [PATCH] selftests/kselftest/runner.sh: Add 45 second timeout per test

From: Kees Cook
Date: Thu Sep 19 2019 - 16:58:57 EST


On Thu, Sep 19, 2019 at 08:49:14PM +0000, Tim.Bird@xxxxxxxx wrote:
> > > +export timeout_rc=124
> > what are the units here? I presume seconds?
>
> Nevermind. I misread this. This is the return code from the 'timeout' program, right?

Correct -- given the skip_rc usage, it felt weird to hard-code it in the
if/else statements.

As for the "45" I chose, that was just to be "more than 30" which was
the per-sub-test for tests built using kselftest_harness.h.

--
Kees Cook