diff mbox

[5/5] libata: allow hyphenated pattern ranges (v2)

Message ID 4C326271.4040005@teksavvy.com
State Not Applicable
Delegated to: David Miller
Headers show

Commit Message

Mark Lord July 5, 2010, 10:53 p.m. UTC 
Take further advantage of the new glob_match() function to reduce the blacklist size,
this time making use of "character ranges" in the patterns.

Signed-off-by: Mark Lord <mlord@pobox.com>
---
No change from original post -- false alarm.
(also attached to bypass issues with my b0rked email client).

Comments

Jeff Garzik July 14, 2010, 7:54 a.m. UTC | #1 
On 07/05/2010 06:53 PM, Mark Lord wrote:
> Take further advantage of the new glob_match() function to reduce the
> blacklist size,
> this time making use of "character ranges" in the patterns.
>
> Signed-off-by: Mark Lord <mlord@pobox.com>
> ---
> No change from original post -- false alarm.
> (also attached to bypass issues with my b0rked email client).

applied #4 and #5

had to utilize the attachments, as mail client still does not seem to be 
cooperating.  otherwise, patches applied and lightly tested without a hitch.


--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

Take further advantage of the new glob_match() function to reduce the blacklist size,
this time making use of "character ranges" in the patterns.

Signed-off-by: Mark Lord <mlord@pobox.com>
---

(also attached to bypass issues with my b0rked email client).

--- old/drivers/ata/libata-core.c	2010-07-05 10:23:51.198204000 -0400
+++ linux/drivers/ata/libata-core.c	2010-07-05 18:47:18.817518530 -0400
@@ -4209,22 +4209,16 @@ 
 	{ "OCZ CORE_SSD",	"02.10104",	ATA_HORKAGE_NONCQ },
 
 	/* Seagate NCQ + FLUSH CACHE firmware bug */
-	{ "ST31500341AS",	"SD1[56789]",	ATA_HORKAGE_NONCQ |
+	{ "ST31500341AS",	"SD1[5-9]",	ATA_HORKAGE_NONCQ |
 						ATA_HORKAGE_FIRMWARE_WARN },
 
-	{ "ST31000333AS",	"SD1[56789]",	ATA_HORKAGE_NONCQ |
+	{ "ST31000333AS",	"SD1[5-9]",	ATA_HORKAGE_NONCQ |
 						ATA_HORKAGE_FIRMWARE_WARN },
 
-	{ "ST3640623AS",	"SD1[56789]",	ATA_HORKAGE_NONCQ |
+	{ "ST3640[36]23AS",	"SD1[5-9]",	ATA_HORKAGE_NONCQ |
 						ATA_HORKAGE_FIRMWARE_WARN },
 
-	{ "ST3640323AS",	"SD1[56789]",	ATA_HORKAGE_NONCQ |
-						ATA_HORKAGE_FIRMWARE_WARN },
-
-	{ "ST3320813AS",	"SD1[56789]",	ATA_HORKAGE_NONCQ |
-						ATA_HORKAGE_FIRMWARE_WARN },
-
-	{ "ST3320613AS",	"SD1[56789]",	ATA_HORKAGE_NONCQ |
+	{ "ST3320[68]13AS",	"SD1[5-9]",	ATA_HORKAGE_NONCQ |
 						ATA_HORKAGE_FIRMWARE_WARN },
 
 	/* Blacklist entries taken from Silicon Image 3124/3132