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

[Peter Maydell]

On 26 March 2011 11:41, Blue Swirl <address@hidden> wrote:
> On Mon, Mar 21, 2011 at 7:47 PM, Peter Maydell <address@hidden> wrote:
>> This fairly simple patchset adds a new 'max_ram' field to the QEMUMachine
>> structure so that a board model can specify the maximum RAM it will accept.
>> We can then produce a friendly diagnostic message when the user tries to
>> start qemu with a '-m' option asking for more RAM than that. (Currently
>> most of the ARM devboard models respond with an obscure guest crash when
>> the guest tries to access RAM and finds device registers instead.)

> This could replace the field max_mem in hwdef structures in sun4m.c.

I've had a go at this, and it's revealed a flaw in this patchset,
which is that the max_ram field needs to be type target_physaddr_t
rather than ram_addr_t. I'll post another patchset including a sun4m
cleanup(*) once I've done some testing on it.

(*) includes combining the QEMUMachine struct into the sun4*_hwdef
struct, which is necessary so the init fn can get at it and also
I think neater.

-- PMM