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-hars...@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.



Reply via email to