From: Johan Hovold <jhovold@gmail.com>
Date: Mon, 26 May 2014 17:23:40 +0000 (+0200)
Subject: USB: cdc-acm: fix shutdown and suspend race
X-Git-Tag: submit/tizen_common/20140730.172411~458
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7dfa070023fee95f181e0cd2bec1cb57417f41e2;p=profile%2Fivi%2Fkernel-x86-ivi.git

USB: cdc-acm: fix shutdown and suspend race

commit ed797074031a37bb9bf4a70952fffc606b77274d upstream.

We should stop I/O unconditionally at suspend rather than rely on the
tty-port initialised flag (which is set prior to stopping I/O during
shutdown) in order to prevent suspend returning with URBs still active.

Fixes: 11ea859d64b6 ("USB: additional power savings for cdc-acm devices
that support remote wakeup")

Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c
index 9a3a895..61738d9 100644
--- a/drivers/usb/class/cdc-acm.c
+++ b/drivers/usb/class/cdc-acm.c
@@ -1548,8 +1548,7 @@ static int acm_suspend(struct usb_interface *intf, pm_message_t message)
 	if (cnt)
 		return 0;
 
-	if (test_bit(ASYNCB_INITIALIZED, &acm->port.flags))
-		stop_data_traffic(acm);
+	stop_data_traffic(acm);
 
 	return 0;
 }