genirq/manage: Rework enable_percpu_irq()

Use the new guards to get and lock the interrupt descriptor and tidy up the
code.

No functional change.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lore.kernel.org/all/20250429065422.315844964@linutronix.de
This commit is contained in:
Thomas Gleixner 2025-04-29 08:55:44 +02:00
parent 90140d08ac
commit 508bd94c3a

View file

@ -2286,35 +2286,25 @@ int request_nmi(unsigned int irq, irq_handler_t handler,
void enable_percpu_irq(unsigned int irq, unsigned int type) void enable_percpu_irq(unsigned int irq, unsigned int type)
{ {
unsigned int cpu = smp_processor_id(); scoped_irqdesc_get_and_lock(irq, IRQ_GET_DESC_CHECK_PERCPU) {
unsigned long flags; struct irq_desc *desc = scoped_irqdesc;
struct irq_desc *desc = irq_get_desc_lock(irq, &flags, IRQ_GET_DESC_CHECK_PERCPU);
if (!desc) /*
return; * If the trigger type is not specified by the caller, then
* use the default for this interrupt.
*/
type &= IRQ_TYPE_SENSE_MASK;
if (type == IRQ_TYPE_NONE)
type = irqd_get_trigger_type(&desc->irq_data);
/* if (type != IRQ_TYPE_NONE) {
* If the trigger type is not specified by the caller, then if (__irq_set_trigger(desc, type)) {
* use the default for this interrupt. WARN(1, "failed to set type for IRQ%d\n", irq);
*/ return;
type &= IRQ_TYPE_SENSE_MASK; }
if (type == IRQ_TYPE_NONE)
type = irqd_get_trigger_type(&desc->irq_data);
if (type != IRQ_TYPE_NONE) {
int ret;
ret = __irq_set_trigger(desc, type);
if (ret) {
WARN(1, "failed to set type for IRQ%d\n", irq);
goto out;
} }
irq_percpu_enable(desc, smp_processor_id());
} }
irq_percpu_enable(desc, cpu);
out:
irq_put_desc_unlock(desc, flags);
} }
EXPORT_SYMBOL_GPL(enable_percpu_irq); EXPORT_SYMBOL_GPL(enable_percpu_irq);