aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Beulich <jbeulich@suse.com>2013-06-10 13:43:03 +0200
committerJan Beulich <jbeulich@suse.com>2013-06-10 13:43:03 +0200
commitac9e298cb4bda0238f50da814b8af2b90dc758a1 (patch)
treef3d24865f891f960846a3e91426796cb3fc8b2a5
parente430510e5cbbfcdc1077739292def633e70fedea (diff)
downloadxen-ac9e298cb4bda0238f50da814b8af2b90dc758a1.tar.gz
xen-ac9e298cb4bda0238f50da814b8af2b90dc758a1.tar.bz2
xen-ac9e298cb4bda0238f50da814b8af2b90dc758a1.zip
Revert "irq: Add extra debugging to help track down why an assertion is failing"
This reverts commits 2ae8b9173fb2388af6514c730d620ed5f450bc34 and 98e10364bde098e12104caa4f566b17d05f8b791. This was never reported to be hit, and we assume to have taken care of the problem by excluding legacy IRQs from the IRQ move cleanup logic. Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com> Signed-off-by: Jan Beulich <jbeulich@suse.com> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com> Acked-by: George Dunlap <george.dunlap@eu.citrix.com>
-rw-r--r--xen/arch/x86/irq.c16
1 files changed, 1 insertions, 15 deletions
diff --git a/xen/arch/x86/irq.c b/xen/arch/x86/irq.c
index 699221ae99..a4da7868cb 100644
--- a/xen/arch/x86/irq.c
+++ b/xen/arch/x86/irq.c
@@ -624,8 +624,6 @@ void move_native_irq(struct irq_desc *desc)
desc->handler->enable(desc);
}
-static void dump_irqs(unsigned char key);
-
void irq_move_cleanup_interrupt(struct cpu_user_regs *regs)
{
unsigned vector, me;
@@ -687,19 +685,7 @@ void irq_move_cleanup_interrupt(struct cpu_user_regs *regs)
if ( desc->arch.used_vectors )
{
- if ( unlikely(!test_bit(vector, desc->arch.used_vectors)) )
- {
- spin_unlock(&desc->lock);
- bitmap_scnlistprintf(keyhandler_scratch,
- sizeof(keyhandler_scratch),
- desc->arch.used_vectors->_bits,
- NR_VECTORS);
- printk("*** IRQ BUG found ***\n"
- "CPU%d -Testing vector %d from bitmap %s\n",
- me, vector, keyhandler_scratch);
- dump_irqs('i');
- BUG();
- }
+ ASSERT(test_bit(vector, desc->arch.used_vectors));
clear_bit(vector, desc->arch.used_vectors);
}
}