Commit a5606c3d authored by Marc Zyngier's avatar Marc Zyngier Committed by YunYi Yang
Browse files

genirq: Always limit the affinity to online CPUs 

mainline inclusion
from mainline-v5.19-rc1
commit 33de0aa4
category: bugfix
bugzilla: https://gitee.com/src-openeuler/kernel/issues/I8GYPY

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=33de0aa4bae982ed6f7c777f86b5af3e627ac937



----------------------------------------------------------------------

When booting with maxcpus=<small number> (or even loading a driver
while most CPUs are offline), it is pretty easy to observe managed
affinities containing a mix of online and offline CPUs being passed
to the irqchip driver.

This means that the irqchip cannot trust the affinity passed down
from the core code, which is a bit annoying and requires (at least
in theory) all drivers to implement some sort of affinity narrowing.

In order to address this, always limit the cpumask to the set of
online CPUs.

Signed-off-by: default avatarMarc Zyngier <maz@kernel.org>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/r/20220405185040.206297-3-maz@kernel.org



Signed-off-by: default avatarChen Jiahao <chenjiahao16@huawei.com>
Reviewed-by: default avatarZhang Jianhua <chris.zjh@huawei.com>
Reviewed-by: default avatarLiao Chang <liaochang1@huawei.com>
Signed-off-by: default avatarZheng Zengkai <zhengzengkai@huawei.com>
Signed-off-by: default avatarYunYi Yang <yangyunyi2@huawei.com>

conflict:
    kernel/irq/manage.c
parent 5867a9b1

kernel/irq/manage.c

+14 −1
Original line number Diff line number Diff line
@@ -224,10 +224,23 @@ int irq_do_set_affinity(struct irq_data *data, const struct cpumask *mask, 
	struct irq_chip *chip = irq_data_get_irq_chip(data);

	int ret;



	static DEFINE_RAW_SPINLOCK(tmp_mask_lock);

	static struct cpumask tmp_mask;



	if (!chip || !chip->irq_set_affinity)

		return -EINVAL;



	ret = chip->irq_set_affinity(data, mask, force);

	raw_spin_lock(&tmp_mask_lock);



	/* Make sure we only provide online CPUs to the irqchip */

	cpumask_and(&tmp_mask, mask, cpu_online_mask);

	if (!cpumask_empty(&tmp_mask))

		ret = chip->irq_set_affinity(data, &tmp_mask, force);

	else

		ret = -EINVAL;



	raw_spin_unlock(&tmp_mask_lock);



	switch (ret) {

	case IRQ_SET_MASK_OK:

	case IRQ_SET_MASK_OK_DONE: