diff options
author | Simon Wunderlich <simon@open-mesh.com> | 2015-11-30 17:34:01 +0100 |
---|---|---|
committer | Antonio Quartulli <a@unstable.cc> | 2016-01-13 19:26:25 +0800 |
commit | af63cf51b7f960aa73b32bac683cd4078f08fa0e (patch) | |
tree | 225c81e4b114a56f4b75f2a9d11b5d1046a04650 /net | |
parent | f1640c3ddeec12804bc9a21feee85fc15aca95f6 (diff) | |
download | linux-af63cf51b7f960aa73b32bac683cd4078f08fa0e.tar.gz linux-af63cf51b7f960aa73b32bac683cd4078f08fa0e.tar.bz2 linux-af63cf51b7f960aa73b32bac683cd4078f08fa0e.zip |
batman-adv: fix lockdep splat when doing mcast_free
While testing, we got something like this:
WARNING: CPU: 0 PID: 238 at net/batman-adv/multicast.c:142 batadv_mcast_mla_tt_retract+0x94/0x205 [batman_adv]()
[...]
Call Trace:
[<ffffffff815fc597>] dump_stack+0x4b/0x64
[<ffffffff810b34dc>] warn_slowpath_common+0xbc/0x120
[<ffffffffa0024ec5>] ? batadv_mcast_mla_tt_retract+0x94/0x205 [batman_adv]
[<ffffffff810b3705>] warn_slowpath_null+0x15/0x20
[<ffffffffa0024ec5>] batadv_mcast_mla_tt_retract+0x94/0x205 [batman_adv]
[<ffffffffa00273fe>] batadv_mcast_free+0x36/0x39 [batman_adv]
[<ffffffffa0020c77>] batadv_mesh_free+0x7d/0x13f [batman_adv]
[<ffffffffa0036a6b>] batadv_softif_free+0x15/0x25 [batman_adv]
[...]
Signed-off-by: Simon Wunderlich <simon@open-mesh.com>
Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
Signed-off-by: Antonio Quartulli <a@unstable.cc>
Diffstat (limited to 'net')
-rw-r--r-- | net/batman-adv/multicast.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/batman-adv/multicast.c b/net/batman-adv/multicast.c index eb76386f8d4b..75fa5013af72 100644 --- a/net/batman-adv/multicast.c +++ b/net/batman-adv/multicast.c @@ -802,7 +802,9 @@ void batadv_mcast_free(struct batadv_priv *bat_priv) batadv_tvlv_container_unregister(bat_priv, BATADV_TVLV_MCAST, 1); batadv_tvlv_handler_unregister(bat_priv, BATADV_TVLV_MCAST, 1); + spin_lock_bh(&bat_priv->tt.commit_lock); batadv_mcast_mla_tt_retract(bat_priv, NULL); + spin_unlock_bh(&bat_priv->tt.commit_lock); } /** |