Re: [PATCH v3 07/14] powerpc/traps: Check for prefixed instructions in facility_unavailable_exception()
Jordan Niethe's on February 27, 2020 9:52 am: > On Wed, Feb 26, 2020 at 5:53 PM Nicholas Piggin wrote: >> >> Jordan Niethe's on February 26, 2020 2:07 pm: >> > If prefixed instructions are made unavailable by the [H]FSCR, attempting >> > to use them will cause a facility unavailable exception. Add "PREFIX" to >> > the facility_strings[]. >> > >> > Currently there are no prefixed instructions that are actually emulated >> > by emulate_instruction() within facility_unavailable_exception(). >> > However, when caused by a prefixed instructions the SRR1 PREFIXED bit is >> > set. Prepare for dealing with emulated prefixed instructions by checking >> > for this bit. >> > >> > Signed-off-by: Jordan Niethe >> >> Oh you've got it here, I would just squash this together with the first >> patch. > Sure, I'll put them together. When you mentioned a couple more things > to do in traps.c, was it just this? Or is there still more to be done > adding an FSCR? Hmm... maybe it's just that one. Thanks, Nick
Re: [PATCH v3 07/14] powerpc/traps: Check for prefixed instructions in facility_unavailable_exception()
On Wed, Feb 26, 2020 at 5:53 PM Nicholas Piggin wrote: > > Jordan Niethe's on February 26, 2020 2:07 pm: > > If prefixed instructions are made unavailable by the [H]FSCR, attempting > > to use them will cause a facility unavailable exception. Add "PREFIX" to > > the facility_strings[]. > > > > Currently there are no prefixed instructions that are actually emulated > > by emulate_instruction() within facility_unavailable_exception(). > > However, when caused by a prefixed instructions the SRR1 PREFIXED bit is > > set. Prepare for dealing with emulated prefixed instructions by checking > > for this bit. > > > > Signed-off-by: Jordan Niethe > > Oh you've got it here, I would just squash this together with the first > patch. Sure, I'll put them together. When you mentioned a couple more things to do in traps.c, was it just this? Or is there still more to be done adding an FSCR? > > Thanks, > Nick
Re: [PATCH v3 07/14] powerpc/traps: Check for prefixed instructions in facility_unavailable_exception()
Jordan Niethe's on February 26, 2020 2:07 pm: > If prefixed instructions are made unavailable by the [H]FSCR, attempting > to use them will cause a facility unavailable exception. Add "PREFIX" to > the facility_strings[]. > > Currently there are no prefixed instructions that are actually emulated > by emulate_instruction() within facility_unavailable_exception(). > However, when caused by a prefixed instructions the SRR1 PREFIXED bit is > set. Prepare for dealing with emulated prefixed instructions by checking > for this bit. > > Signed-off-by: Jordan Niethe Oh you've got it here, I would just squash this together with the first patch. Thanks, Nick
[PATCH v3 07/14] powerpc/traps: Check for prefixed instructions in facility_unavailable_exception()
If prefixed instructions are made unavailable by the [H]FSCR, attempting to use them will cause a facility unavailable exception. Add "PREFIX" to the facility_strings[]. Currently there are no prefixed instructions that are actually emulated by emulate_instruction() within facility_unavailable_exception(). However, when caused by a prefixed instructions the SRR1 PREFIXED bit is set. Prepare for dealing with emulated prefixed instructions by checking for this bit. Signed-off-by: Jordan Niethe --- arch/powerpc/kernel/traps.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc/kernel/traps.c index d80b82fc1ae3..cd8b3043c268 100644 --- a/arch/powerpc/kernel/traps.c +++ b/arch/powerpc/kernel/traps.c @@ -1739,6 +1739,7 @@ void facility_unavailable_exception(struct pt_regs *regs) [FSCR_TAR_LG] = "TAR", [FSCR_MSGP_LG] = "MSGP", [FSCR_SCV_LG] = "SCV", + [FSCR_PREFIX_LG] = "PREFIX", }; char *facility = "unknown"; u64 value; -- 2.17.1