public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andi Kleen <ak@linux.intel.com>
To: Dmitry Vyukov <dvyukov@google.com>
Cc: namhyung@kernel.org, irogers@google.com,
	linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org,
	Arnaldo Carvalho de Melo <acme@kernel.org>
Subject: Re: [PATCH v5 0/8] perf report: Add latency and parallelism profiling
Date: Mon, 10 Feb 2025 09:11:57 -0800	[thread overview]
Message-ID: <Z6ozXeOaZJMNRy3x@tassilo> (raw)
In-Reply-To: <CACT4Y+bOVd4hw2fkE4ahJnbyAuc_=wMu_HMkAUbAhL--dTxBiA@mail.gmail.com>

> > But yes that perf has too many options and is not intuitive and most
> > people miss most of the features is an inherent problem. I don't have
> > a good solution for that unfortunately, other than perhaps better
> > documentation.
> 
> I don't think this is a solution :(
> 
> I provided lots of rationale for making this latency profiling enabled
> by default in this patch series for this reason. If we just capture
> context switches, then we can show both overhead and latency, even if
> we sort by overhead by default, people would still see the latency
> column and may start thinking/asking questions.
> But this is not happening, so mostly people on this thread will know about it :)

Maybe something that could be done is to have some higher level configurations
for perf report that are easier to understand.

This kind of already exists e.g. in perf mem report which is just a
wrapper around perf report with some magic flags.

In this case you could have perf latency report (or maybe some other
syntax like perf report --mode latency) 

There are a few others that would make sense, like basic time series
or disabling children aggregation.

Another possibility would be to find a heuristic where perf report
can detect that a latency problem might be there (e.g. varying
usage of CPUs) and suggest it automatically.

-Andi

  reply	other threads:[~2025-02-10 17:11 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-05 16:27 [PATCH v5 0/8] perf report: Add latency and parallelism profiling Dmitry Vyukov
2025-02-05 16:27 ` [PATCH v5 1/8] perf report: Add machine parallelism Dmitry Vyukov
2025-02-05 16:27 ` [PATCH v5 2/8] perf report: Add parallelism sort key Dmitry Vyukov
2025-02-05 16:27 ` [PATCH v5 3/8] perf report: Switch filtered from u8 to u16 Dmitry Vyukov
2025-02-05 16:27 ` [PATCH v5 4/8] perf report: Add parallelism filter Dmitry Vyukov
2025-02-05 16:27 ` [PATCH v5 5/8] perf report: Add latency output field Dmitry Vyukov
2025-02-05 16:27 ` [PATCH v5 6/8] perf report: Add --latency flag Dmitry Vyukov
2025-02-07  3:44   ` Namhyung Kim
2025-02-07  7:23     ` Dmitry Vyukov
2025-02-11  1:02       ` Namhyung Kim
2025-02-11  8:30         ` Dmitry Vyukov
2025-02-11  8:42         ` Dmitry Vyukov
2025-02-11 17:42           ` Namhyung Kim
2025-02-11 20:23             ` Dmitry Vyukov
2025-02-12 19:47               ` Namhyung Kim
2025-02-13  9:09                 ` Dmitry Vyukov
2025-02-07  3:53   ` Namhyung Kim
2025-02-07 11:42     ` Dmitry Vyukov
2025-02-05 16:27 ` [PATCH v5 7/8] perf report: Add latency and parallelism profiling documentation Dmitry Vyukov
2025-02-05 16:27 ` [PATCH v5 8/8] perf hist: Shrink struct hist_entry size Dmitry Vyukov
2025-02-06 18:30 ` [PATCH v5 0/8] perf report: Add latency and parallelism profiling Andi Kleen
2025-02-06 18:41   ` Dmitry Vyukov
2025-02-06 18:51     ` Ian Rogers
2025-02-07  3:57       ` Namhyung Kim
2025-02-07 11:44         ` Dmitry Vyukov
2025-02-06 18:57     ` Andi Kleen
2025-02-06 19:07       ` Andi Kleen
2025-02-07  8:16   ` Dmitry Vyukov
2025-02-07 18:30     ` Andi Kleen
2025-02-10  7:17       ` Dmitry Vyukov
2025-02-10 17:11         ` Andi Kleen [this message]
2025-02-13  9:09         ` Dmitry Vyukov

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=Z6ozXeOaZJMNRy3x@tassilo \
    --to=ak@linux.intel.com \
    --cc=acme@kernel.org \
    --cc=dvyukov@google.com \
    --cc=irogers@google.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-perf-users@vger.kernel.org \
    --cc=namhyung@kernel.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