APPLIED/cmt: [F][SRU][PATCH 0/2] selftests/powerpc: Turn off timeout setting for benchmarks, dscr, signal, tm

Po-Hsu Lin po-hsu.lin at canonical.com
Wed Apr 8 07:55:22 UTC 2020


Oh indeed I forgot to add -s when reverting that patch.
Thank you!
Sam

On Wed, Apr 8, 2020 at 3:44 AM Seth Forshee <seth.forshee at canonical.com> wrote:
>
> 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