| Message ID | 20231006075247.403364-1-thuth@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | docs/about: Mark the old pc-i440fx-2.0 - 2.3 machine types as deprecated | expand |
On 6/10/23 09:52, Thomas Huth wrote: > As we've seen in the past, it's useful for deprecating old machine > types to finally be able to get of legacy code or do other clean-ups > (see e.g. commit ea985d235b868047 that was used to drop the PCI code in > the 128k bios binaries to free some precious space in those binaries). > > So let's continue deprecating the oldest pc machine types. QEMU 2.3 > has been released 8 years ago, so that's plenty of time since such > machine types have been used by default, thus deprecating pc-i440fx-2.0 > up to pc-i440fx-2.3 should be fine nowadays. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > docs/about/deprecated.rst | 8 ++++++++ > hw/i386/pc_piix.c | 1 + > 2 files changed, 9 insertions(+) Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
On Fri, Oct 06, 2023 at 09:52:47AM +0200, Thomas Huth wrote: > As we've seen in the past, it's useful for deprecating old machine > types to finally be able to get of legacy code or do other clean-ups > (see e.g. commit ea985d235b868047 that was used to drop the PCI code in > the 128k bios binaries to free some precious space in those binaries). > > So let's continue deprecating the oldest pc machine types. QEMU 2.3 > has been released 8 years ago, so that's plenty of time since such > machine types have been used by default, thus deprecating pc-i440fx-2.0 > up to pc-i440fx-2.3 should be fine nowadays. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > docs/about/deprecated.rst | 8 ++++++++ > hw/i386/pc_piix.c | 1 + > 2 files changed, 9 insertions(+) > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst > index e43de9c183..c016bb9b20 100644 > --- a/docs/about/deprecated.rst > +++ b/docs/about/deprecated.rst > @@ -277,6 +277,14 @@ deprecated; use the new name ``dtb-randomness`` instead. The new name > better reflects the way this property affects all random data within > the device tree blob, not just the ``kaslr-seed`` node. > > +``pc-i440fx-2.0`` up to ``pc-i440fx-2.3`` (since 8.2) > +''''''''''''''''''''''''''''''''''''''''''''''''''''' > + > +These old machine types are quite neglected nowadays and thus might have > +various pitfalls with regards to live migration. Use a newer machine type > +instead. > + > + > Backend options > --------------- Wait a sec. Which tree is this against? I don't see this context ever since: commit f59fb1889f480b0324a7ed0404a68ad5a0ad4f6c Author: Thomas Huth <thuth@redhat.com> Date: Mon Jan 17 20:16:39 2022 +0100 hw/i386/pc_piix: Mark the machine types from version 1.4 to 1.7 as deprecated Also, do we intentionally keep 1.8 and 1.9 as not deprecated? > diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c > index ff8654ecda..9789e9e75c 100644 > --- a/hw/i386/pc_piix.c > +++ b/hw/i386/pc_piix.c > @@ -784,6 +784,7 @@ static void pc_i440fx_2_3_machine_options(MachineClass *m) > { > pc_i440fx_2_4_machine_options(m); > m->hw_version = "2.3.0"; > + m->deprecation_reason = "old and unattended - use a newer version instead"; > compat_props_add(m->compat_props, hw_compat_2_3, hw_compat_2_3_len); > compat_props_add(m->compat_props, pc_compat_2_3, pc_compat_2_3_len); > } > -- > 2.41.0
On 18/10/2023 15.17, Michael S. Tsirkin wrote: > On Fri, Oct 06, 2023 at 09:52:47AM +0200, Thomas Huth wrote: >> As we've seen in the past, it's useful for deprecating old machine >> types to finally be able to get of legacy code or do other clean-ups >> (see e.g. commit ea985d235b868047 that was used to drop the PCI code in >> the 128k bios binaries to free some precious space in those binaries). >> >> So let's continue deprecating the oldest pc machine types. QEMU 2.3 >> has been released 8 years ago, so that's plenty of time since such >> machine types have been used by default, thus deprecating pc-i440fx-2.0 >> up to pc-i440fx-2.3 should be fine nowadays. >> >> Signed-off-by: Thomas Huth <thuth@redhat.com> >> --- >> docs/about/deprecated.rst | 8 ++++++++ >> hw/i386/pc_piix.c | 1 + >> 2 files changed, 9 insertions(+) >> >> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst >> index e43de9c183..c016bb9b20 100644 >> --- a/docs/about/deprecated.rst >> +++ b/docs/about/deprecated.rst >> @@ -277,6 +277,14 @@ deprecated; use the new name ``dtb-randomness`` instead. The new name >> better reflects the way this property affects all random data within >> the device tree blob, not just the ``kaslr-seed`` node. >> >> +``pc-i440fx-2.0`` up to ``pc-i440fx-2.3`` (since 8.2) >> +''''''''''''''''''''''''''''''''''''''''''''''''''''' >> + >> +These old machine types are quite neglected nowadays and thus might have >> +various pitfalls with regards to live migration. Use a newer machine type >> +instead. >> + >> + >> Backend options >> --------------- > > Wait a sec. Which tree is this against? It should be against master. The patch still applies cleanly for me... could you please double-check whether your master branch is up to date? > I don't see this context ever since: > > commit f59fb1889f480b0324a7ed0404a68ad5a0ad4f6c > Author: Thomas Huth <thuth@redhat.com> > Date: Mon Jan 17 20:16:39 2022 +0100 > > hw/i386/pc_piix: Mark the machine types from version 1.4 to 1.7 as deprecated It's the context that has been left after Paolo moved the previous entries from deprecated.rst to removed-features.rst: https://gitlab.com/thuth/qemu/-/commit/ea985d235b868047cb4d8cb5657bc > Also, do we intentionally keep 1.8 and 1.9 as not deprecated? There never was a QEMU version 1.8 or 1.9, so there were never such machine types. Thomas
On Wed, Oct 18, 2023 at 03:28:16PM +0200, Thomas Huth wrote: > On 18/10/2023 15.17, Michael S. Tsirkin wrote: > > On Fri, Oct 06, 2023 at 09:52:47AM +0200, Thomas Huth wrote: > > > As we've seen in the past, it's useful for deprecating old machine > > > types to finally be able to get of legacy code or do other clean-ups > > > (see e.g. commit ea985d235b868047 that was used to drop the PCI code in > > > the 128k bios binaries to free some precious space in those binaries). > > > > > > So let's continue deprecating the oldest pc machine types. QEMU 2.3 > > > has been released 8 years ago, so that's plenty of time since such > > > machine types have been used by default, thus deprecating pc-i440fx-2.0 > > > up to pc-i440fx-2.3 should be fine nowadays. > > > > > > Signed-off-by: Thomas Huth <thuth@redhat.com> > > > --- > > > docs/about/deprecated.rst | 8 ++++++++ > > > hw/i386/pc_piix.c | 1 + > > > 2 files changed, 9 insertions(+) > > > > > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst > > > index e43de9c183..c016bb9b20 100644 > > > --- a/docs/about/deprecated.rst > > > +++ b/docs/about/deprecated.rst > > > @@ -277,6 +277,14 @@ deprecated; use the new name ``dtb-randomness`` instead. The new name > > > better reflects the way this property affects all random data within > > > the device tree blob, not just the ``kaslr-seed`` node. > > > +``pc-i440fx-2.0`` up to ``pc-i440fx-2.3`` (since 8.2) > > > +''''''''''''''''''''''''''''''''''''''''''''''''''''' > > > + > > > +These old machine types are quite neglected nowadays and thus might have > > > +various pitfalls with regards to live migration. Use a newer machine type > > > +instead. > > > + > > > + > > > Backend options > > > --------------- > > > > Wait a sec. Which tree is this against? > > It should be against master. The patch still applies cleanly for me... could > you please double-check whether your master branch is up to date? It is ... can you rebase on latest and repost? > > I don't see this context ever since: > > > > commit f59fb1889f480b0324a7ed0404a68ad5a0ad4f6c > > Author: Thomas Huth <thuth@redhat.com> > > Date: Mon Jan 17 20:16:39 2022 +0100 > > > > hw/i386/pc_piix: Mark the machine types from version 1.4 to 1.7 as deprecated > > It's the context that has been left after Paolo moved the previous entries > from deprecated.rst to removed-features.rst: > > https://gitlab.com/thuth/qemu/-/commit/ea985d235b868047cb4d8cb5657bc > > > Also, do we intentionally keep 1.8 and 1.9 as not deprecated? > > There never was a QEMU version 1.8 or 1.9, so there were never such machine > types. > > Thomas >
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst index e43de9c183..c016bb9b20 100644 --- a/docs/about/deprecated.rst +++ b/docs/about/deprecated.rst @@ -277,6 +277,14 @@ deprecated; use the new name ``dtb-randomness`` instead. The new name better reflects the way this property affects all random data within the device tree blob, not just the ``kaslr-seed`` node. +``pc-i440fx-2.0`` up to ``pc-i440fx-2.3`` (since 8.2) +''''''''''''''''''''''''''''''''''''''''''''''''''''' + +These old machine types are quite neglected nowadays and thus might have +various pitfalls with regards to live migration. Use a newer machine type +instead. + + Backend options --------------- diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c index ff8654ecda..9789e9e75c 100644 --- a/hw/i386/pc_piix.c +++ b/hw/i386/pc_piix.c @@ -784,6 +784,7 @@ static void pc_i440fx_2_3_machine_options(MachineClass *m) { pc_i440fx_2_4_machine_options(m); m->hw_version = "2.3.0"; + m->deprecation_reason = "old and unattended - use a newer version instead"; compat_props_add(m->compat_props, hw_compat_2_3, hw_compat_2_3_len); compat_props_add(m->compat_props, pc_compat_2_3, pc_compat_2_3_len); }
As we've seen in the past, it's useful for deprecating old machine types to finally be able to get of legacy code or do other clean-ups (see e.g. commit ea985d235b868047 that was used to drop the PCI code in the 128k bios binaries to free some precious space in those binaries). So let's continue deprecating the oldest pc machine types. QEMU 2.3 has been released 8 years ago, so that's plenty of time since such machine types have been used by default, thus deprecating pc-i440fx-2.0 up to pc-i440fx-2.3 should be fine nowadays. Signed-off-by: Thomas Huth <thuth@redhat.com> --- docs/about/deprecated.rst | 8 ++++++++ hw/i386/pc_piix.c | 1 + 2 files changed, 9 insertions(+)