[Libvirt-cim] [PATCH] VirtualSystemManagementService: Always update domain device information

Sharad Mishra snmishra at us.ibm.com
Tue Sep 20 21:33:35 UTC 2011


Eduardo,

	What happens when you try to do a dynamic update on a defined guest
after applying this patch? I am guessing that libvirt should fail with
similar message.
                In any case, libvirt-cim should not be doing any check like
the one you patched. It  should  just call appropriate libvirt API.

-Sharad

libvirt-cim-bounces at redhat.com wrote on 09/20/2011 01:53:00 PM:

> "Eduardo Lima \(Etrunko\)" <eblima at linux.vnet.ibm.com>
> Sent by: libvirt-cim-bounces at redhat.com
>
> 09/20/2011 01:53 PM
>
> Please respond to
> List for discussion and development of libvirt CIM
<libvirt-cim at redhat.com>
>
> To
>
> libvirt-cim at redhat.com
>
> cc
>
> Subject
>
> [Libvirt-cim] [PATCH] VirtualSystemManagementService: Always update
> domain device information
>
>  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) {
>
> _______________________________________________
> Libvirt-cim mailing list
> Libvirt-cim at redhat.com
> https://www.redhat.com/mailman/listinfo/libvirt-cim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvirt-cim/attachments/20110920/a4c58b53/attachment.htm>


More information about the Libvirt-cim mailing list