From: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>,
Tony Lindgren <tony@atomide.com>, Petr Mladek <pmladek@suse.com>,
Raul Rangel <rrangel@google.com>,
Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>,
linux-kernel <linux-kernel@vger.kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
kurt@linutronix.de, "S, Shirish" <Shirish.S@amd.com>,
Peter Zijlstra <peterz@infradead.org>,
John Ogness <john.ogness@linutronix.de>,
Steven Rostedt <rostedt@goodmis.org>
Subject: Re: UART/TTY console deadlock
Date: Fri, 3 Jul 2020 19:53:02 +0900 [thread overview]
Message-ID: <20200703105302.GC182102@jagdpanzerIV.localdomain> (raw)
In-Reply-To: <20200702082008.GE3703480@smile.fi.intel.com>
On (20/07/02 11:20), Andy Shevchenko wrote:
>
> I didn't look into this deeply, but my understanding that this is something for
> special case when you have several UART ports sharing the IRQ (multi-port card)
> and IRQ even maybe undesirable b/c it will confuse real IRQ handler. I don't
> remember details, but AFAIR IRQ handler does a busyloop to service as much as
> possible and in between it may release the lock (again, multi-port UART cards),
> that's why we better avoid IRQ event in the first place.
> But it's my pure speculation.
Hmm. Would different IRQ handlers use different UART port structures? ->irq
might be the only thing they share. Even if they use the same port, we
keep IRQs disabled on local CPU, and UART port locked. To unlock the UART
port during IRQ, the handler (on another CPU) first needs to acquire the
port->lock, which is locked by serial8250_do_startup().
-ss
next prev parent reply other threads:[~2020-07-03 10:53 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-22 17:30 UART/TTY console deadlock Raul Rangel
2020-06-22 17:37 ` Andy Shevchenko
2020-06-30 3:58 ` Sergey Senozhatsky
2020-06-30 10:21 ` Petr Mladek
2020-06-30 10:55 ` Sergey Senozhatsky
2020-06-30 11:40 ` Andy Shevchenko
2020-06-30 12:22 ` Petr Mladek
2020-06-30 13:05 ` Sergey Senozhatsky
2020-06-30 18:02 ` Tony Lindgren
2020-07-01 6:44 ` S, Shirish
2020-07-02 3:48 ` S, Shirish
2020-07-02 6:11 ` Greg Kroah-Hartman
2020-07-02 6:14 ` S, Shirish
2020-07-02 6:34 ` Sergey Senozhatsky
2020-07-02 7:11 ` Greg Kroah-Hartman
2020-07-02 5:12 ` Sergey Senozhatsky
2020-07-02 5:40 ` Sergey Senozhatsky
2020-07-02 8:20 ` Andy Shevchenko
2020-07-03 10:53 ` Sergey Senozhatsky [this message]
2020-07-04 11:37 ` Andy Shevchenko
2020-07-02 16:05 ` Tony Lindgren
2020-07-03 10:32 ` Sergey Senozhatsky
2020-07-04 11:35 ` Andy Shevchenko
2020-07-04 11:59 ` Andy Shevchenko
2020-07-06 11:31 ` Kurt Kanzenbach
2020-07-06 14:43 ` Sergey Senozhatsky
2020-07-08 7:40 ` Kurt Kanzenbach
2020-07-08 8:07 ` Sergey Senozhatsky
2020-07-08 9:52 ` Petr Mladek
2020-07-09 13:22 ` Sergey Senozhatsky
2020-07-14 16:16 ` Raul Rangel
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=20200703105302.GC182102@jagdpanzerIV.localdomain \
--to=sergey.senozhatsky@gmail.com \
--cc=Shirish.S@amd.com \
--cc=andy.shevchenko@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=john.ogness@linutronix.de \
--cc=kurt@linutronix.de \
--cc=linux-kernel@vger.kernel.org \
--cc=peterz@infradead.org \
--cc=pmladek@suse.com \
--cc=rostedt@goodmis.org \
--cc=rrangel@google.com \
--cc=sergey.senozhatsky.work@gmail.com \
--cc=tony@atomide.com \
/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.