diff options
| author | Paolo Bonzini | 2017-05-08 16:13:10 +0200 |
|---|---|---|
| committer | Jeff Cody | 2017-05-24 22:38:51 +0200 |
| commit | eb05e011e248c6fb6baee295e14fd206e136028c (patch) | |
| tree | ab38894e8cee57f1e1d2ca690b74bc00800673cb /include/exec | |
| parent | blockjob: reorganize block_job_completed_txn_abort (diff) | |
| download | qemu-eb05e011e248c6fb6baee295e14fd206e136028c.tar.gz qemu-eb05e011e248c6fb6baee295e14fd206e136028c.tar.xz qemu-eb05e011e248c6fb6baee295e14fd206e136028c.zip | |
blockjob: use deferred_to_main_loop to indicate the coroutine has ended
All block jobs are using block_job_defer_to_main_loop as the final
step just before the coroutine terminates. At this point,
block_job_enter should do nothing, but currently it restarts
the freed coroutine.
Now, the job->co states should probably be changed to an enum
(e.g. BEFORE_START, STARTED, YIELDED, COMPLETED) subsuming
block_job_started, job->deferred_to_main_loop and job->busy.
For now, this patch eliminates the problematic reenter by
removing the reset of job->deferred_to_main_loop (which served
no purpose, as far as I could see) and checking the flag in
block_job_enter.
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 20170508141310.8674-12-pbonzini@redhat.com
Signed-off-by: Jeff Cody <jcody@redhat.com>
Diffstat (limited to 'include/exec')
0 files changed, 0 insertions, 0 deletions