mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: ebiederm@xmission.com (Eric W. Biederman)
To: Jiri Kosina <jikos@jikos.cz>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel@vger.kernel.org, e1000-devel@lists.sourceforge.net,
	Len Brown <len.brown@intel.com>,
	Natalie Protasevich <nataliep@google.com>,
	Andi Kleen <ak@suse.de>,
	Michal Piotrowski <michal.k.k.piotrowski@gmail.com>,
	auke-jan.h.kok@intel.com
Subject: Re: 2.6.21-rc5-mm4
Date: Wed, 04 Apr 2007 11:22:21 -0600	[thread overview]
Message-ID: <m1r6r0awc2.fsf@ebiederm.dsl.xmission.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0704041748330.28249@twin.jikos.cz> (Jiri Kosina's message of "Wed, 4 Apr 2007 18:10:02 +0200 (CEST)")

Jiri Kosina <jikos@jikos.cz> writes:

> On Tue, 3 Apr 2007, Jiri Kosina wrote:
>
>> > we're also having problems reproducing it on that same combination 
>> > (2.6.21-rc4 + my tree), so it points to something in -mm. Since your 
>> > trace is completely different right now it looks like something else 
>> > is fuzzing it up. Since the e1000 changes are in rc5-mm3 as well, that 
>> > might help to narrow it down quickly.
>> I don't know (yet) whether rc5-mm3 was OK in this respect, I didn't boot 
>> it on this machine. I only know that both rc5 and rc5 + e1000 tree are 
>> OK, but rc5-mm4 panics on ifconfig/dhclient on e1000 card immediately on 
>> my system.
>> I will start bisection when I get back to the respective machine 
>> (tomorrow) and will let you know.
>
> And the bisection winner is
>
> 	i386-irq-kill-nr_irq_vectors-and-increase-nr_irqs.patch
>
> I don't immediately see how it could be causing it, so adding CCs which 
> are listed in the patch.

Weird.  I will have to look at that in a little more detail.

Do you know if this problem happens on x86_64?
What does your .config look like?
What does /proc/interrupts look like?
What kind of hardware you running this kernel on?
Can anyone else reproduce this?

The oops clearly shows something using -1 and calling that as an
address I don't know why, but I'm guessing I have triggered a memory
stomp somewhere.  I think this is the first time I have seen a small
negative number causing a NULL pointer dereference.

That patch looks innocuous enough that either:
- I just missed changing something I should have.
- Your configuration has an increase in NR_IRQS and that triggered
  something.
- The patch simply permuted things so a memory stomp now happens
  on the e1000 data structures instead of somewhere else.
- Something doesn't like large irq numbers.

This work is essentially a backport from x86_64 so if your hardware
is 64bit capable testing that should be a fairly easy test, and be
able to rule out large irq numbers as the culprit.

Until I get a good look at -mm I'm going to have a hard time guessing.
But a roving memory stomp is my best guess.


Eric

  reply	other threads:[~2007-04-04 17:23 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-03  5:47 2.6.21-rc5-mm4 Andrew Morton
2007-04-03 12:57 ` 2.6.21-rc5-mm4 Jiri Kosina
2007-04-03 15:11   ` 2.6.21-rc5-mm4 Kok, Auke
2007-04-03 16:30     ` 2.6.21-rc5-mm4 Jiri Kosina
2007-04-03 17:19       ` 2.6.21-rc5-mm4 Kok, Auke
2007-04-03 17:42         ` 2.6.21-rc5-mm4 Jiri Kosina
2007-04-04 16:10           ` 2.6.21-rc5-mm4 Jiri Kosina
2007-04-04 17:22             ` Eric W. Biederman [this message]
2007-04-06 13:23               ` 2.6.21-rc5-mm4 Jiri Kosina
2007-04-06 14:32                 ` 2.6.21-rc5-mm4 Eric W. Biederman
2007-04-03 16:11 ` 2.6.21-rc5-mm4 Michal Piotrowski
2007-04-03 22:20   ` 2.6.21-rc5-mm4 Michal Piotrowski
2007-04-03 22:31     ` 2.6.21-rc5-mm4 Con Kolivas
2007-04-03 23:31       ` 2.6.21-rc5-mm4 Michal Piotrowski
2007-04-04 13:22         ` [PATCH] sched: implement staircase deadline cpu scheduler improvements fix Con Kolivas
2007-04-03 19:48 ` 2.6.21-rc5-mm4 Randy Dunlap
2007-04-03 19:51   ` 2.6.21-rc5-mm4 Davide Libenzi
2007-04-03 20:31 ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-03 20:40   ` 2.6.21-rc5-mm4 Christoph Lameter
2007-04-03 22:53     ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-03 22:59       ` 2.6.21-rc5-mm4 Christoph Lameter
2007-04-03 23:40         ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-03 23:41           ` 2.6.21-rc5-mm4 Christoph Lameter
2007-04-04  1:16       ` 2.6.21-rc5-mm4 Christoph Lameter
2007-04-04 15:12         ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-04 15:27           ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-04 17:13           ` 2.6.21-rc5-mm4 (SLUB powerpc) Christoph Lameter
2007-04-04 17:33             ` Badari Pulavarty
2007-04-04 17:35               ` Christoph Lameter
2007-04-04 20:53                 ` Badari Pulavarty
2007-04-03 21:55 ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-03 23:19   ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-03 23:27     ` 2.6.21-rc5-mm4 Christoph Lameter
2007-04-03 23:38       ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-03 23:40         ` 2.6.21-rc5-mm4 Christoph Lameter
2007-04-03 23:51           ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-03 23:55             ` 2.6.21-rc5-mm4 Christoph Lameter
2007-04-04 14:53               ` 2.6.21-rc5-mm4 Badari Pulavarty
2007-04-04 17:03                 ` 2.6.21-rc5-mm4 (SLUB) Christoph Lameter
2007-04-04 18:02                   ` Badari Pulavarty
2007-04-04 18:04                     ` Christoph Lameter
2007-04-04 18:22                       ` Christoph Lameter
2007-04-04 22:53                         ` Badari Pulavarty
2007-04-04 22:59                           ` Christoph Lameter
2007-04-04 23:42                             ` Badari Pulavarty
2007-04-05  0:31                               ` Christoph Lameter
2007-04-05  2:44                                 ` Badari Pulavarty
2007-04-05  3:15                                   ` Christoph Lameter
2007-04-05  4:29                                   ` Christoph Lameter
2007-04-05 15:16                                     ` Badari Pulavarty
2007-04-05 18:19                                       ` Christoph Lameter
2007-04-13 15:13                       ` [PATCH] cache_k8_northbridges() overflows beyond allocation (Was: 2.6.21-rc5-mm4 (SLUB)) Badari Pulavarty
2007-04-13 15:45                         ` Andi Kleen
2007-04-13 16:42                           ` Chuck Ebbert
2007-04-13 16:52                             ` Andi Kleen
2007-04-13 16:57                           ` Andrew Morton
2007-04-13 18:07                         ` Christoph Lameter
2007-04-04 18:09                     ` 2.6.21-rc5-mm4 (SLUB) Christoph Lameter
2007-04-03 22:40 ` 2.6.21-rc5-mm4 J.A. Magallón
2007-04-03 22:51   ` 2.6.21-rc5-mm4 Andrew Morton
2007-04-03 22:58     ` 2.6.21-rc5-mm4 J.A. Magallón
2007-04-03 23:22       ` 2.6.21-rc5-mm4 Valdis.Kletnieks
2007-04-03 23:43         ` 2.6.21-rc5-mm4 J.A. Magallón
2007-04-04  1:35 ` 2.6.21-rc5-mm4 Valdis.Kletnieks
2007-04-04  2:20   ` 2.6.21-rc5-mm4 Randy Dunlap
2007-04-04  2:46     ` 2.6.21-rc5-mm4 Valdis.Kletnieks
2007-04-04  3:37       ` 2.6.21-rc5-mm4 Randy Dunlap
2007-04-04 17:55         ` 2.6.21-rc5-mm4 Valdis.Kletnieks
2007-04-05 11:14           ` 2.6.21-rc5-mm4 Sam Ravnborg
2007-04-04 15:25 ` [-mm patch] net/: possible cleanups Adrian Bunk
2007-04-04 15:25 ` [-mm patch] unexport pci_proc_attach_device Adrian Bunk
2007-04-04 21:56 ` 2.6.21-rc5-mm4 Antonino A. Daplas
2007-04-04 22:10   ` 2.6.21-rc5-mm4 Andrew Morton
2007-04-04 22:38     ` 2.6.21-rc5-mm4 Con Kolivas
2007-04-04 23:55       ` 2.6.21-rc5-mm4 Antonino A. Daplas
2007-04-05  0:56 ` 2.6.21-rc5-mm4 Antonino A. Daplas
2007-04-05 13:53 ` [-mm patch] pcm_native: lockdep warning when launching jack Frederik Deweerdt
2007-04-05 15:00   ` Takashi Iwai
2007-04-05 16:33 ` RAID1 "out of memory" error, was Re: 2.6.21-rc5-mm4 Reuben Farrelly
2007-04-05 20:21   ` Andrew Morton
2007-04-06  5:34     ` Dan Williams
2007-04-11  3:31   ` Neil Brown
2007-04-05 17:02 ` 2.6.21-rc5-mm4 Valdis.Kletnieks
2007-04-05 20:31   ` 2.6.21-rc5-mm4 Andrew Morton
2007-04-06 15:26     ` 2.6.21-rc5-mm4 Valdis.Kletnieks
2007-04-06 19:33       ` 2.6.21-rc5-mm4 Andrew Morton

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=m1r6r0awc2.fsf@ebiederm.dsl.xmission.com \
    --to=ebiederm@xmission.com \
    --cc=ak@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=auke-jan.h.kok@intel.com \
    --cc=e1000-devel@lists.sourceforge.net \
    --cc=jikos@jikos.cz \
    --cc=len.brown@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=michal.k.k.piotrowski@gmail.com \
    --cc=nataliep@google.com \
    /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

all inboxes | Powered by JetHome®