From: "Aaron Tiensivu" <mojomofo@mojomofo.com>
To: <linux-kernel@vger.kernel.org>
Cc: <linux-ppp@vger.kernel.org>
Subject: [2.4.1-p10] Multilink PPP crash fix
Date: Tue, 23 Jan 2001 04:59:20 -0500 [thread overview]
Message-ID: <01e901c08523$5b6af660$0300a8c0@methusela> (raw)
[-- Attachment #1: Type: text/plain, Size: 460 bytes --]
I took out my can of RAID and went bug hunting.
The kernel was always crashing in this subroutine and the comment near the
list walk clued me in.
I've run this fixed kernel all night and can no longer make it OOPS from a
racy list walk.
It may be overkill to use ppp_lock instead of the finer grained locks but it
works great so far
and I don't believe this routine gets called a lot.
---
Life is like a box of chocolates: Mass produced and usually stale.
[-- Attachment #2: 2.4.1-p10-mppp-fix.patch --]
[-- Type: application/octet-stream, Size: 537 bytes --]
--- linux/drivers/net/ppp_generic.c.virgin Mon Jan 22 02:37:42 2001
+++ linux/drivers/net/ppp_generic.c Tue Jan 23 04:36:48 2001
@@ -1569,11 +1569,12 @@
struct sk_buff_head *list = &ppp->mrq;
u32 seq = skb->sequence;
- /* N.B. we don't need to lock the list lock because we have the
- ppp unit receive-side lock. */
+ ppp_lock(ppp);
for (p = list->next; p != (struct sk_buff *)list; p = p->next)
if (seq_before(seq, p->sequence))
break;
+ ppp_unlock(ppp);
+
__skb_insert(skb, p->prev, p, list);
}
reply other threads:[~2001-01-23 10:01 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='01e901c08523$5b6af660$0300a8c0@methusela' \
--to=mojomofo@mojomofo.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-ppp@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox