This is a note to let you know that I've just added the patch titled

    USB: option: reduce interrupt-urb logging verbosity

to the 3.16-stable tree which can be found at:
    
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     usb-option-reduce-interrupt-urb-logging-verbosity.patch
and it can be found in the queue-3.16 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <[email protected]> know about it.


>From f0e4cba2534cd88476dff920727c81350130f3c5 Mon Sep 17 00:00:00 2001
From: Johan Hovold <[email protected]>
Date: Tue, 29 Jul 2014 14:14:55 +0200
Subject: USB: option: reduce interrupt-urb logging verbosity

From: Johan Hovold <[email protected]>

commit f0e4cba2534cd88476dff920727c81350130f3c5 upstream.

Do not log normal interrupt-urb shutdowns as errors.

The option driver has always been logging any nonzero interrupt-urb
status as an error, including when the urb is killed during normal
operation.

Commit 9096f1fbba91 ("USB: usb_wwan: fix potential NULL-deref at
resume") moved the interrupt urb submission from port probe and release
to open and close, thus potentially increasing the number of these
false-positive error messages dramatically.

Reported-by: Ed Butler <[email protected]>
Tested-by: Ed Butler <[email protected]>
Signed-off-by: Johan Hovold <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>

---
 drivers/usb/serial/option.c |    2 ++
 1 file changed, 2 insertions(+)

--- a/drivers/usb/serial/option.c
+++ b/drivers/usb/serial/option.c
@@ -1916,6 +1916,8 @@ static void option_instat_callback(struc
                        dev_dbg(dev, "%s: type %x req %x\n", __func__,
                                req_pkt->bRequestType, req_pkt->bRequest);
                }
+       } else if (status == -ENOENT || status == -ESHUTDOWN) {
+               dev_dbg(dev, "%s: urb stopped: %d\n", __func__, status);
        } else
                dev_err(dev, "%s: error %d\n", __func__, status);
 


Patches currently in stable-queue which might be from [email protected] are

queue-3.16/usb-serial-fix-potential-heap-buffer-overflow.patch
queue-3.16/revert-usb-option-zte_ev-move-most-zte-cdma-devices-to-zte_ev.patch
queue-3.16/usb-zte_ev-remove-duplicate-qualcom-pid.patch
queue-3.16/usb-serial-fix-potential-stack-buffer-overflow.patch
queue-3.16/usb-option-add-via-telecom-cds7-chipset-device-id.patch
queue-3.16/usb-option-reduce-interrupt-urb-logging-verbosity.patch
queue-3.16/usb-serial-pl2303-add-device-id-for-ztek-device.patch
queue-3.16/usb-zte_ev-remove-duplicate-gobi-pid.patch
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to