Re: [PATCH v2 4/9] workqueue: account nr_active by the backing pool

From: Tejun Heo

Date: Mon Aug 10 2026 - 17:41:30 EST


On Wed, Aug 05, 2026 at 07:52:31AM -0700, Breno Leitao wrote:
> static struct wq_node_nr_active *wq_node_nr_active(struct workqueue_struct *wq,
> int node)
> {
> - if (!(wq->flags & WQ_UNBOUND))
> + if (WARN_ON_ONCE(!(wq->flags & WQ_UNBOUND)))
> return NULL;

This just pushes the crash to the caller, right? Maybe do BUG_ON() instead?

Thanks.

--
tejun