[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 06/15] block: Fix crash on block_resize on inactive node
From: |
Eric Blake |
Subject: |
Re: [PATCH v2 06/15] block: Fix crash on block_resize on inactive node |
Date: |
Thu, 30 Jan 2025 14:11:35 -0600 |
User-agent: |
NeoMutt/20250113 |
On Thu, Jan 30, 2025 at 06:12:37PM +0100, Kevin Wolf wrote:
> In order for block_resize to fail gracefully on an inactive node instead
> of crashing with an assertion failure in bdrv_co_write_req_prepare()
> (called from bdrv_co_truncate()), we need to check for inactive nodes
> also when they are attached as a root node and make sure that
> BLK_PERM_RESIZE isn't among the permissions allowed for inactive nodes.
> To this effect, don't enumerate the permissions that are incompatible
> with inactive nodes any more, but allow only BLK_PERM_CONSISTENT_READ
> for them.
>
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
> block.c | 7 +++++++
> block/block-backend.c | 2 +-
> 2 files changed, 8 insertions(+), 1 deletion(-)
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc.
Virtualization: qemu.org | libguestfs.org
- [PATCH v2 00/15] block: Managing inactive nodes (QSD migration), Kevin Wolf, 2025/01/30
- [PATCH v2 01/15] block: Add 'active' field to BlockDeviceInfo, Kevin Wolf, 2025/01/30
- [PATCH v2 02/15] block: Inactivate external snapshot overlays when necessary, Kevin Wolf, 2025/01/30
- [PATCH v2 03/15] migration/block-active: Remove global active flag, Kevin Wolf, 2025/01/30
- [PATCH v2 04/15] block: Don't attach inactive child to active node, Kevin Wolf, 2025/01/30
- [PATCH v2 06/15] block: Fix crash on block_resize on inactive node, Kevin Wolf, 2025/01/30
- Re: [PATCH v2 06/15] block: Fix crash on block_resize on inactive node,
Eric Blake <=
- [PATCH v2 05/15] block: Allow inactivating already inactive nodes, Kevin Wolf, 2025/01/30
- [PATCH v2 07/15] block: Add option to create inactive nodes, Kevin Wolf, 2025/01/30
- [PATCH v2 08/15] block: Add blockdev-set-active QMP command, Kevin Wolf, 2025/01/30
- [PATCH v2 09/15] block: Support inactive nodes in blk_insert_bs(), Kevin Wolf, 2025/01/30