|
From: | Anthony Liguori |
Subject: | Re: [Qemu-devel] [PATCH 08/40] xenner: kernel: 64-bit files |
Date: | Mon, 01 Nov 2010 10:59:16 -0500 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101027 Lightning/1.0b1 Thunderbird/3.0.10 |
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
[Prev in Thread] | Current Thread | [Next in Thread] |