From: Peter Xu <pet...@redhat.com>

Postcopy can trigger this tracepoint twice, while only the 1st one is
valid.  Avoid triggering the 2nd tracepoint just like what we do with
recording the total downtime.

Signed-off-by: Peter Xu <pet...@redhat.com>
Tested-by: Jiri Denemark <jdene...@redhat.com>
Reviewed-by: Juraj Marcin <jmar...@redhat.com>
Link: https://lore.kernel.org/r/20250114230746.3268797-5-pet...@redhat.com
Signed-off-by: Fabiano Rosas <faro...@suse.de>
---
 migration/migration.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/migration/migration.c b/migration/migration.c
index a9fe9c2821..07b6b730b7 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -129,9 +129,8 @@ static void migration_downtime_end(MigrationState *s)
      */
     if (!s->downtime) {
         s->downtime = now - s->downtime_start;
+        trace_vmstate_downtime_checkpoint("src-downtime-end");
     }
-
-    trace_vmstate_downtime_checkpoint("src-downtime-end");
 }
 
 static bool migration_needs_multiple_sockets(void)
-- 
2.35.3


Reply via email to