Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1340007 > unrolled thread

[patch V3 09/28] perf: Allow storage of pmu private data in event

Started byThomas Gleixner <tglx@linutronix.de>
First post2016-02-22 23:30 +0100
Last post2016-02-29 12:10 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [patch V3 09/28] perf: Allow storage of pmu private data in event Thomas Gleixner <tglx@linutronix.de> - 2016-02-22 23:30 +0100
    [tip:perf/core] perf: Allow storage of PMU private data in event tip-bot for Thomas Gleixner <tipbot@zytor.com> - 2016-02-29 12:10 +0100

#1340007 — [patch V3 09/28] perf: Allow storage of pmu private data in event

FromThomas Gleixner <tglx@linutronix.de>
Date2016-02-22 23:30 +0100
Subject[patch V3 09/28] perf: Allow storage of pmu private data in event
Message-ID<r579q-4ce-81@gated-at.bofh.it>
For pmus which are not per cpu, but e.g. per package/socket, we want to be
able to store a reference to the underlying per package/socket facility in the
event at init time so we can avoid magic storage constructs in the pmu driver.

This allows us to get rid of the per cpu dance in the intel uncore and rapl
drivers and avoids a lookup of the per package data in the perf hotpath.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
---
 include/linux/perf_event.h |    1 +
 1 file changed, 1 insertion(+)

--- a/include/linux/perf_event.h
+++ b/include/linux/perf_event.h
@@ -467,6 +467,7 @@ struct perf_event {
 	int				group_flags;
 	struct perf_event		*group_leader;
 	struct pmu			*pmu;
+	void				*pmu_private;
 
 	enum perf_event_active_state	state;
 	unsigned int			attach_state;

[toc] | [next] | [standalone]


#1345646 — [tip:perf/core] perf: Allow storage of PMU private data in event

Fromtip-bot for Thomas Gleixner <tipbot@zytor.com>
Date2016-02-29 12:10 +0100
Subject[tip:perf/core] perf: Allow storage of PMU private data in event
Message-ID<r7tS9-2et-1@gated-at.bofh.it>
In reply to#1340007
Commit-ID:  54d751d4ad357c817907fe89db3222b97ff66db3
Gitweb:     http://git.kernel.org/tip/54d751d4ad357c817907fe89db3222b97ff66db3
Author:     Thomas Gleixner <tglx@linutronix.de>
AuthorDate: Mon, 22 Feb 2016 22:19:14 +0000
Committer:  Ingo Molnar <mingo@kernel.org>
CommitDate: Mon, 29 Feb 2016 09:35:16 +0100

perf: Allow storage of PMU private data in event

For PMUs which are not per CPU, but e.g. per package/socket, we want to be
able to store a reference to the underlying per package/socket facility in the
event at init time so we can avoid magic storage constructs in the PMU driver.

This allows us to get rid of the per CPU dance in the intel uncore and RAPL
drivers and avoids a lookup of the per package data in the perf hotpath.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Andi Kleen <andi.kleen@intel.com>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Harish Chegondi <harish.chegondi@intel.com>
Cc: Jacob Pan <jacob.jun.pan@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Kan Liang <kan.liang@intel.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: Vince Weaver <vincent.weaver@maine.edu>
Cc: linux-kernel@vger.kernel.org
Link: http://lkml.kernel.org/r/20160222221011.364140369@linutronix.de
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
 include/linux/perf_event.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h
index f5c5a3f..a9d8cab 100644
--- a/include/linux/perf_event.h
+++ b/include/linux/perf_event.h
@@ -468,6 +468,7 @@ struct perf_event {
 	int				group_flags;
 	struct perf_event		*group_leader;
 	struct pmu			*pmu;
+	void				*pmu_private;
 
 	enum perf_event_active_state	state;
 	unsigned int			attach_state;

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web