From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40905) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V1lLV-0002mT-Vu for qemu-devel@nongnu.org; Tue, 23 Jul 2013 18:46:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V1lLT-0001hh-DW for qemu-devel@nongnu.org; Tue, 23 Jul 2013 18:46:17 -0400 Received: from usindpps06.hds.com ([207.126.252.19]:39932) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V1lLT-0001hb-90 for qemu-devel@nongnu.org; Tue, 23 Jul 2013 18:46:15 -0400 From: Tomoki Sekiyama Date: Tue, 23 Jul 2013 18:46:06 -0400 Message-ID: <20130723224606.26273.97063.stgit@outback> In-Reply-To: <20130723224525.26273.28321.stgit@outback> References: <20130723224525.26273.28321.stgit@outback> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] [PATCH v8 10/10] QMP/qemu-ga-client: Make timeout longer for guest-fsfreeze-freeze command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: libaiqing@huawei.com, ghammer@redhat.com, stefanha@gmail.com, mdroth@linux.vnet.ibm.com, lcapitulino@redhat.com, vrozenfe@redhat.com, Paolo Bonzini , seiji.aguchi@hds.com, Laszlo Ersek , areis@redhat.com guest-fsfreeze-freeze command can take longer than 3 seconds when heavy disk I/O is running. To avoid unexpected timeout, this changes the timeout to 60 seconds (timeout of pre-commit phase of VSS). Signed-off-by: Tomoki Sekiyama Reviewed-by: Paolo Bonzini Reviewed-by: Laszlo Ersek --- QMP/qemu-ga-client | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/QMP/qemu-ga-client b/QMP/qemu-ga-client index 46676c3..b5f7e7c 100755 --- a/QMP/qemu-ga-client +++ b/QMP/qemu-ga-client @@ -267,7 +267,9 @@ def main(address, cmd, args): print('Hint: qemu is not running?') sys.exit(1) - if cmd != 'ping': + if cmd == 'fsfreeze' and args[0] == 'freeze': + client.sync(60) + elif cmd != 'ping': client.sync() globals()['_cmd_' + cmd](client, args)