[v6] block: deal with errp: part I

[PATCH v6 00/14] block: deal with errp: part I

Vladimir Sementsov-Ogievskiy posted 14 patches 3 years, 3 months ago

Diff against v1 v2 v3 v4 v5 v7
Download series mbox

Test checkpatch passed

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

Maintainers: Markus Armbruster <armbru@redhat.com>, Pavel Dovgalyuk <pavel.dovgaluk@ispras.ru>, Eric Blake <eblake@redhat.com>, Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>, Alberto Garcia <berto@igalia.com>, Paolo Bonzini <pbonzini@redhat.com>, Max Reitz <mreitz@redhat.com>, Ari Sundholm <ari@tuxera.com>, John Snow <jsnow@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>, Kevin Wolf <kwolf@redhat.com>

There is a newer version of this series

block/qcow2.h               |  9 ++---
include/block/block.h       | 12 +++----
include/block/blockjob.h    |  2 +-
block.c                     | 70 ++++++++++++++++++++++---------------
block/backup-top.c          | 23 ++++++------
block/blkdebug.c            |  6 ++--
block/blklogwrites.c        | 10 +++---
block/blkreplay.c           |  6 ++--
block/blkverify.c           | 11 +++---
block/commit.c              |  6 ++--
block/mirror.c              | 18 ++++------
block/qcow2-bitmap.c        | 67 ++++++++++++++++++-----------------
block/qcow2.c               | 64 +++++++++++++++------------------
block/qed.c                 | 24 ++++++++-----
block/quorum.c              |  6 ++--
blockdev.c                  | 10 +++---
blockjob.c                  | 18 +++++-----
tests/test-bdrv-graph-mod.c |  6 ++--
18 files changed, 180 insertions(+), 188 deletions(-)

Expand all Fold all

[PATCH v6 00/14] block: deal with errp: part I

Posted by Vladimir Sementsov-Ogievskiy 3 years, 3 months ago

v6:
01: fix uninitialized ret
02,14: add Alberto's r-b

Vladimir Sementsov-Ogievskiy (14):
  block: return status from bdrv_append and friends
  block: use return status of bdrv_append()
  block: check return value of bdrv_open_child and drop error
    propagation
  blockdev: fix drive_backup_prepare() missed error
  block: drop extra error propagation for bdrv_set_backing_hd
  block/mirror: drop extra error propagation in commit_active_start()
  blockjob: return status from block_job_set_speed()
  block/qcow2: qcow2_get_specific_info(): drop error propagation
  block/qcow2-bitmap: improve qcow2_load_dirty_bitmaps() interface
  block/qcow2-bitmap: return status from
    qcow2_store_persistent_dirty_bitmaps
  block/qcow2: read_cache_sizes: return status value
  block/qcow2: simplify qcow2_co_invalidate_cache()
  block/qed: bdrv_qed_do_open: deal with errp
  block/qcow2: refactor qcow2_update_options_prepare error paths

 block/qcow2.h               |  9 ++---
 include/block/block.h       | 12 +++----
 include/block/blockjob.h    |  2 +-
 block.c                     | 70 ++++++++++++++++++++++---------------
 block/backup-top.c          | 23 ++++++------
 block/blkdebug.c            |  6 ++--
 block/blklogwrites.c        | 10 +++---
 block/blkreplay.c           |  6 ++--
 block/blkverify.c           | 11 +++---
 block/commit.c              |  6 ++--
 block/mirror.c              | 18 ++++------
 block/qcow2-bitmap.c        | 67 ++++++++++++++++++-----------------
 block/qcow2.c               | 64 +++++++++++++++------------------
 block/qed.c                 | 24 ++++++++-----
 block/quorum.c              |  6 ++--
 blockdev.c                  | 10 +++---
 blockjob.c                  | 18 +++++-----
 tests/test-bdrv-graph-mod.c |  6 ++--
 18 files changed, 180 insertions(+), 188 deletions(-)

-- 
2.29.2