Mark Lord wrote:
Randy.Dunlap wrote:
From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
Rate-limit a translated ATA status/error message.
I have a kernel log that is 2/3 full of these messages
due to a CD/DVD drive being empty.
...
- printk(KERN_ERR "ata%u: translated ATA stat/err 0x%02x/%02x to "
- "SCSI SK/ASC/ASCQ 0x%x/%02x/%02x\n", id, drv_stat, drv_err,
- *sk, *asc, *ascq);
+ if (printk_ratelimit())
+ printk(KERN_ERR "ata%u: translated ATA stat/err 0x%02x/%02x to "
+ "SCSI SK/ASC/ASCQ 0x%x/%02x/%02x\n",
+ id, drv_stat, drv_err, *sk, *asc, *ascq);
That message shouldn't even be enabled for non DEBUG cases.
I suspect Jeff will remove it for 2.6.17..
Kinda sorta: ATAPI errors are still shaking out, in a big way. Under
ATA, almost any error should be reported. Under ATAPI, certain classes
of errors are quite normal, and should not fill the logs with spam.
The proper fix is to avoid reporting errors for those few ATAPI cases
that occur in normal operation, rather than ratelimiting all libata
error messages.
IOW don't report an error, when it isn't really an error...
Jeff
-
: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
