Re: [PATCH 1/2] genirq: Fix IRQ threads affinity VS cpuset isolated partitions

From: Waiman Long

Date: Tue Nov 18 2025 - 12:29:26 EST


On 11/18/25 12:23 PM, Thomas Gleixner wrote:
On Tue, Nov 18 2025 at 11:27, Waiman Long wrote:
+static inline void irq_thread_set_affinity(struct task_struct *t,
+ struct irq_desc *desc)
+{
+ kthread_bind_mask(t, irq_data_get_effective_affinity_mask(&desc->irq_data));
+}
According to irq_thread_check_affinity(), accessing the cpumask returned
from irq_data_get_effective_affinity_mask(&desc->irq_data) requires
taking desc->lock to ensure its stability. Do we need something similar
here? Other than that, it looks good to me.
That's during interrupt setup so it should be stable (famous last words)

Thanks for the clarification. We should probably add a comment to mention that.

Cheers,
Longman