[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH] PPC: mac newworld: fix cpu NIP reset
From: |
Andreas Färber |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH] PPC: mac newworld: fix cpu NIP reset value |
Date: |
Thu, 04 Apr 2013 21:06:58 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130307 Thunderbird/17.0.4 |
Am 04.04.2013 18:47, schrieb Alexander Graf:
> On -M mac99, we can run 970 CPUs. However, these CPUs define the initial
> instruction pointer they start execution at as part of their bootup protocol,
> so effectively it's up to the board to decide where they start.
>
> This went unnoticed, because they used to boot at the same location our flash
> was mapped to, but due to the recent reset changes our 970 CPUs want to reset
> to 0x100 now, which is always a 0 instruction.
>
> Set the initial IP to something reasonable for -M mac99.
>
> Signed-off-by: Alexander Graf <address@hidden>
> ---
> hw/ppc/mac_newworld.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c
> index a08a6b2..ca7d98f 100644
> --- a/hw/ppc/mac_newworld.c
> +++ b/hw/ppc/mac_newworld.c
> @@ -126,6 +126,8 @@ static void ppc_core99_reset(void *opaque)
> PowerPCCPU *cpu = opaque;
>
> cpu_reset(CPU(cpu));
> + /* 970 CPUs want to get their initial IP as part of their boot protocol
> */
> + cpu->env.nip = PROM_ADDR + 0x100;
> }
>
> /* PowerPC Mac99 hardware initialisation */
That is lacking a restriction to 970... :) There's probably some -cpu to
make it start from -4 instead, no?
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg