On 6/3/2014 4:27 AM, Alexey Kardashevskiy wrote: > @@ -7303,8 +7304,21 @@ static void init_proc_970 (CPUPPCState *env) > SPR_NOACCESS, SPR_NOACCESS, > &spr_read_hior, &spr_write_hior, > 0x00000000); > + > + spr_register(env, SPR_CTRL, "SPR_CTRL", > + SPR_NOACCESS, SPR_NOACCESS, > + SPR_NOACCESS, &spr_write_generic, > + 0x00000000); > + spr_register(env, SPR_UCTRL, "SPR_UCTRL", > + SPR_NOACCESS, SPR_NOACCESS, > + &spr_read_generic, SPR_NOACCESS, > + 0x00000000);
This doesn't look quite right .... UCTRL is readable also from both user & supervisor mode. And UCTRL should alias CTRL (a la your read_ureg helper). But you've only re-arranged existing code ... not regressed anything. This should be fixed in a follow up patch.