diff options
author | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2013-04-16 14:33:20 -0400 |
---|---|---|
committer | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2013-04-16 16:05:15 -0400 |
commit | cb9c6f15f318aa3aeb62fe525aa5c6dcf6eee159 (patch) | |
tree | 804a98f6f928a5f27bf69f1b31f40d26867b78aa | |
parent | ef35a4e6d9eef3bd904b8c6b85618a2e878e801e (diff) |
xen/spinlock: Check against default value of -1 for IRQ line.
The default (uninitialized) value of the IRQ line is -1.
Check if we already have allocated an spinlock interrupt line
and if somebody is trying to do it again. Also set it to -1
when we offline the CPU.
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
-rw-r--r-- | arch/x86/xen/spinlock.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/arch/x86/xen/spinlock.c b/arch/x86/xen/spinlock.c index f7a080ef0354..47ae032f8d97 100644 --- a/arch/x86/xen/spinlock.c +++ b/arch/x86/xen/spinlock.c | |||
@@ -364,6 +364,9 @@ void __cpuinit xen_init_lock_cpu(int cpu) | |||
364 | int irq; | 364 | int irq; |
365 | const char *name; | 365 | const char *name; |
366 | 366 | ||
367 | WARN(per_cpu(lock_kicker_irq, cpu) > 0, "spinlock on CPU%d exists on IRQ%d!\n", | ||
368 | cpu, per_cpu(lock_kicker_irq, cpu)); | ||
369 | |||
367 | name = kasprintf(GFP_KERNEL, "spinlock%d", cpu); | 370 | name = kasprintf(GFP_KERNEL, "spinlock%d", cpu); |
368 | irq = bind_ipi_to_irqhandler(XEN_SPIN_UNLOCK_VECTOR, | 371 | irq = bind_ipi_to_irqhandler(XEN_SPIN_UNLOCK_VECTOR, |
369 | cpu, | 372 | cpu, |
@@ -383,6 +386,7 @@ void __cpuinit xen_init_lock_cpu(int cpu) | |||
383 | void xen_uninit_lock_cpu(int cpu) | 386 | void xen_uninit_lock_cpu(int cpu) |
384 | { | 387 | { |
385 | unbind_from_irqhandler(per_cpu(lock_kicker_irq, cpu), NULL); | 388 | unbind_from_irqhandler(per_cpu(lock_kicker_irq, cpu), NULL); |
389 | per_cpu(lock_kicker_irq, cpu) = -1; | ||
386 | } | 390 | } |
387 | 391 | ||
388 | void __init xen_init_spinlocks(void) | 392 | void __init xen_init_spinlocks(void) |