[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 08/13] migration: Don't abuse qemu_file transferred for RDMA
From: |
Juan Quintela |
Subject: |
[PULL 08/13] migration: Don't abuse qemu_file transferred for RDMA |
Date: |
Mon, 2 Oct 2023 14:20:16 +0200 |
Just create a variable for it, the same way that multifd does. This
way it is safe to use for other thread, etc, etc.
Reviewed-by: Leonardo Bras <leobras@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Message-Id: <20230515195709.63843-11-quintela@redhat.com>
---
migration/migration-stats.h | 4 ++++
migration/migration-stats.c | 5 +++--
migration/rdma.c | 22 ++++++++++++++++++++--
migration/trace-events | 2 +-
4 files changed, 28 insertions(+), 5 deletions(-)
diff --git a/migration/migration-stats.h b/migration/migration-stats.h
index ac2260e987..2358caad63 100644
--- a/migration/migration-stats.h
+++ b/migration/migration-stats.h
@@ -89,6 +89,10 @@ typedef struct {
* Maximum amount of data we can send in a cycle.
*/
Stat64 rate_limit_max;
+ /*
+ * Number of bytes sent through RDMA.
+ */
+ Stat64 rdma_bytes;
/*
* Total number of bytes transferred.
*/
diff --git a/migration/migration-stats.c b/migration/migration-stats.c
index 095d6d75bb..84e11e6dd8 100644
--- a/migration/migration-stats.c
+++ b/migration/migration-stats.c
@@ -61,8 +61,9 @@ void migration_rate_reset(QEMUFile *f)
uint64_t migration_transferred_bytes(QEMUFile *f)
{
uint64_t multifd = stat64_get(&mig_stats.multifd_bytes);
+ uint64_t rdma = stat64_get(&mig_stats.rdma_bytes);
uint64_t qemu_file = qemu_file_transferred(f);
- trace_migration_transferred_bytes(qemu_file, multifd);
- return qemu_file + multifd;
+ trace_migration_transferred_bytes(qemu_file, multifd, rdma);
+ return qemu_file + multifd + rdma;
}
diff --git a/migration/rdma.c b/migration/rdma.c
index a2a3db35b1..2a3c784328 100644
--- a/migration/rdma.c
+++ b/migration/rdma.c
@@ -2122,9 +2122,18 @@ retry:
return -EIO;
}
+ /*
+ * TODO: Here we are sending something, but we are not
+ * accounting for anything transferred. The following is
wrong:
+ *
+ * stat64_add(&mig_stats.rdma_bytes, sge.length);
+ *
+ * because we are using some kind of compression. I
+ * would think that head.len would be the more similar
+ * thing to a correct value.
+ */
stat64_add(&mig_stats.zero_pages,
sge.length / qemu_target_page_size());
-
return 1;
}
@@ -2232,8 +2241,17 @@ retry:
set_bit(chunk, block->transit_bitmap);
stat64_add(&mig_stats.normal_pages, sge.length / qemu_target_page_size());
+ /*
+ * We are adding to transferred the amount of data written, but no
+ * overhead at all. I will asume that RDMA is magicaly and don't
+ * need to transfer (at least) the addresses where it wants to
+ * write the pages. Here it looks like it should be something
+ * like:
+ * sizeof(send_wr) + sge.length
+ * but this being RDMA, who knows.
+ */
+ stat64_add(&mig_stats.rdma_bytes, sge.length);
ram_transferred_add(sge.length);
- qemu_file_credit_transfer(f, sge.length);
rdma->total_writes++;
return 0;
diff --git a/migration/trace-events b/migration/trace-events
index 4666f19325..63483732ce 100644
--- a/migration/trace-events
+++ b/migration/trace-events
@@ -191,7 +191,7 @@ 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
+migration_transferred_bytes(uint64_t qemu_file, uint64_t multifd, uint64_t
rdma) "qemu_file %" PRIu64 " multifd %" PRIu64 " RDMA %" PRIu64
# channel.c
migration_set_incoming_channel(void *ioc, const char *ioctype) "ioc=%p
ioctype=%s"
--
2.41.0
- [PULL 00/13] Migration 20231002 patches, Juan Quintela, 2023/10/02
- [PULL 01/13] migration-test: Create kvm_opts, Juan Quintela, 2023/10/02
- [PULL 02/13] migration-test: bootpath is the same for all tests and for all archs, Juan Quintela, 2023/10/02
- [PULL 04/13] migration-test: dirtylimit checks for x86_64 arch before, Juan Quintela, 2023/10/02
- [PULL 03/13] migration-test: Add bootfile_create/delete() functions, Juan Quintela, 2023/10/02
- [PULL 06/13] migration: Refactor repeated call of yank_unregister_instance, Juan Quintela, 2023/10/02
- [PULL 07/13] migration: Use qemu_file_transferred_noflush() for block migration., Juan Quintela, 2023/10/02
- [PULL 05/13] migration-test: simplify shmem_opts handling, Juan Quintela, 2023/10/02
- [PULL 08/13] migration: Don't abuse qemu_file transferred for RDMA,
Juan Quintela <=
- [PULL 09/13] migration/RDMA: It is accounting for zero/normal pages in two places, Juan Quintela, 2023/10/02
- [PULL 10/13] migration/rdma: Remove QEMUFile parameter when not used, Juan Quintela, 2023/10/02
- [PULL 11/13] migration/rdma: Don't use imaginary transfers, Juan Quintela, 2023/10/02
- [PULL 12/13] migration: Remove unused qemu_file_credit_transfer(), Juan Quintela, 2023/10/02
- [PULL 13/13] migration/rdma: Simplify the function that saves a page, Juan Quintela, 2023/10/02
- Re: [PULL 00/13] Migration 20231002 patches, Stefan Hajnoczi, 2023/10/02