From: Gregoire Gentil <gregoire@gentil.com>
To: linux-rt-users@vger.kernel.org
Subject: "g_serial: fix deadlock with PREEMPT_RT enabled" still not integrated
Date: Sun, 02 Sep 2012 23:10:23 -0700 [thread overview]
Message-ID: <504449CF.5050604@gentil.com> (raw)
In-Reply-To: <20120902194911.GT28643@pengutronix.de>
Hello,
I think that the g_serial patch is still not integrated and it's still
needed at least as I experienced it on 3.4-rt17 on ARM:
http://www.spinics.net/lists/linux-rt-users/msg07158.html
Here is an update of the patch:
--- a/drivers/usb/gadget/u_serial.c 2012-08-29 12:17:48.607922510 -0700
+++ b/drivers/usb/gadget/u_serial.c 2012-08-29 12:23:21.305572267 -0700
@@ -554,7 +554,15 @@
* a workqueue, so we won't get callbacks and can hold port_lock
*/
if (tty && do_push)
- tty_flip_buffer_push(tty);
+ /*
+ * Drop the lock here since it might end up calling
+ * gs_flush_chars, which takes the lock.
+ */
+ spin_unlock_irq(&port->port_lock);
+ tty_flip_buffer_push(tty);
+ spin_lock_irq(&port->port_lock);
+ /* tty may have been closed */
+ tty = port->port_tty;
/* We want our data queue to become empty ASAP, keeping data
Grégoire
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2012-09-03 6:16 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-13 20:48 Don't reference non-standard realtime group Uwe Kleine-König
2012-09-02 19:49 ` Uwe Kleine-König
2012-09-03 6:10 ` Gregoire Gentil [this message]
2012-09-03 6:24 ` TI wl1271 wireless bug with 3.4-rt17 Gregoire Gentil
2012-09-04 14:11 ` Josh Cartwright
2012-09-05 1:22 ` Gregoire Gentil
2012-09-06 3:48 ` Josh Cartwright
2012-09-06 4:22 ` Gregoire Gentil
2012-09-09 22:42 ` Gregoire Gentil
2012-09-09 22:51 ` how to put in higher priority a thread in rt-kernel? Gregoire Gentil
2012-09-10 3:52 ` Mike Galbraith
2012-09-10 5:23 ` Gregoire Gentil
2012-09-10 16:20 ` Clark Williams
2012-10-15 23:13 ` Don't reference non-standard realtime group John Kacur
2012-10-16 18:26 ` Josh Cartwright
2012-10-18 16:13 ` Josh Cartwright
2012-10-19 2:12 ` Clark Williams
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=504449CF.5050604@gentil.com \
--to=gregoire@gentil.com \
--cc=linux-rt-users@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.