From: Mahesh Jagannath Salgaonkar <mahesh@linux.vnet.ibm.com>
To: Nicholas Piggin <npiggin@gmail.com>
Cc: linuxppc-dev <linuxppc-dev@ozlabs.org>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
Paul Mackerras <paulus@samba.org>
Subject: Re: [RFC PATCH 5/7] powerpc/book3s: Don't turn on the MSR[ME] bit until opal processes the reason.
Date: Tue, 21 Feb 2017 09:47:53 +0530 [thread overview]
Message-ID: <c3a85fbf-29e8-e20d-dc68-6c1f1413d063@linux.vnet.ibm.com> (raw)
In-Reply-To: <20170221124728.675af9f9@roar.ozlabs.ibm.com>
On 02/21/2017 08:17 AM, Nicholas Piggin wrote:
> On Tue, 21 Feb 2017 07:22:56 +0530
> Mahesh J Salgaonkar <mahesh@linux.vnet.ibm.com> wrote:
>
>> From: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com>
>>
>> Delay it until we are done with machine_check_early() call. Turn on MSR[ME]
>> once opal is done with processing MCE.
>
> Why? This seems like quite a regression -- the MCE handler today
> has about 60 instructions and 30 l/st with ME clear.
I understand that this is bit long window. But we are in MCE handling
code and if we hit MCE while doing that we may anyway end up with
recursive MCE interrupts without really be able to recover from it.
Instead lets risk checkstop which would get us rebooted with hostboot
throwing proper error call out.
-Mahesh.
next prev parent reply other threads:[~2017-02-21 4:18 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-21 1:51 [RFC PATCH 0/7] Machine check handling for Power9 with bacward compatibility Mahesh J Salgaonkar
2017-02-21 1:51 ` [RFC PATCH 1/7] powerpc/book3s: Move machine check event structure to opal-api.h Mahesh J Salgaonkar
2017-02-21 2:35 ` Nicholas Piggin
2017-02-21 6:51 ` Mahesh Jagannath Salgaonkar
2017-02-21 1:52 ` [RFC PATCH 2/7] powerpc/book3s: mce: Call opal mce handler to extract MCE error reason Mahesh J Salgaonkar
2017-02-21 1:52 ` [RFC PATCH 3/7] powerpc/book3s: mce: Process the MCE event and recover if possible Mahesh J Salgaonkar
2017-02-21 1:52 ` [RFC PATCH 4/7] powerpc/book3s: Print additional MCE errors introduced in power9 Mahesh J Salgaonkar
2017-02-21 1:52 ` [RFC PATCH 5/7] powerpc/book3s: Don't turn on the MSR[ME] bit until opal processes the reason Mahesh J Salgaonkar
2017-02-21 2:47 ` Nicholas Piggin
2017-02-21 4:17 ` Mahesh Jagannath Salgaonkar [this message]
2017-02-21 4:43 ` Nicholas Piggin
2017-02-21 1:53 ` [RFC PATCH 6/7] powerpc/book3s: Display more info for MCE error console log Mahesh J Salgaonkar
2017-02-21 1:53 ` [RFC PATCH 7/7] powerpc/book3s: Display task info for MCE error in user mode Mahesh J Salgaonkar
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=c3a85fbf-29e8-e20d-dc68-6c1f1413d063@linux.vnet.ibm.com \
--to=mahesh@linux.vnet.ibm.com \
--cc=benh@kernel.crashing.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=npiggin@gmail.com \
--cc=paulus@samba.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;
as well as URLs for NNTP newsgroup(s).