LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Alexander Shishkin <alexander.shishkin@linux.intel.com>
To: Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Ingo Molnar <mingo@redhat.com>,
	linux-kernel@vger.kernel.org, Will Deacon <will.deacon@arm.com>,
	Adrian Hunter <adrian.hunter@intel.com>,
	Alexander Shishkin <alexander.shishkin@linux.intel.com>
Subject: [PATCH v1 0/4] perf: Allow suppressing useless AUX records
Date: Tue, 14 Nov 2017 14:30:20 +0200	[thread overview]
Message-ID: <20171114123024.11517-1-alexander.shishkin@linux.intel.com> (raw)

Hi guys,

[Fixed the braindamage of the previous version as pointed out by
Adrian]

It's been brought to my attention many times that the AUX records are
not always useful. One instance is the AUX overwrite mode, where we
generate one such record every time the target task schedules out.

This patchset adds a bit to the attribute structure that would enable
suppressing the AUX records when the only the only set flag is OVERWRITE.
In this case, at least for the PT and BTS data we just ignore these
records. But at the same time, AUX overwrite events are likely to be
kept running over longer periods of time, making the buildup of AUX
records an unnecessary annoyance.

Alexander Shishkin (4):
  perf: Allow suppressing AUX records
  tools, perf_event.h: Synchronize
  perf tools: Add 'suppress_aux' attribute bit definition and fallback
  perf intel-pt, intel-bts: Suppress useless AUX records by default

 include/uapi/linux/perf_event.h       |  3 ++-
 kernel/events/core.c                  |  5 +++++
 kernel/events/ring_buffer.c           | 12 ++++++++++--
 tools/include/uapi/linux/perf_event.h |  3 ++-
 tools/perf/arch/x86/util/auxtrace.c   |  2 ++
 tools/perf/util/evsel.c               |  9 +++++++++
 6 files changed, 30 insertions(+), 4 deletions(-)

-- 
2.15.0

             reply	other threads:[~2017-11-14 12:33 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-14 12:30 Alexander Shishkin [this message]
2017-11-14 12:30 ` [PATCH v1 1/4] perf: Allow suppressing " Alexander Shishkin
2017-11-15 12:00   ` Peter Zijlstra
2018-01-15 15:00     ` [PATCH] " Alexander Shishkin
2018-03-29 11:54       ` Peter Zijlstra
2018-03-31  9:35         ` Ingo Molnar
2018-04-03 17:32           ` Will Deacon
2018-04-04 14:53             ` [PATCH v2] perf: Suppress AUX/OVERWRITE records Alexander Shishkin
2018-05-04 12:09               ` Alexander Shishkin
2018-05-04 15:35                 ` Arnaldo Carvalho de Melo
2018-05-04 15:36                   ` Arnaldo Carvalho de Melo
2018-09-25  9:27               ` [tip:perf/core] " tip-bot for Alexander Shishkin
2017-11-14 12:30 ` [PATCH v1 2/4] tools, perf_event.h: Synchronize Alexander Shishkin
2017-11-14 12:30 ` [PATCH v1 3/4] perf tools: Add 'suppress_aux' attribute bit definition and fallback Alexander Shishkin
2017-11-14 12:30 ` [PATCH v1 4/4] perf intel-pt, intel-bts: Suppress useless AUX records by default Alexander Shishkin

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=20171114123024.11517-1-alexander.shishkin@linux.intel.com \
    --to=alexander.shishkin@linux.intel.com \
    --cc=a.p.zijlstra@chello.nl \
    --cc=acme@redhat.com \
    --cc=adrian.hunter@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=will.deacon@arm.com \
    --subject='Re: [PATCH v1 0/4] perf: Allow suppressing useless AUX records' \
    /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

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).