RE: [RFC PATCH 15/35] hw/i386/xen/xen-hvm: Emit warning when old code is used

[Paul Durrant <xadimgnik@xxxxxxxxx>]

> -----Original Message-----
> From: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
> Sent: 08 June 2020 17:00
> To: qemu-devel@xxxxxxxxxx
> Cc: qemu-arm@xxxxxxxxxx; Markus Armbruster <armbru@xxxxxxxxxx>; Max Filippov 
> <jcmvbkbc@xxxxxxxxx>;
> Marcel Apfelbaum <marcel.apfelbaum@xxxxxxxxx>; Peter Maydell 
> <peter.maydell@xxxxxxxxxx>; Michael Walle
> <michael@xxxxxxxx>; Edgar E. Iglesias <edgar.iglesias@xxxxxxxxx>; Aurelien 
> Jarno
> <aurelien@xxxxxxxxxxx>; Gerd Hoffmann <kraxel@xxxxxxxxxx>; Stafford Horne 
> <shorne@xxxxxxxxx>; Andrzej
> Zaborowski <balrogg@xxxxxxxxx>; qemu-ppc@xxxxxxxxxx; Alistair Francis 
> <alistair@xxxxxxxxxxxxx>;
> Richard Henderson <rth@xxxxxxxxxxx>; Mark Cave-Ayland 
> <mark.cave-ayland@xxxxxxxxxxxx>; Marc-André
> Lureau <marcandre.lureau@xxxxxxxxxx>; Daniel P . Berrange 
> <berrange@xxxxxxxxxx>; qemu-
> riscv@xxxxxxxxxx; Michael S. Tsirkin <mst@xxxxxxxxxx>; 
> xen-devel@xxxxxxxxxxxxxxxxxxxx; Sagar
> Karandikar <sagark@xxxxxxxxxxxxxxxxx>; Anthony Perard 
> <anthony.perard@xxxxxxxxxx>; Palmer Dabbelt
> <palmer@xxxxxxxxxxx>; Stefano Stabellini <sstabellini@xxxxxxxxxx>; Paul 
> Durrant <paul@xxxxxxx>; Paolo
> Bonzini <pbonzini@xxxxxxxxxx>; Alistair Francis <Alistair.Francis@xxxxxxx>; 
> Eduardo Habkost
> <ehabkost@xxxxxxxxxx>; Thomas Huth <huth@xxxxxxxxxxxxx>; Bastian Koppelmann 
> <kbastian@xxxxxxxx-
> paderborn.de>; David Gibson <david@xxxxxxxxxxxxxxxxxxxxx>; Magnus Damm 
> <magnus.damm@xxxxxxxxx>;
> Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
> Subject: [RFC PATCH 15/35] hw/i386/xen/xen-hvm: Emit warning when old code is 
> used
> 
> This code hasn't been QOM'ified yet. Warn the user.

"Based on today's IRC chat, this is a trivial RFC series
to anotate pre-qdev/QOM devices so developers using them
without knowing they are not QOM'ified yet can realize
it and convert them if they have time."

So, how should this be coded then? The XenIOState doesn't really qualify as a 
'device', does it?

  Paul

> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
> ---
>  hw/i386/xen/xen-hvm.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/hw/i386/xen/xen-hvm.c b/hw/i386/xen/xen-hvm.c
> index 82ece6b9e7..a1163b1529 100644
> --- a/hw/i386/xen/xen-hvm.c
> +++ b/hw/i386/xen/xen-hvm.c
> @@ -31,7 +31,7 @@
>  #include "sysemu/xen-mapcache.h"
>  #include "trace.h"
>  #include "exec/address-spaces.h"
> -
> +#include "hw/qdev-deprecated.h"
>  #include <xen/hvm/ioreq.h>
>  #include <xen/hvm/e820.h>
> 
> @@ -1401,6 +1401,8 @@ void xen_hvm_init(PCMachineState *pcms, MemoryRegion 
> **ram_memory)
>      xen_pfn_t ioreq_pfn;
>      XenIOState *state;
> 
> +    qdev_warn_deprecated_function_used();
> +
>      state = g_malloc0(sizeof (XenIOState));
> 
>      state->xce_handle = xenevtchn_open(NULL, 0);
> --
> 2.21.3