Re: [PATCH 1/2] staging: rtl8192u: fix control-message timeouts

From: Greg Kroah-Hartman
Date: Mon Oct 25 2021 - 11:51:30 EST


On Mon, Oct 25, 2021 at 08:41:36AM -0700, Joe Perches wrote:
> On Mon, 2021-10-25 at 10:06 -0500, Larry Finger wrote:
> > On 10/25/21 07:09, Johan Hovold wrote:
> > > USB control-message timeouts are specified in milliseconds and should
> > > specifically not vary with CONFIG_HZ.
>
> There appears to be more than a few of these in the kernel.
>
> $ cat usb_hz.cocci
> @@
> expression e;
> @@
> * usb_control_msg(..., HZ * e)
>
> @@
> expression e;
> @@
> * usb_control_msg(..., HZ / e)
>
> @@
> @@
> * usb_control_msg(..., HZ)
>
> $ spatch --very-quiet -U 0 -sp-file usb_hz.cocci .
> warning: line 4: should HZ be a metavariable?
> warning: line 9: should HZ be a metavariable?
> warning: line 13: should HZ be a metavariable?
> 50 files match

Look at the lists, he's sent a bunch of fixes for this today to all the
subsystems...

greg k-h