[Skiboot] [PATCH] dts: spl_wakeup: Remove all workarounds in the spl wakeup logic
Stewart Smith
stewart at linux.vnet.ibm.com
Fri Mar 16 07:59:38 AEDT 2018
Vaidyanathan Srinivasan <svaidy at linux.vnet.ibm.com> writes:
> * Stewart Smith <stewart at linux.vnet.ibm.com> [2018-03-15 20:13:45]:
>
>> Shilpasri G Bhat <shilpa.bhat at linux.vnet.ibm.com> writes:
>> > We coded few workarounds in special wakeup logic to handle the
>> > buggy firmware. Now that is fixed remove them as they break the
>> > special wakeup protocol. As per the spec we should not de-assert
>> > beofre assert is complete. So follow this protocol.
>> >
>> > Signed-off-by: Shilpasri G Bhat <shilpa.bhat at linux.vnet.ibm.com>
>> > ---
>> > core/direct-controls.c | 59 +++++++++++++++++++++++++-------------------------
>> > hw/dts.c | 30 +------------------------
>> > 2 files changed, 30 insertions(+), 59 deletions(-)
>>
>> great! Merged to master as of b5c9d09d067743be6ab12eef49f80562f0d41f02
>>
>> Is it worth trying re-enabling stop4 and stop5 now? Was this one of the
>> issues causing failures there?
>
> This patch helped capture CME/microcode issues at the first fail, then
> root-caused and fixed. The HCODE has to roll out into a release before
> we can re-enable stop4/5. Special wakeup with stop4/5 has been stress
> tested with the new HCODE that is yet to release.
Ok.
Corey - do you know which hcode this is and how far away it is from
hitting op-build?
> Further we are removing DTS reads in favour of OCC sensors in the
> other patch to avoid poking cores.
Yeah, I think that ends up making sense, I just have a desire to see the
op-test sensors test pass with poking the OCC sensors and stop 4/5 as
that's been really good at triggering these bugs.
--
Stewart Smith
OPAL Architect, IBM.
More information about the Skiboot
mailing list