lib/irq_poll: Prevent softirq pending leak in irq_poll_cpu_dead()
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Sun, 10 Apr 2022 12:49:36 +0000 (14:49 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Wed, 13 Apr 2022 19:32:21 +0000 (21:32 +0200)
commit75d8cce128c516fe6cf4b8683e8fe1a59e919902
treec90aa58d0e8ae63b4915d5af63c63ebc499fb561
parentce522ba9ef7e2d9fb22a39eb3371c0c64e2a433e
lib/irq_poll: Prevent softirq pending leak in irq_poll_cpu_dead()

irq_poll_cpu_dead() pulls the blk_cpu_iopoll backlog from the dead CPU and
raises the POLL softirq with __raise_softirq_irqoff() on the CPU it is
running on. That just sets the bit in the pending softirq mask.

This means the handling of the softirq is delayed until the next interrupt
or a local_bh_disable/enable() pair. As a consequence the CPU on which this
code runs can reach idle with the POLL softirq pending, which triggers a
warning in the NOHZ idle code.

Add a local_bh_disable/enable() pair around the interrupts disabled section
in irq_poll_cpu_dead(). local_bh_enable will handle the pending softirq.

[tglx: Massaged changelog and comment]

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/r/87k0bxgl27.ffs@tglx
lib/irq_poll.c