LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Matt Mackall <mpm@selenic.com>
To: Andi Kleen <andi@firstfloor.org>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
Sam Ravnborg <sam@ravnborg.org>,
Linux-tiny@selenic.com, Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org, "H. Anvin" <hpa@zytor.com>,
Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH] Configure out doublefault exception handler (Linux Tiny)
Date: Fri, 15 Feb 2008 11:02:16 -0600 [thread overview]
Message-ID: <1203094937.12383.212.camel@cinder.waste.org> (raw)
In-Reply-To: <p73tzkazaxh.fsf@bingen.suse.de>
On Fri, 2008-02-15 at 13:00 +0100, Andi Kleen wrote:
> Matt Mackall <mpm@selenic.com> writes:
> >
> > I bet there's some doublefault-handling code hiding somewhere. It's not
> > the sort of thing it'd make sense to take out of the architecture.
>
> The big question is if it makes sense taking out of a kernel at all.
> I still think the answer is no.
>
> Or have you considered replacing die() and show_trace() etc. with a single
> panic("the tiny gods say this won't happen") yet? That would be roughly
> equivalent.
It's not a matter of "won't happen" so much as "not a damn thing we can
do when it does". There's very little point in having this sort of code
in a mass-market camera, phone, DVR, TV, etc. (of which there are
already millions running Linux). These devices have no console and
basically zero serviceability beyond firmware upgrades. If taking these
vestigial debugging features out means we can cram in more features that
consumers can actually see and will pay for, that's precisely what's
going to happen.
--
Mathematics is the supreme nostalgia of our time.
next prev parent reply other threads:[~2008-02-15 17:03 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-12 10:08 Thomas Petazzoni
2008-02-12 13:04 ` Sam Ravnborg
2008-02-12 14:00 ` Thomas Petazzoni
2008-02-12 16:02 ` Matt Mackall
2008-02-15 12:00 ` Andi Kleen
2008-02-15 17:02 ` Matt Mackall [this message]
2008-02-15 18:04 ` Andi Kleen
2008-02-15 17:52 ` Matt Mackall
2008-02-17 17:59 ` Ingo Molnar
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=1203094937.12383.212.camel@cinder.waste.org \
--to=mpm@selenic.com \
--cc=Linux-tiny@selenic.com \
--cc=akpm@linux-foundation.org \
--cc=andi@firstfloor.org \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=sam@ravnborg.org \
--cc=tglx@linutronix.de \
--cc=thomas.petazzoni@free-electrons.com \
--subject='Re: [PATCH] Configure out doublefault exception handler (Linux Tiny)' \
/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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).