Re: [PATCH 2/2] arm64: use the correct function type in SYSCALL_DEFINE0
From: Sami Tolvanen
Date: Fri May 03 2019 - 13:03:20 EST
Hi Mark,
On Fri, May 03, 2019 at 11:21:28AM +0100, Mark Rutland wrote:
> Generally, this makes sense, but I'm not sure that this is complete.
>
> IIUC this introduces a new type mismatch with sys_ni_syscall() in some
> cases.
Thanks for the review. You're correct, sys_ni_syscall needs to be fixed
too. I'll include this in v2.
> We probably need that to use SYSCALL_DEFINE0(), and maybe have a
> ksys_ni_syscall() for in-kernel wrappers.
Why would we need ksys_ni_syscall? It seems something like this should
be sufficient:
asmlinkage long sys_ni_syscall(void);
SYSCALL_DEFINE0(ni_syscall)
{
return sys_ni_syscall();
}
Sami