2024-02-20 00:03:12

by SeongJae Park

Subject: [PATCH v2] 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 100 seconds.

Link: https://lore.kernel.org/r/[email protected]
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 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-20 00:06:27

by SeongJae Park

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

On Mon, 19 Feb 2024 16:02:43 -0800 SeongJae Park <[email protected]> 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 100 seconds.

Oops, I forgot updating this message.

>
> Link: https://lore.kernel.org/r/[email protected]

Also, this Link: line should be removed.

I will send v3 with above fixes.


Thanks,
SJ

> 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 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