Skip to content

Commit 7f2b41a

Browse files
neerajupadhyabp3tk0v
authored andcommitted
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: Neeraj Upadhyay <[email protected]> Signed-off-by: Borislav Petkov (AMD) <[email protected]> Link: https://lore.kernel.org/[email protected]
1 parent 0877ad1 commit 7f2b41a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

arch/x86/kernel/apic/vector.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -183,6 +183,7 @@ static void apic_update_vector(struct irq_data *irqd, unsigned int newvec,
183183
apicd->cpu = newcpu;
184184
BUG_ON(!IS_ERR_OR_NULL(per_cpu(vector_irq, newcpu)[newvec]));
185185
per_cpu(vector_irq, newcpu)[newvec] = desc;
186+
apic_update_irq_cfg(irqd, newvec, newcpu);
186187
}
187188

188189
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)
261262
if (vector < 0)
262263
return vector;
263264
apic_update_vector(irqd, vector, cpu);
264-
apic_update_irq_cfg(irqd, vector, cpu);
265265

266266
return 0;
267267
}
@@ -338,7 +338,7 @@ assign_managed_vector(struct irq_data *irqd, const struct cpumask *dest)
338338
if (vector < 0)
339339
return vector;
340340
apic_update_vector(irqd, vector, cpu);
341-
apic_update_irq_cfg(irqd, vector, cpu);
341+
342342
return 0;
343343
}
344344

0 commit comments

Comments
 (0)