diff mbox series

[1/5] selftests/powerpc: Lower run time of count_stcx_fail test

Message ID 20241106130453.1741013-1-mpe@ellerman.id.au
State New
Headers show
Series [1/5] selftests/powerpc: Lower run time of count_stcx_fail test | expand

Commit Message

Michael Ellerman Nov. 6, 2024, 1:04 p.m. UTC
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(-)

Comments

Michael Ellerman Nov. 17, 2024, 12:09 p.m. UTC | #1
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 mbox series

Patch

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]);