[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20221030111759.95092-1-wuchi.zero@gmail.com>
Date: Sun, 30 Oct 2022 19:17:59 +0800
From: wuchi <wuchi.zero@...il.com>
To: peterz@...radead.org, mingo@...hat.com, will@...nel.org,
longman@...hat.com, boqun.feng@...il.com
Cc: linux-kernel@...r.kernel.org
Subject: [PATCH] locking/lockdep: Remove unused variable in __lockdep_count*()
The target_entry variable will never be referenced because the
function noop_count() always returns false in __bfs().
So just remove that.
Signed-off-by: wuchi <wuchi.zero@...il.com>
---
kernel/locking/lockdep.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/kernel/locking/lockdep.c b/kernel/locking/lockdep.c
index e3375bc40dad..04d2ce5d0215 100644
--- a/kernel/locking/lockdep.c
+++ b/kernel/locking/lockdep.c
@@ -2075,9 +2075,8 @@ static bool noop_count(struct lock_list *entry, void *data)
static unsigned long __lockdep_count_forward_deps(struct lock_list *this)
{
unsigned long count = 0;
- struct lock_list *target_entry;
- __bfs_forwards(this, (void *)&count, noop_count, NULL, &target_entry);
+ __bfs_forwards(this, (void *)&count, noop_count, NULL, NULL);
return count;
}
@@ -2100,9 +2099,8 @@ unsigned long lockdep_count_forward_deps(struct lock_class *class)
static unsigned long __lockdep_count_backward_deps(struct lock_list *this)
{
unsigned long count = 0;
- struct lock_list *target_entry;
- __bfs_backwards(this, (void *)&count, noop_count, NULL, &target_entry);
+ __bfs_backwards(this, (void *)&count, noop_count, NULL, NULL);
return count;
}
--
2.20.1
Powered by blists - more mailing lists