Re: [Qemu-devel] [RFC PATCH] arm boot: added QOM device definition

[Anthony Liguori <anthony@codemonkey.ws>]

On 02/08/2012 10:03 AM, Peter Crosthwaite wrote:
> 2012/2/9 Paul Brook<paul@codesourcery.com>
>
>>> So here are some of the problems im trying to solve with the bootloader:
>>>
>>> Smp bootstrap secondary CPUs while loading an elf (currently elfs will be
>>> assumed to be not kernels).
>>> Change the kernel, initrd and dtb load address on the command line.
>>> Use my own SMP secondary bootloop.
>>>
>>> My intention with this patch was to set myself to do boot
>> parameterizations
>>> on the command line by just adding them as qdev props to the
>>> arm_linux_loader and set them using -device instantiation. E.G. -device
>>> arm_boot_loader,initrd_addr=0x10000000. But if I take the approach you
>> are
>>> suggesting, the for this initrd load address option, I would need to add
>>> myself a command line option, fetch that command line option in every arm
>>> machine model and then pass it to arm_load_kernel.
>>
>> No.  You just set/override properties on the device that the machine
>> created.
>> I thought we already had this, but it looks like the closest we have is
>> -global.
>>
>> Something like "-property /devicepath/propertyname=value".

A lot of refactoring is needed to make this a reality.  Please read through the 
various QOM threads for details of what needs to be done.

Regards,

Anthony Liguori