Commit ac70a964 authored by Tejun Heo's avatar Tejun Heo Committed by Jeff Garzik
Browse files

libata: blacklist Seagate drives which time out FLUSH_CACHE when used with NCQ 

Some recent Seagate harddrives have firmware bug which causes FLUSH
CACHE to timeout under certain circumstances if NCQ is being used.
This can be worked around by disabling NCQ and fixed by updating the
firmware.  Implement ATA_HORKAGE_FIRMWARE_UPDATE and blacklist these
devices.

The wiki page has been updated to contain information on this issue.

  http://ata.wiki.kernel.org/index.php/Known_issues



Signed-off-by: default avatarTejun Heo <tj@kernel.org>
Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
parent 03f60840

drivers/ata/libata-core.c

+21 −0
Original line number Diff line number Diff line
@@ -2492,6 +2492,13 @@ int ata_dev_configure(struct ata_device *dev) 
		}

	}



	if ((dev->horkage & ATA_HORKAGE_FIRMWARE_WARN) && print_info) {

		ata_dev_printk(dev, KERN_WARNING, "WARNING: device requires "

			       "firmware update to be fully functional.\n");

		ata_dev_printk(dev, KERN_WARNING, "         contact the vendor "

			       "or visit http://ata.wiki.kernel.org.\n");

	}



	return 0;



err_out_nosup:
@@ -4042,6 +4049,20 @@ static const struct ata_blacklist_entry ata_device_blacklist [] = { 
	{ "ST380817AS",		"3.42",		ATA_HORKAGE_NONCQ },

	{ "ST3160023AS",	"3.42",		ATA_HORKAGE_NONCQ },



	/* Seagate NCQ + FLUSH CACHE firmware bug */

	{ "ST31500341AS",	"9JU138",	ATA_HORKAGE_NONCQ |

						ATA_HORKAGE_FIRMWARE_WARN },

	{ "ST31000333AS",	"9FZ136",	ATA_HORKAGE_NONCQ |

						ATA_HORKAGE_FIRMWARE_WARN },

	{ "ST3640623AS",	"9FZ164",	ATA_HORKAGE_NONCQ |

						ATA_HORKAGE_FIRMWARE_WARN },

	{ "ST3640323AS",	"9FZ134",	ATA_HORKAGE_NONCQ |

						ATA_HORKAGE_FIRMWARE_WARN },

	{ "ST3320813AS",	"9FZ182",	ATA_HORKAGE_NONCQ |

						ATA_HORKAGE_FIRMWARE_WARN },

	{ "ST3320613AS",	"9FZ162",	ATA_HORKAGE_NONCQ |

						ATA_HORKAGE_FIRMWARE_WARN },



	/* Blacklist entries taken from Silicon Image 3124/3132

	   Windows driver .inf file - also several Linux problem reports */

	{ "HTS541060G9SA00",    "MB3OC60D",     ATA_HORKAGE_NONCQ, },

include/linux/libata.h

+1 −0
Original line number Diff line number Diff line
@@ -375,6 +375,7 @@ enum { 
	ATA_HORKAGE_BRIDGE_OK	= (1 << 10),	/* no bridge limits */

	ATA_HORKAGE_ATAPI_MOD16_DMA = (1 << 11), /* use ATAPI DMA for commands

						    not multiple of 16 bytes */

	ATA_HORKAGE_FIRMWARE_WARN = (1 << 12),	/* firwmare update warning */



	 /* DMA mask for user DMA control: User visible values; DO NOT

	    renumber */

CRA Git | Maintained and supported by SUSTech CRA and CCSE