lkml.org 
[lkml]   [2004]   [Sep]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 2.6.9-rc2-mm3
On Fri, Sep 24, 2004 at 10:25:11PM -0500, Paul Fulghum wrote:
> My suggestion was flawed in that it could
> violate POSIX requirements (as Russell pointed out).
>
> Removing the lock from tty_termios_baud_rate(), tty_io.c
> corrects the problem for the path from change_termios()
> to tty_termios_baud_rate(), which is causing the deadlock.
>
> This may not be, and probably is not,
> correct for all paths to tty_termios_baud_rate().

I wonder if we should consider adding:

WARN_ON(!spin_is_locked(&tty_termios_lock));

in there.

However, the one annoying thing about "spin_is_locked" is that, on UP,
it defaults to "unlocked" which makes these kinds of checks too noisy.
Maybe we need a spin_is_locked() with a bias towards being locked for UP?

--
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
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:06    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site