Revert r2828 - problem appears to be fixed now
authormentor <mentor@0192ed92-7a03-0410-a25b-9323aeb14dbd>
Mon, 12 Nov 2007 19:20:11 +0000 (19:20 +0000)
committermentor <mentor@0192ed92-7a03-0410-a25b-9323aeb14dbd>
Mon, 12 Nov 2007 19:20:11 +0000 (19:20 +0000)
git-svn-id: http://madwifi-project.org/svn/madwifi/trunk@2845 0192ed92-7a03-0410-a25b-9323aeb14dbd

net80211/ieee80211_node.c

index 4ec0dc9bbcb528e290722a913724fc801b6e8ff6..9ad6ae653066a9f0536711e4623a105de28e8750 100644 (file)
@@ -738,21 +738,6 @@ static __inline void _node_table_join(struct ieee80211_node_table *nt, struct ie
 
        ni->ni_table = nt;
        tni = ieee80211_ref_node(ni);
-       /* MT: BEGIN HACK 
-         * XXX: r2792 fixed a bug where ieee80211_ref_node was being
-        * invoked five times on ni because it was passed to TAILQ_INSERT_TAIL
-         * which happens to evaluate it's value five times.  This fixed a problem
-         * where we were gaining four extra references, but unfortunately exposed
-         * other bugs which cause kernel panics.  I am working on a fix for this as part
-         * of ticket #1621.  In the meantime, adding these references back is 'wrong'
-         * but better than getting five new critical defects per day while debugging.
-         */
-       ieee80211_ref_node(ni);ieee80211_ref_node(ni); 
-       ieee80211_ref_node(ni);ieee80211_ref_node(ni); 
-       /*
-        * END HACK
-         */
-
        TAILQ_INSERT_TAIL(&nt->nt_node, tni, ni_list);
        tni = NULL;