bianbu-repo-mirror/bianbu-linux-6.6
1
0
Fork 0
mirror of https://gitee.com/bianbu-linux/linux-6.6 synced 2025-04-24 14:07:52 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

genirq/cpuhotplug: Skip suspended interrupts when restoring affinity

Browse source 
commit a60dd06af674d3bb76b40da5d722e4a0ecefe650 upstream.

irq_restore_affinity_of_irq() restarts managed interrupts unconditionally
when the first CPU in the affinity mask comes online. That's correct during
normal hotplug operations, but not when resuming from S3 because the
drivers are not resumed yet and interrupt delivery is not expected by them.

Skip the startup of suspended interrupts and let resume_device_irqs() deal
with restoring them. This ensures that irqs are not delivered to drivers
during the noirq phase of resuming from S3, after non-boot CPUs are brought
back online.

Signed-off-by: David Stevens <stevensd@chromium.org>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/r/20240424090341.72236-1-stevensd@chromium.org
Cc: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
David Stevens 2024-04-24 18:03:41 +09:00 • committed by Greg Kroah-Hartman
parent a47b54846a
commit 20dbad7525
2 changed files with 16 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

11
kernel/irq/cpuhotplug.c
View file

@ -195,10 +195,15 @@ static void irq_restore_affinity_of_irq(struct irq_desc *desc, unsigned int cpu)
!irq_data_get_irq_chip(data) || !cpumask_test_cpu(cpu, affinity)) !irq_data_get_irq_chip(data) || !cpumask_test_cpu(cpu, affinity))
return; return;
if (irqd_is_managed_and_shutdown(data)) { /*
irq_startup(desc, IRQ_RESEND, IRQ_START_COND); * Don't restore suspended interrupts here when a system comes back
* from S3. They are reenabled via resume_device_irqs().
*/
if (desc->istate & IRQS_SUSPENDED)
return; return;
}
if (irqd_is_managed_and_shutdown(data))
irq_startup(desc, IRQ_RESEND, IRQ_START_COND);
/* /*
* If the interrupt can only be directed to a single target * If the interrupt can only be directed to a single target

12
kernel/irq/manage.c
View file

@ -796,10 +796,14 @@ void __enable_irq(struct irq_desc *desc)
irq_settings_set_noprobe(desc); irq_settings_set_noprobe(desc);
/* /*
* Call irq_startup() not irq_enable() here because the * Call irq_startup() not irq_enable() here because the
* interrupt might be marked NOAUTOEN. So irq_startup() * interrupt might be marked NOAUTOEN so irq_startup()
* needs to be invoked when it gets enabled the first * needs to be invoked when it gets enabled the first time.
* time. If it was already started up, then irq_startup() * This is also required when __enable_irq() is invoked for
* will invoke irq_enable() under the hood. * a managed and shutdown interrupt from the S3 resume
* path.
*
* If it was already started up, then irq_startup() will
* invoke irq_enable() under the hood.
*/ */
irq_startup(desc, IRQ_RESEND, IRQ_START_FORCE); irq_startup(desc, IRQ_RESEND, IRQ_START_FORCE);
break; break;