From: Steven Rostedt <rostedt@goodmis.org>
To: Jason Behmer <jbehmer@google.com>
Cc: tom.zanussi@linux.intel.com, linux-kernel@vger.kernel.org
Subject: Re: Nested events with zero deltas, can use absolute timestamps instead?
Date: Fri, 29 Mar 2019 12:52:58 -0400 [thread overview]
Message-ID: <20190329125258.2c6fe0cb@gandalf.local.home> (raw)
In-Reply-To: <CAMmhGqKc27W03roONYXhmwB0dtz5Z8nGoS2MLSsKJ3Zotv5-JA@mail.gmail.com>
On Fri, 29 Mar 2019 09:30:36 -0700
Jason Behmer <jbehmer@google.com> wrote:
> Hi Steven and Tom,
> I recently ran into a problem with correlating timestamps across CPUs and
> finally was able to track it down to the fact that nested events are given
> zero deltas, and so are placed earlier in time than they actually occurred.
>
> I'm on an older version and wanted to check if this problem still exists on
> head and it appears that it does, but I also ran into Tom's commits adding
> the ability to use absolute timestamps instead of deltas. It seems like
> this option could fix the problem.
>
> It seems like ideally nested events could just always use these absolute
> timestamps instead of zero deltas, as it appears the implementation of
> absolute timestamps is flexible enough to have a mix of deltas and
> absolutes in the stream of events. A fix that would require less work
> would be for me to just turn on absolute timestamps for my entire trace,
> but I see that the file to do that isn't yet writable to support that.
>
> Before trying to do either of these things I wanted to run it by both of
> you to see if there's something I'm missing here. What do you think?
>
Yes this is an issue I need to fix. I have some ideas on how to handle
this. I may need to go about looking into them again.
It may be possible to add an option to have all nested events use the
full timestamp as well.
I'll have to think about this a bit more.
-- Steve
next parent reply other threads:[~2019-03-29 16:53 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CAMmhGqKc27W03roONYXhmwB0dtz5Z8nGoS2MLSsKJ3Zotv5-JA@mail.gmail.com>
2019-03-29 16:52 ` Steven Rostedt [this message]
2019-04-01 22:54 ` Jason Behmer
2019-04-02 2:20 ` Steven Rostedt
2019-04-02 4:42 ` Jason Behmer
2019-05-24 14:17 ` Jason Behmer
2019-05-24 15:00 ` Steven Rostedt
2019-05-24 15:11 ` Jason Behmer
2019-05-24 15:24 ` Steven Rostedt
2019-05-24 15:48 ` Jason Behmer
2019-05-24 15:54 ` Steven Rostedt
2019-05-24 15:58 ` Steven Rostedt
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=20190329125258.2c6fe0cb@gandalf.local.home \
--to=rostedt@goodmis.org \
--cc=jbehmer@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=tom.zanussi@linux.intel.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®