From: Peter Xu <peterx@redhat.com>
To: qemu-devel@nongnu.org
Cc: Fabiano Rosas <farosas@suse.de>,
Juraj Marcin <jmarcin@redhat.com>,
"Dr. David Alan Gilbert" <dave@treblig.org>,
Markus Armbruster <armbru@redhat.com>
Subject: Re: [PATCH] migration/postcopy: Add latency distribution report for blocktime
Date: Fri, 13 Jun 2025 10:25:09 -0400 [thread overview]
Message-ID: <aEw0xUO6t_Z44ruM@x1.local> (raw)
In-Reply-To: <20250609223607.34387-1-peterx@redhat.com>
On Mon, Jun 09, 2025 at 06:36:07PM -0400, Peter Xu wrote:
> Add the latency distribution too for blocktime, using order-of-two buckets.
> It accounts for all the faults, from either vCPU or non-vCPU threads. With
> prior rework, it's very easy to achieve by adding an array to account for
> faults in each buckets.
>
> Sample output for HMP (while for QMP it's simply an array):
>
> Postcopy Latency Distribution:
> [ 1 us - 2 us ]: 0
> [ 2 us - 4 us ]: 0
> [ 4 us - 8 us ]: 1
> [ 8 us - 16 us ]: 2
> [ 16 us - 32 us ]: 2
> [ 32 us - 64 us ]: 3
> [ 64 us - 128 us ]: 10169
> [ 128 us - 256 us ]: 50151
> [ 256 us - 512 us ]: 12876
> [ 512 us - 1 ms ]: 97
> [ 1 ms - 2 ms ]: 42
> [ 2 ms - 4 ms ]: 44
> [ 4 ms - 8 ms ]: 93
> [ 8 ms - 16 ms ]: 138
> [ 16 ms - 32 ms ]: 0
> [ 32 ms - 65 ms ]: 0
> [ 65 ms - 131 ms ]: 0
> [ 131 ms - 262 ms ]: 0
> [ 262 ms - 524 ms ]: 0
> [ 524 ms - 1 sec ]: 0
> [ 1 sec - 2 sec ]: 0
> [ 2 sec - 4 sec ]: 0
> [ 4 sec - 8 sec ]: 0
> [ 8 sec - 16 sec ]: 0
>
> Cc: Dr. David Alan Gilbert <dave@treblig.org>
> Cc: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Peter Xu <peterx@redhat.com>
Just to keep a record, the new version of this patch is included here:
https://lore.kernel.org/all/20250613141217.474825-15-peterx@redhat.com/
There're trivial changes there, on marking the feature unstable, meanwhile
adjustments to suite the us->ns conversion suggested by Dave.
--
Peter Xu
prev parent reply other threads:[~2025-06-13 14:26 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-09 22:36 [PATCH] migration/postcopy: Add latency distribution report for blocktime Peter Xu
2025-06-10 0:27 ` Dr. David Alan Gilbert
2025-06-10 17:46 ` Fabiano Rosas
2025-06-13 14:25 ` Peter Xu [this message]
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=aEw0xUO6t_Z44ruM@x1.local \
--to=peterx@redhat.com \
--cc=armbru@redhat.com \
--cc=dave@treblig.org \
--cc=farosas@suse.de \
--cc=jmarcin@redhat.com \
--cc=qemu-devel@nongnu.org \
/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.