Re: [PATCH v2 00/11] ppc/spapr: remove deprecated machines specific code

["Cédric Le Goater" <clg@kaod.org>]

Hello Harsh

On 10/3/24 07:16, Harsh Prateek Bora wrote:
> Hi Cedric,
> 
> On 10/2/24 12:26, Cédric Le Goater wrote:
>> Hello Harsh,
>>
>> On 10/1/24 11:28, Harsh Prateek Bora wrote:
>>> Commit 1392617d3576 intended to tag pseries-2.1 - 2.11 machines as
>>> deprecated with reasons mentioned in its commit log.
>>> Removing the arch specific code for the now deprecated machine types.
>>>
>>> v2: Addressed review comments from Cedric
>>> v1: <20240917060300.943496-1-harshpb@linux.ibm.com>
>>> Harsh Prateek Bora (11):
>>>    ppc/spapr: remove deprecated machine pseries-2.1
>>>    ppc/spapr: remove deprecated machine pseries-2.2
>>>    ppc/spapr: remove deprecated machine pseries-2.3
>>>    ppc/spapr: remove deprecated machine pseries-2.4
>>>    ppc/spapr: remove deprecated machine pseries-2.5
>>>    ppc/spapr: remove deprecated machine pseries-2.6
>>>    ppc/spapr: remove deprecated machine pseries-2.7
>>>    ppc/spapr: remove deprecated machine pseries-2.8
>>>    ppc/spapr: remove deprecated machine pseries-2.9
>>>    ppc/spapr: remove deprecated machine pseries-2.10
>>>    ppc/spapr: remove deprecated machine pseries-2.11
>>
>> LGTM, I had a few questions.
>>
> 
> Thanks, I shall address the review comments in next iteration.

While at it, could you please also remove the pseries paragraph in
the 'docs/about/deprecated.rst' file  ?

>> Please be aware that the next QEMU cycle will have more machines
>> to deprecate.
>>
> 
> I realized that 2.12 had also been marked as deprecated earlier which I
> can include in v3, however would like to understand the rationale behind
> choosing the remaining until pseries-6.1 for deprecating in next release
> cycle. 

pseries-6.1 is not for this release.

> Please enlighten.

It's in the docs :

   https://www.qemu.org/docs/master/about/deprecated.html
   
   As a special exception to this general timeframe, rather than have an
   indefinite lifetime, versioned machine types are only intended to be
   supported for a period of 6 years, equivalent to 18 QEMU releases. All
   versioned machine types will be automatically marked deprecated after
   an initial 3 years (9 QEMU releases) has passed, and will then be
   deleted after a further 3 year period has passed. It is recommended
   that a deprecated machine type is only used for incoming migrations
   and restore of saved state, for pre-existing VM deployments. They
   should be scheduled for updating to a newer machine type during an
   appropriate service window. Newly deployed VMs should exclusively use
   a non-deprecated machine type, with use of the most recent version
   highly recommended. Non-versioned machine types follow the general
   feature deprecation policy.

So, since the tags date are :

   refs/tags/v2.0.0 Thu Apr 17 13:41:45 2014 +0100
   refs/tags/v2.1.0 Fri Aug 1 13:31:29 2014 +0100
   refs/tags/v2.2.0 Tue Dec 9 12:13:37 2014 +0000
   refs/tags/v2.3.0 Fri Apr 24 15:05:06 2015 +0100
   refs/tags/v2.4.0 Tue Aug 11 15:30:34 2015 +0100
   refs/tags/v2.5.0 Wed Dec 16 16:10:14 2015 +0000
   refs/tags/v2.6.0 Wed May 11 16:44:26 2016 +0100
   refs/tags/v2.7.0 Fri Sep 2 13:44:11 2016 +0100
   refs/tags/v2.8.0 Tue Dec 20 16:17:01 2016 +0000
   refs/tags/v2.9.0 Thu Apr 20 15:31:34 2017 +0100
   refs/tags/v2.10.0 Wed Aug 30 17:02:54 2017 +0100
   refs/tags/v2.11.0 Wed Dec 13 14:31:09 2017 +0000
   refs/tags/v2.12.0 Tue Apr 24 16:44:55 2018 +0100
   refs/tags/v3.0.0 Tue Aug 14 16:38:43 2018 +0100
                                                    <--- ~6y
   refs/tags/v3.1.0 Tue Dec 11 17:18:37 2018 +0000
   refs/tags/v4.0.0 Tue Apr 23 17:32:17 2019 +0100
   refs/tags/v4.1.0 Thu Aug 15 13:03:37 2019 +0100
   refs/tags/v4.2.0 Thu Dec 12 16:45:57 2019 +0000
   refs/tags/v5.0.0 Tue Apr 28 17:46:57 2020 +0100
   refs/tags/v5.1.0 Tue Aug 11 17:07:03 2020 +0100
   refs/tags/v5.2.0 Tue Dec 8 15:55:19 2020 +0000
   refs/tags/v6.0.0 Thu Apr 29 18:05:29 2021 +0100
   refs/tags/v6.1.0 Tue Aug 24 17:59:52 2021 +0100
   refs/tags/v6.2.0 Tue Dec 14 12:35:02 2021 -0800
   refs/tags/v7.0.0 Tue Apr 19 18:44:36 2022 +0100
   refs/tags/v7.1.0 Tue Aug 30 09:40:11 2022 -0700
   refs/tags/v7.2.0 Tue Dec 13 15:54:59 2022 -0500
   refs/tags/v8.0.0 Wed Apr 19 17:27:13 2023 +0100
   refs/tags/v8.1.0 Tue Aug 22 07:13:44 2023 -0700
   refs/tags/v8.2.0 Tue Dec 19 09:45:04 2023 -0500
   refs/tags/v9.0.0 Tue Apr 23 19:14:06 2024 +0100
   refs/tags/v9.1.0 Tue Sep 3 09:18:26 2024 -0700
   
We can remove up to v3.0.0. v3.1.0 could be a candidate too,
I will check.

Thanks,

C.