public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@kernel.org>
To: Seiji Aguchi <seiji.aguchi@hds.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"x86@kernel.org" <x86@kernel.org>,
	"Thomas Gleixner (tglx@linutronix.de)" <tglx@linutronix.de>,
	"'mingo@elte.hu' (mingo@elte.hu)" <mingo@elte.hu>,
	"H. Peter Anvin (hpa@zytor.com)" <hpa@zytor.com>,
	"dzickus@redhat.com" <dzickus@redhat.com>,
	"dle-develop@lists.sourceforge.net" 
	<dle-develop@lists.sourceforge.net>,
	Satoru Moriya <satoru.moriya@hds.com>
Subject: Re: [PATCH]Skip unnecessary WARN_ON in panic case
Date: Mon, 11 Mar 2013 09:28:51 +0100	[thread overview]
Message-ID: <20130311082851.GC12742@gmail.com> (raw)
In-Reply-To: <A5ED84D3BB3A384992CBB9C77DEDA4D41AF5CB8B@USINDEM103.corp.hds.com>


* Seiji Aguchi <seiji.aguchi@hds.com> wrote:

> [Problem]
> 
> When kernel panics, unnecessary WARN_ON() may be printed after panic messages in a following scenario.
>  - A panicked cpu stops other cpus via smp_send_stop().
>  - Other cpus turn to be offline in stop_this_cpu().
>  - The panicked cpu enables interrupt.
>  - native_smp_send_reschedule() is called via a timer interrupt on the panicked cpu.
>  - The panicked cpu tries to send a reschedule IPI to other cpus
>  - The panicked cpu hits WARN_ON() because other cpus have already been offlined.
> 
> If an user has just a VGA console, panic messages may be missed because 
> they are floated outside a screen due to messages of the WARN_ON(). In 
> this case, it could be difficult to investigate the reason why a kernel 
> panicked.

So it appears the problem occurs because the other CPUs are not offlined 
correctly, and still have pending work which may generate wakeup IPIs to 
them?

The fix would be to either offline them properly - or, if we don't want to 
do that to keep panicking simple, why do we mark them offline? They aren't 
really offline in a proper way.

Thanks,

	Ingo

  reply	other threads:[~2013-03-11  8:28 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-06 19:06 [PATCH]Skip unnecessary WARN_ON in panic case Seiji Aguchi
2013-03-11  8:28 ` Ingo Molnar [this message]
2013-03-15 15:39   ` Seiji Aguchi

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=20130311082851.GC12742@gmail.com \
    --to=mingo@kernel.org \
    --cc=dle-develop@lists.sourceforge.net \
    --cc=dzickus@redhat.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=satoru.moriya@hds.com \
    --cc=seiji.aguchi@hds.com \
    --cc=tglx@linutronix.de \
    --cc=x86@kernel.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