mirror of https://gitee.com/openkylin/linux.git
selftests/powerpc: Fixup clobbers for TM tests
Some of our TM (Transactional Memory) tests, list "r1" (the stack pointer) as a clobbered register. GCC >= 9 doesn't accept this, and the build breaks: ptrace-tm-spd-tar.c: In function 'tm_spd_tar': ptrace-tm-spd-tar.c:31:2: error: listing the stack pointer register 'r1' in a clobber list is deprecated [-Werror=deprecated] 31 | asm __volatile__( | ^~~ ptrace-tm-spd-tar.c:31:2: note: the value of the stack pointer after an 'asm' statement must be the same as it was before the statement We do have some fairly large inline asm blocks in these tests, and some of them do change the value of r1. However they should all return to C with the value in r1 restored, so I think it's legitimate to say r1 is not clobbered. As Segher points out, the r1 clobbers may have been added because of the use of `or 1,1,1`, however that doesn't actually clobber r1. Segher also points out that some of these tests do clobber LR, because they call functions, and that is not listed in the clobbers, so add that where appropriate. Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Link: https://lore.kernel.org/r/20191029095324.14669-1-mpe@ellerman.id.au
This commit is contained in:
parent
d78d5dace5
commit
a02cbc7ffe
|
@ -73,7 +73,7 @@ void tm_spd_tar(void)
|
|||
[sprn_texasr]"i"(SPRN_TEXASR), [tar_1]"i"(TAR_1),
|
||||
[dscr_1]"i"(DSCR_1), [tar_2]"i"(TAR_2), [dscr_2]"i"(DSCR_2),
|
||||
[tar_3]"i"(TAR_3), [dscr_3]"i"(DSCR_3)
|
||||
: "memory", "r0", "r1", "r3", "r4", "r5", "r6"
|
||||
: "memory", "r0", "r3", "r4", "r5", "r6", "lr"
|
||||
);
|
||||
|
||||
/* TM failed, analyse */
|
||||
|
|
|
@ -74,8 +74,8 @@ void tm_spd_vsx(void)
|
|||
"3: ;"
|
||||
: [res] "=r" (result), [texasr] "=r" (texasr)
|
||||
: [sprn_texasr] "i" (SPRN_TEXASR)
|
||||
: "memory", "r0", "r1", "r3", "r4",
|
||||
"r7", "r8", "r9", "r10", "r11"
|
||||
: "memory", "r0", "r3", "r4",
|
||||
"r7", "r8", "r9", "r10", "r11", "lr"
|
||||
);
|
||||
|
||||
if (result) {
|
||||
|
|
|
@ -62,7 +62,7 @@ void tm_tar(void)
|
|||
[sprn_ppr]"i"(SPRN_PPR), [sprn_texasr]"i"(SPRN_TEXASR),
|
||||
[tar_1]"i"(TAR_1), [dscr_1]"i"(DSCR_1), [tar_2]"i"(TAR_2),
|
||||
[dscr_2]"i"(DSCR_2), [cptr1] "b" (&cptr[1])
|
||||
: "memory", "r0", "r1", "r3", "r4", "r5", "r6"
|
||||
: "memory", "r0", "r3", "r4", "r5", "r6"
|
||||
);
|
||||
|
||||
/* TM failed, analyse */
|
||||
|
|
|
@ -62,8 +62,8 @@ void tm_vsx(void)
|
|||
"3: ;"
|
||||
: [res] "=r" (result), [texasr] "=r" (texasr)
|
||||
: [sprn_texasr] "i" (SPRN_TEXASR), [cptr1] "b" (&cptr[1])
|
||||
: "memory", "r0", "r1", "r3", "r4",
|
||||
"r7", "r8", "r9", "r10", "r11"
|
||||
: "memory", "r0", "r3", "r4",
|
||||
"r7", "r8", "r9", "r10", "r11", "lr"
|
||||
);
|
||||
|
||||
if (result) {
|
||||
|
|
Loading…
Reference in New Issue