Skip to content

Commit d1781f2

Browse files
mikeympe
authored andcommitted
selftests/powerpc: Make alignment handler test P9N DD2.1 vector CI load workaround
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 5080332 ("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
1 parent 1da4a02 commit d1781f2

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

tools/testing/selftests/powerpc/alignment/alignment_handler.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -266,8 +266,12 @@ int do_test(char *test_name, void (*test_func)(char *, char *))
266266
}
267267

268268
rc = 0;
269-
/* offset = 0 no alignment fault, so skip */
270-
for (offset = 1; offset < 16; offset++) {
269+
/*
270+
* offset = 0 is aligned but tests the workaround for the P9N
271+
* DD2.1 vector CI load issue (see 5080332c2c89 "powerpc/64s:
272+
* Add workaround for P9 vector CI load issue")
273+
*/
274+
for (offset = 0; offset < 16; offset++) {
271275
width = 16; /* vsx == 16 bytes */
272276
r = 0;
273277

0 commit comments

Comments
 (0)