On Sat, Jan 01, 2011 at 11:50:50AM +0200, Kalle Valo wrote: > Mohammed Shafi Shajakhan <mshajakhan@xxxxxxxxxxx> writes: > > * Preventing association with broken AP's > > How can different dtim value cause problems with association? Power > save should not be even enabled during association. Or do you mean > there are problems after association, for example during eap > negotation? It is not the DTIM value, it is the part of having to receive a Beacon frame before even trying to associate. This is very much a corner case, but well, that behavior did change with the commit. > > * Adds latency in roaming > > We should try to solve this problem differently than hardcoding dtim. > There are other ways to trigger roaming than just following beacons. > And if there's no data, there's no need to roam either. This is about the part of waiting for Beacon frame before trying to associate, not about hardcoding DTIM for any other purpose. > > So its better to always use the safe value of '1' for dtim period > > By hardcoding dtim you increase the power consumption in the case when > there is no data traffic, so it's not definitely better in all cases. > > I would prefer to fix the real issues instead of hardcoding dtim. And > if it's really essential to hardcode it now, please add a big fat > warning indicating that this is a temporary solution and should be > fixed properly without hardcoding anything. The real issue (as far as I can tell) was that ath9k was changed to require mac80211 to get it DTIM period before association. I'm not aware of any need for that with ath9k. It should have no problems updating PS parameters after association and as such, should not require mac80211 to figure DTIM period out before being able to associate. This change is not about hardcoding DTIM period; it is about removing unneeded change that added extra latency without any real gain. -- 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
