From nobody Wed Nov 27 10:24:19 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1552319059880227.17747806768637; Mon, 11 Mar 2019 08:44:19 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 10AC45AFF4; Mon, 11 Mar 2019 15:44:18 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id DB9AD60BF1; Mon, 11 Mar 2019 15:44:17 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 98199247E1; Mon, 11 Mar 2019 15:44:17 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id x2BFe58P002328 for ; Mon, 11 Mar 2019 11:40:05 -0400 Received: by smtp.corp.redhat.com (Postfix) id 56AC060C60; Mon, 11 Mar 2019 15:40:05 +0000 (UTC) Received: from mx1.redhat.com (ext-mx04.extmail.prod.ext.phx2.redhat.com [10.5.110.28]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 5008060BF1 for ; Mon, 11 Mar 2019 15:40:00 +0000 (UTC) Received: from mail-qt1-f197.google.com (mail-qt1-f197.google.com [209.85.160.197]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 73E7D88306 for ; Mon, 11 Mar 2019 15:40:00 +0000 (UTC) Received: by mail-qt1-f197.google.com with SMTP id 35so5629986qtq.5 for ; Mon, 11 Mar 2019 08:40:00 -0700 (PDT) Received: from cube-1.lan (pool-173-73-12-108.washdc.fios.verizon.net. [173.73.12.108]) by smtp.gmail.com with ESMTPSA id l129sm3381742qkb.44.2019.03.11.08.39.58 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 11 Mar 2019 08:39:58 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=h3kSry78a01jssNQuKLc0kxevahYtUuD7nG1yYHRccs=; b=QYk6Kxd3b7ELh6wPe+n0PTgUf10IVEwUMYU/BTV6lkG+jODvVo/6XHkvdYaL5EO6d3 o15PqqBfymtNLHp/Lnn0iBE3UweClMO42Eo/zA8HIox0nZDVSRdQl9okUWZPNvSjCDVT zUSMLt2r7VNdVrhUbpztlfUHGP7Mk9Y65Bve055L5IieaQ9j4UC0jqsx3OBT1n85NSsA 8KKgPiIiqZy12zJp5CXHdEcEB/5w9A7fzmaTvVeW4TXU0+1qsX+LHzU2iQvS7JLBmTwQ mU+r1XVun1MZIuSCAhE9E7sQD4CmZbuD4slptcb2NtSTntH/jIxr+wNju77EK+Kb+2/0 CfUA== X-Gm-Message-State: APjAAAUJvqUPqJ8UHgNYS5AO/TgNBtORznQZPwMHWDd9eIX3sOK0VXOl AN+3D1vo0gic9ba2JyM3xwROecL51ezAMX+631O7cQwxbPYNyzsl66MTFEO3vikLdxJ41jYlciL A5d7Br1z6vYvzEXED7VU= X-Received: by 2002:ac8:7545:: with SMTP id b5mr8855954qtr.239.1552318799566; Mon, 11 Mar 2019 08:39:59 -0700 (PDT) X-Google-Smtp-Source: APXvYqxrbUHhJcGH+NA4T4oHevCeotRttb7ujPWfoJQK06Re5YYvWk/ye7EgQLNhFFCWVjwG53LpZA== X-Received: by 2002:ac8:7545:: with SMTP id b5mr8855938qtr.239.1552318799332; Mon, 11 Mar 2019 08:39:59 -0700 (PDT) From: jdillama@redhat.com X-Google-Original-From: dillaman@redhat.com To: libvir-list@redhat.com Date: Mon, 11 Mar 2019 11:39:55 -0400 Message-Id: <20190311153955.4131-3-dillaman@redhat.com> In-Reply-To: <20190311153955.4131-1-dillaman@redhat.com> References: <20190311153955.4131-1-dillaman@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-loop: libvir-list@redhat.com X-Mailman-Approved-At: Mon, 11 Mar 2019 11:44:16 -0400 Cc: Jason Dillaman Subject: [libvirt] [PATCH 2/2] rbd: optionally disable actual disk-usage during pool/volume refresh X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.39]); Mon, 11 Mar 2019 15:44:18 +0000 (UTC) Content-Type: text/plain; charset="utf-8" From: Jason Dillaman For pools with numerous volumes or very large volumes, the disk-usage calculation can take a non-trivial amount of time even with the fast-diff feature enabled (especially since the the usage is calculated serially for each image in the pool). The "rbd:config_opts" node now supports a new libvirt-internal "libvirt_calculate_disk_usage" option which can be set to false to disable this calculation as needed. Signed-off-by: Jason Dillaman --- docs/formatstorage.html.in | 13 ++++++++-- src/storage/storage_backend_rbd.c | 41 ++++++++++++++++++++++++++++--- 2 files changed, 48 insertions(+), 6 deletions(-) diff --git a/docs/formatstorage.html.in b/docs/formatstorage.html.in index 968651330f..eb65edc4d4 100644 --- a/docs/formatstorage.html.in +++ b/docs/formatstorage.html.in @@ -613,12 +613,21 @@ <rbd:option name=3D'client_mount_timeout' value=3D'45'/> <rbd:option name=3D'rados_mon_op_timeout' value=3D'20'/> <rbd:option name=3D'rados_osd_op_timeout' value=3D'10'/> + <rbd:option name=3D'libvirt_calculate_disk_usage' value=3D'false'/&= gt; </rbd:config_opts> </pool> =20 - Since 5.1.0. - + Since 5.1.0. +
+
libvirt_calculate_disk_usage
+
Disable calculating the actual disk usage of RBD volumes when + set to false. This will speed up pool and volume + refresh operations for pools with many volumes or pools with + large images. +
+
+ =20

