[v1] Basic implementation of persistent reservations

[libvirt] [PATCH 12/14] qemu_process.c: Introduce qemuProcessSetupPRDaemon

Posted by Michal Privoznik 8 years ago

Again, for hotplug we need to be able to spawn just one process.
Not all of them. Expose the static function we already have for
that.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
---
 src/qemu/qemu_process.c | 9 +++++++++
 src/qemu/qemu_process.h | 4 ++++
 2 files changed, 13 insertions(+)

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index 02608c1f3..b4c4c64fa 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -2635,6 +2635,15 @@ qemuProcessSetupOnePRDaemon(void *payload,
 }
 
 
+int
+qemuProcessSetupPRDaemon(virDomainObjPtr vm,
+                         qemuDomainDiskPRObjectPtr prObj,
+                         const char *prAlias)
+{
+    return qemuProcessSetupOnePRDaemon(prObj, prAlias, vm);
+}
+
+
 static int
 qemuProcessSetupPRDaemons(virDomainObjPtr vm)
 {
diff --git a/src/qemu/qemu_process.h b/src/qemu/qemu_process.h
index cd9a72031..50946e16d 100644
--- a/src/qemu/qemu_process.h
+++ b/src/qemu/qemu_process.h
@@ -199,4 +199,8 @@ int qemuProcessRefreshDisks(virQEMUDriverPtr driver,
                             virDomainObjPtr vm,
                             qemuDomainAsyncJob asyncJob);
 
+int qemuProcessSetupPRDaemon(virDomainObjPtr vm,
+                             qemuDomainDiskPRObjectPtr prObj,
+                             const char *prAlias);
+
 #endif /* __QEMU_PROCESS_H__ */
-- 
2.13.6

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 12/14] qemu_process.c: Introduce qemuProcessSetupPRDaemon

Posted by Peter Krempa 7 years, 12 months ago

On Thu, Jan 18, 2018 at 17:04:44 +0100, Michal Privoznik wrote:
> Again, for hotplug we need to be able to spawn just one process.
> Not all of them. Expose the static function we already have for
> that.
> 
> Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
> ---
>  src/qemu/qemu_process.c | 9 +++++++++
>  src/qemu/qemu_process.h | 4 ++++
>  2 files changed, 13 insertions(+)
> 
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index 02608c1f3..b4c4c64fa 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -2635,6 +2635,15 @@ qemuProcessSetupOnePRDaemon(void *payload,
>  }
>  
>  
> +int
> +qemuProcessSetupPRDaemon(virDomainObjPtr vm,
> +                         qemuDomainDiskPRObjectPtr prObj,
> +                         const char *prAlias)
> +{
> +    return qemuProcessSetupOnePRDaemon(prObj, prAlias, vm);

Do we really need a wrapper that only modifies argument order?
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list