[libvirt] [PATCH v2 2/3] virsh: Expose virDomainPMWakeup

Eric Blake eblake at redhat.com
Thu Feb 16 01:05:24 UTC 2012


On 02/15/2012 08:04 AM, Michal Privoznik wrote:
> ---
>  tools/virsh.c   |   46 ++++++++++++++++++++++++++++++++++++++++++++++
>  tools/virsh.pod |    6 ++++++
>  2 files changed, 52 insertions(+), 0 deletions(-)

ACK.

I'm wondering, though, if we should make this command smarter (as a
followup patch).  The API for dompmsuspend made it into 0.9.10, but
dompmwakeup depends on 0.9.11.  Fortunately, out of the box, 0.9.10
always fails on dompmsuspend (we didn't implement anything in qemu).
But for backport situations (such as what I'm guessing RHEL will want to
do), where you can backport implementation but not API, then we will
have the (unfortunate) case that dompmsuspend might work while
dompmwakeup fails for anyone that bases a release on the 0.9.10 API.  Is
it worth adding a virsh flag, so that 'dompmwakeup dom --qemu' tries
virDomainPMWakeup, and if it fails, it then falls back to trying
virDomainQemuMonitorCommand to issue a raw "system_wakeup" monitor
command rather than completely giving up?

-- 
Eric Blake   eblake at 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: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120215/9840c548/attachment-0001.sig>


More information about the libvir-list mailing list