From: Dario Faggioli <dfaggioli@suse.com>
To: Sean Christopherson <seanjc@google.com>
Cc: Stefano De Venuto <stefano.devenuto99@gmail.com>,
linux-kernel@vger.kernel.org, pbonzini@redhat.com,
vkuznets@redhat.com, wanpengli@tencent.com, jmattson@google.com,
x86@kernel.org, hpa@zytor.com, kvm@vger.kernel.org,
rostedt@goodmis.org, y.karadz@gmail.com
Subject: Re: [PATCH] Move VMEnter and VMExit tracepoints closer to the actual event
Date: Fri, 28 May 2021 19:03:53 +0200 [thread overview]
Message-ID: <9e9a9aeefd288c70bdf493601f99820e10dd9eea.camel@suse.com> (raw)
In-Reply-To: <YK0j6MrOCFeQSHCa@google.com>
[-- Attachment #1: Type: text/plain, Size: 1730 bytes --]
On Tue, 2021-05-25 at 16:20 +0000, Sean Christopherson wrote:
> On Fri, May 21, 2021, Dario Faggioli wrote:
> > >
> > Indeed. So, do you happen to have in mind what could be the best
> > place
> > and the best way for documenting this?
>
> I didn't have anything in mind, but my gut reaction is to add a new
> file dedicated
> to tracing/tracepoints in KVM, e.g.
>
> Documentation/virt/kvm/tracepoints.rst or
> Documentation/virt/kvm/tracing.rst
>
Ok. Well, FWIW, this seems a good idea to me. :-)
> I'm sure there are all sorts of tips and tricks people have for using
> KVM's
> tracepoints, it would be nice to provide a way to capture and
> disseminate them.
> My only hesitation is that Documentation/virt/kvm/ might be too
> formal for what
> would effectively be a wiki of sorts.
>
Yeah, understand the concerns, I think. However, it seems to me that
how to interpret the kernel KVM tracepoint (i.e., this fact that they
mark the rather the beginning of the "logical" entry and exit sequences
rather than the actual instructions) does belong in the kernel's own
documentation, i.e., where you proposed above.
Surely when we'll have something like that, it seems natural that we'd
want to have more stuff there, and we'll have to judge what's best
suited for it and what should perhaps be somewhere else... But I think
it's worth a try, and I probably will try to put something together.
Thanks and Regards
--
Dario Faggioli, Ph.D
http://about.me/dario.faggioli
Virtualization Software Engineer
SUSE Labs, SUSE https://www.suse.com/
-------------------------------------------------------------------
<<This happens because _I_ choose it to happen!>> (Raistlin Majere)
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
prev parent reply other threads:[~2021-05-28 17:03 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-19 18:23 Stefano De Venuto
2021-05-20 6:05 ` Dario Faggioli
2021-05-20 7:21 ` Thomas Gleixner
2021-05-21 7:13 ` Dario Faggioli
2021-05-20 15:32 ` Sean Christopherson
2021-05-20 16:18 ` Paolo Bonzini
2021-05-21 7:58 ` Dario Faggioli
2021-05-28 16:55 ` Dario Faggioli
2021-05-28 16:57 ` Paolo Bonzini
2021-05-21 7:51 ` Dario Faggioli
2021-05-25 16:20 ` Sean Christopherson
2021-05-28 17:03 ` Dario Faggioli [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=9e9a9aeefd288c70bdf493601f99820e10dd9eea.camel@suse.com \
--to=dfaggioli@suse.com \
--cc=hpa@zytor.com \
--cc=jmattson@google.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pbonzini@redhat.com \
--cc=rostedt@goodmis.org \
--cc=seanjc@google.com \
--cc=stefano.devenuto99@gmail.com \
--cc=vkuznets@redhat.com \
--cc=wanpengli@tencent.com \
--cc=x86@kernel.org \
--cc=y.karadz@gmail.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
all inboxes | Powered by JetHome®