Re: [PATCH] xenapi: support xenapi 5.6.0 headers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2010/8/21 Eric Blake <eblake@xxxxxxxxxx>:
> * src/xenapi/xenapi_driver.c (xenapiDomainGetInfo): Avoid using
> XEN_VM_POWER_STATE_UNKNOWN, which disappeared in newer xenapi.
> * src/xenapi/xenapi_utils.c (mapPowerState): Likewise.
> ---

> I guess I could fold the default: and UNDEFINED: case labels
> into one, if that is desirable.

You could make XEN_VM_POWER_STATE_UNDEFINED fall through to default as in

case XEN_VM_POWER_STATE_UNDEFINED:
default:
    /* Includes XEN_VM_POWER_STATE_UNKNOWN from libxenserver
     * 5.5.0, which is gone in 5.6.0.  */
    virState = VIR_DOMAIN_NOSTATE;
    break;

I think that it's not desirable to remove XEN_VM_POWER_STATE_UNDEFINED
completely.

ACK.

Matthias

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]