From: Thomas Huth <thuth@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel <qemu-devel@nongnu.org>, "Hoffmann, Gerd" <kraxel@redhat.com>
Subject: Re: [PATCH 1/2] pc_piix: remove pc-i440fx-1.4 up to pc-i440fx-1.7
Date: Fri, 22 Sep 2023 12:21:40 +0200 [thread overview]
Message-ID: <d2af7bf6-3079-366b-1ee4-280e2ca38659@redhat.com> (raw)
In-Reply-To: <CABgObfYgc2iF-89rPGL1siqvwXCGWPgP9Zs-2uOjj0j6jXVEfA@mail.gmail.com>
On 22/09/2023 11.19, Paolo Bonzini wrote:
>
>
> Il ven 22 set 2023, 08:43 Thomas Huth <thuth@redhat.com
> <mailto:thuth@redhat.com>> ha scritto:
>
>
> While you're at it ... do we maybe want to start deprecating the next batch
> of machine types already? (Say pc-i440fx-2.0 up to pc-i440fx-2.2 maybe?)
>
>
> It depends on the benefit. We would have to check the compat options that
> are not needed anymore, and whether they'd be something that is useful
> anyway for debugging.
>
> Also it would be useful to check if isapc can drop some of the compat code
> and realign itself to the 2.0 i440fx machine in terms of QEMU-specific features.
>
> Because of all this todo, I decided not to proceed further with
> deprecations. The 128k ROM on the other hand does provide immediate benefit.
FWIW, when I was working on deprecating the old pc machine types, I
originally wanted to stop with pc-1.3 (since I was most bugged by the
different naming between "pc" and "pc-i44fx" in the help output). I then
only put the 1.4 to 1.7 machine types on the deprecation list without
anything in mind. Now this became useful for the 128k bios rework... so
maybe we should continue deprecating older machine types for other future
reworks that we don't envision yet.
Thomas
next prev parent reply other threads:[~2023-09-22 10:22 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-21 12:10 [PATCH 0/2] pc_piix: remove machines using a 128K BIOS Paolo Bonzini
2023-09-21 12:10 ` [PATCH 1/2] pc_piix: remove pc-i440fx-1.4 up to pc-i440fx-1.7 Paolo Bonzini
2023-09-22 6:43 ` Thomas Huth
2023-09-22 9:19 ` Paolo Bonzini
2023-09-22 10:21 ` Thomas Huth [this message]
2023-09-21 12:10 ` [PATCH 2/2] seabios: remove PCI drivers from bios.bin Paolo Bonzini
2023-09-22 6:48 ` Thomas Huth
2023-09-22 9:20 ` Paolo Bonzini
2023-09-23 6:28 ` Volker Rümelin
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=d2af7bf6-3079-366b-1ee4-280e2ca38659@redhat.com \
--to=thuth@redhat.com \
--cc=kraxel@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.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).