Commit 7f2b41ac authored by Neeraj Upadhyay's avatar Neeraj Upadhyay Committed by Borislav Petkov (AMD)
Browse files

x86/apic: Move apic_update_irq_cfg() call to apic_update_vector() 



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: default avatarNeeraj Upadhyay <Neeraj.Upadhyay@amd.com>
Signed-off-by: default avatarBorislav Petkov (AMD) <bp@alien8.de>
Link: https://lore.kernel.org/20250709033242.267892-18-Neeraj.Upadhyay@amd.com
parent 0877ad1c

arch/x86/kernel/apic/vector.c

+2 −2
Original line number Diff line number Diff line
@@ -183,6 +183,7 @@ static void apic_update_vector(struct irq_data *irqd, unsigned int newvec, 
	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;

}