Re: [Qemu-devel] [PATCHv2 2/4] spapr: Remove obsolete ram_limit field from sPAPRMachineState

2015-05-28 Thread David Gibson
On Tue, May 26, 2015 at 10:00:31AM +0200, Thomas Huth wrote: > On Tue, 26 May 2015 12:22:57 +1000 > David Gibson wrote: > > > The ram_limit field was imported from sPAPREnvironment where it predates > > the machine's ram size being available generically from machine->ram_size. > > > > Worse, the

Re: [Qemu-devel] [PATCHv2 2/4] spapr: Remove obsolete ram_limit field from sPAPRMachineState

2015-05-26 Thread Thomas Huth
On Tue, 26 May 2015 12:22:57 +1000 David Gibson wrote: > The ram_limit field was imported from sPAPREnvironment where it predates > the machine's ram size being available generically from machine->ram_size. > > Worse, the existing code was inconsistent about where it got the ram size > from. So

[Qemu-devel] [PATCHv2 2/4] spapr: Remove obsolete ram_limit field from sPAPRMachineState

2015-05-25 Thread David Gibson
The ram_limit field was imported from sPAPREnvironment where it predates the machine's ram size being available generically from machine->ram_size. Worse, the existing code was inconsistent about where it got the ram size from. Sometimes it used spapr->ram_limit, sometimes the global 'ram_size' a