Hi Rob.
> diff --git a/arch/sparc/kernel/traps_64.c b/arch/sparc/kernel/traps_64.c
> index ad31af1..c49949c 100644
> --- a/arch/sparc/kernel/traps_64.c
> +++ b/arch/sparc/kernel/traps_64.c
> @@ -265,6 +265,34 @@ void sun4v_insn_access_exception_tl1(struct pt_regs *regs, unsigned long addr, u
> sun4v_insn_access_exception(regs, addr, type_ctx);
> }
>
> +bool is_no_fault_exception(struct pt_regs *regs)
> +{
> + unsigned char asi;
> + u32 insn;
> +
> + if (get_user(insn, (u32 __user *)regs->tpc) == -EFAULT)
> + return false;
> +
> + if ((insn & 0xc0800000) == 0xc0800000) {
> + if (insn & 0x2000)
> + asi = (regs->tstate >> 24);
> + else
> + asi = (insn >> 5);
The constants used in the above code is pure magic
for peopel without any initimate knowledge of sparc.
Could at a abre minimum add a few comments that explains
what is going on.
> + if ((asi & 0xf2) == ASI_PNF) {
> + if (insn & 0x1000000) {
> + handle_ldf_stq(insn, regs);
> + return true;
> + } else if (insn & 0x200000) {
> + /* store instruction, do not handle */
> + return false;
> + }
Likewise
> + handle_ld_nf(insn, regs);
> + return true;
> + }
> + }
> + return false;
> +}
Sam
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
