qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>, Kevin O'Connor <kevin@koconnor.net>
Cc: Phil Dennis-Jordan <phil@philjordan.eu>,
	seabios@seabios.org,
	"qemu-devel@nongnu.org qemu-devel" <qemu-devel@nongnu.org>,
	Programmingkid <programmingkidx@gmail.com>,
	Phil Dennis-Jordan <lists@philjordan.eu>,
	Igor Mammedov <imammedo@redhat.com>,
	Laszlo Ersek <lersek@redhat.com>,
	Richard Henderson <rth@twiddle.net>
Subject: Re: [Qemu-devel] [SeaBIOS] Commit 77af8a2b95b79699de650965d5228772743efe84 breaks Windows 2000 support
Date: Thu, 27 Jul 2017 10:39:10 +0200	[thread overview]
Message-ID: <1501144750.27172.1.camel@redhat.com> (raw)
In-Reply-To: <1419439438.19249924.1501100483485.JavaMail.zimbra@redhat.com>

  Hi,

> > SeaBIOS is used with both modern and legacy OSes, and it doesn't
> > have
> > any knowledge about what kind of OS will be used.  If anything, I'd
> > argue that QEMU has more knowledge about the guest OS than SeaBIOS
> > does (due to command-line options like machine version).
> 
> No, the machine type is independent from the guest OS.

Well, not really.  Old guests (winxp & older) don't do very well on pc.
 Modern ones are better served with q35, because they either don't boot
on pc (macos I think) or can use the features of the modern hardware,
like pci express.

I still like the idea to have pc machine type provide the older stuff. 
The hardware we emulate for the pc machine type is old enough that old
acpi versions should have no problems describing it.

> > A - It would require deploying SeaBIOS and QEMU in lock-step.  To
> > get
> >     this in for QEMU v2.10 would require making QEMU changes during
> >     the soft freeze and would require a SeaBIOS "stable" release
> > that
> >     introduces ACPI table manipulation.
> 
> Yes.

Since the switch to qemu-generated acpi tables we were able to avoid
that kind of lockstep updates, I'd prefer to not loose that.

> > B - I don't have full confidence the proposed ACPI changes wont
> > expose
> >     a quirk in some obscure OS from the last 25 years.  If it does
> >     expose a quirk, any work-around would likely require deploying
> > a
> >     new SeaBIOS and QEMU in lock-step.
> 
> Well, the solution is proposed by ACPI itself, and the worst that can
> happen is that some OS prefers the RSDT to the XSDT (which we already
> test on Windows 2000).
> 
> > C - We'd be introducing "shared ownership" of the acpi
> > tables.  Some
> >     of the tables would be produced by QEMU and some of them by
> >     SeaBIOS.  Explaining when and why to future developers would be
> > a
> >     challenge.
> 
> The advantage is that the same shared ownership is already present in
> OVMF.  The RSDP/RSDT/XSDT are entirely created by the firmware in
> OVMF.

Hmm, seabios could do the same then, and we would not need a lockstep
update?

cheers,
  Gerd

  reply	other threads:[~2017-07-27  8:39 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-20 16:40 [Qemu-devel] Commit 77af8a2b95b79699de650965d5228772743efe84 breaks Windows 2000 support Programmingkid
2017-07-20 19:29 ` Phil Dennis-Jordan
2017-07-21  0:00   ` Programmingkid
2017-07-21  9:06   ` Igor Mammedov
2017-07-21  9:11     ` Phil Dennis-Jordan
2017-07-21  9:23     ` Daniel P. Berrange
2017-07-21 12:34       ` Igor Mammedov
2017-07-21 18:29         ` Phil Dennis-Jordan
2017-07-25 16:14           ` Laszlo Ersek
2017-07-25 16:23             ` Paolo Bonzini
2017-07-25 17:10               ` Paolo Bonzini
2017-07-25 21:25                 ` Phil Dennis-Jordan
2017-07-26  8:53                   ` Paolo Bonzini
2017-07-26 11:42                     ` Laszlo Ersek
2017-07-26 12:06                       ` Paolo Bonzini
2017-07-25 22:01                 ` [Qemu-devel] [SeaBIOS] " Kevin O'Connor
2017-07-26  7:20                   ` Paolo Bonzini
2017-07-26 19:12                     ` Kevin O'Connor
2017-07-26 20:21                       ` Paolo Bonzini
2017-07-27  8:39                         ` Gerd Hoffmann [this message]
2017-07-27 12:26                           ` Paolo Bonzini
2017-07-27 14:59                         ` Kevin O'Connor
2017-07-27 17:46                           ` Laszlo Ersek
2017-07-28  6:57                             ` Gerd Hoffmann
2017-07-26 13:08               ` [Qemu-devel] " Igor Mammedov
2017-07-26 13:10                 ` Paolo Bonzini
2017-07-26 13:30                   ` Igor Mammedov
2017-07-26 13:33                     ` Paolo Bonzini
2017-07-26 13:43                       ` Igor Mammedov
2017-07-26 14:04                         ` Daniel P. Berrange
2017-07-26 16:13                           ` Michael S. Tsirkin
2017-07-26 13:57                     ` Michael S. Tsirkin
2017-07-24 12:45     ` Gerd Hoffmann
2017-07-24 16:43     ` John Snow
2017-07-24 17:30       ` Programmingkid
2017-07-21  9:20   ` Daniel P. Berrange
2017-07-21  9:46     ` Igor Mammedov
2017-07-21 10:39       ` Phil Dennis-Jordan
2017-07-21 10:50       ` BALATON Zoltan
2017-07-21 11:46         ` Phil Dennis-Jordan
2017-07-21 17:17           ` BALATON Zoltan

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=1501144750.27172.1.camel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=kevin@koconnor.net \
    --cc=lersek@redhat.com \
    --cc=lists@philjordan.eu \
    --cc=pbonzini@redhat.com \
    --cc=phil@philjordan.eu \
    --cc=programmingkidx@gmail.com \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    --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).