Having an Error parameter for these functions makes sense because we sometimes want a bit more information than just "Something failed". Some drivers already use error_report() and the like to emit this additional information, so it's rather obvious that we do want a real error object here.
Max Reitz (3): block: Add errp to b{lk,drv}_truncate() block: Add errp to BD.bdrv_truncate() block: Add some bdrv_truncate() error messages include/block/block.h | 2 +- include/block/block_int.h | 2 +- include/sysemu/block-backend.h | 2 +- block.c | 18 +++++++++++++----- block/archipelago.c | 3 ++- block/blkdebug.c | 4 ++-- block/block-backend.c | 5 +++-- block/commit.c | 5 +++-- block/crypto.c | 5 +++-- block/file-posix.c | 10 ++++++++-- block/file-win32.c | 6 +++--- block/gluster.c | 3 ++- block/iscsi.c | 4 ++-- block/mirror.c | 2 +- block/nfs.c | 2 +- block/parallels.c | 13 ++++++++----- block/qcow.c | 6 +++--- block/qcow2-refcount.c | 5 ++++- block/qcow2.c | 23 ++++++++++++++--------- block/qed.c | 8 +++++--- block/raw-format.c | 6 ++++-- block/rbd.c | 2 +- block/sheepdog.c | 14 ++++++-------- block/vdi.c | 4 ++-- block/vhdx-log.c | 2 +- block/vhdx.c | 6 +++--- block/vmdk.c | 13 +++---------- block/vpc.c | 2 +- blockdev.c | 21 +-------------------- qemu-img.c | 17 ++++------------- qemu-io-cmds.c | 2 +- 31 files changed, 107 insertions(+), 110 deletions(-) -- 2.12.0