RE: [PATCH rtw-next] Revert "wifi: rtw88: add WQ_UNBOUND to alloc_workqueue users"

[Ping-Ke Shih <pkshih@realtek.com>]

Marco Crivellari <marco.crivellari@suse.com> wrote:
> Sent: Tuesday, December 9, 2025 5:57 PM
> rtw89On Mon, Dec 8, 2025 at 1:40 AM Ping-Ke Shih <pkshih@realtek.com> wrote:
> > Hi Marco,
> >
> > Will you send a patch to point out WQ_PERCPU explicitly?
> >
> > > +       rtwusb->rxwq = alloc_workqueue("rtw88_usb: rx wq", WQ_BH, 0);
> > >         if (!rtwusb->rxwq) {
> > >                 rtw_err(rtwdev, "failed to create RX work queue\n");
> > >                 return -ENOMEM;
> 
> Hi,
> 
> The v1 is indeed correct, sorry for that: WQ_BH should be per-cpu
> because it is executed in softirq context.
> 
> Should I send a patch prefixed with [BUG] in order to fix it or the v1
> could be used and we're ok?

I think v1 is ok, so I re-open v1 [1], and plan to merge this in -next cycle.

[1] https://patchwork.kernel.org/project/linux-wireless/patch/20251113160605.381777-3-marco.crivellari@suse.com/

> 
> I noticed also in 6.18 "rtw89_usb_init_rx" is allocating a workqueue
> and is missing WQ_PERCPU.

Please prepare another patch for this.