Re: [PATCH] USB: legousbtower: use stream_open for stream semantics

From: Greg KH

Date: Tue Nov 10 2026 - 13:25:10 EST


On Tue, Nov 10, 2026 at 11:01:36PM +0530, mdshahid03@xxxxxxxxx wrote:
> Hi,
>
> Could you please review patch and give your feedback,when time permits.
>
> Thanks
> Shahid.
>
> On 2026-10-28 23:58 +0530, mdshahid03@xxxxxxxxx wrote:
> > From: Mohammad Shahid <mdshahid03@xxxxxxxxx>
> >
> > The LEGO USB Tower driver uses nonseekable_open() in tower_open()
> > However, its read and write have stream semantics.
> >
> > Use stream_open instead to mark the file as a stream and disable
> > unnecessary file position handling.
> >
> > The issue is reported by stream_open.cocci script as potential
> > deadlock between read and write operation.
> >
> > Signed-off-by: Mohammad Shahid <mdshahid03@xxxxxxxxx>
> > ---
> > drivers/usb/misc/legousbtower.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)

How was this tested?