Re: [Qemu-devel] [PATCH v3 00/12] block: Remove BB interface from blockdev-add/del

2016-09-21 Thread Kevin Wolf
Am 21.09.2016 um 14:55 hat Kevin Wolf geschrieben:
> This series makes the next step towards a QAPI interface that doesn't require
> clients to know about BlockBackends. By removing the support for 'id' from
> blockdev-add, it becomes a command that always only creates a BDS (with a node
> name). Existing interfaces have already been changed to accept node names
> everywhere and BlockBackends are created whenever they are needed.
> 
> The actual change is in the last patch and pretty trivial, but many test cases
> use blockdev-add to create a BlockBackend and must be updated first.

Applied to the block branch.

Kevin



[Qemu-devel] [PATCH v3 00/12] block: Remove BB interface from blockdev-add/del

2016-09-21 Thread Kevin Wolf
This series makes the next step towards a QAPI interface that doesn't require
clients to know about BlockBackends. By removing the support for 'id' from
blockdev-add, it becomes a command that always only creates a BDS (with a node
name). Existing interfaces have already been changed to accept node names
everywhere and BlockBackends are created whenever they are needed.

The actual change is in the last patch and pretty trivial, but many test cases
use blockdev-add to create a BlockBackend and must be updated first.

v3:
- Patch 8: Fixed whitespace error
- Patch 9: Mention dropped cases in commit message
- Patch 10: New: Fix NULL use for %s in error messages
- Patch 11 (was 10): Update output after new patch 10
- Patch 12 (was 11):
  Remove 'id' from blockdev-add error message
  Remove spurious # characters in QAPI documentation

v2:
- Rebased on top of qmp-commands.hx removal

Kevin Wolf (12):
  qemu-iotests/041: Avoid blockdev-add with id
  qemu-iotests/067: Avoid blockdev-add with id
  qemu-iotests/071: Avoid blockdev-add with id
  qemu-iotests/081: Avoid blockdev-add with id
  qemu-iotests/087: Avoid blockdev-add with id
  qemu-iotests/117: Avoid blockdev-add with id
  qemu-iotests/118: Avoid blockdev-add with id
  qemu-iotests/124: Avoid blockdev-add with id
  qemu-iotests/139: Avoid blockdev-add with id
  block: Avoid printing NULL string in error messages
  qemu-iotests/141: Avoid blockdev-add with id
  block: Remove BB interface from blockdev-add/del

 blockdev.c | 131 
 docs/qmp-commands.txt  |  24 ++
 qapi/block-core.json   |  30 ++-
 tests/qemu-iotests/041 |  71 +++
 tests/qemu-iotests/067 |   6 +-
 tests/qemu-iotests/067.out | 211 +++--
 tests/qemu-iotests/071 |   8 +-
 tests/qemu-iotests/081 |   2 +-
 tests/qemu-iotests/085.out |   6 +-
 tests/qemu-iotests/087 |  62 ++---
 tests/qemu-iotests/087.out |   6 +-
 tests/qemu-iotests/117 |   4 +-
 tests/qemu-iotests/118 |   6 +-
 tests/qemu-iotests/124 |  17 ++--
 tests/qemu-iotests/139 | 178 --
 tests/qemu-iotests/139.out |   4 +-
 tests/qemu-iotests/141 |  24 +++---
 tests/qemu-iotests/141.out |  24 +++---
 18 files changed, 324 insertions(+), 490 deletions(-)

-- 
1.8.3.1