Re: [Patch] 2.4.28-pre3 tty/ldisc fixes

[Jason Baron]

On Thu, 28 Oct 2004, Sergey Vlasov wrote:

> On Fri, 24 Sep 2004 13:49:33 -0400, Jason Baron wrote:
> 
> > Here is a first attempt at bringing Alan's 2.6 tty/ldisc fixes to 2.4.  
> > I've done some testing with it, but was hoping for broader
> > testing/feedback while all the issues get ironed out. The most notable
> > change is the addition of a wakeup at the end of tty_set_ldisc, for
> > threads waiting for the TTY_LDISC bit to be set.
> 
> > +
> > +	/* Defer ldisc switch */
> > +	/* tty_deferred_ldisc_switch(N_TTY);
> > +
> >  	read_lock(&tasklist_lock);
> >   	for_each_task(p) {
> >  		if ((tty->session > 0) && (p->session == tty->session) &&
> 
> Here the comment is unclosed; is this intentional?  Simply closing it
> at the same line gives a kernel which cannot complete the system boot
> process: it prints "init_dev but no ldisc", and then init hangs in
> uninterruptible sleep with this backtrace:

That comment should be closed. oops. I'm working on a fixed up patch now 
(for that and a few other things).

-Jason


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