[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/1] block: improve error logging in bdrv_reopen_prepare()
From: |
Kevin Wolf |
Subject: |
Re: [PATCH 1/1] block: improve error logging in bdrv_reopen_prepare() |
Date: |
Thu, 16 Feb 2023 11:56:14 +0100 |
Am 13.02.2023 um 11:31 hat Denis V. Lunev geschrieben:
> The error generated when the option could not be changed inside
> bdrv_reopen_prepare() does not give a clue about problematic
> BlockDriverState as we could get very long tree of devices.
>
> The patch adds node name to the error report in the same way as done
> above.
>
> Signed-off-by: Denis V. Lunev <den@openvz.org>
> CC: Kevin Wolf <kwolf@redhat.com>
> CC: Hanna Reitz <hreitz@redhat.com>
> CC: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
Truly fascinating how inconsistent error reporting is in
bdrv_reopen_prepare(). Some places use the node name, some places device
or node name, some places filename and some places nothing. Your choice
is as good as any.
The only problem I can see with this patch is that qemu-iotests 245
needs an update, too.
Kevin