On 21/01/21 10:18 pm, Ananth N Mavinakayanahalli wrote: > We currently just percolate the return value from analyze_instr() > to the caller of emulate_step(), especially if it is a -1. > > For one particular case (opcode = 4) for instructions that > aren't currently emulated, we are returning 'should not be > single-stepped' while we should have returned 0 which says > 'did not emulate, may have to single-step'. > > Signed-off-by: Ananth N Mavinakayanahalli <ana...@linux.ibm.com> > Tested-by: Naveen N. Rao <naveen.n....@linux.vnet.ibm.com> > --- > arch/powerpc/lib/sstep.c | 49 > +++++++++++++++++++++++++--------------------- > 1 file changed, 27 insertions(+), 22 deletions(-) >
Thanks for fixing this. Reviewed-by: Sandipan Das <sandi...@linux.ibm.com>