From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:50027) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rlmb9-0000Sg-BQ for qemu-devel@nongnu.org; Fri, 13 Jan 2012 14:15:36 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Rlmb7-0007lo-Su for qemu-devel@nongnu.org; Fri, 13 Jan 2012 14:15:35 -0500 Received: from mx1.redhat.com ([209.132.183.28]:17053) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rlmb7-0007lh-MJ for qemu-devel@nongnu.org; Fri, 13 Jan 2012 14:15:33 -0500 From: Luiz Capitulino Date: Fri, 13 Jan 2012 17:15:20 -0200 Message-Id: <1326482122-12619-1-git-send-email-lcapitulino@redhat.com> Subject: [Qemu-devel] [PATCH v5 0/2]: qemu-ga: Add the guest-suspend command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: eblake@redhat.com, jcody@redhat.com, mdroth@linux.vnet.ibm.com I've tried to address all review comments in this new version. The two most important changes is that I've added the 'sleep' and 'hybrid' modes back and now the guest is queried for suspend support (the way I'm doing this is also worth reviewing). This series depends on this patch from Michael: http://lists.gnu.org/archive/html/qemu-devel/2012-01/msg01382.html and from a patch I've submitted to seabios to disable S3 advertise, as S3 is broken in qemu today. v5 o add 'sleep' and 'hybrid' modes back o query for suspend support using pm-is-supported & manual check o use _exit() [Daniel] o reopen standard file-descriptors to /dev/null [Eric] o make the SIGCHLD handler more portable by calling waitpid() in a loop, as not all unix versions will raise SIGCHLD multiple times if several children terminate at once (I don't even know if Linux does that) qapi-schema-guest.json | 29 ++++++ qemu-ga.c | 20 ++++- qga/guest-agent-commands.c | 212 ++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 259 insertions(+), 2 deletions(-)