qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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



  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).