selftests/powerpc: Improve tm-resched-dscr
authorSam Bobroff <sam.bobroff@au1.ibm.com>
Thu, 17 Aug 2017 01:06:47 +0000 (11:06 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 17 Aug 2017 11:56:36 +0000 (21:56 +1000)
The tm-resched-dscr self test can, in some situations, run for
several minutes before being successfully interrupted by the context
switch it needs in order to perform the test. This often seems to
occur when the test is being run in a virtual machine.

Improve the test by running it under eat_cpu() to guarantee
contention for the CPU and increase the chance of a context switch.

In practice this seems to reduce the test time, in some cases, from
more than two minutes to under a second.

Also remove the "progress dots" so that if the test does run for a
long time, it doesn't produce large amounts of unnecessary output.

Signed-off-by: Sam Bobroff <sam.bobroff@au1.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
tools/testing/selftests/powerpc/tm/Makefile
tools/testing/selftests/powerpc/tm/tm-resched-dscr.c

index 958c11c14acd017a6c3afe79f02d9c6e8fad2240..7bfcd454fb2aa0e3655522f8fa664a54dc9937c4 100644 (file)
@@ -15,6 +15,7 @@ $(OUTPUT)/tm-syscall: tm-syscall-asm.S
 $(OUTPUT)/tm-syscall: CFLAGS += -I../../../../../usr/include
 $(OUTPUT)/tm-tmspr: CFLAGS += -pthread
 $(OUTPUT)/tm-vmx-unavail: CFLAGS += -pthread -m64
+$(OUTPUT)/tm-resched-dscr: ../pmu/lib.o
 
 SIGNAL_CONTEXT_CHK_TESTS := $(patsubst %,$(OUTPUT)/%,$(SIGNAL_CONTEXT_CHK_TESTS))
 $(SIGNAL_CONTEXT_CHK_TESTS): tm-signal.S
index e79ccd6aada1a9acf10bb2860e76ddb444415d52..a7ac2e4c60d9135d9d939fd06ad41c7d3f18c384 100644 (file)
@@ -30,6 +30,7 @@
 
 #include "utils.h"
 #include "tm.h"
+#include "../pmu/lib.h"
 
 #define SPRN_DSCR       0x03
 
@@ -75,8 +76,6 @@ int test_body(void)
                );
                assert(rv); /* make sure the transaction aborted */
                if ((texasr >> 56) != TM_CAUSE_RESCHED) {
-                       putchar('.');
-                       fflush(stdout);
                        continue;
                }
                if (dscr2 != dscr1) {
@@ -89,7 +88,12 @@ int test_body(void)
        }
 }
 
-int main(void)
+static int tm_resched_dscr(void)
 {
-       return test_harness(test_body, "tm_resched_dscr");
+       return eat_cpu(test_body);
+}
+
+int main(int argc, const char *argv[])
+{
+       return test_harness(tm_resched_dscr, "tm_resched_dscr");
 }