From patchwork Thu Jan 19 03:07:42 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Wen Congyang X-Patchwork-Id: 136744 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [140.186.70.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id E0B3E1007D1 for ; Thu, 19 Jan 2012 14:06:03 +1100 (EST) Received: from localhost ([::1]:53663 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RniK4-0005Kn-PY for incoming@patchwork.ozlabs.org; Wed, 18 Jan 2012 22:05:56 -0500 Received: from eggs.gnu.org ([140.186.70.92]:38784) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RniJv-0005FX-FJ for qemu-devel@nongnu.org; Wed, 18 Jan 2012 22:05:51 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RniJk-0000dM-1c for qemu-devel@nongnu.org; Wed, 18 Jan 2012 22:05:42 -0500 Received: from [222.73.24.84] (port=59495 helo=song.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RniJj-0000ct-0d for qemu-devel@nongnu.org; Wed, 18 Jan 2012 22:05:36 -0500 Received: from tang.cn.fujitsu.com (tang.cn.fujitsu.com [10.167.250.3]) by song.cn.fujitsu.com (Postfix) with ESMTP id 08687170130; Thu, 19 Jan 2012 11:05:24 +0800 (CST) Received: from mailserver.fnst.cn.fujitsu.com (tang.cn.fujitsu.com [127.0.0.1]) by tang.cn.fujitsu.com (8.14.3/8.13.1) with ESMTP id q0J35Nvn020443; Thu, 19 Jan 2012 11:05:23 +0800 Received: from [10.167.225.226] ([10.167.225.226]) by mailserver.fnst.cn.fujitsu.com (Lotus Domino Release 8.5.1FP4) with ESMTP id 2012011911041541-379196 ; Thu, 19 Jan 2012 11:04:15 +0800 Message-ID: <4F1788FE.2030708@cn.fujitsu.com> Date: Thu, 19 Jan 2012 11:07:42 +0800 From: Wen Congyang User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100413 Fedora/3.0.4-2.fc13 Thunderbird/3.0.4 MIME-Version: 1.0 To: qemu-devel , Jan Kiszka , Dave Anderson , HATAYAMA Daisuke , Luiz Capitulino References: <4F1784EE.2040800@cn.fujitsu.com> In-Reply-To: <4F1784EE.2040800@cn.fujitsu.com> X-MIMETrack: Itemize by SMTP Server on mailserver/fnst(Release 8.5.1FP4|July 25, 2010) at 2012-01-19 11:04:15, Serialize by Router on mailserver/fnst(Release 8.5.1FP4|July 25, 2010) at 2012-01-19 11:04:16, Serialize complete at 2012-01-19 11:04:16 X-detected-operating-system: by eggs.gnu.org: FreeBSD 6.x (1) X-Received-From: 222.73.24.84 Subject: [Qemu-devel] [RFC][PATCH 13/15 v5] support to set dumping speed X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Add API to allow the user to control dumping speed Signed-off-by: Wen Congyang --- dump.c | 12 ++++++++++++ hmp-commands.hx | 15 +++++++++++++++ hmp.c | 6 ++++++ hmp.h | 1 + qapi-schema.json | 15 +++++++++++++++ qmp-commands.hx | 22 ++++++++++++++++++++++ 6 files changed, 71 insertions(+), 0 deletions(-) diff --git a/dump.c b/dump.c index 485e3f3..39dc892 100644 --- a/dump.c +++ b/dump.c @@ -86,6 +86,7 @@ typedef struct DumpState { } DumpState; #define DEFAULT_THROTTLE (32 << 20) /* Default dump speed throttling */ +#define MIN_THROTTLE (1 << 10) /* Miniumum dump speed */ static DumpState *dump_get_current(void) { @@ -722,3 +723,14 @@ void qmp_dump_cancel(Error **errp) dump_cleanup(s); return; } + +void qmp_dump_set_speed(int64_t value, Error **errp) +{ + DumpState *s = dump_get_current(); + + if (value < MIN_THROTTLE) { + value = MIN_THROTTLE; + } + s->bandwidth = value; + return; +} diff --git a/hmp-commands.hx b/hmp-commands.hx index 57f5a20..e71a174 100644 --- a/hmp-commands.hx +++ b/hmp-commands.hx @@ -858,6 +858,21 @@ Cancel the current VM dumping. ETEXI { + .name = "dump_set_speed", + .args_type = "value:o", + .params = "value", + .help = "set maximum speed (in bytes) for dumping. " + "Defaults to MB if no size suffix is specified, ie. B/K/M/G/T", + .mhandler.cmd = hmp_dump_set_speed, + }, + +STEXI +@item dump_set_speed @var{value} +@findex dump_set_speed +Set maximum speed to @var{value} (in bytes) for dumping. +ETEXI + + { .name = "snapshot_blkdev", .args_type = "device:B,snapshot-file:s?,format:s?", .params = "device [new-image-file] [format]", diff --git a/hmp.c b/hmp.c index bca411b..4c015f9 100644 --- a/hmp.c +++ b/hmp.c @@ -696,3 +696,9 @@ void hmp_dump_cancel(Monitor *mon, const QDict *qdict) { qmp_dump_cancel(NULL); } + +void hmp_dump_set_speed(Monitor *mon, const QDict *qdict) +{ + int64_t value = qdict_get_int(qdict, "value"); + qmp_dump_set_speed(value, NULL); +} diff --git a/hmp.h b/hmp.h index c0037b6..2d71343 100644 --- a/hmp.h +++ b/hmp.h @@ -51,5 +51,6 @@ void hmp_migrate_set_downtime(Monitor *mon, const QDict *qdict); void hmp_migrate_set_speed(Monitor *mon, const QDict *qdict); void hmp_dump(Monitor *mon, const QDict *qdict); void hmp_dump_cancel(Monitor *mon, const QDict *qdict); +void hmp_dump_set_speed(Monitor *mon, const QDict *qdict); #endif diff --git a/qapi-schema.json b/qapi-schema.json index bbd2011..ec19631 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -1301,3 +1301,18 @@ # Since: 1.1 ## { 'command': 'dump_cancel' } + +## +# @dump_set_speed +# +# Set maximum speed for dumping. +# +# @value: maximum speed in bytes. +# +# Returns: nothing on success +# +# Notes: A value lesser than 1024 will be automatically round up to 1024. +# +# Since: 1.1 +## +{ 'command': 'dump_set_speed', 'data': {'value': 'int'} } diff --git a/qmp-commands.hx b/qmp-commands.hx index 055072d..509273e 100644 --- a/qmp-commands.hx +++ b/qmp-commands.hx @@ -620,6 +620,28 @@ Example: EQMP { + .name = "dump_set_speed", + .args_type = "value:o", + .mhandler.cmd_new = qmp_marshal_input_dump_set_speed, + }, + +SQMP +dump_set_speed + +Set maximum speed for dumping. + +Arguments: + +- "value": maximum speed, in bytes per second (json-int) + +Example: + +-> { "execute": "dump_set_speed", "arguments": { "value": 1024 } } +<- { "return": {} } + +EQMP + + { .name = "netdev_add", .args_type = "netdev:O", .params = "[user|tap|socket],id=str[,prop=value][,...]",