diff options
| author | Vladimir Sementsov-Ogievskiy | 2021-06-01 09:52:15 +0200 |
|---|---|---|
| committer | Kevin Wolf | 2021-06-02 14:23:20 +0200 |
| commit | 2c0a3acb9570a9e1ffae3c73ef94bc826dc9dd1d (patch) | |
| tree | 0e6b71b1725ffa73172770876c01886849c32aa4 /block.c | |
| parent | block-backend: improve blk_root_get_parent_desc() (diff) | |
| download | qemu-2c0a3acb9570a9e1ffae3c73ef94bc826dc9dd1d.tar.gz qemu-2c0a3acb9570a9e1ffae3c73ef94bc826dc9dd1d.tar.xz qemu-2c0a3acb9570a9e1ffae3c73ef94bc826dc9dd1d.zip | |
block: improve bdrv_child_get_parent_desc()
We have different types of parents: block nodes, block backends and
jobs. So, it makes sense to specify type together with name.
Next, this handler us used to compose an error message about permission
conflict. And permission conflict occurs in a specific place of block
graph. We shouldn't report name of parent device (as it refers another
place in block graph), but exactly and only the name of the node. So,
use bdrv_get_node_name() directly.
iotest 283 output is updated.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
Message-Id: <20210601075218.79249-4-vsementsov@virtuozzo.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block.c')
| -rw-r--r-- | block.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -1149,7 +1149,7 @@ int bdrv_parse_cache_mode(const char *mode, int *flags, bool *writethrough) static char *bdrv_child_get_parent_desc(BdrvChild *c) { BlockDriverState *parent = c->opaque; - return g_strdup(bdrv_get_device_or_node_name(parent)); + return g_strdup_printf("node '%s'", bdrv_get_node_name(parent)); } static void bdrv_child_cb_drained_begin(BdrvChild *child) |