[libvirt] [Qemu-devel] [PATCH v2 4/4] hw/core/machine: Officially deprecate the enforce-config-section parameter

Markus Armbruster armbru at redhat.com
Thu Sep 20 18:13:37 UTC 2018


Thomas Huth <thuth at redhat.com> writes:

> Commit 16f7244842b5135543ef068a1adafd94c6965953 added this parameter
> to the documentation, including a note that it is deprecated. But it
> has never been added to the "Deprecated features" appendix, which is
> our official way to deprecate legacy parameters. So let's do this now.
>
> Signed-off-by: Thomas Huth <thuth at redhat.com>
> ---
>  hw/core/machine.c    | 3 +++
>  qemu-deprecated.texi | 5 +++++
>  2 files changed, 8 insertions(+)
>
> diff --git a/hw/core/machine.c b/hw/core/machine.c
> index 6b68e12..882e7b4 100644
> --- a/hw/core/machine.c
> +++ b/hw/core/machine.c
> @@ -324,6 +324,9 @@ static void machine_set_enforce_config_section(Object *obj, bool value,
>  {
>      MachineState *ms = MACHINE(obj);
>  
> +    warn_report("enforce-config-section is deprecated. Use "

Comma, please.

> +                "-global migration.send-configuration=on|off instead");
> +
>      ms->enforce_config_section = value;
>  }
>  
> diff --git a/qemu-deprecated.texi b/qemu-deprecated.texi
> index 23fa78b..f1e807c 100644
> --- a/qemu-deprecated.texi
> +++ b/qemu-deprecated.texi
> @@ -35,6 +35,11 @@ which is the default.
>  
>  @section System emulator command line arguments
>  
> + at subsection -machine enforce-config-section=on|off (since 3.1)
> +
> +The @option{enforce-config-section} parameter is replaced by the
> + at option{-global migration.send-configuration=@var{on|off}} option.
> +
>  @subsection -no-kvm (since 1.3.0)
>  
>  The ``-no-kvm'' argument is now a synonym for setting

You keep the @item enforce-config-section=on|off in qemu-options.hx.
Keeping it until we kill the option parameter is okay, but I'd drop now,
since like to keep all the junk we don't want people to use anymore out
of the first place they look for stuff they can use.




More information about the libvir-list mailing list