[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <YG3ORQcI5DRw7QL/@kroah.com>
Date: Wed, 7 Apr 2021 17:22:45 +0200
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Johan Hovold <johan@...nel.org>
Cc: Jiri Slaby <jirislaby@...nel.org>, linux-staging@...ts.linux.dev,
greybus-dev@...ts.linaro.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 00/16] tty: TIOCSSERIAL fixes and clean ups
On Wed, Apr 07, 2021 at 12:23:18PM +0200, Johan Hovold wrote:
> TIOCSSERIAL is a horrid, underspecified, legacy interface which for most
> serial devices is only useful for setting the close_delay and
> closing_wait parameters.
>
> This series fixes up the various ways in which driver authors have
> gotten the implementation wrong over the years, like, for example,
> jiffies conversions, permissions checks and error handling.
>
> The de-facto standard for error handling is to ignore any unsupported
> features and immutable parameters (cf. UPF_FIXED_PORT and deprecated
> ASYNC flags).
>
> Permission checking should prevent an unprivileged user from changing
> anything but the ASYNC_USR flags (and custom divisor) by returning
> -EPERM.
>
> These patches are against tty-next, but the staging ones could otherwise
> go through either tree.
I've taken all of these through my tty tree, thanks for cleaning this
mess up.
greg k-h
Powered by blists - more mailing lists