diff options
author | Albert Lee <albertcc@tw.ibm.com> | 2005-12-26 03:48:00 -0500 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2006-01-17 19:55:03 -0500 |
commit | 000080c3499cd5037e60c08a8053efb9e48aa9c0 (patch) | |
tree | 9b3534675d986ad1d2982bc78a93f1a3912d61bc | |
parent | a4f16610081001640ffe0314024bc31c10f69757 (diff) |
[PATCH] libata-dev: 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()
Signed-off-by: Albert Lee <albertcc@tw.ibm.com>
======
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
-rw-r--r-- | drivers/scsi/libata-core.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c index 41f76b9bf992..911151db8c25 100644 --- a/drivers/scsi/libata-core.c +++ b/drivers/scsi/libata-core.c | |||
@@ -3520,11 +3520,12 @@ static void ata_pio_error(struct ata_port *ap) | |||
3520 | { | 3520 | { |
3521 | struct ata_queued_cmd *qc; | 3521 | struct ata_queued_cmd *qc; |
3522 | 3522 | ||
3523 | printk(KERN_WARNING "ata%u: PIO error\n", ap->id); | ||
3524 | |||
3525 | qc = ata_qc_from_tag(ap, ap->active_tag); | 3523 | qc = ata_qc_from_tag(ap, ap->active_tag); |
3526 | assert(qc != NULL); | 3524 | assert(qc != NULL); |
3527 | 3525 | ||
3526 | if (qc->tf.command != ATA_CMD_PACKET) | ||
3527 | printk(KERN_WARNING "ata%u: PIO error\n", ap->id); | ||
3528 | |||
3528 | /* make sure qc->err_mask is available to | 3529 | /* make sure qc->err_mask is available to |
3529 | * know what's wrong and recover | 3530 | * know what's wrong and recover |
3530 | */ | 3531 | */ |
@@ -4404,8 +4405,9 @@ fsm_start: | |||
4404 | break; | 4405 | break; |
4405 | 4406 | ||
4406 | case HSM_ST_ERR: | 4407 | case HSM_ST_ERR: |
4407 | printk(KERN_ERR "ata%u: command error, drv_stat 0x%x host_stat 0x%x\n", | 4408 | if (qc->tf.command != ATA_CMD_PACKET) |
4408 | ap->id, status, host_stat); | 4409 | printk(KERN_ERR "ata%u: command error, drv_stat 0x%x host_stat 0x%x\n", |
4410 | ap->id, status, host_stat); | ||
4409 | 4411 | ||
4410 | /* make sure qc->err_mask is available to | 4412 | /* make sure qc->err_mask is available to |
4411 | * know what's wrong and recover | 4413 | * know what's wrong and recover |