Re: [Qemu-devel] [RFC PATCH V1 1/2] arm_boot: added linux switch

[Andreas Färber]

Am 25.06.2012 03:51, schrieb Peter Crosthwaite:
> Ping! @PMM
> 
> You rejected my workaround in V2 (i.e. using -dtb to force is_linux)
> on the grounds that youd "accept some reasonable way of saying "this
> ELF file is a Linux kernel"," That brings us back to V1 - this patch.
> Any objections?

I won't object to it, but I don't like it: If it's really needed, it
should be os=linux or something (enum?) for extensibility or at some
point we'll need windows=on, vxworks=on, aix=on, etc.

Andreas

> 
> Regards,
> Peter
> 
> On Fri, Jun 1, 2012 at 11:41 AM, Peter Maydell <address@hidden> wrote:
>> On 1 June 2012 02:16, Peter A. G. Crosthwaite
>> <address@hidden> wrote:
>>> Added a switch to tell the bootloader that the image is linux and should be
>>> bootstrapped as such. This is needed to boot an elf that is linux.
>>>
>>> Syntax would be:
>>>
>>> qemu-system-arm ... -kernel linux.elf -machine linux=on
>>
>> This whole area is a mess. Personally I think we should have a
>> standard way to say "load this ELF file into memory" which isn't
>> target-dependent, and then we could make -kernel actually load
>> a kernel rather than its current "load a kernel unless it's an
>> ELF file in which case do something else".
>>
>> There are some back-compatibility issues there, though.
>>
>> -- PMM
> 


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg