Hello, Tvrtko.

On Wed, Oct 21, 2026 at 01:53:01PM +0100, Tvrtko Ursulin wrote:
> +static void
> +sanitize_workqueue_attrs(struct workqueue_struct *wq,
> +                        struct workqueue_attrs *attrs)
> +{
> +       if (wq->flags & WQ_HIGHPRI)
> +               attrs->nice = HIGHPRI_NICE_LEVEL;

WQ_HIGHPRI only sets the initial nice level, which can be changed
afterwards, e.g. through sysfs. Forcing it here would silently override such
changes. Maybe keep setting it at allocation instead?

> Could you please confirm (or deny :)) this is what you had in mind?

The rest sounds sane to me, but it's difficult to evaluate without looking
at the patches. Can you please post them?

Thanks.

--
tejun

Reply via email to