Re: [PATCH 02/50] move compat handling of tty ioctls to tty_compat_ioctl()

[gregkh]

On Fri, Sep 14, 2018 at 05:15:52PM +0200, Arnd Bergmann wrote:
> On Thu, Sep 13, 2018 at 4:40 AM Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote:
> >
> > From: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
> >
> > ioctls that are
> >         * callable only via tty_ioctl()
> >         * not driver-specific
> >         * not demand data structure conversions
> >         * either always need passing arg as is or always demand compat_ptr()
> > get intercepted in tty_compat_ioctl() from the very beginning and
> > redirecter to tty_ioctl().  As the result, their entries in fs/compat_ioctl.c
> > (some of those had been missing, BTW) got removed, as well as
> > n_tty_compat_ioctl_helper() (now it's never called with any cmd it would accept).
> >
> > Signed-off-by: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
> 
> One more thing I just remembered when looking at my obsolete patches:
> Your patch is fixing a couple of ioctl commands (I had identified only
> TIOCVHANGUP and TIOCGPTPEER, you found a couple more)
> 
> It would be good to mark those as cc:stable@xxxxxxxxxxxxxxx so we
> can also get the fixes into older kernels. Looking through the
> series again, I did not spot anything else that we need in stable,
> in particular I would not bother with the compat_ptr() fixups there,
> and the many cleanups of dead code and setserial wouldn't
> be stable material anyway, but missing translation seems important
> enough.
> 
> Looking at the changes in your patches, these are the ones
> that you add in tty_compat_ioctl() that did not have a corresponding
> entry in fs/compat_ioctl.c:
> 
> +       case TCSETX:
> +       case TCSETXF:
> +       case TCSETXW:
> +       case TIOCGETC:
> +       case TIOCGETP:
> +       case TIOCGPTPEER:
> +       case TIOCSERCONFIG:
> +       case TIOCSETC:
> +       case TIOCSETN:
> +       case TIOCSETP:
> +       case TIOCVHANGUP:
> 
> Should these all be added in a stable backport patch?

Why?  What is being "fixed" here that anyone has noticed before that is
causing problems in those old kernels?  If no one has complained about
the lack of compat fixups, well...  :)

thanks,

greg k-h