qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [PATCH v4 0/4] Add 'x-blockdev-del' command


From: Max Reitz
Subject: Re: [Qemu-block] [PATCH v4 0/4] Add 'x-blockdev-del' command
Date: Wed, 4 Nov 2015 17:15:08 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0

On 02.11.2015 15:51, Alberto Garcia wrote:
> This version has only documentation and test fixes.
> 
> Most are cosmetic changes, but there were two missing wait_ready() and
> wait_until_completed() calls that could break one of the tests.
> 
> Regards,
> 
> Berto
> 
> v4:
> - patch 3: s/being used/in use/ [Max]
> - patch 3: add 'blockdev-add' command to the 'x-blockdev-del' example
>   in qmp-commands.hx [Max]
> - patch 4: clarify that the BlkDebug and BlkVerify are not meant to be
>   real use cases of those drivers, but only to test the sanity checks
>   of 'x-blockdev-del' [Max]
> - patch 4: fix addBlockDriverStateOverlay() documentation [Max]
> - patch 4: add missing wait_ready() and wait_until_completed() calls
> 
> v3: https://lists.gnu.org/archive/html/qemu-block/2015-10/msg00854.html
> - Remove the extra references added in v2 to the mirror and backup
>   jobs, and use op blockers instead (for the mirror case only).
> 
> v2: https://lists.gnu.org/archive/html/qemu-block/2015-10/msg00828.html
> - Rename it as 'x-blockdev-del' and label it as experimental.
> - Use two parameters instead of just one. If you try to delete a BDS,
>   it must not be attached to any backend.
> - New test cases.
> - Hold extra references during the mirror and backup block jobs.
> 
> v1: https://lists.gnu.org/archive/html/qemu-devel/2015-10/msg02999.html
> - Initial implementation
> 
> Alberto Garcia (4):
>   mirror: block all operations on the target image during the job
>   block: Add blk_get_refcnt()
>   block: Add 'x-blockdev-del' QMP command
>   iotests: Add tests for the x-blockdev-del command
> 
>  block/block-backend.c          |   5 +
>  block/mirror.c                 |   4 +
>  blockdev.c                     |  66 +++++++
>  include/sysemu/block-backend.h |   1 +
>  qapi/block-core.json           |  32 +++-
>  qmp-commands.hx                |  61 +++++-
>  tests/qemu-iotests/139         | 414 
> +++++++++++++++++++++++++++++++++++++++++
>  tests/qemu-iotests/139.out     |   5 +
>  tests/qemu-iotests/group       |   1 +
>  9 files changed, 585 insertions(+), 4 deletions(-)
>  create mode 100644 tests/qemu-iotests/139
>  create mode 100644 tests/qemu-iotests/139.out

Thanks, applied to my block tree:

https://github.com/XanClic/qemu/commits/block


Max

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]