[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] s390: Fix ram_size updating in machine init
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] s390: Fix ram_size updating in machine init |
Date: |
Tue, 20 Nov 2012 10:31:33 +0100 |
On 20.11.2012, at 10:28, Christian Borntraeger wrote:
> From: Heinz Graalfs <address@hidden>
>
> The global variable 'ram_size' is hidden by the local variable
> declaration in s390_init(). Since we want to update the global
> ram size in certain cases we must not use a local ram_size
> variable.
> - This fixes booting with unusual ram sizes like -m 67001
> - This changes behaviour back to the situation before commit
> 5f072e1f3075bd869e0ace9f2545a85992ac0084
> (create struct for machine initialization arguments)
>
> Signed-off-by: Heinz Graalfs <address@hidden>
> Reviewed-by: Eduardo Habkost <address@hidden>
> Signed-off-by: Christian Borntraeger <address@hidden>
Thanks, applied to s390-next.
Alex
> ---
> hw/s390-virtio.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/hw/s390-virtio.c b/hw/s390-virtio.c
> index ebec844..78477af 100644
> --- a/hw/s390-virtio.c
> +++ b/hw/s390-virtio.c
> @@ -155,7 +155,6 @@ unsigned s390_del_running_cpu(CPUS390XState *env)
> static void s390_init(QEMUMachineInitArgs *args)
> {
> ram_addr_t my_ram_size = args->ram_size;
> - ram_addr_t ram_size = args->ram_size;
> const char *cpu_model = args->cpu_model;
> const char *kernel_filename = args->kernel_filename;
> const char *kernel_cmdline = args->kernel_cmdline;
> --
> 1.7.10.1
>