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

From: Thomas Gleixner

Date: Tue Nov 18 2025 - 12:24:00 EST


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)