Storage volume XML

diff --git a/src/storage/storage_backend_rbd.c b/src/storage/storage_backen= d_rbd.c index e67911f928..07ab72f97c 100644 --- a/src/storage/storage_backend_rbd.c +++ b/src/storage/storage_backend_rbd.c @@ -61,6 +61,9 @@ struct _virStoragePoolRBDConfigOptionsDef { =20 #define STORAGE_POOL_RBD_NAMESPACE_HREF "http://libvirt.org/schemas/storag= epool/rbd/1.0" =20 +#define CONFIG_OPTION_INTERNAL_PREFIX "libvirt_" +#define CONFIG_OPTION_CALCULATE_DISK_USAGE CONFIG_OPTION_INTERNAL_PREFIX "= calculate_disk_usage" + static void virStoragePoolDefRBDNamespaceFree(void *nsdata) { @@ -321,6 +324,9 @@ virStorageBackendRBDOpenRADOSConn(virStorageBackendRBDS= tatePtr ptr, char uuidstr[VIR_UUID_STRING_BUFLEN]; =20 for (i =3D 0; i < cmdopts->noptions; i++) { + if (STRPREFIX(cmdopts->names[i], CONFIG_OPTION_INTERNAL_PREFIX= )) + continue; + if (virStorageBackendRBDRADOSConfSet(ptr->cluster, cmdopts->names[i], cmdopts->values[i]) < 0) @@ -527,10 +533,30 @@ virStorageBackendRBDSetAllocation(virStorageVolDefPtr= vol ATTRIBUTE_UNUSED, } #endif =20 +static bool +virStorageBackendRBDCalculateDiskUsage(virStoragePoolDefPtr def) +{ + size_t i; + bool calculate_disk_usage =3D true; + + if (def->namespaceData) { + virStoragePoolRBDConfigOptionsDefPtr cmdopts =3D def->namespaceDat= a; + + for (i =3D 0; i < cmdopts->noptions; i++) { + if (STREQ(cmdopts->names[i], CONFIG_OPTION_CALCULATE_DISK_USAG= E)) { + calculate_disk_usage =3D STRCASEEQ(cmdopts->values[i], "tr= ue"); + break; + } + } + } + return calculate_disk_usage; +} + static int volStorageBackendRBDRefreshVolInfo(virStorageVolDefPtr vol, virStoragePoolObjPtr pool, - virStorageBackendRBDStatePtr ptr) + virStorageBackendRBDStatePtr ptr, + bool calculate_disk_usage) { int ret =3D -1; virStoragePoolDefPtr def =3D virStoragePoolObjGetDef(pool); @@ -564,7 +590,8 @@ volStorageBackendRBDRefreshVolInfo(virStorageVolDefPtr = vol, vol->type =3D VIR_STORAGE_VOL_NETWORK; vol->target.format =3D VIR_STORAGE_FILE_RAW; =20 - if (volStorageBackendRBDUseFastDiff(features, flags)) { + if (calculate_disk_usage && + volStorageBackendRBDUseFastDiff(features, flags)) { VIR_DEBUG("RBD image %s/%s has fast-diff feature enabled. " "Querying for actual allocation", def->source.name, vol->name); @@ -610,8 +637,10 @@ virStorageBackendRBDRefreshPool(virStoragePoolObjPtr p= ool) virStorageBackendRBDStatePtr ptr =3D NULL; struct rados_cluster_stat_t clusterstat; struct rados_pool_stat_t poolstat; + bool calculate_disk_usage =3D virStorageBackendRBDCalculateDiskUsage(d= ef); VIR_AUTOFREE(char *) names =3D NULL; =20 + if (!(ptr =3D virStorageBackendRBDNewState(pool))) goto cleanup; =20 @@ -663,7 +692,8 @@ virStorageBackendRBDRefreshPool(virStoragePoolObjPtr po= ol) =20 name +=3D strlen(name) + 1; =20 - r =3D volStorageBackendRBDRefreshVolInfo(vol, pool, ptr); + r =3D volStorageBackendRBDRefreshVolInfo(vol, pool, ptr, + calculate_disk_usage); =20 /* It could be that a volume has been deleted through a different = route * then libvirt and that will cause a -ENOENT to be returned. @@ -1238,12 +1268,15 @@ virStorageBackendRBDRefreshVol(virStoragePoolObjPtr= pool, virStorageVolDefPtr vol) { virStorageBackendRBDStatePtr ptr =3D NULL; + virStoragePoolDefPtr def =3D virStoragePoolObjGetDef(pool); + bool calculate_disk_usage =3D virStorageBackendRBDCalculateDiskUsage(d= ef); int ret =3D -1; =20 if (!(ptr =3D virStorageBackendRBDNewState(pool))) goto cleanup; =20 - if (volStorageBackendRBDRefreshVolInfo(vol, pool, ptr) < 0) + if (volStorageBackendRBDRefreshVolInfo(vol, pool, ptr, + calculate_disk_usage) < 0) goto cleanup; =20 ret =3D 0; --=20 2.20.1 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list