diff options
| author | Michael Neuling <mikey@neuling.org> | 2020-10-13 15:37:41 +1100 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2020-10-20 08:00:42 +1100 |
| commit | d1781f23704707d350b8c9006e2bdf5394bf91b2 (patch) | |
| tree | 5ed3239645c68bbfd529011ca095f477d6b78233 /tools | |
| parent | 1da4a0272c5469169f78cd76cf175ff984f52f06 (diff) | |
| download | lwn-d1781f23704707d350b8c9006e2bdf5394bf91b2.tar.gz lwn-d1781f23704707d350b8c9006e2bdf5394bf91b2.zip | |
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 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
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/testing/selftests/powerpc/alignment/alignment_handler.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/tools/testing/selftests/powerpc/alignment/alignment_handler.c b/tools/testing/selftests/powerpc/alignment/alignment_handler.c index 2a0503bc7e49..cb53a8b777e6 100644 --- a/tools/testing/selftests/powerpc/alignment/alignment_handler.c +++ b/tools/testing/selftests/powerpc/alignment/alignment_handler.c @@ -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; |