APPLIED/cmt: [F][SRU][PATCH 0/2] selftests/powerpc: Turn off timeout setting for benchmarks, dscr, signal, tm
Seth Forshee
seth.forshee at canonical.com
Tue Apr 7 19:44:31 UTC 2020
On Mon, Apr 06, 2020 at 01:57:43PM +0800, Po-Hsu Lin wrote:
> == SRU Justification ==
> Some specific tests in powerpc can take longer than the default 45
> seconds that added in commit 852c8cbf34d3 ("selftests/kselftest/runner.sh:
> Add 45 second timeout per test") to run, the following test result was
> collected across 2 Power8 nodes and 1 Power9 node in our pool:
> powerpc/benchmarks/futex_bench - 52s
> powerpc/dscr/dscr_sysfs_test - 116s
> powerpc/signal/signal_fuzzer - 88s
> powerpc/tm/tm_unavailable_test - 168s
> powerpc/tm/tm-poison - 240s
>
> Thus they will fail with TIMEOUT error.
>
> == Fix ==
> * 850507f3 ("selftests/powerpc: Turn off timeout setting for benchmarks, dscr, signal, tm")
>
> Only Eoan and newer kernel will need this fix.
> For Eoan, this patch can be applied with some context adjustment.
> For Focal, there is a SAUCE patch which turned off timeout setting for
> benchmarks and tm that needs to be reverted first, and this patch can
> be applied with some context adjustment too
>
> == Test ==
> Patch tested on PowerPC and it works as expected.
>
> == Regression Potential ==
> Low, changes limited to testing tools for PowerPC.
You forgot your sign-off on the first patch. I added it when applying.
Applied to focal/master-next and unstable/master, thanks!
More information about the kernel-team
mailing list