2024-02-20 00:08:15

by SeongJae Park

Subject: [PATCH v3] selftests/mqueue: Set timeout to 180 seconds

While mq_perf_tests runs with the default kselftest timeout limit, which
is 45 seconds, the test takes about 60 seconds to complete on i3.metal
AWS instances. Hence, the test always times out. Increase the timeout
to 180 seconds.

Fixes: 852c8cbf34d3 ("selftests/kselftest/runner.sh: Add 45 second timeout per test")
Cc: <[email protected]> # 5.4.x
Signed-off-by: SeongJae Park <[email protected]>
Reviewed-by: Kees Cook <[email protected]>
---
Changes from v2
(https://lore.kernel.org/r/[email protected])
- Update commit message about the new timeout limit to 180 seconds
- Remove wrong Link: line

Changes from v1
(https://lore.kernel.org/r/[email protected])
- Use 180 seconds timeout instead of 100 seconds

tools/testing/selftests/mqueue/setting | 1 +
1 file changed, 1 insertion(+)
create mode 100644 tools/testing/selftests/mqueue/setting

diff --git a/tools/testing/selftests/mqueue/setting b/tools/testing/selftests/mqueue/setting
new file mode 100644
index 000000000000..a953c96aa16e
--- /dev/null
+++ b/tools/testing/selftests/mqueue/setting
@@ -0,0 +1 @@
+timeout=180
--
2.39.2

2024-02-21 00:14:32

by Shuah Khan

Subject: Re: [PATCH v3] selftests/mqueue: Set timeout to 180 seconds

On 2/19/24 17:08, SeongJae Park wrote:
> While mq_perf_tests runs with the default kselftest timeout limit, which
> is 45 seconds, the test takes about 60 seconds to complete on i3.metal
> AWS instances. Hence, the test always times out. Increase the timeout
> to 180 seconds.
>
> Fixes: 852c8cbf34d3 ("selftests/kselftest/runner.sh: Add 45 second timeout per test")
> Cc: <[email protected]> # 5.4.x
> Signed-off-by: SeongJae Park <[email protected]>
> Reviewed-by: Kees Cook <[email protected]>
> ---
> Changes from v2
> (https://lore.kernel.org/r/[email protected])
> - Update commit message about the new timeout limit to 180 seconds
> - Remove wrong Link: line
>
> Changes from v1
> (https://lore.kernel.org/r/[email protected])
> - Use 180 seconds timeout instead of 100 seconds
>
> tools/testing/selftests/mqueue/setting | 1 +
> 1 file changed, 1 insertion(+)
> create mode 100644 tools/testing/selftests/mqueue/setting
>
> diff --git a/tools/testing/selftests/mqueue/setting b/tools/testing/selftests/mqueue/setting
> new file mode 100644
> index 000000000000..a953c96aa16e
> --- /dev/null
> +++ b/tools/testing/selftests/mqueue/setting
> @@ -0,0 +1 @@
> +timeout=180

Applied to next for Linux 6.9-rc1

thanks,
-- Shuah