Re: [Qemu-devel] [REBASE PATCH v5 1/2] machine: add default_ram_size to machine class

From: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
To: Paolo Bonzini <pbonzini@redhat.com>,
	David Gibson <david@gibson.dropbear.id.au>
Cc: thuth@redhat.com, aik@ozlabs.ru, armbru@redhat.com,
	agraf@suse.de, qemu-devel@nongnu.org, qemu-ppc@nongnu.org,
	marcel.apfelbaum@gmail.com, imammedo@redhat.com
Subject: Re: [Qemu-devel] [REBASE PATCH v5 1/2] machine: add default_ram_size to machine class
Date: Wed, 29 Apr 2015 14:36:06 +0530	[thread overview]
Message-ID: <87pp6nxq0x.fsf@abhimanyu.in.ibm.com> (raw)
In-Reply-To: <5540975F.3090804@redhat.com>

Hi Paolo,

Paolo Bonzini <pbonzini@redhat.com> writes:
> On 29/04/2015 10:14, Nikunj A Dadhania wrote:
>> 
>> Hi Paolo,
>> 
>> David Gibson <david@gibson.dropbear.id.au> writes:
>>> On Tue, Apr 28, 2015 at 01:30:29PM +0200, Thomas Huth wrote:
>>>> On Tue, 28 Apr 2015 12:23:25 +0530
>>>> Nikunj A Dadhania <nikunj@linux.vnet.ibm.com> wrote:
>>> [snip]
>>>> Looks good to me.
>>>>
>>>> Reviewed-by: Thomas Huth <thuth@redhat.com>
>>>
>>> Same here,
>>>
>>> Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
>> 
>> Can you push this patch via you tree?
>> 
>> David will push the spapr changes dependent on this patch through his
>> tree.
>> 
>> https://lists.gnu.org/archive/html/qemu-devel/2015-04/msg04006.html
>> 
>> Regards,
>> Nikunj
>> 
>
> Acked-by: Paolo Bonzini <pbonzini@redhat.com>
>
> so David can push both patches.
>
> But isn't 1G a bit too much?  At least on x86 you can easily boot with 512M.

I understood this number as not the _minimum memory_ to boot the
VM. And this will only come in picture when the user has not specified
any memory.

I have seen the PPC64 VM booting with 256M.

Regards,
Nikunj