This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH2] multiboot module support

To: Samuel Thibault <samuel.thibault@xxxxxxxxxxxx>, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH2] multiboot module support
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Tue, 15 Dec 2009 22:18:16 +0000
Delivery-date: Tue, 15 Dec 2009 14:18:40 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20091215204236.GI4962@xxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acp9xyRVXhpfHg8GRzaZP5aJSYPMDAADVs6l
Thread-topic: [Xen-devel] [PATCH2] multiboot module support
User-agent: Microsoft-Entourage/
Your original patch is checked in already, you know?

 -- Keir

On 15/12/2009 20:42, "Samuel Thibault" <samuel.thibault@xxxxxxxxxxxx> wrote:

> So, what's the preferred way?
> - either completely multiboot compliant, has to patch libxc and modify
>   start_info to add the number of modules (my second patch)
> - multiboot compliant except the module table has to end with a NULL
>   entry to let the domU compute the number of modules, has to patch
>   libxc but leave start_info as is (my third patch).
> - multiboot-like, offsets have to be recomputed by the domU, no need to
>   patch libxc for now, until somebody wants to add xend support (my
>   first patch).
> Samuel

Xen-devel mailing list