Re: [Qemu-devel] [PATCH 08/40] xenner: kernel: 64-bit files

[Blue Swirl]

On Mon, Nov 1, 2010 at 7:05 PM, Alexander Graf <address@hidden> wrote:
>
> On 01.11.2010, at 15:02, Anthony Liguori wrote:
>
>> On 11/01/2010 02:00 PM, Blue Swirl wrote:
>>> On Mon, Nov 1, 2010 at 3:47 PM, Alexander Graf<address@hidden>  wrote:
>>>
>>>> On 01.11.2010, at 11:44, Anthony Liguori wrote:
>>>>
>>>>
>>>>> On 11/01/2010 10:01 AM, Alexander Graf wrote:
>>>>>
>>>>>> This patch adds various header files required for the xenner kernel on 
>>>>>> 64 bit
>>>>>> systems.
>>>>>>
>>>>>> Signed-off-by: Alexander Graf<address@hidden>
>>>>>>
>>>>>>
>>>>> I think it might make more sense to put this on a separate git.qemu.org 
>>>>> repository and then use a submodule in roms/.
>>>>>
>>>> The main reason why I wanted to have it fully in the qemu tree is so that 
>>>> we can guarantee the qemu<->xenner interface is 100% private. I'm not sure 
>>>> how much it'll be in a flux, but I don't want to go through the hassle of 
>>>> defining stable ABIs where we don't have to :)
>>>>
>>> How is this different from SeaBIOS or OpenBIOS, both use fw_cfg
>>> interface?
>>
>> At least with SeaBIOS, we don't provide a stable ABI.  That's why we ship a 
>> specific version in roms/.
>
> So if we can assume that the interface is 100% internal and can break at any 
> time and there's no guarantee that a newer version of qemu works with an 
> older version of xenner or vice versa, I'm perfectly fine in making it its 
> own project :)

This is the same situation as with OpenBIOS. We also ship known good
pc-bios/openbios-* files to help bisection etc.