From: "Tzvetomir Stoyanov (VMware)" <tz.stoyanov@gmail.com>
To: rostedt@goodmis.org
Cc: linux-trace-devel@vger.kernel.org
Subject: RFC [PATCH 0/5] tsc2nsec fixes
Date: Wed, 28 Apr 2021 15:28:34 +0300 [thread overview]
Message-ID: <20210428122839.805296-1-tz.stoyanov@gmail.com> (raw)
Removed the not so stable logic for tsc2nsec offset and replaced it with
trace-cmd library APIs which can be used from the library users to achieve
the same functionality.
These changes ignore the offset, part of the tsc2nsec data written in the
trace file metadata. Is it save to completely remove the offset field and to
break the format of this trace file option?
This is a RFC patchset, additional updates are required in case the changes
are approved:
- Update trace-cmd documentation.
- Remove offset field from tsc2nsec structures.
Tzvetomir Stoyanov (VMware) (5):
trace-cmd: Remove ts offset from tsc2nsec conversion
trace-cmd library: Remove useless check before applying ts offset
trace-cmd library: Store the timestamp of the first event when reading
a trace file
trace-cmd library: New API for modifyning the timestamp offset
trace-cmd report: New option --align-ts
include/trace-cmd/trace-cmd.h | 2 ++
lib/trace-cmd/trace-input.c | 39 ++++++++++++++++++++--
tracecmd/trace-read.c | 25 ++++++++++++--
tracecmd/trace-record.c | 62 -----------------------------------
tracecmd/trace-usage.c | 1 +
5 files changed, 61 insertions(+), 68 deletions(-)
--
2.30.2
next reply other threads:[~2021-04-28 12:28 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-28 12:28 Tzvetomir Stoyanov (VMware) [this message]
2021-04-28 12:28 ` [PATCH 1/5] trace-cmd: Remove ts offset from tsc2nsec conversion Tzvetomir Stoyanov (VMware)
2021-04-28 12:28 ` [PATCH 2/5] trace-cmd library: Remove useless check before applying ts offset Tzvetomir Stoyanov (VMware)
2021-04-28 12:28 ` [PATCH 3/5] trace-cmd library: Store the timestamp of the first event when reading a trace file Tzvetomir Stoyanov (VMware)
2021-04-28 12:28 ` [PATCH 4/5] trace-cmd library: New API for modifyning the timestamp offset Tzvetomir Stoyanov (VMware)
2021-04-28 12:28 ` [PATCH 5/5] trace-cmd report: New option --align-ts Tzvetomir Stoyanov (VMware)
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=20210428122839.805296-1-tz.stoyanov@gmail.com \
--to=tz.stoyanov@gmail.com \
--cc=linux-trace-devel@vger.kernel.org \
--cc=rostedt@goodmis.org \
/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.