The qemuDomainSetMemoryFlags() allows for memballoon
(<currentMemory/>) changes for both active and inactive guests.
And just before doing any change, we have to make sure that the
new size is not greater than the total memory (<memory/>).
However, the total memory includes not only the regular guest
memory, but also sum of maximum sizes of all virtio-mems (in fact
all memory devices for that matter). But virtio-mem devices are
modified differently (via virDomainUpdateDevice()) and thus the
upper limit for new balloon size has to be lowered.
Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com>
---
src/qemu/qemu_driver.c | 20 ++++++++++++++++++--
1 file changed, 18 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 735e383da6..b5610ca8e4 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -2416,12 +2416,28 @@ static int qemuDomainSetMemoryFlags(virDomainPtr dom, unsigned
long newmem,
} else {
/* resize the current memory */
unsigned long oldmax = 0;
+ size_t i;
- if (def)
+ if (def) {
oldmax = virDomainDefGetMemoryTotal(def);
+
+ /* While virtio-mem is regular mem from guest POV, it can't be
+ * modified through this API. */
+ for (i = 0; i < def->nmems; i++) {
+ if (def->mems[i]->model == VIR_DOMAIN_MEMORY_MODEL_VIRTIO_MEM)
+ oldmax -= def->mems[i]->size;
+ }
+ }
+
if (persistentDef) {
- if (!oldmax || oldmax > virDomainDefGetMemoryTotal(persistentDef))
+ if (!def || oldmax > virDomainDefGetMemoryTotal(persistentDef)) {
oldmax = virDomainDefGetMemoryTotal(persistentDef);
+
+ for (i = 0; i < persistentDef->nmems; i++) {
+ if (persistentDef->mems[i]->model ==
VIR_DOMAIN_MEMORY_MODEL_VIRTIO_MEM)
+ oldmax -= persistentDef->mems[i]->size;
+ }
+ }
}
if (newmem > oldmax) {
--
2.31.1