Re: [Qemu-devel] [PATCH v3 0/7] Let boards state maximum RAM limits in QEMUMachine struct

[Peter Maydell]

On 30 March 2011 14:56, Anthony Liguori <address@hidden> wrote:
> On 03/30/2011 08:22 AM, Peter Maydell wrote:
>> Not really, typically they're just filled up in some particular
>> order (main RAM in one place and expansion RAM elsewhere).
>> Since the board init function is only passed a single "ram_size"
>> parameter that's all you can do anyhow.
>
> FWIW, I don't think any static data is going to be perfect here.  A lot of
> boards have strict requirements on ram_size based on plausible combinations
> of DIMMs.  Arbitrary values up to ram_size is not good enough.
>
> ram_size ought to be viewed as a hint, not a mechanism to allow common code
> to completely validate the passed in ram size parameter.  It's still up to
> the board to validate that the given ram size makes sense.

Yes, I agree, so we shouldn't try to specify some complicated
set of static data that still won't be good enough.

I'm trying to make it easy for boards to avoid crashing horribly
when the user passes a bad value; that's all.

-- PMM