[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata upd
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates |
Date: |
Fri, 8 May 2015 15:55:12 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 08.05.2015 um 15:14 hat Max Reitz geschrieben:
> On 07.05.2015 17:16, Zhe Qiu wrote:
> >In reference to b0ad5a45...078a458e, metadata writes to
> >qcow2/cow/qcow/vpc/vmdk are all synced prior to succeeding writes.
> >
> >Only when write is successful that bdrv_flush is called.
> >
> >Signed-off-by: Zhe Qiu <address@hidden>
> >---
> > block/vdi.c | 3 +++
> > 1 file changed, 3 insertions(+)
>
> I missed Kevin's arguments before, but I think that adding this is
> more correct than not having it; and when thinking about speed, this
> is vdi, a format supported for compatibility.
If you use it only as a convert target, you probably care more about
speed than about leaks in case of a host crash.
> So if we wanted to optimize it, we'd probably have to cache multiple
> allocations, do them at once and then flush afterwards (like the
> metadata cache we have in qcow2?)
That would defeat the purpose of this patch which aims at having
metadata and data written out almost at the same time. On the other
hand, fully avoiding the problem instead of just making the window
smaller would require a journal, which VDI just doesn't have.
I'm not convinced of this patch, but I'll defer to Stefan Weil as the
VDI maintainer.
Kevin
- [Qemu-block] [PATCH v3] block/vdi: Use bdrv_flush after metadata updates, Zhe Qiu, 2015/05/07
- Re: [Qemu-block] [Qemu-devel] [PATCH v3] block/vdi: Use bdrv_flush after metadata updates, Stefan Hajnoczi, 2015/05/07
- Re: [Qemu-block] [PATCH v3] block/vdi: Use bdrv_flush after metadata updates, Eric Blake, 2015/05/07
- [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Zhe Qiu, 2015/05/07
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Max Reitz, 2015/05/08
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates,
Kevin Wolf <=
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/08
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Stefan Weil, 2015/05/08
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/08
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/08
- Re: [Qemu-block] [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Stefan Weil, 2015/05/09
- Re: [Qemu-block] [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/09
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Paolo Bonzini, 2015/05/10
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Stefan Weil, 2015/05/10
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/10
- Re: [Qemu-block] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Paolo Bonzini, 2015/05/10