[PATCH v2 10/17] CH: use virDomainObjBeginJob()

Ján Tomko jtomko at redhat.com
Mon Sep 5 16:21:16 UTC 2022


On a Monday in 2022, Kristina Hanicova wrote:
>This patch removes virCHDomainObjBeginJob() and replaces it with
>call to the generalized virDomainObjBeginJob().
>
>Signed-off-by: Kristina Hanicova <khanicov at redhat.com>
>---
> src/ch/ch_domain.c | 51 +---------------------------------------------
> src/ch/ch_domain.h |  4 ----
> src/ch/ch_driver.c | 20 +++++++++---------
> 3 files changed, 11 insertions(+), 64 deletions(-)
>
>diff --git a/src/ch/ch_domain.c b/src/ch/ch_domain.c
>index 9ddf9a8584..c592c6ffbb 100644
>--- a/src/ch/ch_domain.c
>+++ b/src/ch/ch_domain.c
>@@ -32,60 +32,11 @@
>
> VIR_LOG_INIT("ch.ch_domain");
>
>-/*
>- * obj must be locked before calling, virCHDriver must NOT be locked
>- *
>- * This must be called by anything that will change the VM state
>- * in any way
>- *
>- * Upon successful return, the object will have its ref count increased.
>- * Successful calls must be followed by EndJob eventually.
>- */
>-int
>-virCHDomainObjBeginJob(virDomainObj *obj, virDomainJob job)
>-{
>-    unsigned long long now;
>-    unsigned long long then;
>-
>-    if (virTimeMillisNow(&now) < 0)
>-        return -1;
>-    then = now + CH_JOB_WAIT_TIME;
>-

This removes the last use of CH_JOB_WAIT_TIME. I can squash in the
removal of it before pushing.

Jano


More information about the libvir-list mailing list