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

[Anthony Liguori]

On 11/01/2010 10:47 AM, Alexander Graf 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 :)
>    

Yeah, I'm not suggesting it be a separate project.  But we've been 
keeping firmware out of tree for the most part and using submodules.  I 
like this model (even though git submodules don't always work the way 
you expect them to..).

I'm not passionate either way.  If you feel strongly about it being in 
the main source tree, I don't have a problem with it.  But stick it in 
roms/ instead of pc-bios.

Regards,

Anthony Liguori

> Alex
>
>