[v1] drop unallocated_blocks_are_zero

[PATCH 0/8] drop unallocated_blocks_are_zero

Vladimir Sementsov-Ogievskiy posted 8 patches 5 years, 6 months ago

Diff against v2 v3
Download series mbox

Test docker-mingw@fedora passed

Test checkpatch passed

Test asan passed

Test docker-quick@centos7 passed

Test FreeBSD passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200506092513.20904-1-vsementsov@virtuozzo.com

Maintainers: Kevin Wolf <kwolf@redhat.com>, Jeff Cody <codyprime@gmail.com>, Fam Zheng <fam@euphon.net>, Stefan Hajnoczi <stefanha@redhat.com>, Ronnie Sahlberg <ronniesahlberg@gmail.com>, Max Reitz <mreitz@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Stefan Weil <sw@weilnetz.de>, Peter Lieven <pl@kamp.de>

There is a newer version of this series

include/block/block.h     |  6 ------
include/block/block_int.h | 13 ++++++++++++-
block.c                   | 15 ---------------
block/crypto.c            |  1 -
block/file-posix.c        |  3 ---
block/io.c                |  8 ++++----
block/iscsi.c             |  1 -
block/qcow2.c             |  1 -
block/qed.c               |  1 -
block/vdi.c               |  3 +--
block/vhdx.c              |  3 ---
block/vpc.c               |  3 +--
qemu-img.c                |  4 +---
13 files changed, 19 insertions(+), 43 deletions(-)

Expand all Fold all

[PATCH 0/8] drop unallocated_blocks_are_zero

Posted by Vladimir Sementsov-Ogievskiy 5 years, 6 months ago

Hi all!

This is first step to block-status refactoring, and solves most simple
problem mentioned in my investigation of block-status described in
the thread "backing chain & block status & filters":
  https://lists.gnu.org/archive/html/qemu-devel/2020-04/msg04706.html


unallocated_blocks_are_zero doesn't simplify all the logic about
block-status, and happily it's not needed, as shown by the following
patches. So, let's get rid of it.

Vladimir Sementsov-Ogievskiy (8):
  block/vdi: return ZERO block-status when appropriate
  block/vpc: return ZERO block-status when appropriate
  block/crypto: drop unallocated_blocks_are_zero
  block/iscsi: drop unallocated_blocks_are_zero
  block/file-posix: drop unallocated_blocks_are_zero
  block/vhdx: drop unallocated_blocks_are_zero
  qemu-img: convert: don't use unallocated_blocks_are_zero
  block: drop unallocated_blocks_are_zero

 include/block/block.h     |  6 ------
 include/block/block_int.h | 13 ++++++++++++-
 block.c                   | 15 ---------------
 block/crypto.c            |  1 -
 block/file-posix.c        |  3 ---
 block/io.c                |  8 ++++----
 block/iscsi.c             |  1 -
 block/qcow2.c             |  1 -
 block/qed.c               |  1 -
 block/vdi.c               |  3 +--
 block/vhdx.c              |  3 ---
 block/vpc.c               |  3 +--
 qemu-img.c                |  4 +---
 13 files changed, 19 insertions(+), 43 deletions(-)

-- 
2.21.0