All of lore.kernel.org
 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 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.