[v3] genirq: Fix IRQ threads VS cpuset

[PATCH 0/3 v3] genirq: Fix IRQ threads VS cpuset

Posted by Frederic Weisbecker 1 week, 3 days ago

Hi,

Here is another take after some last minutes issues reported by
Marek Szyprowski <m.szyprowski@samsung.com>:

https://lore.kernel.org/all/73356b5f-ab5c-4e9e-b57f-b80981c35998@samsung.com/


Changes since v2:

* Fix early spurious IRQ thread wake-up (to be SOB'ed by Thomas)

* Instead of applying the affinity remotely, set PF_NO_SETAFFINITY
  early, right after kthread creation, and wait for the thread to
  apply the affinity by itself. This is to prevent from early wake-up
  to mess up with kthread_bind_mask(), as reported by
  Marek Szyprowski <m.szyprowski@samsung.com>

Frederic Weisbecker (2):
  genirq: Fix interrupt threads affinity vs. cpuset isolated partitions
  genirq: Remove cpumask availability check on kthread affinity setting

Thomas Gleixner (1):
  genirq: Prevent from early irq thread spurious wake-ups

 kernel/irq/handle.c | 10 +++++++++-
 kernel/irq/manage.c | 40 +++++++++++++++++++---------------------
 2 files changed, 28 insertions(+), 22 deletions(-)

-- 
2.51.1

Re: [PATCH 0/3 v3] genirq: Fix IRQ threads VS cpuset

Posted by Marek Szyprowski 1 week, 3 days ago

On 21.11.2025 15:34, Frederic Weisbecker wrote:
> Here is another take after some last minutes issues reported by
> Marek Szyprowski <m.szyprowski@samsung.com>:
>
> https://lore.kernel.org/all/73356b5f-ab5c-4e9e-b57f-b80981c35998@samsung.com/

Works fine on my test systems.

Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>

> Changes since v2:
>
> * Fix early spurious IRQ thread wake-up (to be SOB'ed by Thomas)
>
> * Instead of applying the affinity remotely, set PF_NO_SETAFFINITY
>    early, right after kthread creation, and wait for the thread to
>    apply the affinity by itself. This is to prevent from early wake-up
>    to mess up with kthread_bind_mask(), as reported by
>    Marek Szyprowski <m.szyprowski@samsung.com>
>
> Frederic Weisbecker (2):
>    genirq: Fix interrupt threads affinity vs. cpuset isolated partitions
>    genirq: Remove cpumask availability check on kthread affinity setting
>
> Thomas Gleixner (1):
>    genirq: Prevent from early irq thread spurious wake-ups
>
>   kernel/irq/handle.c | 10 +++++++++-
>   kernel/irq/manage.c | 40 +++++++++++++++++++---------------------
>   2 files changed, 28 insertions(+), 22 deletions(-)
>
Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland