[Pdbg] [PATCH v2 7/7] libpdbg/p8chip.c: Emulate sreset using ramming for active threads

Stewart Smith stewart at linux.ibm.com
Tue Mar 26 16:38:20 AEDT 2019


Nicholas Piggin <npiggin at gmail.com> writes:
> Based on patch from Alistair, some fixes and changes:
> - account HILE bit, set/clear MSR_LE
> - clear MSR_PR
> - don't use raw ramming (clearer this way, not perf critical)
>
> At the moment, must manually stop all threads in the core, and manually
> restart them. Can change behaviour depending on what exactly we want
> (e.g., sreset all threads may be good for debugging).
> ---
>  libpdbg/p8chip.c | 131 +++++++++++++++++++++++++++++++++++++++++++++--
>  1 file changed, 128 insertions(+), 3 deletions(-)

I've decently hammered this with a WIP op-test patch that does a lot of
enter/exit xmon by using pdbg to inject an NMI.

Well, decently hammered enough to find some good skiboot bugs. I'm not
quite convinced as to where one bug is (the symptom is we start dumping
a whole bunch of memory out to the console), but I'm sure this is better
than what we had before.

Tested-by: Stewart Smith <stewart at linux.ibm.com>

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Pdbg mailing list