mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] perf/x86/intel/bts: Fix comment about default perf_event_paranoid setting
@ 2024-08-02 10:52 James Clark
  2024-08-07 10:51 ` [tip: perf/core] " tip-bot2 for James Clark
  0 siblings, 1 reply; 2+ messages in thread
From: James Clark @ 2024-08-02 10:52 UTC (permalink / raw)
  To: linux-kernel
  Cc: James Clark, Peter Zijlstra, Ingo Molnar,
	Arnaldo Carvalho de Melo, Namhyung Kim, Mark Rutland,
	Alexander Shishkin, Jiri Olsa, Ian Rogers, Adrian Hunter, Liang,
	Kan, Thomas Gleixner, Borislav Petkov, Dave Hansen, x86,
	H. Peter Anvin, linux-perf-users

The default paranoid setting was updated in commit 0161028b7c8a
("perf/core: Change the default paranoia level to 2") so this comment is
no longer true.

Signed-off-by: James Clark <james.clark@linaro.org>
---
 arch/x86/events/intel/bts.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/x86/events/intel/bts.c b/arch/x86/events/intel/bts.c
index 974e917e65b2..8f78b0c900ef 100644
--- a/arch/x86/events/intel/bts.c
+++ b/arch/x86/events/intel/bts.c
@@ -557,9 +557,6 @@ static int bts_event_init(struct perf_event *event)
 	 * disabled, so disallow intel_bts driver for unprivileged
 	 * users on paranoid systems since it provides trace data
 	 * to the user in a zero-copy fashion.
-	 *
-	 * Note that the default paranoia setting permits unprivileged
-	 * users to profile the kernel.
 	 */
 	if (event->attr.exclude_kernel) {
 		ret = perf_allow_kernel(&event->attr);
-- 
2.34.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [tip: perf/core] perf/x86/intel/bts: Fix comment about default perf_event_paranoid setting
  2024-08-02 10:52 [PATCH] perf/x86/intel/bts: Fix comment about default perf_event_paranoid setting James Clark
@ 2024-08-07 10:51 ` tip-bot2 for James Clark
  0 siblings, 0 replies; 2+ messages in thread
From: tip-bot2 for James Clark @ 2024-08-07 10:51 UTC (permalink / raw)
  To: linux-tip-commits; +Cc: James Clark, Peter Zijlstra (Intel), x86, linux-kernel

The following commit has been merged into the perf/core branch of tip:

Commit-ID:     ea1992f36b894fe60cc3537a7f6a7af4087b999a
Gitweb:        https://git.kernel.org/tip/ea1992f36b894fe60cc3537a7f6a7af4087b999a
Author:        James Clark <james.clark@linaro.org>
AuthorDate:    Fri, 02 Aug 2024 11:52:55 +01:00
Committer:     Peter Zijlstra <peterz@infradead.org>
CommitterDate: Mon, 05 Aug 2024 16:54:47 +02:00

perf/x86/intel/bts: Fix comment about default perf_event_paranoid setting

The default paranoid setting was updated in commit 0161028b7c8a
("perf/core: Change the default paranoia level to 2") so this comment is
no longer true.

Signed-off-by: James Clark <james.clark@linaro.org>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20240802105256.335961-1-james.clark@linaro.org
---
 arch/x86/events/intel/bts.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/x86/events/intel/bts.c b/arch/x86/events/intel/bts.c
index 974e917..8f78b0c 100644
--- a/arch/x86/events/intel/bts.c
+++ b/arch/x86/events/intel/bts.c
@@ -557,9 +557,6 @@ static int bts_event_init(struct perf_event *event)
 	 * disabled, so disallow intel_bts driver for unprivileged
 	 * users on paranoid systems since it provides trace data
 	 * to the user in a zero-copy fashion.
-	 *
-	 * Note that the default paranoia setting permits unprivileged
-	 * users to profile the kernel.
 	 */
 	if (event->attr.exclude_kernel) {
 		ret = perf_allow_kernel(&event->attr);

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-08-07 10:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-08-02 10:52 [PATCH] perf/x86/intel/bts: Fix comment about default perf_event_paranoid setting James Clark
2024-08-07 10:51 ` [tip: perf/core] " tip-bot2 for James Clark

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®