Re: [PATCH v2] spapr: Tag pseries-2.1 - 2.11 machines as deprecated

[Cédric Le Goater]

On 3/11/24 18:46, Philippe Mathieu-Daudé wrote:
> Hi,
>
> On 14/12/23 19:17, Cédric Le Goater wrote:
> > pseries machines before version 2.11 have undergone many changes to
> > correct issues, mostly regarding migration compatibility. This is
> > obfuscating the code uselessly and makes maintenance more difficult.
> > Remove them and only keep the last version of the 2.x series, 2.12,
> > still in use by old distros.
>
> By the time we get to QEMU v9.2, will pseries-2.12 still be used
> by old distros? (which ones btw?)

Using info from https://repology.org/api/v1/project/qemu :

alpine_3_8: 2.12.0
aur: 2.10.0, 2.7.1, 8.2.0.r1767.g91e3bf2e92
epel_7: 2.0.0
fedora_26: 2.9.0, 2.9.1
fedora_27: 2.10.0, 2.10.2
fedora_28: 2.11.1, 2.11.2
freshcode: 2.8.0
gnuguix: 2.10.2, 8.1.3
opensuse_leap_15_0: 2.11.1, 2.11.2, 3.1.1.1, 4.2.1, 5.2.0
opensuse_leap_42_3: 2.9.0, 2.9.1
openwrt_17_01_x86_64: 2.6.2
openwrt_18_06_x86_64: 2.11.1
rosa_2016_1: 2.10.1, 2.5.1.1
slackbuilds: 2.12.0, 7.2.1
slitaz_next: 2.0.2
ubuntu_14_04: 2.0.0, 2.0.0~rc1
ubuntu_16_04: 2.5
ubuntu_18_04: 2.11

Thanks to Thomas !

C.


> What about also deprecating pseries-2.12 and pseries-2.12-sxxm
> which both set pre_3_0_migration, and suggest to use pseries-3.0
> instead?
>
> > Reviewed-by: Thomas Huth <thuth@redhat.com>
> > Signed-off-by: Cédric Le Goater <clg@kaod.org>
> > ---
> >   docs/about/deprecated.rst | 7 +++++++
> >   hw/ppc/spapr.c            | 1 +
> >   2 files changed, 8 insertions(+)
> >
> > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > index 2e1504024658..c0e22b2b568f 100644
> > --- a/docs/about/deprecated.rst
> > +++ b/docs/about/deprecated.rst
> > @@ -269,6 +269,13 @@ Nios II ``10m50-ghrd`` and ``nios2-generic-nommu`` 
> > machines (since 8.2)
> >   The Nios II architecture is orphan.
> > +``pseries-2.1`` up to ``pseries-2.11`` (since 9.0)
> > +''''''''''''''''''''''''''''''''''''''''''''''''''
> > +
> > +Older pseries machines before version 2.12 have undergone many changes
> > +to correct issues, mostly regarding migration compatibility. These are
> > +no longer maintained and removing them will make the code easier to
> > +read and maintain. Use versions 2.12 and above as a replacement.