| Message ID | 20200225101809.9986-1-po-hsu.lin@canonical.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | selftests/timers: Turn off timeout setting | expand |
On 2/25/20 5:18 AM, Po-Hsu Lin wrote: > Tests in timers especially nsleep-lat, set-timer-lat, > inconsistency-check and raw_skew these 4 tests can take longer than > the default 45 seconds that introduced in commit 852c8cbf > (selftests/kselftest/runner.sh: Add 45 second timeout per test) to run. > > Disable the timeout setting for timers instead of looking for an proper > value to make it more general. > > Signed-off-by: Po-Hsu Lin <po-hsu.lin@canonical.com> > --- > tools/testing/selftests/timers/settings | 1 + > 1 file changed, 1 insertion(+) > create mode 100644 tools/testing/selftests/timers/settings > > diff --git a/tools/testing/selftests/timers/settings b/tools/testing/selftests/timers/settings > new file mode 100644 > index 0000000..e7b9417 > --- /dev/null > +++ b/tools/testing/selftests/timers/settings > @@ -0,0 +1 @@ > +timeout=0 > Hi, Michael recently posted a patch that adds the various selftest settings files to the TEST_FILES list, ensuring that they get copied to the install directory along with the tests: https://lore.kernel.org/linux-kselftest/20200220044241.2878-1-mpe@ellerman.id.au/ I think if you add that to the corresponding Makefile, you should be set. Regards, -- Joe
On 2/25/20 3:18 AM, Po-Hsu Lin wrote: > Tests in timers especially nsleep-lat, set-timer-lat, > inconsistency-check and raw_skew these 4 tests can take longer than > the default 45 seconds that introduced in commit 852c8cbf > (selftests/kselftest/runner.sh: Add 45 second timeout per test) to run. > > Disable the timeout setting for timers instead of looking for an proper > value to make it more general. > > Signed-off-by: Po-Hsu Lin <po-hsu.lin@canonical.com> > --- > tools/testing/selftests/timers/settings | 1 + > 1 file changed, 1 insertion(+) > create mode 100644 tools/testing/selftests/timers/settings > > diff --git a/tools/testing/selftests/timers/settings b/tools/testing/selftests/timers/settings > new file mode 100644 > index 0000000..e7b9417 > --- /dev/null > +++ b/tools/testing/selftests/timers/settings > @@ -0,0 +1 @@ > +timeout=0 > What happens when you disable this and set it to 0? How long does the rest run in this case? Can you add the details to the change log please. thanks, -- Shuah
Sure, I will send V2 that include the change for makefile and more detail about the change. Thanks for the feedback, On Tue, Feb 25, 2020 at 11:30 PM shuah <shuah@kernel.org> wrote: > > On 2/25/20 3:18 AM, Po-Hsu Lin wrote: > > Tests in timers especially nsleep-lat, set-timer-lat, > > inconsistency-check and raw_skew these 4 tests can take longer than > > the default 45 seconds that introduced in commit 852c8cbf > > (selftests/kselftest/runner.sh: Add 45 second timeout per test) to run. > > > > Disable the timeout setting for timers instead of looking for an proper > > value to make it more general. > > > > Signed-off-by: Po-Hsu Lin <po-hsu.lin@canonical.com> > > --- > > tools/testing/selftests/timers/settings | 1 + > > 1 file changed, 1 insertion(+) > > create mode 100644 tools/testing/selftests/timers/settings > > > > diff --git a/tools/testing/selftests/timers/settings b/tools/testing/selftests/timers/settings > > new file mode 100644 > > index 0000000..e7b9417 > > --- /dev/null > > +++ b/tools/testing/selftests/timers/settings > > @@ -0,0 +1 @@ > > +timeout=0 > > > > What happens when you disable this and set it to 0? How long does the > rest run in this case? Can you add the details to the change log please. > > thanks, > -- Shuah
diff --git a/tools/testing/selftests/timers/settings b/tools/testing/selftests/timers/settings new file mode 100644 index 0000000..e7b9417 --- /dev/null +++ b/tools/testing/selftests/timers/settings @@ -0,0 +1 @@ +timeout=0
Tests in timers especially nsleep-lat, set-timer-lat, inconsistency-check and raw_skew these 4 tests can take longer than the default 45 seconds that introduced in commit 852c8cbf (selftests/kselftest/runner.sh: Add 45 second timeout per test) to run. Disable the timeout setting for timers instead of looking for an proper value to make it more general. Signed-off-by: Po-Hsu Lin <po-hsu.lin@canonical.com> --- tools/testing/selftests/timers/settings | 1 + 1 file changed, 1 insertion(+) create mode 100644 tools/testing/selftests/timers/settings