LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] perf/x86: remove unused assignment to pointer e
@ 2021-08-04 11:57 Colin King
2021-08-05 9:34 ` [tip: perf/core] " tip-bot2 for Colin Ian King
2021-08-26 8:09 ` [tip: perf/core] perf/x86: Remove unused assignment to pointer 'e' tip-bot2 for Colin Ian King
0 siblings, 2 replies; 3+ messages in thread
From: Colin King @ 2021-08-04 11:57 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Namhyung Kim,
Thomas Gleixner, Borislav Petkov, x86, H . Peter Anvin,
linux-perf-users
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
The pointer e is being assigned a value that is never read, the assignment
is redundant and can be removed.
Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
arch/x86/events/core.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch/x86/events/core.c b/arch/x86/events/core.c
index 1eb45139fcc6..4110fe1b783e 100644
--- a/arch/x86/events/core.c
+++ b/arch/x86/events/core.c
@@ -1087,10 +1087,8 @@ int x86_schedule_events(struct cpu_hw_events *cpuc, int n, int *assign)
* validate an event group (assign == NULL)
*/
if (!unsched && assign) {
- for (i = 0; i < n; i++) {
- e = cpuc->event_list[i];
+ for (i = 0; i < n; i++)
static_call_cond(x86_pmu_commit_scheduling)(cpuc, i, assign[i]);
- }
} else {
for (i = n0; i < n; i++) {
e = cpuc->event_list[i];
--
2.31.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* [tip: perf/core] perf/x86: remove unused assignment to pointer e
2021-08-04 11:57 [PATCH] perf/x86: remove unused assignment to pointer e Colin King
@ 2021-08-05 9:34 ` tip-bot2 for Colin Ian King
2021-08-26 8:09 ` [tip: perf/core] perf/x86: Remove unused assignment to pointer 'e' tip-bot2 for Colin Ian King
1 sibling, 0 replies; 3+ messages in thread
From: tip-bot2 for Colin Ian King @ 2021-08-05 9:34 UTC (permalink / raw)
To: linux-tip-commits
Cc: Colin Ian King, Peter Zijlstra (Intel), x86, linux-kernel
The following commit has been merged into the perf/core branch of tip:
Commit-ID: 79551ec0782895af27d6aa9b3abb6d547b7260d3
Gitweb: https://git.kernel.org/tip/79551ec0782895af27d6aa9b3abb6d547b7260d3
Author: Colin Ian King <colin.king@canonical.com>
AuthorDate: Wed, 04 Aug 2021 12:57:10 +01:00
Committer: Peter Zijlstra <peterz@infradead.org>
CommitterDate: Wed, 04 Aug 2021 15:16:39 +02:00
perf/x86: remove unused assignment to pointer e
The pointer e is being assigned a value that is never read, the assignment
is redundant and can be removed.
Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20210804115710.109608-1-colin.king@canonical.com
---
arch/x86/events/core.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch/x86/events/core.c b/arch/x86/events/core.c
index c0167d5..f4e5fa7 100644
--- a/arch/x86/events/core.c
+++ b/arch/x86/events/core.c
@@ -1085,10 +1085,8 @@ int x86_schedule_events(struct cpu_hw_events *cpuc, int n, int *assign)
* validate an event group (assign == NULL)
*/
if (!unsched && assign) {
- for (i = 0; i < n; i++) {
- e = cpuc->event_list[i];
+ for (i = 0; i < n; i++)
static_call_cond(x86_pmu_commit_scheduling)(cpuc, i, assign[i]);
- }
} else {
for (i = n0; i < n; i++) {
e = cpuc->event_list[i];
^ permalink raw reply [flat|nested] 3+ messages in thread
* [tip: perf/core] perf/x86: Remove unused assignment to pointer 'e'
2021-08-04 11:57 [PATCH] perf/x86: remove unused assignment to pointer e Colin King
2021-08-05 9:34 ` [tip: perf/core] " tip-bot2 for Colin Ian King
@ 2021-08-26 8:09 ` tip-bot2 for Colin Ian King
1 sibling, 0 replies; 3+ messages in thread
From: tip-bot2 for Colin Ian King @ 2021-08-26 8:09 UTC (permalink / raw)
To: linux-tip-commits
Cc: Colin Ian King, Peter Zijlstra (Intel), Ingo Molnar, x86, linux-kernel
The following commit has been merged into the perf/core branch of tip:
Commit-ID: 4f32da76a1401dcd088930f0ac8658425524368b
Gitweb: https://git.kernel.org/tip/4f32da76a1401dcd088930f0ac8658425524368b
Author: Colin Ian King <colin.king@canonical.com>
AuthorDate: Wed, 04 Aug 2021 12:57:10 +01:00
Committer: Ingo Molnar <mingo@kernel.org>
CommitterDate: Thu, 26 Aug 2021 09:14:36 +02:00
perf/x86: Remove unused assignment to pointer 'e'
The pointer 'e' is being assigned a value that is never read, the assignment
is redundant and can be removed.
Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lkml.kernel.org/r/20210804115710.109608-1-colin.king@canonical.com
---
arch/x86/events/core.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch/x86/events/core.c b/arch/x86/events/core.c
index 3092fbf..2a57dbe 100644
--- a/arch/x86/events/core.c
+++ b/arch/x86/events/core.c
@@ -1087,10 +1087,8 @@ int x86_schedule_events(struct cpu_hw_events *cpuc, int n, int *assign)
* validate an event group (assign == NULL)
*/
if (!unsched && assign) {
- for (i = 0; i < n; i++) {
- e = cpuc->event_list[i];
+ for (i = 0; i < n; i++)
static_call_cond(x86_pmu_commit_scheduling)(cpuc, i, assign[i]);
- }
} else {
for (i = n0; i < n; i++) {
e = cpuc->event_list[i];
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-08-26 8:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-04 11:57 [PATCH] perf/x86: remove unused assignment to pointer e Colin King
2021-08-05 9:34 ` [tip: perf/core] " tip-bot2 for Colin Ian King
2021-08-26 8:09 ` [tip: perf/core] perf/x86: Remove unused assignment to pointer 'e' tip-bot2 for Colin Ian King
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).