- CIM - libvirt List Archives

Is virStoragePoolRefresh() synchronous or asynchronous?
by Gareth S Bestor 12 Aug '11

12 Aug '11

Basically, when you call virStoragePoolRefresh(), eg to pickup any new disks that may have just been created under the pool path, when the API returns are you guaranteed that the refresh operation is completed, or might the pool still be refreshing itself? I ask because I understand this refresh operation can be 'expensive', so I'm wondering if in fact things may still be going on in the background after the API returns... - Gareth Dr. Gareth S. Bestor IBM Senior Software Engineer Systems & Technology Group - Systems Management Standards 971-285-6375 (mobile) bestor(a)us.ibm.com

3 2

[PATCH] VirtualSystemManagementService: Fixing potential null dereferences and leaks
by Eduardo Lima (Etrunko) 12 Aug '11

12 Aug '11

# HG changeset patch # User Eduardo Lima (Etrunko) <eblima(a)br.ibm.com> # Date 1312918075 10800 # Node ID 8759e60c17c42101118f914215d071138340c70f # Parent 0291fb05e93a0cbcbf2b80c894a47d58f7c37d23 VirtualSystemManagementService: Fixing potential null dereferences and leaks As reported in https://bugzilla.redhat.com/show_bug.cgi?id=728245 line 1048 - Comparing "path" to null implies that "path" might be null. line 1057 - Dereferencing null variable "path". line 1088 - Comparing "port" to null implies that "port" might be null. line 1094 - Dereferencing null variable "port". Signed-off-by: Eduardo Lima (Etrunko) <eblima(a)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 @@ -1054,8 +1054,12 @@ ret = 1; out: - CU_DEBUG("Exiting parse_console_address, ip is %s, port is %s", - *path, *port); + free(tmp_path); + free(tmp_port); + + if (path && port) + CU_DEBUG("Exiting parse_console_address, ip is %s, port is %s", + *path, *port); return ret; } @@ -1091,8 +1095,12 @@ ret = 1; out: - CU_DEBUG("Exiting parse_vnc_address, ip is %s, port is %s", - *ip, *port); + free(tmp_ip); + free(tmp_port); + + if (ip && port) + CU_DEBUG("Exiting parse_vnc_address, ip is %s, port is %s", + *ip, *port); return ret; }

3 2

[PATCH 0 of 2] Add SDL frame buffer support in libvirt-cim, V4
by Wayne Xia 12 Aug '11

12 Aug '11

these patches would add support of SDL frame buffer configuration.

2 5

Network-pool seems not conform to profile
by Wayne Xia 11 Aug '11

11 Aug '11

By executing following command: wbemcli -nl ei http://root:123456@localhost:5988/root/virt:KVM_NetworkPool There would be 2 network pool, NetworkPool/default and NetworkPool/0. I think NetworkPool/0 is OK, which represent the primordial pool for network device, but NetworkPool/default should not be there, because it comes from the calling of the libvirt's API which returns the existing logical network device on the host, not the network pool. Another issue is that, it seems now libvirt-cim supports the creation of NetworkPool as a child, but the libvirt now does not support that. So I think this function should be removed, otherwise we need extra files to record the organization of the network pools. -- Best Regards Wayne Xia mail:xiawenc@linux.vnet.ibm.com tel:86-010-82450803

2 2

[PATCH] Check to see if stream is non-null before closing
by Sharad Mishra 11 Aug '11

11 Aug '11

# HG changeset patch # User Sharad Mishra <snmishra(a)us.ibm.com> # Date 1312828809 25200 # Node ID 8b4f6d69f40904ced24f7948f04f89e770e37bce # Parent dd060dd8dbbd6c1d9a73de4dffa9fb2ab84d62b9 Check to see if stream is non-null before closing. Added a check to see if the stream is non-null before calling close. Signed-off-by: Sharad Mishra <snmishra(a)us.ibm.com> diff -r dd060dd8dbbd -r 8b4f6d69f409 src/Virt_SwitchService.c --- a/src/Virt_SwitchService.c Thu Aug 04 11:22:32 2011 -0400 +++ b/src/Virt_SwitchService.c Mon Aug 08 11:40:09 2011 -0700 @@ -91,7 +91,8 @@ "No VSI Support found"); out: - pclose(stream); + if (stream != NULL) + pclose(stream); return s; }

