rpms/libblockdev
5
0
Fork 0
Code Issues Pull Requests Releases Activity

Update to 3.2.0

Browse Source 
Resolves: RHEL-60272
Resolves: RHEL-50820
This commit is contained in:
Vojtech Trefny 2024-10-30 10:01:12 +01:00
parent de1e1d2f17
commit 226f8b9a9f
10 changed files with 10 additions and 11490 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.gitignore vendored
View File

@ -56,3 +56,4 @@
/libblockdev-3.0.4.tar.gz
/libblockdev-3.1.0.tar.gz
/smart-tests.tar.gz
/libblockdev-3.2.0.tar.gz

107
0001-misc-RHEL-10-beta-backport.patch
View File

@ -1,107 +0,0 @@
From 9ca6684291e8a30cce2158285078664f8c771638 Mon Sep 17 00:00:00 2001
From: guazhang <guazhang@redhat.com>
Date: Wed, 27 Mar 2024 11:53:21 +0800
Subject: [PATCH 1/3] fixed md_create issue #1013
---
src/python/gi/overrides/BlockDev.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/python/gi/overrides/BlockDev.py b/src/python/gi/overrides/BlockDev.py
index d830e485..25852f66 100644
--- a/src/python/gi/overrides/BlockDev.py
+++ b/src/python/gi/overrides/BlockDev.py
@@ -1018,7 +1018,7 @@ __all__.append("md_get_superblock_size")
_md_create = BlockDev.md_create
@override(BlockDev.md_create)
-def md_create(device_name, level, disks, spares=0, version=None, bitmap=False, chunk_size=0, extra=None, **kwargs):
+def md_create(device_name, level, disks, spares=0, version=None, bitmap=None, chunk_size=0, extra=None, **kwargs):
extra = _get_extra(extra, kwargs)
return _md_create(device_name, level, disks, spares, version, bitmap, chunk_size, extra)
__all__.append("md_create")
--
2.45.0
From 829cea63351c8d5264573eb3d8953cd0069d2566 Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Wed, 3 Apr 2024 15:58:04 +0200
Subject: [PATCH 2/3] crypto: Fix double free in bd_crypto_luks_remove_key
---
src/plugins/crypto.c | 1 -
tests/crypto_test.py | 6 ++++++
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/plugins/crypto.c b/src/plugins/crypto.c
index 5120ec57..f764593f 100644
--- a/src/plugins/crypto.c
+++ b/src/plugins/crypto.c
@@ -1480,7 +1480,6 @@ gboolean bd_crypto_luks_remove_key (const gchar *device, BDCryptoKeyslotContext
return FALSE;
}
- crypt_safe_free (key_buf);
crypt_free (cd);
bd_utils_report_finished (progress_id, "Completed");
return TRUE;
diff --git a/tests/crypto_test.py b/tests/crypto_test.py
index 6fe15a07..54ec0176 100644
--- a/tests/crypto_test.py
+++ b/tests/crypto_test.py
@@ -524,6 +524,9 @@ class CryptoTestRemoveKey(CryptoTestCase):
succ = BlockDev.crypto_luks_add_key(self.loop_dev, ctx, nctx2)
self.assertTrue(succ)
+ nctx3 = BlockDev.CryptoKeyslotContext(keyfile=self.keyfile)
+ succ = BlockDev.crypto_luks_add_key(self.loop_dev, ctx, nctx3)
+
with self.assertRaises(GLib.GError):
wctx = BlockDev.CryptoKeyslotContext(passphrase="wrong-passphrase")
BlockDev.crypto_luks_remove_key(self.loop_dev, wctx)
@@ -534,6 +537,9 @@ class CryptoTestRemoveKey(CryptoTestCase):
succ = BlockDev.crypto_luks_remove_key(self.loop_dev, nctx2)
self.assertTrue(succ)
+ succ = BlockDev.crypto_luks_remove_key(self.loop_dev, nctx3)
+ self.assertTrue(succ)
+
@tag_test(TestTags.SLOW)
def test_luks_remove_key(self):
self._remove_key(self._luks_format)
--
2.45.0
From c50d43b4d8e9d10b410265c72d2bbf5372eb41e6 Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Tue, 16 Apr 2024 12:02:27 +0200
Subject: [PATCH 3/3] tests: Fix running tests without ntfsprogs
---
tests/fs_tests/generic_test.py | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/tests/fs_tests/generic_test.py b/tests/fs_tests/generic_test.py
index fb890229..df25e25a 100644
--- a/tests/fs_tests/generic_test.py
+++ b/tests/fs_tests/generic_test.py
@@ -342,8 +342,12 @@ class CanResizeRepairCheckLabel(GenericNoDevTestCase):
self.assertEqual(util, "resize2fs")
avail, util = BlockDev.fs_can_get_min_size("ntfs")
- self.assertTrue(avail)
- self.assertEqual(util, None)
+ if self.ntfs_avail:
+ self.assertTrue(avail)
+ self.assertEqual(util, None)
+ else:
+ self.assertFalse(avail)
+ self.assertEqual(util, "ntfsresize")
with self.assertRaises(GLib.GError):
BlockDev.fs_can_get_min_size("xfs")
--
2.45.0

86
0002-Check-also-for-aliases-in-bd_utils_have_kernel_module.patch
View File

