On Wed, Feb 10, 2016 at 07:09:11PM +0100, Thomas Huth wrote: > The H_SET_XDABR hypercall is similar to H_SET_DABR, but also sets > the extended DABR (DABRX) register. > > Signed-off-by: Thomas Huth > --- > hw/ppc/spapr_hcall.c | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c > index 6b9d512..f14f849 100644 > --- a/hw/ppc/spapr_hcall.c > +++ b/hw/ppc/spapr_hcall.c > @@ -366,6 +366,27 @@ static target_ulong h_set_dabr(PowerPCCPU *cpu, sPAPRMachineState *spapr, > return H_SUCCESS; > } > > +static target_ulong h_set_xdabr(PowerPCCPU *cpu, sPAPRMachineState *spapr, > + target_ulong opcode, target_ulong *args) > +{ > + CPUState *cs = CPU(cpu); > + target_ulong dabrx = args[1]; > + > + if (!has_spr(cpu, SPR_DABR) || !has_spr(cpu, SPR_DABRX)) { > + return H_HARDWARE; > + } > + > + if ((dabrx & ~0xfULL) != 0 || (dabrx & H_DABRX_HYPERVISOR) != 0 > + || (dabrx & (H_DABRX_KERNEL | H_DABRX_USER)) == 0) { > + return H_PARAMETER; > + } > + > + set_spr(cs, SPR_DABRX, dabrx, -1L); > + set_spr(cs, SPR_DABR, args[0], -1L); Same comments for set_spr(). > + > + return H_SUCCESS; > +} > + > #define FLAGS_REGISTER_VPA 0x0000200000000000ULL > #define FLAGS_REGISTER_DTL 0x0000400000000000ULL > #define FLAGS_REGISTER_SLBSHADOW 0x0000600000000000ULL > @@ -1024,6 +1045,7 @@ static void hypercall_register_types(void) > /* processor register resource access h-calls */ > spapr_register_hypercall(H_SET_SPRG0, h_set_sprg0); > spapr_register_hypercall(H_SET_DABR, h_set_dabr); > + spapr_register_hypercall(H_SET_XDABR, h_set_xdabr); > spapr_register_hypercall(H_SET_MODE, h_set_mode); > > /* "debugger" hcalls (also used by SLOF). Note: We do -not- differenciate -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson