Linux-Doc Archive mirror
 help / color / mirror / Atom feed
From: Daniel Bristot de Oliveira <bristot@kernel.org>
To: Daniel Bristot de Oliveira <bristot@kernel.org>,
	Steven Rostedt <rostedt@goodmis.org>,
	linux-trace-kernel@vger.kernel.org
Cc: Jonathan Corbet <corbet@lwn.net>,
	Juri Lelli <juri.lelli@redhat.com>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 0/8] rtla usage improvements
Date: Wed, 24 Apr 2024 16:36:49 +0200	[thread overview]
Message-ID: <cover.1713968967.git.bristot@kernel.org> (raw)

Hi

These are some changes I have accumulated in the last
weeks. Mostly improving the tool's user experience.

 - On timerlat top, remove an extra \n that was breaking
   the output.

 - Replace \t with '       ' on the auto analysis, fixing some
   copy & paste.

 - On timerlat top, do not print TTY formatting characters when
   the output is not sent to a terminal, because they break the
   output.

 - on timerlat top and hist, add a summary with the overall results.
   For instance, the minimum value for all CPUs, the overall average
   and the maximum value from all CPUs.

 - Add a --warm-up <seconds> option, allowing the workload to run for
   <seconds> before starting to collect results.

 - Add a --trace-buffer-size option, allowing the user to set the
   tracing buffer size. This is mainly useful for reducing the trace
   file.

 - Make the user-space threads the default choice, also adding the
   a -k option, allowing the user to switch to kerne-threads.

Daniel Bristot de Oliveira (8):
  rtla/timerlat: Simplify "no value" printing on top
  rtla/auto-analysis: Replace \t with spaces
  rtla/timerlat: Use pretty formatting only on interactive tty
  rtla/timerlat: Add a summary for top mode
  rtla/timerlat: Add a summary for hist mode
  rtla: Add the --warm-up option
  rtla/timerlat: Make user-space threads the default
  rtla: Add --trace-buffer-size option

 Documentation/tools/rtla/common_options.rst   |   7 +
 .../tools/rtla/common_timerlat_options.rst    |   6 +-
 tools/tracing/rtla/src/osnoise_hist.c         |  41 ++-
 tools/tracing/rtla/src/osnoise_top.c          |  41 ++-
 tools/tracing/rtla/src/timerlat_aa.c          | 109 ++++----
 tools/tracing/rtla/src/timerlat_hist.c        | 220 ++++++++++++++--
 tools/tracing/rtla/src/timerlat_top.c         | 236 +++++++++++++++---
 tools/tracing/rtla/src/trace.c                |  15 ++
 tools/tracing/rtla/src/trace.h                |   1 +
 9 files changed, 566 insertions(+), 110 deletions(-)

-- 
2.44.0


             reply	other threads:[~2024-04-24 14:37 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-24 14:36 Daniel Bristot de Oliveira [this message]
2024-04-24 14:36 ` [PATCH 1/8] rtla/timerlat: Simplify "no value" printing on top Daniel Bristot de Oliveira
2024-04-24 14:36 ` [PATCH 2/8] rtla/auto-analysis: Replace \t with spaces Daniel Bristot de Oliveira
2024-04-24 14:36 ` [PATCH 3/8] rtla/timerlat: Use pretty formatting only on interactive tty Daniel Bristot de Oliveira
2024-04-24 14:36 ` [PATCH 4/8] rtla/timerlat: Add a summary for top mode Daniel Bristot de Oliveira
2024-04-24 14:36 ` [PATCH 5/8] rtla/timerlat: Add a summary for hist mode Daniel Bristot de Oliveira
2024-04-24 14:36 ` [PATCH 6/8] rtla: Add the --warm-up option Daniel Bristot de Oliveira
2024-04-24 14:36 ` [PATCH 7/8] rtla/timerlat: Make user-space threads the default Daniel Bristot de Oliveira
2024-04-24 14:36 ` [PATCH 8/8] rtla: Add --trace-buffer-size option Daniel Bristot de Oliveira
2024-05-16 13:25   ` Daniel Bristot de Oliveira

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=cover.1713968967.git.bristot@kernel.org \
    --to=bristot@kernel.org \
    --cc=corbet@lwn.net \
    --cc=juri.lelli@redhat.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-trace-kernel@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 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 read-only IMAP folder(s) and NNTP newsgroup(s).