[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/4] hw/core/machine: Officially deprecate th
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/4] hw/core/machine: Officially deprecate the enforce-config-section parameter |
Date: |
Fri, 21 Sep 2018 10:22:21 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 2018-09-20 20:13, Markus Armbruster wrote:
> 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.
Sure, I'll change it.
>> + "-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.
The text from 16f7244842b5135543ef06 also contains the hint what should
be used instead, so I think it's ok if we keep it until the option gets
removed completely - and that's also the way we did it so far with all
other deprecated options.
Thomas
- Re: [Qemu-devel] [PATCH v2 2/4] net: Deprecate the "name" parameter of -net, (continued)