Kernel KVM virtualization development
 help / color / mirror / Atom feed
From: Kevin O'Connor <kevin@koconnor.net>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Juan Quintela <quintela@redhat.com>,
	KVM devel mailing list <kvm@vger.kernel.org>,
	qemu-devel qemu-devel <qemu-devel@nongnu.org>,
	seabios@seabios.org, ddutile@redhat.com, dwmw2@infradead.org,
	lersek@redhat.com, "Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: KVM call agenda for 2013-05-28
Date: Fri, 31 May 2013 23:11:38 -0400	[thread overview]
Message-ID: <20130601031138.GA6131@morn.localdomain> (raw)
In-Reply-To: <87ehcnmhkz.fsf@codemonkey.ws>

On Fri, May 31, 2013 at 07:58:36AM -0500, Anthony Liguori wrote:
> Kevin O'Connor <kevin@koconnor.net> writes:
> > Given the objections to implementing ACPI directly in QEMU, one
> > possible way forward would be to split the current SeaBIOS rom into
> > two roms: "qvmloader" and "seabios".  The "qvmloader" would do the
> > qemu specific platform init (pci init, smm init, mtrr init, bios
> > tables) and then load and run the regular seabios rom.
> What about a small change to the SeaBIOS build system to allow ACPI
> table generation to be done via a "plugin".

Using a runtime plugin (eg, "qplugin") would require a more complex
handoff then qvmloader.  With qplugin, seabios would need to know what
memory qplugin is compiled to run in and make sure it didn't allocate
anything there.  Similarly, qplugin would need to not stomp on seabios
while it runs, and it would need to coordinate with seabios where to
place the final tables.  With qvmloader, there is no need to
coordinate memory addresses, so it can run anywhere, deploy the tables
in their final location, and then launch seabios.

> This could be as simple as moving acpi.c and *.dsl into the QEMU build
> tree and then having a way to point the SeaBIOS makefiles to our copy of
> it.

I don't see how that would work.  It would complicate the seabios
build (as it would require a copy of qemu source to compile), and the
resulting seabios binary would be strongly tied to the qemu version it
was compiled with and vice-versa.  This would break distro seabios
rpms.  It would also cause great pain when bisecting and would be
confusing even during regular compile/debug cycles.  Internal seabios
calls (eg, memory allocations, pci config accesses) would need to be
static interfaces, etc.

-Kevin

  parent reply	other threads:[~2013-06-01  3:11 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-23 12:41 KVM call agenda for 2013-05-28 Michael S. Tsirkin
2013-05-24  3:02 ` [SeaBIOS] " li guang
2013-05-28 23:53 ` Kevin O'Connor
2013-05-29  8:45   ` Michael S. Tsirkin
2013-05-29 16:12     ` Anthony Liguori
2013-05-29 16:19       ` Michael S. Tsirkin
2013-05-30  6:37       ` [Qemu-devel] " Gerd Hoffmann
2013-06-02 15:05     ` [SeaBIOS] " Gleb Natapov
2013-06-02 15:09       ` Michael S. Tsirkin
2013-06-02 15:40         ` Gleb Natapov
2013-06-02 15:53           ` [SeaBIOS] " Michael S. Tsirkin
2013-06-03  6:25       ` Paolo Bonzini
2013-05-29  8:49   ` Gerd Hoffmann
2013-05-29  9:17     ` Michael S. Tsirkin
2013-05-29  9:42       ` Gerd Hoffmann
2013-05-29  9:46         ` Michael S. Tsirkin
2013-05-29 16:18     ` [SeaBIOS] " Anthony Liguori
2013-05-29 16:28       ` Michael S. Tsirkin
2013-05-29 18:17         ` Michael S. Tsirkin
2013-05-29 16:35       ` [SeaBIOS] " Markus Armbruster
2013-05-30  1:12       ` Kevin O'Connor
2013-05-31 12:16         ` David Woodhouse
2013-05-30  6:12       ` Gerd Hoffmann
2013-05-30  9:23       ` David Woodhouse
2013-05-30 11:13         ` [Qemu-devel] " Laszlo Ersek
2013-05-30 12:19           ` [Qemu-devel] [SeaBIOS] " David Woodhouse
2013-05-30 12:27             ` [Qemu-devel] " Michael S. Tsirkin
2013-05-30 12:43             ` Laszlo Ersek
2013-05-30 16:20             ` [Qemu-devel] [SeaBIOS] " Jordan Justen
2013-05-30 16:41               ` Laszlo Ersek
2013-05-30 16:57                 ` [Qemu-devel] " Jordan Justen
2013-05-30 17:37                   ` Laszlo Ersek
2013-05-30 17:45                   ` [Qemu-devel] " Michael S. Tsirkin
2013-05-31  9:32                 ` [SeaBIOS] " Gerd Hoffmann
2013-05-31  9:55                   ` [SeaBIOS] [Qemu-devel] " Peter Stuge
2013-05-31 23:01                   ` [Qemu-devel] [SeaBIOS] " Jordan Justen
2013-06-03  5:28                     ` Gerd Hoffmann
2013-05-30 17:44               ` [Qemu-devel] " Michael S. Tsirkin
2013-05-31 12:09               ` [Qemu-devel] [SeaBIOS] " David Woodhouse
2013-05-31 19:48                 ` Patrick Georgi
2013-05-29  9:54   ` Michael S. Tsirkin
2013-05-31  2:34   ` Kevin O'Connor
2013-05-31  7:09     ` Jordan Justen
2013-05-31 11:45       ` Laszlo Ersek
2013-05-31 13:04         ` Anthony Liguori
2013-05-31 14:01           ` Laszlo Ersek
2013-05-31 14:38             ` Anthony Liguori
2013-05-31 16:36               ` Laszlo Ersek
2013-05-31 17:10                 ` Anthony Liguori
2013-05-31 19:02               ` Jordan Justen
2013-05-31 20:27                 ` Anthony Liguori
2013-05-31 21:03                   ` Jordan Justen
2013-06-01  0:01                     ` Laszlo Ersek
2013-06-01  3:16                       ` Jordan Justen
2013-05-31 14:08           ` David Woodhouse
2013-05-31 14:28             ` Laszlo Ersek
2013-05-31 15:43             ` Anthony Liguori
2013-05-31 16:33               ` David Woodhouse
2013-05-31 16:54                 ` Laszlo Ersek
2013-05-31 17:06                 ` Anthony Liguori
2013-05-31 18:09                   ` Paolo Bonzini
2013-05-31 18:35                     ` Anthony Liguori
2013-05-31 19:28                       ` Jordan Justen
2013-05-31 20:44                         ` Anthony Liguori
2013-05-31 16:45               ` Laszlo Ersek
2013-06-02  9:43         ` Michael S. Tsirkin
2013-06-03  7:24           ` Jordan Justen
2013-05-31  8:13     ` [SeaBIOS] " Peter Stuge
2013-05-31 10:05       ` Gerd Hoffmann
2013-05-31 13:03       ` Laszlo Ersek
2013-06-01  3:41       ` Kevin O'Connor
2013-05-31 12:58     ` Anthony Liguori
2013-05-31 13:02       ` David Woodhouse
2013-06-01  3:11       ` Kevin O'Connor [this message]
2013-06-02  9:54     ` Michael S. Tsirkin

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=20130601031138.GA6131@morn.localdomain \
    --to=kevin@koconnor.net \
    --cc=anthony@codemonkey.ws \
    --cc=ddutile@redhat.com \
    --cc=dwmw2@infradead.org \
    --cc=kvm@vger.kernel.org \
    --cc=lersek@redhat.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=seabios@seabios.org \
    /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