Skip to content

Commit 605df8d

Browse files
cyrilbur-ibmmpe
authored andcommitted
selftests/powerpc: Replace stxvx and lxvx with stxvd2x/lxvd2x
On POWER8 (ISA 2.07) lxvx and stxvx are defined to be extended mnemonics of lxvd2x and stxvd2x. For POWER9 (ISA 3.0) the HW architects in their infinite wisdom made lxvx and stxvx instructions in their own right. POWER9 aware GCC will use the POWER9 instruction for lxvx and stxvx causing these selftests to fail on POWER8. Further compounding the issue, because of the way -mvsx works it will cause the power9 instructions to be used regardless of -mcpu=power8 to GCC or -mpower8 to AS. The safest way to address the problem for now is to not use the extended mnemonic. We don't care how the CPU loads the values from memory since the tests only performs register comparisons, so using stdvd2x/lxvd2x does not impact the test. Signed-off-by: Cyril Bur <cyrilbur@gmail.com> Acked-by: Balbir Singh<bsingharora@gmail.com> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
1 parent 78b4416 commit 605df8d

File tree

1 file changed

+24
-24
lines changed
  • tools/testing/selftests/powerpc/include

1 file changed

+24
-24
lines changed

tools/testing/selftests/powerpc/include/vsx_asm.h

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -16,56 +16,56 @@
1616
*/
1717
FUNC_START(load_vsx)
1818
li r5,0
19-
lxvx vs20,r5,r3
19+
lxvd2x vs20,r5,r3
2020
addi r5,r5,16
21-
lxvx vs21,r5,r3
21+
lxvd2x vs21,r5,r3
2222
addi r5,r5,16
23-
lxvx vs22,r5,r3
23+
lxvd2x vs22,r5,r3
2424
addi r5,r5,16
25-
lxvx vs23,r5,r3
25+
lxvd2x vs23,r5,r3
2626
addi r5,r5,16
27-
lxvx vs24,r5,r3
27+
lxvd2x vs24,r5,r3
2828
addi r5,r5,16
29-
lxvx vs25,r5,r3
29+
lxvd2x vs25,r5,r3
3030
addi r5,r5,16
31-
lxvx vs26,r5,r3
31+
lxvd2x vs26,r5,r3
3232
addi r5,r5,16
33-
lxvx vs27,r5,r3
33+
lxvd2x vs27,r5,r3
3434
addi r5,r5,16
35-
lxvx vs28,r5,r3
35+
lxvd2x vs28,r5,r3
3636
addi r5,r5,16
37-
lxvx vs29,r5,r3
37+
lxvd2x vs29,r5,r3
3838
addi r5,r5,16
39-
lxvx vs30,r5,r3
39+
lxvd2x vs30,r5,r3
4040
addi r5,r5,16
41-
lxvx vs31,r5,r3
41+
lxvd2x vs31,r5,r3
4242
blr
4343
FUNC_END(load_vsx)
4444

4545
FUNC_START(store_vsx)
4646
li r5,0
47-
stxvx vs20,r5,r3
47+
stxvd2x vs20,r5,r3
4848
addi r5,r5,16
49-
stxvx vs21,r5,r3
49+
stxvd2x vs21,r5,r3
5050
addi r5,r5,16
51-
stxvx vs22,r5,r3
51+
stxvd2x vs22,r5,r3
5252
addi r5,r5,16
53-
stxvx vs23,r5,r3
53+
stxvd2x vs23,r5,r3
5454
addi r5,r5,16
55-
stxvx vs24,r5,r3
55+
stxvd2x vs24,r5,r3
5656
addi r5,r5,16
57-
stxvx vs25,r5,r3
57+
stxvd2x vs25,r5,r3
5858
addi r5,r5,16
59-
stxvx vs26,r5,r3
59+
stxvd2x vs26,r5,r3
6060
addi r5,r5,16
61-
stxvx vs27,r5,r3
61+
stxvd2x vs27,r5,r3
6262
addi r5,r5,16
63-
stxvx vs28,r5,r3
63+
stxvd2x vs28,r5,r3
6464
addi r5,r5,16
65-
stxvx vs29,r5,r3
65+
stxvd2x vs29,r5,r3
6666
addi r5,r5,16
67-
stxvx vs30,r5,r3
67+
stxvd2x vs30,r5,r3
6868
addi r5,r5,16
69-
stxvx vs31,r5,r3
69+
stxvd2x vs31,r5,r3
7070
blr
7171
FUNC_END(store_vsx)

0 commit comments

Comments
 (0)