@ -1,86 +0,0 @@
From 91c22078e9d369d266c044f949b65ebd611f31b0 Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Tue, 21 May 2024 15:42:29 +0200
Subject: [PATCH] utils: Check also for aliases in bd_utils_have_kernel_module
We want to return true for ext2/3 which are now supported by ext4
module.
---
src/utils/module.c | 12 +++++++++++-
tests/utils_test.py | 22 ++++++++++++++++++++++
2 files changed, 33 insertions(+), 1 deletion(-)
diff --git a/src/utils/module.c b/src/utils/module.c
index 66d02343..71d53b46 100644
--- a/src/utils/module.c
+++ b/src/utils/module.c
@@ -85,6 +85,7 @@ gboolean bd_utils_have_kernel_module (const gchar *module_name, GError **error)
gint ret = 0;
struct kmod_ctx *ctx = NULL;
struct kmod_module *mod = NULL;
+ struct kmod_list *list = NULL;
gchar *null_config = NULL;
const gchar *path = NULL;
gboolean have_path = FALSE;
@@ -100,21 +101,30 @@ gboolean bd_utils_have_kernel_module (const gchar *module_name, GError **error)
}
set_kmod_logging (ctx);
- ret = kmod_module_new_from_name (ctx, module_name, &mod);
+ ret = kmod_module_new_from_lookup (ctx, module_name, &list);
if (ret < 0) {
g_set_error (error, BD_UTILS_MODULE_ERROR, BD_UTILS_MODULE_ERROR_FAIL,
"Failed to get the module: %s", strerror_l (-ret, c_locale));
kmod_unref (ctx);
+ kmod_module_unref_list (list);
+ freelocale (c_locale);
+ return FALSE;
+ }
+
+ if (list == NULL) {
+ kmod_unref (ctx);
freelocale (c_locale);
return FALSE;
}
+ mod = kmod_module_get_module (list);
path = kmod_module_get_path (mod);
have_path = (path != NULL) && (g_strcmp0 (path, "") != 0);
if (!have_path) {
builtin = kmod_module_get_initstate (mod) == KMOD_MODULE_BUILTIN;
}
kmod_module_unref (mod);
+ kmod_module_unref_list (list);
kmod_unref (ctx);
freelocale (c_locale);
diff --git a/tests/utils_test.py b/tests/utils_test.py
index d656d309..372228f8 100644
--- a/tests/utils_test.py
+++ b/tests/utils_test.py
@@ -484,3 +484,25 @@ def test_initialization(self):
self.assertEqual(ver.major, ver2.major)
self.assertEqual(ver.minor, ver2.minor)
self.assertEqual(ver.micro, ver2.micro)
+
+
+class UtilsKernelModuleTest(UtilsTestCase):
+ @tag_test(TestTags.NOSTORAGE, TestTags.CORE)
+ def test_have_kernel_module(self):
+ """ Test checking for kernel modules """
+
+ have = BlockDev.utils_have_kernel_module("definitely-not-existing-kernel-module")
+ self.assertFalse(have)
+
+ # loop should be everywhere, right?
+ have = BlockDev.utils_have_kernel_module("loop")
+ self.assertTrue(have)
+
+ # lets check some filesystems support and compare with 'modprobe' results
+ for fs in ("ext2", "ext3", "ext4", "xfs", "btrfs"):
+ have_fs = BlockDev.utils_have_kernel_module(fs)
+ ret, _out, _err = run_command("modprobe --dry-run %s" % fs)
+ if ret == 0:
+ self.assertTrue(have_fs)
+ else:
+ self.assertFalse(have_fs)

65
0003-Fix-passing-size-for-pvresize-over-DBus.patch
View File

@ -1,65 +0,0 @@
From 7afe64265c9fc6ac67bfc412e43d00e5a21c867a Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Fri, 22 Mar 2024 14:54:15 +0100
Subject: [PATCH] lvm-dbus: Fix passing size for pvresize over DBus
"u" is UINT32, we need to use "t" for UINT64
---
src/plugins/lvm-dbus.c | 2 +-
tests/lvm_dbus_tests.py | 6 ++++++
tests/lvm_test.py | 6 ++++++
3 files changed, 13 insertions(+), 1 deletion(-)
diff --git a/src/plugins/lvm-dbus.c b/src/plugins/lvm-dbus.c
index 5ecf47f6..ff9eb24b 100644
--- a/src/plugins/lvm-dbus.c
+++ b/src/plugins/lvm-dbus.c
@@ -1776,7 +1776,7 @@ gboolean bd_lvm_pvresize (const gchar *device, guint64 size, const BDExtraArg **
if (!obj_path)
return FALSE;
- params = g_variant_new ("(u)", size);
+ params = g_variant_new ("(t)", size);
return call_lvm_method_sync (obj_path, PV_INTF, "ReSize", params, NULL, extra, TRUE, error);
}
diff --git a/tests/lvm_dbus_tests.py b/tests/lvm_dbus_tests.py
index 013a29c3..c179a82c 100644
--- a/tests/lvm_dbus_tests.py
+++ b/tests/lvm_dbus_tests.py
@@ -431,9 +431,15 @@ def test_pvresize(self):
succ = BlockDev.lvm_pvresize(self.loop_dev, 200 * 1024**2, None)
self.assertTrue(succ)
+ info = BlockDev.lvm_pvinfo(self.loop_dev)
+ self.assertEqual(info.pv_size, 200 * 1024**2)
+
succ = BlockDev.lvm_pvresize(self.loop_dev, 200 * 1024**3, None)
self.assertTrue(succ)
+ info = BlockDev.lvm_pvinfo(self.loop_dev)
+ self.assertEqual(info.pv_size, 200 * 1024**3)
+
@unittest.skipUnless(lvm_dbus_running, "LVM DBus not running")
class LvmTestPVscan(LvmPVonlyTestCase):
def test_pvscan(self):
diff --git a/tests/lvm_test.py b/tests/lvm_test.py
index e7800d8f..261615e9 100644
--- a/tests/lvm_test.py
+++ b/tests/lvm_test.py
@@ -434,9 +434,15 @@ def test_pvresize(self):
succ = BlockDev.lvm_pvresize(self.loop_dev, 200 * 1024**2, None)
self.assertTrue(succ)
+ info = BlockDev.lvm_pvinfo(self.loop_dev)
+ self.assertEqual(info.pv_size, 200 * 1024**2)
+
succ = BlockDev.lvm_pvresize(self.loop_dev, 200 * 1024**3, None)
self.assertTrue(succ)
+ info = BlockDev.lvm_pvinfo(self.loop_dev)
+ self.assertEqual(info.pv_size, 200 * 1024**3)
+
class LvmTestPVscan(LvmPVonlyTestCase):
def test_pvscan(self):
"""Verify that pvscan runs without issues with cache or without"""

571
0004-Upstream-kernel-VDO-support.patch
View File

