qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] qapi: add info about reset to SHUTDOWN event


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH] qapi: add info about reset to SHUTDOWN event
Date: Wed, 17 Oct 2018 08:51:35 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1

On 10/17/18 2:17 AM, Dominik Csapak wrote:
On 10/8/18 3:19 PM, Dominik Csapak wrote:
when '-no-reboot' is set, it is interesting if the guest was originally
shutdown or reset, so save and return that info

Signed-off-by: Dominik Csapak <address@hidden>
---
  qapi/run-state.json | 5 ++++-
  vl.c                | 5 ++++-
  2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/qapi/run-state.json b/qapi/run-state.json
index 332e44897b..ec1769777d 100644
--- a/qapi/run-state.json
+++ b/qapi/run-state.json
@@ -107,6 +107,9 @@
  # a guest-initiated ACPI shutdown request or other hardware-specific action)   # rather than a host request (such as sending qemu a SIGINT). (since 2.10)
  #
+# @was_reset: If true, the shutdown was actually a reset, but no-reboot
+# was set, so it got converted to a shutdown

New additions should prefer naming like 'was-reset' rather than 'was_reset', if we still think this particular name is appropriate. My personal take: what does the 'was' add, which would prevent us from just using the name 'reset' and avoiding the separator spelling issue?

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org



reply via email to

[Prev in Thread] Current Thread [Next in Thread]