[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-block] [PATCH v3] block/vdi: Use bdrv_flush after
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [Qemu-block] [PATCH v3] block/vdi: Use bdrv_flush after metadata updates |
Date: |
Thu, 07 May 2015 09:05:16 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 |
On 05/06/2015 10:04 PM, Zhe Qiu wrote:
> From: phoeagon <address@hidden>
>
> In reference to
> b0ad5a455d7e5352d4c86ba945112011dbeadfb8~078a458e077d6b0db262c4b05fee51d01de2d1d2,
> metadata writes to qcow2/cow/qcow/vpc/vmdk are all synced prior to
> succeeding writes.
>
Please wrap commit comments to be under 80 columns (in fact, under 72 is
good, because 'git log' adds spaces when displaying commit bodies).
Your notation of commit~commit is unusual; ranges in git are usually
spelled commit..commit. Also, it's okay to abbreviate commit SHAs to 8
bytes or so. So to shrink your long line, I'd write b0ad5a45..078a458e.
> Only when write is successful that bdrv_flush is called.
>
> Signed-off-by: Zhe Qiu <address@hidden>
Your 'From:' and 'Signed-off-by:' lines have different spellings, which
makes it more confusing when searching for patches by you. You can add
an entry to .mailmap (as a separate patch) to retro-actively consolidate
entries, but it is better to catch things like this up front before we
even have the problem of separate names.
I suspect you want "Zhe Qiu" as the spelling of your name in both lines
(git config can be taught to set up the preferred name to attribute both
for signing patches and for sending email). It is also acceptable to
use UTF-8 to spell your name in native characters, or even a combination
of "native name (ascii counterpart)"
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v3] block/vdi: Use bdrv_flush after metadata updates, Zhe Qiu, 2015/05/07
- Re: [Qemu-devel] [PATCH v3] block/vdi: Use bdrv_flush after metadata updates, Stefan Hajnoczi, 2015/05/07
- Re: [Qemu-devel] [Qemu-block] [PATCH v3] block/vdi: Use bdrv_flush after metadata updates,
Eric Blake <=
- [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Zhe Qiu, 2015/05/07
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Max Reitz, 2015/05/08
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Kevin Wolf, 2015/05/08
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/08
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Stefan Weil, 2015/05/08
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/08
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/08
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Stefan Weil, 2015/05/09
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, phoeagon, 2015/05/09
- Re: [Qemu-devel] [PATCH v4] block/vdi: Use bdrv_flush after metadata updates, Paolo Bonzini, 2015/05/10