Re: [Qemu-block] [PATCH 1/2] ignore bdrv_flush operation when no qcow2 cache item is dirty

[Eric Blake]

On 07/20/2015 07:51 PM, Qingshu Chen wrote:
> I've made a mistake on the series. Following is the new patch:

When re-sending a patch, it's better to start a new thread and include a
'v2' in the subject line.

> 
>>From ef1079b422eef40a802ca13e249795005efa441d Mon Sep 17 00:00:00 2001
> From: Qingshu Chen <address@hidden>
> Date: Tue, 21 Jul 2015 09:46:08 +0800
> Subject: [PATCH] ignore bdrv_flush operation when no qcow2 cache item is
> dirty
> Signed-off-by: Qingshu Chen <address@hidden>
> 
> ---
>  block/qcow2-cache.c | 16 ++++++++++++----
>  block/qcow2.c       |  2 ++
>  2 files changed, 14 insertions(+), 4 deletions(-)

>      if (qcow2_need_accurate_refcounts(s)) {
> --
> 1.9.1
> 
> 
> 2015-07-20 23:03 GMT+08:00 Eric Blake <address@hidden>:
>>
>> [patches should always be sent to qemu-devel, even if qemu-block is also
>> in the to/cc list]
>>
>> On 07/08/2015 01:26 AM, Qingshu Chen wrote:
>>> qcow2_cache_flush() writes dirty cache to the disk and invokes
> bdrv_flush()

Replying to an existing mail, and worse, keeping the old patch in the
reply, makes it harder for maintainers to extract your patch.

For more hints on patch submissions, see
http://wiki.qemu.org/Contribute/SubmitAPatch

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature