diff options
| author | Vladimir Sementsov-Ogievskiy | 2021-06-01 09:52:14 +0200 |
|---|---|---|
| committer | Kevin Wolf | 2021-06-02 14:23:20 +0200 |
| commit | fd240a184b0e8a9889097216d182def6aece30cb (patch) | |
| tree | ce37631224a6c6d4c417cc7640bd370252ab2336 /block | |
| parent | block: document child argument of bdrv_attach_child_common() (diff) | |
| download | qemu-fd240a184b0e8a9889097216d182def6aece30cb.tar.gz qemu-fd240a184b0e8a9889097216d182def6aece30cb.tar.xz qemu-fd240a184b0e8a9889097216d182def6aece30cb.zip | |
block-backend: improve blk_root_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.
While being here also use g_autofree.
iotest 307 output is updated.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
Message-Id: <20210601075218.79249-3-vsementsov@virtuozzo.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block')
| -rw-r--r-- | block/block-backend.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/block/block-backend.c b/block/block-backend.c index d1a33a2c8e..5be32c0c42 100644 --- a/block/block-backend.c +++ b/block/block-backend.c @@ -141,19 +141,18 @@ static void blk_root_set_aio_ctx(BdrvChild *child, AioContext *ctx, static char *blk_root_get_parent_desc(BdrvChild *child) { BlockBackend *blk = child->opaque; - char *dev_id; + g_autofree char *dev_id = NULL; if (blk->name) { - return g_strdup(blk->name); + return g_strdup_printf("block device '%s'", blk->name); } dev_id = blk_get_attached_dev_id(blk); if (*dev_id) { - return dev_id; + return g_strdup_printf("block device '%s'", dev_id); } else { /* TODO Callback into the BB owner for something more detailed */ - g_free(dev_id); - return g_strdup("a block device"); + return g_strdup("an unnamed block device"); } } |