mac80211: remove pointless mesh path timer RCU code
authorJohannes Berg <johannes.berg@intel.com>
Thu, 12 May 2011 13:03:32 +0000 (15:03 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 12 May 2011 18:10:52 +0000 (14:10 -0400)
The code here to RCU-dereference a pointer that's
on the stack is totally pointless, RCU isn't magic
(like say Java's weak references are), so the code
can't work like whoever wrote it thought it might.

Remove it so readers don't get confused. Note that
it seems that a bug is there anyway: I don't see
any code that cancels the timer when a mesh path
struct is destroyed.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/mesh_hwmp.c

index 849fecd0820e49dd0dcbbac89d1d2ee275edba6f..2aec7c4f357b53248ae47982d7b8bf420c2bd683 100644 (file)
@@ -966,20 +966,11 @@ endlookup:
 
 void mesh_path_timer(unsigned long data)
 {
-       struct ieee80211_sub_if_data *sdata;
-       struct mesh_path *mpath;
-
-       rcu_read_lock();
-       mpath = (struct mesh_path *) data;
-       mpath = rcu_dereference(mpath);
-       if (!mpath)
-               goto endmpathtimer;
-       sdata = mpath->sdata;
+       struct mesh_path *mpath = (void *) data;
+       struct ieee80211_sub_if_data *sdata = mpath->sdata;
 
-       if (sdata->local->quiescing) {
-               rcu_read_unlock();
+       if (sdata->local->quiescing)
                return;
-       }
 
        spin_lock_bh(&mpath->state_lock);
        if (mpath->flags & MESH_PATH_RESOLVED ||
@@ -996,8 +987,6 @@ void mesh_path_timer(unsigned long data)
        }
 
        spin_unlock_bh(&mpath->state_lock);
-endmpathtimer:
-       rcu_read_unlock();
 }
 
 void