Skip to contentSkip to conversation

libata-dev: filter out noisy ATAPI error messages

PATCH In review Albert Lee 1 participant 1 message Dec 26, 2005

Subsystems (2)
lore
Share & tools
lore

Albert Lee

author
#0 lore raw

PATCH 2/2 libata-dev: filter out noisy ATAPI error messages

Jeff,

Patch 2/2: Minor patch to filter out noisy ATAPI error messages.

Changes:
- Filter out ATAPI packet command error messages in ata_pio_error()
- Filter out ATAPI packet command error messages in ata_host_intr()

Patch against the libata-dev irq-pio branch
(3d0a59c02303df01848537b3bf938dc11e9a0ded).

For your review, thanks.

Albert

======

drivers/scsi/libata-core.c +6 -4
@@ -3464,11 +3464,12 @@ static void ata_pio_error(struct ata_por
 {
 	struct ata_queued_cmd *qc;
 
-	printk(KERN_WARNING "ata%u: PIO error\n", ap->id);
-
 	qc = ata_qc_from_tag(ap, ap->active_tag);
 	assert(qc != NULL);
 
+	if (qc->tf.command != ATA_CMD_PACKET)
+		printk(KERN_WARNING "ata%u: PIO error\n", ap->id);
+
 	/* make sure qc->err_mask is available to 
 	 * know what's wrong and recover
 	 */
@@ -4348,8 +4349,9 @@ fsm_start:
 		break;
 
 	case HSM_ST_ERR:
-		printk(KERN_ERR "ata%u: command error, drv_stat 0x%x host_stat 0x%x\n",
-		       ap->id, status, host_stat);
+		if (qc->tf.command != ATA_CMD_PACKET)
+			printk(KERN_ERR "ata%u: command error, drv_stat 0x%x host_stat 0x%x\n",
+			       ap->id, status, host_stat);
 
 		/* make sure qc->err_mask is available to 
 		 * know what's wrong and recover