| Message ID | 20200618201234.795692-2-bigeasy@linutronix.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/2] slub: Cure list_slab_objects() from double fix | expand |
diff --git a/mm/slub.c b/mm/slub.c index 72195cafbb503..5a43ad225427f 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -469,8 +469,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); }
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(). Cc: Yu Zhao <yuzhao@google.com> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> --- mm/slub.c | 2 -- 1 file changed, 2 deletions(-)