On Tue, Feb 08, 2011 at 10:22:39PM +0100, BjÃrn Smedman wrote: > On Mon, Feb 7, 2011 at 4:38 PM, Mohammed Shafi Shajakhan > <mshajakhan@xxxxxxxxxxx> wrote: > > - Â Â Â Â* Some times we dont parse dtim period from mac80211, in that case > > - Â Â Â Â* use a default value > > + Â Â Â Â* We don't parse dtim period from mac80211 during the driver > > + Â Â Â Â* initialization as it breaks association with hidden-ssid > > + Â Â Â Â* AP and it causes latency in roaming > I realize I'm not the perfect representative for the intended audience > but I don't understand that comment at all. The previous one made > sense to me, but the new one seems to refer to some logic somewhere > else. In ath_beacon_config() I expect AP/IBSS logic but the comment > seems to be about some special case for a STA vif associating with a > hidden access point, no? I don't understand that special case either > but that is more easily attributable to ignorance. :) While I agree that the proposed comment is quite confusing, this function is actually used even in station mode to set beacon timers for when to wake up for receiving beacon frames, etc. However, I don't see why this comment should really be here. In addition, this should really be fixed by updating DTIM period after the first Beacon frame is received. Forcing a wait for that Beacon frame is the one that causes the extra latency; I don't remember why exactly this would break hidden SSID case unless there is another bug somewhere. ath9k does not need to know the DTIM period before association and the first call to this function may need to handle the 0 -> 1 safety check. This should then be followed by another call (likely new code in mac80211 needed for this) to update the timers after the DTIM period has been learned post-association. -- Jouni Malinen PGP id EFC895FA -- 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
