diff options
author | Hanna Reitz <hreitz@redhat.com> | 2021-10-06 17:19:31 +0200 |
---|---|---|
committer | Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> | 2021-10-07 10:40:48 +0200 |
commit | 1d4a43e9464f8945bd8aa2ed9d95f184b011befe (patch) | |
tree | 2c19c140dfe2d4419349846a5c7e8154678fc5c4 /job.c | |
parent | 447162242803bdc38bb198312e1d0a0675948424 (diff) | |
download | qemu-1d4a43e9464f8945bd8aa2ed9d95f184b011befe.zip qemu-1d4a43e9464f8945bd8aa2ed9d95f184b011befe.tar.gz qemu-1d4a43e9464f8945bd8aa2ed9d95f184b011befe.tar.bz2 |
job: Force-cancel jobs in a failed transaction
When a transaction is aborted, no result matters, and so all jobs within
should be force-cancelled.
Signed-off-by: Hanna Reitz <hreitz@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Message-Id: <20211006151940.214590-5-hreitz@redhat.com>
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Diffstat (limited to 'job.c')
-rw-r--r-- | job.c | 7 |
1 files changed, 6 insertions, 1 deletions
@@ -766,7 +766,12 @@ static void job_completed_txn_abort(Job *job) if (other_job != job) { ctx = other_job->aio_context; aio_context_acquire(ctx); - job_cancel_async(other_job, false); + /* + * This is a transaction: If one job failed, no result will matter. + * Therefore, pass force=true to terminate all other jobs as quickly + * as possible. + */ + job_cancel_async(other_job, true); aio_context_release(ctx); } } |