[libvirt] [PATCH] vcpu: teach getVcpusFlags about current

[Eric Blake]

On 07/15/2011 05:38 PM, Eric Blake wrote:
> Now that virDomainSetVcpusFlags knows about VIR_DOMAIN_AFFECT_CURRENT,
> so should virDomainGetVcpusFlags.
>
> Unfortunately, the virsh counterpart 'virsh vcpucount' has already
> commandeered --current for a different meaning, so virsh does not
> have a way to expose this new calling capability unless we either
> break backward compatibility or consistency with other virsh commands
> that take --live and --config.
>
> * src/libvirt.c (virDomainGetVcpusFlags): Allow
> VIR_DOMAIN_AFFECT_CURRENT.
> * src/libxl/libxl_driver.c (libxlDomainGetVcpusFlags): Likewise.
> * src/qemu/qemu_driver.c (qemudDomainGetVcpusFlags): Likewise.
> * src/test/test_driver.c (testDomainGetVcpusFlags): Likewise.
> * src/xen/xen_driver.c (xenUnifiedDomainGetVcpusFlags): Likewise.
> ---
>
> In response to:
> https://www.redhat.com/archives/libvir-list/2011-July/msg00967.html
>

Also, squash in some docs:

diff --git i/src/libvirt.c w/src/libvirt.c
index c34b46c..4b22b3a 100644
--- i/src/libvirt.c
+++ w/src/libvirt.c
@@ -6933,10 +6933,13 @@ error:
   * not support it.  This function requires privileged access to the
   * hypervisor.
   *
- * @flags must include either VIR_DOMAIN_AFFECT_LIVE to query a
- * running domain (which will fail if domain is not active), or
- * VIR_DOMAIN_AFFECT_CONFIG to query the XML description of the
- * domain.  It is an error to set both flags.
+ * If @flags includes VIR_DOMAIN_AFFECT_LIVE, this will query a
+ * running domain (which will fail if domain is not active); if
+ * it includes VIR_DOMAIN_AFFECT_CONFIG, this will query the XML
+ * description of the domain.  It is an error to set both flags.
+ * If neither flag is set (that is, VIR_DOMAIN_AFFECT_CURRENT),
+ * then the configuration queried depends on whether the domain
+ * is currently running.
   *
   * If @flags includes VIR_DOMAIN_VCPU_MAXIMUM, then the maximum
   * virtual CPU limit is queried.  Otherwise, this call queries the

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org