[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH for-6.1? 4/6] job: Add job_cancel_requested()

From: Vladimir Sementsov-Ogievskiy
Subject: Re: [PATCH for-6.1? 4/6] job: Add job_cancel_requested()
Date: Tue, 27 Jul 2021 17:47:12 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.12.0

26.07.2021 10:09, Max Reitz wrote:

          job->ret = -ECANCELED;
      if (job->ret) {
@@ -704,7 +709,7 @@ static int job_finalize_single(Job *job)
        /* Emit events only if we actually started */
      if (job_started(job)) {
-        if (job_is_cancelled(job)) {
+        if (job_cancel_requested(job)) {

Same question here.. Shouldn't mirror report COMPLETED event in case of 
not-force cancelled in READY state?

Same here, I thought this is user-visible, nothing internal, so I should leave 
it as-is.

Now I see that cancelling mirror post-READY indeed should result in a COMPLETED 
event.  So I’m actually not exactly sure how mirror does that, despite this 
code here

Hmm. Now looking at mirror code, I see that it does "s->common.job.cancelled = 

Best regards,

reply via email to

[Prev in Thread] Current Thread [Next in Thread]