Reinette Chatre <reinette.chatre@xxxxxxxxx> writes: > From: Wey-Yi Guy <wey-yi.w.guy@xxxxxxxxx> > > The enabling of powersaving had a big impact on reliability and > performance. Throughput on n band suffered especially. With power save > support enabled we see significant ping response times, dropped frames, and > reduced throughput. These are captured in the following bug reports: > > http://bugzilla.intellinuxwireless.org/show_bug.cgi?id=2051 > http://bugzilla.intellinuxwireless.org/show_bug.cgi?id=2053 > > Disable power save by default, add "iwl_support_ps" module parameter > that can be used to enable it again. You are again creating driver specific parameters. But we should be moving away from this and instead use just generic interfaces. Please, think three times (or even more) whenever creating new driver specific interfaces, be it module parameters, sysfs files etc.. It's a maintenance nightmare and also very confusing for the users. We need to focus on nl80211 and make it work properly with all drivers. For the problem at hand, I see two options: 1. Users seeing the problem disable power save either via wext or with CONFIG_CFG80211_DEFAULT_PS and everyone else still can use power save. The issue will be investigated and fixed. If the AP is buggy, there isn't much we can do. 2. If you think the problem is widespread, remove IEEE80211_HW_SUPPORTS_PS from the driver, fix the issues and renable power save support. -- Kalle Valo -- 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
