linux-kernel - Re: [PATCH] usb: core: support interface node for simple USB devices

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]

Message-ID: <Z5Ilm06wcuNsqtod@hovoldconsulting.com>
Date: Thu, 23 Jan 2025 12:18:51 +0100
From: Johan Hovold <johan@...nel.org>
To: POPESCU Catalin <catalin.popescu@...ca-geosystems.com>
Cc: Greg KH <gregkh@...uxfoundation.org>,
	"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"m.felsch@...gutronix.de" <m.felsch@...gutronix.de>,
	GEO-CHHER-bsp-development <bsp-development.geo@...ca-geosystems.com>
Subject: Re: [PATCH] usb: core: support interface node for simple USB devices

On Wed, Jan 22, 2025 at 05:17:36PM +0000, POPESCU Catalin wrote:
> On 22/01/2025 14:53, Greg KH wrote:

> > serdev can not use usb-serial devices due to the lack of hotplugging, so
> > why is this an issue?  I thought that just would not work.
> 
> Well, I can say that on 6.12.0 it's working :
> 
> [    0.929493] usb 1-1: new full-speed USB device number 2 using xhci-hcd
> [    5.906574] usbcore: registered new interface driver ftdi_sio
> [    5.906632] usbserial: USB Serial support registered for FTDI USB 
> Serial Device
> [    5.906727] ftdi_sio 1-1:1.0: FTDI USB Serial Device converter detected
> [    5.906842] usb 1-1: Detected FT-X
> [    5.907809] serial serial0: tty port ttyUSB0 registered

You'd need to patch USB serial too for this to happen so this in not
mainline 6.12.

And if the port is open during disconnect you'd see a bunch of warnings
because the serdev tty port implementation does not handle hangups.

Johan