From nobody Sat Apr 20 09:46:09 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=virtuozzo.com ARC-Seal: i=1; a=rsa-sha256; t=1573208393; cv=none; d=zoho.com; s=zohoarc; b=VOyKbmp+L1Czo6Wh5zlwFguuKsqHvTsav9ZbqNolcvY/3F2KD8Nm4XDyuQlFoeFD7EizkeLTCmcoVXAFaZABniKthRHN4HO1YOuYkcXk5W1ZY8Eag1EqkdmLeC5Gs2nPfSYLxM/Es+QS5u+iIfZ5D1noD72XFVmWzvR/qt9859Y= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1573208393; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=uSRM3X2le0Z9HkfZHBcDLV1YAaLYK5GKSUzcjRKARKU=; b=nvZ5np8On1yRfEdixpAfZgqh+Djur6VLBM2lpklMbgrtQv6Bws4YeTfCidDCksbd0i+z27Ijsjb2hkCbAvuFkzNt8WXJrV+alIQBBiIfc1bcxYiEPB9we8kSGF4sbq0m5Ou/GOez6pfcYIdhjKNae8mJM95Z7YrAE0d2nLhf36o= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1573208393613291.0537301757889; Fri, 8 Nov 2019 02:19:53 -0800 (PST) Received: from localhost ([::1]:51636 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iT1Ms-00081T-Ah for importer@patchew.org; Fri, 08 Nov 2019 05:19:50 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:46291) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iT1KI-00068z-1z for qemu-devel@nongnu.org; Fri, 08 Nov 2019 05:17:11 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iT1KG-0004xM-31 for qemu-devel@nongnu.org; Fri, 08 Nov 2019 05:17:09 -0500 Received: from relay.sw.ru ([185.231.240.75]:56108) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iT1K8-0004ua-UE; Fri, 08 Nov 2019 05:17:01 -0500 Received: from [10.94.3.0] (helo=kvm.qa.sw.ru) by relay.sw.ru with esmtp (Exim 4.92.3) (envelope-from ) id 1iT1K3-0002Yp-Os; Fri, 08 Nov 2019 13:16:55 +0300 From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Subject: [PATCH v2 1/2] qapi: add filter-node-name option to drive-mirror Date: Fri, 8 Nov 2019 13:16:54 +0300 Message-Id: <20191108101655.10611-2-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20191108101655.10611-1-vsementsov@virtuozzo.com> References: <20191108101655.10611-1-vsementsov@virtuozzo.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 185.231.240.75 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, pkrempa@redhat.com, libvir-list@redhat.com, armbru@redhat.com, philmd@redhat.com, qemu-devel@nongnu.org, mreitz@redhat.com, jsnow@redhat.com, vsementsov@virtuozzo.com, dinechin@redhat.com, den@openvz.org, mlevitsk@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" To correspond to blockdev-mirror command and make it possible to deprecate implicit filters in the next commit. Signed-off-by: Vladimir Sementsov-Ogievskiy --- qapi/block-core.json | 7 +++++++ blockdev.c | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/qapi/block-core.json b/qapi/block-core.json index aa97ee2641..93530d3a13 100644 --- a/qapi/block-core.json +++ b/qapi/block-core.json @@ -1992,6 +1992,12 @@ # @on-target-error: the action to take on an error on the target, # default 'report' (no limitations, since this applies to # a different block device than @device). +# +# @filter-node-name: the node name that should be assigned to the +# filter driver that the mirror job inserts into the gr= aph +# above @device. If this option is not given, a node na= me is +# autogenerated. (Since: 4.2) +# # @unmap: Whether to try to unmap target sectors where source has # only zero. If true, and target unallocated sectors will read as = zero, # target image sectors will be unmapped; otherwise, zeroes will be @@ -2022,6 +2028,7 @@ '*speed': 'int', '*granularity': 'uint32', '*buf-size': 'int', '*on-source-error': 'BlockdevOnError', '*on-target-error': 'BlockdevOnError', + '*filter-node-name': 'str', '*unmap': 'bool', '*copy-mode': 'MirrorCopyMode', '*auto-finalize': 'bool', '*auto-dismiss': 'bool' } } =20 diff --git a/blockdev.c b/blockdev.c index 8e029e9c01..2ca614c77f 100644 --- a/blockdev.c +++ b/blockdev.c @@ -4008,7 +4008,7 @@ void qmp_drive_mirror(DriveMirror *arg, Error **errp) arg->has_on_source_error, arg->on_source_error, arg->has_on_target_error, arg->on_target_error, arg->has_unmap, arg->unmap, - false, NULL, + arg->has_filter_node_name, arg->filter_node_nam= e, arg->has_copy_mode, arg->copy_mode, arg->has_auto_finalize, arg->auto_finalize, arg->has_auto_dismiss, arg->auto_dismiss, --=20 2.21.0 From nobody Sat Apr 20 09:46:09 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=virtuozzo.com ARC-Seal: i=1; a=rsa-sha256; t=1573208419; cv=none; d=zoho.com; s=zohoarc; b=PHjb0MqMYSzSthr8QNl8AD9IF08DNUYoPq1/W97vQfN9kjQwkZa9LAQ2DxEYOoTYGYIUk1UGNl5/3CzMl86fgXN2IYLLsIdKEELIsUiodO79O2M4O/yZ/7tzLfQdCbdT9Ucslp9K6mdojenuofbCI0Qu/p91trZm4xX9IZKdAUg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1573208419; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=iPMa+LhKspwIaLqtSJwyWRkRpZE63uDcclVL+zD9A0Y=; b=NulviEDzUyoMle9mSO2KBVUX7rE/cpAQaluAUYxL2aY7CmU53ADnlm6MLlFIRM4YEGdQUGYLcmCqNJkBKFOZCVfGU/RisJU+NFjsanHuZbNvb8hFw/BbgUMQb93su6xYZiGR1lwKiHYBp31sl+ECF5tBuhQVpEZD6vN3X/Nbf+4= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1573208419643106.56526338010224; Fri, 8 Nov 2019 02:20:19 -0800 (PST) Received: from localhost ([::1]:51644 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iT1NK-0000JY-7A for importer@patchew.org; Fri, 08 Nov 2019 05:20:18 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:46335) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iT1KN-0006AZ-Rq for qemu-devel@nongnu.org; Fri, 08 Nov 2019 05:17:18 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iT1KJ-0004yV-W5 for qemu-devel@nongnu.org; Fri, 08 Nov 2019 05:17:15 -0500 Received: from relay.sw.ru ([185.231.240.75]:56102) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iT1K8-0004uZ-UC; Fri, 08 Nov 2019 05:17:01 -0500 Received: from [10.94.3.0] (helo=kvm.qa.sw.ru) by relay.sw.ru with esmtp (Exim 4.92.3) (envelope-from ) id 1iT1K3-0002Yp-TA; Fri, 08 Nov 2019 13:16:55 +0300 From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Subject: [PATCH v2 2/2] qapi: deprecate implicit filters Date: Fri, 8 Nov 2019 13:16:55 +0300 Message-Id: <20191108101655.10611-3-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20191108101655.10611-1-vsementsov@virtuozzo.com> References: <20191108101655.10611-1-vsementsov@virtuozzo.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 185.231.240.75 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, pkrempa@redhat.com, libvir-list@redhat.com, armbru@redhat.com, philmd@redhat.com, qemu-devel@nongnu.org, mreitz@redhat.com, jsnow@redhat.com, vsementsov@virtuozzo.com, dinechin@redhat.com, den@openvz.org, mlevitsk@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" To get rid of implicit filters related workarounds in future let's deprecate them now. Deprecation warning breaks some bash iotests output, so fix it here too: in most of cases just add filter-node-name in test. In 161 add FIXME and deprecation warning into 161.out. In 249, the test case is changed, as we don't need to test "default" filter node name anymore. Signed-off-by: Vladimir Sementsov-Ogievskiy --- qemu-deprecated.texi | 6 ++++++ qapi/block-core.json | 9 ++++++--- include/block/block_int.h | 10 +++++++++- blockdev.c | 10 ++++++++++ tests/qemu-iotests/094 | 1 + tests/qemu-iotests/095 | 6 ++++-- tests/qemu-iotests/109 | 1 + tests/qemu-iotests/127 | 1 + tests/qemu-iotests/141 | 5 ++++- tests/qemu-iotests/144 | 3 ++- tests/qemu-iotests/156 | 1 + tests/qemu-iotests/161 | 7 +++++++ tests/qemu-iotests/161.out | 1 + tests/qemu-iotests/185 | 3 +++ tests/qemu-iotests/191 | 2 ++ tests/qemu-iotests/229 | 1 + tests/qemu-iotests/247 | 8 +++++--- tests/qemu-iotests/249 | 5 +++-- tests/qemu-iotests/249.out | 2 +- 19 files changed, 68 insertions(+), 14 deletions(-) diff --git a/qemu-deprecated.texi b/qemu-deprecated.texi index 296bfc93a3..c969faf55a 100644 --- a/qemu-deprecated.texi +++ b/qemu-deprecated.texi @@ -204,6 +204,12 @@ and accurate ``query-qmp-schema'' command. Character devices creating sockets in client mode should not specify the 'wait' field, which is only applicable to sockets in server mode =20 +@subsection implicit filters in mirror and commit (since 4.2) + +Mirror and commit jobs insert filters, which becomes implicit if user +omitted @option(filter-node-name) parameter. So omitting it is deprecated +in ``blockdev-mirror'', ``drive-mirror'' and ``block-commit'', set it alwa= ys. + @section Human Monitor Protocol (HMP) commands =20 @subsection The hub_id parameter of 'hostfwd_add' / 'hostfwd_remove' (sinc= e 3.1) diff --git a/qapi/block-core.json b/qapi/block-core.json index 93530d3a13..37caed775f 100644 --- a/qapi/block-core.json +++ b/qapi/block-core.json @@ -1659,7 +1659,8 @@ # @filter-node-name: the node name that should be assigned to the # filter driver that the commit job inserts into the gr= aph # above @top. If this option is not given, a node name = is -# autogenerated. (Since: 2.9) +# autogenerated. Omitting this option is deprecated, it= will +# be required in future. (Since: 2.9) # # @auto-finalize: When false, this job will wait in a PENDING state after = it has # finished its work, waiting for @block-job-finalize before @@ -1996,7 +1997,8 @@ # @filter-node-name: the node name that should be assigned to the # filter driver that the mirror job inserts into the gr= aph # above @device. If this option is not given, a node na= me is -# autogenerated. (Since: 4.2) +# autogenerated. Omitting this option is deprecated, it= will +# be required in future. (Since: 4.2) # # @unmap: Whether to try to unmap target sectors where source has # only zero. If true, and target unallocated sectors will read as = zero, @@ -2311,7 +2313,8 @@ # @filter-node-name: the node name that should be assigned to the # filter driver that the mirror job inserts into the gr= aph # above @device. If this option is not given, a node na= me is -# autogenerated. (Since: 2.9) +# autogenerated. Omitting this option is deprecated, it= will +# be required in future. (Since: 2.9) # # @copy-mode: when to copy data to the destination; defaults to 'backgroun= d' # (Since: 3.0) diff --git a/include/block/block_int.h b/include/block/block_int.h index dd033d0b37..48ff3af48d 100644 --- a/include/block/block_int.h +++ b/include/block/block_int.h @@ -793,7 +793,15 @@ struct BlockDriverState { bool sg; /* if true, the device is a /dev/sg* */ bool probed; /* if true, format was probed rather than specified */ bool force_share; /* if true, always allow all shared permissions */ - bool implicit; /* if true, this filter node was automatically inserte= d */ + + /* + * @implicit field is deprecated, don't set it to true for new filters. + * If true, this filter node was automatically inserted and user don't + * know about it and unprepared for any effects of it. So, implicit + * filters are workarounded and skipped in many places of the block + * layer code. + */ + bool implicit; =20 BlockDriver *drv; /* NULL means no media */ void *opaque; diff --git a/blockdev.c b/blockdev.c index 2ca614c77f..8c3a409c94 100644 --- a/blockdev.c +++ b/blockdev.c @@ -3333,6 +3333,11 @@ void qmp_block_commit(bool has_job_id, const char *j= ob_id, const char *device, BlockdevOnError on_error =3D BLOCKDEV_ON_ERROR_REPORT; int job_flags =3D JOB_DEFAULT; =20 + if (!has_filter_node_name) { + warn_report("Omitting filter-node-name parameter is deprecated, it= " + "will be required in future"); + } + if (!has_speed) { speed =3D 0; } @@ -3780,6 +3785,11 @@ static void blockdev_mirror_common(const char *job_i= d, BlockDriverState *bs, { int job_flags =3D JOB_DEFAULT; =20 + if (!has_filter_node_name) { + warn_report("Omitting filter-node-name parameter is deprecated, it= " + "will be required in future"); + } + if (!has_speed) { speed =3D 0; } diff --git a/tests/qemu-iotests/094 b/tests/qemu-iotests/094 index 9343e09492..07017d5821 100755 --- a/tests/qemu-iotests/094 +++ b/tests/qemu-iotests/094 @@ -61,6 +61,7 @@ _send_qemu_cmd $QEMU_HANDLE \ _send_qemu_cmd $QEMU_HANDLE \ "{'execute': 'drive-mirror', 'arguments': {'device': 'src', + 'filter-node-name': 'filter0', 'target': '$TEST_IMG', 'format': 'nbd', 'sync':'full', diff --git a/tests/qemu-iotests/095 b/tests/qemu-iotests/095 index 58fe174b5e..3dc206ba9d 100755 --- a/tests/qemu-iotests/095 +++ b/tests/qemu-iotests/095 @@ -69,8 +69,10 @@ h=3D$QEMU_HANDLE _send_qemu_cmd $h "{ 'execute': 'qmp_capabilities' }" "return" =20 _send_qemu_cmd $h "{ 'execute': 'block-commit', - 'arguments': { 'device': 'test', - 'top': '"${TEST_IMG}.snp1"' } }" '"status= ": "null"' + 'arguments': { 'device': 'test', + 'filter-node-name': 'filter0', + 'top': '"${TEST_IMG}.snp1"' } }" \ + '"status": "null"' =20 _cleanup_qemu =20 diff --git a/tests/qemu-iotests/109 b/tests/qemu-iotests/109 index 9897ceb6cd..aa7da57f8c 100755 --- a/tests/qemu-iotests/109 +++ b/tests/qemu-iotests/109 @@ -57,6 +57,7 @@ run_qemu() =20 _send_qemu_cmd $QEMU_HANDLE \ "{'execute':'drive-mirror', 'arguments':{ + 'filter-node-name': 'filter0', 'device': 'src', 'target': '$raw_img', $qmp_format 'mode': 'existing', 'sync': 'full'}}" \ "return" diff --git a/tests/qemu-iotests/127 b/tests/qemu-iotests/127 index b64926ab31..b2ba9c66cb 100755 --- a/tests/qemu-iotests/127 +++ b/tests/qemu-iotests/127 @@ -70,6 +70,7 @@ _send_qemu_cmd $QEMU_HANDLE \ "{ 'execute': 'drive-mirror', 'arguments': { 'job-id': 'mirror', + 'filter-node-name': 'filter0', 'device': 'source', 'target': '$TEST_IMG.overlay1', 'mode': 'existing', diff --git a/tests/qemu-iotests/141 b/tests/qemu-iotests/141 index 8c2ae79f2b..178cee460d 100755 --- a/tests/qemu-iotests/141 +++ b/tests/qemu-iotests/141 @@ -123,6 +123,7 @@ echo test_blockjob \ "{'execute': 'drive-mirror', 'arguments': {'job-id': 'job0', + 'filter-node-name': 'filter0', 'device': 'drv0', 'target': '$TEST_DIR/o.$IMGFMT', 'format': '$IMGFMT', @@ -140,7 +141,8 @@ echo =20 test_blockjob \ "{'execute': 'block-commit', - 'arguments': {'job-id': 'job0', 'device': 'drv0'}}" \ + 'arguments': {'job-id': 'job0', 'filter-node-name': 'filter0', + 'device': 'drv0'}}" \ 'BLOCK_JOB_READY' \ '"status": "null"' =20 @@ -157,6 +159,7 @@ $QEMU_IO -c 'write 0 1M' "$TEST_DIR/m.$IMGFMT" | _filte= r_qemu_io test_blockjob \ "{'execute': 'block-commit', 'arguments': {'job-id': 'job0', + 'filter-node-name': 'filter0', 'device': 'drv0', 'top': '$TEST_DIR/m.$IMGFMT', 'speed': 1}}" \ diff --git a/tests/qemu-iotests/144 b/tests/qemu-iotests/144 index 011ed4f2bc..de60b0ccba 100755 --- a/tests/qemu-iotests/144 +++ b/tests/qemu-iotests/144 @@ -83,7 +83,8 @@ echo # Block commit on active layer, push the new overlay into base _send_qemu_cmd $h "{ 'execute': 'block-commit', 'arguments': { - 'device': 'virtio0' + 'device': 'virtio0', + 'filter-node-name': 'filt= er0' } }" "READY" =20 diff --git a/tests/qemu-iotests/156 b/tests/qemu-iotests/156 index 2ffa3ca942..2dc50444db 100755 --- a/tests/qemu-iotests/156 +++ b/tests/qemu-iotests/156 @@ -87,6 +87,7 @@ TEST_IMG=3D"$TEST_IMG.target.overlay" _make_test_img -u -= b "$TEST_IMG.target" 1M _send_qemu_cmd $QEMU_HANDLE \ "{ 'execute': 'drive-mirror', 'arguments': { 'device': 'source', + 'filter-node-name': 'filter0', 'target': '$TEST_IMG.target.overlay', 'mode': 'existing', 'sync': 'top' } }" \ diff --git a/tests/qemu-iotests/161 b/tests/qemu-iotests/161 index 456a4bd8c4..9cf3e96c74 100755 --- a/tests/qemu-iotests/161 +++ b/tests/qemu-iotests/161 @@ -120,6 +120,13 @@ _send_qemu_cmd $QEMU_HANDLE \ 'top': '${TEST_IMG}.int' } }" \ 'return' =20 +# FIXME +# Need to use filter-node-name in previous command, as it's omitting is +# deprecated, as described by warning in test output. But if just put +# filter-node-name into previous command the following reopen is failed wi= th +# messasge "Cannot change the option 'backing.detect-zeroes". It's a bug a= nd +# shoud be fixed. + # Wait for block-commit to finish sleep 0.5 =20 diff --git a/tests/qemu-iotests/161.out b/tests/qemu-iotests/161.out index 39951993ee..497f98f3c3 100644 --- a/tests/qemu-iotests/161.out +++ b/tests/qemu-iotests/161.out @@ -27,6 +27,7 @@ Formatting 'TEST_DIR/t.IMGFMT.base', fmt=3DIMGFMT size=3D= 1048576 Formatting 'TEST_DIR/t.IMGFMT.int', fmt=3DIMGFMT size=3D1048576 backing_fi= le=3DTEST_DIR/t.IMGFMT.base Formatting 'TEST_DIR/t.IMGFMT', fmt=3DIMGFMT size=3D1048576 backing_file= =3DTEST_DIR/t.IMGFMT.int {"return": {}} +warning: Omitting filter-node-name parameter is deprecated, it will be req= uired in future {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event"= : "JOB_STATUS_CHANGE", "data": {"status": "created", "id": "none0"}} {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event"= : "JOB_STATUS_CHANGE", "data": {"status": "running", "id": "none0"}} {"return": {}} diff --git a/tests/qemu-iotests/185 b/tests/qemu-iotests/185 index 454ff600cc..65ef64f0c0 100755 --- a/tests/qemu-iotests/185 +++ b/tests/qemu-iotests/185 @@ -107,6 +107,7 @@ echo _send_qemu_cmd $h \ "{ 'execute': 'block-commit', 'arguments': { 'device': 'disk', + 'filter-node-name': 'filter0', 'base':'$TEST_IMG.base', 'top': '$TEST_IMG.mid', 'speed': 65536 } }" \ @@ -132,6 +133,7 @@ _send_qemu_cmd $h "{ 'execute': 'qmp_capabilities' }" '= return' _send_qemu_cmd $h \ "{ 'execute': 'block-commit', 'arguments': { 'device': 'disk', + 'filter-node-name': 'filter0', 'base':'$TEST_IMG.base', 'speed': 65536 } }" \ "return" @@ -154,6 +156,7 @@ _send_qemu_cmd $h "{ 'execute': 'qmp_capabilities' }" '= return' _send_qemu_cmd $h \ "{ 'execute': 'drive-mirror', 'arguments': { 'device': 'disk', + 'filter-node-name': 'filter0', 'target': '$TEST_IMG.copy', 'format': '$IMGFMT', 'sync': 'full', diff --git a/tests/qemu-iotests/191 b/tests/qemu-iotests/191 index 528022e8d8..79d572f6d5 100755 --- a/tests/qemu-iotests/191 +++ b/tests/qemu-iotests/191 @@ -76,6 +76,7 @@ echo _send_qemu_cmd $h \ "{ 'execute': 'block-commit', 'arguments': { 'job-id': 'commit0', + 'filter-node-name': 'filter0', 'device': 'top', 'base':'$TEST_IMG.base', 'top': '$TEST_IMG.mid' } }" \ @@ -124,6 +125,7 @@ echo _send_qemu_cmd $h \ "{ 'execute': 'block-commit', 'arguments': { 'job-id': 'commit0', + 'filter-node-name': 'filter0', 'device': 'top', 'base':'$TEST_IMG.base', 'top': '$TEST_IMG.mid' } }" \ diff --git a/tests/qemu-iotests/229 b/tests/qemu-iotests/229 index e18a464fe0..08a7887e90 100755 --- a/tests/qemu-iotests/229 +++ b/tests/qemu-iotests/229 @@ -69,6 +69,7 @@ echo _send_qemu_cmd $QEMU_HANDLE \ "{'execute': 'drive-mirror', 'arguments': {'device': 'testdisk', + 'filter-node-name': 'filter0', 'format': '$IMGFMT', 'target': '$DEST_IMG', 'sync': 'full', diff --git a/tests/qemu-iotests/247 b/tests/qemu-iotests/247 index c853b73819..394f4e221c 100755 --- a/tests/qemu-iotests/247 +++ b/tests/qemu-iotests/247 @@ -53,9 +53,11 @@ TEST_IMG=3D"$TEST_IMG.3" _make_test_img $size TEST_IMG=3D"$TEST_IMG.4" _make_test_img $size =20 (cat <