> -----Original Message-----
> From: Li Zhijian <lizhij...@fujitsu.com>
> Sent: Thursday, May 9, 2024 11:31 AM
> To: Peter Xu <pet...@redhat.com>; Fabiano Rosas <faro...@suse.de>
> Cc: Zhang, Hailiang <zhanghaili...@xfusion.com>; qemu-
> de...@nongnu.org; Zhang, Chen <chen.zh...@intel.com>; Li Zhijian
> <lizhij...@fujitsu.com>
> Subject: [PATCH 1/3] migration/colo: Minor fix for colo error message
> 
> - Explicitly show the missing module name: replication
> - Fix capability name to x-colo
> 
> Signed-off-by: Li Zhijian <lizhij...@fujitsu.com>

Reviewed-by: Zhang Chen <chen.zh...@intel.com>

> ---
>  migration/migration.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/migration/migration.c b/migration/migration.c index
> 6502e169a3..b4a09c561c 100644
> --- a/migration/migration.c
> +++ b/migration/migration.c
> @@ -517,13 +517,13 @@ void migration_incoming_disable_colo(void)
>  int migration_incoming_enable_colo(void)
>  {
>  #ifndef CONFIG_REPLICATION
> -    error_report("ENABLE_COLO command come in migration stream, but
> COLO "
> -                 "module is not built in");
> +    error_report("ENABLE_COLO command come in migration stream, but the
> "
> +                 "replication module is not built in");
>      return -ENOTSUP;
>  #endif
> 
>      if (!migrate_colo()) {
> -        error_report("ENABLE_COLO command come in migration stream, but c-
> colo "
> +        error_report("ENABLE_COLO command come in migration stream, but
> x-colo "
>                       "capability is not set");
>          return -EINVAL;
>      }
> --
> 2.31.1


Reply via email to