Re: 2.5.74 / oops with ppp_synctty and local_bh_enable

Ravikiran G Thirumalai (kiran@in.ibm.com)
Tue, 8 Jul 2003 14:57:12 +0530


On Mon, Jul 07, 2003 at 10:47:48AM -0500, Paul Fulghum wrote:
> ...
> Perhaps changing ppp_synctty.c to use spin_lock_irqsave
> and spin_lock_irqrestore is the best fix.

My 2 cents would be to keep it as it is. BK says WARN_ON was used
in place of BUG_ON in local_bh_enable just for the the ppp ldisc.

We all know tty locking is broken and needs rewrite, but
if we kill such symptoms at driver level, it'll probably deprive
some data points to whoever rewrites the tty layer.

Thanks,
Kiran
-
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/