[Qemu-devel] [PATCH 00/10] block: Op blocker fixes

Kevin Wolf posted 10 patches 7 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1488817322-11397-1-git-send-email-kwolf@redhat.com
Test checkpatch passed
Test docker passed
block.c                   | 182 ++++++++++++++++++++++++++++++----------------
block/commit.c            |   2 +-
block/mirror.c            |  35 +++++----
blockdev.c                |   6 +-
include/block/block.h     |   4 +-
include/block/block_int.h |   6 +-
6 files changed, 152 insertions(+), 83 deletions(-)
[Qemu-devel] [PATCH 00/10] block: Op blocker fixes
Posted by Kevin Wolf 7 years ago
This series fixes a few problems introduced recently with the op blocker
series. It includes mainly fix for cases where qemu would abort()
instead of doing proper error handling previously. These changes also
happen to result in more complete and correct permission checking.

Kevin Wolf (10):
  commit: Fix error handling
  mirror: Fix permission problem with 'replaces'
  mirror: Fix permissions for removing mirror_top_bs
  mirror: Fix error path for dirty bitmap creation
  block: Fix blockdev-snapshot error handling
  block: Factor out should_update_child()
  block: Factor out bdrv_replace_child_noperm()
  block: Ignore multiple children in bdrv_check_update_perm()
  block: Handle permission errors in change_parent_backing_link()
  block: Fix error handling in bdrv_replace_in_backing_chain()

 block.c                   | 182 ++++++++++++++++++++++++++++++----------------
 block/commit.c            |   2 +-
 block/mirror.c            |  35 +++++----
 blockdev.c                |   6 +-
 include/block/block.h     |   4 +-
 include/block/block_int.h |   6 +-
 6 files changed, 152 insertions(+), 83 deletions(-)

-- 
1.8.3.1


Re: [Qemu-devel] [PATCH 00/10] block: Op blocker fixes
Posted by Fam Zheng 7 years ago
On Mon, 03/06 17:21, Kevin Wolf wrote:
> This series fixes a few problems introduced recently with the op blocker
> series. It includes mainly fix for cases where qemu would abort()
> instead of doing proper error handling previously. These changes also
> happen to result in more complete and correct permission checking.
> 
> Kevin Wolf (10):
>   commit: Fix error handling
>   mirror: Fix permission problem with 'replaces'
>   mirror: Fix permissions for removing mirror_top_bs
>   mirror: Fix error path for dirty bitmap creation
>   block: Fix blockdev-snapshot error handling
>   block: Factor out should_update_child()
>   block: Factor out bdrv_replace_child_noperm()
>   block: Ignore multiple children in bdrv_check_update_perm()
>   block: Handle permission errors in change_parent_backing_link()
>   block: Fix error handling in bdrv_replace_in_backing_chain()
> 
>  block.c                   | 182 ++++++++++++++++++++++++++++++----------------
>  block/commit.c            |   2 +-
>  block/mirror.c            |  35 +++++----
>  blockdev.c                |   6 +-
>  include/block/block.h     |   4 +-
>  include/block/block_int.h |   6 +-
>  6 files changed, 152 insertions(+), 83 deletions(-)
> 
> -- 
> 1.8.3.1
> 

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

Re: [Qemu-devel] [PATCH 00/10] block: Op blocker fixes
Posted by Kevin Wolf 7 years ago
Am 06.03.2017 um 17:21 hat Kevin Wolf geschrieben:
> This series fixes a few problems introduced recently with the op blocker
> series. It includes mainly fix for cases where qemu would abort()
> instead of doing proper error handling previously. These changes also
> happen to result in more complete and correct permission checking.

Applied to the block branch.

Kevin