[Skiboot] [PATCH] SLW: Add init for power9 power management
Akshay Adiga
akshay.adiga at linux.vnet.ibm.com
Thu Mar 23 19:52:31 AEDT 2017
On 03/23/2017 01:08 PM, Vaidyanathan Srinivasan wrote:
> * Stewart Smith <stewart at linux.vnet.ibm.com> [2017-03-23 15:30:09]:
>
> > Akshay Adiga <akshay.adiga at linux.vnet.ibm.com> writes:
> >> This patch adds new function to init core for power9 power management.
> >> SPECIAL_WKUP_* SCOM registers, if set, can hold the cores from going into
> >> idle states. Hence, clear PPM SPECIAL_WKUP_HYP_REG scom register for each
> >> core during init.
> >>
> >> Signed-off-by: Akshay Adiga <akshay.adiga at linux.vnet.ibm.com>
> >> ---
> >> hw/slw.c | 53 ++++++++++++++++++++++++++++++++++++++++++++++-------
> >> include/xscom.h | 6 ++++++
> >> 2 files changed, 52 insertions(+), 7 deletions(-)
> >
> > Hi! This breaks Mambo tests.
> >
> > FYI, you can run them automatically yourself by pushing your code up to
> > github after enabling travis-ci.org for your personal repository. The
> > .travis.yml file in skiboot runs a wide variety of OSs and runs P8 and
> > P9 mambo boot tests.
> >
> > The failure is this:
> >
> > 6352335: (6352335): [ 0.006348678,5] SLW: Init chip 0x0
> > WARNING: 6352731: (6352731): Invalid address 0x0000000100780868 in XSCOM range, SCOM=
> > 0x01007808d
> > WARNING: 6352731: (6352731): Attempt to store non-existent address 0x00001A0100780868
> > 6352731: (6352731): 0x000000003002E65C : stdcix r26,r0,r3
> > FATAL ERROR: 6352731: (6352731): Check Stop for 0:0: Machine Check with ME bit of MSR
> > off
>
> Hi Stewart,
>
> Thanks for catching this issue. We need to skip this xscoms access
> and setup on mambo. We do on P8, need to fix that code path.
>
> I will re-spin this patch with the fix so as to not break P9 mambo.
>
> --Vaidy
>
Posted v2 for the patch with the fix for p9 mambo.
--Akshay
More information about the Skiboot
mailing list