qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Jordan Justen <jljusten@gmail.com>
Cc: KVM devel mailing list <kvm@vger.kernel.org>,
	Juan Quintela <quintela@redhat.com>,
	David Woodhouse <dwmw2@infradead.org>,
	seabios@seabios.org,
	qemu-devel qemu-devel <qemu-devel@nongnu.org>,
	Kevin O'Connor <kevin@koconnor.net>,
	Gerd Hoffmann <kraxel@redhat.com>,
	Anthony Liguori <anthony@codemonkey.ws>,
	"Jordan Justen (Intel address)" <jordan.l.justen@intel.com>,
	Laszlo Ersek <lersek@redhat.com>
Subject: Re: [Qemu-devel] [SeaBIOS] KVM call agenda for 2013-05-28
Date: Thu, 30 May 2013 20:44:05 +0300	[thread overview]
Message-ID: <20130530174405.GE29091@redhat.com> (raw)
In-Reply-To: <CAFe8ug_=WJEvJspRRWsTG7POWBxmeO3+kcUVQHmntt0gxZqfRA@mail.gmail.com>

On Thu, May 30, 2013 at 09:20:42AM -0700, Jordan Justen wrote:
> On Thu, May 30, 2013 at 5:19 AM, David Woodhouse <dwmw2@infradead.org> wrote:
> > On Thu, 2013-05-30 at 13:13 +0200, Laszlo Ersek wrote:
> >> Where is CorebootPkg available from?
> >
> > https://github.com/pgeorgi/edk2/tree/coreboot-pkg
> 
> Is the license on this actually BSD as the License.txt indicates?
> 
> Is this planned to be upstreamed?
> 
> Does this support UEFI variables?
> 
> Does this support UEFI IA32 / X64?
> 
> >> > And it helps to dispel the stupid misconception in some quarters that
> >> > Coreboot *competes* with UEFI and thus cannot possibly be supported
> >> > because helping something that competes with UEFI would be bad.
> 
> Coreboot and EDK II both provide a good infrastructure for
> initializing hardware. So, they compete on that point.
> 
> Coreboot then focuses on booting coreboot payloads, while EDK II
> focuses on UEFI support. On that point they don't compete, but the
> focus is different.
> 
> Of course, you can build a layer of EDK II => Coreboot payload
> support, or Coreboot => EDK II (CorebootPkg, I guess?), but the match
> will not be perfect. (That is not to say it can't work.)
> 
> >> I'm not sure who do you mean by "some quarters", but for some
> >> distributions Coreboot would be yet another component (package) to
> >> support, for no obvious benefit.
> >>
> >> (Gerd said it better than I possibly could:
> >> <http://thread.gmane.org/gmane.comp.bios.coreboot.seabios/5685/focus=5705>.)
> >
> > Yeah, but if we're shoving a lot of hardware-specific ACPI table
> > generation into the guest's firmware, instead of just doing it on the
> > qemu side where a number of us seem to think it belongs, then there *is*
> > a benefit to using Coreboot. When stuff changes on the qemu side and we
> > have to update the table generation to match, you end up having to
> > update just the Coreboot package, and *not* having to patch both SeaBIOS
> > and OVMF.
> 
> I think ACPI table generation lives in firmware on real products,
> because on real products the firmware is the point that best
> understands the actual hardware layout for the machine. In qemu, I
> would say that qemu best knows the hardware layout, given that the
> firmware is generally a slightly separate project from qemu.

Of course ACPI tables are firmware.
Please note that what my patches do is simply supply
templates for ACPI tables on a ROM device separate
from where bios code resides.
bios still tweaks them in minor ways.
I am guessing this is what happens on real hardware too:
most tables pre-generated in ROM, firmware shadows
them and tweaks them in minor ways.

> I don't think adding a coreboot layer into the picture helps, if it
> brings along the coreboot payload boot interface as a requirement.
> 
> Then again, I don't really understand how firmware could be swapped
> out in this case. What would -bios do? How would the coreboot ACPI
> shim layer be specified to qemu?
> 
> -Jordan

  parent reply	other threads:[~2013-05-30 17:45 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
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 [this message]
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=20130530174405.GE29091@redhat.com \
    --to=mst@redhat.com \
    --cc=anthony@codemonkey.ws \
    --cc=dwmw2@infradead.org \
    --cc=jljusten@gmail.com \
    --cc=jordan.l.justen@intel.com \
    --cc=kevin@koconnor.net \
    --cc=kraxel@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=lersek@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).