[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/47] block: add block_job_query
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 04/47] block: add block_job_query |
Date: |
Mon, 30 Jul 2012 16:47:47 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120605 Thunderbird/13.0 |
Am 24.07.2012 13:03, schrieb Paolo Bonzini:
> Extract it out of the implementation of query-block-jobs.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> blockdev.c | 15 ++-------------
> blockjob.c | 11 +++++++++++
> blockjob.h | 8 ++++++++
> 3 files changed, 21 insertions(+), 13 deletions(-)
>
> diff --git a/blockdev.c b/blockdev.c
> index e066f8f..dc099f9 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -1166,19 +1166,8 @@ static void do_qmp_query_block_jobs_one(void *opaque,
> BlockDriverState *bs)
> BlockJob *job = bs->job;
>
> if (job) {
> - BlockJobInfoList *elem;
> - BlockJobInfo *info = g_new(BlockJobInfo, 1);
> - *info = (BlockJobInfo){
> - .type = g_strdup(job->job_type->job_type),
> - .device = g_strdup(bdrv_get_device_name(bs)),
> - .len = job->len,
> - .offset = job->offset,
> - .speed = job->speed,
> - };
> -
> - elem = g_new0(BlockJobInfoList, 1);
> - elem->value = info;
> -
> + BlockJobInfoList *elem = g_new0(BlockJobInfoList, 1);
> + elem->value = block_job_query(bs->job);
> (*prev)->next = elem;
> *prev = elem;
> }
> diff --git a/blockjob.c b/blockjob.c
> index 9737a43..a947a6e 100644
> --- a/blockjob.c
> +++ b/blockjob.c
> @@ -161,3 +161,14 @@ void block_job_sleep_ns(BlockJob *job, QEMUClock *clock,
> int64_t ns)
> job->busy = true;
> }
> }
> +
> +BlockJobInfo *block_job_query(BlockJob *job)
> +{
> + BlockJobInfo *info = g_new(BlockJobInfo, 1);
> + info->type = g_strdup(job->job_type->job_type);
> + info->device = g_strdup(bdrv_get_device_name(job->bs));
> + info->len = job->len;
> + info->offset = job->offset;
> + info->speed = job->speed;
> + return info;
> +}
Why did you convert the initialisation to separate statement? If you
really want to do this, I think using g_new0 would be safer now, but I
actually like compound literals better.
Kevin
[Qemu-devel] [PATCH 05/47] block: add support for job pause/resume, Paolo Bonzini, 2012/07/24
[Qemu-devel] [PATCH 04/47] block: add block_job_query, Paolo Bonzini, 2012/07/24
- Re: [Qemu-devel] [PATCH 04/47] block: add block_job_query,
Kevin Wolf <=
[Qemu-devel] [PATCH 06/47] qmp: add block-job-pause and block-job-resume, Paolo Bonzini, 2012/07/24
[Qemu-devel] [PATCH 03/47] block: move job APIs to separate files, Paolo Bonzini, 2012/07/24
[Qemu-devel] [PATCH 07/47] qemu-iotests: add test for pausing a streaming operation, Paolo Bonzini, 2012/07/24
[Qemu-devel] [PATCH 08/47] block: rename block_job_complete to block_job_completed, Paolo Bonzini, 2012/07/24
[Qemu-devel] [PATCH 10/47] block: move BlockdevOnError declaration to QAPI, Paolo Bonzini, 2012/07/24
[Qemu-devel] [PATCH 09/47] block: rename BlockErrorAction, BlockQMPEventAction, Paolo Bonzini, 2012/07/24