From: Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
To: "Yordan Karadzhov (VMware)" <y.karadz@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>, linux-trace-devel@vger.kernel.org
Subject: Re: [PATCH] kernel-shark-2.alpha: Use new tracecmd API to open guest tracing file
Date: Mon, 6 Apr 2020 17:33:00 +0300 [thread overview]
Message-ID: <CAPpZLN5d4Q9k48hoC=VBTAs3g=KGTL11f_dT+5HYVf1jwq6NJw@mail.gmail.com> (raw)
In-Reply-To: <3e45fc77-619f-3d3e-3f48-47d0fcf29f1d@gmail.com>
On Mon, Apr 6, 2020 at 4:24 PM Yordan Karadzhov (VMware)
<y.karadz@gmail.com> wrote:
>
>
>
> On 3.04.20 г. 20:12 ч., Tzvetomir Stoyanov wrote:
> >> Maybe it has to be a bit more sophisticated. Does the "primary" stream
> >> know how to recognize its "secondary" streams?
> >>
> > Yes, there are tracing IDs in both host and guest files. When these IDs match,
> > files are from a same tracing session.
> > I can add "parent_stream_id" member in "struct kshark_data_stream" and
> > initialize it
> > when reading the files, according to the trace IDs.
>
> Not sure if this is the right place to put this information. I do not
> want to have "tep data" specific fields in "struct kshark_data_stream".
> Is it possible to have this mapping stored in in "struct tepdata_handle"
> instead.
Actually, the only new thing in "struct kshark_data_stream" will be the id of
the parent stream, "int parent_stream_id". All tracecmd specific mappings are in
"struct tracecmd_input" and will be accessible via tracecmd APIs.
Currently I assume that the stream with ID 0 is parent to all other
streams, opened with "-a" option.
This assumption will be removed when this "parent_stream_id" is introduced.
>
> Thanks!
> Yordan
--
Tzvetomir (Ceco) Stoyanov
VMware Open Source Technology Center
prev parent reply other threads:[~2020-04-06 14:33 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-01 16:45 [PATCH] kernel-shark-2.alpha: Use new tracecmd API to open guest tracing file Tzvetomir Stoyanov (VMware)
2020-04-03 11:39 ` Yordan Karadzhov (VMware)
2020-04-03 12:26 ` Tzvetomir Stoyanov
2020-04-03 13:25 ` Steven Rostedt
2020-04-03 15:09 ` Yordan Karadzhov (VMware)
2020-04-03 17:12 ` Tzvetomir Stoyanov
2020-04-06 13:24 ` Yordan Karadzhov (VMware)
2020-04-06 14:33 ` Tzvetomir Stoyanov [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='CAPpZLN5d4Q9k48hoC=VBTAs3g=KGTL11f_dT+5HYVf1jwq6NJw@mail.gmail.com' \
--to=tz.stoyanov@gmail.com \
--cc=linux-trace-devel@vger.kernel.org \
--cc=rostedt@goodmis.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;
as well as URLs for NNTP newsgroup(s).