[Qemu-devel] [PATCH for-2.9 0/2] commit: Fix commit_top_bs problems

Kevin Wolf posted 2 patches 6 years, 11 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1491498851-16709-1-git-send-email-kwolf@redhat.com
Test checkpatch passed
Test docker passed
Test s390x passed
block/commit.c | 3 +++
1 file changed, 3 insertions(+)
[Qemu-devel] [PATCH for-2.9 0/2] commit: Fix commit_top_bs problems
Posted by Kevin Wolf 6 years, 11 months ago
The same problem that Fam's patch "mirror: Fix aio context of
mirror_top_bs" fixes for the mirror block job also exists for the
commit block job and the 'commit' HMP command. While comparing the
respective places in mirror.c and commit.c, I also noticed that
commit neglects to set an image size.

This series fixes both problems.

Kevin Wolf (2):
  commit: Set commit_top_bs->aio_context
  commit: Set commit_top_bs->total_sectors

 block/commit.c | 3 +++
 1 file changed, 3 insertions(+)

-- 
1.8.3.1


Re: [Qemu-devel] [PATCH for-2.9 0/2] commit: Fix commit_top_bs problems
Posted by Fam Zheng 6 years, 11 months ago
On Thu, 04/06 19:14, Kevin Wolf wrote:
> The same problem that Fam's patch "mirror: Fix aio context of
> mirror_top_bs" fixes for the mirror block job also exists for the
> commit block job and the 'commit' HMP command. While comparing the
> respective places in mirror.c and commit.c, I also noticed that
> commit neglects to set an image size.
> 
> This series fixes both problems.
> 
> Kevin Wolf (2):
>   commit: Set commit_top_bs->aio_context
>   commit: Set commit_top_bs->total_sectors
> 
>  block/commit.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> -- 
> 1.8.3.1
> 

Reviewed-by: Fam Zheng <famz@redhat.com>