[libvirt] [PATCH 1/1] dirty-bitmaps: remove deprecated autoload parameter
Vladimir Sementsov-Ogievskiy
vsementsov at virtuozzo.com
Wed Sep 25 07:20:06 UTC 2019
25.09.2019 2:01, John Snow wrote:
> This parameter has been deprecated since 2.12.0 and is eligible for
> removal. Remove this parameter as it is actually completely ignored;
> let's not give false hope.
>
> Signed-off-by: John Snow <jsnow at redhat.com>
> ---
> qemu-deprecated.texi | 20 +++++++++++++++-----
> qapi/block-core.json | 6 +-----
> blockdev.c | 6 ------
> 3 files changed, 16 insertions(+), 16 deletions(-)
>
> diff --git a/qemu-deprecated.texi b/qemu-deprecated.texi
> index 01245e0b1c..d60246d5d6 100644
> --- a/qemu-deprecated.texi
> +++ b/qemu-deprecated.texi
> @@ -149,11 +149,6 @@ QEMU 4.1 has three options, please migrate to one of these three:
>
> @section QEMU Machine Protocol (QMP) commands
>
> - at subsection block-dirty-bitmap-add "autoload" parameter (since 2.12.0)
> -
> -"autoload" parameter is now ignored. All bitmaps are automatically loaded
> -from qcow2 images.
> -
> @subsection query-block result field dirty-bitmaps[i].status (since 4.0)
>
> The ``status'' field of the ``BlockDirtyInfo'' structure, returned by
> @@ -356,3 +351,18 @@ existing CPU models. Management software that needs runnability
> guarantees must resolve the CPU model aliases using te
> ``alias-of'' field returned by the ``query-cpu-definitions'' QMP
> command.
> +
> +
> + at node Recently removed features
> + at appendix Recently removed features
> +
> +What follows is a record of recently removed, formerly deprecated
> +features that serves as a record for users who have encountered
> +trouble after a recent upgrade.
> +
> + at section QEMU Machine Protocol (QMP) commands
> +
> + at subsection block-dirty-bitmap-add "autoload" parameter (since 2.12.0)
Agree with Eric that it should be 4.2 - as this section is about removing
> +
> +"autoload" parameter is now ignored. All bitmaps are automatically loaded
> +from qcow2 images.
Maybe, rephrase it as s/is now ignored/is now removed (ignored since 2.12.0)/ ,
so that this paragraph don't mislead without a context.
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov at virtuozzo.com>
(Yay, deprecation works!)
--
Best regards,
Vladimir
More information about the libvir-list
mailing list