Re: [PATCH 1/6] powerpc/syscalls: Switch trivial cases to SYSCALL_DEFINE

From: Michael Ellerman
Date: Sun May 06 2018 - 22:43:01 EST


"Naveen N. Rao" <naveen.n.rao@xxxxxxxxxxxxxxxxxx> writes:

> Michael Ellerman wrote:
>> From: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
>>
>> Signed-off-by: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
>> ---
>> arch/powerpc/kernel/pci_32.c | 6 +++---
>> arch/powerpc/kernel/pci_64.c | 4 ++--
>> arch/powerpc/mm/subpage-prot.c | 4 +++-
>> arch/powerpc/platforms/cell/spu_syscalls.c | 3 ++-
>> 4 files changed, 10 insertions(+), 7 deletions(-)
>>
>
> I suppose we can also do this for switch_endian?

Yeah. It'd nice to have an automated test that every syscall is
available via FTRACE_SYSCALLS, I just haven't got around to writing one.

cheers

> diff --git a/arch/powerpc/kernel/syscalls.c b/arch/powerpc/kernel/syscalls.c
> index 466216506eb2..290265f2700c 100644
> --- a/arch/powerpc/kernel/syscalls.c
> +++ b/arch/powerpc/kernel/syscalls.c
> @@ -123,7 +123,7 @@ long ppc_fadvise64_64(int fd, int advice, u32 offset_high, u32 offset_low,
> (u64)len_high << 32 | len_low, advice);
> }
>
> -long sys_switch_endian(void)
> +SYSCALL_DEFINE0(switch_endian)
> {
> struct thread_info *ti;
>
>
> - Naveen