Messages in this thread | | | Date | Fri, 28 Jan 2000 12:26:50 -0500 | From | "Theodore Y. Ts'o" <> | Subject | Re: 2.2.15pre4 VM fix |
| |
Date: Fri, 28 Jan 2000 19:48:27 +0300 From: Ivan Kokshaysky <ink@jurassic.park.msu.ru>
On Fri, Jan 28, 2000 at 02:40:30PM +0000, Alan Cox wrote: > > n_tty_open() has been caught with your patch. > > Thanks! > > Do you know which drivers (serial,tty) you were using it. n_tty_open itself > seems ok, but the caller may be guilty
It happened when ppp connection was terminated (remote end hangup). Serial driver is Comtrol Rocketport. The problem is repeatable (3 times last 20 hours), so I can investigate further to see who is the caller.
I think it's a flase positive. It's happeninig because tty_do_hangup() is calling ldisc.open --- which means n_tty_open() inside an interrupt context. n_tty_open() makes a check to see whether it is being called inside an interrupt, and uses GFP_ATOMIC to avoid blocking inside the interrupt.
static int n_tty_open(struct tty_struct *tty) { .... get_zeroed_page(in_interrupt() ? GFP_ATOMIC : GFP_KERNEL); ... }
This should be OK, I think.
- Ted
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/
| |