From: Keith Owens <kaos@ocs.com.au>
To: Linus Torvalds <torvalds@transmeta.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [patch] 2.4.13-pre3 arm/i386/mips/mips64/s390/s390x/sh die() deadlock
Date: Tue, 16 Oct 2001 14:27:55 +1000 [thread overview]
Message-ID: <19892.1003206475@kao2.melbourne.sgi.com> (raw)
In-Reply-To: Your message of "Tue, 16 Oct 2001 12:58:21 +1000." <18966.1003201101@kao2.melbourne.sgi.com>
On Tue, 16 Oct 2001 12:58:21 +1000,
Keith Owens <kaos@ocs.com.au> wrote:
>On Mon, 15 Oct 2001 19:36:02 -0700 (PDT),
>Linus Torvalds <torvalds@transmeta.com> wrote:
>>I much prefer a dead machine with a partially visible oops over a oops
>>where the original oops has scrolled away due to recursive faults.
>
>IMHO it is unrealistic to expect that all code inside die() will never
>fail. Any unexpected kernel corruption could cause the register or
>backtrace dump to fail. The patch gets the best of both worlds. It
>protects against recursive errors and against concurrent calls to
>die().
Previous message sent too soon.
The patch makes two attempts at dumping registers, one for the original
oops and one if die() fails, then it gives up. The second attempt is
useful for diagnosing why die() is failing, without that data it is
difficult to fix die() itself.
I was aiming to improve error handling in the rare case that die()
failed so we could get better diagnostics in the long term, by fixing
the problems that make die() fail. If you think that this would scroll
away useful data then we can compromise.
if (++die_lock_owner_depth < 2+(CONFIG_DIAGNOSE_RECURSIVE_DIE+0)) {
CONFIG_DIAGNOSE_RECURSIVE_DIE
If this variable is selected then the kernel will attempt to provide
extra diagnostics in the rare cases when the kernel die() routine
itself dies. This may cause useful information from the first
failure to be lost. Unless you want to diagnose the die() and
show_regs() code in the kernel, say N here.
Acceptable?
next prev parent reply other threads:[~2001-10-16 4:28 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-10-16 2:23 [patch] 2.4.13-pre3 arm/i386/mips/mips64/s390/s390x/sh die() deadlock Keith Owens
2001-10-16 2:36 ` Linus Torvalds
2001-10-16 2:58 ` Keith Owens
2001-10-16 4:27 ` Keith Owens [this message]
-- strict thread matches above, loose matches on Subject: below --
2001-10-16 3:35 Strumila, John
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=19892.1003206475@kao2.melbourne.sgi.com \
--to=kaos@ocs.com.au \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@transmeta.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