diff options
author | Sebastian Andrzej Siewior <[email protected]> | 2020-08-06 23:19:09 -0700 |
---|---|---|
committer | Linus Torvalds <[email protected]> | 2020-08-07 11:33:23 -0700 |
commit | b3cb9fc3aeaf9d52967c87d54a98f035e96279dc (patch) | |
tree | c503cbb2063ed9d78211856bcf6fb16d8f6d2fcf | |
parent | e42f174e43e47b623d9dbf814521c4961000c962 (diff) |
mm/slub.c: drop lockdep_assert_held() from put_map()
There is no point in using lockdep_assert_held() unlock that is about to
be unlocked. It works only with lockdep and lockdep will complain if
spin_unlock() is used on a lock that has not been locked.
Remove superfluous lockdep_assert_held().
Signed-off-by: Sebastian Andrzej Siewior <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Cc: Yu Zhao <[email protected]>
Cc: Christopher Lameter <[email protected]>
Cc: Pekka Enberg <[email protected]>
Cc: David Rientjes <[email protected]>
Cc: Joonsoo Kim <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Linus Torvalds <[email protected]>
-rw-r--r-- | mm/slub.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/mm/slub.c b/mm/slub.c index 629a58b56728..e5fc31c37b37 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -473,8 +473,6 @@ static unsigned long *get_map(struct kmem_cache *s, struct page *page) static void put_map(unsigned long *map) __releases(&object_map_lock) { VM_BUG_ON(map != object_map); - lockdep_assert_held(&object_map_lock); - spin_unlock(&object_map_lock); } |