All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juan Quintela <quintela@redhat.com>
To: "Gonglei (Arei)" <arei.gonglei@huawei.com>
Cc: "chenliang (T)" <chenliang88@huawei.com>,
	"Huangweidong (C)" <weidong.huang@huawei.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	"owasserm@redhat.com" <owasserm@redhat.com>,
	"pbonzini@redhat.com" <pbonzini@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 02/10] migration: Add counters of updating the dirty bitmap
Date: Tue, 11 Mar 2014 21:28:18 +0100	[thread overview]
Message-ID: <87siqo32fx.fsf@elfo.mitica> (raw)
In-Reply-To: <33183CC9F5247A488A2544077AF19020815D6567@SZXEMA503-MBS.china.huawei.com> (Gonglei's message of "Tue, 11 Mar 2014 13:34:45 +0000")

"Gonglei (Arei)" <arei.gonglei@huawei.com> wrote:
>> On 03/11/2014 06:53 AM, arei.gonglei@huawei.com wrote:
>> > From: ChenLiang <chenliang88@huawei.com>
>> >
>> > Add counters to log the times of updating the dirty bitmap.
>> >
>> > Signed-off-by: ChenLiang <chenliang88@huawei.com>
>> > Signed-off-by: Gonglei <arei.gonglei@huawei.com>
>> > Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
>> > Reviewed-by: Eric Blake <eblake@redhat.com>
>> 
>> Wait - how did my Reviewed-by get here?  There is no [PATCHv2] in the
>> subject line to indicate that I reviewed an earlier revision. and the
>> only other mail I see from me in the archives with the same subject line
>> is here:
>> https://lists.gnu.org/archive/html/qemu-devel/2014-02/msg05143.html
>> 
>> where I made a comment, but did NOT give my signature on the patch.
>> 
>> Please follow http://wiki.qemu.org/Contribute/SubmitAPatch when using
>> Reviewed-by: tags and subject lines.  Otherwise, you are making life
>> harder for yourself and for reviewers.
>> 
> I'm so sorry for those, and will post a new version.

>> > +static uint64_t bitmap_sync_cnt;

If you change, you can do s/_cnt/_count/?  As suggested in the output
string?

Thanks, Juan.

Later, Juan.

  reply	other threads:[~2014-03-11 20:28 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-11 12:53 [Qemu-devel] [PATCH 00/10] migration: Optimization the xbzrle and fix two corruption issues arei.gonglei
2014-03-11 12:53 ` [Qemu-devel] [PATCH 01/10] XBZRLE: Fix one XBZRLE " arei.gonglei
2014-03-11 20:26   ` Juan Quintela
2014-03-11 12:53 ` [Qemu-devel] [PATCH 02/10] migration: Add counters of updating the dirty bitmap arei.gonglei
2014-03-11 13:09   ` Eric Blake
2014-03-11 13:34     ` Gonglei (Arei)
2014-03-11 20:28       ` Juan Quintela [this message]
2014-03-11 12:53 ` [Qemu-devel] [PATCH 03/10] XBZRLE: optimize XBZRLE to decrease the cache missing arei.gonglei
2014-03-11 20:34   ` Juan Quintela
2014-03-18 12:20     ` Gonglei (Arei)
2014-03-11 20:52   ` Eric Blake
2014-03-11 12:53 ` [Qemu-devel] [PATCH 04/10] XBZRLE: rebuild the cache_is_cached function arei.gonglei
2014-03-11 13:11   ` Eric Blake
2014-03-11 20:37   ` Juan Quintela
2014-03-11 12:53 ` [Qemu-devel] [PATCH 05/10] migration: Fix the migrate auto converge process arei.gonglei
2014-03-11 20:48   ` Juan Quintela
2014-03-11 20:55     ` Eric Blake
2014-03-11 22:56     ` Chegu Vinod
2014-03-18 12:23       ` Gonglei (Arei)
2014-03-11 12:53 ` [Qemu-devel] [PATCH 06/10] migraion: optimiztion xbzrle by reducing data copy arei.gonglei
2014-03-11 20:56   ` Juan Quintela
2014-03-11 20:56   ` Juan Quintela
2014-03-11 20:58   ` Eric Blake
2014-03-11 12:53 ` [Qemu-devel] [PATCH 07/10] migraion: clear the death code arei.gonglei
2014-03-11 20:58   ` Juan Quintela
2014-03-11 20:59   ` Eric Blake
2014-03-11 12:53 ` [Qemu-devel] [PATCH 08/10] migration: s/uint64_t/int64_t the definitions of it_age arei.gonglei
2014-03-11 21:02   ` Eric Blake
2014-03-11 21:08   ` Juan Quintela
2014-03-11 12:53 ` [Qemu-devel] [PATCH 09/10] migration: expose the bitmap_sync_cnt to the end user arei.gonglei
2014-03-11 13:22   ` Eric Blake
2014-03-11 21:10   ` Juan Quintela
2014-03-11 12:53 ` [Qemu-devel] [PATCH 10/10] XBZRLE: update the doc of XBZRLE arei.gonglei
2014-03-11 21:09   ` Juan Quintela

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87siqo32fx.fsf@elfo.mitica \
    --to=quintela@redhat.com \
    --cc=arei.gonglei@huawei.com \
    --cc=chenliang88@huawei.com \
    --cc=owasserm@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=weidong.huang@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.