[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 07/15] blockjob: Add "completed" and "ret" in
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH v3 07/15] blockjob: Add "completed" and "ret" in BlockJob |
Date: |
Mon, 13 Jul 2015 19:08:21 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
On 07/09/2015 11:46 PM, Fam Zheng wrote:
> They are set when block_job_completed is called.
>
> Signed-off-by: Fam Zheng <address@hidden>
> ---
> blockjob.c | 3 +++
> include/block/blockjob.h | 9 +++++++++
> 2 files changed, 12 insertions(+)
>
> diff --git a/blockjob.c b/blockjob.c
> index 62bb906..fb1d9e7 100644
> --- a/blockjob.c
> +++ b/blockjob.c
> @@ -89,6 +89,9 @@ void block_job_completed(BlockJob *job, int ret)
> BlockDriverState *bs = job->bs;
>
> assert(bs->job == job);
> + assert(!job->completed);
> + job->completed = true;
> + job->ret = ret;
> job->cb(job->opaque, ret);
> block_job_release(bs);
> }
> diff --git a/include/block/blockjob.h b/include/block/blockjob.h
> index a7b7f66..40d0776 100644
> --- a/include/block/blockjob.h
> +++ b/include/block/blockjob.h
> @@ -134,6 +134,15 @@ struct BlockJob {
>
> /** The opaque value that is passed to the completion function. */
> void *opaque;
> +
> + /* True if this job has reported completion by calling
> block_job_completed.
> + */
The end comment marker looks a little doofy on a newline, no?
> + bool completed;
> +
> + /* ret code passed to block_job_completed.
> + */
> + int ret;
> +
> };
>
> /**
>
Reviewed-by: John Snow <address@hidden>
- Re: [Qemu-devel] [PATCH v3 04/15] block: keep bitmap if incremental backup job is cancelled, (continued)
- [Qemu-devel] [PATCH v3 05/15] backup: Extract dirty bitmap handling as a separate function, Fam Zheng, 2015/07/09
- [Qemu-devel] [PATCH v3 06/15] blockjob: Add .txn_commit and .txn_abort transaction actions, Fam Zheng, 2015/07/09
- [Qemu-devel] [PATCH v3 07/15] blockjob: Add "completed" and "ret" in BlockJob, Fam Zheng, 2015/07/09
- Re: [Qemu-devel] [PATCH v3 07/15] blockjob: Add "completed" and "ret" in BlockJob,
John Snow <=
- [Qemu-devel] [PATCH v3 08/15] blockjob: Simplify block_job_finish_sync, Fam Zheng, 2015/07/09
- [Qemu-devel] [PATCH v3 09/15] blockjob: Move BlockJobDeferToMainLoopData into BlockJob, Fam Zheng, 2015/07/09
- [Qemu-devel] [PATCH v3 10/15] block: add block job transactions, Fam Zheng, 2015/07/09
- Re: [Qemu-devel] [PATCH v3 10/15] block: add block job transactions, Stefan Hajnoczi, 2015/07/14