xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: xen-devel@lists.xensource.com
Cc: Ian Campbell <ian.campbell@citrix.com>
Subject: [PATCH 00 of 11] RFC: hvmloader SeaBIOS integration
Date: Tue, 05 Apr 2011 16:46:25 +0100	[thread overview]
Message-ID: <patchbomb.1302018385@cosworth.uk.xensource.com> (raw)

The following patchset is a WIP/RFC/vagueidea enabling the build of
two versions of hvmloader, the first using ROMBIOS and the second
using SeaBIOS.

This is achieved by compiling hvmloader.c multiple times which
required a certain amount of disentanglement and reworking to avoid
the need to compile other files more than once. The bulk of the series
consists of that work.

It's not clear to me that this is preferable to building a single
hvmloader which can pollinate a VM with either BIOS.

The final patch, which actually adds the SeaBIOS support, includes a
slew of ugly #ifdef'ery. Since it is not clear at the moment which of
this functionality is not needed with SeaBIOS and which is simply not
ready/working/tested yet I didn't put a great deal of effort into
refactoring that aspect.

It's a great shame that the choice of hvmloader binary was exposed to
users through the config file format. We'll have to cross that bridge
later I guess...

             reply	other threads:[~2011-04-05 15:46 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-05 15:46 Ian Campbell [this message]
2011-04-05 15:46 ` [PATCH 01 of 11] tools: hvmloader: move ROMBIOS configuration into tools/firmware/rombios/ Ian Campbell
2011-04-05 15:46 ` [PATCH 02 of 11] tools: hvmloader: move e820 definitions into rombios Ian Campbell
2011-04-05 15:46 ` [PATCH 03 of 11] tools: hvmloader: pass ACPI_PHYSICAL_ADDRESS as a runtime parameter Ian Campbell
2011-04-05 15:46 ` [PATCH 04 of 11] tools: hvmloader: pass SMBIOS location " Ian Campbell
2011-04-05 15:46 ` [PATCH 05 of 11] tools: hvmloader: split scratch and hypercall addressing from ROMBIOS low HEAP Ian Campbell
2011-04-05 15:46 ` [PATCH 06 of 11] tools: hvmloader: only hvmloader.c needs to know about rombios Ian Campbell
2011-04-05 15:46 ` [PATCH 07 of 11] tools: hvmloader: abort build if BIOS is too big Ian Campbell
2011-04-05 15:46 ` [PATCH 08 of 11] tools: hvmloader: Use more generic name for BIOS ROM image Ian Campbell
2011-04-05 15:46 ` [PATCH 09 of 11] tools: hvmloader: add Makefile option to use debug video BIOS Ian Campbell
2011-04-05 15:46 ` [PATCH 10 of 11] tools: hvmloader: build and install hvmloader-rombios Ian Campbell
2011-04-05 15:46 ` [PATCH 11 of 11] tools: hvmloader: build and install hvmloader-seabios Ian Campbell
2011-04-05 16:09 ` libxl: hiding hvmloader and device_model options from users by default (Was: Re: [PATCH 00 of 11] RFC: hvmloader SeaBIOS integration) Ian Campbell
2011-04-06 13:18   ` Ian Jackson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=patchbomb.1302018385@cosworth.uk.xensource.com \
    --to=ian.campbell@citrix.com \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).