[Xen-devel] [PATCH] Revert "irq: Add extra debugging to help track down why an assertion is failing"

["Jan Beulich" <JBeulich@xxxxxxxx>]

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@xxxxxxxxxx>
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

--- a/xen/arch/x86/irq.c
+++ b/xen/arch/x86/irq.c
@@ -624,8 +624,6 @@ void move_native_irq(struct irq_desc *de
     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 c
 
             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);
             }
         }

Attachment: x86-revert-irq-debugging.patch
Description: Text document

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel