public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] selftests/mqueue: Set timeout to 180 seconds
@ 2024-02-20  0:02 SeongJae Park
  2024-02-20  0:06 ` SeongJae Park
  0 siblings, 1 reply; 2+ messages in thread
From: SeongJae Park @ 2024-02-20  0:02 UTC (permalink / raw)
  To: shuah
  Cc: ryan.roberts, abuehaze, brauner, jlayton, jack, keescook,
	linux-kselftest, linux-kernel, stable, Vijaikumar_Kanagarajan,
	SeongJae Park

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/20240208212925.68286-1-sj@kernel.org
Fixes: 852c8cbf34d3 ("selftests/kselftest/runner.sh: Add 45 second timeout per test")
Cc: <stable@vger.kernel.org> # 5.4.x
Signed-off-by: SeongJae Park <sj@kernel.org>
Reviewed-by: Kees Cook <keescook@chromium.org>
---
Changes from v1
(https://lore.kernel.org/r/20240208212925.68286-1-sj@kernel.org)
- 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


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] selftests/mqueue: Set timeout to 180 seconds
  2024-02-20  0:02 [PATCH v2] selftests/mqueue: Set timeout to 180 seconds SeongJae Park
@ 2024-02-20  0:06 ` SeongJae Park
  0 siblings, 0 replies; 2+ messages in thread
From: SeongJae Park @ 2024-02-20  0:06 UTC (permalink / raw)
  To: SeongJae Park
  Cc: shuah, ryan.roberts, abuehaze, brauner, jlayton, jack, keescook,
	linux-kselftest, linux-kernel, stable, Vijaikumar_Kanagarajan

On Mon, 19 Feb 2024 16:02:43 -0800 SeongJae Park <sj@kernel.org> 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/20240208212925.68286-1-sj@kernel.org

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: <stable@vger.kernel.org> # 5.4.x
> Signed-off-by: SeongJae Park <sj@kernel.org>
> Reviewed-by: Kees Cook <keescook@chromium.org>
> ---
> Changes from v1
> (https://lore.kernel.org/r/20240208212925.68286-1-sj@kernel.org)
> - 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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-02-20  0:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-20  0:02 [PATCH v2] selftests/mqueue: Set timeout to 180 seconds SeongJae Park
2024-02-20  0:06 ` SeongJae Park

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox