Commit 2579a4ee authored by Thomas Gleixner's avatar Thomas Gleixner Committed by Ingo Molnar

x86/ioapic: Rename misnamed functions

ioapic_irqd_[un]mask() are misnomers as both functions do way more than
masking and unmasking the interrupt line. Both deal with the moving the
affinity of the interrupt within interrupt context. The mask/unmask is just
a tiny part of the functionality.

Rename them to ioapic_prepare/finish_move(), fixup the call sites and
rename the related variables in the code to reflect what this is about.

No functional change.
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Cc: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Sebastian Siewior <bigeasy@linutronix.de>
Link: https://lkml.kernel.org/r/20191017101938.412489856@linutronix.deSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
parent df439342
...@@ -1725,7 +1725,7 @@ static bool io_apic_level_ack_pending(struct mp_chip_data *data) ...@@ -1725,7 +1725,7 @@ static bool io_apic_level_ack_pending(struct mp_chip_data *data)
return false; return false;
} }
static inline bool ioapic_irqd_mask(struct irq_data *data) static inline bool ioapic_prepare_move(struct irq_data *data)
{ {
/* If we are moving the IRQ we need to mask it */ /* If we are moving the IRQ we need to mask it */
if (unlikely(irqd_is_setaffinity_pending(data))) { if (unlikely(irqd_is_setaffinity_pending(data))) {
...@@ -1736,9 +1736,9 @@ static inline bool ioapic_irqd_mask(struct irq_data *data) ...@@ -1736,9 +1736,9 @@ static inline bool ioapic_irqd_mask(struct irq_data *data)
return false; return false;
} }
static inline void ioapic_irqd_unmask(struct irq_data *data, bool masked) static inline void ioapic_finish_move(struct irq_data *data, bool moveit)
{ {
if (unlikely(masked)) { if (unlikely(moveit)) {
/* Only migrate the irq if the ack has been received. /* Only migrate the irq if the ack has been received.
* *
* On rare occasions the broadcast level triggered ack gets * On rare occasions the broadcast level triggered ack gets
...@@ -1773,11 +1773,11 @@ static inline void ioapic_irqd_unmask(struct irq_data *data, bool masked) ...@@ -1773,11 +1773,11 @@ static inline void ioapic_irqd_unmask(struct irq_data *data, bool masked)
} }
} }
#else #else
static inline bool ioapic_irqd_mask(struct irq_data *data) static inline bool ioapic_prepare_move(struct irq_data *data)
{ {
return false; return false;
} }
static inline void ioapic_irqd_unmask(struct irq_data *data, bool masked) static inline void ioapic_finish_move(struct irq_data *data, bool moveit)
{ {
} }
#endif #endif
...@@ -1786,11 +1786,11 @@ static void ioapic_ack_level(struct irq_data *irq_data) ...@@ -1786,11 +1786,11 @@ static void ioapic_ack_level(struct irq_data *irq_data)
{ {
struct irq_cfg *cfg = irqd_cfg(irq_data); struct irq_cfg *cfg = irqd_cfg(irq_data);
unsigned long v; unsigned long v;
bool masked; bool moveit;
int i; int i;
irq_complete_move(cfg); irq_complete_move(cfg);
masked = ioapic_irqd_mask(irq_data); moveit = ioapic_prepare_move(irq_data);
/* /*
* It appears there is an erratum which affects at least version 0x11 * It appears there is an erratum which affects at least version 0x11
...@@ -1845,7 +1845,7 @@ static void ioapic_ack_level(struct irq_data *irq_data) ...@@ -1845,7 +1845,7 @@ static void ioapic_ack_level(struct irq_data *irq_data)
eoi_ioapic_pin(cfg->vector, irq_data->chip_data); eoi_ioapic_pin(cfg->vector, irq_data->chip_data);
} }
ioapic_irqd_unmask(irq_data, masked); ioapic_finish_move(irq_data, moveit);
} }
static void ioapic_ir_ack_level(struct irq_data *irq_data) static void ioapic_ir_ack_level(struct irq_data *irq_data)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment