linux-kernel - Re: [RFC PATCH 1/2] serial: core: Move tty and serdev to be children of serial core port device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 25 Oct 2023 09:33:09 +0200
From:   Johan Hovold <johan@...nel.org>
To:     Tony Lindgren <tony@...mide.com>
Cc:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Jiri Slaby <jirislaby@...nel.org>,
        Rob Herring <robh@...nel.org>,
        Andy Shevchenko <andriy.shevchenko@...el.com>,
        Dhruva Gole <d-gole@...com>,
        Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,
        John Ogness <john.ogness@...utronix.de>,
        Sebastian Andrzej Siewior <bigeasy@...utronix.de>,
        Vignesh Raghavendra <vigneshr@...com>,
        linux-kernel@...r.kernel.org, linux-serial@...r.kernel.org,
        Maximilian Luz <luzmaximilian@...il.com>
Subject: Re: [RFC PATCH 1/2] serial: core: Move tty and serdev to be children
 of serial core port device

On Wed, Oct 25, 2023 at 09:51:52AM +0300, Tony Lindgren wrote:
> * Greg Kroah-Hartman <gregkh@...uxfoundation.org> [231024 14:01]:

> > > > What does this change the sysfs tree to look like?

> Here's the diff of the same command before and after:
> 
> --- /tmp/before 2023-10-25 09:45:12.197283690 +0300
> +++ /tmp/after  2023-10-25 09:43:30.681797899 +0300
> @@ -1,7 +1,9 @@
>  # find /sys -name tty
>  /sys/class/tty
>  /sys/class/tty/tty
> -/sys/devices/pnp0/00:04/tty
> -/sys/devices/platform/serial8250/tty
> +/sys/devices/pnp0/00:04/00:04:0/00:04:0.0/tty
> +/sys/devices/platform/serial8250/serial8250:0/serial8250:0.3/tty
> +/sys/devices/platform/serial8250/serial8250:0/serial8250:0.1/tty
> +/sys/devices/platform/serial8250/serial8250:0/serial8250:0.2/tty
>  /sys/devices/virtual/tty
>  /sys/devices/virtual/tty/tty

Your diff is missing the actual tty devices. 'tty' is just the class
directory. 

And can you post the equivalent diff for serdev as well for completeness?

> There are multiple ports claimed by serial8250. So I think the new sysfs
> output is correct showing more ttys. If there's some reason why serial8250
> should only have one tty and this output is not correct let me know too..

There should not be more class devices, you've just moved them and thus
there are more class directories (with one device per directory).

Johan