]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
x86/apic: Move apic_update_irq_cfg() call to apic_update_vector()
authorNeeraj Upadhyay <Neeraj.Upadhyay@amd.com>
Wed, 9 Jul 2025 03:32:24 +0000 (09:02 +0530)
committerBorislav Petkov (AMD) <bp@alien8.de>
Tue, 15 Jul 2025 09:54:09 +0000 (11:54 +0200)
All callers of apic_update_vector() also call apic_update_irq_cfg() after it.
So, move the apic_update_irq_cfg() call to apic_update_vector().

No functional change intended.

Signed-off-by: Neeraj Upadhyay <Neeraj.Upadhyay@amd.com>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Link: https://lore.kernel.org/20250709033242.267892-18-Neeraj.Upadhyay@amd.com
arch/x86/kernel/apic/vector.c

index 93069b13d3af234ade9e754dd97ec78982138115..a947b46a8b642bcaf6ba6847d2d2320c1ae1f0d5 100644 (file)
@@ -183,6 +183,7 @@ setnew:
        apicd->cpu = newcpu;
        BUG_ON(!IS_ERR_OR_NULL(per_cpu(vector_irq, newcpu)[newvec]));
        per_cpu(vector_irq, newcpu)[newvec] = desc;
+       apic_update_irq_cfg(irqd, newvec, newcpu);
 }
 
 static void vector_assign_managed_shutdown(struct irq_data *irqd)
@@ -261,7 +262,6 @@ assign_vector_locked(struct irq_data *irqd, const struct cpumask *dest)
        if (vector < 0)
                return vector;
        apic_update_vector(irqd, vector, cpu);
-       apic_update_irq_cfg(irqd, vector, cpu);
 
        return 0;
 }
@@ -338,7 +338,7 @@ assign_managed_vector(struct irq_data *irqd, const struct cpumask *dest)
        if (vector < 0)
                return vector;
        apic_update_vector(irqd, vector, cpu);
-       apic_update_irq_cfg(irqd, vector, cpu);
+
        return 0;
 }