On 6 May 2016 at 19:11, Alistair Francis <[email protected]> wrote: > Add the Hypervisor System Trap Register for EL2. > > This register is used early in the Linux boot and without it the kernel > aborts with a "Synchronous Abort" error. > > Signed-off-by: Alistair Francis <[email protected]> > --- > > target-arm/helper.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/target-arm/helper.c b/target-arm/helper.c > index 09638b2..65e8ff1 100644 > --- a/target-arm/helper.c > +++ b/target-arm/helper.c > @@ -3470,6 +3470,9 @@ static const ARMCPRegInfo el3_no_el2_cp_reginfo[] = { > .opc0 = 3, .opc1 = 4, .crn = 6, .crm = 0, .opc2 = 4, > .access = PL2_RW, .accessfn = access_el3_aa32ns_aa64any, > .type = ARM_CP_CONST, .resetvalue = 0 }, > + { .name = "HSTR_EL2", .state = ARM_CP_STATE_AA64, > + .opc0 = 3, .opc1 = 4, .crn = 1, .crm = 1, .opc2 = 3, > + .access = PL2_RW, .type = ARM_CP_CONST, .resetvalue = 0 }, > REGINFO_SENTINEL > }; > > @@ -3703,6 +3706,10 @@ static const ARMCPRegInfo el2_cp_reginfo[] = { > .opc0 = 3, .opc1 = 4, .crn = 6, .crm = 0, .opc2 = 4, > .access = PL2_RW, > .fieldoffset = offsetof(CPUARMState, cp15.hpfar_el2) }, > + { .name = "HSTR_EL2", .state = ARM_CP_STATE_AA64, > + .opc0 = 3, .opc1 = 4, .crn = 1, .crm = 1, .opc2 = 3, > + .access = PL2_RW, > + .readfn = arm_cp_read_zero, .writefn = arm_cp_write_ignore, }, > REGINFO_SENTINEL > };
Can we have the 32-bit version of the register too, please? (I think it should be possible to do this with a STATE_BOTH stanza). Can we have the el2_cp_reginfo implemented to be a reads-as-written to a hstr_el2 field in the env->cp15 struct? That way we won't have to come back to the regdef struct as and when we actually implement the trap bits to do something... thanks -- PMM
