Message ID | 20241106130453.1741013-1-mpe@ellerman.id.au (mailing list archive) |
---|---|
State | Accepted |
Commit | 3b9bde403aafa55dcbe7dc250b95af917610f139 |
Headers | show |
Series | [1/5] selftests/powerpc: Lower run time of count_stcx_fail test | expand |
On Thu, 07 Nov 2024 00:04:49 +1100, Michael Ellerman wrote: > The count_stcx_fail test runs for close to or just over 2 minutes, which > means it sometimes times out. > > That's overkill for a test that just demonstrates some PMU counters > are working. Drop the 64 billion instruction case, to lower the runtime > to ~30s. > > [...] Applied to powerpc/next. [1/5] selftests/powerpc: Lower run time of count_stcx_fail test https://git.kernel.org/powerpc/c/3b9bde403aafa55dcbe7dc250b95af917610f139 [2/5] selftests/powerpc: Give all tests 2 minutes timeout https://git.kernel.org/powerpc/c/5543d595954eefb3a6faa18a6dc7b1b3d6022052 [3/5] selftests/powerpc: Fix 32-bit BE build errors on Ubuntu 24.04 https://git.kernel.org/powerpc/c/d5f578f90a34d85f1cabd4c27af1b2d9fbffe64b [4/5] selftests/powerpc: Return errors from all tests https://git.kernel.org/powerpc/c/c6a75555b4b2643365a007b7162a670d69aa28fe [5/5] selftests/powerpc: Detect taint change in mitigation patching test https://git.kernel.org/powerpc/c/a8a54a65cac4f8202df36f925b6746328802d05f cheers
diff --git a/tools/testing/selftests/powerpc/pmu/count_stcx_fail.c b/tools/testing/selftests/powerpc/pmu/count_stcx_fail.c index 2070a1e2b3a5..d8dd9a9c6c1b 100644 --- a/tools/testing/selftests/powerpc/pmu/count_stcx_fail.c +++ b/tools/testing/selftests/powerpc/pmu/count_stcx_fail.c @@ -144,9 +144,6 @@ static int test_body(void) /* Run for 16Bi instructions */ FAIL_IF(do_count_loop(events, 16000000000, overhead, true)); - /* Run for 64Bi instructions */ - FAIL_IF(do_count_loop(events, 64000000000, overhead, true)); - event_close(&events[0]); event_close(&events[1]);
The count_stcx_fail test runs for close to or just over 2 minutes, which means it sometimes times out. That's overkill for a test that just demonstrates some PMU counters are working. Drop the 64 billion instruction case, to lower the runtime to ~30s. Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> --- tools/testing/selftests/powerpc/pmu/count_stcx_fail.c | 3 --- 1 file changed, 3 deletions(-)