From nobody Thu Mar 28 21:03:57 2024 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass(p=quarantine dis=none) header.from=virtuozzo.com ARC-Seal: i=2; a=rsa-sha256; t=1620121691; cv=pass; d=zohomail.com; s=zohoarc; b=DlfMAYk1PaOwVYFa1pYIzzAdRvFz+5mD84Bu91DAQKdlXFH4qEoic86ikVlmGseIncQe6c6G+8O2r9U9lVG8/KufVGxbiAOIbFp9teUAzD3WcDXNryrJxl7rnCefzkI3UyFb7dyg5ClRZFBplYA9Y4ryCkBxgPVvfDMz7kTSqOk= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1620121691; 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=0IrSJAcH+pqt2aZWAHJQINU4Rl6ephvHojeX+7yt+vo=; b=gUxcGg5puNunOTUzj3mEQINSYnTuUnCO6W31sMMaAc0C+CJgK2w8wRu1zb3taT8Yu9p8JUOiPV1WgfKOfaksQX6AivRuOEjpDoBfNmzd2U5e7Fq3/BIFyE1JTndvNlLHmsuIZcD1eYa4xw1ZNtCxns94NX2TwqZXABBYh2mqh94= 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass header.from= (p=quarantine 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 162012169191196.68953115258807; Tue, 4 May 2021 02:48:11 -0700 (PDT) Received: from localhost ([::1]:39386 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ldrf0-0000gK-RF for importer@patchew.org; Tue, 04 May 2021 05:48:10 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51414) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ldrd0-0007QK-41; Tue, 04 May 2021 05:46:06 -0400 Received: from mail-vi1eur05on2137.outbound.protection.outlook.com ([40.107.21.137]:63329 helo=EUR05-VI1-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 1ldrcv-0001jr-JM; Tue, 04 May 2021 05:46:05 -0400 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) by AS8PR08MB6152.eurprd08.prod.outlook.com (2603:10a6:20b:298::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4087.35; Tue, 4 May 2021 09:45:25 +0000 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133]) by AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133%7]) with mapi id 15.20.4108.024; Tue, 4 May 2021 09:45:25 +0000 Received: from localhost.localdomain (185.215.60.251) by HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.24 via Frontend Transport; Tue, 4 May 2021 09:45:24 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mpiAor4gne1R3Oz1QPGnTsEOTYO9ka3i0Jm0X3Zz43aTE5HAoUmxuB6beNllWgwotEasEYu5NhCHIQPFEVFdGB0tpjSGl4LnB1KuFL+Q79/yz9i9sAThJOuaXSSTInBu9x+MyilnsqRBksq+M+f9IfFzxJPyJOIwxnBrbRZf3x2F3HEk+mxr9NM9DjnTRVDgAmM+HNv+Zj8uPlMLtq22yJbYKcBFmnKaRYRhPIaqjtbeZo7PM26LyXVlJo0o2hmxJ+2+uIJuFLIkArjZwHSYlXefe0O7CQI56mikjMHLb9LSQUwBXrSxR7PZbTHSgoKnJqJp2v/TAT5aA0kFZKdg+w== 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-SenderADCheck; bh=0IrSJAcH+pqt2aZWAHJQINU4Rl6ephvHojeX+7yt+vo=; b=P5A9fHKsmptUTdisHHM+ofIKJFKvIRdvuRG6UTCRenEXjKRtiUjOM0Hayu7u56zksSPsWaDi/SyDgRnIlvTJm/83Vx6Wwy79+VB0KHNF27HuwVOzkP91gPJoGuFCwIFkyYr3q8yT+O5lWXdFEQMGD+JP/e+GZKyRamtNS5I+gV6NhiRlwozpateWhqVwA+WEDpjdJz1722BoPrWs5qyHjJvWWM0K7yP8S2d2kNCVlTKBo1db7a+smfdKMdlyM/xHhCMNxsukUEWwgxeI8T+hnO8poTbafOPciBGfzvIMHS0U5QSANvE7830MsFO311czR8St6EvpgLT8HkIVnn+6dg== 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=0IrSJAcH+pqt2aZWAHJQINU4Rl6ephvHojeX+7yt+vo=; b=FifmVNFnpONGAssZaFP7bIVtH4mJbtgPpwP5oP74excT4TQIRvZoyYAQ9eivke5zmgJRQnSNXwwl+Inr1BGL6jZooqwYwgDzW3FcPfwE/IU00aKhevUrLlBF13z6P5SEmqikj2sI/J07zErztUIQqt6rQIx9ot6y0t7svgkCOzU= Authentication-Results: nongnu.org; dkim=none (message not signed) header.d=none;nongnu.org; dmarc=none action=none header.from=virtuozzo.com; From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Cc: qemu-devel@nongnu.org, mreitz@redhat.com, kwolf@redhat.com, vsementsov@virtuozzo.com, berto@igalia.com Subject: [PATCH v2 1/5] block: document child argument of bdrv_attach_child_common() Date: Tue, 4 May 2021 12:45:06 +0300 Message-Id: <20210504094510.25032-2-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210504094510.25032-1-vsementsov@virtuozzo.com> References: <20210504094510.25032-1-vsementsov@virtuozzo.com> Content-Transfer-Encoding: quoted-printable X-Originating-IP: [185.215.60.251] X-ClientProxiedBy: HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) To AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0f333c3a-c78d-4375-ee0f-08d90ee157ed X-MS-TrafficTypeDiagnostic: AS8PR08MB6152: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:6430; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: fFcv6oDK0zOHwvdCrYcIuTbrVOfKg0nHARyP/kegy0yVtGOeOE++6Iu1VUA35qF5aEr4SOIUrnMYERyZ7h07LVR50tv8/PAP+ZBn+zv24lWWP9nrjC0P2E4sIuFKtphKhyW3FLh1QMC/9Mt9r4DN9Zx8LbhCmQPUlcLyt8jHYRols5NU5kLbkPNDtr29uRzp9A0LrbpLcrHKVC7qdIZWzHiiXEsaOSRyfUaIDQTra2MmqbuGmYtkPBXjH4WwTj5Lo/W9Z1rDOLg7WvzwpdlrqVm3pJc8RxsjNaKRnc7klw9XIL5wRnoGoZygstNlZ8aHwsUjOk0oCQfjT9Hch3Qo0idekthj9/P7bSCaTljM026tu2khpjrH1cVsYn1U6695NYNuxQvysASG02KlAtMfszOGuCLGEM0gCSDoDoVBQASLFNiLRoVESKnT0K2uk8+QSL3BG40ltFZYuSBfiJb13EO0rkEYC3qTTwRFgm+FcNC9RUvpnnRUcKc3HNsKXK61SzkfzArEMnIZ8Gddm0QVSYKtIsmiqHzE1RsSWmFzvHx09pKmMsSSyeQ2GZFwkAUfFkSMxZJAXzNvlPpL3W+tjzR8XbnGnhocwZkOcCn82eI9V7A7aBoh4er4sy+e3I+EW6NcFAbblLFFdT6cwO2MB88SOu97o9yGDHBhzT1L7/TFVWXiIvxkCO2JUY80Mqoy X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM7PR08MB5494.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(39840400004)(346002)(376002)(396003)(366004)(136003)(6512007)(52116002)(2616005)(4326008)(8936002)(2906002)(86362001)(6506007)(5660300002)(6486002)(38100700002)(38350700002)(316002)(956004)(478600001)(16526019)(186003)(6916009)(8676002)(1076003)(66556008)(6666004)(66476007)(66946007)(83380400001)(36756003)(26005)(69590400013); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?HoplVySTGy7JOKvz82ZMa06c0/Ak2Fu5tXRX6n5BXoEk4W+rLaWkCa3qcDtE?= =?us-ascii?Q?TjTNAcVKn7Ul0cbTWiw4GHjzhtUdJGqOLms6urAT2aNRQsyLwiv3sWiVkoxv?= =?us-ascii?Q?1kGtVs7/NqG52+39nxOjuWmAV51mO3Q62LnafeRDbQWmIMj3J+7HQNo97e8N?= =?us-ascii?Q?g0vWgiac5HkUSAuKymtsa6QQJRV2FPs+9rGKu2OzIJ6BCgR/UeUfdhGXUn0F?= =?us-ascii?Q?1gMMK17UewbEn0yT2+mdNk4f2tF5eiryTGZ4HmsqSrHL5NaeFMXtu/neHMe3?= =?us-ascii?Q?vkkCyqYuitM8FzqEd1IT9kbrhoZrLxahq/ugqKgB7TPKu8EH3hM9Y1cAn4dP?= =?us-ascii?Q?c0ji2ebCTtr00nZF6PipafLWKkW68Tar9UBsEZSPlF1JJlZNYRv85fYEVVd8?= =?us-ascii?Q?peATN1NOMS4A04gW+N7Ju/xP9jpKfJ6DdfjuvZ+0gRVjPBtnQGwgiSrL7gLU?= =?us-ascii?Q?0Zr5OkWhheOuvvSE46v4SqjT0EmSlnH7M1IPKCzCgT8TirNSrUVPSqmd6Ftg?= =?us-ascii?Q?/r4+c5GG0KPoAHIInPF/6BKwLrf4mlTU7oS6RYGAtQtDeHolYEXNiKWlI+yn?= =?us-ascii?Q?VAKchh0qEDcv+neVA/GGkKH8Nl484wHeDQlgEG2p9F+pqJ69h7ogxCm2Fvju?= =?us-ascii?Q?qqaML6pzHu4WecLbXHqSlD9fAUW3S9K7hP0UNhBm82afPW3jIOilSxeFnQnB?= =?us-ascii?Q?qbSJWB5Pj4Wv6VowBIjXGQRYZB2XZ2C7KsbnpCeSr1iBgVDxWhc7V13szV45?= =?us-ascii?Q?Yi82nwLPifW2gmPSIz1GtkGc1G7e4ToLK9phwyf+byRPhQTT6zSsbFNLe99E?= =?us-ascii?Q?Z0oEv6H8rms23DOD+tL4mComiNmAXv69hEIvc0faOrbo5rLVmYE+VdspLrIk?= =?us-ascii?Q?M8p/2s0He4ZOntVBGHE5bVRcPzk9ozqtre81PMPR5mus05MO25jpTSHWEcug?= =?us-ascii?Q?WoPXZXVZ4jDupTYKDWgfXg6R05fQVfOHdvk17fW4ZoxJ9W4h70meleQJHvjQ?= =?us-ascii?Q?9n24r95UuYLSVTuOn+ikwoHC0m4umgwzfa+nVB+TkxL0Kog5Cc2GOvctSQkd?= =?us-ascii?Q?IJ74bf/vt/+qnx4pHZaN7BSKji0wXx6KkwmC73PIgaL25vwuorSm5ViVVdHa?= =?us-ascii?Q?wYcosByqOXoBDN3cV0S7DMIie2MpGEMCS2JCnJgEysj80OZ3q4/wzDP4qt7R?= =?us-ascii?Q?o183rsis5e4mXmAEt5ONaDyeRNTeQXrFFTtnD1PS9AqUP8b7gNajrkphEcal?= =?us-ascii?Q?8IyCrDYF+MQPefc4R7jG1Xpel75fG3kv5eKePCtMkPeBtrr5k9KtKCaV2pT6?= =?us-ascii?Q?k3OZRdlK5wnKoJJY4QpOX7MD?= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0f333c3a-c78d-4375-ee0f-08d90ee157ed X-MS-Exchange-CrossTenant-AuthSource: AM7PR08MB5494.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2021 09:45:25.6631 (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: ACgWiE2NyiCQ6XQAlJVIB+ylmboIHBl0MiJhePVQKSUdCZUMiiFiV1o+6jQipGXieL+8xaP6AVaRxCMOoQIt2AQ7Rw6paeF/THFIQRhKpYM= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR08MB6152 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=40.107.21.137; envelope-from=vsementsov@virtuozzo.com; helo=EUR05-VI1-obe.outbound.protection.outlook.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 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, MSGID_FROM_MTA_HEADER=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action 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: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @virtuozzo.com) Content-Type: text/plain; charset="utf-8" The logic around **child is not obvious: this reference is used not only to return resulting child, but also to rollback NULL value on transaction abort. So, let's add documentation and some assertions. While being here, drop extra declaration of bdrv_attach_child_noperm(). Signed-off-by: Vladimir Sementsov-Ogievskiy --- block.c | 24 +++++++++++++++--------- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/block.c b/block.c index 69615fabd1..b9df90d61d 100644 --- a/block.c +++ b/block.c @@ -85,14 +85,6 @@ static BlockDriverState *bdrv_open_inherit(const char *f= ilename, =20 static void bdrv_replace_child_noperm(BdrvChild *child, BlockDriverState *new_bs); -static int bdrv_attach_child_noperm(BlockDriverState *parent_bs, - BlockDriverState *child_bs, - const char *child_name, - const BdrvChildClass *child_class, - BdrvChildRole child_role, - BdrvChild **child, - Transaction *tran, - Error **errp); static void bdrv_remove_filter_or_cow_child(BlockDriverState *bs, Transaction *tran); =20 @@ -2762,6 +2754,12 @@ static TransactionActionDrv bdrv_attach_child_common= _drv =3D { =20 /* * Common part of attaching bdrv child to bs or to blk or to job + * + * Resulting new child is returned through @child. + * At start *@child must be NULL. + * @child is saved to a new entry of @tran, so that *@child could be rever= ted to + * NULL on abort(). So referenced variable must live at least until transa= ction + * end. */ static int bdrv_attach_child_common(BlockDriverState *child_bs, const char *child_name, @@ -2836,6 +2834,10 @@ static int bdrv_attach_child_common(BlockDriverState= *child_bs, return 0; } =20 +/* + * Variable referenced by @child must live at least until transaction end. + * (see bdrv_attach_child_common() doc for details) + */ static int bdrv_attach_child_noperm(BlockDriverState *parent_bs, BlockDriverState *child_bs, const char *child_name, @@ -2918,7 +2920,6 @@ BdrvChild *bdrv_root_attach_child(BlockDriverState *c= hild_bs, child_role, perm, shared_perm, opaque, &child, tran, errp); if (ret < 0) { - assert(child =3D=3D NULL); goto out; } =20 @@ -2926,6 +2927,9 @@ BdrvChild *bdrv_root_attach_child(BlockDriverState *c= hild_bs, =20 out: tran_finalize(tran, ret); + /* child is unset on failure by bdrv_attach_child_common_abort() */ + assert((ret < 0) =3D=3D !child); + bdrv_unref(child_bs); return child; } @@ -2965,6 +2969,8 @@ BdrvChild *bdrv_attach_child(BlockDriverState *parent= _bs, =20 out: tran_finalize(tran, ret); + /* child is unset on failure by bdrv_attach_child_common_abort() */ + assert((ret < 0) =3D=3D !child); =20 bdrv_unref(child_bs); =20 --=20 2.29.2 From nobody Thu Mar 28 21:03:57 2024 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass(p=quarantine dis=none) header.from=virtuozzo.com ARC-Seal: i=2; a=rsa-sha256; t=1620121779; cv=pass; d=zohomail.com; s=zohoarc; b=OZViN7OPg9tuhEGc79QlDHRCOJayeM2Kv+Y5lm91f65G678j2bcK4dVWz2qpcroY+8eW1k3OOeJFudk5c9iBULornXMLlJYive4ssKBJEU820Z94r0SL70HgTb5bKIl6wmvtNuycI1y0yVCERFexJtPNidx88P8xGXzPFF+1kT8= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1620121779; 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=YSN/kJlrkZVbL67g+PWHPKfNTKWkIBebr4QNs1KP4aI=; b=V1kK8GaFYFmAcYOp3yNLULkyOPsUOu9FSEtq/DjIyeNn83DzSzJvgluuU1k56cUJI+6/j3EMW1hqTXmf4ta9KIo+44J/fBhmtNHo0cNhIwI8LJbjhYg0/8aYyqyTGYvO7levCj5i4Hjcp3PE1x76Pz06vVDo8EBoblzQtZqPgIs= 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass header.from= (p=quarantine 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 1620121779299402.96391259166353; Tue, 4 May 2021 02:49:39 -0700 (PDT) Received: from localhost ([::1]:44468 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ldrgQ-0002mS-8q for importer@patchew.org; Tue, 04 May 2021 05:49:38 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51442) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ldrd5-0007So-QX; Tue, 04 May 2021 05:46:13 -0400 Received: from mail-vi1eur05on2137.outbound.protection.outlook.com ([40.107.21.137]:63329 helo=EUR05-VI1-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 1ldrd1-0001jr-BS; Tue, 04 May 2021 05:46:09 -0400 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) by AS8PR08MB6152.eurprd08.prod.outlook.com (2603:10a6:20b:298::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4087.35; Tue, 4 May 2021 09:45:26 +0000 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133]) by AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133%7]) with mapi id 15.20.4108.024; Tue, 4 May 2021 09:45:26 +0000 Received: from localhost.localdomain (185.215.60.251) by HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.24 via Frontend Transport; Tue, 4 May 2021 09:45:25 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IfI/O5s2/UlINwM7FCG3vXRJSIPcEwioe+hA67XmO0CO+lEma0Ju8oWbD0OM0EAhNeRNbuFbbHVOYImUnC1gm371EKoKhpSi2mx7vGyG8YC+TWBgzVEzFy6dyt2xWP8vybzr6+9mDyH7hcmOu5/BzQkLCl2BR5SYLzYp/aZH8mlCka3jEhgRDH9GJbZf9jLW1KS6M6ypcDlCkyhqoTJAwHBYQv1QaM+FeN6r1Vuy3bK2bgdMZwFhxNExJAWkS/HdXEyqxTjuYJNeMNNn342vG4qm6zPIScUowXRNn8OpT7DS7RSzn+yfiC4TlG2VhgaawLtPfQxR9DBQvfCnmzTrPg== 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-SenderADCheck; bh=YSN/kJlrkZVbL67g+PWHPKfNTKWkIBebr4QNs1KP4aI=; b=PSWQ7VEtEVsIslgGEegIHGSOAqkHcMZzHFrOJ64YGNebC9w41rh+P8rfzO0JNBhLzz1gj+nuLY0V6tL1GEq7DrWWiRt59+SgplOfgxuHpXC1w7nS34Gwaau0x8DL4DAbcrKMY5iTx3DjsLrrvkbaWixuyfYEwJQuXQWLMZgwwriiIJRe4liZGnilbTxaQYaQoUHk4uV1GqZyKOuGQfpMUTc0ZfFpsyYhDTNsXoPfuHvUhFAKqtwj+AadG4QZSjpF10dvurVfg3qtK2nk0BkvTvIFAdxUCaoO4TwFEVa1IzxiZ6I8//iBUgq5KspSvctJFVysuvW3rLk4++4dZZ2YWA== 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=YSN/kJlrkZVbL67g+PWHPKfNTKWkIBebr4QNs1KP4aI=; b=iR1IVjgxJPELlAV6J4F6qG6FZfl7hGLCsOB2lG2UJTTNMX0oKjPACh1xyIpO7/S2loGKul46P8IQrw1ZK7RjWD8QcBOgSUYBrKk923RqdvK02lgiVPS6EfDhVbUPlKi9uaq8sjtGsjyO9KXhbgcKZaT/6uOt5cwodniX5a9vI5w= Authentication-Results: nongnu.org; dkim=none (message not signed) header.d=none;nongnu.org; dmarc=none action=none header.from=virtuozzo.com; From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Cc: qemu-devel@nongnu.org, mreitz@redhat.com, kwolf@redhat.com, vsementsov@virtuozzo.com, berto@igalia.com Subject: [PATCH v2 2/5] block-backend: improve blk_root_get_parent_desc() Date: Tue, 4 May 2021 12:45:07 +0300 Message-Id: <20210504094510.25032-3-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210504094510.25032-1-vsementsov@virtuozzo.com> References: <20210504094510.25032-1-vsementsov@virtuozzo.com> Content-Transfer-Encoding: quoted-printable X-Originating-IP: [185.215.60.251] X-ClientProxiedBy: HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) To AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0e1104eb-64ed-43fc-91f5-08d90ee15877 X-MS-TrafficTypeDiagnostic: AS8PR08MB6152: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:334; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: r24/Tc23+WUR8gAV0IZfh3hmTjB2w957dPgDAiviYkxU2cmFrsuYDUYvVSndvoXEiyAe4Cf5eLpqYGgtLrx6LjfW4T4EEzuxfqoC+zV8JUouFM7ydN0sI+mhaeXJ3Hwg2ZAd8wY3LhMBXwAXg40pSOT7vZNr3MG5bPlDnYBlIfbtymwA74RURFxtvglSlEPQm19I0BEMyWigvDE03T5sehJfYYEkvP1TTrmvubF1NpuTIfUrzpwGjWgVAS3Cn2spev2aGdfyGwPMOS08jU8l626oCN7ieoVgxjRnsoTKYpgmQ2Y6112FyOlmFPWQqkibhnkLezef+ImLLKgEyvYQQiFXbj0HPJXQRrCrRV7M1SN/zJKG6SiY2WZ520rr1ZDWtZKNlYrSp7fyAVqUwJQUxdRsZJe4VrbBjIvbKVfGYTmrejnh25BI3GUWw7Swm6w9OfnBhKRpw16J8T77leospSxMTf1BRVEKaz2xXFO93YG92Jfqasm4kPAWFC0s6pa3BBiKmgxeXmVWopVs2+Np/wHn6R2PX0ROknPHmf5igOFx3hLJb8pxTn7t54Tdh8rFMeoD2+0k/6vc43n75oDesaqiA7zCQLv4bRagp5zPnTaHAJohQhFoQu1q25DTWSxXUN6N779RcWD6quDuD2/R8nWUCpUJMyyggrnYKiNB0MP8QJ6rhgbVRbgSMh88BfTN X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM7PR08MB5494.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(39840400004)(346002)(376002)(396003)(366004)(136003)(6512007)(52116002)(2616005)(4326008)(8936002)(2906002)(86362001)(6506007)(5660300002)(6486002)(38100700002)(38350700002)(316002)(956004)(478600001)(16526019)(186003)(6916009)(8676002)(1076003)(66556008)(6666004)(66476007)(66946007)(83380400001)(36756003)(26005)(69590400013); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?lm4lJxOneKA99eoyLuYTO8wRtQAFFF4b3NujpyfacO6MPUBqgVWiiCwh8uZ6?= =?us-ascii?Q?yLedcwrw4yuh8bBbXvwPjlvT/dZGVKQV+1e2FTO1fnkNyglaMqAa2uTqOeGE?= =?us-ascii?Q?eVZX7KuJNPGebocrOn5+G719uYBR9x3ST7LMnR1TAEefAf73/9XsthlN0u+u?= =?us-ascii?Q?urTFLafNy/LM+pfLeDJ7756Qm0wJZxfCms9ymZ1rFzfTO69vzRhK3T7UOmSl?= =?us-ascii?Q?LvUQc0bRIkPJyS2ldBn/Mv7AeN2Dm5adUIggA9O42BrMDvErssXANpo1IYJh?= =?us-ascii?Q?HL4Ilh61dOOnFrTmrtQLOjlfUImXfYrpeKeE0d7pagpevWwLgWhsCzUg4KqN?= =?us-ascii?Q?Asojy7E2PLbpmY8jHZNdnDfzIkmXZfz3ld8hA6cVZKQ7HtjdL7Yb5u9De/Br?= =?us-ascii?Q?wcRwOEUBLyr5RHZDmNwTJYtIYZCiZBwjHlSiX88pM2xZIFyRcSPK6ZfgNpE7?= =?us-ascii?Q?p1TROL36kTs31U0kZj6KkHemKNVmDaKZiSmXvC8uSpfkQZenXUAZJOo4yYz3?= =?us-ascii?Q?GSx5KJ5I3IEeB1UqIguB3UHCyvLYEN0izWm1f6Q8erRZkzxwjoZAc879grLD?= =?us-ascii?Q?huBDWWaUV/V6TcgxdSZXE8KHir6RgtClnOWf04hgXFfXYMfM5G8FEHgHWChy?= =?us-ascii?Q?8Tskk/dd9Kp98i+mArPqy4KLpFgj6l0wD1uPard0I8Ho5TjGMpo5lNZejOmT?= =?us-ascii?Q?yPw2BuRzZj57Sufg2zOuCxWpZg/XlJvIlkdM+ltvfnj2KiPxCZhP0Ttb1MMA?= =?us-ascii?Q?m+PEqROL2wcDeJDfDRwZXuutbSanfuUyugNbrDs2rL+G1ZaC3corMEbR8d8y?= =?us-ascii?Q?ZwctKDF4Tzkmw5f6S6+9KSTVCcetS3QA837LYMuqtzGGnkaUxb9+cUHiOA5H?= =?us-ascii?Q?nzHPE815oN65jkx3FbJ+8EOdfzrG+2TR7TQI9WkgcxCPC7jndVfZyweJhogQ?= =?us-ascii?Q?GDgtsuYrqfFh0KQfvVlImkS2v5s1Dm2eDI40zeR9S8MYKaqg3vuD1pLRkJ0o?= =?us-ascii?Q?19D9CdIoeiG1u7W0dfsO+tTRwvfygTjKTw5UViyr+CXFO3IlKl5yoACiGlJK?= =?us-ascii?Q?i0W9lGv6vXWAA9CA1UFd1r8nZY3B5rTnMdqeVl4aaCY/YvjEUQGFgQMPGRRM?= =?us-ascii?Q?Eo0SUjpHk3UdnnkeZZ7cqrg/tZDukaGIQBHYTg6uIeCXbHqshkvgjtMmFG61?= =?us-ascii?Q?l1cFAE/QJPkIH8R4dRkM4+M6Jo2EQcwsBcJXDjqRbHhqPrZfWFGUycno106l?= =?us-ascii?Q?dd8QjxNsh8MOaKpyT3U2L7pyGIJtg9llDSqpevzyQp8BUspWkMO1UtgU6kWg?= =?us-ascii?Q?I4x6u0zEMj7lqHMOddrX4cfg?= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0e1104eb-64ed-43fc-91f5-08d90ee15877 X-MS-Exchange-CrossTenant-AuthSource: AM7PR08MB5494.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2021 09:45:26.5571 (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: QVOo3wRTzUkhvli3P3FELEvmktuusw21yjF9M8DhCp5nw8EetioXmpMpg04KoRtauoF25Lx9yZGk1vAQx8+j/WX3NH+YP7v1DYiQliOgBI4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR08MB6152 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=40.107.21.137; envelope-from=vsementsov@virtuozzo.com; helo=EUR05-VI1-obe.outbound.protection.outlook.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 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, MSGID_FROM_MTA_HEADER=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action 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: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @virtuozzo.com) Content-Type: text/plain; charset="utf-8" We have different types of parents: block nodes, block backends and jobs. So, it makes sense to specify type together with name. While being here also use g_autofree. iotest 307 output is updated. Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/block-backend.c | 9 ++++----- tests/qemu-iotests/307.out | 2 +- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/block/block-backend.c b/block/block-backend.c index 6fca9853e1..2b7e9b5192 100644 --- a/block/block-backend.c +++ b/block/block-backend.c @@ -142,19 +142,18 @@ static void blk_root_set_aio_ctx(BdrvChild *child, Ai= oContext *ctx, static char *blk_root_get_parent_desc(BdrvChild *child) { BlockBackend *blk =3D child->opaque; - char *dev_id; + g_autofree char *dev_id =3D NULL; =20 if (blk->name) { - return g_strdup(blk->name); + return g_strdup_printf("block device '%s'", blk->name); } =20 dev_id =3D blk_get_attached_dev_id(blk); if (*dev_id) { - return dev_id; + return g_strdup_printf("block device '%s'", dev_id); } else { /* TODO Callback into the BB owner for something more detailed */ - g_free(dev_id); - return g_strdup("a block device"); + return g_strdup("unnamed block device"); } } =20 diff --git a/tests/qemu-iotests/307.out b/tests/qemu-iotests/307.out index daa8ad2da0..66bf2ddb74 100644 --- a/tests/qemu-iotests/307.out +++ b/tests/qemu-iotests/307.out @@ -53,7 +53,7 @@ exports available: 1 =20 =3D=3D=3D Add a writable export =3D=3D=3D {"execute": "block-export-add", "arguments": {"description": "This is the = writable second export", "id": "export1", "name": "export1", "node-name": "= fmt", "type": "nbd", "writable": true, "writethrough": true}} -{"error": {"class": "GenericError", "desc": "Conflicts with use by sda as = 'root', which does not allow 'write' on fmt"}} +{"error": {"class": "GenericError", "desc": "Conflicts with use by block d= evice 'sda' as 'root', which does not allow 'write' on fmt"}} {"execute": "device_del", "arguments": {"id": "sda"}} {"return": {}} {"data": {"device": "sda", "path": "/machine/peripheral/sda"}, "event": "D= EVICE_DELETED", "timestamp": {"microseconds": "USECS", "seconds": "SECS"}} --=20 2.29.2 From nobody Thu Mar 28 21:03:57 2024 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass(p=quarantine dis=none) header.from=virtuozzo.com ARC-Seal: i=2; a=rsa-sha256; t=1620121692; cv=pass; d=zohomail.com; s=zohoarc; b=O5fIRlQz6ebuBL6NiHq4scrQKd2gMqrlp+szNdDSbXI0enj6nmVWW4bJdeKBA/OoE5XXC6eymQnblT+VBw7Ap/DxAibUk3KsHG3Y81/2op5K1aVvn47d9ThhLd7Bv8/npA1qQ7uRrcQFlb0nIIwUI5p6pM8uBjs6Ucy/ZaU5YB0= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1620121692; 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=b+lD9dsyde2gPEXgbkI09VYAPvmNkKEFe5BMnXZKMCk=; b=FD3ocTYPjWiJAvwlevEqQ0lFlWDX230Q10SrxfYEP/PanyFlZGRDKE8/X2pE99ciqe8kHx1DXvj87uRno+1p3Ry8FkDFsJVcqJRIFclk60WH06GkzAyZeFyhvIdQznexVDhc3D2CTO9GSqkEbw/5yx7Txsn4+ymUrBYwQusWc9c= 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass header.from= (p=quarantine 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 1620121692508517.4695075590894; Tue, 4 May 2021 02:48:12 -0700 (PDT) Received: from localhost ([::1]:39442 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ldrf1-0000ha-EU for importer@patchew.org; Tue, 04 May 2021 05:48:11 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51328) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ldrcT-0006r9-Qr; Tue, 04 May 2021 05:45:33 -0400 Received: from mail-vi1eur05on2125.outbound.protection.outlook.com ([40.107.21.125]:31488 helo=EUR05-VI1-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 1ldrcR-0001RP-7C; Tue, 04 May 2021 05:45:33 -0400 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) by AS8PR08MB6947.eurprd08.prod.outlook.com (2603:10a6:20b:346::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.24; Tue, 4 May 2021 09:45:27 +0000 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133]) by AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133%7]) with mapi id 15.20.4108.024; Tue, 4 May 2021 09:45:27 +0000 Received: from localhost.localdomain (185.215.60.251) by HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.24 via Frontend Transport; Tue, 4 May 2021 09:45:26 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Cw35bbtB8mF05yOtHdSD2g2XA590ScIY1J1Zi0bRNJY+/vqP2B8bKeQ07ZrMovM6A6+gs4kiyLyZpQr2WyS6G4uaXh7PIMhWvBLK2biIYc+w8c7T6giJhuc5HRsLbgxGetqbEPWwcDFW5tvHi2ousSFy3e3suGcSWAI/NtcegdyGgepRBbXAPZvRmMNBe4tUANlQliLjlECaGNKSbEa/5Il9mSeKBAdkm6eQUoJBwrElyzZ3G5SPvAvOSdFIoXGP7k9OPpxxjaY46zR21jRBQ3K6Or6WCoj1fywdXLqBp+MjkWKG3wOHfa+wU2sOzyUqZ6OpSlf6gMO62X//yenYMA== 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-SenderADCheck; bh=b+lD9dsyde2gPEXgbkI09VYAPvmNkKEFe5BMnXZKMCk=; b=KjMbUioOxc7NEv1xD9cOQgBaf2N+MxVkuxrw7X2z/IBbETAX3QwKjtC/6kFsusP27aj43HbhEng4FwcKr4y1I0c6snU7vc57CUmv4g70GkAot/N7muqMP4oajt+Fq4gUYPAjYZThDcOpakqBTjNOfaraORRGnv9ci10mHWIFl312SSaf/zxFmttTZcFZRLQtS7X6SF/A5fmWchx4mzCK7rey+vqfpWTRGIJ92Hnzw1n5Dx7odBVYuT8dZNjRJTRtcz/Sq769YV1SqhzspFTIqehU5NscWsVVjO/g6LKqkwGsiD9yinAX95D8+6qOZ33RDIdbHm/wprbfAq9lvZYIGg== 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=b+lD9dsyde2gPEXgbkI09VYAPvmNkKEFe5BMnXZKMCk=; b=TmNy3gGHdoQE+twb57F+cAJIbbj7778tFYs3UIPSbRiBeJkeCPtoyQMl+CbuHFOvhLXIkQMfFZFfZ34xC+qVtiSHTqXMSbvg2iE++R+SF8eljcgv8I6deRZdJmy3QNv3oCfKzGQyvTGuh60F1JewAxSh41COpGOpUxGrwOYdVCc= Authentication-Results: nongnu.org; dkim=none (message not signed) header.d=none;nongnu.org; dmarc=none action=none header.from=virtuozzo.com; From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Cc: qemu-devel@nongnu.org, mreitz@redhat.com, kwolf@redhat.com, vsementsov@virtuozzo.com, berto@igalia.com Subject: [PATCH v2 3/5] block: improve bdrv_child_get_parent_desc() Date: Tue, 4 May 2021 12:45:08 +0300 Message-Id: <20210504094510.25032-4-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210504094510.25032-1-vsementsov@virtuozzo.com> References: <20210504094510.25032-1-vsementsov@virtuozzo.com> Content-Transfer-Encoding: quoted-printable X-Originating-IP: [185.215.60.251] X-ClientProxiedBy: HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) To AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: be0a86fe-0da9-4c92-d7e8-08d90ee158ff X-MS-TrafficTypeDiagnostic: AS8PR08MB6947: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8882; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HA7ZpT/7/3AvSwrsuhnw608E4rD9C0DV4O8zUSNaS46+3pr9WJsuuDLhRWFTO3RxO5r4F5U5VQ7J+UH0Yo+nFsMyvIps+y1TgdDT5wWg5DFR1iiawdf6CTqUJRXjy/swxmuA0XrwD7Q3PZrvKJyGhKMY108hBrYUgzw7thXUGFiiDDIiAt7e5z1to16W/yMMQ7uIInlt+6XSNvM3Hqutg+VL/aE6E/Autd58nYDQd8zczB+mussXvitOYlLheU3ytY87FA49Kg53YLBMA0X+UHxKJ3yNJJvCtzaFCy/+nJ0lLL2F8qnSBn4CADph5+fmHzB5OYOs7AFssLpZbTe+EI5iZeO/7KUB/iJ/e6o3nmR6J8vxMgqkQHQUAKDJ2JGQRwKfsWlsP9gjd5xWz3hUtA0LDfju5rd1gN2vXj0W4gLcV33CLOgDyJnXWDkUAtYV7Ahu7XVQ+B2fGSD1hmZGyG2EHK4sNKXKttql/ODrIiGKmQuZGQQczBFoeLV9grq1h5hormIiMyyjFdinZeuOxFiTqnUhc3u9LYC9iJVmlqspQprq444+DbZDyIcTxMvPHqURaIG5v8lT7W5LHX+TywtxvQ5QXDPfrCTf9Rq00NivZAoO19NQzn5oAuY1R3GSyd2nXL5zZ0DRSxUragdOnKWDTygVrtv1Zcx2+aefGQwk/seksK8bd6sXuBFWNos1 X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM7PR08MB5494.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(396003)(136003)(376002)(366004)(346002)(39840400004)(6506007)(66946007)(66476007)(16526019)(52116002)(36756003)(83380400001)(6666004)(186003)(6512007)(86362001)(38350700002)(38100700002)(66556008)(478600001)(4326008)(1076003)(5660300002)(316002)(2906002)(8676002)(8936002)(6486002)(6916009)(956004)(2616005)(26005)(69590400013); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?o23YKlp96sSEG+Ky7Pkl+ujJkqMTKZWyyANhk/aVveJ5cgjrQiBxT3ubSD2A?= =?us-ascii?Q?ylQreMcgp37VMCWryVtJW0+CH+sMUlBHKeq1ECwFd1QzVvSDzRGpYkA5dqF3?= =?us-ascii?Q?9IUkG2RXKfqws5eNOsYnBTc+AnHPlFeYGoQ+7M0WSGEGedV8PNOO7USBTFz8?= =?us-ascii?Q?Afhbh/WFROOxVp6+VyHfsUTjup0OGTsJdVPQfibJQ/jF6G8ezd7iKc1aOiFw?= =?us-ascii?Q?FSefSqf9bYZhYoT+PlDOpdeanwtQTCRZvK6ZYzcKOIul7Y9aFp9TPzf3cuL5?= =?us-ascii?Q?aSLDlVZ8mD8wgMbfQbmOZL4VoTFFFiRltB91R2PiDdNcLj8ZhqIVTqrrKVIW?= =?us-ascii?Q?SPe6gbyi+b/ZN7MUR1YZEvIkwZVRCQ0rhMEpSTVxB+WGm6zscDaMn5hIyHCU?= =?us-ascii?Q?UCyFM91RhgSQcfMQmj+R+XpJhWGYHEMjxz+oc3w7t9BT65oIOO8eUcT7u/bz?= =?us-ascii?Q?b4N56gWg0WilqCfNgM/EdItweZMZlBBw0pCYTCpxTPsQRmJyza6xPIbtdEHh?= =?us-ascii?Q?xc09AkVnQtywpGjSwUbTZ1Ty1gFAjNzTTPL4HIv5mJw+vNS15HA9z61rDIA2?= =?us-ascii?Q?0YSf6LKJJRDhvbG3ISLBhpxMmqGsjV9DvBs9ojLUYn8KJbtClOXemXKk4xQ4?= =?us-ascii?Q?uXRrEyfuqXvwkwRrHmH6ThN28S78jPj5B9klnPxmXZAePzrhMAiIRfBXD8z1?= =?us-ascii?Q?5JeyhicYSP/fKlxIMilV4DRtqCUXvYNR/MPEXAeF1GHOQ1gxNEi/b4Be/nzP?= =?us-ascii?Q?lCxn8gaZgx1FljQTZWGpC5X/C8y4h35qPeSmiDSp127hGx867uhe6x0WbweB?= =?us-ascii?Q?S9pKsHPd+e2cYMis4pFuJpmstetRxaF3+zzar2akh9ElRpxGkf3NMfxDxbXa?= =?us-ascii?Q?rDImS1JSa+IGAMDFRYLPcYjVntTiq4096LdKwTwBhXjg1QufdZKQl3bYhTPd?= =?us-ascii?Q?P8Bg5HlHRdANVA/6CEVgfBnj2zO/iyGtRLZLZA5Y+l5DeExdxNz9/LPRkfk1?= =?us-ascii?Q?vfseNcSB4e8LPK6WLFKFErV3n43rnKY2tdQFg9Btjj3kTO3HpCqoRco8q0W5?= =?us-ascii?Q?w5n+N+lV3lu0ndjhFQNiKsaI9rKJ+U7/W2Cl0o8kjrtkTZY6/1avo/OW2sFm?= =?us-ascii?Q?yQeXtvoqfeLbz6j59gzZ6AISDXxoda7S0pc9Rg4ZE5EdxKy+MoG7DDamnEhi?= =?us-ascii?Q?peGkAbe3a3xM6ROmFLRU+fEWc16ifhNlmkj0/2etJMYyOmk21lnd+JuLbIyC?= =?us-ascii?Q?bpIiCIjJ52AbZgrTLTc8Rf477242giCCMBJDCY0F/BSzh40rJylf68659i3T?= =?us-ascii?Q?0j9FUnt4rlz6tplkZgVa5C1O?= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: be0a86fe-0da9-4c92-d7e8-08d90ee158ff X-MS-Exchange-CrossTenant-AuthSource: AM7PR08MB5494.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2021 09:45:27.4283 (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: kr551PZMUxWOGrLYA2V1H36EstMHeZEOkmicZuKO2ou2zuTti8lCovvFh39PqpRyLRwFjrlN01/T7yh9OB16X3Zdo1v6bq8QHkU/s6cS0qU= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR08MB6947 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=40.107.21.125; envelope-from=vsementsov@virtuozzo.com; helo=EUR05-VI1-obe.outbound.protection.outlook.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 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, MSGID_FROM_MTA_HEADER=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action 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: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @virtuozzo.com) Content-Type: text/plain; charset="utf-8" We have different types of parents: block nodes, block backends and jobs. So, it makes sense to specify type together with name. Next, this handler us used to compose an error message about permission conflict. And permission conflict occurs in a specific place of block graph. We shouldn't report name of parent device (as it refers another place in block graph), but exactly and only the name of the node. So, use bdrv_get_node_name() directly. iotest 283 output is updated. Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block.c | 2 +- tests/qemu-iotests/283.out | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/block.c b/block.c index b9df90d61d..54a3da9311 100644 --- a/block.c +++ b/block.c @@ -1152,7 +1152,7 @@ int bdrv_parse_cache_mode(const char *mode, int *flag= s, bool *writethrough) static char *bdrv_child_get_parent_desc(BdrvChild *c) { BlockDriverState *parent =3D c->opaque; - return g_strdup(bdrv_get_device_or_node_name(parent)); + return g_strdup_printf("node '%s'", bdrv_get_node_name(parent)); } =20 static void bdrv_child_cb_drained_begin(BdrvChild *child) diff --git a/tests/qemu-iotests/283.out b/tests/qemu-iotests/283.out index 97e62a4c94..c9397bfc44 100644 --- a/tests/qemu-iotests/283.out +++ b/tests/qemu-iotests/283.out @@ -5,7 +5,7 @@ {"execute": "blockdev-add", "arguments": {"driver": "blkdebug", "image": "= base", "node-name": "other", "take-child-perms": ["write"]}} {"return": {}} {"execute": "blockdev-backup", "arguments": {"device": "source", "sync": "= full", "target": "target"}} -{"error": {"class": "GenericError", "desc": "Cannot append backup-top filt= er: Conflicts with use by source as 'image', which does not allow 'write' o= n base"}} +{"error": {"class": "GenericError", "desc": "Cannot append backup-top filt= er: Conflicts with use by node 'source' as 'image', which does not allow 'w= rite' on base"}} =20 =3D=3D=3D backup-top should be gone after job-finalize =3D=3D=3D =20 --=20 2.29.2 From nobody Thu Mar 28 21:03:57 2024 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass(p=quarantine dis=none) header.from=virtuozzo.com ARC-Seal: i=2; a=rsa-sha256; t=1620121797; cv=pass; d=zohomail.com; s=zohoarc; b=FXxZHCrKPIFQklyo8aGXS9K9xZoKiqRnIovxhmBAw7J2fiR/fJo4bFh/B3to3HqrDwtvALA80LWtaCHTlNFCSOo5Au9aatyQl1KHnV9rA4MjTC01AkTYfe3U44C2x9wFH7SQt1r4ggrJp9egZeHR7fC6S5vsJwZyeeCk7kCvbfg= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1620121797; 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=2hhSFkrAqW0yMvUxJpy40QDVG+y88h6pemFJZzGMVpI=; b=S4O9cvu+xsF43tM4LNjb6ccTYPY0RpLSoMxK59AAUkzGpJrCYI5pnZ3tFvSw82r4rWMTc/cnzrczduYLVhUkXvvua4d7BV3LJRayQ4OPNukN17RAZTZ+1nScepa1dklu1gOx+hbbiFHzo+Rv7vG3oMCkzqgE447442QTYHxqI0U= 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass header.from= (p=quarantine 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 1620121797660824.0377957996456; Tue, 4 May 2021 02:49:57 -0700 (PDT) Received: from localhost ([::1]:45234 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ldrgi-00035y-Mt for importer@patchew.org; Tue, 04 May 2021 05:49:56 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51360) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ldrcW-0006v4-Q9; Tue, 04 May 2021 05:45:36 -0400 Received: from mail-vi1eur05on2125.outbound.protection.outlook.com ([40.107.21.125]:31488 helo=EUR05-VI1-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 1ldrcU-0001RP-87; Tue, 04 May 2021 05:45:36 -0400 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) by AS8PR08MB6947.eurprd08.prod.outlook.com (2603:10a6:20b:346::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.24; Tue, 4 May 2021 09:45:28 +0000 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133]) by AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133%7]) with mapi id 15.20.4108.024; Tue, 4 May 2021 09:45:28 +0000 Received: from localhost.localdomain (185.215.60.251) by HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.24 via Frontend Transport; Tue, 4 May 2021 09:45:27 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SclTk+VkFfpetcuu+VJ6vhXWAzrMHaSJRbOpODvoTrr34fktVZiEeVUbjdZH1wyxo2ybSOnQdX2Mx0q/VvtOXVf5kNtBBOt1cxp7MVHMDqYOt6lgU0fV/awcKAicYFPwfpzmHYipbJAV0akOMk1sAYLlLx9ZmtSmy5RKv0vCq/HzYQ7klmeQqP9FA016OC9EUTXpkuPvR27RvDNS/omCeBTM46f+jFdPSBt7PLFrFEJgB/6aYKrK7g5kF2h/j5qBy0gow+dbQB5UXd7iyE5VBEpKrxpEckkblre+n855iYWdkMNAMaexrM4sVFh1WMzMCKKtWIDzoCDNWmw8w2cjbA== 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-SenderADCheck; bh=2hhSFkrAqW0yMvUxJpy40QDVG+y88h6pemFJZzGMVpI=; b=TU25NL6YTFRxqR0ft3wgT8q0ZI40TOmEaXVMxu0pAzK+90wHPHcsWVoeObvneI+2FdVj6WVIMf7xndbAOUu3iX4B2T0sX3Va99J5e64xwNV22Y8cEsHb429e58Yrvofhezpj9IVhBGeEE4UsGxzREjy3ZEMe9yKj5tN0pMGF2AeE8absfwQd4aNfrdGx6M7krdCaUqob9UG85VTpgtYg0iJ6U6PtXjrtuBfjijid52UDPTKzMJ1lN3j8P9WX2ischfYdYmr1LOiaB8jvyqixCso6fLNiZVAEhRXhzAHY7qrkpyMUs953ik81dhGHwODJiUBXTnOGjs29p+vqkXrMcQ== 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=2hhSFkrAqW0yMvUxJpy40QDVG+y88h6pemFJZzGMVpI=; b=TD5I9vnqIhEjE43Qk65cUD/Dk7rDuLfXwWo8d/QM8txr/q3UbePXjSnCA8Bso1fAOtHkHScbjOKhRYY1jA1fSheAslM5RE7juRPjfsXE8DZJmYcJxLdAv5hkfm2bPHM8sHut+ju7SS597HMzp9gUmDS5leUhGMQtNJ5c9/UB04g= Authentication-Results: nongnu.org; dkim=none (message not signed) header.d=none;nongnu.org; dmarc=none action=none header.from=virtuozzo.com; From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Cc: qemu-devel@nongnu.org, mreitz@redhat.com, kwolf@redhat.com, vsementsov@virtuozzo.com, berto@igalia.com Subject: [PATCH v2 4/5] block: simplify bdrv_child_user_desc() Date: Tue, 4 May 2021 12:45:09 +0300 Message-Id: <20210504094510.25032-5-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210504094510.25032-1-vsementsov@virtuozzo.com> References: <20210504094510.25032-1-vsementsov@virtuozzo.com> Content-Transfer-Encoding: quoted-printable X-Originating-IP: [185.215.60.251] X-ClientProxiedBy: HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) To AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 6aa0207c-529d-4210-ed16-08d90ee15984 X-MS-TrafficTypeDiagnostic: AS8PR08MB6947: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:238; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: eF/YWXWFOh8UlnXUq35g0YuKjBqunkllWH4OiWksSBiFHBGRpEYdR5KIfUV3ZBjd08rsOoaG6Ww1rxVIA7Vs0xDxJOXuZEZDtiyucbk7GKKuon7tYB3Mw1a+VJcgQ8YrataFlgZLzDB+vm5xwg13m6fEqzgm2QVRyHLuHBPNaEs0T5selyYVRtlPrJWSYpRr3foPCYFkJ4k8BANoUFGdU2017MGHBPoZd1P9ZMFBVxhRlSeywADmPcr9TVJ9zhiqN+Vy+YsaQ4YR4drl3bmRbSJWJ/XWtEBug3UA/rHbxyasA9gmPuVNa6MJxITxyc34h1XUbINOTNl6LgSE7BXMy8JQugyhU7JfFn+oSbvHW9tULnz+fqnw4u/W1H39cU/6csWPeQSc2Et5hBFlon/2i4GuUVRlQCSlZO0KdMYrn5mTFLPdMyTpjVrMcmlyTbniw/0SMUyfdve7UyR3OJneQojzqca7PvfC+T7/29gC3xyByFp6wPNw1I5G5WkwuELj2kh1z+XYF8CB8DWnUKXmJCfJESe1yUnpak5zIoh6gWS4OvHl76AwvFHHM6T2+QtzwPx+/cDokmrjCMuCydDoWpGZhiD7ZqMrcrO8/RUOq8uw9ZPkZqFT76ItE2ISWwS5KDy4LuI82raqM6q+W06m8T//fQ9T0jrVgQ1b+hWdBpxa8dI3HoXCguvADyMiXswK X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM7PR08MB5494.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(396003)(136003)(376002)(366004)(346002)(39840400004)(6506007)(66946007)(66476007)(16526019)(52116002)(36756003)(83380400001)(6666004)(186003)(6512007)(86362001)(38350700002)(38100700002)(66556008)(478600001)(4326008)(1076003)(5660300002)(316002)(2906002)(8676002)(8936002)(6486002)(6916009)(956004)(2616005)(4744005)(26005)(69590400013); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?21jD1piUmuF/CDnM75a9qbUuLCrPXQuF0kYN9vAXkvcJFG38RRrXTemkXCeK?= =?us-ascii?Q?NZHRi1Pq2ncsxzuvmBXCzuE289NXqi1Ujpyc+m0H4fWl9h/ZFIcPnzcro0Av?= =?us-ascii?Q?gDz2ihnfsPGkh4TbKD0NMPeAQsg+3AhfY+nskeIKhofMmGUiwhohMOyZlp/p?= =?us-ascii?Q?wxMzxedocmXpm+X67oW8B8djSFSoCUUt85e1oeDDYvTGFqjT4x2FvGCo/6zm?= =?us-ascii?Q?4hjx10NdHF72jC6NOJ1OL+KyM7QP5fxZ4qx+G1+wBzSPr+2Tcyf4khQMNJCv?= =?us-ascii?Q?pPI26Ked/tZvZ/MH9s5xTap80BhPranBFnpcM39YJwsXOHsyHnzKsJqlzBBd?= =?us-ascii?Q?8ClyMey67NvqximYXjvmyMilf44qwWTkaD4g8dl/XUvO53Xmq8+liPGrsQMj?= =?us-ascii?Q?N5GRkVRzAOFIJ0JP5fHPUQFEuOvkIq97phTUE4mT45iDcKjS6JTN7RjSVzHr?= =?us-ascii?Q?cpMZsCWguL5yRFzDD/FEIusLJiJmiKhlWm4nTComKPcRYYSFeal3pT4tKbAr?= =?us-ascii?Q?Pm98aE/fY2T9XTKNyxG+i9wI0QAOMTRpJ7eY/hFrPDlhPtN6MqbyTyEiZ5PM?= =?us-ascii?Q?Y4ujLM2n8l/jq/A4JRst2voNUXo3EBSTXYVsUQGfTwKH73a/xBlcPTR+Y15Q?= =?us-ascii?Q?3SwZW0oEtdQ5ORBIB2lqaTYwhsp8MbZHeJ2vO+SUFh+K9ye4eRaze77jxEk3?= =?us-ascii?Q?5Q9vOrXpxEW35sf14E2GgfwzDecW3htJbTJnMTN+CI6THYrDtuQMpTLs0XC4?= =?us-ascii?Q?cwaN1LcH9E+aUiRC3p78kDDJTyjdNeQ5c4LvL0zrkRsWr1GoUD9R+0GkXf23?= =?us-ascii?Q?N1OQqEE3m4Riw3ATl6Z+Ci9TLt5qHVj4kUZoglqkBOEV3s02kg2LGkIDQtSY?= =?us-ascii?Q?R2fTy2jgkDgHNzFRkfMa/5rGSJgmPTSzeFnql6cg9zKZhOv1DyLR6qHsVTV3?= =?us-ascii?Q?mc1eR3AGsrsf7BfASvrotWBdYrjOiudmtiwVjJ2I155IkEDTdwC0dXHb8W0X?= =?us-ascii?Q?3wr6XQaCxcdFVrYMDVaknvxLX6ho9Kuyjc0o2QuWNov+6MzOnjvAlPscmPpG?= =?us-ascii?Q?uI4Kzapx60r9zciXGBGR6hUhfSiwBhDm3BKzn+V+6h6QKZV5j1OI+NIj/Rtq?= =?us-ascii?Q?Crp/x+BFkU66/4LbFP2Z7F9We9n62Vm/f/6MA7YW74MV8k0VD73cjX0VzcuF?= =?us-ascii?Q?OwAIyUj7XsNHZzK4OQOm+qcgrMjXZmVR8xvZHiy+tl6r5Ruo5bmGQn6898aX?= =?us-ascii?Q?8m+phkGIowRJNP7Q0r0ZNExpaa1k3C1S5l49qy4g1XBznxPZlzuBNpbdKUw7?= =?us-ascii?Q?ijfP64r3DznSCiywrpwkWSJY?= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6aa0207c-529d-4210-ed16-08d90ee15984 X-MS-Exchange-CrossTenant-AuthSource: AM7PR08MB5494.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2021 09:45:28.3841 (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: 5JYOOkf2TUeSMiBVhY3UKuaXX70x8AaUnfZ8yurEeFklxAJlCOfTsZYO+C16wx4KzjnbryxUNqleCao0kdYevhD3zG/l31sB5ENzrXI9KRE= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR08MB6947 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=40.107.21.125; envelope-from=vsementsov@virtuozzo.com; helo=EUR05-VI1-obe.outbound.protection.outlook.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 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, MSGID_FROM_MTA_HEADER=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action 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: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @virtuozzo.com) Content-Type: text/plain; charset="utf-8" All existing parent types (block nodes, block devices, jobs) has the realization. So, drop unreachable code. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/block.c b/block.c index 54a3da9311..2f73523285 100644 --- a/block.c +++ b/block.c @@ -2029,11 +2029,7 @@ bool bdrv_is_writable(BlockDriverState *bs) =20 static char *bdrv_child_user_desc(BdrvChild *c) { - if (c->klass->get_parent_desc) { - return c->klass->get_parent_desc(c); - } - - return g_strdup("another user"); + return c->klass->get_parent_desc(c); } =20 static bool bdrv_a_allow_b(BdrvChild *a, BdrvChild *b, Error **errp) --=20 2.29.2 From nobody Thu Mar 28 21:03:57 2024 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass(p=quarantine dis=none) header.from=virtuozzo.com ARC-Seal: i=2; a=rsa-sha256; t=1620121678; cv=pass; d=zohomail.com; s=zohoarc; b=FKtZ+YTf6YO4RRsL7LvbZdLDMWY0KNqs7eHB6UGzEvGiOLMHyNf0zJcTqD8EfNNLUJPgTGcCqbeGNBMuiTsgBd4bPp/tLEKy8h0reCQrFLG2zDc9r6zc8q730XFCef/fh5dDvIKd1ITA3aK+MHSi+SqMDKXLPUbccYS7yca7dJ4= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1620121678; 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=/OvbAEACRnaVoGw9KkxV2qLMM5Yh4YimQFkpvJb3EBU=; b=hcnG3R9QWE7gwKOzj04zb9Egk7QeNprhErTdwuK4PHBz5U+xrnvVCGv4MBlsSdWrexKlaQJKAvAXF6VJ2irNI7xt6yGzBYxzs61nnk/UuDcl0qCTjFICyzF1v35tHDLdd14b+SBtFgUGgnGCdpj9svTLvaBCl4Fg3N4YUYrow2E= 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=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass header.from= (p=quarantine 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 1620121678137854.2891901595953; Tue, 4 May 2021 02:47:58 -0700 (PDT) Received: from localhost ([::1]:38358 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ldrem-0000Fi-Oq for importer@patchew.org; Tue, 04 May 2021 05:47:56 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51378) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ldrcc-0006yV-Tp; Tue, 04 May 2021 05:45:43 -0400 Received: from mail-eopbgr00109.outbound.protection.outlook.com ([40.107.0.109]:61086 helo=EUR02-AM5-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 1ldrcZ-0001Xl-6t; Tue, 04 May 2021 05:45:42 -0400 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) by AS8PR08MB6947.eurprd08.prod.outlook.com (2603:10a6:20b:346::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.24; Tue, 4 May 2021 09:45:29 +0000 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133]) by AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::b403:c1a9:6bb7:133%7]) with mapi id 15.20.4108.024; Tue, 4 May 2021 09:45:29 +0000 Received: from localhost.localdomain (185.215.60.251) by HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.24 via Frontend Transport; Tue, 4 May 2021 09:45:28 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=CMfmTMEHNRX6j3J+oiMZM2EHMmwX1aWVVE7BkDI4O5XDI7QNJKkCrHn+dUWg/raZf2QI9iqtFDebduFeI9jEeYU/sPsH3x5YL8dTdorC9u7L3SCGQ2yZiRiev6MRzdYcxQxz+zeQQrse6OiSfJoFNnWc5E1OwW7wpJmdbUe1cOrXwhxAFshH9qZQrCE986zHX9HF/y6kgwyvdpm1duJTVlGvgDs2kBlTR47uLVwo3is0NbDRKFP4wfpHx0EF361cv1Q9YJcaNNrZtHkzeTZCFGocxiSdehN0cHQhqQQIbkrF+VYOzV6NViidk/icHV9FWO8jent8CIecx+QoJW7qDw== 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-SenderADCheck; bh=/OvbAEACRnaVoGw9KkxV2qLMM5Yh4YimQFkpvJb3EBU=; b=FWSxfHY7Mb90YRl7uFi+5gkaYGRN3VvH+Jva2aXifhUXNYJGvT8U0s8hQkSIqefwE6h4vuwzTADg1Ddi2Jp6KAnWxVVsX28hj04syNznkkL5wQkk3y5hQBqaHkSnl3mQQCtNgoA4oWlTlD0atx3v2ABsn28E8wQwrxn6JwEshhtmuNph1pq1RSUzKtseH32u/IprafC6hWOMwpH0sCsnETakAoUBA5MthBIChWcmRUI1voHHhfunppPuXrZGLkn9avJcaQfs8F7TYSK/2tdkAOfulurvnPTfabJKDJrpLUnskXSvIJ2gDelzlaalla0ldUnkrHV8DL6fGidJfTSb+A== 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=/OvbAEACRnaVoGw9KkxV2qLMM5Yh4YimQFkpvJb3EBU=; b=PX8riebd7XTqVbGQRRaSdZ4IElfHgirjz9BVNhJ3f2DdzXm1vFt/X5aZGYbc8CwhaICNzYf6mcABNcMiX00W2O5ujGeAcxTGT/Fu0tptL7XqxNSw/hxv1WbgXMswRe6zaTYTyO7eWCncwSLpJ70FqEdGpWqF8WDKVfjAJ1TbCrE= Authentication-Results: nongnu.org; dkim=none (message not signed) header.d=none;nongnu.org; dmarc=none action=none header.from=virtuozzo.com; From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Cc: qemu-devel@nongnu.org, mreitz@redhat.com, kwolf@redhat.com, vsementsov@virtuozzo.com, berto@igalia.com Subject: [PATCH v2 5/5] block: improve permission conflict error message Date: Tue, 4 May 2021 12:45:10 +0300 Message-Id: <20210504094510.25032-6-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210504094510.25032-1-vsementsov@virtuozzo.com> References: <20210504094510.25032-1-vsementsov@virtuozzo.com> Content-Transfer-Encoding: quoted-printable X-Originating-IP: [185.215.60.251] X-ClientProxiedBy: HE1PR02CA0107.eurprd02.prod.outlook.com (2603:10a6:7:29::36) To AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: e4efd6d2-ec4a-42d2-0ffa-08d90ee15a1e X-MS-TrafficTypeDiagnostic: AS8PR08MB6947: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:229; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: WdsOhWh4y7LHJJHJKH/aBHd92cCXYl6uHIZqZBd928U8eoGC/VkZ8WbhuaSRoX8KPMkIwnYrV4HZS3x5T0cj9YIvQiWqhM5vKzbBIXp4PGqS2C+6mLB7YfI0JovYW+REfPb61rwuCLzOr1rCV5DD6HXTp93iwfElTrI3MYgXMevLM55mqfsOJ6nVDIf8/nG+CcmttvDigkRHmwMB4uuNbngplxdnij5YodbpyoB7K/wCK5TTcJh7cWbDI9rA1WvYcdY9NREGiPEsOdH+iRaJsND0EvYT95zN383IqZ1m/JMKjc/6oe6h8T8iexA+iGAcHstig8O57pILMF2qfjclftILljBpw87TiHBUTh2Bu+Qfy3OFYYJ7jVvYqVLMYPOSdDdD64wrhXds1nGixR+izOjUJgvIh1FUOCtkaHZUwuLDJjX/7O5tLjWOiJa74HQ3IR01F77h8ATmuZuavZXJ4OxxDXqA87GC9ysvOPaev4XX3WzVHGCTlJ1XoGvXl0rUxcmKt201NBAJPXa3ENLL6n9WohUbUbylYVqoZ+51usLpsQR8xi8kf145PlBFtHQ8RsWZHeR3LHKWMG6+Qjdf47rBZXwakR+aUOZNsLMp4mTWxoCrGz4zyCqRolRpjd9yXsV2uyM+nAlaSiW0H5NBvwdKj3wfs4yu19rohNxTTbtynbNbd4m7ck3cYd4Skk4K X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM7PR08MB5494.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(396003)(136003)(376002)(366004)(346002)(39840400004)(6506007)(15650500001)(66946007)(66476007)(16526019)(52116002)(36756003)(83380400001)(6666004)(186003)(6512007)(86362001)(38350700002)(38100700002)(66556008)(478600001)(4326008)(1076003)(5660300002)(316002)(2906002)(8676002)(8936002)(6486002)(6916009)(956004)(2616005)(26005)(69590400013); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?nihas6wYcOuEgXz6E+s4gI6OmMZ90rXIa5qjcEribsUw1Fwu7frAnIrxpOpg?= =?us-ascii?Q?xbDRoGHzEFMV+6wrGKO0oHwnsiWYkzXz9eI+FvuoVeQm1TJuuHTI4O8gttOh?= =?us-ascii?Q?MzrjqpmSz7x2q7aX3IH7Gn8/N6iOG31sgy7TiF6oVUw20guhiwi78sB3cHnU?= =?us-ascii?Q?J1ZIRQDBavtxN3AsyOsdVLzPhnCUuVDkRJp2hII3V/XdygMftgI9EKtHxazd?= =?us-ascii?Q?nXHwt4eb52r1A6Xd/gvMG9oG9PXqkRh+HMEPv4YcnjyPTMbBYodjPDcq/cda?= =?us-ascii?Q?k0NaARWwpXwfCiFAQf2TKMnR9t6Ab6OIPEEcOGwjekxiVUBKmuSCLQyPqJvg?= =?us-ascii?Q?ty9E1Cz7qqoOxfNIqXjTy6vJRu4SwrDEmoWlcZmEc8y8BSUPTfvQ9lOrbc0k?= =?us-ascii?Q?cgFgbpOYMajPa/IyfBezkCMliv2U/ws6lph7V1aRalWdVmdLwOVm2tAOKu/s?= =?us-ascii?Q?5JpBL8o3V83KLE8EXl3XwYry+gjnTU1PhhrlXqedXp9TVnlE+TNsLvMUvc9w?= =?us-ascii?Q?dCep+2KQrsNmVOMrlqFgRwTUQ5/WOS66uVM6n3dFOcCIHkFQQ+q7nMSs4B0a?= =?us-ascii?Q?nmOWQTzQO1mTQLYmE9h/bKqBdxge/0h/8/0PS3FK6IfuMIRYNPwzX9l2sI3i?= =?us-ascii?Q?IdFsQ6x1jjPrLSzqZQtxyQfsiFx/eYmvinwkCDDDImSea5YJCjhMf9Px4YZI?= =?us-ascii?Q?C8UGtFSD9wQyIKhyUFagRGk0M6G7/tWpCgz1sUZEvnyvZDS1L5bXachoOqq0?= =?us-ascii?Q?Y93KtAUaTqw0ACwjp31TLb6nxBJTFMSPLrhAbakpMajDsv/0Y6eLLgSyvR43?= =?us-ascii?Q?JCbSxyHkmE6q/ZIm9pq9PxRnvjlH3+KJecedXiA7S+8qTyzAMBf9BknJgzB9?= =?us-ascii?Q?YTbSX4a+ZqlzS86D4a7e0Y62mPYjn4J5+/oh5yMuzrbrh7wIFQv3QpDnV0iE?= =?us-ascii?Q?eqdLBkuYF+5+x8jUm0Tx0q1JXXAeyw+c3AOx3p4rsddDdEKJUJxx2IsWlPyT?= =?us-ascii?Q?5LPQtb4yPgK7fyykYQvmURkr6WszPWzyELKqipNE8eP9lP3++9A2bKdd4SbW?= =?us-ascii?Q?gPPmyViBAfI8KqS8UYE513QPVocIvKlDEtrspk2hM0L13lRppUqiSrQAGTud?= =?us-ascii?Q?L5wiSMSEpU06Qat5nfhKaIejIzjpd0S+Ife1DRies2Mgp9vb6E8xw5N1Hh6R?= =?us-ascii?Q?BOoVOOfKReED6PK4koD/3EnzD1laAHlmD2oRtZf92bWa93JEqCtEEl4jA8qU?= =?us-ascii?Q?6ffBR2inrkA1bVRx8UBZq6absEfkc6SKFdSb0CFujQrJvo4zXdYMozzFM1Jg?= =?us-ascii?Q?nVehFhw9uuMGcWaGgsrb42wd?= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: e4efd6d2-ec4a-42d2-0ffa-08d90ee15a1e X-MS-Exchange-CrossTenant-AuthSource: AM7PR08MB5494.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2021 09:45:29.3598 (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: 93Zk06YyhECKgZCGnHbUcCJkoEY/attQzA7Ktn7zKBerx360F6SuYTeqbvPIb+KVaWco1EW7Ae78wVvQb20yqbL7rx616NPRZIdCf9PRq9c= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR08MB6947 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=40.107.0.109; envelope-from=vsementsov@virtuozzo.com; helo=EUR02-AM5-obe.outbound.protection.outlook.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 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, MSGID_FROM_MTA_HEADER=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action 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: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @virtuozzo.com) Content-Type: text/plain; charset="utf-8" Now permissions are updated as follows: 1. do graph modifications ignoring permissions 2. do permission update (of course, we rollback [1] if [2] fails) So, on stage [2] we can't say which users are "old" and which are "new" and exist only since [1]. And current error message is a bit outdated. Let's improve it, to make everything clean. While being here, add also a comment and some good assertions. iotests 283, 307, qsd-jobs outputs are updated. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block.c | 29 ++++++++++++++++++++------- tests/qemu-iotests/283.out | 2 +- tests/qemu-iotests/307.out | 2 +- tests/qemu-iotests/tests/qsd-jobs.out | 2 +- 4 files changed, 25 insertions(+), 10 deletions(-) diff --git a/block.c b/block.c index 2f73523285..354438d918 100644 --- a/block.c +++ b/block.c @@ -2032,20 +2032,35 @@ static char *bdrv_child_user_desc(BdrvChild *c) return c->klass->get_parent_desc(c); } =20 +/* + * Check that @a allows everything that @b needs. @a and @b must reference= same + * child node. + */ static bool bdrv_a_allow_b(BdrvChild *a, BdrvChild *b, Error **errp) { - g_autofree char *user =3D NULL; - g_autofree char *perm_names =3D NULL; + g_autofree char *a_user =3D NULL; + g_autofree char *a_against =3D NULL; + g_autofree char *b_user =3D NULL; + g_autofree char *b_perm =3D NULL; + + assert(a->bs); + assert(a->bs =3D=3D b->bs); =20 if ((b->perm & a->shared_perm) =3D=3D b->perm) { return true; } =20 - perm_names =3D bdrv_perm_names(b->perm & ~a->shared_perm); - user =3D bdrv_child_user_desc(a); - error_setg(errp, "Conflicts with use by %s as '%s', which does not " - "allow '%s' on %s", - user, a->name, perm_names, bdrv_get_node_name(b->bs)); + a_user =3D bdrv_child_user_desc(a); + a_against =3D bdrv_perm_names(b->perm & ~a->shared_perm); + + b_user =3D bdrv_child_user_desc(b); + b_perm =3D bdrv_perm_names(b->perm); + + error_setg(errp, "Permission conflict on node '%s': %s wants to use it= as " + "'%s', which requires these permissions: %s. On the other h= and %s " + "wants to use it as '%s', which doesn't share: %s", + bdrv_get_node_name(b->bs), + b_user, b->name, b_perm, a_user, a->name, a_against); =20 return false; } diff --git a/tests/qemu-iotests/283.out b/tests/qemu-iotests/283.out index c9397bfc44..92f3cc1ed5 100644 --- a/tests/qemu-iotests/283.out +++ b/tests/qemu-iotests/283.out @@ -5,7 +5,7 @@ {"execute": "blockdev-add", "arguments": {"driver": "blkdebug", "image": "= base", "node-name": "other", "take-child-perms": ["write"]}} {"return": {}} {"execute": "blockdev-backup", "arguments": {"device": "source", "sync": "= full", "target": "target"}} -{"error": {"class": "GenericError", "desc": "Cannot append backup-top filt= er: Conflicts with use by node 'source' as 'image', which does not allow 'w= rite' on base"}} +{"error": {"class": "GenericError", "desc": "Cannot append backup-top filt= er: Permission conflict on node 'base': node 'other' wants to use it as 'im= age', which requires these permissions: write. On the other hand node 'sour= ce' wants to use it as 'image', which doesn't share: write"}} =20 =3D=3D=3D backup-top should be gone after job-finalize =3D=3D=3D =20 diff --git a/tests/qemu-iotests/307.out b/tests/qemu-iotests/307.out index 66bf2ddb74..e03932ba4f 100644 --- a/tests/qemu-iotests/307.out +++ b/tests/qemu-iotests/307.out @@ -53,7 +53,7 @@ exports available: 1 =20 =3D=3D=3D Add a writable export =3D=3D=3D {"execute": "block-export-add", "arguments": {"description": "This is the = writable second export", "id": "export1", "name": "export1", "node-name": "= fmt", "type": "nbd", "writable": true, "writethrough": true}} -{"error": {"class": "GenericError", "desc": "Conflicts with use by block d= evice 'sda' as 'root', which does not allow 'write' on fmt"}} +{"error": {"class": "GenericError", "desc": "Permission conflict on node '= fmt': unnamed block device wants to use it as 'root', which requires these = permissions: consistent read, write. On the other hand block device 'sda' w= ants to use it as 'root', which doesn't share: write"}} {"execute": "device_del", "arguments": {"id": "sda"}} {"return": {}} {"data": {"device": "sda", "path": "/machine/peripheral/sda"}, "event": "D= EVICE_DELETED", "timestamp": {"microseconds": "USECS", "seconds": "SECS"}} diff --git a/tests/qemu-iotests/tests/qsd-jobs.out b/tests/qemu-iotests/tes= ts/qsd-jobs.out index 9f52255da8..b0596d2c95 100644 --- a/tests/qemu-iotests/tests/qsd-jobs.out +++ b/tests/qemu-iotests/tests/qsd-jobs.out @@ -16,7 +16,7 @@ QMP_VERSION {"return": {}} {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event"= : "JOB_STATUS_CHANGE", "data": {"status": "created", "id": "job0"}} {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event"= : "JOB_STATUS_CHANGE", "data": {"status": "null", "id": "job0"}} -{"error": {"class": "GenericError", "desc": "Conflicts with use by stream = job 'job0' as 'intermediate node', which does not allow 'write' on fmt_base= "}} +{"error": {"class": "GenericError", "desc": "Permission conflict on node '= fmt_base': unnamed block device wants to use it as 'root', which requires t= hese permissions: consistent read, write. On the other hand stream job 'job= 0' wants to use it as 'intermediate node', which doesn't share: write"}} {"return": {}} {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event"= : "BLOCK_EXPORT_DELETED", "data": {"id": "export1"}} *** done --=20 2.29.2