[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [Linaro-uefi] [PATCH] xen: arm: implement generic multiboot compatibility strings (Was: Re: The GRUB multiboot support patch for aarch64(V3.1))



On 06/06/2014 01:24 PM, Fu Wei wrote:
> On 06/06/2014 02:31 AM, Ian Campbell wrote:
>> On Thu, 2014-06-05 at 18:03 +0100, Julien Grall wrote:
>>>>> While we are modifying the protocol, "linux-zImage" is confusing in the
>>>>> name. Actually we can use it for an ELF, another OS... I don't think Xen
>>>>> will change his behavior depending of the DOM0 image.
>>
>> Actually thinking about this some more I think you are right. Xen
>> already probes the kernel it gets so we can safely implement this as
>> multiboot,kernel, since we don't really need the more specific type. If
>> in the future some non-probable kernel comes along which we want to
>> support we still have the option of adding more specific compatibility
>> strings.
>>
>> Fu Wei -- if this is OK with you I will modify the wiki page to
>> s/multiboot,linux-zimage/multiboot,kernel/ and rev this patch to suit.
> 
> This is OK for me, And I think the "multiboot,kernel" is better and more 
> generic. :-)
> 
>>
>> Can we do something similar with linux-ramdisk? I'm not sure since we
>> cannot easily probe the ramdisk contents. We could base the ramdisk
>> behaviour on the probed behaviour of the kernel. Anyone got any
>> thoughts?
> 
> My thought looks exactly the same as yours :
> The cpio utility can detect the cpio file format. Maybe we can just probe the 
> file, see if this is a cpio or cpio.gz.

cpio is not Linux specific. Probing just the file won't help here to to
determine if we have to add the properties linux,initrd-* or another set.

Regards,

-- 
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.