blockjob: Remove the job from the list earlier in block_job_unref()
authorAlberto Garcia <berto@igalia.com>
Tue, 28 Nov 2017 14:53:27 +0000 (16:53 +0200)
committerKevin Wolf <kwolf@redhat.com>
Tue, 28 Nov 2017 15:59:24 +0000 (16:59 +0100)
When destroying a block job in block_job_unref() we should remove it
from the job list before calling block_job_remove_all_bdrv().

This is because removing the BDSs can trigger an aio_poll() and wake
up other jobs that might attempt to use the block job list. If that
happens the job we're currently destroying should not be in that list
anymore.

Signed-off-by: Alberto Garcia <berto@igalia.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
blockjob.c

index ff9a614531cb7a8bf1b8d25caec92bccf4addc87..2f0cc1528b140bab3fdf4d5aed5de01762875673 100644 (file)
@@ -152,6 +152,7 @@ void block_job_unref(BlockJob *job)
 {
     if (--job->refcnt == 0) {
         BlockDriverState *bs = blk_bs(job->blk);
+        QLIST_REMOVE(job, job_list);
         bs->job = NULL;
         block_job_remove_all_bdrv(job);
         blk_remove_aio_context_notifier(job->blk,
@@ -160,7 +161,6 @@ void block_job_unref(BlockJob *job)
         blk_unref(job->blk);
         error_free(job->blocker);
         g_free(job->id);
-        QLIST_REMOVE(job, job_list);
         g_free(job);
     }
 }