[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0703201145570.6730@woody.linux-foundation.org>
Date: Tue, 20 Mar 2007 11:49:39 -0700 (PDT)
From: Linus Torvalds <torvalds@...ux-foundation.org>
To: Andi Kleen <ak@...e.de>
cc: "Eric W. Biederman" <ebiederm@...ssion.com>,
David Miller <davem@...emloft.net>,
virtualization@...ts.linux-foundation.org, jbeulich@...ell.com,
jeremy@...p.org, xen-devel@...ts.xensource.com,
netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
chrisw@...s-sol.org, virtualization@...ts.osdl.org,
anthony@...emonkey.ws, akpm@...ux-foundation.org, mingo@...e.hu
Subject: Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops
callsites to make them patchable
On Tue, 20 Mar 2007, Andi Kleen wrote:
>
> So what is your proposed alternative to handle long backtraces?
> You didn't answer that question. Please do, I'm curious about your thoughts
> in this area.
the thing is, I'd rather see a long backtrace that is hard to decipher but
that *never* *ever* causes any additional problems, over a pretty one.
Because that's really the issue: do you want a "pretty" backtrace, or do
you want one that is rock solid but has some crud in it.
I'll take the rock solid one any day. Especially as even the pretty one
won't fix the most common problem, which is "I don't see the caller" (due
to inlining or tail-calls).
In contrast, the ugly backtrace will have some "garbage entries" from
previous frames that didn't get overwritten, but there have actually been
(admittedly rare) cases where those garbage entries have given hints about
what happened just before.
Linus
-
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