|
 |
1be5c7 |
From 6a9a5a2809cbbe2982df156722b88efeec998e3d Mon Sep 17 00:00:00 2001
|
|
|
1be5c7 |
From: Juan Quintela <quintela@redhat.com>
|
|
|
1be5c7 |
Date: Wed, 18 May 2022 02:52:22 -0300
|
|
|
1be5c7 |
Subject: [PATCH 01/37] migration: Never call twice qemu_target_page_size()
|
|
|
1be5c7 |
MIME-Version: 1.0
|
|
|
1be5c7 |
Content-Type: text/plain; charset=UTF-8
|
|
|
1be5c7 |
Content-Transfer-Encoding: 8bit
|
|
|
1be5c7 |
|
|
|
1be5c7 |
RH-Author: Leonardo Brás <leobras@redhat.com>
|
|
|
1be5c7 |
RH-MergeRequest: 191: MSG_ZEROCOPY + Multifd @ rhel8.7
|
|
|
1be5c7 |
RH-Commit: [1/26] 809ca84dec80bafc1959df8c9e57f482ee752a97
|
|
|
1be5c7 |
RH-Bugzilla: 2072049
|
|
|
1be5c7 |
RH-Acked-by: Peter Xu <peterx@redhat.com>
|
|
|
1be5c7 |
RH-Acked-by: Daniel P. Berrangé <berrange@redhat.com>
|
|
|
1be5c7 |
RH-Acked-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
|
|
|
1be5c7 |
|
|
|
1be5c7 |
Signed-off-by: Juan Quintela <quintela@redhat.com>
|
|
|
1be5c7 |
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
|
|
|
1be5c7 |
(cherry picked from commit 144fa06b3431e806057ce1438338395b35a3e544)
|
|
|
1be5c7 |
Signed-off-by: Leonardo Bras <leobras@redhat.com>
|
|
|
1be5c7 |
---
|
|
|
1be5c7 |
migration/migration.c | 7 ++++---
|
|
|
1be5c7 |
migration/multifd.c | 7 ++++---
|
|
|
1be5c7 |
migration/savevm.c | 5 +++--
|
|
|
1be5c7 |
3 files changed, 11 insertions(+), 8 deletions(-)
|
|
|
1be5c7 |
|
|
|
1be5c7 |
diff --git a/migration/migration.c b/migration/migration.c
|
|
|
1be5c7 |
index a87ff01b81..8a13294da6 100644
|
|
|
1be5c7 |
--- a/migration/migration.c
|
|
|
1be5c7 |
+++ b/migration/migration.c
|
|
|
1be5c7 |
@@ -992,6 +992,8 @@ static void populate_time_info(MigrationInfo *info, MigrationState *s)
|
|
|
1be5c7 |
|
|
|
1be5c7 |
static void populate_ram_info(MigrationInfo *info, MigrationState *s)
|
|
|
1be5c7 |
{
|
|
|
1be5c7 |
+ size_t page_size = qemu_target_page_size();
|
|
|
1be5c7 |
+
|
|
|
1be5c7 |
info->has_ram = true;
|
|
|
1be5c7 |
info->ram = g_malloc0(sizeof(*info->ram));
|
|
|
1be5c7 |
info->ram->transferred = ram_counters.transferred;
|
|
|
1be5c7 |
@@ -1000,12 +1002,11 @@ static void populate_ram_info(MigrationInfo *info, MigrationState *s)
|
|
|
1be5c7 |
/* legacy value. It is not used anymore */
|
|
|
1be5c7 |
info->ram->skipped = 0;
|
|
|
1be5c7 |
info->ram->normal = ram_counters.normal;
|
|
|
1be5c7 |
- info->ram->normal_bytes = ram_counters.normal *
|
|
|
1be5c7 |
- qemu_target_page_size();
|
|
|
1be5c7 |
+ info->ram->normal_bytes = ram_counters.normal * page_size;
|
|
|
1be5c7 |
info->ram->mbps = s->mbps;
|
|
|
1be5c7 |
info->ram->dirty_sync_count = ram_counters.dirty_sync_count;
|
|
|
1be5c7 |
info->ram->postcopy_requests = ram_counters.postcopy_requests;
|
|
|
1be5c7 |
- info->ram->page_size = qemu_target_page_size();
|
|
|
1be5c7 |
+ info->ram->page_size = page_size;
|
|
|
1be5c7 |
info->ram->multifd_bytes = ram_counters.multifd_bytes;
|
|
|
1be5c7 |
info->ram->pages_per_second = s->pages_per_second;
|
|
|
1be5c7 |
|
|
|
1be5c7 |
diff --git a/migration/multifd.c b/migration/multifd.c
|
|
|
1be5c7 |
index 7c9deb1921..8125d0015c 100644
|
|
|
1be5c7 |
--- a/migration/multifd.c
|
|
|
1be5c7 |
+++ b/migration/multifd.c
|
|
|
1be5c7 |
@@ -289,7 +289,8 @@ static void multifd_send_fill_packet(MultiFDSendParams *p)
|
|
|
1be5c7 |
static int multifd_recv_unfill_packet(MultiFDRecvParams *p, Error **errp)
|
|
|
1be5c7 |
{
|
|
|
1be5c7 |
MultiFDPacket_t *packet = p->packet;
|
|
|
1be5c7 |
- uint32_t pages_max = MULTIFD_PACKET_SIZE / qemu_target_page_size();
|
|
|
1be5c7 |
+ size_t page_size = qemu_target_page_size();
|
|
|
1be5c7 |
+ uint32_t pages_max = MULTIFD_PACKET_SIZE / page_size;
|
|
|
1be5c7 |
RAMBlock *block;
|
|
|
1be5c7 |
int i;
|
|
|
1be5c7 |
|
|
|
1be5c7 |
@@ -358,14 +359,14 @@ static int multifd_recv_unfill_packet(MultiFDRecvParams *p, Error **errp)
|
|
|
1be5c7 |
for (i = 0; i < p->pages->used; i++) {
|
|
|
1be5c7 |
uint64_t offset = be64_to_cpu(packet->offset[i]);
|
|
|
1be5c7 |
|
|
|
1be5c7 |
- if (offset > (block->used_length - qemu_target_page_size())) {
|
|
|
1be5c7 |
+ if (offset > (block->used_length - page_size)) {
|
|
|
1be5c7 |
error_setg(errp, "multifd: offset too long %" PRIu64
|
|
|
1be5c7 |
" (max " RAM_ADDR_FMT ")",
|
|
|
1be5c7 |
offset, block->used_length);
|
|
|
1be5c7 |
return -1;
|
|
|
1be5c7 |
}
|
|
|
1be5c7 |
p->pages->iov[i].iov_base = block->host + offset;
|
|
|
1be5c7 |
- p->pages->iov[i].iov_len = qemu_target_page_size();
|
|
|
1be5c7 |
+ p->pages->iov[i].iov_len = page_size;
|
|
|
1be5c7 |
}
|
|
|
1be5c7 |
|
|
|
1be5c7 |
return 0;
|
|
|
1be5c7 |
diff --git a/migration/savevm.c b/migration/savevm.c
|
|
|
1be5c7 |
index d59e976d50..0bef031acb 100644
|
|
|
1be5c7 |
--- a/migration/savevm.c
|
|
|
1be5c7 |
+++ b/migration/savevm.c
|
|
|
1be5c7 |
@@ -1685,6 +1685,7 @@ static int loadvm_postcopy_handle_advise(MigrationIncomingState *mis,
|
|
|
1be5c7 |
{
|
|
|
1be5c7 |
PostcopyState ps = postcopy_state_set(POSTCOPY_INCOMING_ADVISE);
|
|
|
1be5c7 |
uint64_t remote_pagesize_summary, local_pagesize_summary, remote_tps;
|
|
|
1be5c7 |
+ size_t page_size = qemu_target_page_size();
|
|
|
1be5c7 |
Error *local_err = NULL;
|
|
|
1be5c7 |
|
|
|
1be5c7 |
trace_loadvm_postcopy_handle_advise();
|
|
|
1be5c7 |
@@ -1741,13 +1742,13 @@ static int loadvm_postcopy_handle_advise(MigrationIncomingState *mis,
|
|
|
1be5c7 |
}
|
|
|
1be5c7 |
|
|
|
1be5c7 |
remote_tps = qemu_get_be64(mis->from_src_file);
|
|
|
1be5c7 |
- if (remote_tps != qemu_target_page_size()) {
|
|
|
1be5c7 |
+ if (remote_tps != page_size) {
|
|
|
1be5c7 |
/*
|
|
|
1be5c7 |
* Again, some differences could be dealt with, but for now keep it
|
|
|
1be5c7 |
* simple.
|
|
|
1be5c7 |
*/
|
|
|
1be5c7 |
error_report("Postcopy needs matching target page sizes (s=%d d=%zd)",
|
|
|
1be5c7 |
- (int)remote_tps, qemu_target_page_size());
|
|
|
1be5c7 |
+ (int)remote_tps, page_size);
|
|
|
1be5c7 |
return -1;
|
|
|
1be5c7 |
}
|
|
|
1be5c7 |
|
|
|
1be5c7 |
--
|
|
|
1be5c7 |
2.35.3
|
|
|
1be5c7 |
|