[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 4/5] pc-dimm: Don't check dimm->node when there is no
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] [PULL 4/5] pc-dimm: Don't check dimm->node when there is non-NUMA config |
Date: |
Tue, 30 Sep 2014 12:40:10 +0300 |
From: zhanghailiang <address@hidden>
It should not break memory hotplug feature if there is non-NUMA option.
This patch would also allow to use pc-dimm as replacement for initial memory
for non-NUMA configs.
Note: After this patch, the memory hotplug can work normally for Linux guest OS
when there is non-NUMA option and NUMA option. But not support Windows guest OS
to hotplug memory with no-NUMA config, actully, it's Windows limitation.
Reviewed-By: Igor Mammedov <address@hidden>
Signed-off-by: zhanghailiang <address@hidden>
Acked-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
---
hw/mem/pc-dimm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/mem/pc-dimm.c b/hw/mem/pc-dimm.c
index 5bfc5b7..a800ea7 100644
--- a/hw/mem/pc-dimm.c
+++ b/hw/mem/pc-dimm.c
@@ -252,7 +252,7 @@ static void pc_dimm_realize(DeviceState *dev, Error **errp)
error_setg(errp, "'" PC_DIMM_MEMDEV_PROP "' property is not set");
return;
}
- if (dimm->node >= nb_numa_nodes) {
+ if ((nb_numa_nodes > 0) && (dimm->node >= nb_numa_nodes)) {
error_setg(errp, "'DIMM property " PC_DIMM_NODE_PROP " has value %"
PRIu32 "' which exceeds the number of numa nodes: %d",
dimm->node, nb_numa_nodes);
--
MST
- [Qemu-devel] [PULL 0/5] pci, pc, virtio, misc bugfixes, Michael S. Tsirkin, 2014/09/30
- [Qemu-devel] [PULL 1/5] loader: g_realloc(p, 0) frees and returns NULL, simplify, Michael S. Tsirkin, 2014/09/30
- [Qemu-devel] [PULL 2/5] Revert "virtio-pci: fix migration for pci bus master", Michael S. Tsirkin, 2014/09/30
- [Qemu-devel] [PULL 3/5] pci-hotplug-old: avoid losing error message, Michael S. Tsirkin, 2014/09/30
- [Qemu-devel] [PULL 5/5] vl: Adjust the place of calling mlockall to speedup VM's startup, Michael S. Tsirkin, 2014/09/30
- [Qemu-devel] [PULL 4/5] pc-dimm: Don't check dimm->node when there is non-NUMA config,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PULL 0/5] pci, pc, virtio, misc bugfixes, Peter Maydell, 2014/09/30