qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Kevin Wolf <kwolf@redhat.com>
To: qemu-block@nongnu.org
Cc: kwolf@redhat.com, mreitz@redhat.com, jcody@redhat.com,
	armbru@redhat.com, eblake@redhat.com, qemu-devel@nongnu.org
Subject: [Qemu-devel] [PATCH v2 0/9] block: Remove BlockDriverState.blk
Date: Wed, 27 Apr 2016 15:20:22 +0200	[thread overview]
Message-ID: <1461763231-17598-1-git-send-email-kwolf@redhat.com> (raw)

This is the final patch series that is required before we can start allowing
setups with more than one BlockBackend per BlockDriverState.

Depends on 'block: Move I/O throttling to BlockBackend'.

v2:
- Patch 1 ('block: Use BdrvChild callbacks for change_media/resize')
  Bring back an accidentally removed comment [Max]

- Patch 2 ('block: User BdrvChild callback for device name')
  Documented BdrvChildRole.get_name() callback [Max]

- Patch 3 ('blockjob: Don't set iostatus of target') and
  Patch 4 ('Don't touch BDS iostatus')

  Replaces 'Use BdrvChild callbacks for iostatus operations'. The whole
  problematic iostatus manipulation of block jobs only affected the target node
  where it wasn't visible, and it doesn't make sense, so it can simply go away.

- Patch 6 ('block: Add bdrv_has_blk()') and
  Patch 7 ('block: Avoid bs->blk in bdrv_next()')

  Split differently to keep the bdrv_next() changes in one patch [Max]
  Return BDSes only once in bdrv_next() even with multiple BBs [Max]
  Use enum for iteration phases instead of magic int [Max]

- Patch 9 ('block: Remove BlockDriverState.blk')
  Use bdrv_get_parent_name() to keep error message unchanged [Max]

Kevin Wolf (9):
  block: Use BdrvChild callbacks for change_media/resize
  block: User BdrvChild callback for device name
  blockjob: Don't set iostatus of target
  blockjob: Don't touch BDS iostatus
  block: Remove bdrv_aio_multiwrite()
  block: Add bdrv_has_blk()
  block: Avoid bs->blk in bdrv_next()
  block: Don't return throttling info in query-named-block-nodes
  block: Remove BlockDriverState.blk

 block.c                        | 101 ++++++++++----------
 block/backup.c                 |  34 +------
 block/block-backend.c          | 116 +++++++++++++++--------
 block/commit.c                 |   7 --
 block/io.c                     | 207 ++---------------------------------------
 block/mirror.c                 |  38 ++------
 block/qapi.c                   |   6 +-
 block/snapshot.c               |  30 +++---
 block/stream.c                 |  10 +-
 blockdev.c                     |  19 ++--
 blockjob.c                     |   6 +-
 include/block/block.h          |  10 +-
 include/block/block_int.h      |  12 ++-
 include/block/blockjob.h       |   4 +-
 include/sysemu/block-backend.h |   3 +-
 migration/block.c              |   4 +-
 monitor.c                      |   6 +-
 qemu-io-cmds.c                 | 203 ----------------------------------------
 qmp.c                          |   5 +-
 tests/qemu-iotests/096         |   3 +-
 tests/qemu-iotests/100         | 145 -----------------------------
 tests/qemu-iotests/100.out     |  89 ------------------
 tests/qemu-iotests/136         |  20 +---
 tests/qemu-iotests/136.out     |   4 +-
 tests/qemu-iotests/group       |   2 +-
 trace-events                   |   2 -
 26 files changed, 218 insertions(+), 868 deletions(-)
 delete mode 100755 tests/qemu-iotests/100
 delete mode 100644 tests/qemu-iotests/100.out

-- 
1.8.3.1

             reply	other threads:[~2016-04-27 13:20 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-27 13:20 Kevin Wolf [this message]
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 1/9] block: Use BdrvChild callbacks for change_media/resize Kevin Wolf
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 2/9] block: User BdrvChild callback for device name Kevin Wolf
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 3/9] blockjob: Don't set iostatus of target Kevin Wolf
2016-05-06 12:01   ` Max Reitz
2016-05-06 12:32     ` Max Reitz
2016-05-06 13:31       ` Kevin Wolf
2016-05-06 13:40         ` Max Reitz
2016-05-06 14:12           ` Kevin Wolf
2016-05-11 15:02             ` Max Reitz
2016-05-06 14:34         ` Eric Blake
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 4/9] blockjob: Don't touch BDS iostatus Kevin Wolf
2016-05-06 13:37   ` Max Reitz
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 5/9] block: Remove bdrv_aio_multiwrite() Kevin Wolf
2016-05-06 12:29   ` Eric Blake
2016-05-12 20:18   ` Eric Blake
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 6/9] block: Add bdrv_has_blk() Kevin Wolf
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 7/9] block: Avoid bs->blk in bdrv_next() Kevin Wolf
2016-05-06 12:54   ` Max Reitz
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 8/9] block: Don't return throttling info in query-named-block-nodes Kevin Wolf
2016-04-27 13:20 ` [Qemu-devel] [PATCH v2 9/9] block: Remove BlockDriverState.blk Kevin Wolf
2016-05-17 14:28 ` [Qemu-devel] [PATCH v2 0/9] " 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=1461763231-17598-1-git-send-email-kwolf@redhat.com \
    --to=kwolf@redhat.com \
    --cc=armbru@redhat.com \
    --cc=eblake@redhat.com \
    --cc=jcody@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=qemu-block@nongnu.org \
    --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 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).