Re: [Qemu-devel] [PATCH] blockjob: add block_job_release function

2015-07-07 Thread Stefan Hajnoczi
On Fri, Jun 26, 2015 at 05:37:35PM +0800, Ting Wang wrote: > There is job resource leak in function mirror_start_job, > although bdrv_create_dirty_bitmap is unlikely failed. > Add block_job_release for each release when needed. > > Signed-off-by: Ting Wang > --- > block/mirror.c | 2

Re: [Qemu-devel] [PATCH] blockjob: add block_job_release function

2015-07-05 Thread Ting Wang
Ping? Regards, Ting On 2015-6-26 17:37, Ting Wang wrote: > There is job resource leak in function mirror_start_job, > although bdrv_create_dirty_bitmap is unlikely failed. > Add block_job_release for each release when needed. > > Signed-off-by: Ting Wang > --- > block/mirror.c | 2 +

Re: [Qemu-devel] [PATCH] blockjob: add block_job_release function

2015-06-26 Thread John Snow
On 06/26/2015 05:37 AM, Ting Wang wrote: > There is job resource leak in function mirror_start_job, > although bdrv_create_dirty_bitmap is unlikely failed. > Add block_job_release for each release when needed. > > Signed-off-by: Ting Wang > --- > block/mirror.c | 2 ++ > blockjob.c

[Qemu-devel] [PATCH] blockjob: add block_job_release function

2015-06-26 Thread Ting Wang
There is job resource leak in function mirror_start_job, although bdrv_create_dirty_bitmap is unlikely failed. Add block_job_release for each release when needed. Signed-off-by: Ting Wang --- block/mirror.c | 2 ++ blockjob.c | 20 include/block/blo