Martin K. Petersen wrote:
"Matthew" == Matthew Wilcox <matthew@xxxxxx> writes:Matthew> I already did that. The only problem is that you made me Matthew> include the stupid: Matthew> if (ata_id_major_version(args->id) > 7) { Matthew> so of course it doesn't work on any existing hardware. How Matthew> about applying this patch: Maybe we could incubate your patch in the next tree for a bit and see what breaks without the version check? We could even be somewhat conservative like we were with RC16 in SCSI. The SATA devices I have here with valid rotational flags all report version 7. I wonder if > 6 do the trick?
linux/ata.h illustrates the standard ATA rules for validating bits of IDENTIFY DEVICE.
Just checking the version was always just a simplistic hack... we are talking specifically about trusting values listed as undefined in the relevant specs. That requires more, not less, gymnastics :)
Jeff -- 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
