[libvirt] [PATCH 02/15] Introduce virDomainResumeFlags

Eric Blake eblake at redhat.com
Mon Feb 3 17:24:56 UTC 2014


On 02/03/2014 09:16 AM, Michal Privoznik wrote:
> So far, we have just bare virDomainResume() API that resumes a domain.
> However, in the future there might occur a case, in which we may want
> to modify resume behavior slightly. In that case, @flags are useful.
> 
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
>  include/libvirt/libvirt.h.in |  2 ++
>  src/driver.h                 |  5 ++++
>  src/libvirt.c                | 63 ++++++++++++++++++++++++++++++++++++--------
>  src/libvirt_public.syms      |  1 +
>  src/remote/remote_driver.c   |  1 +
>  src/remote/remote_protocol.x | 13 ++++++++-
>  src/remote_protocol-structs  |  5 ++++
>  7 files changed, 78 insertions(+), 12 deletions(-)

Same story as 1/15, where I like the change even if you don't have a
plan for the flags, but you should get a second opinion.

Also, I could see squashing 1 and 2 into a single patch - it doesn't
make sense to introduce one function without the other, especially since
you are touching cross-documentation to call out the new function names.

>  
>  /**
> + * virDomainResumeFlags:
> + * @domain: a domain object
> + * @flags: extra flags, not used yet, so callers should always pass 0
> + *
> + * Resume a suspended domain, the process is restarted from the state
> + * where it was frozen by calling virDomainSuspend() or
> + * virDomainSuspendFlags(). This function may require privileged
> + * access Moreover, resume may not be supported if domain is in some
> + * special state like VIR_DOMAIN_PMSUSPENDED.
> + *

As in patch 1, you need to add a comment to virDomainResume() mentioning
that it is short for virDomainResumeFlags(,0).


> +++ b/src/libvirt_public.syms
> @@ -648,6 +648,7 @@ LIBVIRT_1.2.1 {
>  LIBVIRT_1.2.2 {
>      global:
>          virDomainSuspendFlags;
> +        virDomainResumeFlags;

Not essential, but I like listing new APIs in alphabetical order.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140203/3dfd9ef3/attachment-0001.sig>


More information about the libvir-list mailing list