<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Sep 5, 2022 at 6:21 PM Ján Tomko <<a href="mailto:jtomko@redhat.com">jtomko@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On a Monday in 2022, Kristina Hanicova wrote:<br>
>This patch removes virCHDomainObjBeginJob() and replaces it with<br>
>call to the generalized virDomainObjBeginJob().<br>
><br>
>Signed-off-by: Kristina Hanicova <<a href="mailto:khanicov@redhat.com" target="_blank">khanicov@redhat.com</a>><br>
>---<br>
> src/ch/ch_domain.c | 51 +---------------------------------------------<br>
> src/ch/ch_domain.h |  4 ----<br>
> src/ch/ch_driver.c | 20 +++++++++---------<br>
> 3 files changed, 11 insertions(+), 64 deletions(-)<br>
><br>
>diff --git a/src/ch/ch_domain.c b/src/ch/ch_domain.c<br>
>index 9ddf9a8584..c592c6ffbb 100644<br>
>--- a/src/ch/ch_domain.c<br>
>+++ b/src/ch/ch_domain.c<br>
>@@ -32,60 +32,11 @@<br>
><br>
> VIR_LOG_INIT("ch.ch_domain");<br>
><br>
>-/*<br>
>- * obj must be locked before calling, virCHDriver must NOT be locked<br>
>- *<br>
>- * This must be called by anything that will change the VM state<br>
>- * in any way<br>
>- *<br>
>- * Upon successful return, the object will have its ref count increased.<br>
>- * Successful calls must be followed by EndJob eventually.<br>
>- */<br>
>-int<br>
>-virCHDomainObjBeginJob(virDomainObj *obj, virDomainJob job)<br>
>-{<br>
>-    unsigned long long now;<br>
>-    unsigned long long then;<br>
>-<br>
>-    if (virTimeMillisNow(&now) < 0)<br>
>-        return -1;<br>
>-    then = now + CH_JOB_WAIT_TIME;<br>
>-<br>
<br>
This removes the last use of CH_JOB_WAIT_TIME. I can squash in the<br>
removal of it before pushing.<br></blockquote><div><br></div><div><br></div><div>Thank you for noticing, that would be great.</div><div><br></div><div>Kristina <br></div></div></div>