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

2019-09-20 Thread shuah
On 9/19/19 3:17 PM, Kees Cook wrote: On Thu, Sep 19, 2019 at 02:09:37PM -0600, shuah wrote: On 9/19/19 12:55 PM, Alexandre Belloni wrote: On 19/09/2019 11:06:44-0700, Kees Cook wrote: Commit a745f7af3cbd ("selftests/harness: Add 30 second timeout per test") solves the problem of

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

2019-09-19 Thread Kees Cook
On Thu, Sep 19, 2019 at 02:09:37PM -0600, shuah wrote: > On 9/19/19 12:55 PM, Alexandre Belloni wrote: > > On 19/09/2019 11:06:44-0700, Kees Cook wrote: > > > Commit a745f7af3cbd ("selftests/harness: Add 30 second timeout per > > > test") solves the problem of kselftest_harness.h-using binary

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

2019-09-19 Thread Kees Cook
On Thu, Sep 19, 2019 at 08:41:42PM +, tim.b...@sony.com wrote: > > -Original Message- > > From: Kees Cook > > > > Commit a745f7af3cbd ("selftests/harness: Add 30 second timeout per > > test") solves the problem of kselftest_harness.h-using binary tests > > possibly hanging forever.

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

2019-09-19 Thread Kees Cook
On Thu, Sep 19, 2019 at 08:49:14PM +, tim.b...@sony.com 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

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

2019-09-19 Thread Tim.Bird
> -Original Message- > From tim.b...@sony.com > > > -Original Message- > > From: Kees Cook > > > > Commit a745f7af3cbd ("selftests/harness: Add 30 second timeout per > > test") solves the problem of kselftest_harness.h-using binary tests > > possibly hanging forever. However,

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

2019-09-19 Thread Tim.Bird
> -Original Message- > From: shuah > > On 9/19/19 12:55 PM, Alexandre Belloni wrote: > > On 19/09/2019 11:06:44-0700, Kees Cook wrote: > >> Commit a745f7af3cbd ("selftests/harness: Add 30 second timeout per > >> test") solves the problem of kselftest_harness.h-using binary tests > >>

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

2019-09-19 Thread Tim.Bird
> -Original Message- > From: Kees Cook > > Commit a745f7af3cbd ("selftests/harness: Add 30 second timeout per > test") solves the problem of kselftest_harness.h-using binary tests > possibly hanging forever. However, scripts and other binaries can still > hang forever. This adds a

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

2019-09-19 Thread shuah
On 9/19/19 12:55 PM, Alexandre Belloni wrote: On 19/09/2019 11:06:44-0700, Kees Cook wrote: Commit a745f7af3cbd ("selftests/harness: Add 30 second timeout per test") solves the problem of kselftest_harness.h-using binary tests possibly hanging forever. However, scripts and other binaries can

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

2019-09-19 Thread Alexandre Belloni
On 19/09/2019 11:06:44-0700, Kees Cook wrote: > Commit a745f7af3cbd ("selftests/harness: Add 30 second timeout per > test") solves the problem of kselftest_harness.h-using binary tests > possibly hanging forever. However, scripts and other binaries can still > hang forever. This adds a global