|
 |
76daa3 |
From 79e13ab8c86fc0a78ad396f0b88635d88e8ce103 Mon Sep 17 00:00:00 2001
|
|
|
76daa3 |
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
|
|
|
76daa3 |
Date: Thu, 18 May 2017 11:17:27 +0200
|
|
|
76daa3 |
Subject: [PATCH 20/27] postcopy: Require RAMBlocks that are whole pages
|
|
|
76daa3 |
|
|
|
76daa3 |
RH-Author: Dr. David Alan Gilbert <dgilbert@redhat.com>
|
|
|
76daa3 |
Message-id: <20170518111727.7433-3-dgilbert@redhat.com>
|
|
|
76daa3 |
Patchwork-id: 75315
|
|
|
76daa3 |
O-Subject: [RHEL-7.4 qemu-kvm-rhev PATCH 2/2] postcopy: Require RAMBlocks that are whole pages
|
|
|
76daa3 |
Bugzilla: 1449037
|
|
|
76daa3 |
RH-Acked-by: Juan Quintela <quintela@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Peter Xu <peterx@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
It turns out that it's legal to create a VM with RAMBlocks that aren't
|
|
|
76daa3 |
a multiple of the pagesize in use; e.g. a 1025M main memory using
|
|
|
76daa3 |
2M host pages. That breaks postcopy's atomic placement of pages,
|
|
|
76daa3 |
so disallow it.
|
|
|
76daa3 |
|
|
|
76daa3 |
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
Upstream URL:
|
|
|
76daa3 |
http://lists.nongnu.org/archive/html/qemu-devel/2017-05/msg04393.html
|
|
|
76daa3 |
Upstream Message-Id: <20170518172453.32599-3-quintela@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
---
|
|
|
76daa3 |
migration/postcopy-ram.c | 16 +++++++++++++---
|
|
|
76daa3 |
1 file changed, 13 insertions(+), 3 deletions(-)
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/migration/postcopy-ram.c b/migration/postcopy-ram.c
|
|
|
76daa3 |
index dc80dbb..46d5a83 100644
|
|
|
76daa3 |
--- a/migration/postcopy-ram.c
|
|
|
76daa3 |
+++ b/migration/postcopy-ram.c
|
|
|
76daa3 |
@@ -97,14 +97,24 @@ static bool ufd_version_check(int ufd)
|
|
|
76daa3 |
|
|
|
76daa3 |
/* Callback from postcopy_ram_supported_by_host block iterator.
|
|
|
76daa3 |
*/
|
|
|
76daa3 |
-static int test_range_shared(const char *block_name, void *host_addr,
|
|
|
76daa3 |
+static int test_ramblock_postcopiable(const char *block_name, void *host_addr,
|
|
|
76daa3 |
ram_addr_t offset, ram_addr_t length, void *opaque)
|
|
|
76daa3 |
{
|
|
|
76daa3 |
- if (qemu_ram_is_shared(qemu_ram_block_by_name(block_name))) {
|
|
|
76daa3 |
+ RAMBlock *rb = qemu_ram_block_by_name(block_name);
|
|
|
76daa3 |
+ size_t pagesize = qemu_ram_pagesize(rb);
|
|
|
76daa3 |
+
|
|
|
76daa3 |
+ if (qemu_ram_is_shared(rb)) {
|
|
|
76daa3 |
error_report("Postcopy on shared RAM (%s) is not yet supported",
|
|
|
76daa3 |
block_name);
|
|
|
76daa3 |
return 1;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
+
|
|
|
76daa3 |
+ if (length % pagesize) {
|
|
|
76daa3 |
+ error_report("Postcopy requires RAM blocks to be a page size multiple,"
|
|
|
76daa3 |
+ " block %s is 0x" RAM_ADDR_FMT " bytes with a "
|
|
|
76daa3 |
+ "page size of 0x%zx", block_name, length, pagesize);
|
|
|
76daa3 |
+ return 1;
|
|
|
76daa3 |
+ }
|
|
|
76daa3 |
return 0;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
@@ -141,7 +151,7 @@ bool postcopy_ram_supported_by_host(void)
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
/* We don't support postcopy with shared RAM yet */
|
|
|
76daa3 |
- if (qemu_ram_foreach_block(test_range_shared, NULL)) {
|
|
|
76daa3 |
+ if (qemu_ram_foreach_block(test_ramblock_postcopiable, NULL)) {
|
|
|
76daa3 |
goto out;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
--
|
|
|
76daa3 |
1.8.3.1
|
|
|
76daa3 |
|