| Message ID | 1320731536-484-1-git-send-email-david@gibson.dropbear.id.au |
|---|---|
| State | New |
| Headers | show |
Am Tue, 8 Nov 2011 16:52:16 +1100 schrieb David Gibson <david@gibson.dropbear.id.au>: > 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 think? Thomas
On 11/08/2011 11:12 AM, Thomas Huth wrote: > Am Tue, 8 Nov 2011 16:52:16 +1100 > schrieb David Gibson<david@gibson.dropbear.id.au>: > >> 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 > think? Yes, please. That's basically what the patch is changing, so the printf should be changed as well :). If you find it too confusing, you can say "%ldM guest RAM (RMA)". Whoever uses KVM on a 970 will have to manually allocate RMA regions in the first place, so he will know what it means. Everyone else will read RAM (which should be correct for p7 KVM and TCG). Alex
On Fri, Nov 11, 2011 at 11:51:12AM +0100, Alexander Graf wrote: > On 11/08/2011 11:12 AM, Thomas Huth wrote: > >Am Tue, 8 Nov 2011 16:52:16 +1100 > >schrieb David Gibson<david@gibson.dropbear.id.au>: > > > >>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 > >think? > > Yes, please. That's basically what the patch is changing, so the > printf should be changed as well :). If you find it too confusing, > you can say "%ldM guest RAM (RMA)". Whoever uses KVM on a 970 will > have to manually allocate RMA regions in the first place, so he will > know what it means. Everyone else will read RAM (which should be > correct for p7 KVM and TCG). Duh, yeah, oversight. New version coming.
diff --git a/hw/spapr.c b/hw/spapr.c index 40cfc9b..e826e0b 100644 --- a/hw/spapr.c +++ b/hw/spapr.c @@ -57,7 +57,7 @@ #define FW_MAX_SIZE 0x400000 #define FW_FILE_NAME "slof.bin" -#define MIN_RAM_SLOF 512UL +#define MIN_RMA_SLOF 128UL #define TIMEBASE_FREQ 512000000ULL @@ -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); exit(1); } filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, FW_FILE_NAME);
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. In addition, more recent versions of SLOF that were committed earlier don't need quite as much memory as earlier versions. Therefore, this patch also reduces the amount of RAM we require to run SLOF. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> --- hw/spapr.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-)