{"id":809753,"url":"http://patchwork.ozlabs.org/api/1.0/patches/809753/?format=json","project":{"id":14,"url":"http://patchwork.ozlabs.org/api/1.0/projects/14/?format=json","name":"QEMU Development","link_name":"qemu-devel","list_id":"qemu-devel.nongnu.org","list_email":"qemu-devel@nongnu.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<1504541267-36954-2-git-send-email-pradeep.jagadeesh@huawei.com>","date":"2017-09-04T16:07:42","name":"[v10,1/6] throttle: factor out duplicate code","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"4140aef1b67a8c5fc74201af6fd5caf56e397ad9","submitter":{"id":69852,"url":"http://patchwork.ozlabs.org/api/1.0/people/69852/?format=json","name":"Pradeep Jagadeesh","email":"pradeepkiruvale@gmail.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/qemu-devel/patch/1504541267-36954-2-git-send-email-pradeep.jagadeesh@huawei.com/mbox/","series":[{"id":1415,"url":"http://patchwork.ozlabs.org/api/1.0/series/1415/?format=json","date":"2017-09-04T16:07:42","name":"fsdev: qmp interface for io throttling","version":10,"mbox":"http://patchwork.ozlabs.org/series/1415/mbox/"}],"check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/809753/checks/","tags":{},"headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"UnqyTmnT\"; dkim-atps=neutral"],"Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xmFCb1bB3z9sNr\n\tfor ;\n\tTue, 5 Sep 2017 02:09:15 +1000 (AEST)","from localhost ([::1]:54734 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t)\n\tid 1dotw1-0007OT-56\n\tfor incoming@patchwork.ozlabs.org; Mon, 04 Sep 2017 12:09:13 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:46862)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from ) id 1dotvJ-0007KJ-1d\n\tfor qemu-devel@nongnu.org; Mon, 04 Sep 2017 12:08:34 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from ) id 1dotvD-0008R0-MK\n\tfor qemu-devel@nongnu.org; Mon, 04 Sep 2017 12:08:28 -0400","from mail-wm0-x244.google.com ([2a00:1450:400c:c09::244]:33839)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.71) (envelope-from )\n\tid 1dotvD-0008QL-D8\n\tfor qemu-devel@nongnu.org; Mon, 04 Sep 2017 12:08:23 -0400","by mail-wm0-x244.google.com with SMTP id r75so722856wmf.1\n\tfor ; Mon, 04 Sep 2017 09:08:23 -0700 (PDT)","from localhost.localdomain.localdomain ([217.110.131.66])\n\tby smtp.gmail.com with ESMTPSA id\n\tx67sm976582wma.7.2017.09.04.09.08.21\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tMon, 04 Sep 2017 09:08:21 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=irDKxKGkaoOLI6Ox8CLrOXgSHf87WhGrI0kxMmsev4c=;\n\tb=UnqyTmnToqv4TMo5KAVTHxfitiwjlbHUcqcU8swDF2ZjXcTycV6SBS8F/vbN8JCvuc\n\tr6uTIWryLCeKTeOjXChFjxAlpFm3RwCGlqOG1heRz9jkHOA0BDAvbtDmhx7vUaGEWS33\n\tSV1mKBeL7Q2Xsm3NnnUqRc8SwNKdiCpZ6CGFY/D0bDd8gTtxISnve+sjo6ckIxisEO6x\n\t/hPvUr1JYXclUclzBtLOGFgWOJxf66de3t47uLeIUSBowyemMH4uDh4Pw5Q/NlNTIfTT\n\tfNDejbXqzhCIj3nButSZp/klaygG+u4C/klJnON745a5JqDqhL8wnOsZw55B0F/yzMki\n\t1mRQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=irDKxKGkaoOLI6Ox8CLrOXgSHf87WhGrI0kxMmsev4c=;\n\tb=lsg5XNYnLb8wokyzgSVCo12H73kWUppd5glL9QGLUB5KTAM9BNxy7ejCw+QKM64yxd\n\tV/GSU/catc0Xy5h7Lolzq/Rty8CPF21fHGM7LGQUtF7VAQXo+cZ+RqHQL6byrGKOMMvf\n\tbLmtEzsHz7AaWON7QPnBCNOzTClL5zF523X6LC5odP+znhLy+hFLuuuBv9xWwB5sutM6\n\tOcJe8U+rML1++UDoOyyiaAE8aWAHLKFjyN+vwBOPsBv5iuNMgzGN3KlT4yd9kHgu5ppo\n\taETiUIg6eXeeZhEpzobB4YsP35XXOWUMs2h8QnpnzRhCVZHdTfQu5OdK4c38epBDusWO\n\tPx1A==","X-Gm-Message-State":"AHPjjUiBmWVRthesedVMg4bpv2DM5c1hy0vJZ6KhYQRA0dCn9OzMew1x\n\tu1x8sIJXtsLjkg==","X-Google-Smtp-Source":"ADKCNb7XixnNj286/XXzsGFzTGpXy2Zzcg9DNxlKQ+U2UmvmgJWVzygoC59n9Ey51XNlDL50ZjbdYg==","X-Received":"by 10.28.45.209 with SMTP id t200mr794718wmt.168.1504541302294; \n\tMon, 04 Sep 2017 09:08:22 -0700 (PDT)","From":"Pradeep Jagadeesh ","X-Google-Original-From":"Pradeep Jagadeesh ","To":"\"eric blake\" ,\n\t\"greg kurz\" ","Date":"Mon, 4 Sep 2017 12:07:42 -0400","Message-Id":"<1504541267-36954-2-git-send-email-pradeep.jagadeesh@huawei.com>","X-Mailer":"git-send-email 1.8.3.1","In-Reply-To":"<1504541267-36954-1-git-send-email-pradeep.jagadeesh@huawei.com>","References":"<1504541267-36954-1-git-send-email-pradeep.jagadeesh@huawei.com>","X-detected-operating-system":"by eggs.gnu.org: Genre and OS details not\n\trecognized.","X-Received-From":"2a00:1450:400c:c09::244","Subject":"[Qemu-devel] [PATCH v10 1/6] throttle: factor out duplicate code","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"qemu-devel@nongnu.org, jani kokkonen ,\n\talberto garcia ,\n\tPradeep Jagadeesh ,\n\tMarkus Armbruster ","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t"},"content":"This patch factor out the duplicate throttle code that was present in\nblock and fsdev devices.\n\nSigned-off-by: Pradeep Jagadeesh \nReviewed-by: Alberto Garcia \n---\n blockdev.c | 44 +----------------------------------\n fsdev/qemu-fsdev-throttle.c | 44 ++---------------------------------\n include/qemu/throttle-options.h | 4 ++++\n include/qemu/throttle.h | 4 ++--\n include/qemu/typedefs.h | 1 +\n util/throttle.c | 51 +++++++++++++++++++++++++++++++++++++++++\n 6 files changed, 61 insertions(+), 87 deletions(-)","diff":"diff --git a/blockdev.c b/blockdev.c\nindex 02cd69b..780ae58 100644\n--- a/blockdev.c\n+++ b/blockdev.c\n@@ -388,49 +388,7 @@ static void extract_common_blockdev_options(QemuOpts *opts, int *bdrv_flags,\n }\n \n if (throttle_cfg) {\n- throttle_config_init(throttle_cfg);\n- throttle_cfg->buckets[THROTTLE_BPS_TOTAL].avg =\n- qemu_opt_get_number(opts, \"throttling.bps-total\", 0);\n- throttle_cfg->buckets[THROTTLE_BPS_READ].avg =\n- qemu_opt_get_number(opts, \"throttling.bps-read\", 0);\n- throttle_cfg->buckets[THROTTLE_BPS_WRITE].avg =\n- qemu_opt_get_number(opts, \"throttling.bps-write\", 0);\n- throttle_cfg->buckets[THROTTLE_OPS_TOTAL].avg =\n- qemu_opt_get_number(opts, \"throttling.iops-total\", 0);\n- throttle_cfg->buckets[THROTTLE_OPS_READ].avg =\n- qemu_opt_get_number(opts, \"throttling.iops-read\", 0);\n- throttle_cfg->buckets[THROTTLE_OPS_WRITE].avg =\n- qemu_opt_get_number(opts, \"throttling.iops-write\", 0);\n-\n- throttle_cfg->buckets[THROTTLE_BPS_TOTAL].max =\n- qemu_opt_get_number(opts, \"throttling.bps-total-max\", 0);\n- throttle_cfg->buckets[THROTTLE_BPS_READ].max =\n- qemu_opt_get_number(opts, \"throttling.bps-read-max\", 0);\n- throttle_cfg->buckets[THROTTLE_BPS_WRITE].max =\n- qemu_opt_get_number(opts, \"throttling.bps-write-max\", 0);\n- throttle_cfg->buckets[THROTTLE_OPS_TOTAL].max =\n- qemu_opt_get_number(opts, \"throttling.iops-total-max\", 0);\n- throttle_cfg->buckets[THROTTLE_OPS_READ].max =\n- qemu_opt_get_number(opts, \"throttling.iops-read-max\", 0);\n- throttle_cfg->buckets[THROTTLE_OPS_WRITE].max =\n- qemu_opt_get_number(opts, \"throttling.iops-write-max\", 0);\n-\n- throttle_cfg->buckets[THROTTLE_BPS_TOTAL].burst_length =\n- qemu_opt_get_number(opts, \"throttling.bps-total-max-length\", 1);\n- throttle_cfg->buckets[THROTTLE_BPS_READ].burst_length =\n- qemu_opt_get_number(opts, \"throttling.bps-read-max-length\", 1);\n- throttle_cfg->buckets[THROTTLE_BPS_WRITE].burst_length =\n- qemu_opt_get_number(opts, \"throttling.bps-write-max-length\", 1);\n- throttle_cfg->buckets[THROTTLE_OPS_TOTAL].burst_length =\n- qemu_opt_get_number(opts, \"throttling.iops-total-max-length\", 1);\n- throttle_cfg->buckets[THROTTLE_OPS_READ].burst_length =\n- qemu_opt_get_number(opts, \"throttling.iops-read-max-length\", 1);\n- throttle_cfg->buckets[THROTTLE_OPS_WRITE].burst_length =\n- qemu_opt_get_number(opts, \"throttling.iops-write-max-length\", 1);\n-\n- throttle_cfg->op_size =\n- qemu_opt_get_number(opts, \"throttling.iops-size\", 0);\n-\n+ throttle_parse_options(throttle_cfg, opts);\n if (!throttle_is_valid(throttle_cfg, errp)) {\n return;\n }\ndiff --git a/fsdev/qemu-fsdev-throttle.c b/fsdev/qemu-fsdev-throttle.c\nindex 49eebb5..0e6fb86 100644\n--- a/fsdev/qemu-fsdev-throttle.c\n+++ b/fsdev/qemu-fsdev-throttle.c\n@@ -16,6 +16,7 @@\n #include \"qemu/error-report.h\"\n #include \"qemu-fsdev-throttle.h\"\n #include \"qemu/iov.h\"\n+#include \"qemu/throttle-options.h\"\n \n static void fsdev_throttle_read_timer_cb(void *opaque)\n {\n@@ -31,48 +32,7 @@ static void fsdev_throttle_write_timer_cb(void *opaque)\n \n void fsdev_throttle_parse_opts(QemuOpts *opts, FsThrottle *fst, Error **errp)\n {\n- throttle_config_init(&fst->cfg);\n- fst->cfg.buckets[THROTTLE_BPS_TOTAL].avg =\n- qemu_opt_get_number(opts, \"throttling.bps-total\", 0);\n- fst->cfg.buckets[THROTTLE_BPS_READ].avg =\n- qemu_opt_get_number(opts, \"throttling.bps-read\", 0);\n- fst->cfg.buckets[THROTTLE_BPS_WRITE].avg =\n- qemu_opt_get_number(opts, \"throttling.bps-write\", 0);\n- fst->cfg.buckets[THROTTLE_OPS_TOTAL].avg =\n- qemu_opt_get_number(opts, \"throttling.iops-total\", 0);\n- fst->cfg.buckets[THROTTLE_OPS_READ].avg =\n- qemu_opt_get_number(opts, \"throttling.iops-read\", 0);\n- fst->cfg.buckets[THROTTLE_OPS_WRITE].avg =\n- qemu_opt_get_number(opts, \"throttling.iops-write\", 0);\n-\n- fst->cfg.buckets[THROTTLE_BPS_TOTAL].max =\n- qemu_opt_get_number(opts, \"throttling.bps-total-max\", 0);\n- fst->cfg.buckets[THROTTLE_BPS_READ].max =\n- qemu_opt_get_number(opts, \"throttling.bps-read-max\", 0);\n- fst->cfg.buckets[THROTTLE_BPS_WRITE].max =\n- qemu_opt_get_number(opts, \"throttling.bps-write-max\", 0);\n- fst->cfg.buckets[THROTTLE_OPS_TOTAL].max =\n- qemu_opt_get_number(opts, \"throttling.iops-total-max\", 0);\n- fst->cfg.buckets[THROTTLE_OPS_READ].max =\n- qemu_opt_get_number(opts, \"throttling.iops-read-max\", 0);\n- fst->cfg.buckets[THROTTLE_OPS_WRITE].max =\n- qemu_opt_get_number(opts, \"throttling.iops-write-max\", 0);\n-\n- fst->cfg.buckets[THROTTLE_BPS_TOTAL].burst_length =\n- qemu_opt_get_number(opts, \"throttling.bps-total-max-length\", 1);\n- fst->cfg.buckets[THROTTLE_BPS_READ].burst_length =\n- qemu_opt_get_number(opts, \"throttling.bps-read-max-length\", 1);\n- fst->cfg.buckets[THROTTLE_BPS_WRITE].burst_length =\n- qemu_opt_get_number(opts, \"throttling.bps-write-max-length\", 1);\n- fst->cfg.buckets[THROTTLE_OPS_TOTAL].burst_length =\n- qemu_opt_get_number(opts, \"throttling.iops-total-max-length\", 1);\n- fst->cfg.buckets[THROTTLE_OPS_READ].burst_length =\n- qemu_opt_get_number(opts, \"throttling.iops-read-max-length\", 1);\n- fst->cfg.buckets[THROTTLE_OPS_WRITE].burst_length =\n- qemu_opt_get_number(opts, \"throttling.iops-write-max-length\", 1);\n- fst->cfg.op_size =\n- qemu_opt_get_number(opts, \"throttling.iops-size\", 0);\n-\n+ throttle_parse_options(&fst->cfg, opts);\n throttle_is_valid(&fst->cfg, errp);\n }\n \ndiff --git a/include/qemu/throttle-options.h b/include/qemu/throttle-options.h\nindex 3133d1c..f63d38c 100644\n--- a/include/qemu/throttle-options.h\n+++ b/include/qemu/throttle-options.h\n@@ -10,6 +10,8 @@\n #ifndef THROTTLE_OPTIONS_H\n #define THROTTLE_OPTIONS_H\n \n+#include \"typedefs.h\"\n+\n #define THROTTLE_OPTS \\\n { \\\n .name = \"throttling.iops-total\",\\\n@@ -89,4 +91,6 @@\n .help = \"when limiting by iops max size of an I/O in bytes\",\\\n }\n \n+void throttle_parse_options(ThrottleConfig *, QemuOpts *);\n+\n #endif\ndiff --git a/include/qemu/throttle.h b/include/qemu/throttle.h\nindex d056008..75d930c 100644\n--- a/include/qemu/throttle.h\n+++ b/include/qemu/throttle.h\n@@ -89,10 +89,10 @@ typedef struct LeakyBucket {\n * However it allows to keep the code clean and the bucket field is reset to\n * zero at the right time.\n */\n-typedef struct ThrottleConfig {\n+struct ThrottleConfig {\n LeakyBucket buckets[BUCKETS_COUNT]; /* leaky buckets */\n uint64_t op_size; /* size of an operation in bytes */\n-} ThrottleConfig;\n+};\n \n typedef struct ThrottleState {\n ThrottleConfig cfg; /* configuration */\ndiff --git a/include/qemu/typedefs.h b/include/qemu/typedefs.h\nindex 39bc835..90fe0f9 100644\n--- a/include/qemu/typedefs.h\n+++ b/include/qemu/typedefs.h\n@@ -100,6 +100,7 @@ typedef struct uWireSlave uWireSlave;\n typedef struct VirtIODevice VirtIODevice;\n typedef struct Visitor Visitor;\n typedef struct node_info NodeInfo;\n+typedef struct ThrottleConfig ThrottleConfig;\n typedef void SaveStateHandler(QEMUFile *f, void *opaque);\n typedef int LoadStateHandler(QEMUFile *f, void *opaque, int version_id);\n \ndiff --git a/util/throttle.c b/util/throttle.c\nindex b2a52b8..95c2ecf 100644\n--- a/util/throttle.c\n+++ b/util/throttle.c\n@@ -27,6 +27,7 @@\n #include \"qemu/throttle.h\"\n #include \"qemu/timer.h\"\n #include \"block/aio.h\"\n+#include \"qemu/throttle-options.h\"\n \n /* This function make a bucket leak\n *\n@@ -502,3 +503,53 @@ void throttle_account(ThrottleState *ts, bool is_write, uint64_t size)\n }\n }\n \n+/* parse the throttle options\n+ *\n+ * @opts: qemu options\n+ * @throttle_cfg: throttle configuration\n+ */\n+void throttle_parse_options(ThrottleConfig *throttle_cfg, QemuOpts *opts)\n+{\n+ throttle_config_init(throttle_cfg);\n+ throttle_cfg->buckets[THROTTLE_BPS_TOTAL].avg =\n+ qemu_opt_get_number(opts, \"throttling.bps-total\", 0);\n+ throttle_cfg->buckets[THROTTLE_BPS_READ].avg =\n+ qemu_opt_get_number(opts, \"throttling.bps-read\", 0);\n+ throttle_cfg->buckets[THROTTLE_BPS_WRITE].avg =\n+ qemu_opt_get_number(opts, \"throttling.bps-write\", 0);\n+ throttle_cfg->buckets[THROTTLE_OPS_TOTAL].avg =\n+ qemu_opt_get_number(opts, \"throttling.iops-total\", 0);\n+ throttle_cfg->buckets[THROTTLE_OPS_READ].avg =\n+ qemu_opt_get_number(opts, \"throttling.iops-read\", 0);\n+ throttle_cfg->buckets[THROTTLE_OPS_WRITE].avg =\n+ qemu_opt_get_number(opts, \"throttling.iops-write\", 0);\n+\n+ throttle_cfg->buckets[THROTTLE_BPS_TOTAL].max =\n+ qemu_opt_get_number(opts, \"throttling.bps-total-max\", 0);\n+ throttle_cfg->buckets[THROTTLE_BPS_READ].max =\n+ qemu_opt_get_number(opts, \"throttling.bps-read-max\", 0);\n+ throttle_cfg->buckets[THROTTLE_BPS_WRITE].max =\n+ qemu_opt_get_number(opts, \"throttling.bps-write-max\", 0);\n+ throttle_cfg->buckets[THROTTLE_OPS_TOTAL].max =\n+ qemu_opt_get_number(opts, \"throttling.iops-total-max\", 0);\n+ throttle_cfg->buckets[THROTTLE_OPS_READ].max =\n+ qemu_opt_get_number(opts, \"throttling.iops-read-max\", 0);\n+ throttle_cfg->buckets[THROTTLE_OPS_WRITE].max =\n+ qemu_opt_get_number(opts, \"throttling.iops-write-max\", 0);\n+\n+ throttle_cfg->buckets[THROTTLE_BPS_TOTAL].burst_length =\n+ qemu_opt_get_number(opts, \"throttling.bps-total-max-length\", 1);\n+ throttle_cfg->buckets[THROTTLE_BPS_READ].burst_length =\n+ qemu_opt_get_number(opts, \"throttling.bps-read-max-length\", 1);\n+ throttle_cfg->buckets[THROTTLE_BPS_WRITE].burst_length =\n+ qemu_opt_get_number(opts, \"throttling.bps-write-max-length\", 1);\n+ throttle_cfg->buckets[THROTTLE_OPS_TOTAL].burst_length =\n+ qemu_opt_get_number(opts, \"throttling.iops-total-max-length\", 1);\n+ throttle_cfg->buckets[THROTTLE_OPS_READ].burst_length =\n+ qemu_opt_get_number(opts, \"throttling.iops-read-max-length\", 1);\n+ throttle_cfg->buckets[THROTTLE_OPS_WRITE].burst_length =\n+ qemu_opt_get_number(opts, \"throttling.iops-write-max-length\", 1);\n+\n+ throttle_cfg->op_size =\n+ qemu_opt_get_number(opts, \"throttling.iops-size\", 0);\n+}\n","prefixes":["v10","1/6"]}