[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-ppc] [PATCH] pseries: Correct RAM size check for SLOF

From: Thomas Huth
Subject: Re: [Qemu-ppc] [PATCH] pseries: Correct RAM size check for SLOF
Date: Tue, 8 Nov 2011 11:12:21 +0100

Am Tue,  8 Nov 2011 16:52:16 +1100
schrieb David Gibson <address@hidden>:

> The SLOF firmware used on the pseries machine needs a reasonable amount of
> (guest) RAM in order to run, so we have a check in the machine init
> function to check that this is available.  However, SLOF runs in real mode
> (MMU off) which means it can only actually access the RMA (Real Mode Area),
> not all of RAM.  In many cases the RMA is the same as all RAM, but when
> running with Book3S HV KVM on PowerPC 970, the RMA must be especially
> allocated to be (host) physically contiguous.  In this case, the RMA size
> is determined by what the host admin allocated at boot time, and will
> usually be less than the whole guest RAM size.
> This patch corrects the test to see if SLOF has enough memory for this
> case.
> @@ -562,9 +562,9 @@ static void ppc_spapr_init(ram_addr_t ram_size,
>          spapr->entry_point = KERNEL_LOAD_ADDR;
>      } else {
> -        if (ram_size < (MIN_RAM_SLOF << 20)) {
> +        if (rma_size < (MIN_RMA_SLOF << 20)) {
>              fprintf(stderr, "qemu: pSeries SLOF firmware requires >= "
> -                    "%ldM guest RAM\n", MIN_RAM_SLOF);
> +                    "%ldM guest RAM\n", MIN_RMA_SLOF);

Maybe it's more helpful for the unexperienced users when we change the
term "guest RAM" to "real mode area (RMA) memory" here? What do you


reply via email to

[Prev in Thread] Current Thread [Next in Thread]