Re: [RFC PATCH 13/35] syscalls: do not call sys_ioperm() within the kernel

From: Andy Lutomirski
Date: Sun Mar 11 2018 - 11:20:29 EST


On Sun, Mar 11, 2018 at 10:55 AM, Dominik Brodowski
<linux@xxxxxxxxxxxxxxxxxxxx> wrote:
> While at it, use SYSCALL_DEFINE3() instead of a hand-crafted syscall
> definition.

Great!

> static void complete_change_console(struct vc_data *vc);
> @@ -420,12 +420,12 @@ int vt_ioctl(struct tty_struct *tty,
> ret = -EINVAL;
> break;
> }
> - ret = sys_ioperm(arg, 1, (cmd == KDADDIO)) ? -ENXIO : 0;
> + ret = ksys_ioperm(arg, 1, (cmd == KDADDIO)) ? -ENXIO : 0;
> break;
>
> case KDENABIO:
> case KDDISABIO:
> - ret = sys_ioperm(GPFIRST, GPNUM,
> + ret = ksys_ioperm(GPFIRST, GPNUM,
> (cmd == KDENABIO)) ? -ENXIO : 0;
> break;
> #endif

I wonder if we can just delete this code.