Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote:
> Syzbot reports the following WARNING:
>
> [200~raw_local_irq_restore() called with IRQs enabled
> WARNING: CPU: 1 PID: 1206 at kernel/locking/irqflag-debug.c:10
> warn_bogus_irq_restore+0x1d/0x20 kernel/locking/irqflag-debug.c:10
>
> Hardware initialization for the rtl8188cu can run for as long as 350 ms,
> and the routine may be called with interrupts disabled. To avoid locking
> the machine for this long, the current routine saves the interrupt flags
> and enables local interrupts. The problem is that it restores the flags
> at the end without disabling local interrupts first.
>
> This patch fixes commit a53268be0cb9 ("rtlwifi: rtl8192cu: Fix too long
> disable of IRQs").
>
> Reported-by: syzbot+cce1ee31614c171f5595@xxxxxxxxxxxxxxxxxxxxxxxxx
> Cc: stable@xxxxxxxxxxxxxxx
> Fixes: a53268be0cb9 ("rtlwifi: rtl8192cu: Fix too long disable of IRQs")
> Signed-off-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx>
Patch applied to wireless-drivers-next.git, thanks.
8b144dedb928 rtlwifi: rtl8192cu: Fix WARNING when calling local_irq_restore() with interrupts enabled
--
https://patchwork.kernel.org/project/linux-wireless/patch/20211215171105.20623-1-Larry.Finger@xxxxxxxxxxxx/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
