Re: [Qemu-stable] [PATCH for-1.5] ppc: do not register IABR SPR twice for 603e

[Hervé Poussineau]

mdroth a écrit :
> On Mon, May 20, 2013 at 08:17:47PM +0200, Hervé Poussineau wrote:
> > Ping.
> > This should be IMO committed to stable, as it fixes a crash with
> > qemu-system-ppc -M prep -cpu 603e
>
> Ping^2. Looking to pull this in for 1.5.1

Ping^3

> > Hervé
> >
> > Hervé Poussineau a écrit :
> > > IABR SPR is already registered in gen_spr_603(), called from init_proc_603E().
> > >
> > > Signed-off-by: Hervé Poussineau <address@hidden>
> > > ---
> > > target-ppc/translate_init.c |    5 -----
> > > 1 file changed, 5 deletions(-)
> > >
> > > diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c
> > > index 6feb62a..248d3e0 100644
> > > --- a/target-ppc/translate_init.c
> > > +++ b/target-ppc/translate_init.c
> > > @@ -4980,11 +4980,6 @@ static void init_proc_603E (CPUPPCState *env)
> > >                  SPR_NOACCESS, SPR_NOACCESS,
> > >                  &spr_read_generic, &spr_write_generic,
> > >                  0x00000000);
> > > -    /* XXX : not implemented */
> > > -    spr_register(env, SPR_IABR, "IABR",
> > > -                 SPR_NOACCESS, SPR_NOACCESS,
> > > -                 &spr_read_generic, &spr_write_generic,
> > > -                 0x00000000);
> > >     /* Memory management */
> > >     gen_low_BATs(env);
> > >     gen_6xx_7xx_soft_tlb(env, 64, 2);