diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2017-05-08 16:13:10 +0200 |
|---|---|---|
| committer | Jeff Cody <jcody@redhat.com> | 2017-05-24 16:38:51 -0400 |
| commit | eb05e011e248c6fb6baee295e14fd206e136028c (patch) | |
| tree | ab38894e8cee57f1e1d2ca690b74bc00800673cb /hw/usb/dev-serial.c | |
| parent | 4fb588e95bacb770746cc794ef165fd71a4d27ea (diff) | |
| download | focaccia-qemu-eb05e011e248c6fb6baee295e14fd206e136028c.tar.gz focaccia-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 'hw/usb/dev-serial.c')
0 files changed, 0 insertions, 0 deletions