[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 12/12] suspend: add qmp events
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v5 12/12] suspend: add qmp events |
Date: |
Tue, 21 Feb 2012 09:30:14 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120209 Thunderbird/10.0.1 |
On 02/21/2012 03:00 AM, Gerd Hoffmann wrote:
> [ added libvirt to Cc:, leaving full context.
> this is about qmp events when the guest enters/leaves s3 ].
>
>>> @@ -1436,12 +1437,26 @@ void qemu_register_suspend_notifier(Notifier
>>> *notifier)
>>>
>>> void qemu_system_wakeup_request(WakeupReason reason)
>>> {
>>> + static const char *names[] = {
>>> + [QEMU_WAKEUP_REASON_OTHER] = "other",
>>> + [QEMU_WAKEUP_REASON_RTC] = "rtc",
>>> + [QEMU_WAKEUP_REASON_PMTIMER] = "pmtimer",
>>
>> Is the reason really important for mngt? Can we just leave it out?
>
> I assumed the wakeup reason could be useful, but dunno.
> Zapping the code is no problem of course.
>
> Lets ask mgmt aka libvirt folks ;)
If you present the reason, libvirt will pass it on to the management
app. But I don't see any reason why libvirt itself would care about the
reason, so it won't hurt libvirt if you drop a reason field. I don't
know whether higher-level management apps, like oVirt, would care or not.
Isn't this something where it is easier to omit first and add later once
we have a use case, than to add up front only to find that no one cares?
--
Eric Blake address@hidden +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v5 07/12] suspend: add system_wakeup monitor command, (continued)
- [Qemu-devel] [PATCH v5 07/12] suspend: add system_wakeup monitor command, Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 02/12] acpi: add ACPIREGS, Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 09/12] suspend: make serial ports wakeup the guest., Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 10/12] suspend: make rtc alarm wakeup the guest., Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 12/12] suspend: add qmp events, Gerd Hoffmann, 2012/02/15
[Qemu-devel] [PATCH v5 06/12] suspend: switch acpi s3 to new infrastructure., Gerd Hoffmann, 2012/02/15
[Qemu-devel] [PATCH v5 05/12] suspend: add infrastructure, Gerd Hoffmann, 2012/02/15