selftests/powerpc: Don't use setaffinity in tm-tmspr
authorMichael Ellerman <mpe@ellerman.id.au>
Thu, 13 Aug 2020 01:34:44 +0000 (11:34 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 8 Sep 2020 12:23:46 +0000 (22:23 +1000)
This test tries to set affinity to CPUs that don't exist, especially
if the set of online CPUs doesn't start at 0.

But there's no real reason for it to use setaffinity in the first
place, it's just trying to create lots of threads to cause contention.
So drop the setaffinity entirely.

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20200813013445.686464-2-mpe@ellerman.id.au
tools/testing/selftests/powerpc/tm/tm-tmspr.c

index 17becf3..2ff329e 100644 (file)
@@ -38,14 +38,8 @@ int  passed = 1;
 
 void tfiar_tfhar(void *in)
 {
-       int i, cpu;
        unsigned long tfhar, tfhar_rd, tfiar, tfiar_rd;
-       cpu_set_t cpuset;
-
-       CPU_ZERO(&cpuset);
-       cpu = (unsigned long)in >> 1;
-       CPU_SET(cpu, &cpuset);
-       sched_setaffinity(0, sizeof(cpuset), &cpuset);
+       int i;
 
        /* TFIAR: Last bit has to be high so userspace can read register */
        tfiar = ((unsigned long)in) + 1;