Re: [Xen-devel] [PATCH V3] tools/libxl: Add iothread support for COLO

[Anthony PERARD <anthony.perard@xxxxxxxxxx>]

On Fri, Jul 26, 2019 at 11:43:40PM +0800, Zhang Chen wrote:
> From: Zhang Chen <chen.zhang@xxxxxxxxx>
> 
> Xen COLO and KVM COLO shared lots of code in Qemu.
> The colo-compare object in Qemu now requires an 'iothread' property since 
> QEMU 2.11.
> 
> Detail:
> https://wiki.qemu.org/Features/COLO
> 
> Signed-off-by: Zhang Chen <chen.zhang@xxxxxxxxx>
> ---
>  tools/libxl/libxl_dm.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/libxl/libxl_dm.c b/tools/libxl/libxl_dm.c
> index f4fc96415d..4b029d6c8e 100644
> --- a/tools/libxl/libxl_dm.c
> +++ b/tools/libxl/libxl_dm.c
> @@ -1635,7 +1635,10 @@ static int 
> libxl__build_device_model_args_new(libxl__gc *gc,
>                          nics[i].colo_compare_notify_dev) {
>                          flexarray_append(dm_args, "-object");
>                          flexarray_append(dm_args,
> -                           
> GCSPRINTF("colo-compare,id=c1,primary_in=%s,secondary_in=%s,outdev=%s,notify_dev=%s",
> +                           GCSPRINTF("iothread,id=colo-compare-iothread-1"));

You don't need GCSPRINTF here anymore ;-), in fact, it doesn't build with
it. With that fix:
Reviewed-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

Thanks,

-- 
Anthony PERARD

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel