[Libvirt-cim] [PATCH] VirtualSystemManagementService: Always update domain device information
Eduardo Lima (Etrunko)
eblima at linux.vnet.ibm.com
Tue Sep 20 20:53:00 UTC 2011
src/Virt_VirtualSystemManagementService.c | 9 ---------
1 files changed, 0 insertions(+), 9 deletions(-)
# HG changeset patch
# User Eduardo Lima (Etrunko) <eblima at br.ibm.com>
# Date 1316551891 10800
# Node ID 25c6de6ebf094bf602e95ddf01dd39a6c5c1804a
# Parent 9a59a56e226f3f800ff7214b81ab5f2fe6362fcc
VirtualSystemManagementService: Always update domain device information
I noticed that a call to ModifyResourceSettings was causing devices description
to be lost. It happens that there was an explicit check in the code that
resulted in the device information to be updated only if the specified domain
was running.
This patch removes that block and now the resulting domain xml after the
ModifyResourceSettings call will keep the device information.
Signed-off-by: Eduardo Lima (Etrunko) <eblima at br.ibm.com>
diff --git a/src/Virt_VirtualSystemManagementService.c b/src/Virt_VirtualSystemManagementService.c
--- a/src/Virt_VirtualSystemManagementService.c
+++ b/src/Virt_VirtualSystemManagementService.c
@@ -2349,15 +2349,6 @@
update_dominfo(dominfo, refcn);
- if (!domain_online(dom)) {
- CU_DEBUG("VS `%s' not online; skipping dynamic update",
- dominfo->name);
- cu_statusf(_BROKER, &s,
- CMPI_RC_OK,
- "");
- goto out;
- }
-
CU_DEBUG("Doing dynamic device update for `%s'", dominfo->name);
if (func(dom, dev) == 0) {
More information about the Libvirt-cim
mailing list