From nobody Sun Feb 8 18:44:16 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; arc=pass (i=1 dmarc=pass fromdomain=virtuozzo.com); dmarc=pass(p=quarantine dis=none) header.from=virtuozzo.com ARC-Seal: i=2; a=rsa-sha256; t=1645723030; cv=pass; d=zohomail.com; s=zohoarc; b=YukfsM08YXGPBFL7+qMUtC+HxGZ9wx7TDGXSQjauAdV/rSxNYZBvDuCgfwkK3OQtOuCnlhbGTM2NIQzVFhQVweJg7q2uDW/JdkHTkrDumtbRRnuB5q7uvlDTsOUmjmNkDXFCj0jlMMiKtNqIjkk43L28zYuEyrlC51jWDa5IO2M= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1645723030; h=Content-Type: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=5PEIUGCBX9t1e2g5owSOOuCbnMvA4gr6n9U2JaJhar8=; b=hiUkGenKqVPnh0V5RPuPmVBVyALFBLM9bKljvVtyun76CgtjIJELTfzEVCXtSUQlL88HjPpseDD9L/wHSKduBZ7Jl32e1mDuAOQKKFNQIajkpt3bMB8s+/Xn7JMdE6R2Ds10+2bRGNdc94KLUphz0DUkodubPTJZ3eElGWahICw= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; arc=pass (i=1 dmarc=pass fromdomain=virtuozzo.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1645723030766457.65281988214963; Thu, 24 Feb 2022 09:17:10 -0800 (PST) Received: from localhost ([::1]:50248 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1nNHjp-0007rx-4H for importer@patchew.org; Thu, 24 Feb 2022 12:17:09 -0500 Received: from eggs.gnu.org ([209.51.188.92]:60870) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1nNHgt-0005do-3p; Thu, 24 Feb 2022 12:14:07 -0500 Received: from [2a01:111:f400:fe1e::726] (port=14723 helo=EUR01-HE1-obe.outbound.protection.outlook.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1nNHgq-0005pu-NX; Thu, 24 Feb 2022 12:14:06 -0500 Received: from AM9PR08MB6737.eurprd08.prod.outlook.com (2603:10a6:20b:304::18) by VI1PR08MB3197.eurprd08.prod.outlook.com (2603:10a6:803:3e::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.22; Thu, 24 Feb 2022 17:13:54 +0000 Received: from AM9PR08MB6737.eurprd08.prod.outlook.com ([fe80::49c:67e9:3e24:8714]) by AM9PR08MB6737.eurprd08.prod.outlook.com ([fe80::49c:67e9:3e24:8714%3]) with mapi id 15.20.5017.021; Thu, 24 Feb 2022 17:13:54 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IaCIlnQLjIDvnYpF1D+w37nBWUHK1S/i4EQruM1EOn6BYZUMVyH8CXbHVa3waZhK3I++BtwO/xrWstgaQ7hSal5swzvZVDGUGpVymfALDMwV3fxrtGhzUYZnVUzaACSnvwD1pQgNFyOo6F4RjdoLme8yZ06HCuKzFmqF+O/I3kNUQtjlkKEzUYp4vMktqvKfMgYZzt8A/mftWesi1U+FVHxLrJlpx3An1wfpZucx5lvn8uXd55/sAUTXod5EweAn91ZqZxHXlwyM39ao0QFbrERXsunIkm3OyPeo4BypkeJRyHnTRi60E0bEBIMxQLOcteMDN3tfaCgfJxMKa0J1YA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=5PEIUGCBX9t1e2g5owSOOuCbnMvA4gr6n9U2JaJhar8=; b=WgSSSjaljyuLLC6sT3xl1LbeTjDSH2jMMeKQu//wb9O8JjC9UDTJlwojSaNfvcnyD+leWLf3Qo/Olq63Ycb3lkTmtvY5U21uvDc/8zwIvdnwvfyuAItMFULYzmtXaTGmKtJPvYklaDs0+DDaHh12T0xZ2PlFQb7MEPIFwYXvWbOWUEtqcbJM46YVbwefOZi/0WUI757xEfXc93UlSBiSRFYajKYSWATkGpqKxcguKBJkS+YFvwYjEMUcuojZzqTkfHTrihfi+coOvAHTUglDm7ehDfzP8SOSwiC4+wG+tGFTYmFDFUPZIpaWcyrqs66/OuugCIe+sRvcLlD6NNkQYA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=virtuozzo.com; dmarc=pass action=none header.from=virtuozzo.com; dkim=pass header.d=virtuozzo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=virtuozzo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=5PEIUGCBX9t1e2g5owSOOuCbnMvA4gr6n9U2JaJhar8=; b=JFG7tZ/TGCabAx1APmw0x1lX5YLk5k1AWI1N7cAEdg6zXEFNB/xtWab39bkJl4rrmTAvgj3/M9M8wtbxcFpgj0TTSTMP7t6z2w48WKO+jXN5hyoNK00lvhDcpKrunXNCZ910cUPWO1+zEoc7Uz2EbH3r/7XwgvqxO2Ngj7DD8jQ= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=virtuozzo.com; From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Cc: qemu-devel@nongnu.org, eblake@redhat.com, armbru@redhat.com, hreitz@redhat.com, kwolf@redhat.com, Vladimir Sementsov-Ogievskiy Subject: [PATCH 1/2] block: transaction support for blockdev-add Date: Thu, 24 Feb 2022 18:13:27 +0100 Message-Id: <20220224171328.1628047-2-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20220224171328.1628047-1-vsementsov@virtuozzo.com> References: <20220224171328.1628047-1-vsementsov@virtuozzo.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: VI1P195CA0016.EURP195.PROD.OUTLOOK.COM (2603:10a6:800:d0::26) To AM9PR08MB6737.eurprd08.prod.outlook.com (2603:10a6:20b:304::18) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 1acca4f6-453c-48a7-1624-08d9f7b908cd X-MS-TrafficTypeDiagnostic: VI1PR08MB3197:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 9NiMzOWQ/OmwpMnhLJAdx2BT3RaGamphQB/Lv0VXp7c2UMfmktkDSWLntn5MlIHAuKvjC9XQFT+ziVd1H7pkM8J/vUP3eFuNvqiOzD6kOwTAXcfhxRTs9VK3YGtKRuESlmnRLAxK3r7zleMm5qnA7QqsBOBT8/76K5VphBobpzMeDma7BNkumxVxc94faM9P5s1ZfCxtC9/urCuNpM0x87Xe4jW2z9idYUtC/dPSQGRgINRQAxSTX/odzPKkPW7DNAf+7NPHFaRbpGEnb+xGYF7r6F16TlKDHKhpclCMYmBKCtZH5gev3RqaJ8+ZYUqs5od4d5Q1uDIjfHxRew/c8a07KjbaFYc0Pmxf9HOB1wFrlA/CI/bxn8bNgJEzU2IToH8PxSsAxz1NdQPVA5viIKsGM0wedEaHQoFWKheWsQTAzWETimR8Wp8EW9qJX9iexrThq4iiViLcVq85kBxlvMHo4TnR/TT7B0tt8dbKrgd2CK01x2mxGsBUEBYGJqA9jaBQTCFAwQD3QzSnyz8LqTPK1P1S6dqo4nUiuCOfMIYyT4uPG5inI/Lp+QxNjEhOH4X40JyjrWxAxntGsSzbni7zE1YngvENtXfwrJ1cO22+m2EksZw6JVxvAxyl1fe9Ml/lRyuMQa6DWoTJQgttNzJmEUwuSP54ITmhBa+u9AST0esAUHhIXW4RzSw+bimOTEwQO95PBHEKmm/069SEww== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM9PR08MB6737.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(4636009)(366004)(6506007)(52116002)(508600001)(6666004)(2616005)(86362001)(38100700002)(38350700002)(6486002)(4326008)(8676002)(66476007)(66556008)(66946007)(6916009)(316002)(1076003)(8936002)(107886003)(26005)(2906002)(83380400001)(186003)(6512007)(5660300002)(36756003); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?R0xRfNxtJ5CG+y4lC9i6HttCA4nOH7rEJ36RVfbNGDuLhv7MldvxvZsooiV0?= =?us-ascii?Q?3q1fwLMkeWEjMfTb2pz/WgfjnvfNUjDVOTsWNJm41UrD8imoNdkfVY7lPIG6?= =?us-ascii?Q?eSvFFFlvTShFUNsv+Ow+qiR84/ycbMvgy05VyQsbeZuxznInHglpYRndxoQU?= =?us-ascii?Q?7hFbq7DHyykTDR/ltosY+VUx4T9UQZCMnps1migkkMCmTFNRBQhuhsId6n2V?= =?us-ascii?Q?JYd07Rb9TpJXvhBr/QwM+w5JboR4mwkjqqNf+hR6NxHNcTFoAQzkerrjWsek?= =?us-ascii?Q?JF2dKzc1ZsqqNtOOMpjtgjJ6RlQecfI75bQ9StP7VDrM2kbSNkJ9EBEe6J4y?= =?us-ascii?Q?TeAQJ82jKf9nxBIurZqhGFX8T5IKZ2bddBuntcKF7YsUEL5+2UN2A6/F1CWa?= =?us-ascii?Q?UJFw5MSp33HSi2T7X+hg/YN0rK0DM5DPagqAhG34UTJ2sEVKhUXsNajckwRv?= =?us-ascii?Q?Tmjat5vcm1PaZz9MUtseBEMsUWSdzi/NkBUKOUeRJ4CDbWyWgTuRY335qNHP?= =?us-ascii?Q?OdkNRn8dFYEaCKc7yIIICsEJqsTYzYV1j6+xNUJj0IBG8qynf5n8+bqVboFR?= =?us-ascii?Q?QaSNLBioDfxSKPzwLvKOZCvbm15whGBEMMGqUNjrDjpFdWTUrxYF0jEVZ8Gq?= =?us-ascii?Q?05DbA0mseIDfexMEaSTLPc3chMaR1o7SiLSkgUlvj0ZosbGFcj6G4xMksnFo?= =?us-ascii?Q?UFQMOEVCoeI4EbVxHRHAtIJcFnjgjSNbt0jD3aSzbgpw838HoqKf4psL6Nwl?= =?us-ascii?Q?WekKMG+YxQslX3KF4gJ4aNiZZOoCb0ySMGd0Ym3N5KK/TmTv9AM6TWdMtaEF?= =?us-ascii?Q?Azf4TTAelnZEaiIy0bZ0OihSZs7/KCWwapvc03LAF0Yb3BSRbWuihFF1BZRZ?= =?us-ascii?Q?FeOTMMWxe3saKkLvdjijBHNVlplvLTJEDbhf3UWzvGUBuEICSC4J2OovzxLo?= =?us-ascii?Q?9kRDj7wfG6wfFSp715MmEREVGeeFYA8w3R5xwRPmiQHKQRTdz3kQucRrggZ5?= =?us-ascii?Q?IisFWzvGChbppkdCGq1Xv45q3xxcV6YMEzMuxCT+xeHqb2qX2w7J5+noDalL?= =?us-ascii?Q?1vJoh7rF/IobiFiJJmNDV++hNDYswKqTuoHR8OkisE993KohO2/Wm8UfovoQ?= =?us-ascii?Q?2Gn8ZkcN442Txd7vK1xXD0JriGBQ7iI3J60xOn89zRT8JTuBRkMMTwgZL84Y?= =?us-ascii?Q?EieBmifVQD0ehu1MH2ekk2VO6BiU4OROdbuDhWvVobViOPLcAzejkwQVyCOC?= =?us-ascii?Q?6OYYtt5VQfTYYNZCve00qRQq8dd8rU6dDMo6Pnl23D6Xnus+WPP2FD9sQnNB?= =?us-ascii?Q?kLtt5HCclv/e3ZNWcVr4pCOpu48jWCTAnkeF+4hh5mMBebUbcOwF3wOVhiMe?= =?us-ascii?Q?6Qj6e4uttlqs6wte5F2zPJmmI0YEmwMFgzZ48fT4DK10sTrFUCCMIDT7zsAG?= =?us-ascii?Q?ergEbHSKI/69JrfSdjfOajd20ptfGgcgU2fJB6ZJmoswxDvmAuG0uNsjXG8V?= =?us-ascii?Q?JK+T2ZMn/3gtOe2eCn1JED+intl2u+MdV5pg4pgnUtcZcz9JaRDLG1UVYJq7?= =?us-ascii?Q?mfy0X7+NOW8BzzA8N2CMktkvDotihwBWJNy+ET3w7G8KzyF5tedk4E+wsQup?= =?us-ascii?Q?fgaoQTvQmn1JBk5S+SNAPb2bnPsKy4+GiSQu6WXDbdgSp3rKPQT3hehFJI1L?= =?us-ascii?Q?CNZEajT+UWeUtTnMLpfx9Nzz7F0=3D?= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1acca4f6-453c-48a7-1624-08d9f7b908cd X-MS-Exchange-CrossTenant-AuthSource: AM9PR08MB6737.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Feb 2022 17:13:54.0022 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 0bc7f26d-0264-416e-a6fc-8352af79c58f X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 0LhGpf/sR+RwTS/iJl06bTOFENUSWs4uXlJzaopt9UKLG9RxCrmYt5oyGubqUMsuDLvhNY03F6NGvc9D516WfKcoFiDlywXMC/9c6HuPHc8= X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB3197 X-Host-Lookup-Failed: Reverse DNS lookup failed for 2a01:111:f400:fe1e::726 (failed) Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a01:111:f400:fe1e::726; envelope-from=vsementsov@virtuozzo.com; helo=EUR01-HE1-obe.outbound.protection.outlook.com X-Spam_score_int: -6 X-Spam_score: -0.7 X-Spam_bar: / X-Spam_report: (-0.7 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, PDS_HP_HELO_NORDNS=0.659, RDNS_NONE=0.793, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @virtuozzo.com) X-ZM-MESSAGEID: 1645723032835100001 Content-Type: text/plain; charset="utf-8" Simply do blockdev_add() in .prepare() and bdrv_unref() in .abort() and that's it. Signed-off-by: Vladimir Sementsov-Ogievskiy --- qapi/transaction.json | 11 ++++++ blockdev.c | 80 +++++++++++++++++++++++++++++-------------- 2 files changed, 66 insertions(+), 25 deletions(-) diff --git a/qapi/transaction.json b/qapi/transaction.json index 381a2df782..a938dc7d10 100644 --- a/qapi/transaction.json +++ b/qapi/transaction.json @@ -53,6 +53,7 @@ # @blockdev-snapshot-internal-sync: Since 1.7 # @blockdev-snapshot-sync: since 1.1 # @drive-backup: Since 1.6 +# @blockdev-add: since 7.0 # # Features: # @deprecated: Member @drive-backup is deprecated. Use member @@ -66,6 +67,7 @@ 'block-dirty-bitmap-disable', 'block-dirty-bitmap-merge', 'blockdev-backup', 'blockdev-snapshot', 'blockdev-snapshot-internal-sync', 'blockdev-snapshot-sync', + 'blockdev-add', { 'name': 'drive-backup', 'features': [ 'deprecated' ] } ] } =20 ## @@ -140,6 +142,14 @@ { 'struct': 'DriveBackupWrapper', 'data': { 'data': 'DriveBackup' } } =20 +## +# @BlockdevAddWrapper: +# +# Since: 7.0 +## +{ 'struct': 'BlockdevAddWrapper', + 'data': { 'data': 'BlockdevOptions' } } + ## # @TransactionAction: # @@ -163,6 +173,7 @@ 'blockdev-snapshot': 'BlockdevSnapshotWrapper', 'blockdev-snapshot-internal-sync': 'BlockdevSnapshotInternalWrapper= ', 'blockdev-snapshot-sync': 'BlockdevSnapshotSyncWrapper', + 'blockdev-add': 'BlockdevAddWrapper', 'drive-backup': 'DriveBackupWrapper' } } =20 diff --git a/blockdev.c b/blockdev.c index 42e098b458..eb9ad9cb89 100644 --- a/blockdev.c +++ b/blockdev.c @@ -2180,6 +2180,55 @@ static void abort_commit(BlkActionState *common) g_assert_not_reached(); /* this action never succeeds */ } =20 +static BlockDriverState *blockdev_add(BlockdevOptions *options, Error **er= rp) +{ + BlockDriverState *bs =3D NULL; + QObject *obj; + Visitor *v =3D qobject_output_visitor_new(&obj); + QDict *qdict; + + visit_type_BlockdevOptions(v, NULL, &options, &error_abort); + visit_complete(v, &obj); + qdict =3D qobject_to(QDict, obj); + + qdict_flatten(qdict); + + if (!qdict_get_try_str(qdict, "node-name")) { + error_setg(errp, "'node-name' must be specified for the root node"= ); + goto fail; + } + + bs =3D bds_tree_init(qdict, errp); + if (!bs) { + goto fail; + } + + bdrv_set_monitor_owned(bs); + +fail: + visit_free(v); + return bs; +} + +typedef struct BlockdevAddState { + BlkActionState common; + BlockDriverState *bs; +} BlockdevAddState; + +static void blockdev_add_prepare(BlkActionState *common, Error **errp) +{ + BlockdevAddState *s =3D DO_UPCAST(BlockdevAddState, common, common); + + s->bs =3D blockdev_add(common->action->u.blockdev_add.data, errp); +} + +static void blockdev_add_abort(BlkActionState *common) +{ + BlockdevAddState *s =3D DO_UPCAST(BlockdevAddState, common, common); + + bdrv_unref(s->bs); +} + static const BlkActionOps actions[] =3D { [TRANSACTION_ACTION_KIND_BLOCKDEV_SNAPSHOT] =3D { .instance_size =3D sizeof(ExternalSnapshotState), @@ -2253,6 +2302,11 @@ static const BlkActionOps actions[] =3D { .commit =3D block_dirty_bitmap_remove_commit, .abort =3D block_dirty_bitmap_remove_abort, }, + [TRANSACTION_ACTION_KIND_BLOCKDEV_ADD] =3D { + .instance_size =3D sizeof(BlockdevAddState), + .prepare =3D blockdev_add_prepare, + .abort =3D blockdev_add_abort, + }, /* Where are transactions for MIRROR, COMMIT and STREAM? * Although these blockjobs use transaction callbacks like the backup = job, * these jobs do not necessarily adhere to transaction semantics. @@ -3499,31 +3553,7 @@ out: =20 void qmp_blockdev_add(BlockdevOptions *options, Error **errp) { - BlockDriverState *bs; - QObject *obj; - Visitor *v =3D qobject_output_visitor_new(&obj); - QDict *qdict; - - visit_type_BlockdevOptions(v, NULL, &options, &error_abort); - visit_complete(v, &obj); - qdict =3D qobject_to(QDict, obj); - - qdict_flatten(qdict); - - if (!qdict_get_try_str(qdict, "node-name")) { - error_setg(errp, "'node-name' must be specified for the root node"= ); - goto fail; - } - - bs =3D bds_tree_init(qdict, errp); - if (!bs) { - goto fail; - } - - bdrv_set_monitor_owned(bs); - -fail: - visit_free(v); + blockdev_add(options, errp); } =20 void qmp_blockdev_reopen(BlockdevOptionsList *reopen_list, Error **errp) --=20 2.31.1 From nobody Sun Feb 8 18:44:16 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; arc=pass (i=1 dmarc=pass fromdomain=virtuozzo.com); dmarc=pass(p=quarantine dis=none) header.from=virtuozzo.com ARC-Seal: i=2; a=rsa-sha256; t=1645723054; cv=pass; d=zohomail.com; s=zohoarc; b=mTb2HoiQ35p77/vY1CIpveqSJoYr9yorWj3UoIUwOIA9g/rg3mATVLujRnggY/YA0FZgeFOO/9ENqtNtg/KDRoxyY9HyRD6ldITta4CSjY3hSnYiiuxrFxLVUSNTCHeasA2vd4drnFzq9KcflNwYiugbrwjO2CaiYBRGRALPpQA= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1645723054; h=Content-Type: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=G47iRQIOgfT0wAzgPUnf4mXbZDrHrM3d/9Bijf+gOsQ=; b=G7wfvDCPVX1vNw3txLGvEn7sG/SJWaq3F0/cYl7kc5RdeutnMcr4D66lrdwa3ifjEKpkUfsXMGkPL8TndqBtNLJvQqaB/U1oHO+h3jEv8PHJlDhTCU/omXo0YSMQMFynv8L+idh+wbx1m1V2NrzlTn4UW+dVoGNF7AAMbUOefCc= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; arc=pass (i=1 dmarc=pass fromdomain=virtuozzo.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1645723054477225.83263004114815; Thu, 24 Feb 2022 09:17:34 -0800 (PST) Received: from localhost ([::1]:51818 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1nNHkD-0000a1-Br for importer@patchew.org; Thu, 24 Feb 2022 12:17:33 -0500 Received: from eggs.gnu.org ([209.51.188.92]:60896) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1nNHgw-0005jp-B6; Thu, 24 Feb 2022 12:14:10 -0500 Received: from [2a01:111:f400:fe1e::726] (port=14723 helo=EUR01-HE1-obe.outbound.protection.outlook.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1nNHgt-0005pu-QC; Thu, 24 Feb 2022 12:14:09 -0500 Received: from AM9PR08MB6737.eurprd08.prod.outlook.com (2603:10a6:20b:304::18) by VI1PR08MB3197.eurprd08.prod.outlook.com (2603:10a6:803:3e::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.22; Thu, 24 Feb 2022 17:13:54 +0000 Received: from AM9PR08MB6737.eurprd08.prod.outlook.com ([fe80::49c:67e9:3e24:8714]) by AM9PR08MB6737.eurprd08.prod.outlook.com ([fe80::49c:67e9:3e24:8714%3]) with mapi id 15.20.5017.021; Thu, 24 Feb 2022 17:13:54 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=RmwMtwIPWSvJas60ySqjbIsx3KCSGprCBNObKb8ysZV6dHQHpstKtsC979sHZQxx5nVm/0824XmiupGz0yh/NkiNzcbyi3/Q/3CzRBvBGEFzMWOXVCzIdoycxqIHG3U61Qw2h/aAaT9GNpA1PvA30tGy2+RPQmyXbjKk8MK0eIrowU91lLEgaJGc96I15O/uypb2PZ99ksc8mTonJSvkLsyW0vBkuNrsp7UxAWOgov5wFJ6gIsmfXtyOKehO3Jnk0R5FiIY1ydhqfGMIz0OwT6Ob5DOn2GLvn6ETqLxCUydfD0gjGBkGA0Epvj+uuwOAOvdeo0OLbrFzyi6FeTpUgw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=G47iRQIOgfT0wAzgPUnf4mXbZDrHrM3d/9Bijf+gOsQ=; b=G3CFctxGgoizKVflYG/OqzjEBOwwr6zfQsSEnDniiDLY2S/3F2xY+zvAzTTS9mGI/xHYRNvQJzR/8FvxRU0QubczBkE0YVot/DFP2jgcCu/vt8bHTUsVPZsYrHoYZFfqeguJjECFZdUFDd9VAKB2NmNKPe5wgx9+oUVLhEToltp7cFmMr0lqVvifJl/QJcCp0bUFxYkC3YLWAQb5VsN2akdLpuuJMkgMX3gRnvbGTY3yk/Cfbig7xeWV3ebXNy5LNK5TspsRIrx8ZOsq2TGVMo1B+i8imI6z1QdpX1Gp8MspeBJ1bJNxGdi+cf9HB/Na9QkUYnMZ1osBPSWOpm7TDA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=virtuozzo.com; dmarc=pass action=none header.from=virtuozzo.com; dkim=pass header.d=virtuozzo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=virtuozzo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=G47iRQIOgfT0wAzgPUnf4mXbZDrHrM3d/9Bijf+gOsQ=; b=pjPfquiVZuqpejuPVizCj7j2jle4ch4lp5hnuOdoLv8mHPEwVUcHaie1sKY3PjwX5jBmLcbYTtCxhqXHeyv4A/t/ccBTYDvNQkjzNdhzzkslQBDMUKBoEFzFwaMR3ibbeBWsmMyGDpi+fTlVt8jwayUPVcf1RNpkaTqYvVaabyg= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=virtuozzo.com; From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Cc: qemu-devel@nongnu.org, eblake@redhat.com, armbru@redhat.com, hreitz@redhat.com, kwolf@redhat.com, Vladimir Sementsov-Ogievskiy Subject: [PATCH 2/2] iotests: add blockdev-add-transaction Date: Thu, 24 Feb 2022 18:13:28 +0100 Message-Id: <20220224171328.1628047-3-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20220224171328.1628047-1-vsementsov@virtuozzo.com> References: <20220224171328.1628047-1-vsementsov@virtuozzo.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: VI1P195CA0016.EURP195.PROD.OUTLOOK.COM (2603:10a6:800:d0::26) To AM9PR08MB6737.eurprd08.prod.outlook.com (2603:10a6:20b:304::18) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 476cda8b-9da4-47b6-1a26-08d9f7b90942 X-MS-TrafficTypeDiagnostic: VI1PR08MB3197:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 7bTDAfPYbHiifpnX3PyExjjziZhHQkTqBrLD6Hp6rF/G7f15nk8hU6vp/Zse/LlxEJOLEu3iVfdWT1Dib5/Ci51Q4kk6u69uvFPBnQf0xgiYQoENvdNR40tcYti+DVCsQRhKkbMsSV3HLwRmvm2Hj21FEMV9vXtC9BsteQBuodXCNlQx3H458OlZUXaL6L6PovMSwKa1VuPSj8C2gTu9w69dcnKJo/2yRkm1m1XgmKrco/vLPOLtNrdYpiReyWl/ZYxFrcBl522uHRimycGfNuMa1LH3RWoyvcJL1bJZpwTiWC/Nv/V1Dt2t9LgLJ0ircUXEonVlfkO24MwlH7fM34kxEFPSPrA0g8/vv1ufck9lEMJJyyufFnSTfm3tbVBIS3yUpq+UgNArg8FLSwR5F1zJBFiKBOpHGtOvemAIjepRqy7DSsqo+ZpeFPFjkXaMPUlgI4991ChW5GMhsilfGnLKBjrpkWG3Mjk4LGZfYfeWFTLJ7MvBQ2bXS1EM9NlAW/AkYSkWdxDae594Oz7NH6EHiB3P1e9WGjMvpR/eSGHfFStOY9+V+nlL3qnBjsHysNn1J0myySHY/Y3c7oMBwbl5CuuOqI+ObzprO3M7qogezBUj2rSDLFvpcucU7R2JCYi1O+OhEMfjSs5wbXjTyqTwXLNZJoYSHPkp26KyEYLPtjuq90r1vtCFxj+7NSG01srpzqBBe4xcsMbLsmxxEYkMe1DwQqSm+umVj0HXV1qShfNhBIesRwO0w+Ug6OXaA3Sv3XqqsaIUgjkp8K7CE/3HwGF/qEeQa4oQecTqtM4l9kjoO0SGOtsB1pEDqdzM X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM9PR08MB6737.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(4636009)(366004)(6506007)(52116002)(508600001)(6666004)(2616005)(86362001)(38100700002)(38350700002)(6486002)(4326008)(8676002)(66476007)(66556008)(66946007)(6916009)(316002)(1076003)(8936002)(107886003)(26005)(2906002)(83380400001)(186003)(6512007)(5660300002)(36756003)(2004002); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?reBzyBXVdB2kJCs7qRkyXslQMm7s1v9bexhEDPnN2fF+fTyEnVSd6QmFkaZ3?= =?us-ascii?Q?eVWS/UOjB03WcU2gV5ughvDiwhEfc9mPXQ7Kvumjm3uUySudJnrrTiKdmusb?= =?us-ascii?Q?KLPVfJQp0N6jx2f9lBj1Lfu+Vcx3RwE7W3JWOyUX32N3AfG+n128RUp7gXN9?= =?us-ascii?Q?C6yWNELQamjQJhj9TFBMGiUXgmXGG4bJ2FxD1MWmxGKACd6BCbZ/OC96Ti5M?= =?us-ascii?Q?lrHOLvLAp013l8nDfEKrm4UrKotmcWkSFNalU46Rb1wK1QVViwscw67+UtGy?= =?us-ascii?Q?PBJVY+UW+k0rQm/WBkAb0e3rSzF4xLUiVbQvDsIG1FojxGJxe5SgD4/dI1hj?= =?us-ascii?Q?eM7LjibQ6ZtJz7eACBKk63tNmRN91eiGU82K8Aon8R1eZlsLjHchOmmw8UyY?= =?us-ascii?Q?Lt7rT3PEefsYg03OSD/FxTiI+15OSi+MuIkqiCE0BTXO6vXkvCmbGxL5fdOW?= =?us-ascii?Q?o1uJgoZDPMWIrDaS/uxz2JfQ5dx4WxpkoZqM+bOTbu4CFI8wYnjdoKef+H0z?= =?us-ascii?Q?7IjErsdSyC6y6uMliPrjGLcmg0dwiW6kLQfCh8/pZgIODC4l6Sy602orqRnF?= =?us-ascii?Q?QVwOAaseaiCkwY3x1JADy2Ewlh5/Y1dbaVx0utm/o6StnQ4oDS7THGyk5L6c?= =?us-ascii?Q?5UIup7xUE0mjSvTrzYQbesmWmE9ZVuQynbBVUtrHwc/yV41XsVqL7ifDrfGC?= =?us-ascii?Q?OewlnvSq0zmgAjd3K1lvdal3x1Y+SnsspBTva9LIgO76uZVK60Vu9a0cP2ja?= =?us-ascii?Q?cgoFLtDSgcf5RSYpNaydUVoYBhFV8mOXqH7E3FqKKWSNtsP+57KXae9MbDIX?= =?us-ascii?Q?5rE5Y5suN5QSDA+U7XP4IKukRZuA8B24kG7GK9I38jMnN+5I/m22PFQKYw3N?= =?us-ascii?Q?8D5tjMEA+P6mMIY9J9xhQSsxs/i0uDoSCW80Vbjy53MZkhxkE0VMx8dIqiIW?= =?us-ascii?Q?6T8LqSHdAOBadBAFylQazadTOfEbXG9theDRs/T0e3TkPkGojIvUZsGd2866?= =?us-ascii?Q?WxfUiq74OpmLM3H8w0LujA+f0xB9Jm838OsqlxKV+2PBv9ekv3y4Y1XNWD1l?= =?us-ascii?Q?0CTehqey2ujQE/Ex8oGgqtELeu65nkz9G7V0+sJiA/rHtYnCsKK788ZFMjkH?= =?us-ascii?Q?0OYJkFdfBuRtbOq+A0bg2qA6cuNPMQFkNFdFc0+mTTMpBcGT6E6ax+jheGJi?= =?us-ascii?Q?6G24B6fiOUOHfX0RI6fNkrsu9ecu263ITXgpeuaHGRAiSulhs9AUG9c1XB8x?= =?us-ascii?Q?mDxurWNIZARDspB7z3h74hQqtIqQZPMPXgmbETAMyX/J0OYZgSz7wIKcR38s?= =?us-ascii?Q?qyitlYLQP+Yfa9PNf0OFUDZUtLOy1xQjlR9ADHBtojlDMt7ddqiXQtotkflp?= =?us-ascii?Q?jxhzLhhDLND0VpnGjwAmRpRV2dHdwBWY14jp4iPVoVi6A2cLWtgIWONJ5GXW?= =?us-ascii?Q?Tmhx6xLGQ0luZruJEmR+UgaBLQnQWJoxslKO6g3N6ft5rQ1hf885LEN/taCM?= =?us-ascii?Q?0mWtvD5vxocOD60AHqRH2UvYxrxRL59mWbaCCcxOfemjbau2KZ5gOzChgFaW?= =?us-ascii?Q?G15jtPxV4DKTet387tkQ6tbODq4Ou9A/3HS25tZto1AlUi5mVsW2PGL4HBbE?= =?us-ascii?Q?O+tCIqOj/GM/vvKWDhYg1O9RJ7pOIVnTelD6hnK+MO7obaUw9F0obU0MePPe?= =?us-ascii?Q?qDh289PA4+C1D5egEsR3dTgoaGw=3D?= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 476cda8b-9da4-47b6-1a26-08d9f7b90942 X-MS-Exchange-CrossTenant-AuthSource: AM9PR08MB6737.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Feb 2022 17:13:54.6910 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 0bc7f26d-0264-416e-a6fc-8352af79c58f X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: RQiytWTxV8lXaMG9JSfc5J8j6nB17m2o/ZvGy4oRqsfH8Til7CNXVeOCNXyGPicaY2jm+AxupBxA8/LtZJL2QVpao8ks8mizIu/Ia5Uyn9E= X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB3197 X-Host-Lookup-Failed: Reverse DNS lookup failed for 2a01:111:f400:fe1e::726 (failed) Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a01:111:f400:fe1e::726; envelope-from=vsementsov@virtuozzo.com; helo=EUR01-HE1-obe.outbound.protection.outlook.com X-Spam_score_int: -6 X-Spam_score: -0.7 X-Spam_bar: / X-Spam_report: (-0.7 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, PDS_HP_HELO_NORDNS=0.659, RDNS_NONE=0.793, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @virtuozzo.com) X-ZM-MESSAGEID: 1645723056825100003 Content-Type: text/plain; charset="utf-8" Add a test for transaction support of blockdev-add. Test is format-agnostic, so limit it to qcow2 to avoid extra test runs. Signed-off-by: Vladimir Sementsov-Ogievskiy --- .../tests/blockdev-add-transaction | 52 +++++++++++++++++++ .../tests/blockdev-add-transaction.out | 6 +++ 2 files changed, 58 insertions(+) create mode 100755 tests/qemu-iotests/tests/blockdev-add-transaction create mode 100644 tests/qemu-iotests/tests/blockdev-add-transaction.out diff --git a/tests/qemu-iotests/tests/blockdev-add-transaction b/tests/qemu= -iotests/tests/blockdev-add-transaction new file mode 100755 index 0000000000..ce3c1c069b --- /dev/null +++ b/tests/qemu-iotests/tests/blockdev-add-transaction @@ -0,0 +1,52 @@ +#!/usr/bin/env python3 +# +# Test blockdev-add transaction action +# +# Copyright (c) 2022 Virtuozzo International GmbH. +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . +# + +import iotests +from iotests import log + +iotests.script_initialize(supported_fmts=3D['qcow2']) + +with iotests.VM() as vm: + vm.launch() + + # Use same node-name for nodes, neither one should appear. + vm.qmp_log('transaction', actions=3D[ + {'type': 'blockdev-add', + 'data': {'node-name': 'node0', 'driver': 'null-co', + 'size': 1024 * 1024}}, + {'type': 'blockdev-add', + 'data': {'node-name': 'node0', 'driver': 'null-co', + 'size': 1024 * 1024}} + ]) + + n =3D len(vm.qmp('query-named-block-nodes')['return']) + log(f'Created {n} nodes') + + vm.qmp_log('transaction', actions=3D[ + {'type': 'blockdev-add', + 'data': {'node-name': 'node0', 'driver': 'null-co', + 'size': 1024 * 1024}}, + {'type': 'blockdev-add', + 'data': {'node-name': 'node1', 'driver': 'null-co', + 'size': 1024 * 1024}} + ]) + + n =3D len(vm.qmp('query-named-block-nodes')['return']) + log(f'Created {n} nodes') diff --git a/tests/qemu-iotests/tests/blockdev-add-transaction.out b/tests/= qemu-iotests/tests/blockdev-add-transaction.out new file mode 100644 index 0000000000..7e6cd5a9a3 --- /dev/null +++ b/tests/qemu-iotests/tests/blockdev-add-transaction.out @@ -0,0 +1,6 @@ +{"execute": "transaction", "arguments": {"actions": [{"data": {"driver": "= null-co", "node-name": "node0", "size": 1048576}, "type": "blockdev-add"}, = {"data": {"driver": "null-co", "node-name": "node0", "size": 1048576}, "typ= e": "blockdev-add"}]}} +{"error": {"class": "GenericError", "desc": "Duplicate nodes with node-nam= e=3D'node0'"}} +Created 0 nodes +{"execute": "transaction", "arguments": {"actions": [{"data": {"driver": "= null-co", "node-name": "node0", "size": 1048576}, "type": "blockdev-add"}, = {"data": {"driver": "null-co", "node-name": "node1", "size": 1048576}, "typ= e": "blockdev-add"}]}} +{"return": {}} +Created 2 nodes --=20 2.31.1