qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Alberto Garcia <berto@igalia.com>
To: qemu-devel@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>, Alberto Garcia <berto@igalia.com>,
	qemu-block@nongnu.org, Markus Armbruster <armbru@redhat.com>,
	Max Reitz <mreitz@redhat.com>,
	Stefan Hajnoczi <stefanha@redhat.com>
Subject: [Qemu-devel] [PATCH v2 00/17] throttle: Allow I/O bursts for a user-defined period of time
Date: Thu, 18 Feb 2016 12:26:53 +0200	[thread overview]
Message-ID: <cover.1455788710.git.berto@igalia.com> (raw)

Hi,

here's a new version of the series that adds support for performing
I/O bursts for a user-defined period of time. Please follow the link
to the first version of the series for a complete description.

There are two important changes in this version:

a) The previous series was broken because the new parameters were
   missing from qmp-commands.hx. This is fixed now. [patch 10]
b) This series has new tests and documentation [patches 14 and 16]

I also added myself as maintainer of the throttling code [patch 17].

Regards,

Berto

v2:
- Patch 10: Add the new parameters to qmp-commands.hx
- Patch 14: New iotest for this feature
- Patch 15: Fix typo in the API documentation
- Patch 16: New document that explains the throttling infrastructure
- Patch 17: Add myself as maintainer of the throttling code

v1: https://lists.gnu.org/archive/html/qemu-block/2016-02/msg00210.html
- Initial version

Alberto Garcia (17):
  throttle: Make throttle_compute_timer() static
  throttle: Make throttle_conflicting() set errp
  throttle: Make throttle_max_is_missing_limit() set errp
  throttle: Make throttle_is_valid() set errp
  throttle: Set always an average value when setting a maximum value
  throttle: Merge all functions that check the configuration into one
  throttle: Use throttle_config_init() to initialize ThrottleConfig
  throttle: Add support for burst periods
  throttle: Add command-line settings to define the burst periods
  qapi: Add burst length parameters to block_set_io_throttle
  qapi: Add burst length fields to BlockDeviceInfo
  throttle: Check that burst_level leaks correctly
  throttle: Test throttle_compute_wait() during bursts
  qemu-iotests: Extend iotest 093 to test bursts
  qapi: Correct the name of the iops_rd parameter
  docs: Document the throttling infrastructure
  MAINTAINERS: Add myself as maintainer of the throttling code

 MAINTAINERS                |   9 ++
 block/qapi.c               |  20 ++++
 blockdev.c                 |  99 +++++++++++++-----
 docs/throttle.txt          | 252 +++++++++++++++++++++++++++++++++++++++++++++
 hmp.c                      |  12 +++
 include/qemu/throttle.h    |  55 +++++++---
 qapi/block-core.json       |  92 ++++++++++++++---
 qmp-commands.hx            |  25 +++--
 tests/qemu-iotests/093     |  65 +++++++++---
 tests/qemu-iotests/093.out |   4 +-
 tests/test-throttle.c      |  88 ++++++++++++----
 util/throttle.c            | 132 ++++++++++++++++--------
 12 files changed, 708 insertions(+), 145 deletions(-)
 create mode 100644 docs/throttle.txt

-- 
2.7.0

             reply	other threads:[~2016-02-18 10:28 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-18 10:26 Alberto Garcia [this message]
2016-02-18 10:26 ` [Qemu-devel] [PATCH v2 01/17] throttle: Make throttle_compute_timer() static Alberto Garcia
2016-02-18 10:26 ` [Qemu-devel] [PATCH v2 02/17] throttle: Make throttle_conflicting() set errp Alberto Garcia
2016-02-18 10:26 ` [Qemu-devel] [PATCH v2 03/17] throttle: Make throttle_max_is_missing_limit() " Alberto Garcia
2016-02-18 10:26 ` [Qemu-devel] [PATCH v2 04/17] throttle: Make throttle_is_valid() " Alberto Garcia
2016-02-18 10:26 ` [Qemu-devel] [PATCH v2 05/17] throttle: Set always an average value when setting a maximum value Alberto Garcia
2016-02-18 10:26 ` [Qemu-devel] [PATCH v2 06/17] throttle: Merge all functions that check the configuration into one Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 07/17] throttle: Use throttle_config_init() to initialize ThrottleConfig Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 08/17] throttle: Add support for burst periods Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 09/17] throttle: Add command-line settings to define the " Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 10/17] qapi: Add burst length parameters to block_set_io_throttle Alberto Garcia
2016-02-22 16:41   ` Eric Blake
2016-02-23  8:26     ` Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 11/17] qapi: Add burst length fields to BlockDeviceInfo Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 12/17] throttle: Check that burst_level leaks correctly Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 13/17] throttle: Test throttle_compute_wait() during bursts Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 14/17] qemu-iotests: Extend iotest 093 to test bursts Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 15/17] qapi: Correct the name of the iops_rd parameter Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 16/17] docs: Document the throttling infrastructure Alberto Garcia
2016-02-18 10:27 ` [Qemu-devel] [PATCH v2 17/17] MAINTAINERS: Add myself as maintainer of the throttling code Alberto Garcia
2016-02-22 13:47 ` [Qemu-devel] [PATCH v2 00/17] throttle: Allow I/O bursts for a user-defined period of time Kevin Wolf

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=cover.1455788710.git.berto@igalia.com \
    --to=berto@igalia.com \
    --cc=armbru@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).