> -----Original Message-----
> From: Li Zhijian <[email protected]>
> Sent: Thursday, May 9, 2024 11:31 AM
> To: Peter Xu <[email protected]>; Fabiano Rosas <[email protected]>
> Cc: Zhang, Hailiang <[email protected]>; qemu-
> [email protected]; Zhang, Chen <[email protected]>; Li Zhijian
> <[email protected]>
> 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 <[email protected]>

Reviewed-by: Zhang Chen <[email protected]>

> ---
>  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