@ -1,571 +0,0 @@
From 5db24c0542a6c7ee44e2d487969ef71170d9cb33 Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Tue, 2 Jul 2024 12:30:42 +0200
Subject: [PATCH 1/2] lvm: Check for dm-vdo instead of kvdo module for VDO
support
VDO is available in the upstream kernel since 6.9 as dm-vdo so we
should check for it instead of the out-of-tree kvdo module.
---
src/plugins/lvm-dbus.c | 2 +-
src/plugins/lvm.c | 2 +-
tests/lvm_dbus_tests.py | 4 ++--
tests/lvm_test.py | 4 ++--
4 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/plugins/lvm-dbus.c b/src/plugins/lvm-dbus.c
index d080b389..0c017306 100644
--- a/src/plugins/lvm-dbus.c
+++ b/src/plugins/lvm-dbus.c
@@ -321,7 +321,7 @@ static const UtilFeatureDep features[FEATURES_LAST] = {
#define MODULE_DEPS_VDO_MASK (1 << MODULE_DEPS_VDO)
#define MODULE_DEPS_LAST 1
-static const gchar*const module_deps[MODULE_DEPS_LAST] = { "kvdo" };
+static const gchar*const module_deps[MODULE_DEPS_LAST] = { "dm-vdo" };
/**
* bd_lvm_init:
diff --git a/src/plugins/lvm.c b/src/plugins/lvm.c
index 361a084b..dc7491a7 100644
--- a/src/plugins/lvm.c
+++ b/src/plugins/lvm.c
@@ -329,7 +329,7 @@ static const UtilFeatureDep features[FEATURES_LAST] = {
#define MODULE_DEPS_VDO_MASK (1 << MODULE_DEPS_VDO)
#define MODULE_DEPS_LAST 1
-static const gchar*const module_deps[MODULE_DEPS_LAST] = { "kvdo" };
+static const gchar*const module_deps[MODULE_DEPS_LAST] = { "dm-vdo" };
#define UNUSED __attribute__((unused))
diff --git a/tests/lvm_dbus_tests.py b/tests/lvm_dbus_tests.py
index fc270cb5..9f302611 100644
--- a/tests/lvm_dbus_tests.py
+++ b/tests/lvm_dbus_tests.py
@@ -2029,11 +2029,11 @@ class LVMVDOTest(LVMTestCase):
@classmethod
def setUpClass(cls):
- if not BlockDev.utils_have_kernel_module("kvdo"):
+ if not BlockDev.utils_have_kernel_module("dm-vdo"):
raise unittest.SkipTest("VDO kernel module not available, skipping.")
try:
- BlockDev.utils_load_kernel_module("kvdo")
+ BlockDev.utils_load_kernel_module("dm-vdo")
except GLib.GError as e:
if "File exists" not in e.message:
raise unittest.SkipTest("cannot load VDO kernel module, skipping.")
diff --git a/tests/lvm_test.py b/tests/lvm_test.py
index a4c90172..b1d65baf 100644
--- a/tests/lvm_test.py
+++ b/tests/lvm_test.py
@@ -1933,11 +1933,11 @@ class LVMVDOTest(LVMTestCase):
@classmethod
def setUpClass(cls):
- if not BlockDev.utils_have_kernel_module("kvdo"):
+ if not BlockDev.utils_have_kernel_module("dm-vdo"):
raise unittest.SkipTest("VDO kernel module not available, skipping.")
try:
- BlockDev.utils_load_kernel_module("kvdo")
+ BlockDev.utils_load_kernel_module("dm-vdo")
except GLib.GError as e:
if "File exists" not in e.message:
raise unittest.SkipTest("cannot load VDO kernel module, skipping.")
--
2.45.2
From 6cf4de50d385d801c1936fc67ee2902f9c60e04e Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Tue, 2 Jul 2024 12:32:46 +0200
Subject: [PATCH 2/2] lvm: Get VDO stats from device mapper instead of
/sys/kvdo
The /sys/kvdo API has been removed so we need to get the full
statistics using libdevmapper now.
---
configure.ac | 4 +
dist/libblockdev.spec.in | 2 +
src/lib/plugin_apis/lvm.api | 6 +-
src/plugins/Makefile.am | 8 +-
src/plugins/lvm-dbus.c | 18 +--
src/plugins/lvm.c | 18 +--
src/plugins/vdo_stats.c | 213 ++++++++++++++++++++++--------------
tests/lvm_dbus_tests.py | 4 +-
tests/lvm_test.py | 10 +-
9 files changed, 175 insertions(+), 108 deletions(-)
diff --git a/configure.ac b/configure.ac
index 02b26e3e..cc875bb4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -193,6 +193,10 @@ AS_IF([test "x$with_dm" != "xno" -o "x$with_lvm" != "xno" -o "x$with_lvm_dbus" !
[LIBBLOCKDEV_PKG_CHECK_MODULES([DEVMAPPER], [devmapper >= 1.02.93])],
[])
+AS_IF([test "x$with_lvm" != "xno" -o "x$with_lvm_dbus" != "xno"],
+ [LIBBLOCKDEV_PKG_CHECK_MODULES([YAML], [yaml-0.1])],
+ [])
+
AS_IF([test "x$with_part" != "xno"],
[LIBBLOCKDEV_PKG_CHECK_MODULES([FDISK], [fdisk >= 2.31.0])],
[])
diff --git a/dist/libblockdev.spec.in b/dist/libblockdev.spec.in
index 768aa0c0..e8703411 100644
--- a/dist/libblockdev.spec.in
+++ b/dist/libblockdev.spec.in
@@ -290,6 +290,7 @@ with the libblockdev-loop plugin/library.
%if %{with_lvm}
%package lvm
BuildRequires: device-mapper-devel
+BuildRequires: libyaml-devel
Summary: The LVM plugin for the libblockdev library
Requires: %{name}-utils%{?_isa} = %{version}-%{release}
Requires: lvm2
@@ -312,6 +313,7 @@ with the libblockdev-lvm plugin/library.
%if %{with_lvm_dbus}
%package lvm-dbus
BuildRequires: device-mapper-devel
+BuildRequires: libyaml-devel
Summary: The LVM plugin for the libblockdev library
Requires: %{name}-utils%{?_isa} = %{version}-%{release}
Requires: lvm2-dbusd >= 2.02.156
diff --git a/src/lib/plugin_apis/lvm.api b/src/lib/plugin_apis/lvm.api
index 2152aa79..3dd5ea4a 100644
--- a/src/lib/plugin_apis/lvm.api
+++ b/src/lib/plugin_apis/lvm.api
@@ -1965,10 +1965,10 @@ BDLVMVDOWritePolicy bd_lvm_get_vdo_write_policy_from_str (const gchar *policy_st
* statistics or %NULL in case of error
* (@error gets populated in those cases)
*
- * Statistics are collected from the values exposed by the kernel `kvdo` module
- * at the `/sys/kvdo/<VDO_NAME>/statistics/` path.
+ * Statistics are collected from the values exposed by the kernel `dm-vdo` module.
+ *
* Some of the keys are computed to mimic the information produced by the vdo tools.
- * Please note the contents of the hashtable may vary depending on the actual kvdo module version.
+ * Please note the contents of the hashtable may vary depending on the actual dm-vdo module version.
*
* Tech category: %BD_LVM_TECH_VDO-%BD_LVM_TECH_MODE_QUERY
*/
diff --git a/src/plugins/Makefile.am b/src/plugins/Makefile.am
index 0195e9e1..71bacb7c 100644
--- a/src/plugins/Makefile.am
+++ b/src/plugins/Makefile.am
@@ -97,16 +97,16 @@ libbd_loop_la_SOURCES = loop.c loop.h
endif
if WITH_LVM
-libbd_lvm_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(DEVMAPPER_CFLAGS) -Wall -Wextra -Werror
-libbd_lvm_la_LIBADD = ${builddir}/../utils/libbd_utils.la -lm $(GLIB_LIBS) $(GIO_LIBS) $(DEVMAPPER_LIBS)
+libbd_lvm_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(DEVMAPPER_CFLAGS) $(YAML_CFLAGS) -Wall -Wextra -Werror
+libbd_lvm_la_LIBADD = ${builddir}/../utils/libbd_utils.la -lm $(GLIB_LIBS) $(GIO_LIBS) $(DEVMAPPER_LIBS) $(YAML_LIBS)
libbd_lvm_la_LDFLAGS = -L${srcdir}/../utils/ -version-info 3:0:0 -Wl,--no-undefined -export-symbols-regex '^bd_.*'
libbd_lvm_la_CPPFLAGS = -I${builddir}/../../include/
libbd_lvm_la_SOURCES = lvm.c lvm.h check_deps.c check_deps.h dm_logging.c dm_logging.h vdo_stats.c vdo_stats.h
endif
if WITH_LVM_DBUS
-libbd_lvm_dbus_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(DEVMAPPER_CFLAGS) -Wall -Wextra -Werror
-libbd_lvm_dbus_la_LIBADD = ${builddir}/../utils/libbd_utils.la -lm $(GLIB_LIBS) $(GIO_LIBS) $(DEVMAPPER_LIBS)
+libbd_lvm_dbus_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(DEVMAPPER_CFLAGS) $(YAML_CFLAGS) -Wall -Wextra -Werror
+libbd_lvm_dbus_la_LIBADD = ${builddir}/../utils/libbd_utils.la -lm $(GLIB_LIBS) $(GIO_LIBS) $(DEVMAPPER_LIBS) $(YAML_LIBS)
libbd_lvm_dbus_la_LDFLAGS = -L${srcdir}/../utils/ -version-info 3:0:0 -Wl,--no-undefined -export-symbols-regex '^bd_.*'
libbd_lvm_dbus_la_CPPFLAGS = -I${builddir}/../../include/
libbd_lvm_dbus_la_SOURCES = lvm-dbus.c lvm.h check_deps.c check_deps.h dm_logging.c dm_logging.h vdo_stats.c vdo_stats.h
diff --git a/src/plugins/lvm-dbus.c b/src/plugins/lvm-dbus.c
index 0c017306..92d90718 100644
--- a/src/plugins/lvm-dbus.c
+++ b/src/plugins/lvm-dbus.c
@@ -4701,10 +4701,10 @@ BDLVMVDOWritePolicy bd_lvm_get_vdo_write_policy_from_str (const gchar *policy_st
* statistics or %NULL in case of error
* (@error gets populated in those cases)
*
- * Statistics are collected from the values exposed by the kernel `kvdo` module
- * at the `/sys/kvdo/<VDO_NAME>/statistics/` path.
+ * Statistics are collected from the values exposed by the kernel `dm-vdo` module.
+ *
* Some of the keys are computed to mimic the information produced by the vdo tools.
- * Please note the contents of the hashtable may vary depending on the actual kvdo module version.
+ * Please note the contents of the hashtable may vary depending on the actual dm-vdo module version.
*
* Tech category: %BD_LVM_TECH_VDO-%BD_LVM_TECH_MODE_QUERY
*/
@@ -4736,12 +4736,12 @@ BDLVMVDOStats* bd_lvm_vdo_get_stats (const gchar *vg_name, const gchar *pool_nam
return NULL;
stats = g_new0 (BDLVMVDOStats, 1);
- get_stat_val64_default (full_stats, "block_size", &stats->block_size, -1);
- get_stat_val64_default (full_stats, "logical_block_size", &stats->logical_block_size, -1);
- get_stat_val64_default (full_stats, "physical_blocks", &stats->physical_blocks, -1);
- get_stat_val64_default (full_stats, "data_blocks_used", &stats->data_blocks_used, -1);
- get_stat_val64_default (full_stats, "overhead_blocks_used", &stats->overhead_blocks_used, -1);
- get_stat_val64_default (full_stats, "logical_blocks_used", &stats->logical_blocks_used, -1);
+ get_stat_val64_default (full_stats, "blockSize", &stats->block_size, -1);
+ get_stat_val64_default (full_stats, "logicalBlockSize", &stats->logical_block_size, -1);
+ get_stat_val64_default (full_stats, "physicalBlocks", &stats->physical_blocks, -1);
+ get_stat_val64_default (full_stats, "dataBlocksUsed", &stats->data_blocks_used, -1);
+ get_stat_val64_default (full_stats, "overheadBlocksUsed", &stats->overhead_blocks_used, -1);
+ get_stat_val64_default (full_stats, "logicalBlocksUsed", &stats->logical_blocks_used, -1);
get_stat_val64_default (full_stats, "usedPercent", &stats->used_percent, -1);
get_stat_val64_default (full_stats, "savingPercent", &stats->saving_percent, -1);
if (!get_stat_val_double (full_stats, "writeAmplificationRatio", &stats->write_amplification_ratio))
diff --git a/src/plugins/lvm.c b/src/plugins/lvm.c
index dc7491a7..0af9a382 100644
--- a/src/plugins/lvm.c
+++ b/src/plugins/lvm.c
@@ -3799,10 +3799,10 @@ BDLVMVDOWritePolicy bd_lvm_get_vdo_write_policy_from_str (const gchar *policy_st
* statistics or %NULL in case of error
* (@error gets populated in those cases)
*
- * Statistics are collected from the values exposed by the kernel `kvdo` module
- * at the `/sys/kvdo/<VDO_NAME>/statistics/` path.
+ * Statistics are collected from the values exposed by the kernel `dm-vdo` module.
+ *
* Some of the keys are computed to mimic the information produced by the vdo tools.
- * Please note the contents of the hashtable may vary depending on the actual kvdo module version.
+ * Please note the contents of the hashtable may vary depending on the actual dm-vdo module version.
*
* Tech category: %BD_LVM_TECH_VDO-%BD_LVM_TECH_MODE_QUERY
*/
@@ -3834,12 +3834,12 @@ BDLVMVDOStats* bd_lvm_vdo_get_stats (const gchar *vg_name, const gchar *pool_nam
return NULL;
stats = g_new0 (BDLVMVDOStats, 1);
- get_stat_val64_default (full_stats, "block_size", &stats->block_size, -1);
- get_stat_val64_default (full_stats, "logical_block_size", &stats->logical_block_size, -1);
- get_stat_val64_default (full_stats, "physical_blocks", &stats->physical_blocks, -1);
- get_stat_val64_default (full_stats, "data_blocks_used", &stats->data_blocks_used, -1);
- get_stat_val64_default (full_stats, "overhead_blocks_used", &stats->overhead_blocks_used, -1);
- get_stat_val64_default (full_stats, "logical_blocks_used", &stats->logical_blocks_used, -1);
+ get_stat_val64_default (full_stats, "blockSize", &stats->block_size, -1);
+ get_stat_val64_default (full_stats, "logicalBlockSize", &stats->logical_block_size, -1);
+ get_stat_val64_default (full_stats, "physicalBlocks", &stats->physical_blocks, -1);
+ get_stat_val64_default (full_stats, "dataBlocksUsed", &stats->data_blocks_used, -1);
+ get_stat_val64_default (full_stats, "overheadBlocksUsed", &stats->overhead_blocks_used, -1);
+ get_stat_val64_default (full_stats, "logicalBlocksUsed", &stats->logical_blocks_used, -1);
get_stat_val64_default (full_stats, "usedPercent", &stats->used_percent, -1);
get_stat_val64_default (full_stats, "savingPercent", &stats->saving_percent, -1);
if (!get_stat_val_double (full_stats, "writeAmplificationRatio", &stats->write_amplification_ratio))
diff --git a/src/plugins/vdo_stats.c b/src/plugins/vdo_stats.c
index 620e972f..ab914821 100644
--- a/src/plugins/vdo_stats.c
+++ b/src/plugins/vdo_stats.c
@@ -19,10 +19,11 @@
#include <glib.h>
#include <blockdev/utils.h>
+#include <libdevmapper.h>
+#include <yaml.h>
#include "vdo_stats.h"
-
-#define VDO_SYS_PATH "/sys/kvdo"
+#include "lvm.h"
gboolean __attribute__ ((visibility ("hidden")))
@@ -67,9 +68,9 @@ get_stat_val_double (GHashTable *stats, const gchar *key, gdouble *val) {
static void add_write_ampl_r_stats (GHashTable *stats) {
gint64 bios_meta_write, bios_out_write, bios_in_write;
- if (! get_stat_val64 (stats, "bios_meta_write", &bios_meta_write) ||
- ! get_stat_val64 (stats, "bios_out_write", &bios_out_write) ||
- ! get_stat_val64 (stats, "bios_in_write", &bios_in_write))
+ if (! get_stat_val64 (stats, "biosMetaWrite", &bios_meta_write) ||
+ ! get_stat_val64 (stats, "biosOutWrite", &bios_out_write) ||
+ ! get_stat_val64 (stats, "biosInWrite", &bios_in_write))
return;
if (bios_in_write <= 0)
@@ -84,11 +85,11 @@ static void add_block_stats (GHashTable *stats) {
gint64 physical_blocks, block_size, data_blocks_used, overhead_blocks_used, logical_blocks_used;
gint64 savings;
- if (! get_stat_val64 (stats, "physical_blocks", &physical_blocks) ||
- ! get_stat_val64 (stats, "block_size", &block_size) ||
- ! get_stat_val64 (stats, "data_blocks_used", &data_blocks_used) ||
- ! get_stat_val64 (stats, "overhead_blocks_used", &overhead_blocks_used) ||
- ! get_stat_val64 (stats, "logical_blocks_used", &logical_blocks_used))
+ if (! get_stat_val64 (stats, "physicalBlocks", &physical_blocks) ||
+ ! get_stat_val64 (stats, "blockSize", &block_size) ||
+ ! get_stat_val64 (stats, "dataBlocksUsed", &data_blocks_used) ||
+ ! get_stat_val64 (stats, "overheadBlocksUsed", &overhead_blocks_used) ||
+ ! get_stat_val64 (stats, "logicalBlocksUsed", &logical_blocks_used))
return;
g_hash_table_replace (stats, g_strdup ("oneKBlocks"), g_strdup_printf ("%"G_GINT64_FORMAT, physical_blocks * block_size / 1024));
@@ -105,24 +106,24 @@ static void add_journal_stats (GHashTable *stats) {
gint64 journal_entries_committed, journal_entries_started, journal_entries_written;
gint64 journal_blocks_committed, journal_blocks_started, journal_blocks_written;
- if (! get_stat_val64 (stats, "journal_entries_committed", &journal_entries_committed) ||
- ! get_stat_val64 (stats, "journal_entries_started", &journal_entries_started) ||
- ! get_stat_val64 (stats, "journal_entries_written", &journal_entries_written) ||
- ! get_stat_val64 (stats, "journal_blocks_committed", &journal_blocks_committed) ||
- ! get_stat_val64 (stats, "journal_blocks_started", &journal_blocks_started) ||
- ! get_stat_val64 (stats, "journal_blocks_written", &journal_blocks_written))
+ if (! get_stat_val64 (stats, "journalEntriesCommitted", &journal_entries_committed) ||
+ ! get_stat_val64 (stats, "journalEntriesStarted", &journal_entries_started) ||
+ ! get_stat_val64 (stats, "journalEntriesWritten", &journal_entries_written) ||
+ ! get_stat_val64 (stats, "journalBlocksCommitted", &journal_blocks_committed) ||
+ ! get_stat_val64 (stats, "journalBlocksStarted", &journal_blocks_started) ||
+ ! get_stat_val64 (stats, "journalBlocksWritten", &journal_blocks_written))
return;
- g_hash_table_replace (stats, g_strdup ("journal_entries_batching"), g_strdup_printf ("%"G_GINT64_FORMAT, journal_entries_started - journal_entries_written));
- g_hash_table_replace (stats, g_strdup ("journal_entries_writing"), g_strdup_printf ("%"G_GINT64_FORMAT, journal_entries_written - journal_entries_committed));
- g_hash_table_replace (stats, g_strdup ("journal_blocks_batching"), g_strdup_printf ("%"G_GINT64_FORMAT, journal_blocks_started - journal_blocks_written));
- g_hash_table_replace (stats, g_strdup ("journal_blocks_writing"), g_strdup_printf ("%"G_GINT64_FORMAT, journal_blocks_written - journal_blocks_committed));
+ g_hash_table_replace (stats, g_strdup ("journalEntriesBatching"), g_strdup_printf ("%"G_GINT64_FORMAT, journal_entries_started - journal_entries_written));
+ g_hash_table_replace (stats, g_strdup ("journalEntriesWriting"), g_strdup_printf ("%"G_GINT64_FORMAT, journal_entries_written - journal_entries_committed));
+ g_hash_table_replace (stats, g_strdup ("journalBlocksBatching"), g_strdup_printf ("%"G_GINT64_FORMAT, journal_blocks_started - journal_blocks_written));
+ g_hash_table_replace (stats, g_strdup ("journalBlocksWriting"), g_strdup_printf ("%"G_GINT64_FORMAT, journal_blocks_written - journal_blocks_committed));
}
static void add_computed_stats (GHashTable *stats) {
const gchar *s;
- s = g_hash_table_lookup (stats, "logical_block_size");
+ s = g_hash_table_lookup (stats, "logicalBlockSize");
g_hash_table_replace (stats,
g_strdup ("fiveTwelveByteEmulation"),
g_strdup ((g_strcmp0 (s, "512") == 0) ? "true" : "false"));
@@ -132,76 +133,128 @@ static void add_computed_stats (GHashTable *stats) {
add_journal_stats (stats);
}
-static gchar* _dm_node_from_name (const gchar *map_name, GError **error) {
- gchar *dev_path = NULL;
- gchar *ret = NULL;
- gchar *dev_mapper_path = g_strdup_printf ("/dev/mapper/%s", map_name);
+enum parse_flags {
+ PARSE_NEXT_KEY,
+ PARSE_NEXT_VAL,
+ PARSE_NEXT_IGN,
+};
- dev_path = bd_utils_resolve_device (dev_mapper_path, error);
- g_free (dev_mapper_path);
- if (!dev_path)
- /* error is already populated */
+GHashTable __attribute__ ((visibility ("hidden")))
+*vdo_get_stats_full (const gchar *name, GError **error) {
+ struct dm_task *dmt = NULL;
+ const gchar *response = NULL;
+ yaml_parser_t parser;
+ yaml_token_t token;
+ GHashTable *stats = NULL;
+ gchar *key = NULL;
+ gsize len = 0;
+ int next_token = PARSE_NEXT_IGN;
+ gchar *prefix = NULL;
+
+ dmt = dm_task_create (DM_DEVICE_TARGET_MSG);
+ if (!dmt) {
+ g_set_error (error, BD_LVM_ERROR, BD_LVM_ERROR_DM_ERROR,
+ "Failed to create DM task");
return NULL;
+ }
- ret = g_path_get_basename (dev_path);
- g_free (dev_path);
+ if (!dm_task_set_name (dmt, name)) {
+ g_set_error (error, BD_LVM_ERROR, BD_LVM_ERROR_DM_ERROR,
+ "Failed to set name for DM task");
+ dm_task_destroy (dmt);
+ return NULL;
+ }
- return ret;
-}
+ if (!dm_task_set_message (dmt, "stats")) {
+ g_set_error (error, BD_LVM_ERROR, BD_LVM_ERROR_DM_ERROR,
+ "Failed to set message for DM task");
+ dm_task_destroy (dmt);
+ return NULL;
+ }
-GHashTable __attribute__ ((visibility ("hidden")))
-*vdo_get_stats_full (const gchar *name, GError **error) {
- GHashTable *stats;
- GDir *dir;
- gchar *stats_dir;
- const gchar *direntry;
- gchar *s;
- gchar *val = NULL;
- g_autofree gchar *dm_node = NULL;
- GError *l_error = NULL;
-
- /* try "new" (kvdo >= 8) path first -- /sys/block/dm-X/vdo/statistics */
- dm_node = _dm_node_from_name (name, error);
- if (dm_node == NULL) {
- g_prefix_error (error, "Failed to get DM node for %s: ", name);
+ if (!dm_task_run (dmt)) {
+ g_set_error (error, BD_LVM_ERROR, BD_LVM_ERROR_DM_ERROR,
+ "Failed to run DM task");
+ dm_task_destroy (dmt);
+ return NULL;
+ }
+
+ response = dm_task_get_message_response (dmt);
+ if (!response) {
+ g_set_error (error, BD_LVM_ERROR, BD_LVM_ERROR_DM_ERROR,
+ "Failed to get response from the DM task");
+ dm_task_destroy (dmt);
return NULL;
}
- stats_dir = g_build_path (G_DIR_SEPARATOR_S, "/sys/block", dm_node, "vdo/statistics", NULL);
- dir = g_dir_open (stats_dir, 0, &l_error);
- if (dir == NULL) {
- bd_utils_log_format (BD_UTILS_LOG_INFO,
- "Failed to read VDO stats using the new API, falling back to %s: %s",
- VDO_SYS_PATH, l_error->message);
- g_free (stats_dir);
- g_clear_error (&l_error);
-
- /* lets try /sys/kvdo */
- stats_dir = g_build_path (G_DIR_SEPARATOR_S, VDO_SYS_PATH, name, "statistics", NULL);
- dir = g_dir_open (stats_dir, 0, error);
- if (dir == NULL) {
- g_prefix_error (error, "Error reading statistics from %s: ", stats_dir);
- g_free (stats_dir);
- return NULL;
- }
+ if (!yaml_parser_initialize (&parser)) {
+ g_set_error (error, BD_LVM_ERROR, BD_LVM_ERROR_DM_ERROR,
+ "Failed to get initialize YAML parser");
+ dm_task_destroy (dmt);
+ return NULL;
}
stats = g_hash_table_new_full (g_str_hash, g_str_equal, g_free, g_free);
- while ((direntry = g_dir_read_name (dir))) {
- s = g_build_filename (stats_dir, direntry, NULL);
- if (! g_file_get_contents (s, &val, NULL, error)) {
- g_prefix_error (error, "Error reading statistics from %s: ", s);
- g_free (s);
- g_hash_table_destroy (stats);
- stats = NULL;
- break;
- }
- g_hash_table_replace (stats, g_strdup (direntry), g_strdup (g_strstrip (val)));
- g_free (val);
- g_free (s);
- }
- g_dir_close (dir);
- g_free (stats_dir);
+ yaml_parser_set_input_string (&parser, (guchar *) response, strlen (response));
+
+ do {
+ yaml_parser_scan (&parser, &token);
+ switch (token.type) {
+ /* key */
+ case YAML_KEY_TOKEN:
+ next_token = PARSE_NEXT_KEY;
+ break;
+ /* value */
+ case YAML_VALUE_TOKEN:
+ next_token = PARSE_NEXT_VAL;
+ break;
+ /* block mapping */
+ case YAML_BLOCK_MAPPING_START_TOKEN:
+ if (next_token == PARSE_NEXT_VAL)
+ /* we were expecting to read a key-value pair but this is actually
+ a block start, so we need to free the key we're not going to use */
+ g_free (key);
+ break;
+ /* mapping */
+ case YAML_FLOW_MAPPING_START_TOKEN:
+ /* start of flow mapping -> previously read key will be used as prefix
+ for all keys in the mapping:
+ previous key: biosInProgress
+ keys in the mapping: Read, Write...
+ with prefix: biosInProgressRead, biosInProgressWrite...
+ */
+ prefix = key;
+ break;
+ case YAML_FLOW_MAPPING_END_TOKEN:
+ /* end of flow mapping, discard the prefix used */
+ g_free (prefix);
+ prefix = NULL;
+ break;
+ /* actual data */
+ case YAML_SCALAR_TOKEN:
+ if (next_token == PARSE_NEXT_KEY) {
+ if (prefix) {
+ key = g_strdup_printf ("%s%s", prefix, (const gchar *) token.data.scalar.value);
+ len = strlen (prefix);
+ /* make sure the key with the prefix is still camelCase */
+ key[len] = g_ascii_toupper (key[len]);
+ } else
+ key = g_strdup ((const gchar *) token.data.scalar.value);
+ } else if (next_token == PARSE_NEXT_VAL) {
+ gchar *val = g_strdup ((const gchar *) token.data.scalar.value);
+ g_hash_table_insert (stats, key, val);
+ }
+ break;
+ default:
+ break;
+ }
+
+ if (token.type != YAML_STREAM_END_TOKEN)
+ yaml_token_delete (&token);
+ } while (token.type != YAML_STREAM_END_TOKEN);
+
+ yaml_parser_delete (&parser);
+ dm_task_destroy (dmt);
if (stats != NULL)
add_computed_stats (stats);
diff --git a/tests/lvm_dbus_tests.py b/tests/lvm_dbus_tests.py
index 9f302611..db0d5c90 100644
--- a/tests/lvm_dbus_tests.py
+++ b/tests/lvm_dbus_tests.py
@@ -2090,7 +2090,9 @@ class LVMVDOTest(LVMTestCase):
pool_info = BlockDev.lvm_lvinfo("testVDOVG", "vdoPool")
self.assertEqual(pool_info.segtype, "vdo-pool")
self.assertEqual(pool_info.data_lv, "vdoPool_vdata")
- self.assertGreater(pool_info.data_percent, 0)
+ lvm_version = self._get_lvm_version()
+ if lvm_version >= Version("2.03.24"):
+ self.assertGreater(pool_info.data_percent, 0)
pool = BlockDev.lvm_vdolvpoolname("testVDOVG", "vdoLV")
self.assertEqual(pool, lv_info.pool_lv)
diff --git a/tests/lvm_test.py b/tests/lvm_test.py
index b1d65baf..96aa352a 100644
--- a/tests/lvm_test.py
+++ b/tests/lvm_test.py
@@ -1994,7 +1994,9 @@ class LVMVDOTest(LVMTestCase):
pool_info = BlockDev.lvm_lvinfo("testVDOVG", "vdoPool")
self.assertEqual(pool_info.segtype, "vdo-pool")
self.assertEqual(pool_info.data_lv, "vdoPool_vdata")
- self.assertGreater(pool_info.data_percent, 0)
+ lvm_version = self._get_lvm_version()
+ if lvm_version >= Version("2.03.24"):
+ self.assertGreater(pool_info.data_percent, 0)
pool = BlockDev.lvm_vdolvpoolname("testVDOVG", "vdoLV")
self.assertEqual(pool, lv_info.pool_lv)
@@ -2155,7 +2157,11 @@ class LVMVDOTest(LVMTestCase):
self.assertTrue(vdo_info.deduplication)
vdo_stats = BlockDev.lvm_vdo_get_stats("testVDOVG", "vdoPool")
- self.assertEqual(vdo_info.saving_percent, vdo_stats.saving_percent)
+
+ lvm_version = self._get_lvm_version()
+ if lvm_version >= Version("2.03.24"):
+ # saving_percent is incorrect with LVM < 2.03.24
+ self.assertEqual(vdo_info.saving_percent, vdo_stats.saving_percent)
# just sanity check
self.assertNotEqual(vdo_stats.used_percent, -1)
--
2.45.2

86
0005-libext2fs-unused-parameters-fix.patch
View File

@ -1,86 +0,0 @@
From e1c3c3e3555963104a8de00b510b69375eee5ced Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Fri, 7 Jun 2024 10:06:22 +0200
Subject: [PATCH 1/3] fs: Ignore unused-parameter warning in the FS plugin
There are some unused parameters in the libext2fs header which
together with Wall and Werror means the plugin compilation fails.
As a workaround we'll disable the unused-parameter warning for
now.
Fixes: #1026
---
src/plugins/fs/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/plugins/fs/Makefile.am b/src/plugins/fs/Makefile.am
index 7d4849c0..00474d5a 100644
--- a/src/plugins/fs/Makefile.am
+++ b/src/plugins/fs/Makefile.am
@@ -2,7 +2,7 @@ AUTOMAKE_OPTIONS = subdir-objects
lib_LTLIBRARIES = libbd_fs.la
-libbd_fs_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(BLKID_CFLAGS) $(MOUNT_CFLAGS) $(UUID_CFLAGS) $(EXT2FS_CFLAGS) -Wall -Wextra -Werror
+libbd_fs_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(BLKID_CFLAGS) $(MOUNT_CFLAGS) $(UUID_CFLAGS) $(EXT2FS_CFLAGS) -Wall -Wextra -Werror -Wno-unused-parameter
libbd_fs_la_LIBADD = ${builddir}/../../utils/libbd_utils.la $(GLIB_LIBS) $(GIO_LIBS) $(BLKID_LIBS) $(MOUNT_LIBS) $(UUID_LIBS) $(EXT2FS_LIBS)
libbd_fs_la_LDFLAGS = -L${srcdir}/../../utils/ -version-info 3:0:0 -Wl,--no-undefined -export-symbols-regex '^bd_.*'
libbd_fs_la_CPPFLAGS = -I${builddir}/../../../include/ -I${srcdir}/../
--
2.45.2
From 30e7d0bd7b78b5d9a2ddd031d0cdf9f45187a717 Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Fri, 7 Jun 2024 15:21:09 +0200
Subject: [PATCH 2/3] fs: Ignore shift-count-overflow warning in FS plugin
The warning happens in the libext2fs header.
---
src/plugins/fs/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/plugins/fs/Makefile.am b/src/plugins/fs/Makefile.am
index 00474d5a..42e1c777 100644
--- a/src/plugins/fs/Makefile.am
+++ b/src/plugins/fs/Makefile.am
@@ -2,7 +2,7 @@ AUTOMAKE_OPTIONS = subdir-objects
lib_LTLIBRARIES = libbd_fs.la
-libbd_fs_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(BLKID_CFLAGS) $(MOUNT_CFLAGS) $(UUID_CFLAGS) $(EXT2FS_CFLAGS) -Wall -Wextra -Werror -Wno-unused-parameter
+libbd_fs_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(BLKID_CFLAGS) $(MOUNT_CFLAGS) $(UUID_CFLAGS) $(EXT2FS_CFLAGS) -Wall -Wextra -Werror -Wno-unused-parameter -Wno-shift-count-overflow
libbd_fs_la_LIBADD = ${builddir}/../../utils/libbd_utils.la $(GLIB_LIBS) $(GIO_LIBS) $(BLKID_LIBS) $(MOUNT_LIBS) $(UUID_LIBS) $(EXT2FS_LIBS)
libbd_fs_la_LDFLAGS = -L${srcdir}/../../utils/ -version-info 3:0:0 -Wl,--no-undefined -export-symbols-regex '^bd_.*'
libbd_fs_la_CPPFLAGS = -I${builddir}/../../../include/ -I${srcdir}/../
--
2.45.2
From b40ce052f98c9afb3e5bd0f91e7eb8bf6cbacd90 Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Wed, 12 Jun 2024 15:34:02 +0200
Subject: [PATCH 3/3] fs: Fix ignoring errors from libext2fs
Follow-up for #1028, we still want to show the warning and not
completely supress it.
---
src/plugins/fs/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/plugins/fs/Makefile.am b/src/plugins/fs/Makefile.am
index 42e1c777..1ee256f0 100644
--- a/src/plugins/fs/Makefile.am
+++ b/src/plugins/fs/Makefile.am
@@ -2,7 +2,7 @@ AUTOMAKE_OPTIONS = subdir-objects
lib_LTLIBRARIES = libbd_fs.la
-libbd_fs_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(BLKID_CFLAGS) $(MOUNT_CFLAGS) $(UUID_CFLAGS) $(EXT2FS_CFLAGS) -Wall -Wextra -Werror -Wno-unused-parameter -Wno-shift-count-overflow
+libbd_fs_la_CFLAGS = $(GLIB_CFLAGS) $(GIO_CFLAGS) $(BLKID_CFLAGS) $(MOUNT_CFLAGS) $(UUID_CFLAGS) $(EXT2FS_CFLAGS) -Wall -Wextra -Werror -Wno-error=unused-parameter -Wno-error=shift-count-overflow
libbd_fs_la_LIBADD = ${builddir}/../../utils/libbd_utils.la $(GLIB_LIBS) $(GIO_LIBS) $(BLKID_LIBS) $(MOUNT_LIBS) $(UUID_LIBS) $(EXT2FS_LIBS)
libbd_fs_la_LDFLAGS = -L${srcdir}/../../utils/ -version-info 3:0:0 -Wl,--no-undefined -export-symbols-regex '^bd_.*'
libbd_fs_la_CPPFLAGS = -I${builddir}/../../../include/ -I${srcdir}/../
--
2.45.2

10506
0006-smart-plugin.patch
View File

File diff suppressed because it is too large Load Diff

54
0007-nvme-Add-bd_nvme_is_tech_avail-to-the-API-file.patch
View File

@ -1,54 +0,0 @@
From 11c7d2542addb74b62849caa3c8c7057c41fd45f Mon Sep 17 00:00:00 2001
From: Vojtech Trefny <vtrefny@redhat.com>
Date: Thu, 14 Mar 2024 13:53:04 +0100
Subject: [PATCH] nvme: Add bd_nvme_is_tech_avail to the API file
---
src/lib/plugin_apis/nvme.api | 11 +++++++++++
tests/nvme_test.py | 8 ++++++++
2 files changed, 19 insertions(+)
diff --git a/src/lib/plugin_apis/nvme.api b/src/lib/plugin_apis/nvme.api
index 44f8bf73..2f17e0c9 100644
--- a/src/lib/plugin_apis/nvme.api
+++ b/src/lib/plugin_apis/nvme.api
@@ -66,6 +66,17 @@ typedef enum {
BD_NVME_TECH_MODE_INITIATOR = 1 << 2,
} BDNVMETechMode;
+/**
+ * bd_nvme_is_tech_avail:
+ * @tech: the queried tech
+ * @mode: a bit mask of queried modes of operation (#BDNVMETechMode) for @tech
+ * @error: (out) (nullable): place to store error (details about why the @tech-@mode combination is not available)
+ *
+ * Returns: whether the @tech-@mode combination is available -- supported by the
+ * plugin implementation and having all the runtime dependencies available
+ */
+gboolean bd_nvme_is_tech_avail (BDNVMETech tech, G_GNUC_UNUSED guint64 mode, GError **error);
+
/* BpG-skip */
/**
diff --git a/tests/nvme_test.py b/tests/nvme_test.py
index f31de546..df31f112 100644
--- a/tests/nvme_test.py
+++ b/tests/nvme_test.py
@@ -44,6 +44,14 @@ class NVMePluginVersionTestCase(NVMeTest):
def test_plugin_version(self):
self.assertEqual(BlockDev.get_plugin_soname(BlockDev.Plugin.NVME), "libbd_nvme.so.3")
+ @tag_test(TestTags.NOSTORAGE)
+ def test_availability(self):
+ avail = BlockDev.nvme_is_tech_avail(BlockDev.NVMETech.NVME, 0)
+ self.assertTrue(avail)
+
+ avail = BlockDev.nvme_is_tech_avail(BlockDev.NVMETech.FABRICS, 0)
+ self.assertTrue(avail)
+
class NVMeTestCase(NVMeTest):
def setUp(self):
--
2.46.2

21
libblockdev.spec
View File

@ -85,21 +85,12 @@
%define configure_opts %{?python3_copts} %{?lvm_dbus_copts} %{?btrfs_copts} %{?crypto_copts} %{?dm_copts} %{?loop_copts} %{?lvm_copts} %{?lvm_dbus_copts} %{?mdraid_copts} %{?mpath_copts} %{?swap_copts} %{?part_copts} %{?fs_copts} %{?nvdimm_copts} %{?tools_copts} %{?gi_copts} %{?nvme_copts} %{?smart_copts} %{?smartmontools_copts}
Name: libblockdev
Version: 3.1.0
Release: 9%{?dist}
Version: 3.2.0
Release: 1%{?dist}
Summary: A library for low-level manipulation with block devices
License: LGPL-2.1-or-later
URL: https://github.com/storaged-project/libblockdev
Source0: https://github.com/storaged-project/libblockdev/releases/download/%{version}-%{release}/%{name}-%{version}.tar.gz
Source1: smart-tests.tar.gz
Patch0: 0001-misc-RHEL-10-beta-backport.patch
Patch1: 0002-Check-also-for-aliases-in-bd_utils_have_kernel_module.patch
Patch2: 0003-Fix-passing-size-for-pvresize-over-DBus.patch
Patch3: 0004-Upstream-kernel-VDO-support.patch
Patch4: 0005-libext2fs-unused-parameters-fix.patch
Patch5: 0006-smart-plugin.patch
Patch6: 0007-nvme-Add-bd_nvme_is_tech_avail-to-the-API-file.patch
Source0: https://github.com/storaged-project/libblockdev/releases/download/%{version}/%{name}-%{version}.tar.gz
BuildRequires: make
BuildRequires: glib2-devel
@ -641,7 +632,6 @@ A meta-package that pulls all the libblockdev plugins as dependencies.
%prep
%autosetup -n %{name}-%{version} -N -b1
%autosetup -n %{name}-%{version} -p1
%build
@ -956,6 +946,11 @@ find %{buildroot} -type f -name "*.la" | xargs %{__rm}
%files plugins-all
%changelog
* Wed Oct 30 2024 Vojtech Trefny <vtrefny@redhat.com> - 3.2.0-1
- Update to 3.2.0
Resolves: RHEL-60272
Resolves: RHEL-50820
* Tue Oct 29 2024 Troy Dawson <tdawson@redhat.com> - 3.1.0-9
- Bump release for October 2024 mass rebuild:
Resolves: RHEL-64018

3
sources
View File

@ -1,2 +1 @@
SHA512 (libblockdev-3.1.0.tar.gz) = 1e5f8155ceaf1b1f02607d0e2075bfe3b1a2c8d6845de5c90201e6b7aefde3bab7af8e9abf137763e6e9ca42fefe5218697ea63731b3b4952eb004b01cf3719a
SHA512 (smart-tests.tar.gz) = 6ece82eef09c3b22c8f1473653d791bcac7316c3f267368170cea5b152d55603d2a1014c729c4ed190e39660a8734a51989eb53c4a427b843e074f750b1f9a9b
SHA512 (libblockdev-3.2.0.tar.gz) = 02a56f566a768a4f4bc68e3e40c80a080cb5ced58675dfae7986f55aea57f887425ebab19d9c2659887d489f08ace082aca0cf97ed3d95e266fb83a236a2f66c