[PATCH v2 0/3] docs/about: some documentation clean-up/fix

[PATCH v2 0/3] docs/about: some documentation clean-up/fix

[Yanan Wang]

This series makes some clean-up and fix for docs/about, including removing
the duplicated section, unifying the subject format, and adding the missed
release record in the subject.

v1->v2:
- update the commit message, combined with Cornelia's comment
- add the missing release record in the subject (patch #3)
- v1: https://lore.kernel.org/qemu-devel/20210820015628.173532-1-wangyanan55@huawei.com/

Yanan Wang (3):
  docs/about: Remove the duplicated doc
  docs/about: Unify the subject format
  docs/about: Add the missing release record in the subject

 docs/about/deprecated.rst       | 56 ++++++++++++++++-----------------
 docs/about/removed-features.rst | 45 ++++++++++----------------
 2 files changed, 44 insertions(+), 57 deletions(-)

-- 
2.19.1

[PATCH v2 1/3] docs/about: Remove the duplicated doc

[Yanan Wang]

There are two places describing the same thing about deprecation
of invalid topologies of -smp CLI, so remove the duplicated one.

Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
---
 docs/about/removed-features.rst | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
index cbfa1a8e31..6a9c5bb484 100644
--- a/docs/about/removed-features.rst
+++ b/docs/about/removed-features.rst
@@ -647,19 +647,6 @@ as ignored. Currently, users are responsible for making sure the backing storage
 specified with ``-mem-path`` can actually provide the guest RAM configured with
 ``-m`` and QEMU fails to start up if RAM allocation is unsuccessful.
 
-``-smp`` (invalid topologies) (removed 5.2)
-'''''''''''''''''''''''''''''''''''''''''''
-
-CPU topology properties should describe whole machine topology including
-possible CPUs.
-
-However, historically it was possible to start QEMU with an incorrect topology
-where *n* <= *sockets* * *cores* * *threads* < *maxcpus*,
-which could lead to an incorrect topology enumeration by the guest.
-Support for invalid topologies is removed, the user must ensure
-topologies described with -smp include all possible cpus, i.e.
-*sockets* * *cores* * *threads* = *maxcpus*.
-
 ``-machine enforce-config-section=on|off`` (removed 5.2)
 ''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
-- 
2.19.1

[PATCH v2 2/3] docs/about: Unify the subject format

[Yanan Wang]

There is a mixture of "since/removed in X.Y" vs "since/removed in X.Y.Z"
in the subjects in deprecated.rst/removed-features.rst. It will be better
to use an unified format. It seems unlikely that we will ever deprecate
something in a stable release, and even more unlikely that we'll remove
something in one, so the short versions look like the thing we want to
standardize on.

So here we unify the subject format in deprecated.rst to "since X.Y", and
unify the subject format in removed-features.rst to "removed in X.Y".

Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
---
 docs/about/deprecated.rst       | 56 ++++++++++++++++-----------------
 docs/about/removed-features.rst | 28 ++++++++---------
 2 files changed, 42 insertions(+), 42 deletions(-)

diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 6d438f1c8d..8d4fd384a5 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -107,8 +107,8 @@ the process listing. This is replaced by the new ``password-secret``
 option which lets the password be securely provided on the command
 line using a ``secret`` object instance.
 
-``opened`` property of ``rng-*`` objects (since 6.0.0)
-''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``opened`` property of ``rng-*`` objects (since 6.0)
+''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 The only effect of specifying ``opened=on`` in the command line or QMP
 ``object-add`` is that the device is opened immediately, possibly before all
@@ -116,8 +116,8 @@ other options have been processed.  This will either have no effect (if
 ``opened`` was the last option) or cause errors.  The property is therefore
 useless and should not be specified.
 
-``loaded`` property of ``secret`` and ``secret_keyring`` objects (since 6.0.0)
-''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``loaded`` property of ``secret`` and ``secret_keyring`` objects (since 6.0)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 The only effect of specifying ``loaded=on`` in the command line or QMP
 ``object-add`` is that the secret is loaded immediately, possibly before all
@@ -142,33 +142,33 @@ should be used instead.
 QEMU Machine Protocol (QMP) commands
 ------------------------------------
 
-``blockdev-open-tray``, ``blockdev-close-tray`` argument ``device`` (since 2.8.0)
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``blockdev-open-tray``, ``blockdev-close-tray`` argument ``device`` (since 2.8)
+'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 Use argument ``id`` instead.
 
-``eject`` argument ``device`` (since 2.8.0)
-'''''''''''''''''''''''''''''''''''''''''''
+``eject`` argument ``device`` (since 2.8)
+'''''''''''''''''''''''''''''''''''''''''
 
 Use argument ``id`` instead.
 
-``blockdev-change-medium`` argument ``device`` (since 2.8.0)
-''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``blockdev-change-medium`` argument ``device`` (since 2.8)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 Use argument ``id`` instead.
 
-``block_set_io_throttle`` argument ``device`` (since 2.8.0)
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``block_set_io_throttle`` argument ``device`` (since 2.8)
+'''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 Use argument ``id`` instead.
 
-``blockdev-add`` empty string argument ``backing`` (since 2.10.0)
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``blockdev-add`` empty string argument ``backing`` (since 2.10)
+'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 Use argument value ``null`` instead.
 
-``block-commit`` arguments ``base`` and ``top`` (since 3.1.0)
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``block-commit`` arguments ``base`` and ``top`` (since 3.1)
+'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 Use arguments ``base-node`` and ``top-node`` instead.
 
@@ -191,8 +191,8 @@ from Linux upstream kernel, declare it deprecated.
 System emulator CPUS
 --------------------
 
-``Icelake-Client`` CPU Model (since 5.2.0)
-''''''''''''''''''''''''''''''''''''''''''
+``Icelake-Client`` CPU Model (since 5.2)
+''''''''''''''''''''''''''''''''''''''''
 
 ``Icelake-Client`` CPU Models are deprecated. Use ``Icelake-Server`` CPU
 Models instead.
@@ -245,8 +245,8 @@ Device options
 Emulated device options
 '''''''''''''''''''''''
 
-``-device virtio-blk,scsi=on|off`` (since 5.0.0)
-^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+``-device virtio-blk,scsi=on|off`` (since 5.0)
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
 The virtio-blk SCSI passthrough feature is a legacy VIRTIO feature.  VIRTIO 1.0
 and later do not support it because the virtio-scsi device was introduced for
@@ -258,14 +258,14 @@ alias.
 Block device options
 ''''''''''''''''''''
 
-``"backing": ""`` (since 2.12.0)
-^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+``"backing": ""`` (since 2.12)
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
 In order to prevent QEMU from automatically opening an image's backing
 chain, use ``"backing": null`` instead.
 
-``rbd`` keyvalue pair encoded filenames: ``""`` (since 3.1.0)
-^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+``rbd`` keyvalue pair encoded filenames: ``""`` (since 3.1)
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
 Options for ``rbd`` should be specified according to its runtime options,
 like other block drivers.  Legacy parsing of keyvalue pair encoded
@@ -283,8 +283,8 @@ The above, converted to the current supported format::
 linux-user mode CPUs
 --------------------
 
-``ppc64abi32`` CPUs (since 5.2.0)
-'''''''''''''''''''''''''''''''''
+``ppc64abi32`` CPUs (since 5.2)
+'''''''''''''''''''''''''''''''
 
 The ``ppc64abi32`` architecture has a number of issues which regularly
 trip up our CI testing and is suspected to be quite broken. For that
@@ -303,8 +303,8 @@ Related binaries
 Backwards compatibility
 -----------------------
 
-Runnability guarantee of CPU models (since 4.1.0)
-'''''''''''''''''''''''''''''''''''''''''''''''''
+Runnability guarantee of CPU models (since 4.1)
+'''''''''''''''''''''''''''''''''''''''''''''''
 
 Previous versions of QEMU never changed existing CPU models in
 ways that introduced additional host software or hardware
diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
index 6a9c5bb484..1c926a8bc1 100644
--- a/docs/about/removed-features.rst
+++ b/docs/about/removed-features.rst
@@ -194,8 +194,8 @@ by the ``tls-authz`` and ``sasl-authz`` options.
 The ``pretty=on|off`` switch has no effect for HMP monitors and
 its use is rejected.
 
-``-drive file=json:{...{'driver':'file'}}`` (removed 6.0)
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``-drive file=json:{...{'driver':'file'}}`` (removed in 6.0)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 The 'file' driver for drives is no longer appropriate for character or host
 devices and will only accept regular files (S_IFREG). The correct driver
@@ -272,8 +272,8 @@ for the RISC-V ``virt`` machine and ``sifive_u`` machine.
 QEMU Machine Protocol (QMP) commands
 ------------------------------------
 
-``block-dirty-bitmap-add`` "autoload" parameter (removed in 4.2.0)
-''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``block-dirty-bitmap-add`` "autoload" parameter (removed in 4.2)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 The "autoload" parameter has been ignored since 2.12.0. All bitmaps
 are automatically loaded from qcow2 images.
@@ -456,15 +456,15 @@ Nobody was using this CPU emulation in QEMU, and there were no test images
 available to make sure that the code is still working, so it has been removed
 without replacement.
 
-``lm32`` CPUs (removed in 6.1.0)
-''''''''''''''''''''''''''''''''
+``lm32`` CPUs (removed in 6.1)
+''''''''''''''''''''''''''''''
 
 The only public user of this architecture was the milkymist project,
 which has been dead for years; there was never an upstream Linux
 port.  Removed without replacement.
 
-``unicore32`` CPUs (since 6.1.0)
-''''''''''''''''''''''''''''''''
+``unicore32`` CPUs (removed in 6.1)
+'''''''''''''''''''''''''''''''''''
 
 Support for this CPU was removed from the upstream Linux kernel, and
 there is no available upstream toolchain to build binaries for it.
@@ -593,8 +593,8 @@ error when ``-u`` is not used.
 Command line options
 --------------------
 
-``-smp`` (invalid topologies) (removed 5.2)
-'''''''''''''''''''''''''''''''''''''''''''
+``-smp`` (invalid topologies) (removed in 5.2)
+''''''''''''''''''''''''''''''''''''''''''''''
 
 CPU topology properties should describe whole machine topology including
 possible CPUs.
@@ -606,8 +606,8 @@ Support for invalid topologies is removed, the user must ensure
 topologies described with -smp include all possible cpus, i.e.
 *sockets* * *cores* * *threads* = *maxcpus*.
 
-``-numa`` node (without memory specified) (removed 5.2)
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``-numa`` node (without memory specified) (removed in 5.2)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 Splitting RAM by default between NUMA nodes had the same issues as ``mem``
 parameter with the difference that the role of the user plays QEMU using
@@ -647,8 +647,8 @@ as ignored. Currently, users are responsible for making sure the backing storage
 specified with ``-mem-path`` can actually provide the guest RAM configured with
 ``-m`` and QEMU fails to start up if RAM allocation is unsuccessful.
 
-``-machine enforce-config-section=on|off`` (removed 5.2)
-''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``-machine enforce-config-section=on|off`` (removed in 5.2)
+'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 The ``enforce-config-section`` property was replaced by the
 ``-global migration.send-configuration={on|off}`` option.
-- 
2.19.1

[PATCH v2 3/3] docs/about: Add the missing release record in the subject

[Yanan Wang]

Commit 29e0447551
(docs/about/removed-features: Document removed CLI options from QEMU v3.1)
has recorded some CLI options as replaced/removed from QEMU v3.1, but one
of the subjects has missed the release record. Let's fix it.

Reported-by: Cornelia Huck <cohuck@redhat.com>
Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
---
 docs/about/removed-features.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
index 1c926a8bc1..8feeead449 100644
--- a/docs/about/removed-features.rst
+++ b/docs/about/removed-features.rst
@@ -140,8 +140,8 @@ Use ``-rtc driftfix=slew`` instead.
 
 Replaced by ``-rtc base=date``.
 
-``-vnc ...,tls=...``, ``-vnc ...,x509=...`` & ``-vnc ...,x509verify=...``
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+``-vnc ...,tls=...``, ``-vnc ...,x509=...`` & ``-vnc ...,x509verify=...`` (removed in 3.1)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
 
 The "tls-creds" option should be used instead to point to a "tls-creds-x509"
 object created using "-object".
-- 
2.19.1

Re: [PATCH v2 1/3] docs/about: Remove the duplicated doc

[Thomas Huth]

On 23/08/2021 05.00, Yanan Wang wrote:
> There are two places describing the same thing about deprecation
> of invalid topologies of -smp CLI, so remove the duplicated one.
> 
> Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> ---
>   docs/about/removed-features.rst | 13 -------------
>   1 file changed, 13 deletions(-)
> 
> diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
> index cbfa1a8e31..6a9c5bb484 100644
> --- a/docs/about/removed-features.rst
> +++ b/docs/about/removed-features.rst
> @@ -647,19 +647,6 @@ as ignored. Currently, users are responsible for making sure the backing storage
>   specified with ``-mem-path`` can actually provide the guest RAM configured with
>   ``-m`` and QEMU fails to start up if RAM allocation is unsuccessful.
>   
> -``-smp`` (invalid topologies) (removed 5.2)
> -'''''''''''''''''''''''''''''''''''''''''''
> -
> -CPU topology properties should describe whole machine topology including
> -possible CPUs.
> -
> -However, historically it was possible to start QEMU with an incorrect topology
> -where *n* <= *sockets* * *cores* * *threads* < *maxcpus*,
> -which could lead to an incorrect topology enumeration by the guest.
> -Support for invalid topologies is removed, the user must ensure
> -topologies described with -smp include all possible cpus, i.e.
> -*sockets* * *cores* * *threads* = *maxcpus*.
> -
>   ``-machine enforce-config-section=on|off`` (removed 5.2)
>   ''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>

Re: [PATCH v2 2/3] docs/about: Unify the subject format

[Thomas Huth]

On 23/08/2021 05.00, Yanan Wang wrote:
> There is a mixture of "since/removed in X.Y" vs "since/removed in X.Y.Z"
> in the subjects in deprecated.rst/removed-features.rst. It will be better
> to use an unified format. It seems unlikely that we will ever deprecate
> something in a stable release, and even more unlikely that we'll remove
> something in one, so the short versions look like the thing we want to
> standardize on.
> 
> So here we unify the subject format in deprecated.rst to "since X.Y", and
> unify the subject format in removed-features.rst to "removed in X.Y".
> 
> Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> ---
>   docs/about/deprecated.rst       | 56 ++++++++++++++++-----------------
>   docs/about/removed-features.rst | 28 ++++++++---------
>   2 files changed, 42 insertions(+), 42 deletions(-)
> 
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 6d438f1c8d..8d4fd384a5 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -107,8 +107,8 @@ the process listing. This is replaced by the new ``password-secret``
>   option which lets the password be securely provided on the command
>   line using a ``secret`` object instance.
>   
> -``opened`` property of ``rng-*`` objects (since 6.0.0)
> -''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``opened`` property of ``rng-*`` objects (since 6.0)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   The only effect of specifying ``opened=on`` in the command line or QMP
>   ``object-add`` is that the device is opened immediately, possibly before all
> @@ -116,8 +116,8 @@ other options have been processed.  This will either have no effect (if
>   ``opened`` was the last option) or cause errors.  The property is therefore
>   useless and should not be specified.
>   
> -``loaded`` property of ``secret`` and ``secret_keyring`` objects (since 6.0.0)
> -''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``loaded`` property of ``secret`` and ``secret_keyring`` objects (since 6.0)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   The only effect of specifying ``loaded=on`` in the command line or QMP
>   ``object-add`` is that the secret is loaded immediately, possibly before all
> @@ -142,33 +142,33 @@ should be used instead.
>   QEMU Machine Protocol (QMP) commands
>   ------------------------------------
>   
> -``blockdev-open-tray``, ``blockdev-close-tray`` argument ``device`` (since 2.8.0)
> -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``blockdev-open-tray``, ``blockdev-close-tray`` argument ``device`` (since 2.8)
> +'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   Use argument ``id`` instead.
>   
> -``eject`` argument ``device`` (since 2.8.0)
> -'''''''''''''''''''''''''''''''''''''''''''
> +``eject`` argument ``device`` (since 2.8)
> +'''''''''''''''''''''''''''''''''''''''''
>   
>   Use argument ``id`` instead.
>   
> -``blockdev-change-medium`` argument ``device`` (since 2.8.0)
> -''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``blockdev-change-medium`` argument ``device`` (since 2.8)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   Use argument ``id`` instead.
>   
> -``block_set_io_throttle`` argument ``device`` (since 2.8.0)
> -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``block_set_io_throttle`` argument ``device`` (since 2.8)
> +'''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   Use argument ``id`` instead.
>   
> -``blockdev-add`` empty string argument ``backing`` (since 2.10.0)
> -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``blockdev-add`` empty string argument ``backing`` (since 2.10)
> +'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   Use argument value ``null`` instead.
>   
> -``block-commit`` arguments ``base`` and ``top`` (since 3.1.0)
> -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``block-commit`` arguments ``base`` and ``top`` (since 3.1)
> +'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   Use arguments ``base-node`` and ``top-node`` instead.
>   
> @@ -191,8 +191,8 @@ from Linux upstream kernel, declare it deprecated.
>   System emulator CPUS
>   --------------------
>   
> -``Icelake-Client`` CPU Model (since 5.2.0)
> -''''''''''''''''''''''''''''''''''''''''''
> +``Icelake-Client`` CPU Model (since 5.2)
> +''''''''''''''''''''''''''''''''''''''''
>   
>   ``Icelake-Client`` CPU Models are deprecated. Use ``Icelake-Server`` CPU
>   Models instead.
> @@ -245,8 +245,8 @@ Device options
>   Emulated device options
>   '''''''''''''''''''''''
>   
> -``-device virtio-blk,scsi=on|off`` (since 5.0.0)
> -^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> +``-device virtio-blk,scsi=on|off`` (since 5.0)
> +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>   
>   The virtio-blk SCSI passthrough feature is a legacy VIRTIO feature.  VIRTIO 1.0
>   and later do not support it because the virtio-scsi device was introduced for
> @@ -258,14 +258,14 @@ alias.
>   Block device options
>   ''''''''''''''''''''
>   
> -``"backing": ""`` (since 2.12.0)
> -^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> +``"backing": ""`` (since 2.12)
> +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>   
>   In order to prevent QEMU from automatically opening an image's backing
>   chain, use ``"backing": null`` instead.
>   
> -``rbd`` keyvalue pair encoded filenames: ``""`` (since 3.1.0)
> -^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> +``rbd`` keyvalue pair encoded filenames: ``""`` (since 3.1)
> +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>   
>   Options for ``rbd`` should be specified according to its runtime options,
>   like other block drivers.  Legacy parsing of keyvalue pair encoded
> @@ -283,8 +283,8 @@ The above, converted to the current supported format::
>   linux-user mode CPUs
>   --------------------
>   
> -``ppc64abi32`` CPUs (since 5.2.0)
> -'''''''''''''''''''''''''''''''''
> +``ppc64abi32`` CPUs (since 5.2)
> +'''''''''''''''''''''''''''''''
>   
>   The ``ppc64abi32`` architecture has a number of issues which regularly
>   trip up our CI testing and is suspected to be quite broken. For that
> @@ -303,8 +303,8 @@ Related binaries
>   Backwards compatibility
>   -----------------------
>   
> -Runnability guarantee of CPU models (since 4.1.0)
> -'''''''''''''''''''''''''''''''''''''''''''''''''
> +Runnability guarantee of CPU models (since 4.1)
> +'''''''''''''''''''''''''''''''''''''''''''''''
>   
>   Previous versions of QEMU never changed existing CPU models in
>   ways that introduced additional host software or hardware
> diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
> index 6a9c5bb484..1c926a8bc1 100644
> --- a/docs/about/removed-features.rst
> +++ b/docs/about/removed-features.rst
> @@ -194,8 +194,8 @@ by the ``tls-authz`` and ``sasl-authz`` options.
>   The ``pretty=on|off`` switch has no effect for HMP monitors and
>   its use is rejected.
>   
> -``-drive file=json:{...{'driver':'file'}}`` (removed 6.0)
> -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``-drive file=json:{...{'driver':'file'}}`` (removed in 6.0)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   The 'file' driver for drives is no longer appropriate for character or host
>   devices and will only accept regular files (S_IFREG). The correct driver
> @@ -272,8 +272,8 @@ for the RISC-V ``virt`` machine and ``sifive_u`` machine.
>   QEMU Machine Protocol (QMP) commands
>   ------------------------------------
>   
> -``block-dirty-bitmap-add`` "autoload" parameter (removed in 4.2.0)
> -''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``block-dirty-bitmap-add`` "autoload" parameter (removed in 4.2)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   The "autoload" parameter has been ignored since 2.12.0. All bitmaps
>   are automatically loaded from qcow2 images.
> @@ -456,15 +456,15 @@ Nobody was using this CPU emulation in QEMU, and there were no test images
>   available to make sure that the code is still working, so it has been removed
>   without replacement.
>   
> -``lm32`` CPUs (removed in 6.1.0)
> -''''''''''''''''''''''''''''''''
> +``lm32`` CPUs (removed in 6.1)
> +''''''''''''''''''''''''''''''
>   
>   The only public user of this architecture was the milkymist project,
>   which has been dead for years; there was never an upstream Linux
>   port.  Removed without replacement.
>   
> -``unicore32`` CPUs (since 6.1.0)
> -''''''''''''''''''''''''''''''''
> +``unicore32`` CPUs (removed in 6.1)
> +'''''''''''''''''''''''''''''''''''
>   
>   Support for this CPU was removed from the upstream Linux kernel, and
>   there is no available upstream toolchain to build binaries for it.
> @@ -593,8 +593,8 @@ error when ``-u`` is not used.
>   Command line options
>   --------------------
>   
> -``-smp`` (invalid topologies) (removed 5.2)
> -'''''''''''''''''''''''''''''''''''''''''''
> +``-smp`` (invalid topologies) (removed in 5.2)
> +''''''''''''''''''''''''''''''''''''''''''''''
>   
>   CPU topology properties should describe whole machine topology including
>   possible CPUs.
> @@ -606,8 +606,8 @@ Support for invalid topologies is removed, the user must ensure
>   topologies described with -smp include all possible cpus, i.e.
>   *sockets* * *cores* * *threads* = *maxcpus*.
>   
> -``-numa`` node (without memory specified) (removed 5.2)
> -'''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``-numa`` node (without memory specified) (removed in 5.2)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   Splitting RAM by default between NUMA nodes had the same issues as ``mem``
>   parameter with the difference that the role of the user plays QEMU using
> @@ -647,8 +647,8 @@ as ignored. Currently, users are responsible for making sure the backing storage
>   specified with ``-mem-path`` can actually provide the guest RAM configured with
>   ``-m`` and QEMU fails to start up if RAM allocation is unsuccessful.
>   
> -``-machine enforce-config-section=on|off`` (removed 5.2)
> -''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``-machine enforce-config-section=on|off`` (removed in 5.2)
> +'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>   
>   The ``enforce-config-section`` property was replaced by the
>   ``-global migration.send-configuration={on|off}`` option.
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>

Re: [PATCH v2 3/3] docs/about: Add the missing release record in the subject

[Thomas Huth]

On 23/08/2021 05.00, Yanan Wang wrote:
> Commit 29e0447551
> (docs/about/removed-features: Document removed CLI options from QEMU v3.1)
> has recorded some CLI options as replaced/removed from QEMU v3.1, but one
> of the subjects has missed the release record. Let's fix it.
> 
> Reported-by: Cornelia Huck <cohuck@redhat.com>
> Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
> ---
>   docs/about/removed-features.rst | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
> index 1c926a8bc1..8feeead449 100644
> --- a/docs/about/removed-features.rst
> +++ b/docs/about/removed-features.rst
> @@ -140,8 +140,8 @@ Use ``-rtc driftfix=slew`` instead.
>   
>   Replaced by ``-rtc base=date``.
>   
> -``-vnc ...,tls=...``, ``-vnc ...,x509=...`` & ``-vnc ...,x509verify=...``
> -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``-vnc ...,tls=...``, ``-vnc ...,x509=...`` & ``-vnc ...,x509verify=...`` (removed in 3.1)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''

Reviewed-by: Thomas Huth <thuth@redhat.com>

Re: [PATCH v2 1/3] docs/about: Remove the duplicated doc

[Andrew Jones]

On Mon, Aug 23, 2021 at 11:00:03AM +0800, Yanan Wang wrote:
> There are two places describing the same thing about deprecation
> of invalid topologies of -smp CLI, so remove the duplicated one.
> 
> Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> ---
>  docs/about/removed-features.rst | 13 -------------
>  1 file changed, 13 deletions(-)
> 
> diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
> index cbfa1a8e31..6a9c5bb484 100644
> --- a/docs/about/removed-features.rst
> +++ b/docs/about/removed-features.rst
> @@ -647,19 +647,6 @@ as ignored. Currently, users are responsible for making sure the backing storage
>  specified with ``-mem-path`` can actually provide the guest RAM configured with
>  ``-m`` and QEMU fails to start up if RAM allocation is unsuccessful.
>  
> -``-smp`` (invalid topologies) (removed 5.2)
> -'''''''''''''''''''''''''''''''''''''''''''
> -
> -CPU topology properties should describe whole machine topology including
> -possible CPUs.
> -
> -However, historically it was possible to start QEMU with an incorrect topology
> -where *n* <= *sockets* * *cores* * *threads* < *maxcpus*,
> -which could lead to an incorrect topology enumeration by the guest.
> -Support for invalid topologies is removed, the user must ensure
> -topologies described with -smp include all possible cpus, i.e.
> -*sockets* * *cores* * *threads* = *maxcpus*.
> -
>  ``-machine enforce-config-section=on|off`` (removed 5.2)
>  ''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>  
> -- 
> 2.19.1
>

 
Reviewed-by: Andrew Jones <drjones@redhat.com>

Re: [PATCH v2 2/3] docs/about: Unify the subject format

[Andrew Jones]

On Mon, Aug 23, 2021 at 11:00:04AM +0800, Yanan Wang wrote:
> There is a mixture of "since/removed in X.Y" vs "since/removed in X.Y.Z"
> in the subjects in deprecated.rst/removed-features.rst. It will be better
> to use an unified format. It seems unlikely that we will ever deprecate
> something in a stable release, and even more unlikely that we'll remove
> something in one, so the short versions look like the thing we want to
> standardize on.
> 
> So here we unify the subject format in deprecated.rst to "since X.Y", and
> unify the subject format in removed-features.rst to "removed in X.Y".
> 
> Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> ---
>  docs/about/deprecated.rst       | 56 ++++++++++++++++-----------------
>  docs/about/removed-features.rst | 28 ++++++++---------
>  2 files changed, 42 insertions(+), 42 deletions(-)
>

Reviewed-by: Andrew Jones <drjones@redhat.com>

Re: [PATCH v2 3/3] docs/about: Add the missing release record in the subject

[Andrew Jones]

On Mon, Aug 23, 2021 at 11:00:05AM +0800, Yanan Wang wrote:
> Commit 29e0447551
> (docs/about/removed-features: Document removed CLI options from QEMU v3.1)
> has recorded some CLI options as replaced/removed from QEMU v3.1, but one
> of the subjects has missed the release record. Let's fix it.
> 
> Reported-by: Cornelia Huck <cohuck@redhat.com>
> Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
> ---
>  docs/about/removed-features.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
> index 1c926a8bc1..8feeead449 100644
> --- a/docs/about/removed-features.rst
> +++ b/docs/about/removed-features.rst
> @@ -140,8 +140,8 @@ Use ``-rtc driftfix=slew`` instead.
>  
>  Replaced by ``-rtc base=date``.
>  
> -``-vnc ...,tls=...``, ``-vnc ...,x509=...`` & ``-vnc ...,x509verify=...``
> -'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +``-vnc ...,tls=...``, ``-vnc ...,x509=...`` & ``-vnc ...,x509verify=...`` (removed in 3.1)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>  
>  The "tls-creds" option should be used instead to point to a "tls-creds-x509"
>  object created using "-object".
> -- 
> 2.19.1
>

Reviewed-by: Andrew Jones <drjones@redhat.com>