Thomas Huth <th...@linux.vnet.ibm.com> writes:
> On Thu, 05 Mar 2015 16:01:40 +0530
> Nikunj A Dadhania <nik...@linux.vnet.ibm.com> wrote:
>
>> Hi Igor,
>>
>> Thanks for the review.
>>
>> Igor Mammedov <imamm...@redhat.com> writes:
>> > On Thu, 5 Mar 2015 14:36:10 +0530
>> > Nikunj A Dadhania <nik...@linux.vnet.ibm.com> wrote:
>> >
>> >> Machines types can have different requirement for default ram
>> >> size. Introduce a member in the machine class and set the current
>> >> default_ram_size to 128MB.
>> >>
>> >> For QEMUMachine types override the value during the registration of
>> >> the machine and for MachineClass introduce the generic class init
>> >> setting the default_ram_size.
>> >>
>> >> In case the user passes memory that is lesser that the default ram
>> >> size, upscale the value to the machine's default ram size with a
>> >> warning.
> ...
>> >> @@ -2684,6 +2683,12 @@ static void set_memory_options(uint64_t
>> >> *ram_slots, ram_addr_t *maxram_size)
>> >> exit(EXIT_FAILURE);
>> >> }
>> >>
>> >> + if (ram_size < default_ram_size) {
>> >> + fprintf(stderr, "WARNING: qemu: %s guest ram size defaulting to
>> >> %ld MB\n",
>> >> + mc->name, default_ram_size / (1024 * 1024));
>> >> + ram_size = default_ram_size;
>> >> + }
>> > In previous review someone explicitly asked not to override lower ram_size
>> > if it was requested by user on command line.
>>
>> We would get to a state where the VM is not bootable. I understand that
>> user has provided a value, but what if the value is not correct?
>
> Well, as I said before: There are older versions of Linux which run fine
> with 128 MB or even 64 MB of memory. Do you really want to block this
> just because newer Linux distros now need more RAM now by default?
> IMHO if the user specified the amount of RAM at the command line, you
> can assume that they know what they are doing.
Sure, I can then just use that input without warning/rejection.
Regards
Nikunj
