From: Avi Kivity <avi@redhat.com>
To: Cornelia Huck <cornelia.huck@de.ibm.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>,
Christian Borntraeger <borntraeger@de.ibm.com>,
Carsten Otte <cotte@de.ibm.com>, Alexander Graf <agraf@suse.de>,
Heiko Carstens <heiko.carstens@de.ibm.com>,
Martin Schwidefsky <schwidefsky@de.ibm.com>,
KVM <kvm@vger.kernel.org>,
linux-s390 <linux-s390@vger.kernel.org>
Subject: Re: [PATCH 0/3] KVM: s390: Trace events support.
Date: Thu, 26 Jul 2012 12:35:10 +0300 [thread overview]
Message-ID: <50110F4E.6030106@redhat.com> (raw)
In-Reply-To: <1343056830-45290-1-git-send-email-cornelia.huck@de.ibm.com>
On 07/23/2012 06:20 PM, Cornelia Huck wrote:
> Avi, Marcelo,
>
> here's a patch set that introduces trace events for kvm/s390.
>
> It's split into two parts:
>
> - Trace points for architecture-defined events, like intercepts.
> This patch calls into the disassembler via the interface provided
> by the first patch. These trace points show up under events/kvm/.
> - Trace points for implementation-specific events like interrupt
> injection. These show up under a new trace system, kvm-s390.
I don't see what's the difference between the two types. Isn't
interrupt injection architectural?
On x86, the implementation tracepoints are ones that may go away if the
implementation changes significantly, while the architectural ones will
not go away unless the architecture is changed. In fact creation and
destruction of vcpus and reset requests are not only architectural,
they're generic, you may as well add them to the arch independent trace
code.
btw - why are vcpu creation and destruction useful events to trace?
--
error compiling committee.c: too many arguments to function
next prev parent reply other threads:[~2012-07-26 9:35 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-23 15:20 [PATCH 0/3] KVM: s390: Trace events support Cornelia Huck
2012-07-23 15:20 ` [PATCH 1/3] s390/dis: Instruction decoding interface Cornelia Huck
2012-07-23 15:20 ` [PATCH 2/3] KVM: s390: Add architectural trace events Cornelia Huck
2012-07-23 15:20 ` [PATCH 3/3] KVM: s390: Add implementation-specific " Cornelia Huck
2012-07-26 9:35 ` Avi Kivity [this message]
2012-07-26 10:47 ` [PATCH 0/3] KVM: s390: Trace events support Cornelia Huck
2012-07-26 11:05 ` Avi Kivity
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=50110F4E.6030106@redhat.com \
--to=avi@redhat.com \
--cc=agraf@suse.de \
--cc=borntraeger@de.ibm.com \
--cc=cornelia.huck@de.ibm.com \
--cc=cotte@de.ibm.com \
--cc=heiko.carstens@de.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=mtosatti@redhat.com \
--cc=schwidefsky@de.ibm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).