ACK / APPLIED[Unstable]: [F/G/H/Unstable][SRU][PATCH 0/1] selftests/powerpc/eeh: increase timeout for eeh-basic

[Seth Forshee]

On Fri, Dec 18, 2020 at 05:50:13PM +0800, Po-Hsu Lin wrote:
> [Impact]
> The breakable devices test is hardware-dependent. In our test pool
> it will take about:
> * 30 seconds to run on a Power8 system that with 5 breakable devices,
> * 60 seconds to run on a Power9 system that with 4 breakable devices.
> 
> The default 45 seconds kselftest framework timeout is not enough to
> allow this test to finish on some nodes. Thus causing this test to
> fail with TIMEOUT error.
> 
> [Fix]
> * f5eca0b279117f ("selftests/powerpc/eeh: disable kselftest timeout
> setting for eeh-basic")
> 
> We have this testcase since Focal, and this patch can be cherry-picked
> into all affected releases.
> 
> [Test case]
> Run this test on P9 node baltar, on which this timeout issue can be
> 100% reproduced. With this patch applied, the test can finish without
> being terminated by the default timeout.
> 
> [Where problems could occur]
> This will make test takes longer to finish, but it's still being
> controlled by the timeout mechanism both in the test case and
> kselftest framework. It's unlikely to make the test hang forever.
> 
> Po-Hsu Lin (1):
>   selftests/powerpc/eeh: disable kselftest timeout setting for eeh-basic

Acked-by: Seth Forshee <seth.forshee at canonical.com>

Applied to unstable/master, thanks!