From: Kevin O'Connor <kevin@koconnor.net>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: KVM devel mailing list <kvm@vger.kernel.org>,
"Michael S. Tsirkin" <mst@redhat.com>,
seabios@seabios.org,
qemu-devel qemu-devel <qemu-devel@nongnu.org>,
Juan Quintela <quintela@redhat.com>,
Gerd Hoffmann <kraxel@redhat.com>,
dwmw2@infradead.org
Subject: Re: [Qemu-devel] [SeaBIOS] KVM call agenda for 2013-05-28
Date: Wed, 29 May 2013 21:12:29 -0400 [thread overview]
Message-ID: <20130530011229.GA21234@morn.localdomain> (raw)
In-Reply-To: <871u8p92v8.fsf@codemonkey.ws>
On Wed, May 29, 2013 at 11:18:03AM -0500, Anthony Liguori wrote:
> Gerd Hoffmann <kraxel@redhat.com> writes:
> > On 05/29/13 01:53, Kevin O'Connor wrote:
> >> Raised
> >> that QOM interface should be sufficient.
> >
> > Agree on this one. Ideally the acpi table generation code should be
> > able to gather all information it needs from the qom tree, so it can be
> > a standalone C file instead of being scattered over all qemu.
>
> Ack. So my basic argument is why not expose the QOM interfaces to
> firmware and move the generation code there?
I remain doubtful that QOM has all the info needed to generate the
BIOS tables. Does QOM describe how the 5th pci device uses global
interrupt 11 when using global interrupts, legacy interrupt 5 when not
using global interrupts, and that the legacy interrupt can be changed
by writing to the 0x60 address of the 1st pci device's config space?
Does QOM state that the machine supports S3 sleep mode? Does QOM
indicate that an IPMI device supports the 3rd version of the IPMI
device specification?
I don't see how exporting QOM to the firmware will help. I predict we
would continue to see most of the BIOS tables hardcoded in the
firmware and that all but the most minor changes to those tables would
require synchronizing code patches to both QEMU and the firmware. I
also suspect we would end up adding fields to QOM that only the BIOS
tables cared about, and that ever increasing code would be needed in
both QEMU and the firmware to juggle to/from QOM so that the BIOS
tables could be created.
-Kevin
next prev parent reply other threads:[~2013-05-30 1:12 UTC|newest]
Thread overview: 74+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-23 12:41 [Qemu-devel] KVM call agenda for 2013-05-28 Michael S. Tsirkin
2013-05-24 3:02 ` [Qemu-devel] [SeaBIOS] " li guang
2013-05-28 23:53 ` [Qemu-devel] " 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 ` Gerd Hoffmann
2013-06-02 15:05 ` [Qemu-devel] [SeaBIOS] " Gleb Natapov
2013-06-02 15:09 ` Michael S. Tsirkin
2013-06-02 15:40 ` Gleb Natapov
2013-06-02 15:53 ` 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 ` Anthony Liguori
2013-05-29 16:28 ` Michael S. Tsirkin
2013-05-29 18:17 ` Michael S. Tsirkin
2013-05-29 16:35 ` Markus Armbruster
2013-05-30 1:12 ` Kevin O'Connor [this message]
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 ` Laszlo Ersek
2013-05-30 12:19 ` David Woodhouse
2013-05-30 12:27 ` Michael S. Tsirkin
2013-05-30 12:43 ` Laszlo Ersek
2013-05-30 16:20 ` Jordan Justen
2013-05-30 16:41 ` Laszlo Ersek
2013-05-30 16:57 ` Jordan Justen
2013-05-30 17:37 ` Laszlo Ersek
2013-05-30 17:45 ` Michael S. Tsirkin
2013-05-31 9:32 ` Gerd Hoffmann
2013-05-31 9:55 ` Peter Stuge
2013-05-31 23:01 ` Jordan Justen
2013-06-03 5:28 ` Gerd Hoffmann
2013-05-30 17:44 ` Michael S. Tsirkin
2013-05-31 12:09 ` David Woodhouse
2013-05-31 19:48 ` Patrick Georgi
2013-05-29 9:54 ` [Qemu-devel] " Michael S. Tsirkin
2013-05-31 2:34 ` Kevin O'Connor
2013-05-31 7:09 ` Jordan Justen
2013-05-31 8:13 ` [Qemu-devel] [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 11:45 ` [Qemu-devel] " Laszlo Ersek
2013-05-31 13:04 ` Anthony Liguori
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
[not found] ` <51A8AD52.3070901@redhat.com>
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-06-02 9:43 ` Michael S. Tsirkin
2013-06-03 7:24 ` Jordan Justen
2013-05-31 12:58 ` Anthony Liguori
2013-05-31 13:02 ` David Woodhouse
2013-06-01 3:11 ` Kevin O'Connor
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=20130530011229.GA21234@morn.localdomain \
--to=kevin@koconnor.net \
--cc=anthony@codemonkey.ws \
--cc=dwmw2@infradead.org \
--cc=kraxel@redhat.com \
--cc=kvm@vger.kernel.org \
--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;
as well as URLs for NNTP newsgroup(s).