migration: Remove transferred atomic counter

Juan Quintela · Juan Quintela · commit 0f8596180a30 · 2023-10-31T08:44:33.000+01:00
After last commit, it is a write only variable.

Reviewed-by: Fabiano Rosas &lt;farosas@suse.de&gt;
Signed-off-by: Juan Quintela &lt;quintela@redhat.com&gt;
Message-ID: &lt;20231025091117.6342-12-quintela@redhat.com&gt;
diff --git a/migration/migration-stats.h b/migration/migration-stats.h
@@ -97,10 +97,6 @@ typedef struct {
      * Number of bytes sent through RDMA.
      */
     Stat64 rdma_bytes;
-    /*
-     * Total number of bytes transferred.
-     */
-    Stat64 transferred;
     /*
      * Number of pages transferred that were full of zeros.
      */
diff --git a/migration/multifd.c b/migration/multifd.c
@@ -188,7 +188,6 @@ static int multifd_send_initial_packet(MultiFDSendParams *p, Error **errp)
         return -1;
     }
     stat64_add(&mig_stats.multifd_bytes, size);
-    stat64_add(&mig_stats.transferred, size);
     return 0;
 }
 
@@ -733,8 +732,6 @@ static void *multifd_send_thread(void *opaque)
 
             stat64_add(&mig_stats.multifd_bytes,
                        p->next_packet_size + p->packet_len);
-            stat64_add(&mig_stats.transferred,
-                       p->next_packet_size + p->packet_len);
             p->next_packet_size = 0;
             qemu_mutex_lock(&p->mutex);
             p->pending_job--;
diff --git a/migration/ram.c b/migration/ram.c
@@ -448,7 +448,6 @@ void ram_transferred_add(uint64_t bytes)
     } else {
         stat64_add(&mig_stats.downtime_bytes, bytes);
     }
-    stat64_add(&mig_stats.transferred, bytes);
 }
 
 struct MigrationOps {

Original file line number	Diff line number	Diff line change
`@@ -188,7 +188,6 @@ static int multifd_send_initial_packet(MultiFDSendParams p, Error *errp)`
`188`	`188`	`return -1;`
`189`	`189`	`}`
`190`	`190`	`stat64_add(&mig_stats.multifd_bytes, size);`
`191`		`- stat64_add(&mig_stats.transferred, size);`
`192`	`191`	`return 0;`
`193`	`192`	`}`
`194`	`193`
`@@ -733,8 +732,6 @@ static void multifd_send_thread(void opaque)`
`733`	`732`
`734`	`733`	`stat64_add(&mig_stats.multifd_bytes,`
`735`	`734`	`p->next_packet_size + p->packet_len);`
`736`		`- stat64_add(&mig_stats.transferred,`
`737`		`- p->next_packet_size + p->packet_len);`
`738`	`735`	`p->next_packet_size = 0;`
`739`	`736`	`qemu_mutex_lock(&p->mutex);`
`740`	`737`	`p->pending_job--;`
Original file line number	Diff line number	Diff line change
`@@ -448,7 +448,6 @@ void ram_transferred_add(uint64_t bytes)`
`448`	`448`	`} else {`
`449`	`449`	`stat64_add(&mig_stats.downtime_bytes, bytes);`
`450`	`450`	`}`
`451`		`- stat64_add(&mig_stats.transferred, bytes);`
`452`	`451`	`}`
`453`	`452`
`454`	`453`	`struct MigrationOps {`