| Message ID | 20250116064644.65670-1-thuth@redhat.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | docs/about: Change notes on x86 machine type deprecation into a general one | expand |
On Thu, Jan 16, 2025 at 07:46:44AM +0100, Thomas Huth wrote: > Date: Thu, 16 Jan 2025 07:46:44 +0100 > From: Thomas Huth <thuth@redhat.com> > Subject: [PATCH] docs/about: Change notes on x86 machine type deprecation > into a general one > > We now have a general note about versioned machine types getting > deprecated and removed at the beginning of the deprecated.rst file, > so we should also have a general note about this in removed-features.rst > (which will also apply to versioned non-x86 machine types) instead of > listing individual old machine types in the document. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > docs/about/deprecated.rst | 7 ------- > docs/about/removed-features.rst | 11 +++++------ > 2 files changed, 5 insertions(+), 13 deletions(-) > Sounds good to me, so: Reviewed-by: Zhao Liu <zhao1.liu@intel.com> Let's see if others have any feedback :-)
On 16/1/25 07:46, Thomas Huth wrote: > We now have a general note about versioned machine types getting > deprecated and removed at the beginning of the deprecated.rst file, > so we should also have a general note about this in removed-features.rst > (which will also apply to versioned non-x86 machine types) instead of > listing individual old machine types in the document. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > docs/about/deprecated.rst | 7 ------- > docs/about/removed-features.rst | 11 +++++------ > 2 files changed, 5 insertions(+), 13 deletions(-) > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst > index 4a3c302962..7b42d6eecc 100644 > --- a/docs/about/deprecated.rst > +++ b/docs/about/deprecated.rst > @@ -236,13 +236,6 @@ 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.4`` up to ``pc-i440fx-2.12`` (since 9.1) > -'''''''''''''''''''''''''''''''''''''''''''''''''''''' > - > -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. > - > PPC 405 ``ref405ep`` machine (since 9.1) > '''''''''''''''''''''''''''''''''''''''' > > diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst > index c6616ce05e..156c0c253c 100644 > --- a/docs/about/removed-features.rst > +++ b/docs/about/removed-features.rst > @@ -972,6 +972,11 @@ from Linux in 2021, and is not supported anymore by QEMU either. > System emulator machines > ------------------------ > > +Note: Versioned machine types that have been introduced in a QEMU version > +that has initially been released more than 6 years before are considered > +obsolete and will be removed without further notice in this document. > +Please use newer machine types instead. This is an improvement. I still wonder if we shouldn't list them removed machines here, to help old users. Removed in 4.0: - pc-0.10 - ... Removed in 9.0: - pc-i440fx-2.3 Remove in 10.0: - pc-i440fx-2.4 - pc-i440fx-2.5 - pc-i440fx-2.6 - pc-q35-2.4 - pc-q35-2.5 - pc-q35-2.6 - ARM virt-2.6 ... ? > + > ``s390-virtio`` (removed in 2.6) > '''''''''''''''''''''''''''''''' > > @@ -1006,12 +1011,6 @@ mips ``fulong2e`` machine alias (removed in 6.0) > > This machine has been renamed ``fuloong2e``. > > -``pc-0.10`` up to ``pc-i440fx-2.3`` (removed in 4.0 up to 9.0) > -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''' > - > -These machine types were very old and likely could not be used for live > -migration from old QEMU versions anymore. Use a newer machine type instead. > - > Raspberry Pi ``raspi2`` and ``raspi3`` machines (removed in 6.2) > '''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''' >
On Thu, Jan 16, 2025 at 09:48:50AM +0100, Philippe Mathieu-Daudé wrote: > On 16/1/25 07:46, Thomas Huth wrote: > > We now have a general note about versioned machine types getting > > deprecated and removed at the beginning of the deprecated.rst file, > > so we should also have a general note about this in removed-features.rst > > (which will also apply to versioned non-x86 machine types) instead of > > listing individual old machine types in the document. > > > > Signed-off-by: Thomas Huth <thuth@redhat.com> > > --- > > docs/about/deprecated.rst | 7 ------- > > docs/about/removed-features.rst | 11 +++++------ > > 2 files changed, 5 insertions(+), 13 deletions(-) > > > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst > > index 4a3c302962..7b42d6eecc 100644 > > --- a/docs/about/deprecated.rst > > +++ b/docs/about/deprecated.rst > > @@ -236,13 +236,6 @@ 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.4`` up to ``pc-i440fx-2.12`` (since 9.1) > > -'''''''''''''''''''''''''''''''''''''''''''''''''''''' > > - > > -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. > > - > > PPC 405 ``ref405ep`` machine (since 9.1) > > '''''''''''''''''''''''''''''''''''''''' > > diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst > > index c6616ce05e..156c0c253c 100644 > > --- a/docs/about/removed-features.rst > > +++ b/docs/about/removed-features.rst > > @@ -972,6 +972,11 @@ from Linux in 2021, and is not supported anymore by QEMU either. > > System emulator machines > > ------------------------ > > +Note: Versioned machine types that have been introduced in a QEMU version > > +that has initially been released more than 6 years before are considered > > +obsolete and will be removed without further notice in this document. > > +Please use newer machine types instead. > > This is an improvement. I still wonder if we shouldn't list them removed > machines here, to help old users. > > Removed in 4.0: > - pc-0.10 > - ... > Removed in 9.0: > - pc-i440fx-2.3 > Remove in 10.0: > - pc-i440fx-2.4 > - pc-i440fx-2.5 > - pc-i440fx-2.6 > - pc-q35-2.4 > - pc-q35-2.5 > - pc-q35-2.6 > - ARM virt-2.6 > ... > > ? A big part of the reason for the macros for machine type deprecation & then skipping machine registration, was to eliminate repeated manual work that is visible to users if forgotten. We still have the actual code deletion which is manual, but that's harmless as even if forgotten, such that the code exists, the machine type will have been entirely removed from view. As such I'm pretty reluctant to suggest we create a new place which needs manually updating when we remove machine types. I wonder if its possible to get an auto-generated version number into the documentation though ? eg, so we can have an entry in 'deprecated.rst' Versioned machine types for x86_64, i386, aarch64 <other targets here> ----------------------------------------------------------------------- In accordance with our versioned machine type deprecation policy, all machine types older than 7.0.0 have been deprecated. and in 'removed-features.rst' Versioned machine types for x86_64, i386, aarch64 <other targets here> ----------------------------------------------------------------------- In accordance with our versioned machine type removal policy, all machine types older than 4.1.0 have been removed. where both '7.0.0' and '4.1.0' are variables substituted in by the docs generator in some manner, so we never need touch them manually ? NB, this auto-deletion logic is currently hacked to only come into effect from 10.1.0 onwards, so we're 1 cycle too early currently (see commit c9fd2d9a48ee3c195cf83cc611b87b09f02f0013) With regards, Daniel
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst index 4a3c302962..7b42d6eecc 100644 --- a/docs/about/deprecated.rst +++ b/docs/about/deprecated.rst @@ -236,13 +236,6 @@ 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.4`` up to ``pc-i440fx-2.12`` (since 9.1) -'''''''''''''''''''''''''''''''''''''''''''''''''''''' - -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. - PPC 405 ``ref405ep`` machine (since 9.1) '''''''''''''''''''''''''''''''''''''''' diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst index c6616ce05e..156c0c253c 100644 --- a/docs/about/removed-features.rst +++ b/docs/about/removed-features.rst @@ -972,6 +972,11 @@ from Linux in 2021, and is not supported anymore by QEMU either. System emulator machines ------------------------ +Note: Versioned machine types that have been introduced in a QEMU version +that has initially been released more than 6 years before are considered +obsolete and will be removed without further notice in this document. +Please use newer machine types instead. + ``s390-virtio`` (removed in 2.6) '''''''''''''''''''''''''''''''' @@ -1006,12 +1011,6 @@ mips ``fulong2e`` machine alias (removed in 6.0) This machine has been renamed ``fuloong2e``. -``pc-0.10`` up to ``pc-i440fx-2.3`` (removed in 4.0 up to 9.0) -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''' - -These machine types were very old and likely could not be used for live -migration from old QEMU versions anymore. Use a newer machine type instead. - Raspberry Pi ``raspi2`` and ``raspi3`` machines (removed in 6.2) ''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
We now have a general note about versioned machine types getting deprecated and removed at the beginning of the deprecated.rst file, so we should also have a general note about this in removed-features.rst (which will also apply to versioned non-x86 machine types) instead of listing individual old machine types in the document. Signed-off-by: Thomas Huth <thuth@redhat.com> --- docs/about/deprecated.rst | 7 ------- docs/about/removed-features.rst | 11 +++++------ 2 files changed, 5 insertions(+), 13 deletions(-)