On 30.05.2017 13:02, Igor Mammedov wrote:
> On Tue, 30 May 2017 12:35:01 +0200
> Gerd Hoffmann <kra...@redhat.com> wrote:
>
>> Hi,
>>
>>>> v2:
>>>> - Deprecate machines up to pc-1.2
>>>>
>>>> hw/i386/pc_piix.c | 9 +++++++++
>>>> 1 file changed, 9 insertions(+)
>>>>
>>>> diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
>>>> index 9f102aa..aace378 100644
>>>> --- a/hw/i386/pc_piix.c
>>>> +++ b/hw/i386/pc_piix.c
>>>> @@ -38,6 +38,7 @@
>>>> #include "sysemu/kvm.h"
>>>> #include "hw/kvm/clock.h"
>>>> #include "sysemu/sysemu.h"
>>>> +#include "sysemu/qtest.h"
>>>> #include "hw/sysbus.h"
>>>> #include "sysemu/arch_init.h"
>>>> #include "sysemu/block-backend.h"
>>>> @@ -84,6 +85,14 @@ static void pc_init1(MachineState *machine,
>>>> MemoryRegion *pci_memory;
>>>> MemoryRegion *rom_memory;
>>>> ram_addr_t lowmem;
>>>> + char *mc_name = MACHINE_CLASS(pcmc)->name;
>>>> +
>>>> + /* Machines pc-0.10 up to pc-1.2 are considered as deprecated
>>>> */
>>>> + if (!qtest_enabled() && (!strncmp(mc_name, "pc-0.", 5)
>>>> + || (!strncmp(mc_name, "pc-1.", 5) && mc_name[5] < '3'))) {
>>>> + error_report("Machine type '%s' is deprecated, "
>>>> + "please use a newer type instead", mc_name);
>>>> + }
>>>>
>>>> /*
>>>> * Calculate ram split, for memory below and above 4G. It's a
>>>> bit
>>>>
>>>
>>> Ping! Any comments on this version of the patch?
>>
>> Does it make sense to build some infrastructure for this, so we don't
>> have ad-hoc code to print deprecation warnings everywhere?
> we are deprecating incomplete numa mapping but it uses 'obsoleted' word
> somewhere in warning message (in machine_numa_finish_init).
>
> it could be better if if we had error_report_depricated()
> which would add prefix deprecated prefix consistently
> something like: "DEPRECATED: "
I don't mind the exact wording as long as we present a message to the
users to inform them that the interface might get removed soon... But if
you feel that we need to unify all these spots with an
error_report_deprecated() function, feel free to suggest a patch on the
list!
Thomas
