From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55704) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a8a0Z-0005tD-TO for qemu-devel@nongnu.org; Mon, 14 Dec 2015 15:46:12 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a8a0Y-0003s5-Qe for qemu-devel@nongnu.org; Mon, 14 Dec 2015 15:46:11 -0500 References: <1450122916-4706-1-git-send-email-jsnow@redhat.com> From: Max Reitz Message-ID: <566F2A88.60706@redhat.com> Date: Mon, 14 Dec 2015 21:46:00 +0100 MIME-Version: 1.0 In-Reply-To: <1450122916-4706-1-git-send-email-jsnow@redhat.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="BCNNQdDqPdSnbk8oEUCsIeqjjD3fW57Ha" Subject: Re: [Qemu-devel] [PATCH v3 0/5] block: allow partial info-block query List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: John Snow , qemu-block@nongnu.org Cc: kwolf@redhat.com, qemu-devel@nongnu.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --BCNNQdDqPdSnbk8oEUCsIeqjjD3fW57Ha Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: quoted-printable 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. >=20 > v3: > - Fix qemu-img output such that "backing file IS the absolute backing = path" > and "absolute backing path was indeterminate" are distinguishable vi= a > a new error message present in block/qapi.c; see patch 3. > - Output for test 110 modified once more to expect the above error. >=20 > v2: > - Fix qemu-img from now choking when it gets a partial response. >=20 > _______________________________________________________________________= _________ >=20 > 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 >=20 > This version is tagged block-allow-partial-query-v3: > https://github.com/jnsnow/qemu/releases/tag/block-allow-partial-query-v= 3 >=20 > 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 >=20 > 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... =2E..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 --BCNNQdDqPdSnbk8oEUCsIeqjjD3fW57Ha Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJWbyqIAAoJEDuxQgLoOKytmG0H/A9k7htksKsJyGS9llNdoXlH TFwuf1P5YWaR4pCxTPKrTIVFYBl1koN6uszrm2x8wJby76g+ns3Ux3sdMCvlQ3vV /xGygZGEdnktK0292Sc/EcnfnNLRxh1pfeuw+wKKMuYgmokXalAtndZqmEXbmzbo WTw1v54bDUJIxv/ZbLJuKXfXZ49NBnNYCv4ZPgB7Us2gZ+LmLT2STR9WZ9SRoicU uH/xAHxhOVNL5YvnqeYcOLg4Kje3tW2KzriVJl41zelMGgewTKiZH4QRc7uOMg7M 0GCektJ0lAjLaYFcGUp+x+qu5T1c/u+s6gpIj70UzeFWgSrHfVtnJAP1GyDC/c8= =vJoX -----END PGP SIGNATURE----- --BCNNQdDqPdSnbk8oEUCsIeqjjD3fW57Ha--