Re: [Qemu-devel] [PATCH v3 0/5] block: allow partial info-block query

[Max Reitz <mreitz@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1855 bytes --]

On 14.12.2015 20:55, John Snow wrote:
> Allow info-block to succeed even when it cannot reconstruct filenames,
> but be explicit about this failure. Utilities that rely on the current
> behavior are modified to report this explicit failure to users.
> 
> v3:
>  - Fix qemu-img output such that "backing file IS the absolute backing path"
>    and "absolute backing path was indeterminate" are distinguishable via
>    a new error message present in block/qapi.c; see patch 3.
>  - Output for test 110 modified once more to expect the above error.
> 
> v2:
>  - Fix qemu-img from now choking when it gets a partial response.
> 
> ________________________________________________________________________________
> 
> For convenience, this branch is available at:
> https://github.com/jnsnow/qemu.git branch block-allow-partial-query
> https://github.com/jnsnow/qemu/tree/block-allow-partial-query
> 
> This version is tagged block-allow-partial-query-v3:
> https://github.com/jnsnow/qemu/releases/tag/block-allow-partial-query-v3
> 
> John Snow (5):
>   block/qapi: do not redundantly print "actual path"
>   block/qapi: always report full_backing_filename
>   block/qapi: explicitly warn if !has_full_backing_filename
>   qemu-img: abort when full_backing_filename not present
>   block/qapi: allow best-effort query
> 
>  block/qapi.c               | 19 ++++++++++++-------
>  qemu-img.c                 |  5 ++++-
>  tests/qemu-iotests/043.out |  2 ++
>  tests/qemu-iotests/110.out |  5 ++++-
>  4 files changed, 22 insertions(+), 9 deletions(-)

Due to Markus not being CC'd and the get_maintainers script giving both
him and Kevin, I just went ahead and...

...fixed patch 3's commit message and applied the series to my
block-next branch, thanks:

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

Max


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]