From: Shaopeng Tan <[email protected]>
[ Upstream commit e2e3fb6ef0d6548defbe0be6e092397aaa92f3a1 ]
When testing on a Intel(R) Xeon(R) Gold 6254 CPU @ 3.10GHz the resctrl
selftests fail due to timeout after exceeding the default time limit of
45 seconds. On this system the test takes about 68 seconds.
Since the failing test by default accesses a fixed size of memory, the
execution time should not vary significantly between different environment.
A new default of 120 seconds should be sufficient yet easy to customize
with the introduction of the "settings" file for reference.
Reviewed-by: Reinette Chatre <[email protected]>
Reviewed-by: Fenghua Yu <[email protected]>
Signed-off-by: Shaopeng Tan <[email protected]>
Signed-off-by: Shuah Khan <[email protected]>
Signed-off-by: Sasha Levin <[email protected]>
---
tools/testing/selftests/resctrl/settings | 3 +++
1 file changed, 3 insertions(+)
create mode 100644 tools/testing/selftests/resctrl/settings
diff --git a/tools/testing/selftests/resctrl/settings b/tools/testing/selftests/resctrl/settings
new file mode 100644
index 000000000000..a383f3d4565b
--- /dev/null
+++ b/tools/testing/selftests/resctrl/settings
@@ -0,0 +1,3 @@
+# If running time is longer than 120 seconds when new tests are added in
+# the future, increase timeout here.
+timeout=120
--
2.35.1
Dear Stable Team,
I received the below notice about this commit being considered for
4.9 as well as other notices for it being considered for 4.14, 4.19,
5.4, 5.10, 5.15, 5.17, and 5.18.
I do not believe this commit is appropriate for stable because:
- It forms part of a series and without the other accompanying patches
from that series it does not do anything. Series is at:
https://lore.kernel.org/lkml/[email protected]/
- The original series was not not submitted for inclusion to stable and
none of its patches have a Fixes: tag.
- The series this patch forms part of aims to port resctrl_tests to the
kselftest framework and I do not believe such a change matches stable
criteria.
Reinette
On 6/1/2022 7:00 AM, Sasha Levin wrote:
> From: Shaopeng Tan <[email protected]>
>
> [ Upstream commit e2e3fb6ef0d6548defbe0be6e092397aaa92f3a1 ]
>
> When testing on a Intel(R) Xeon(R) Gold 6254 CPU @ 3.10GHz the resctrl
> selftests fail due to timeout after exceeding the default time limit of
> 45 seconds. On this system the test takes about 68 seconds.
> Since the failing test by default accesses a fixed size of memory, the
> execution time should not vary significantly between different environment.
> A new default of 120 seconds should be sufficient yet easy to customize
> with the introduction of the "settings" file for reference.
>
> Reviewed-by: Reinette Chatre <[email protected]>
> Reviewed-by: Fenghua Yu <[email protected]>
> Signed-off-by: Shaopeng Tan <[email protected]>
> Signed-off-by: Shuah Khan <[email protected]>
> Signed-off-by: Sasha Levin <[email protected]>
> ---
> tools/testing/selftests/resctrl/settings | 3 +++
> 1 file changed, 3 insertions(+)
> create mode 100644 tools/testing/selftests/resctrl/settings
>
> diff --git a/tools/testing/selftests/resctrl/settings b/tools/testing/selftests/resctrl/settings
> new file mode 100644
> index 000000000000..a383f3d4565b
> --- /dev/null
> +++ b/tools/testing/selftests/resctrl/settings
> @@ -0,0 +1,3 @@
> +# If running time is longer than 120 seconds when new tests are added in
> +# the future, increase timeout here.
> +timeout=120
On Wed, Jun 01, 2022 at 10:19:45AM -0700, Reinette Chatre wrote:
>Dear Stable Team,
>
>I received the below notice about this commit being considered for
>4.9 as well as other notices for it being considered for 4.14, 4.19,
>5.4, 5.10, 5.15, 5.17, and 5.18.
>
>I do not believe this commit is appropriate for stable because:
>- It forms part of a series and without the other accompanying patches
> from that series it does not do anything. Series is at:
> https://lore.kernel.org/lkml/[email protected]/
>- The original series was not not submitted for inclusion to stable and
> none of its patches have a Fixes: tag.
>- The series this patch forms part of aims to port resctrl_tests to the
> kselftest framework and I do not believe such a change matches stable
> criteria.
Sure, I'll drop it, thanks!
--
Thanks,
Sasha