Hi,
On 12/28/23 19:23, Kevin Brodsky wrote:
Most #ifdef CONFIG_COMPAT in arch/arm64 really are about 32-bit specifically. Many have been converted already, but a few remain: take care of those.
This change is mostly non-functional, with the following exception:
- arch_parse_elf_property() was incorrectly skipped in compat64. This had no impact on Morello as the only allocated property if for BTI, which is not implemented on Morello.
Signed-off-by: Kevin Brodsky kevin.brodsky@arm.com
arch/arm64/include/asm/elf.h | 4 ++-- arch/arm64/include/asm/fpsimd.h | 2 +- arch/arm64/include/asm/mmu.h | 2 +- arch/arm64/include/asm/processor.h | 8 ++++---- arch/arm64/include/asm/ptrace.h | 2 +- arch/arm64/include/asm/seccomp.h | 2 +- arch/arm64/include/asm/unistd.h | 6 +++++- arch/arm64/kernel/entry-common.c | 6 +++--- arch/arm64/kernel/traps.c | 4 ++-- arch/arm64/mm/init.c | 2 +- 10 files changed, 21 insertions(+), 17 deletions(-)
I did a quick search of CONFIG_COMPAT. All the changes look fine to me and probably below code can also be included.
diff --git a/arch/arm64/kernel/syscall.c b/arch/arm64/kernel/syscall.c index 0feb55df863f..2116ee0bb7e2 100644 --- a/arch/arm64/kernel/syscall.c +++ b/arch/arm64/kernel/syscall.c @@ -204,7 +204,7 @@ void do_el0_svc(struct pt_regs *regs) el0_svc_common(regs, regs->regs[8], __NR_syscalls, table); }
-#ifdef CONFIG_COMPAT +#ifdef CONFIG_COMPAT32 void do_el0_svc_compat(struct pt_regs *regs) { el0_svc_common(regs, regs->regs[7], __NR_compat_syscalls,
Thanks, Amit
diff --git a/arch/arm64/include/asm/elf.h b/arch/arm64/include/asm/elf.h index 4c556ac15e04..b54070e80867 100644 --- a/arch/arm64/include/asm/elf.h +++ b/arch/arm64/include/asm/elf.h @@ -186,7 +186,7 @@ extern int purecap_setup_additional_pages(struct linux_binprm *bprm, #endif /* CONFIG_CHERI_PURECAP_UABI */