All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pradeep Jagadeesh <pradeepkiruvale@gmail.com>
To: eric blake <eblake@redhat.com>, greg kurz <groug@kaod.org>
Cc: Pradeep Jagadeesh <pradeep.jagadeesh@huawei.com>,
	alberto garcia <berto@igalia.com>,
	jani kokkonen <jani.kokkonen@huawei.com>,
	qemu-devel@nongnu.org
Subject: [Qemu-devel] [PATCH v3 0/4] fsdev: qmp interface for io throttling
Date: Tue,  2 May 2017 10:29:42 -0400	[thread overview]
Message-ID: <1493735386-39622-1-git-send-email-pradeep.jagadeesh@huawei.com> (raw)

These patches provide the qmp interface, to query the io throttle 
status of the all fsdev devices that are present in a vm.
also, it provides an interface to set the io throttle parameters of a
fsdev to a required value. some of the patches also remove the duplicate
code that was present in block and fsdev files. 

Pradeep Jagadeesh (4):
  Throttle: Create IOThrottle structure
  fsdev: QMP interface for throttling
  qmp: refactor duplicate code
  throttle: factor out duplicate code

 Makefile                        |   3 ++
 Makefile.objs                   |   4 +-
 blockdev.c                      |  97 ++-----------------------------------
 fsdev/qemu-fsdev-dummy.c        |  11 +++++
 fsdev/qemu-fsdev-throttle.c     |  90 ++++++++++++++++++----------------
 fsdev/qemu-fsdev-throttle.h     |  13 +++++
 fsdev/qemu-fsdev.c              |  36 ++++++++++++++
 hmp-commands-info.hx            |  19 ++++++++
 hmp-commands.hx                 |  19 ++++++++
 hmp.c                           |  87 ++++++++++++++++++++++++++++++---
 hmp.h                           |   5 ++
 include/qemu/throttle-options.h |   7 +++
 qapi-schema.json                |   3 ++
 qapi/block-core.json            |  76 ++---------------------------
 qapi/fsdev.json                 |  84 ++++++++++++++++++++++++++++++++
 qapi/iothrottle.json            |  87 +++++++++++++++++++++++++++++++++
 util/Makefile.objs              |   1 +
 util/throttle-options.c         | 105 ++++++++++++++++++++++++++++++++++++++++
 18 files changed, 531 insertions(+), 216 deletions(-)
 create mode 100644 qapi/fsdev.json
 create mode 100644 qapi/iothrottle.json
 create mode 100644 util/throttle-options.c

v0 -> v1:
 Addressed comments from Eric Blake, Greg Kurz and Daniel P.Berrange
 Mainly renaming the functions and removing the redundant code.

v1 -> v2:
 Addressed the comments from Eric Blake and Greg Kurz.
 As per the suggestion I split the patches into smaller patches.
 Removed some more duplicate code.

v2 -> v3:
 Addressed the comments from Alberto Garcia.
 Changed the comment from block to iothrottle in the iothrottle.json 
 Added the dummy functions in qemu-fsdev-dummy.c to address the compilation
 issues that were observed.

-- 
1.8.3.1

             reply	other threads:[~2017-05-02 14:30 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-02 14:29 Pradeep Jagadeesh [this message]
2017-05-02 14:29 ` [Qemu-devel] [PATCH v3 1/4] Throttle: Create IOThrottle structure Pradeep Jagadeesh
2017-05-02 21:56   ` Eric Blake
2017-05-03  6:22   ` Greg Kurz
2017-05-03 14:19     ` Pradeep Jagadeesh
2017-05-03 17:10       ` Greg Kurz
2017-05-02 14:29 ` [Qemu-devel] [PATCH v3 2/4] fsdev: QMP interface for throttling Pradeep Jagadeesh
2017-05-02 22:13   ` Eric Blake
2017-05-03 15:40     ` Pradeep Jagadeesh
2017-05-03 16:32       ` Eric Blake
2017-05-04 15:12         ` Pradeep Jagadeesh
2017-05-04 15:19           ` Eric Blake
2017-05-05 11:09             ` Pradeep Jagadeesh
2017-05-02 14:29 ` [Qemu-devel] [PATCH v3 3/4] qmp: refactor duplicate code Pradeep Jagadeesh
2017-05-02 22:15   ` Eric Blake
2017-05-03 17:21     ` Greg Kurz
2017-05-04  9:44       ` Pradeep Jagadeesh
2017-05-02 14:29 ` [Qemu-devel] [PATCH v3 4/4] throttle: factor out " Pradeep Jagadeesh
2017-05-02 15:03 ` [Qemu-devel] [PATCH v3 0/4] fsdev: qmp interface for io throttling no-reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1493735386-39622-1-git-send-email-pradeep.jagadeesh@huawei.com \
    --to=pradeepkiruvale@gmail.com \
    --cc=berto@igalia.com \
    --cc=eblake@redhat.com \
    --cc=groug@kaod.org \
    --cc=jani.kokkonen@huawei.com \
    --cc=pradeep.jagadeesh@huawei.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.