lkml.org 
[lkml]   [2006]   [Jun]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: About the fixes of /drivers/serial/8250.C in 2.6.17-rc6 for avoiding habbg-up
On Mon, Jun 19, 2006 at 10:23:14AM +0100, Alan Cox wrote:
> Ar Llu, 2006-06-19 am 12:03 +0900, ysgrifennodd gouji:
> > In /drivers/serial/8250.C of 2.6.17-rc6,
> >
> > these fixes are adapted for avoinding the problem of hang-up
> > while TTY write and console write from kernel conflicted.
>
> Yes, there is a bug in this version that was not in the one I submitted,
> someone added an improvement.

I disagree - in the non-oops_in_progress case, your version and the
merged version are 100% identical - see

http://marc.theaimsgroup.com/?l=linux-kernel&m=114657841432447&w=2

However, you never responded to my answers to your two questions in
that email, which came with the patch which was merged.

> > + if (oops_in_progress) {
> > + locked = spin_trylock_irqsave(&up->port.lock, flags);
> > + } else
> > + spin_lock_irqsave(&up->port.lock, flags);
> > +
>
> It could always use spin_trylock_irqsave(). The oops in progress
> optimisation makes some sense initially but there are many console
> printk users that can occur during serial I/O in exceptional cases.
>
> It's not an easy problem to solve with the current serial locking.

I don't have the initial email from ysgrifennodd gouji, and neither
do the lkml archives. What's the problem?

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 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: 2006-06-19 11:34    [W:0.176 / U:0.160 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site