[PATCH 04/31] qemu: block: Add accessors for protocol/storage node names

Peter Krempa posted 31 patches 2 years, 3 months ago
[PATCH 04/31] qemu: block: Add accessors for protocol/storage node names
Posted by Peter Krempa 2 years, 3 months ago
Introduce a set of accessors, which return node names based on
semantics. This will allow to us to modify how we setup the backing
chain in cases when e.g. the format driver can be omitted, without
breaking all the code.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
---
 src/qemu/qemu_block.c | 48 +++++++++++++++++++++++++++++++++++++++++++
 src/qemu/qemu_block.h | 11 ++++++++++
 2 files changed, 59 insertions(+)

diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c
index 07bc8ede76..0c9460f678 100644
--- a/src/qemu/qemu_block.c
+++ b/src/qemu/qemu_block.c
@@ -52,6 +52,54 @@ qemuBlockNodeNameValidate(const char *nn)
 }


+/**
+ * qemuBlockStorageSourceSetStorageNodename:
+ * @src: virStorageSource to set the storage nodename
+ * @nodename: The node name to set (stolen)
+ *
+ * Sets @nodename as the storage node name of @src. Using NULL @nodename clears
+ * the nodename. @src takes ownership of @nodename.
+ */
+void
+qemuBlockStorageSourceSetStorageNodename(virStorageSource *src,
+                                         char *nodename)
+{
+    g_free(src->nodestorage);
+    src->nodestorage = nodename;
+}
+
+
+/**
+ * qemuBlockStorageSourceGetEffectiveStorageNodename:
+ * @src: virStorageSource to get the effective nodename of
+ *
+ * Gets the nodename that exposes the storage corresponding to @src, without
+ * the format driver applied. This function always returns a name.
+ */
+const char *
+qemuBlockStorageSourceGetEffectiveStorageNodename(virStorageSource *src)
+{
+    if (src->sliceStorage &&
+        src->sliceStorage->nodename)
+        return src->sliceStorage->nodename;
+
+    return src->nodestorage;
+}
+
+
+/**
+ * qemuBlockStorageSourceGetStorageNodename:
+ * @src: virStorageSource to get the effective nodename of
+ *
+ * Gets the nodename corresponding to the real backing storage format layer.
+ */
+const char *
+qemuBlockStorageSourceGetStorageNodename(virStorageSource *src)
+{
+    return src->nodestorage;
+}
+
+
 /**
  * qemuBlockStorageSourceSupportsConcurrentAccess:
  * @src: disk storage source
diff --git a/src/qemu/qemu_block.h b/src/qemu/qemu_block.h
index cf5eaf87f3..ecc5711dcd 100644
--- a/src/qemu/qemu_block.h
+++ b/src/qemu/qemu_block.h
@@ -26,6 +26,17 @@
 #include "virjson.h"
 #include "viruri.h"

+void
+qemuBlockStorageSourceSetStorageNodename(virStorageSource *src,
+                                         char *nodename);
+
+const char *
+qemuBlockStorageSourceGetEffectiveStorageNodename(virStorageSource *src);
+
+const char *
+qemuBlockStorageSourceGetStorageNodename(virStorageSource *src);
+
+
 typedef struct qemuBlockNodeNameBackingChainData qemuBlockNodeNameBackingChainData;
 struct qemuBlockNodeNameBackingChainData {
     char *qemufilename; /* name of the image from qemu */
-- 
2.41.0
Re: [PATCH 04/31] qemu: block: Add accessors for protocol/storage node names
Posted by Ján Tomko 2 years, 3 months ago
On a Monday in 2023, Peter Krempa wrote:
>Introduce a set of accessors, which return node names based on
>semantics. This will allow to us to modify how we setup the backing
>chain in cases when e.g. the format driver can be omitted, without
>breaking all the code.
>
>Signed-off-by: Peter Krempa <pkrempa@redhat.com>
>---
> src/qemu/qemu_block.c | 48 +++++++++++++++++++++++++++++++++++++++++++
> src/qemu/qemu_block.h | 11 ++++++++++
> 2 files changed, 59 insertions(+)
>
>diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c
>index 07bc8ede76..0c9460f678 100644
>--- a/src/qemu/qemu_block.c
>+++ b/src/qemu/qemu_block.c
>@@ -52,6 +52,54 @@ qemuBlockNodeNameValidate(const char *nn)
> }
>
>
>+/**
>+ * qemuBlockStorageSourceSetStorageNodename:
>+ * @src: virStorageSource to set the storage nodename
>+ * @nodename: The node name to set (stolen)

But that's stealing!

>+ *
>+ * Sets @nodename as the storage node name of @src. Using NULL @nodename clears
>+ * the nodename. @src takes ownership of @nodename.
>+ */

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano