Re: [Qemu-devel] [PATCH v4 04/29] target-ppc: Copy and split gen_spr_7xx() for 970

2014-06-03 Thread Alexey Kardashevskiy
On 06/04/2014 02:32 AM, Tom Musta wrote: > On 6/3/2014 4:27 AM, Alexey Kardashevskiy wrote: >> +static void gen_spr_book3s_pmu_user(CPUPPCState *env) >> +{ >> +spr_register(env, SPR_POWER_UMMCR0, "UMMCR0", >> + &spr_read_ureg, SPR_NOACCESS, >> + &spr_read_ureg, S

Re: [Qemu-devel] [PATCH v4 04/29] target-ppc: Copy and split gen_spr_7xx() for 970

2014-06-03 Thread Tom Musta
On 6/3/2014 4:27 AM, Alexey Kardashevskiy wrote: > +static void gen_spr_book3s_pmu_user(CPUPPCState *env) > +{ > +spr_register(env, SPR_POWER_UMMCR0, "UMMCR0", > + &spr_read_ureg, SPR_NOACCESS, > + &spr_read_ureg, SPR_NOACCESS, > + 0x); >

[Qemu-devel] [PATCH v4 04/29] target-ppc: Copy and split gen_spr_7xx() for 970

2014-06-03 Thread Alexey Kardashevskiy
This stops using 7xx common SPRs init function and adds separate set of helpers for 970. This does not copy ICTC SPR as neither 970 manual nor PowerISA mention it. This defines 970/book3s PMU SPRs constants as they differs from the ones used for 7XX. This creates 2 helpers for PMU SPRs, one for