Re: [PATCH v4 1/2] workqueue: Make alloc_workqueue() unbound by default

From: Marco Crivellari

Date: Fri Oct 16 2026 - 12:24:41 EST


On Fri, Oct 16, 2026 at 6:18 PM Marco Crivellari
<marco.crivellari@xxxxxxxx> wrote:
>
> On Fri, Oct 16, 2026 at 5:54 PM Breno Leitao <leitao@xxxxxxxxxx> wrote:
> > [...]
> > > @@ -5895,12 +5895,33 @@ static struct workqueue_struct *__alloc_workqueue(const char *fmt,
> > > struct workqueue_struct *wq;
> > > size_t wq_size;
> > > int name_len;
> > > + bool wq_unbound_by_default = false;
> > > + bool wq_dropped_percpu = false;
> >
> > Nit: I think these declarations would look better in reverse xmas tree
> > order. I know Tejun doesn't feel strongly about it, so it's just a nit.
> >
> > --breno
>
> Cool, thanks for the hint, I will change this along with the removal
> of pr_warn_once().
>

Oh wait: only one of the bool, the other is going away with the
removal of the pr_warn_once().

--

Marco Crivellari

SUSE Labs