Excerpts from Christophe Leroy's message of February 5, 2021 6:56 pm: > For unimplemented instructions or unimplemented SPRs, the 8xx triggers > a "Software Emulation Exception" (0x1000). That interrupt doesn't set > reason bits in SRR1 as the "Program Check Exception" does. > > Go through emulation_assist_interrupt() to set REASON_ILLEGAL. > > Fixes: fbbcc3bb139e ("powerpc/8xx: Remove SoftwareEmulation()") > Signed-off-by: Christophe Leroy <christophe.le...@csgroup.eu> > --- > I'm wondering whether it wouldn't be better to set REASON_ILLEGAL > in the exception prolog and still call program_check_exception. > And do the same in book3s/64 to avoid the nightmare of an > INTERRUPT_HANDLER calling another INTERRUPT_HANDLER.
Hmm, I missed this. We just change program_check_exception to a common function which is called by both. Thanks, Nick