mbox series

[SRU,Zesty,0/1] powerpc/powernv: Flush console before platform error reboot

Message ID cover.1512248299.git.joseph.salisbury@canonical.com
Headers show
Series powerpc/powernv: Flush console before platform error reboot | expand

Message

Joseph Salisbury Dec. 2, 2017, 9:15 p.m. UTC
BugLink: http://bugs.launchpad.net/bugs/1735159

== SRU Justification ==
IBM unrecovered MCE and HMI errors are sent through a special restart OPAL
call to log the platform error. The downside is that they don't go
through normal Linux crash paths, so they don't give much information
to the Linux console.

IBM is requesting to have the commit b746e3e01e70d23ef53dcde1203ab78a1b7ac514
in the Zesty kernel.

Commit b746e3e01e70d2 is in mainline as of 4.14-rc1.  

Breno Leitao from IBM did a backport of this commit. 

== Fix ==
commit b746e3e01e70d23ef53dcde1203ab78a1b7ac514
Author: Nicholas Piggin <npiggin@gmail.com>
Date:   Wed Jul 19 16:59:10 2017 +1000

    powerpc/powernv: Flush console before platform error reboot

== Regression Potential ==
This patch is specific to powerpc, and it has been backported, reviewed and 
tested by IBM.

== Test Case ==
A test kernel was built with this patch and tested by the original bug reporter.
The bug reporter states the test kernel resolved the bug.

Nicholas Piggin (1):
  powerpc/powernv: Flush console before platform error reboot

 arch/powerpc/include/asm/opal.h           |  2 +-
 arch/powerpc/platforms/powernv/opal-hmi.c | 22 ++------
 arch/powerpc/platforms/powernv/opal.c     | 88 ++++++++++++++++++-------------
 arch/powerpc/platforms/powernv/powernv.h  |  2 +
 4 files changed, 56 insertions(+), 58 deletions(-)

Comments

Joseph Salisbury Dec. 2, 2017, 9:21 p.m. UTC | #1
On 12/02/2017 04:15 PM, Joseph Salisbury wrote:
> BugLink: http://bugs.launchpad.net/bugs/1735159
>
> == SRU Justification ==
> IBM unrecovered MCE and HMI errors are sent through a special restart OPAL
> call to log the platform error. The downside is that they don't go
> through normal Linux crash paths, so they don't give much information
> to the Linux console.
>
> IBM is requesting to have the commit b746e3e01e70d23ef53dcde1203ab78a1b7ac514
> in the Zesty kernel.
>
> Commit b746e3e01e70d2 is in mainline as of 4.14-rc1.  
>
> Breno Leitao from IBM did a backport of this commit. 
>
> == Fix ==
> commit b746e3e01e70d23ef53dcde1203ab78a1b7ac514
> Author: Nicholas Piggin <npiggin@gmail.com>
> Date:   Wed Jul 19 16:59:10 2017 +1000
>
>     powerpc/powernv: Flush console before platform error reboot
>
> == Regression Potential ==
> This patch is specific to powerpc, and it has been backported, reviewed and 
> tested by IBM.
>
> == Test Case ==
> A test kernel was built with this patch and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
>
> Nicholas Piggin (1):
>   powerpc/powernv: Flush console before platform error reboot
>
>  arch/powerpc/include/asm/opal.h           |  2 +-
>  arch/powerpc/platforms/powernv/opal-hmi.c | 22 ++------
>  arch/powerpc/platforms/powernv/opal.c     | 88 ++++++++++++++++++-------------
>  arch/powerpc/platforms/powernv/powernv.h  |  2 +
>  4 files changed, 56 insertions(+), 58 deletions(-)
>
I just noticed this is the same request sent by Breno on November 29th:
https://lists.ubuntu.com/archives/kernel-team/2017-November/088500.html

I should have just acked that one instead of this request.  I'll ack
Breno's SRU request.

Joe