Commit 4fbd6d38 authored by Bibo Mao's avatar Bibo Mao Committed by Xianglai Li
Browse files

irqchip/loongson-eiointc: Remove explicit interrupt affinity restore on resume 

mainline inclusion
from mainline-v6.9-rc1
commit 83c0708719f77018cd3b98b0011c9526a3e0e2ca
category: feature
bugzilla: https://gitee.com/openeuler/kernel/issues/I9BTOX


CVE: NA

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

During suspend all CPUs except CPU0 are hot-unpluged and all active
interrupts are migrated to CPU0.

On resume eiointc_router_init() affines all interrupts to CPU0, so the
subsequent explicit interrupt affinity restore is redundant.

Remove it.

[ tglx: Rewrote changelog ]

Signed-off-by: default avatarBibo Mao <maobibo@loongson.cn>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/r/20240130082722.2912576-4-maobibo@loongson.cn

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

During suspend and resume, CPUs except CPU0 can be hot-unpluged and IRQs
will be migrated to CPU0. So it is not necessary to restore irq affinity
for eiointc irq controller when system resumes. This patch removes this
piece of code about irq affinity restoring in function eiointc_resume().
parent 81306dbc

drivers/irqchip/irq-loongson-eiointc.c

+0 −16
Original line number Diff line number Diff line
@@ -324,23 +324,7 @@ static int eiointc_suspend(void) 


static void eiointc_resume(void)

{

	int i, j;

	struct irq_desc *desc;

	struct irq_data *irq_data;



	eiointc_router_init(0);



	for (i = 0; i < nr_pics; i++) {

		for (j = 0; j < eiointc_priv[0]->vec_count; j++) {

			desc = irq_resolve_mapping(eiointc_priv[i]->eiointc_domain, j);

			if (desc && desc->handle_irq && desc->handle_irq != handle_bad_irq) {

				raw_spin_lock(&desc->lock);

				irq_data = irq_domain_get_irq_data(eiointc_priv[i]->eiointc_domain, irq_desc_get_irq(desc));

				eiointc_set_irq_affinity(irq_data, irq_data->common->affinity, 0);

				raw_spin_unlock(&desc->lock);

			}

		}

	}

}



static struct syscore_ops eiointc_syscore_ops = {