WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] RFC/Patch: Support for other bootloaders

To: Michal Ostrowski <mostrows@xxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] RFC/Patch: Support for other bootloaders
From: Christian Limpach <christian.limpach@xxxxxxxxx>
Date: Wed, 23 Mar 2005 20:40:58 +0000
Cc: xen-devel@xxxxxxxxxxxxxxxxxxxxx, kaf24@xxxxxxxxxxxx, iap10@xxxxxxxxxxxx
Delivery-date: Wed, 23 Mar 2005 20:44:00 +0000
Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:references; b=XM3kbMU/CFKbT6bTgxSIyb6Ck6jLdzPM1FztHXGflwrvrbHsFT2tFD8sqs7fKqBXkRmxvJsXlj5tVs3JIK25nz8mnlCXChpOFiT8bAy0wbbFo22sEWocpxJqq21pee8t/KDzwxnVl4GUr0v1PBXm3f01jCjvctkCirwXr8bt9fk=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1111419962.4293.128.camel@xxxxxxxxxxxxxxxxxxxx>
List-archive: <http://sourceforge.net/mailarchive/forum.php?forum=xen-devel>
List-help: <mailto:xen-devel-request@lists.sourceforge.net?subject=help>
List-id: List for Xen developers <xen-devel.lists.sourceforge.net>
List-post: <mailto:xen-devel@lists.sourceforge.net>
List-subscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=subscribe>
List-unsubscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=unsubscribe>
References: <1111419962.4293.128.camel@xxxxxxxxxxxxxxxxxxxx>
Reply-to: Christian.Limpach@xxxxxxxxxxxx
Sender: xen-devel-admin@xxxxxxxxxxxxxxxxxxxxx
On Mon, 21 Mar 2005 10:46:02 -0500, Michal Ostrowski
<mostrows@xxxxxxxxxxxxxx> wrote:
> 
> I need to net-boot xen on an HS20 blade.  HS20's have a Broadcom NIC,
> which is not supported by grub.  The attached patch allows me to build a
> Xen image which I can boot using PXELinux  (and the entire SysLinux
> family of boot-loaders).

I think this breaks multiboot support because of:
+void __init __start_xen(void *params)
+{
...
+    multiboot_info_t *mbi = NULL;
+    
+    if (MULTIBOOT_BOOTLOADER_MAGIC == *(u32*)params ) 
+    {
+        mbi = (struct multiboot_info_t*)__va(mbi);

I think this should be:
+    if (MULTIBOOT_BOOTLOADER_MAGIC == *(u32*)__va(params) ) 
+    {
+        mbi = (struct multiboot_info_t*)__va(params);

Also, have you tested your changes on a SMP or HT machine?  I think
you're lucky if it works at all because you rearranged some of the
tests, and now the test for multiboot is no longer protected by the
test if we're running on a secondary cpu (%ebx == 0).

Finally, I think the updated mbootpack is a sufficient solution --
we're trying to fix a limitation in most bootloaders and interposing
something which works around this limitation seems like a sane
approach...

    christian


-------------------------------------------------------
This SF.net email is sponsored by Microsoft Mobile & Embedded DevCon 2005
Attend MEDC 2005 May 9-12 in Vegas. Learn more about the latest Windows
Embedded(r) & Windows Mobile(tm) platforms, applications & content.  Register
by 3/29 & save $300 http://ads.osdn.com/?ad_id=6883&alloc_id=15149&op=click
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/xen-devel