]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
3.17-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 14 Oct 2014 01:26:24 +0000 (03:26 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 14 Oct 2014 01:26:24 +0000 (03:26 +0200)
added patches:
libata-un-break-ata-blacklist.patch

queue-3.17/libata-un-break-ata-blacklist.patch [new file with mode: 0644]
queue-3.17/series

diff --git a/queue-3.17/libata-un-break-ata-blacklist.patch b/queue-3.17/libata-un-break-ata-blacklist.patch
new file mode 100644 (file)
index 0000000..8e47953
--- /dev/null
@@ -0,0 +1,53 @@
+From 1c40279960bcd7d52dbdf1d466b20d24b99176c8 Mon Sep 17 00:00:00 2001
+From: George Spelvin <linux@horizon.com>
+Date: Tue, 7 Oct 2014 07:26:38 -0400
+Subject: libata: Un-break ATA blacklist
+
+From: George Spelvin <linux@horizon.com>
+
+commit 1c40279960bcd7d52dbdf1d466b20d24b99176c8 upstream.
+
+lib/glob.c provides a new glob_match() function, with arguments in
+(pattern, string) order.  It replaced a private function with arguments
+in (string, pattern) order, but I didn't swap the call site...
+
+The result was the entire ATA blacklist was effectively disabled.
+
+The lesson for today is "I f***ed up *how* badly *how* many months ago?",
+er, I mean "Nobody Tests RC Kernels On Legacy Hardware".
+
+This was not a subtle break, but it made it through an entire RC
+cycle unreported, presumably because all the people doing testing
+have full-featured hardware.
+
+(FWIW, the reason for the argument swap was because fnmatch() does it that
+way, and for a while implementing a full fnmatch() was being considered.)
+
+Fixes: 428ac5fc056e0 (libata: Use glob_match from lib/glob.c)
+Reported-by: Steven Honeyman <stevenhoneyman@gmail.com>
+Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=71371#c21
+Signed-off-by: George Spelvin <linux@horizon.com>
+Tested-by: Steven Honeyman <stevenhoneyman@gmail.com>
+Signed-off-by: Tejun Heo <tj@kernel.org>
+Cc: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+
+---
+ drivers/ata/libata-core.c |    4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+--- a/drivers/ata/libata-core.c
++++ b/drivers/ata/libata-core.c
+@@ -4261,10 +4261,10 @@ static unsigned long ata_dev_blacklisted
+       ata_id_c_string(dev->id, model_rev, ATA_ID_FW_REV, sizeof(model_rev));
+       while (ad->model_num) {
+-              if (glob_match(model_num, ad->model_num)) {
++              if (glob_match(ad->model_num, model_num)) {
+                       if (ad->model_rev == NULL)
+                               return ad->horkage;
+-                      if (glob_match(model_rev, ad->model_rev))
++                      if (glob_match(ad->model_rev, model_rev))
+                               return ad->horkage;
+               }
+               ad++;
index a2babbd1e9d06049f22da82584598e37b97f7444..8033f409a472492155fef96822486a100c5e43de 100644 (file)
@@ -23,3 +23,4 @@ usb-add-device-quirk-for-asus-t100-base-station-keyboard.patch
 crypto-caam-fix-addressing-of-struct-member.patch
 driver-base-node-remove-unnecessary-kfree-of-node-struct-from-unregister_one_node.patch
 serial-8250-add-quark-x1000-to-8250_pci.c.patch
+libata-un-break-ata-blacklist.patch