> > - if (print_info) { > > + if (print_info && !(ata_device_blacklisted(dev) & ATA_HORKAGE_DIAGNOSTIC)) { > > ata_dev_printk(dev, KERN_WARNING, > > "Drive reports diagnostics failure. This may indicate a drive\n"); > > ata_dev_printk(dev, KERN_WARNING, > > I don't understand this one at all. > > The original point of the message was to poke people so that they poke > their vendors into fixing the firmware, which is indeed non-spec. I > would rather not silence a "your device is operating out-of-spec" warning. The moment it goes near a business oriented distribution this becomes a problem because the flack all lands on either the system vendor or the OS product vendor both of whom will simply delete the check. At least this way it stays in for cases that actually matter. Alan - To unsubscribe from this list: 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