From: Ingo Molnar <mingo@elte.hu>
To: Mike Snitzer <snitzer@gmail.com>
Cc: Nicholas Miell <nmiell@comcast.net>,
Linus Torvalds <torvalds@linux-foundation.org>,
jim owens <jowens@hp.com>, "H. Peter Anvin" <hpa@zytor.com>,
Chris Mason <chris.mason@oracle.com>,
Peter Zijlstra <peterz@infradead.org>,
Steven Rostedt <rostedt@goodmis.org>,
paulmck@linux.vnet.ibm.com, Gregory Haskins <ghaskins@novell.com>,
Matthew Wilcox <matthew@wil.cx>, Andi Kleen <andi@firstfloor.org>,
Andrew Morton <akpm@linux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
linux-fsdevel <linux-fsdevel@vger.kernel.org>,
linux-btrfs <linux-btrfs@vger.kernel.org>,
Thomas Gleixner <tglx@linutronix.de>,
Nick Piggin <npiggin@suse.de>,
Peter Morreale <pmorreale@novell.com>,
Sven Dietrich <SDietrich@novell.com>,
sam@ravnborg.org, Dave Anderson <anderson@redhat.com>
Subject: Re: source line numbers with x86_64 modules? [Was: Re: [patch] measurements, numbers about CONFIG_OPTIMIZE_INLINING=y impact]
Date: Sat, 10 Jan 2009 16:34:46 +0100 [thread overview]
Message-ID: <20090110153446.GA13976@elte.hu> (raw)
In-Reply-To: <170fa0d20901100621m74680e0ewd1916c70f1636c9b@mail.gmail.com>
* Mike Snitzer <snitzer@gmail.com> wrote:
> Yes, especially from someone who lacks the ability to properly configure
> kdump. I'm fairly surprised others are giving you a free pass when you
> keep asserting how broken kdump is with such hollow criticism. I rely
> heavily on kdump and it works quite well (kvm integration was lacking
> but has improved).
hm, you say you rely heavily on kdump ... for what exactly, and how does
it help the upstream Linux kernel?
I see a single fix from you in the whole repository:
ffc41cf: nbd: prevent sock_xmit from attempting to use a NULL socket
... and that single fix is a NULL pointer dereference that ought to have
been quite debuggable from a plain oops alone.
In practice i rarely see bugfixes that were debugged via kdump. Normal
oops based fixes outnumber kdump based fixes by a ratio of 1:100 or worse
- and kdump is readily available these days - just nobody configures it.
For example, in the whole kernel repo there's just 45 commits that mention
'kdump' [excluding those commits that develop kdump itself]:
$ git log --pretty=format:"%h: %s" --no-merges -i --grep="kdump" |
grep -viE 'kdump|kexec|dump|mem' | wc -l
45
Contrast that to the 1954 commits that contain the string 'oops' or
'crash':
$ git log --pretty=format:"%h: %s" --no-merges -i -E --grep="oops|crash" |
wc -l
5900
That's a ratio of 1:131. (and probably optimistic in favor of kdump.)
Note, i dont have any negative feelings towards kdump - some people use it
and enterprise folks with their frozen, immutable kernels love it - it
just has not yet given me a reason to have particularly positive feelings
towards it in the upstream kernel space.
Ingo
next prev parent reply other threads:[~2009-01-10 15:34 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-10 14:21 source line numbers with x86_64 modules? [Was: Re: [patch] measurements, numbers about CONFIG_OPTIMIZE_INLINING=y impact] Mike Snitzer
2009-01-10 15:34 ` Ingo Molnar [this message]
2009-01-10 18:21 ` Mike Snitzer
2009-01-10 21:15 ` Theodore Tso
2009-01-10 22:21 ` Andi Kleen
2009-01-10 22:21 ` Andi Kleen
2009-01-10 22:21 ` Andi Kleen
2009-01-10 22:58 ` Linus Torvalds
2009-01-10 23:22 ` Ingo Molnar
2009-01-11 10:11 ` Andreas Dilger
2009-01-11 10:11 ` Andreas Dilger
2009-01-11 15:31 ` Mike Snitzer
2009-01-11 15:31 ` Mike Snitzer
2009-01-11 20:45 ` Theodore Tso
2009-01-11 10:11 ` Andreas Dilger
2009-01-11 1:28 ` Ingo Molnar
2009-01-11 4:52 ` Mike Snitzer
2009-01-13 3:19 ` Eric W. Biederman
2009-01-13 3:19 ` Eric W. Biederman
2009-01-13 3:19 ` Eric W. Biederman
2009-01-11 18:05 ` Andi Kleen
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=20090110153446.GA13976@elte.hu \
--to=mingo@elte.hu \
--cc=SDietrich@novell.com \
--cc=akpm@linux-foundation.org \
--cc=anderson@redhat.com \
--cc=andi@firstfloor.org \
--cc=chris.mason@oracle.com \
--cc=ghaskins@novell.com \
--cc=hpa@zytor.com \
--cc=jowens@hp.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=matthew@wil.cx \
--cc=nmiell@comcast.net \
--cc=npiggin@suse.de \
--cc=paulmck@linux.vnet.ibm.com \
--cc=peterz@infradead.org \
--cc=pmorreale@novell.com \
--cc=rostedt@goodmis.org \
--cc=sam@ravnborg.org \
--cc=snitzer@gmail.com \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.