Re: [PATCH 1/5] migration: Add a tracepoint for invoking migration notifiers

[Fabiano Rosas <farosas@suse.de>]

Peter Xu <peterx@redhat.com> writes:

> Signed-off-by: Peter Xu <peterx@redhat.com>
> ---
>  migration/migration.c  | 2 ++
>  migration/trace-events | 1 +
>  2 files changed, 3 insertions(+)
>
> diff --git a/migration/migration.c b/migration/migration.c
> index 1bcde301f7..47d9189aaf 100644
> --- a/migration/migration.c
> +++ b/migration/migration.c
> @@ -1706,6 +1706,8 @@ int migration_call_notifiers(MigrationState *s, MigrationEventType type,
>      GSList *elem, *next;
>      int ret;
>  
> +    trace_migration_call_notifiers(type);
> +
>      e.type = type;
>  
>      for (elem = migration_state_notifiers[mode]; elem; elem = next) {
> diff --git a/migration/trace-events b/migration/trace-events
> index bf11b62b17..f437f0a784 100644
> --- a/migration/trace-events
> +++ b/migration/trace-events
> @@ -198,6 +198,7 @@ process_incoming_migration_co_end(int ret) "ret=%d"
>  process_incoming_migration_co_postcopy_end_main(void) ""
>  postcopy_preempt_enabled(bool value) "%d"
>  migration_precopy_complete(void) ""
> +migration_call_notifiers(int type) "type=%d"
>  
>  # migration-stats
>  migration_transferred_bytes(uint64_t qemu_file, uint64_t multifd, uint64_t rdma) "qemu_file %" PRIu64 " multifd %" PRIu64 " RDMA %" PRIu64

Reviewed-by: Fabiano Rosas <farosas@suse.de>