On Wed, 2010-10-06 at 12:00 +0200, Christian Lamparter wrote: > The timer itself is part of the station's private struct. > The clean-up routine will deactivate the timer as soon as > the station is removed. Therefore the extra sta->lock > protection should not be necessary. > rcu_read_lock(); > - spin_lock(&sta->lock); > ieee80211_release_reorder_timeout(sta, *ptid); > - spin_unlock(&sta->lock); > rcu_read_unlock(); There's a comment on ieee80211_release_reorder_timeout() saying that the lock must be held -- which is probably not true? We don't generally hold that lock on the RX path...? johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
