Re: [Qemu-ppc] [Qemu-devel] [PATCH 2/6] prep: change default cpu to '7448'

[Andreas Färber]

Am 17.03.2012 15:39, schrieb Hervé Poussineau:
> In b37fc148bb81b5b022846156a6cba266e6c23962, PReP default cpu was changed from
> 'default' to '602'. However, '7448' is closer of 'default' than '602'.
> 
> This repairs following command line, which is available in some tutorials:
> qemu-system-ppc -kernel zImage.prep -fda debian_install_root.bin -M prep
> 
> Cc: Gerd Hoffmann <address@hidden>
> Signed-off-by: Hervé Poussineau <address@hidden>

Weak nack. I guess this depends more on the image than on the command
line. If at this point we change the default CPU for -M prep, we would
need to start creating prep-1.0 etc. which I would like to avoid as long
as possible...

Did you happen to found out which physical machine 'prep' corresponds
to? If so, what CPU did it have?
The Sandalfoot supposedly had a 601, the BeBox a dual 603.

Andreas

> ---
>  hw/ppc_prep.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/hw/ppc_prep.c b/hw/ppc_prep.c
> index 06d589d..19a674f 100644
> --- a/hw/ppc_prep.c
> +++ b/hw/ppc_prep.c
> @@ -517,7 +517,7 @@ static void ppc_prep_init (ram_addr_t ram_size,
>  
>      /* init CPUs */
>      if (cpu_model == NULL)
> -        cpu_model = "602";
> +        cpu_model = "7448";
>      for (i = 0; i < smp_cpus; i++) {
>          env = cpu_init(cpu_model);
>          if (!env) {

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg