Re: [Qemu-devel] [PATCH 13/42] job: Move state transitions to Job

2018-05-16 Thread Eric Blake
On 05/09/2018 11:26 AM, Kevin Wolf wrote: This moves BlockJob.status and the closely related functions (block_)job_state_transition() and (block_)job_apply_verb to Job. The two QAPI enums are renamed to JobStatus and JobVerb. Signed-off-by: Kevin Wolf --- qapi/block-core.json | 14 +++---

Re: [Qemu-devel] [PATCH 13/42] job: Move state transitions to Job

2018-05-14 Thread John Snow
On 05/09/2018 12:26 PM, Kevin Wolf wrote: > This moves BlockJob.status and the closely related functions > (block_)job_state_transition() and (block_)job_apply_verb to Job. The > two QAPI enums are renamed to JobStatus and JobVerb. > > Signed-off-by: Kevin Wolf In good faith that the TODOs dis

Re: [Qemu-devel] [PATCH 13/42] job: Move state transitions to Job

2018-05-14 Thread Max Reitz
On 2018-05-09 18:26, Kevin Wolf wrote: > This moves BlockJob.status and the closely related functions > (block_)job_state_transition() and (block_)job_apply_verb to Job. The > two QAPI enums are renamed to JobStatus and JobVerb. > > Signed-off-by: Kevin Wolf > --- > qapi/block-core.json | 1

[Qemu-devel] [PATCH 13/42] job: Move state transitions to Job

2018-05-09 Thread Kevin Wolf
This moves BlockJob.status and the closely related functions (block_)job_state_transition() and (block_)job_apply_verb to Job. The two QAPI enums are renamed to JobStatus and JobVerb. Signed-off-by: Kevin Wolf --- qapi/block-core.json | 14 +++ include/block/blockjob.h | 3 -- include