2022-05-28 19:54:55

by baihaowen

[permalink] [raw]
Subject: [PATCH] perf/core: drop the useless LIST_HEAD in perf_event_remove_on_exec()

Drop LIST_HEAD where the variable it declares is never used.

Signed-off-by: Haowen Bai <[email protected]>
---
kernel/events/core.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/kernel/events/core.c b/kernel/events/core.c
index 950b25c3f210..80782cddb1da 100644
--- a/kernel/events/core.c
+++ b/kernel/events/core.c
@@ -4257,7 +4257,6 @@ static void perf_event_remove_on_exec(int ctxn)
{
struct perf_event_context *ctx, *clone_ctx = NULL;
struct perf_event *event, *next;
- LIST_HEAD(free_list);
unsigned long flags;
bool modified = false;

--
2.7.4



2022-05-28 20:07:03

by tip-bot2 for Jacob Pan

[permalink] [raw]
Subject: [tip: perf/urgent] perf/core: Remove unused local variable

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

Commit-ID: 8b4dd2d8627e88dc3bd71bf29c48aaae2b69572b
Gitweb: https://git.kernel.org/tip/8b4dd2d8627e88dc3bd71bf29c48aaae2b69572b
Author: Haowen Bai <[email protected]>
AuthorDate: Fri, 27 May 2022 18:03:54 +08:00
Committer: Ingo Molnar <[email protected]>
CommitterDate: Fri, 27 May 2022 12:14:16 +02:00

perf/core: Remove unused local variable

Drop LIST_HEAD() where the variable it declares is never used.

Compiler probably never warned us, because the LIST_HEAD()
initializer is technically 'usage'.

[ mingo: Tweak changelog. ]

Signed-off-by: Haowen Bai <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
---
kernel/events/core.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/kernel/events/core.c b/kernel/events/core.c
index 950b25c..80782cd 100644
--- a/kernel/events/core.c
+++ b/kernel/events/core.c
@@ -4257,7 +4257,6 @@ static void perf_event_remove_on_exec(int ctxn)
{
struct perf_event_context *ctx, *clone_ctx = NULL;
struct perf_event *event, *next;
- LIST_HEAD(free_list);
unsigned long flags;
bool modified = false;