[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/4 v2 RESEND] block: add dirty flag status to q
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 3/4 v2 RESEND] block: add dirty flag status to qemu-img |
Date: |
Mon, 12 Mar 2012 18:18:13 +0000 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Mar 07, 2012 at 05:22:58PM +0800, Dong Xu Wang wrote:
> From: Dong Xu Wang <address@hidden>
>
> Some block drivers can verify their image files are clean or not. So we can
> show
> it while using "qemu-img info.
>
> Signed-off-by: Dong Xu Wang <address@hidden>
> ---
> block.c | 14 ++++++++++++++
> block.h | 2 ++
> block_int.h | 1 +
> qemu-img.c | 3 +++
> 4 files changed, 20 insertions(+), 0 deletions(-)
>
> diff --git a/block.c b/block.c
> index 947607b..17e9ba8 100644
> --- a/block.c
> +++ b/block.c
> @@ -193,6 +193,20 @@ static void bdrv_io_limits_intercept(BlockDriverState
> *bs,
> qemu_co_queue_next(&bs->throttled_reqs);
> }
>
> +/* check if the image was cleanly shut down */
> +bool bdrv_not_cleanly_down(BlockDriverState *bs)
The name is a little cryptic to me and I suggest avoiding 'not' in
function names because it easily leads to double-negatives (!not_foo()).
How about:
bool bdrv_was_shutdown_cleanly()
if (!bdrv_was_shutdown_cleanly(bs)) {
printf(...);
}
This patch and the QED patch look fine otherwise.
Stefan
Re: [Qemu-devel] [PATCH 1/4] block: add image fragmentation statistics to qemu-img, Stefan Hajnoczi, 2012/03/12