Re: [PATCH v2] x86/syscalls: Mark expected switch fall-throughs

From: Steven Rostedt
Date: Wed Mar 27 2019 - 19:15:09 EST


On Thu, 28 Mar 2019 02:12:15 +0300
"Dmitry V. Levin" <ldv@xxxxxxxxxxxx> wrote:

> > Seriously. If we keep it can we at least remove all the unused arguments
> > which we have on both functions to simplify the whole mess?
>
> In case of syscall_set_arguments() I think we can safely remove
> "i" and "n" arguments assuming i == 0 and n == 6.
>
> All I can say about syscall_get_arguments() is that
> - all current users invoke it with i == 0,
> - all current users that invoke it with n != 6 are in kernel/trace/trace_syscalls.c
> so it may actually be invoked with n < 6.

Yes, that's what my old (and current) patches address. I removed the
i,n parameters and make it 0,6 hardcoded in the routines.

Patches will be out soon.

-- Steve