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

[Markus Armbruster]

Thomas Huth <address@hidden> 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 <address@hidden>
> ---
>  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
>  
> address@hidden -machine enforce-config-section=on|off (since 3.1)
> +
> +The @option{enforce-config-section} parameter is replaced by the
> address@hidden address@hidden|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.