On Thu, Oct 27, 2011 at 05:22:48PM +0300, Jouni Malinen wrote: > On Wed, Oct 26, 2011 at 03:47:46PM +0200, Arik Nemtsov wrote: > > Initialize rate control algorithms only when supported rates are known > > for a TDLS peer sta. Direct Tx between peers is not allowed before the > > link is enabled. In turn, this only occurs after a change_station() > > call that sets supported rates. > > Thanks - this seems to fix the issues I saw previously. I tested ath9k > with this instead of my earlier patch and was able to run successfully > through my TDLS test scripts against another implementation. I merged this patch from Jouni: commit e3a4cc2f073739c9c9c2e97efc774703061f034a Author: Jouni Malinen <j@xxxxx> Date: Sun Oct 23 22:36:04 2011 +0300 mac80211: Fix TDLS support validation in add_station handler We need to verify whether the command is successful before allocating the station entry to avoid extra processing. This also fixes a memory leak on the error path. Signed-off-by: Jouni Malinen <j@xxxxx> Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx> I'm not sure what to do with this patch now. I think I will just drop it, and you can send me a new patch that applies on top of Jouni's (perhaps even reverting it) as appropriate? Thanks, John -- John W. Linville Someday the world will need a hero, and you linville@xxxxxxxxxxxxx might be all we have. Be ready. -- 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