powerpc64: Select POWER9 machine for the scv instruction
Commit Message
It is not available with the baseline ISA on powerpc64-linux-gnu.
(powerpc64le-linux-gnu enables it unconditionally.)
Fixes commit 68ab82f56690ada86ac1e0c46bad06ba189a10ef
("powerpc: Runtime selection between sc and scv for syscalls").
---
sysdeps/powerpc/powerpc64/sysdep.h | 3 +++
sysdeps/unix/sysv/linux/powerpc/powerpc64/clone.S | 3 +++
sysdeps/unix/sysv/linux/powerpc/sysdep.h | 5 ++++-
3 files changed, 10 insertions(+), 1 deletion(-)
Comments
Florian Weimer via Libc-alpha <libc-alpha@sourceware.org> writes:
> It is not available with the baseline ISA on powerpc64-linux-gnu.
> (powerpc64le-linux-gnu enables it unconditionally.)
>
> Fixes commit 68ab82f56690ada86ac1e0c46bad06ba189a10ef
> ("powerpc: Runtime selection between sc and scv for syscalls").
Confirmed this patch does build with Binutils 2.29.
LGTM.
Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
I'm updating the installation instructions.
Thanks!
* Tulio Magno Quites Machado Filho via Libc-alpha:
> Florian Weimer via Libc-alpha <libc-alpha@sourceware.org> writes:
>
>> It is not available with the baseline ISA on powerpc64-linux-gnu.
>> (powerpc64le-linux-gnu enables it unconditionally.)
>>
>> Fixes commit 68ab82f56690ada86ac1e0c46bad06ba189a10ef
>> ("powerpc: Runtime selection between sc and scv for syscalls").
>
> Confirmed this patch does build with Binutils 2.29.
> LGTM.
>
> Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
>
> I'm updating the installation instructions.
Thanks. I'll drop the refernence to little endian vs big endian because
this looks like something else after all (maybe a downstream patch
causes things to build for us).
Adhemerval, is this okay to push for the release?
Thanks,
Florian
On 22/01/2021 06:44, Florian Weimer wrote:
> * Tulio Magno Quites Machado Filho via Libc-alpha:
>
>> Florian Weimer via Libc-alpha <libc-alpha@sourceware.org> writes:
>>
>>> It is not available with the baseline ISA on powerpc64-linux-gnu.
>>> (powerpc64le-linux-gnu enables it unconditionally.)
>>>
>>> Fixes commit 68ab82f56690ada86ac1e0c46bad06ba189a10ef
>>> ("powerpc: Runtime selection between sc and scv for syscalls").
>>
>> Confirmed this patch does build with Binutils 2.29.
>> LGTM.
>>
>> Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
>>
>> I'm updating the installation instructions.
>
> Thanks. I'll drop the refernence to little endian vs big endian because
> this looks like something else after all (maybe a downstream patch
> causes things to build for us).
>
> Adhemerval, is this okay to push for the release?
It is ok for 2.33, thanks.
@@ -333,7 +333,10 @@ LT_LABELSUFFIX(name,_name_end): ; \
mflr r9; \
std r9,FRAME_LR_SAVE(r1); \
cfi_offset(lr,FRAME_LR_SAVE); \
+ .machine "push"; \
+ .machine "power9"; \
scv 0; \
+ .machine "pop"; \
ld r9,FRAME_LR_SAVE(r1); \
mtlr r9; \
cfi_restore(lr);
@@ -76,7 +76,10 @@ ENTRY (__clone)
because it uses CFI directives and we just called cfi_endproc. */
mflr r9
std r9,FRAME_LR_SAVE(r1)
+ .machine "push"
+ .machine "power9"
scv 0
+ .machine "pop"
ld r9,FRAME_LR_SAVE(r1)
mtlr r9
@@ -76,7 +76,10 @@
#define SYSCALL_SCV(nr) \
({ \
__asm__ __volatile__ \
- ("scv 0\n\t" \
+ (".machine \"push\"\n\t" \
+ ".machine \"power9\"\n\t" \
+ "scv 0\n\t" \
+ ".machine \"pop\"\n\t" \
"0:" \
: "=&r" (r0), \
"=&r" (r3), "=&r" (r4), "=&r" (r5), \