[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 4/7] qed: add bdrv_invalidate_cache to be cal
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH V2 4/7] qed: add bdrv_invalidate_cache to be called after incoming live migration |
Date: |
Thu, 22 Mar 2012 13:21:17 +0000 |
On Wed, Mar 21, 2012 at 3:52 PM, Benoît Canet <address@hidden> wrote:
> The QED image is reopened to flush metadata and check consistency.
>
> Signed-off-by: Benoit Canet <address@hidden>
> ---
> block/qed.c | 15 +++++++++++++++
> block/qed.h | 1 +
> 2 files changed, 16 insertions(+), 0 deletions(-)
>
> diff --git a/block/qed.c b/block/qed.c
> index a041d31..c47272c 100644
> --- a/block/qed.c
> +++ b/block/qed.c
> @@ -375,6 +375,10 @@ static int bdrv_qed_open(BlockDriverState *bs, int flags)
> int ret;
>
> s->bs = bs;
> +
> + /* backup flags for bdrv_qed_invalidate_cache */
> + s->flags = flags;
It's not clear to me why we need to introduce this field to stash
flags values. bs->open_flags already has this information.
Originally this was introduced in 06d9260ffa9 ("qcow2: implement
bdrv_invalidate_cache (v2)") for qcow2. I wonder if that field is
necessary when we already have bs->open_flags.
What I don't like about s->flags is that it duplicates state *and*
it's done in each block driver that supports .bdrv_invalidate_cache().
So I wonder if we can drop it?
Stefan
- [Qemu-devel] [PATCH V2 0/7] Make QED with live migration safe, Benoît Canet, 2012/03/21
- [Qemu-devel] [PATCH V2 2/7] block: add a function to clear incoming live migration flags, Benoît Canet, 2012/03/21
- [Qemu-devel] [PATCH V2 1/7] block: Add new BDRV_O_INCOMING flag to notice incoming live migration, Benoît Canet, 2012/03/21
- [Qemu-devel] [PATCH V2 7/7] qed: remove incoming live migration blocker, Benoît Canet, 2012/03/21
- [Qemu-devel] [PATCH V2 3/7] blockdev: open images with BDRV_O_INCOMING on incoming live migration, Benoît Canet, 2012/03/21
- [Qemu-devel] [PATCH V2 6/7] qed: honor BDRV_O_INCOMING for incoming live migration, Benoît Canet, 2012/03/21
- [Qemu-devel] [PATCH V2 5/7] migration: clear BDRV_O_INCOMING flags on end of incoming live migration, Benoît Canet, 2012/03/21
- [Qemu-devel] [PATCH V2 4/7] qed: add bdrv_invalidate_cache to be called after incoming live migration, Benoît Canet, 2012/03/21
- Re: [Qemu-devel] [PATCH V2 4/7] qed: add bdrv_invalidate_cache to be called after incoming live migration,
Stefan Hajnoczi <=