From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Markus Armbruster <armbru@redhat.com>
Cc: Denis Plotnikov <den-plotnikov@yandex-team.ru>,
qemu-devel@nongnu.org, yc-core@yandex-team.ru,
michael.roth@amd.com, vsementsov@yandex-team.ru,
marcandre.lureau@gmail.com
Subject: Re: [PATCH v3] qapi/qmp: Add timestamps to qmp command responses
Date: Fri, 14 Oct 2022 12:31:09 +0100 [thread overview]
Message-ID: <Y0lIfTS4/5UMItkk@redhat.com> (raw)
In-Reply-To: <8735bqu4ym.fsf@pond.sub.org>
On Fri, Oct 14, 2022 at 11:31:13AM +0200, Markus Armbruster wrote:
> Daniel P. Berrangé <berrange@redhat.com> writes:
>
> > On Thu, Oct 13, 2022 at 05:00:26PM +0200, Markus Armbruster wrote:
> >> Denis Plotnikov <den-plotnikov@yandex-team.ru> writes:
> >>
> >> > Add "start" & "end" time values to qmp command responses.
> >>
> >> Please spell it QMP. More of the same below.
> >>
> >> >
> >> > These time values are added to let the qemu management layer get the exact
> >> > command execution time without any other time variance which might be brought by
> >> > other parts of management layer or qemu internals. This is particulary useful
> >> > for the management layer logging for later problems resolving.
> >>
> >> I'm still having difficulties seeing the value add over existing
> >> tracepoints and logging.
> >>
> >> Can you tell me about a problem you cracked (or could have cracked) with
> >> the help of this?
> >
> > Consider your QMP client is logging all commands and replies in its
> > own logfile (libvirt can do this). Having this start/end timestamps
> > included means the QMP client log is self contained.
>
> A QMP client can include client-side timestamps in its log. What value
> is being added by server-side timestamps? According to the commit
> message, it's for getting "the exact command execution time without any
> other time variance which might be brought by other parts of management
> layer or qemu internals." Why is that useful? In particular, why is
> excluding network and QEMU queueing delays (inbound and outbound)
> useful?
Lets, say some commands normally runs in ~100ms, but occasionally
runs in 2secs, and you want to understand why.
A first step is understanding whether a given command itself is
slow at executing, or whether its execution has merely been
delayed because some other aspect of QEMU has delayed its execution.
If the server timestamps show it was very fast, then that indicates
delayed processing. Thus instead of debugging the slow command, I
can think about what scenarios would be responsible for the delay.
Perhaps a previous QMP command was very slow, or maybe there is
simply a large volume of QMP commands backlogged, or some part of
QEMU got blocked.
Another case would be a command that is normally fast, and sometimes
is slower, but still relatively fast. The network and queueing side
might be a significant enough proportion of the total time to obscure
the slowdown. If you can eliminate the non-execution time, you can
see the performance trends over time to spot the subtle slowdowns
and detect abnormal behaviour before it becomes too terrible.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2022-10-14 11:46 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-11 15:34 [PATCH v3] qapi/qmp: Add timestamps to qmp command responses Denis Plotnikov
2022-10-13 15:00 ` Markus Armbruster
2022-10-13 15:40 ` Daniel P. Berrangé
2022-10-14 9:31 ` Markus Armbruster
2022-10-14 11:31 ` Daniel P. Berrangé [this message]
2022-10-14 12:57 ` Markus Armbruster
2022-10-14 13:19 ` Daniel P. Berrangé
2022-10-16 9:25 ` Denis Plotnikov
2022-10-17 17:23 ` Markus Armbruster
2022-10-14 9:47 ` Denis Plotnikov
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=Y0lIfTS4/5UMItkk@redhat.com \
--to=berrange@redhat.com \
--cc=armbru@redhat.com \
--cc=den-plotnikov@yandex-team.ru \
--cc=marcandre.lureau@gmail.com \
--cc=michael.roth@amd.com \
--cc=qemu-devel@nongnu.org \
--cc=vsementsov@yandex-team.ru \
--cc=yc-core@yandex-team.ru \
/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.