diff options
| author | WANG Cong <xiyou.wangcong@gmail.com> | 2017-03-16 18:32:42 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-03-17 05:29:20 +0100 |
| commit | d12c917691b45d9dffcfe7c2362d25caa40905fd (patch) | |
| tree | f2e7a8d5f346f39bc220147fcfc5b3a70c9f8999 /net/bridge/br_private.h | |
| parent | rxrpc: Ignore BUSY packets on old calls (diff) | |
| download | linux-d12c917691b45d9dffcfe7c2362d25caa40905fd.tar.xz linux-d12c917691b45d9dffcfe7c2362d25caa40905fd.zip | |
bridge: resolve a false alarm of lockdep
Andrei reported a false alarm of lockdep at net/bridge/br_fdb.c:109,
this is because in Andrei's case, a spin_bug() was already triggered
before this, therefore the debug_locks is turned off, lockdep_is_held()
is no longer accurate after that. We should use lockdep_assert_held_once()
instead of lockdep_is_held() to respect debug_locks.
Fixes: 410b3d48f5111 ("bridge: fdb: add proper lock checks in searching functions")
Reported-by: Andrei Vagin <avagin@gmail.com>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
Acked-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bridge/br_private.h')
| -rw-r--r-- | net/bridge/br_private.h | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h index 2288fca7756c..61368186edea 100644 --- a/net/bridge/br_private.h +++ b/net/bridge/br_private.h @@ -531,15 +531,6 @@ int br_fdb_external_learn_add(struct net_bridge *br, struct net_bridge_port *p, int br_fdb_external_learn_del(struct net_bridge *br, struct net_bridge_port *p, const unsigned char *addr, u16 vid); -static inline bool br_hash_lock_held(struct net_bridge *br) -{ -#ifdef CONFIG_LOCKDEP - return lockdep_is_held(&br->hash_lock); -#else - return true; -#endif -} - /* br_forward.c */ enum br_pkt_type { BR_PKT_UNICAST, |