Re: [PATCH 12/21] migration: Add a trace for migration_transferred_bytes

[Cédric Le Goater]

On 5/8/23 15:09, Juan Quintela wrote:
> Signed-off-by: Juan Quintela <quintela@redhat.com>

Reviewed-by: Cédric Le Goater <clg@kaod.org>

Thanks,

C.


> ---
>   migration/migration-stats.c | 8 ++++++--
>   migration/trace-events      | 3 +++
>   2 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/migration/migration-stats.c b/migration/migration-stats.c
> index fba66c4577..46b2b0d06e 100644
> --- a/migration/migration-stats.c
> +++ b/migration/migration-stats.c
> @@ -14,6 +14,7 @@
>   #include "qemu/stats64.h"
>   #include "qemu/timer.h"
>   #include "qemu-file.h"
> +#include "trace.h"
>   #include "migration-stats.h"
>   
>   MigrationAtomicStats mig_stats;
> @@ -66,6 +67,9 @@ void migration_rate_limit_account(uint64_t len)
>   
>   uint64_t migration_transferred_bytes(QEMUFile *f)
>   {
> -    return qemu_file_transferred(f) + stat64_get(&mig_stats.multifd_bytes);
> -}
> +    uint64_t multifd = stat64_get(&mig_stats.multifd_bytes);
> +    uint64_t qemu_file = qemu_file_transferred(f);
>   
> +    trace_migration_transferred_bytes(qemu_file, multifd);
> +    return qemu_file + multifd;
> +}
> diff --git a/migration/trace-events b/migration/trace-events
> index 92161eeac5..4b6e802833 100644
> --- a/migration/trace-events
> +++ b/migration/trace-events
> @@ -186,6 +186,9 @@ process_incoming_migration_co_end(int ret, int ps) "ret=%d 
> postcopy-state=%d"
>   process_incoming_migration_co_postcopy_end_main(void) ""
>   postcopy_preempt_enabled(bool value) "%d"
>   
> +# migration-stats
> +migration_transferred_bytes(uint64_t qemu_file, uint64_t multifd) "qemu_file %" PRIu64 
> " multifd %" PRIu64
> +
>   # channel.c
>   migration_set_incoming_channel(void *ioc, const char *ioctype) "ioc=%p 
> ioctype=%s"
>   migration_set_outgoing_channel(void *ioc, const char *ioctype, const char *hostname, 
> void *err)  "ioc=%p ioctype=%s hostname=%s err=%p"