All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH RFC v3 0/8] I/O Throtting block filter driver
@ 2017-06-23 12:46 Manos Pitsidianakis
  2017-06-23 12:46 ` [Qemu-devel] [PATCH RFC v3 1/8] block: move ThrottleGroup membership to ThrottleGroupMember Manos Pitsidianakis
                   ` (8 more replies)
  0 siblings, 9 replies; 55+ messages in thread
From: Manos Pitsidianakis @ 2017-06-23 12:46 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-block, Stefan Hajnoczi, Kevin Wolf, Alberto Garcia

This patch series adds a filter driver that uses the existing I/O throttle
code. Throttle groups can be then created and managed with QOM.

The 'remove legacy I/O throttling' patch replaces existing interface (command
line, qmp, hmp) to use the throttle filter transparently so as to not break
backwards compatibility. The created throttle node is stored in a field in
BlockBackendPublic so that the legacy interface can access it. The patch
currently breaks test 129 which commits, mirrors and backups a device with
legacy throttling options. I assume the internal throttling node affects this,
so I'd like some advice on how to approach this.

Test 139 could be updated to include the throttle driver though I'm not sure if
it's needed.

I'm also not sure about the test I created for this (last patch).

v2: 
- Move throttle group membership to ThrottleGroupMember struct
v3:
- add ThrottleGroup object
- convert legacy interface to use the driver
- add tests


Manos Pitsidianakis (8):
  block: move ThrottleGroup membership to ThrottleGroupMember
  block: Add aio_context field in ThrottleGroupMember
  block: add throttle block filter driver
  block: convert ThrottleGroup to object with QOM
  block: add BlockDevOptionsThrottle to QAPI
  block: add options parameter to bdrv_new_open_driver()
  block: remove legacy I/O throttling
  block: add throttle block filter driver interface tests

 block.c                         |  13 +-
 block/Makefile.objs             |   1 +
 block/block-backend.c           | 151 +++++----
 block/commit.c                  |   4 +-
 block/mirror.c                  |   2 +-
 block/qapi.c                    |   8 +-
 block/throttle-groups.c         | 660 ++++++++++++++++++++++++++++++----------
 block/throttle.c                | 431 ++++++++++++++++++++++++++
 block/vvfat.c                   |   2 +-
 blockdev.c                      |  55 +++-
 include/block/block.h           |   2 +-
 include/block/throttle-groups.h |  15 +-
 include/qemu/throttle-options.h |  60 ++--
 include/qemu/throttle.h         |  31 ++
 include/sysemu/block-backend.h  |  26 +-
 qapi/block-core.json            |  19 +-
 tests/qemu-iotests/184          | 144 +++++++++
 tests/qemu-iotests/184.out      |  31 ++
 tests/qemu-iotests/group        |   1 +
 tests/test-throttle.c           | 125 ++++----
 20 files changed, 1441 insertions(+), 340 deletions(-)
 create mode 100644 block/throttle.c
 create mode 100755 tests/qemu-iotests/184
 create mode 100644 tests/qemu-iotests/184.out

-- 
2.11.0

^ permalink raw reply	[flat|nested] 55+ messages in thread

end of thread, other threads:[~2017-06-28 16:18 UTC | newest]

Thread overview: 55+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-23 12:46 [Qemu-devel] [PATCH RFC v3 0/8] I/O Throtting block filter driver Manos Pitsidianakis
2017-06-23 12:46 ` [Qemu-devel] [PATCH RFC v3 1/8] block: move ThrottleGroup membership to ThrottleGroupMember Manos Pitsidianakis
2017-06-26 13:23   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-06-27 12:08   ` [Qemu-devel] " Alberto Garcia
2017-06-27 12:24     ` Manos Pitsidianakis
2017-06-23 12:46 ` [Qemu-devel] [PATCH RFC v3 2/8] block: Add aio_context field in ThrottleGroupMember Manos Pitsidianakis
2017-06-26 13:36   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-06-26 14:03     ` Manos Pitsidianakis
2017-06-27 12:39   ` [Qemu-devel] " Alberto Garcia
2017-06-28 11:27   ` Kevin Wolf
2017-06-28 12:15     ` Manos Pitsidianakis
2017-06-28 12:44       ` Kevin Wolf
2017-06-23 12:46 ` [Qemu-devel] [PATCH RFC v3 3/8] block: add throttle block filter driver Manos Pitsidianakis
2017-06-26 14:00   ` [Qemu-devel] [Qemu-block] " Manos Pitsidianakis
2017-06-26 14:30   ` Stefan Hajnoczi
2017-06-26 16:01     ` Manos Pitsidianakis
2017-06-27 12:42       ` Stefan Hajnoczi
2017-06-26 16:26     ` Manos Pitsidianakis
2017-06-27 12:45       ` Stefan Hajnoczi
2017-06-27 13:34         ` Manos Pitsidianakis
2017-06-28 12:11           ` Stefan Hajnoczi
2017-06-26 14:34   ` Stefan Hajnoczi
2017-06-28 14:40   ` [Qemu-devel] " Kevin Wolf
2017-06-28 15:22     ` Manos Pitsidianakis
2017-06-28 15:36       ` Kevin Wolf
2017-06-28 15:50         ` Manos Pitsidianakis
2017-06-23 12:46 ` [Qemu-devel] [PATCH RFC v3 4/8] block: convert ThrottleGroup to object with QOM Manos Pitsidianakis
2017-06-26 14:52   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-06-26 15:24     ` Manos Pitsidianakis
2017-06-27 12:57       ` Stefan Hajnoczi
2017-06-26 16:58     ` Manos Pitsidianakis
2017-06-27 13:02       ` Stefan Hajnoczi
2017-06-27 16:05       ` Alberto Garcia
2017-06-27 16:12         ` Manos Pitsidianakis
2017-06-28 12:07         ` Stefan Hajnoczi
2017-06-23 12:46 ` [Qemu-devel] [PATCH RFC v3 5/8] block: add BlockDevOptionsThrottle to QAPI Manos Pitsidianakis
2017-06-26 14:55   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-06-27 13:12   ` [Qemu-devel] " Eric Blake
2017-06-28 13:35   ` Alberto Garcia
2017-06-28 13:42     ` Manos Pitsidianakis
2017-06-28 15:50   ` Kevin Wolf
2017-06-28 16:02     ` Eric Blake
2017-06-28 16:18       ` Kevin Wolf
2017-06-23 12:46 ` [Qemu-devel] [PATCH RFC v3 6/8] block: add options parameter to bdrv_new_open_driver() Manos Pitsidianakis
2017-06-26 15:11   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-06-28 15:55     ` Kevin Wolf
2017-06-28 13:42   ` [Qemu-devel] " Alberto Garcia
2017-06-28 13:47     ` Manos Pitsidianakis
2017-06-23 12:46 ` [Qemu-devel] [PATCH RFC v3 7/8] block: remove legacy I/O throttling Manos Pitsidianakis
2017-06-26 15:44   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-06-26 22:45     ` Manos Pitsidianakis
2017-06-27 13:08       ` Stefan Hajnoczi
2017-06-23 12:47 ` [Qemu-devel] [PATCH RFC v3 8/8] block: add throttle block filter driver interface tests Manos Pitsidianakis
2017-06-28 11:18   ` Kevin Wolf
2017-06-26 15:46 ` [Qemu-devel] [Qemu-block] [PATCH RFC v3 0/8] I/O Throtting block filter driver Stefan Hajnoczi

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.