[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20060826155159.GJ4765@stusta.de>
Date: Sat, 26 Aug 2006 17:51:59 +0200
From: Adrian Bunk <bunk@...sta.de>
To: Chuck Ebbert <76306.1226@...puserve.com>,
Paul Fulghum <paulkf@...rogate.com>
Cc: linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: Linux 2.6.16.28
On Sat, Aug 26, 2006 at 10:20:43AM -0400, Chuck Ebbert wrote:
> In-Reply-To: <20060826003639.GA4765@...sta.de>
>
> On Sat, 26 Aug 2006 02:36:39 +0200, Adrian Bunk wrote:
>
> > Location:
> > ftp://ftp.kernel.org/pub/linux/kernel/v2.6/
>
> Could you post the incremental patch as a reply like Greg does?
After Michaels reminder I've put it at ftp.kernel.org .
> And are you going to apply Paul Fulghum's tty patch? Without it
> I get solid lockup on pppd disconnect about once a month.
I've still many patches (including parts of four 2.6.17 releases) to
review and I wasn't yet there.
Paul, can you ACK that this patch is OK for 2.6.16?
> Chuck
cu
Adrian
Subject: tty serialize flush_to_ldisc
From: Paul Fulghum <paulkf@...rogate.com>
Serialize processing of tty buffers in flush_to_ldisc
to fix (very rare) corruption of tty buffer free list
on SMP systems.
Signed-off-by: Paul Fulghum <paulkf@...rogate.com>
Acked-by: Alan Cox <alan@...hat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@...e.de>
---
drivers/char/tty_io.c | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
--- linux-2.6.17.7.orig/drivers/char/tty_io.c
+++ linux-2.6.17.7/drivers/char/tty_io.c
@@ -2776,7 +2776,7 @@ static void flush_to_ldisc(void *private
struct tty_struct *tty = (struct tty_struct *) private_;
unsigned long flags;
struct tty_ldisc *disc;
- struct tty_buffer *tbuf;
+ struct tty_buffer *tbuf, *head;
int count;
char *char_buf;
unsigned char *flag_buf;
@@ -2793,7 +2793,9 @@ static void flush_to_ldisc(void *private
goto out;
}
spin_lock_irqsave(&tty->buf.lock, flags);
- while((tbuf = tty->buf.head) != NULL) {
+ head = tty->buf.head;
+ tty->buf.head = NULL;
+ while((tbuf = head) != NULL) {
while ((count = tbuf->commit - tbuf->read) != 0) {
char_buf = tbuf->char_buf_ptr + tbuf->read;
flag_buf = tbuf->flag_buf_ptr + tbuf->read;
@@ -2802,10 +2804,12 @@ static void flush_to_ldisc(void *private
disc->receive_buf(tty, char_buf, flag_buf, count);
spin_lock_irqsave(&tty->buf.lock, flags);
}
- if (tbuf->active)
+ if (tbuf->active) {
+ tty->buf.head = head;
break;
- tty->buf.head = tbuf->next;
- if (tty->buf.head == NULL)
+ }
+ head = tbuf->next;
+ if (head == NULL)
tty->buf.tail = NULL;
tty_buffer_free(tty, tbuf);
}
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists