From: Ingo Molnar <mingo@elte.hu>
To: Don Zickus <dzickus@redhat.com>
Cc: akpm@linux-foundation.org, fweisbec@gmail.com,
LKML <linux-kernel@vger.kernel.org>,
Linus Torvalds <torvalds@linux-foundation.org>,
Peter Zijlstra <a.p.zijlstra@chello.nl>
Subject: Re: [PATCH 1/2] panic: ratelimit panic messages
Date: Wed, 5 Jan 2011 14:21:08 +0100 [thread overview]
Message-ID: <20110105132108.GE10129@elte.hu> (raw)
In-Reply-To: <1294198711-15492-2-git-send-email-dzickus@redhat.com>
* Don Zickus <dzickus@redhat.com> wrote:
> Sometimes when things go bad, so much spew is coming on the console it is hard
> to figure out what happened. This patch allows you to ratelimit the panic
> messages with the intent that the first panic message will provide the info
> we need to figure out what happened.
>
> Adds new kernel param 'panic_ratelimit=on/<integer in seconds>'
>
> Signed-off-by: Don Zickus <dzickus@redhat.com>
> ---
> Documentation/kernel-parameters.txt | 6 ++++++
> kernel/panic.c | 30 ++++++++++++++++++++++++++++++
> 2 files changed, 36 insertions(+), 0 deletions(-)
Well, but it's not just panics that can occur repeatedly (in fact panics tend to
trigger once), but recursive/repeated oopses.
So if we try to do some sort of 'show first crash, rate-limit afterwards' thing (i'm
still not 100% sure about the wiseness of it) then we should cover all common
occurances of system panics/crashes.
btw., while printk-delay exists and can serve a similar purpose, it's pretty
cumbersome for panic ratelimit: it slows down bootups extremely, and it also slows
down regular, harmless printks.
So having some sort of rate-limit for emitting crash screens may make sense, while
also preserving the oops counter or at least some other notification that we 'lost'
messages.
Ingo
next prev parent reply other threads:[~2011-01-05 13:21 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-05 3:38 [PATCH 0/2] touch_nmi_watchdog semantic changes Don Zickus
2011-01-05 3:38 ` [PATCH 1/2] panic: ratelimit panic messages Don Zickus
2011-01-05 13:21 ` Ingo Molnar [this message]
2011-01-05 14:00 ` Don Zickus
2011-01-05 22:51 ` Andrew Morton
2011-01-06 2:05 ` Don Zickus
2011-01-06 2:16 ` Andrew Morton
2011-01-15 8:29 ` Dave Young
2011-01-15 11:03 ` Dave Young
2011-01-05 3:38 ` [PATCH 2/2] watchdog: touch_nmi_watchdog should only touch local cpu not every one Don Zickus
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=20110105132108.GE10129@elte.hu \
--to=mingo@elte.hu \
--cc=a.p.zijlstra@chello.nl \
--cc=akpm@linux-foundation.org \
--cc=dzickus@redhat.com \
--cc=fweisbec@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox