From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36371) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gMcB7-0007Wl-HO for qemu-devel@nongnu.org; Tue, 13 Nov 2018 12:08:45 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gMbt5-0001ec-7B for qemu-devel@nongnu.org; Tue, 13 Nov 2018 11:50:07 -0500 Received: from mail-qk1-x741.google.com ([2607:f8b0:4864:20::741]:33042) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gMbt5-0001e9-24 for qemu-devel@nongnu.org; Tue, 13 Nov 2018 11:50:03 -0500 Received: by mail-qk1-x741.google.com with SMTP id o89so20569415qko.0 for ; Tue, 13 Nov 2018 08:50:03 -0800 (PST) From: Daniel Henrique Barboza References: <20181113160908.12706-1-danielhb413@gmail.com> <7274afaf-06ad-c93f-370a-151f1605cabb@redhat.com> <914ac120-dcd1-06ac-7a97-5cc73b759cb0@gmail.com> Message-ID: <1ff5be0e-0023-2b78-0b4b-19cd23ec6fe3@gmail.com> Date: Tue, 13 Nov 2018 14:49:58 -0200 MIME-Version: 1.0 In-Reply-To: <914ac120-dcd1-06ac-7a97-5cc73b759cb0@gmail.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US Subject: Re: [Qemu-devel] [PATCH 1/1] qga: update docs with systemd suspend support info List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake , qemu-devel@nongnu.org Cc: mdroth@linux.vnet.ibm.com On 11/13/18 2:39 PM, Daniel Henrique Barboza wrote: > > > On 11/13/18 2:32 PM, Eric Blake wrote: >> On 11/13/18 10:09 AM, Daniel Henrique Barboza wrote: >>> Commit 067927d62e ("qga: systemd hibernate/suspend/hybrid-sleep >>> support") failed to update qapi-schema.json after adding systemd >>> hibernate/suspend/hybrid-sleep capabilities to guest-suspend-* QGA >>> commands. >>> >>> Signed-off-by: Daniel Henrique Barboza >>> --- >>>   qga/qapi-schema.json | 37 ++++++++++++++++++++++++++----------- >>>   1 file changed, 26 insertions(+), 11 deletions(-) >> >>> +++ b/qga/qapi-schema.json >>> @@ -532,12 +532,16 @@ >>>   # >>>   # Suspend guest to disk. >>>   # >>> -# This command tries to execute the scripts provided by the >>> pm-utils package. >>> -# If it's not available, the suspend operation will be performed by >>> manually >>> -# writing to a sysfs file. >>> +# This command attempts to suspend the guest using three >>> strategies, in this >>> +# order: >>>   # >>> -# For the best results it's strongly recommended to have the pm-utils >>> -# package installed in the guest. >>> +# - systemd hibernate >>> +# - pm-utils (via pm-hibernate) >>> +# - manual write into sysfs >>> +# >>> +# If the guest does not have systemd hibernate support, the scripts >>> in pm-utils >>> +# package are executed, in case pm-utils is available. Manual write >>> in sysfs >>> +# is a last resort. >> >> Does this paragraph add any information compared to the earlier list? > > Perhaps we can remove the paragraph and add "if available" remarks in > the listing? > Like: > > +# - systemd hibernate, if supported > +# - pm-utils (via pm-hibernate), if pm-utils is installed > +# - manual write into sysfs as a last resort > > Same for the other 2 cases below. Actually, nevermind. Removing the extra paragraph and keeping the list as is seems cleaner. One can easily assume that, in the lack of guest support of systemd hibernate, the command will try the next option. Same thing with pm-utils. > >> >>>   # >>>   # This command does NOT return a response on success. There is a >>> high chance >>>   # the command succeeded if the VM exits with a zero exit status >>> or, when >>> @@ -560,12 +564,16 @@ >>>   # >>>   # Suspend guest to ram. >>>   # >>> -# This command tries to execute the scripts provided by the >>> pm-utils package. >>> -# If it's not available, the suspend operation will be performed by >>> manually >>> -# writing to a sysfs file. >>> +# This command attempts to suspend the guest using three >>> strategies, in this >>> +# order: >>> +# >>> +# - systemd suspend >>> +# - pm-utils (via pm-suspend) >>> +# - manual write into sysfs >>>   # >>> -# For the best results it's strongly recommended to have the pm-utils >>> -# package installed in the guest. >>> +# If the guest does not have systemd suspend support, the scripts >>> in pm-utils >>> +# package are executed, in case pm-utils is available. Manual write >>> in sysfs >>> +# is a last resort. >> >> Likewise. >> >>>   # >>>   # IMPORTANT: guest-suspend-ram requires QEMU to support the >>> 'system_wakeup' >>>   # command.  Thus, it's *required* to query QEMU for the presence >>> of the >>> @@ -592,7 +600,14 @@ >>>   # >>>   # Save guest state to disk and suspend to ram. >>>   # >>> -# This command requires the pm-utils package to be installed in the >>> guest. >>> +# This command attempts to suspend the guest using three >>> strategies, in this >>> +# order: >>> +# >>> +# - systemd hybrid-sleep >>> +# - pm-utils (via pm-suspend-hybrid) >>> +# >>> +# If the guest does not have systemd hybrid-sleep support, the >>> scripts in >>> +# pm-utils package are executed, in case pm-utils is available. >> >> "using three strategies" but a list of only 2 seems odd. >> > > Copy/paste busted!