3 2

[PATCH] (#2) bugfix: KVM_MemResourceAllocationSettingData does not conform to profile
by Wayne Xia 10 Aug '11

10 Aug '11

# HG changeset patch # User Wayne Xia <xiawenc(a)linux.vnet.ibm.com> # Date 1311231387 -28800 # Node ID d4002b23fffc97fd358811d551abbce28659839b # Parent dd060dd8dbbd6c1d9a73de4dffa9fb2ab84d62b9 (#2) bugfix: KVM_MemResourceAllocationSettingData does not conform to profile According to the discuss and profile, the reserved property means the memory actually allocated to support the VM running, and the unit should be byte*2^10. This patch added some code to retrieve VM's state, and the report the memory status according to that. #2 removed the unit changing to avoid impacting to existing user case. Signed-off-by: Wayne Xia <xiawenc(a)linux.vnet.ibm.com> diff -r dd060dd8dbbd -r d4002b23fffc libxkutil/device_parsing.c --- a/libxkutil/device_parsing.c Thu Aug 04 11:22:32 2011 -0400 +++ b/libxkutil/device_parsing.c Thu Jul 21 14:56:27 2011 +0800 @@ -801,6 +801,7 @@ } else if (dev->type == CIM_RES_TYPE_MEM) { dev->dev.mem.size = _dev->dev.mem.size; dev->dev.mem.maxsize = _dev->dev.mem.maxsize; + dev->dev.mem.reserved = _dev->dev.mem.reserved; } else if (dev->type == CIM_RES_TYPE_PROC) { dev->dev.vcpu.quantity = _dev->dev.vcpu.quantity; } else if (dev->type == CIM_RES_TYPE_EMU) { @@ -895,8 +896,22 @@ if (xml == NULL) return 0; - if (type == CIM_RES_TYPE_MEM) + if (type == CIM_RES_TYPE_MEM) { ret = _get_mem_device(xml, list); + if (*list != NULL) { + virDomainInfo dom_info; + if (virDomainGetInfo(dom, &dom_info) == 0) { + (*list)->dev.mem.reserved = dom_info.memory; + if (dom_info.state == 5) { /* VM not active */ + (*list)->dev.mem.reserved = 0; + } + } + else { + CU_DEBUG("failed to get dom state for mem"); + ret = -1; + } + } + } else if (type == CIM_RES_TYPE_PROC) ret = _get_proc_device(xml, list); else diff -r dd060dd8dbbd -r d4002b23fffc libxkutil/device_parsing.h --- a/libxkutil/device_parsing.h Thu Aug 04 11:22:32 2011 -0400 +++ b/libxkutil/device_parsing.h Thu Jul 21 14:56:27 2011 +0800 @@ -72,6 +72,7 @@ struct mem_device { uint64_t size; uint64_t maxsize; + uint64_t reserved; }; struct vcpu_device { diff -r dd060dd8dbbd -r d4002b23fffc src/Virt_RASD.c --- a/src/Virt_RASD.c Thu Aug 04 11:22:32 2011 -0400 +++ b/src/Virt_RASD.c Thu Jul 21 14:56:27 2011 +0800 @@ -633,7 +633,7 @@ CMSetProperty(inst, "VirtualQuantity", (CMPIValue *)&dev->dev.mem.size, CMPI_uint64); CMSetProperty(inst, "Reservation", - (CMPIValue *)&dev->dev.mem.size, CMPI_uint64); + (CMPIValue *)&dev->dev.mem.reserved, CMPI_uint64); CMSetProperty(inst, "Limit", (CMPIValue *)&dev->dev.mem.maxsize, CMPI_uint64); } else if (dev->type == CIM_RES_TYPE_PROC) {

1 0

[PATCH] bug fix: KVM_MemResourceAllocationSettingData does not conform to profile
by Wayne Xia 08 Aug '11

08 Aug '11

# HG changeset patch # User Wayne Xia <xiawenc(a)linux.vnet.ibm.com> # Date 1311231387 -28800 # Node ID 677867c3d2a16a97591bde2828808f9f39b859a7 # Parent 3c90a88a5199a4ed931a4a76097cff8f55deae41 changed a bit to make it conform to CIM profile According to the discuss and profile, the reserved property means the memory actually allocated to support the VM running, and the unit should be byte*2^10. This patch added some code to retrieve VM's state, and the report the memory status according to that. https://bugzilla.linux.ibm.com/show_bug.cgi?id=72759 Signed-off-by: Wayne Xia <xiawenc(a)linux.vnet.ibm.com> diff -r 3c90a88a5199 -r 677867c3d2a1 libxkutil/device_parsing.c --- a/libxkutil/device_parsing.c Mon Jul 18 11:13:40 2011 -0300 +++ b/libxkutil/device_parsing.c Thu Jul 21 14:56:27 2011 +0800 @@ -792,6 +792,7 @@ } else if (dev->type == CIM_RES_TYPE_MEM) { dev->dev.mem.size = _dev->dev.mem.size; dev->dev.mem.maxsize = _dev->dev.mem.maxsize; + dev->dev.mem.reserved = _dev->dev.mem.reserved; } else if (dev->type == CIM_RES_TYPE_PROC) { dev->dev.vcpu.quantity = _dev->dev.vcpu.quantity; } else if (dev->type == CIM_RES_TYPE_EMU) { @@ -885,8 +886,22 @@ if (xml == NULL) return 0; - if (type == CIM_RES_TYPE_MEM) + if (type == CIM_RES_TYPE_MEM) { ret = _get_mem_device(xml, list); + if (*list != NULL) { + virDomainInfo dom_info; + if (virDomainGetInfo(dom, &dom_info) == 0) { + (*list)->dev.mem.reserved = dom_info.memory; + if (dom_info.state == 5) { /* VM not active */ + (*list)->dev.mem.reserved = 0; + } + } + else { + CU_DEBUG("failed to get dom state for mem"); + ret = -1; + } + } + } else if (type == CIM_RES_TYPE_PROC) ret = _get_proc_device(xml, list); else diff -r 3c90a88a5199 -r 677867c3d2a1 libxkutil/device_parsing.h --- a/libxkutil/device_parsing.h Mon Jul 18 11:13:40 2011 -0300 +++ b/libxkutil/device_parsing.h Thu Jul 21 14:56:27 2011 +0800 @@ -71,6 +71,7 @@ struct mem_device { uint64_t size; uint64_t maxsize; + uint64_t reserved; }; struct vcpu_device { diff -r 3c90a88a5199 -r 677867c3d2a1 src/Virt_RASD.c --- a/src/Virt_RASD.c Mon Jul 18 11:13:40 2011 -0300 +++ b/src/Virt_RASD.c Thu Jul 21 14:56:27 2011 +0800 @@ -576,14 +576,14 @@ inst); } else if (dev->type == CIM_RES_TYPE_MEM) { - const char *units = "KiloBytes"; + const char *units = "byte*2^10"; CMSetProperty(inst, "AllocationUnits", (CMPIValue *)units, CMPI_chars); CMSetProperty(inst, "VirtualQuantity", (CMPIValue *)&dev->dev.mem.size, CMPI_uint64); CMSetProperty(inst, "Reservation", - (CMPIValue *)&dev->dev.mem.size, CMPI_uint64); + (CMPIValue *)&dev->dev.mem.reserved, CMPI_uint64); CMSetProperty(inst, "Limit", (CMPIValue *)&dev->dev.mem.maxsize, CMPI_uint64); } else if (dev->type == CIM_RES_TYPE_PROC) {

3 5

[PATCH] Retrun object paths for newly created filter associations
by Chip Vincent 05 Aug '11

05 Aug '11

# HG changeset patch # User Chip Vincent <cvincent(a)us.ibm.com> # Date 1312471352 14400 # Node ID 43d8048faffd40edd88ec339b7e615ff48a70b6a # Parent 5dc300d4dfd7b9741a6086e4e5b8da632bcdcd2d Retrun object paths for newly created filter associations. Fix to return the resulting new association object path at the end CreateInstance for Nested and AppliedFilterList. This is simply passing back the object path that was passed to the provider if there are no failures during the creation process. Signed-off-by: Chip Vincent <cvincent(a)us.ibm.com> diff --git a/src/Virt_AppliedFilterList.c b/src/Virt_AppliedFilterList.c --- a/src/Virt_AppliedFilterList.c +++ b/src/Virt_AppliedFilterList.c @@ -508,6 +508,7 @@ goto out; } + CMReturnObjectPath(results, reference); CU_DEBUG("CreateInstance complete"); out: diff --git a/src/Virt_NestedFilterList.c b/src/Virt_NestedFilterList.c --- a/src/Virt_NestedFilterList.c +++ b/src/Virt_NestedFilterList.c @@ -386,6 +386,7 @@ goto out; } + CMReturnObjectPath(results, reference); CU_DEBUG("CreateInstance completed"); out:

1 0

[PATCH] New entry "poolid" was not getting DUPed
by Sharad Mishra 04 Aug '11

04 Aug '11

# HG changeset patch # User Sharad Mishra <snmishra(a)us.ibm.com> # Date 1311797923 25200 # Node ID 5dc300d4dfd7b9741a6086e4e5b8da632bcdcd2d # Parent 0f42cab9c45c53cc13407b16418399ed8ed4a026 New entry "poolid" was not getting DUPed. A recent patch added support for network poolid, but missed to add DUP for it. This patch fixes the issue. Signed-off-by: Sharad Mishra <snmishra(a)us.ibm.com> diff -r 0f42cab9c45c -r 5dc300d4dfd7 libxkutil/device_parsing.c --- a/libxkutil/device_parsing.c Mon Jul 25 13:14:22 2011 -0700 +++ b/libxkutil/device_parsing.c Wed Jul 27 13:18:43 2011 -0700 @@ -775,6 +775,7 @@ DUP_FIELD(dev, _dev, dev.net.type); DUP_FIELD(dev, _dev, dev.net.source); DUP_FIELD(dev, _dev, dev.net.model); + DUP_FIELD(dev, _dev, dev.net.poolid); DUP_FIELD(dev, _dev, dev.net.device); DUP_FIELD(dev, _dev, dev.net.net_mode); DUP_FIELD(dev, _dev, dev.net.filter_ref);

3 2

[PATCH] [TEST] Reduce the length of bridge name
by Sharad Mishra 04 Aug '11

04 Aug '11

# HG changeset patch # User Sharad Mishra <snmishra(a)us.ibm.com> # Date 1311878184 25200 # Node ID 8c23ed99a218f1d1eea5c662b2fa3c211fe6d531 # Parent 3bc1a0fc5683d705c022014364cacf6d5860ba28 [TEST] Reduce the length of bridge name. Lately I have seen this (VirtualSystemManagementService - 06_addresource.py) test fails with following error - # virsh -c qemu:///system net-create /home/net.xml error: Failed to create network from /home/net.xml error: cannot create dummy tap device 'testbridge55-nic' to set mac address on bridge 'testbridge55': Invalid argument Reducing the size of bridge name fixed this issue. I looked on the net but could not find more info on when this behaviour changed in libvirt. Signed-off-by: Sharad Mishra <snmishra(a)us.ibm.com> diff -r 3bc1a0fc5683 -r 8c23ed99a218 suites/libvirt-cim/lib/XenKvmLib/const.py --- a/suites/libvirt-cim/lib/XenKvmLib/const.py Tue Jul 12 16:02:27 2011 -0300 +++ b/suites/libvirt-cim/lib/XenKvmLib/const.py Thu Jul 28 11:36:24 2011 -0700 @@ -58,7 +58,7 @@ # vxml.NetXML -default_bridge_name = 'testbridge' +default_bridge_name = 'testbr' default_network_name = 'cimtest-networkpool' default_net_type = 'network'

2 1