public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Russell King <rmk+lkml@arm.linux.org.uk>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: "Andrew Morton" <akpm@osdl.org>,
	"Martin Kögler" <e9925248@student.tuwien.ac.at>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>
Subject: Re: Deadlock in serial driver 2.6.x
Date: Sun, 30 Jan 2005 16:48:40 +0000	[thread overview]
Message-ID: <20050130164840.D25000@flint.arm.linux.org.uk> (raw)
In-Reply-To: <1106844084.14782.45.camel@localhost.localdomain>; from alan@lxorguk.ukuu.org.uk on Sun, Jan 30, 2005 at 03:39:32PM +0000

On Sun, Jan 30, 2005 at 03:39:32PM +0000, Alan Cox wrote:
> On Iau, 2005-01-27 at 07:13, Andrew Morton wrote:
> > Martin Kögler <e9925248@student.tuwien.ac.at> wrote:
> > (For some reason the NMI watchdog isn't triggering here, and it's still
> > taking interrupts).
> 
> > Looks like low-latency mode is busted.
> 
> low latency mode is fine, the drivers/serial layer is busted. It workd
> fine with non drivers/serial using hardware still, and it worked fine in
> 2.4

Unfortunately it creates a major locking problem.  We need to hold the
spinlock because we're using the port structures and relying on things
like the tty structure staying around during the interrupt processing.

We call into the tty layer, and the tty layer calls us back via the
write method, (for echo purposes) and that's indistinguishable from
user-level or other kernel-level writes.

Unsolvable as the tty layer currently stands.  tty needs to not call back
into serial drivers when they supply read characters from their interrupt
functions.

tty layer's busted. 8)

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:  2.6 PCMCIA      - http://pcmcia.arm.linux.org.uk/
                 2.6 Serial core

  reply	other threads:[~2005-01-30 16:49 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-26 13:20 Deadlock in serial driver 2.6.x Martin Kögler
2005-01-27  7:13 ` Andrew Morton
2005-01-30 15:39   ` Alan Cox
2005-01-30 16:48     ` Russell King [this message]
2005-01-31  7:37       ` Alan Cox
2005-01-31  8:48         ` Andrew Morton
2005-02-03 10:02           ` Alan Cox
2005-02-03 18:21             ` Andrew Morton
2005-02-04 11:07               ` Martin Kögler
2005-02-04 13:50                 ` Paul Fulghum

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=20050130164840.D25000@flint.arm.linux.org.uk \
    --to=rmk+lkml@arm.linux.org.uk \
    --cc=akpm@osdl.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=e9925248@student.tuwien.ac.at \
    --cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox