[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 01/14] block: return status from bdrv_append and friends
From: |
Alberto Garcia |
Subject: |
Re: [PATCH v6 01/14] block: return status from bdrv_append and friends |
Date: |
Wed, 27 Jan 2021 16:54:48 +0100 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Sat 16 Jan 2021 10:51:56 PM CET, Vladimir Sementsov-Ogievskiy wrote:
> The recommended use of qemu error api assumes returning status together
> with setting errp and avoid void functions with errp parameter. Let's
> improve bdrv_append and some friends to reduce error-propagation
> overhead in further patches.
>
> Choose int return status, because bdrv_replace_node_common() has call
> to bdrv_check_update_perm(), which reports int status, which seems
> correct to propagate.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
Berto
- [PATCH v6 00/14] block: deal with errp: part I, Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 01/14] block: return status from bdrv_append and friends, Vladimir Sementsov-Ogievskiy, 2021/01/16
- Re: [PATCH v6 01/14] block: return status from bdrv_append and friends,
Alberto Garcia <=
- [PATCH v6 02/14] block: use return status of bdrv_append(), Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 04/14] blockdev: fix drive_backup_prepare() missed error, Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 03/14] block: check return value of bdrv_open_child and drop error propagation, Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 05/14] block: drop extra error propagation for bdrv_set_backing_hd, Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 06/14] block/mirror: drop extra error propagation in commit_active_start(), Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 08/14] block/qcow2: qcow2_get_specific_info(): drop error propagation, Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 10/14] block/qcow2-bitmap: return status from qcow2_store_persistent_dirty_bitmaps, Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 07/14] blockjob: return status from block_job_set_speed(), Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 09/14] block/qcow2-bitmap: improve qcow2_load_dirty_bitmaps() interface, Vladimir Sementsov-Ogievskiy, 2021/01/16
- [PATCH v6 11/14] block/qcow2: read_cache_sizes: return status value, Vladimir Sementsov-Ogievskiy, 2021/01/16