selftests/powerpc: Make alignment handler test P9N DD2.1 vector CI load workaround
authorMichael Neuling <mikey@neuling.org>
Tue, 13 Oct 2020 04:37:41 +0000 (15:37 +1100)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 19 Oct 2020 21:00:42 +0000 (08:00 +1100)
alignment_handler currently only tests the unaligned cases but it can
also be useful for testing the workaround for the P9N DD2.1 vector CI
load issue fixed by p9_hmi_special_emu(). This workaround was
introduced in 5080332c2c89 ("powerpc/64s: Add workaround for P9 vector
CI load issue").

This changes the loop to start from offset 0 rather than 1 so that we
test the kernel emulation in p9_hmi_special_emu().

Signed-off-by: Michael Neuling <mikey@neuling.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20201013043741.743413-2-mikey@neuling.org
tools/testing/selftests/powerpc/alignment/alignment_handler.c

index 2a0503bc7e492ed301ee18201859ddf1394b720c..cb53a8b777e68a7fc49c5c7ed1d249cb13c0e1d1 100644 (file)
@@ -266,8 +266,12 @@ int do_test(char *test_name, void (*test_func)(char *, char *))
        }
 
        rc = 0;
-       /* offset = 0 no alignment fault, so skip */
-       for (offset = 1; offset < 16; offset++) {
+       /*
+        * offset = 0 is aligned but tests the workaround for the P9N
+        * DD2.1 vector CI load issue (see 5080332c2c89 "powerpc/64s:
+        * Add workaround for P9 vector CI load issue")
+        */
+       for (offset = 0; offset < 16; offset++) {
                width = 16; /* vsx == 16 bytes */
                r = 0;