[libvirt] virDomainGetInfo() returns wrong domain state

[Panday Ritesh Sharma (rpanday)]

Hi Libvirt support team,
Could you please help me with below query.

Regards
Ritesh Sharma

From: Panday Ritesh Sharma (rpanday)
Sent: Tuesday, September 24, 2013 8:22 PM
To: 'libvir-list at redhat.com'; 'libvirt-users at redhat.com'; Vinay Shankarkumar (vinays)
Cc: Basavaraj Bendigeri (bbendige); q-se-dev(mailer list)
Subject: virDomainGetInfo() returns wrong domain state


Hi Team,
I have written below code to get the VM state at run time. I found, though the VM is in shut-off state, when I use the function virDomainGetInfo();  I get state as running. Could you please let me know what wrong I am doing. To know the actual VM state I used 'virsh list' and it clearly shows the VM is in shut-off state. Please find the log and code snippet below.

Log from virsh:
=================
[host:~]$ virsh list  --all
Id    Name                           State
----------------------------------------------------
1     calvados                       running
2     LCXR                           running
3     default-sdr--1                 running
-     test--2                        shut off

Out put:
=========
04.03.06.698923264:INFO: vm_libvirt_state_to_vmm_state: state returned is 1

Note : Here 1 is actually running.

Code snippet:
===============
enum cidl_vmm_vm_state
vm_libvirt_state_to_vmm_state(unsigned char libvirt_state)
{
    enum cidl_vmm_vm_state state;
    INFO("%s: state returned is %u\n",__FUNCTION__, libvirt_state); <<<<<<<<<<<<<<<<<<
    if (libvirt_state == VIR_DOMAIN_RUNNING) {
        state = cidl_vm_state_running;
    } else if ((libvirt_state == VIR_DOMAIN_PAUSED) ||
               (libvirt_state == VIR_DOMAIN_BLOCKED)) {
        state = cidl_vm_state_paused;
    } else if (libvirt_state == VIR_DOMAIN_SHUTOFF) {
        state = cidl_vm_state_defined;
    } else {
        state = cidl_vm_state_not_defined;
    }
    return state;
}

           virDomainInfo res_util;
            virDomainPtr dom = virDomainLookupByName(virt,
                                                     private_names[vm_idx]);
            res = virDomainGetInfo(dom, &res_util);
            vminfo[vm_idx].vm_state =
                  vm_libvirt_state_to_vmm_state(res_util.state);



Regards
Ritesh Sharma
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130925/c57a1ebd/attachment-0001.htm>