[SCSI] qla4xxx: Add spurious interrupt messages under debug level 2
authorNilesh Javali <nilesh.javali@qlogic.com>
Mon, 21 Jan 2013 04:50:56 +0000 (23:50 -0500)
committerJames Bottomley <JBottomley@Parallels.com>
Wed, 30 Jan 2013 02:32:25 +0000 (13:32 +1100)
The spurious interrupt messages are seen when interrupt lines are
shared between different adapters or modules. Example, interrupt
lines are shared between qlcnic and qla4xxx and USB modules, then the
console is flooded with flurry of spurious interrupt messages
which are expected in such scenario.

Hence put these messages under debug level 2 for INTx interrupt mode.

Signed-off-by: Nilesh Javali <nilesh.javali@qlogic.com>
Signed-off-by: Vikas Chaudhary <vikas.chaudhary@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/qla4xxx/ql4_isr.c

index acbc2feaa0058d8f188815ff2db1ae9d6f595d3b..39669b5cea1c6ceee32ff2ef7cbf9dbb34ce31be 100644 (file)
@@ -1124,17 +1124,18 @@ irqreturn_t qla4_83xx_intr_handler(int irq, void *dev_id)
 
        /* Legacy interrupt is valid if bit31 of leg_int_ptr is set */
        if (!(leg_int_ptr & LEG_INT_PTR_B31)) {
-               ql4_printk(KERN_ERR, ha,
-                          "%s: Legacy Interrupt Bit 31 not set, spurious interrupt!\n",
-                          __func__);
+               DEBUG2(ql4_printk(KERN_ERR, ha,
+                                 "%s: Legacy Interrupt Bit 31 not set, spurious interrupt!\n",
+                                 __func__));
                return IRQ_NONE;
        }
 
        /* Validate the PCIE function ID set in leg_int_ptr bits [19..16] */
        if ((leg_int_ptr & PF_BITS_MASK) != ha->pf_bit) {
-               ql4_printk(KERN_ERR, ha,
-                          "%s: Incorrect function ID 0x%x in legacy interrupt register, ha->pf_bit = 0x%x\n",
-                          __func__, (leg_int_ptr & PF_BITS_MASK), ha->pf_bit);
+               DEBUG2(ql4_printk(KERN_ERR, ha,
+                                 "%s: Incorrect function ID 0x%x in legacy interrupt register, ha->pf_bit = 0x%x\n",
+                                 __func__, (leg_int_ptr & PF_BITS_MASK),
+                                 ha->pf_bit));
                return IRQ_NONE;
        }