From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christian Limpach Subject: Re: RFC/Patch: Support for other bootloaders Date: Wed, 23 Mar 2005 20:40:58 +0000 Message-ID: <3d8eece205032312404294db9d@mail.gmail.com> References: <1111419962.4293.128.camel@brick.watson.ibm.com> Reply-To: Christian.Limpach@cl.cam.ac.uk Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit In-Reply-To: <1111419962.4293.128.camel@brick.watson.ibm.com> Sender: xen-devel-admin@lists.sourceforge.net Errors-To: xen-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Michal Ostrowski Cc: xen-devel@lists.sourceforge.net, kaf24@cl.cam.ac.uk, iap10@cl.cam.ac.uk List-Id: xen-devel@lists.xenproject.org On Mon, 21 Mar 2005 10:46:02 -0500, Michal Ostrowski 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