Arik Nemtsov <arik@xxxxxxxxxx> writes: > On Wed, May 23, 2012 at 1:59 PM, Kalle Valo <kvalo@xxxxxxxxxx> wrote: >>> We want to change this in wlcore. We've encountered compatibility >>> problems with some APs when we set ACs other than VO to U-APSD mode. >> >> Sounds like a hack to do this in the driver. Why can't mac80211 do it? > > Well it may be an issue specific to our HW. We didn't want to change > the default behavior of all other mac80211 drivers. I would prefer to consider all drivers first. But as you are not even describing the problem and which APs are broken it's difficult for anyone comment. Being verbose in the commit log doesn't hurt, quite the opposite. > I'm not sure what you mean by "hack" here. Wikipedia has a good description: "a kludge (or often a "hack") is a solution to a problem, doing a task, or fixing a system that is inefficient, inelegant, or even unfathomable, but which nevertheless (more or less) works." I'm sure the solution you propose works but it's not elegant. > We just enable the driver to override a default value, which is used > by mac80211. You are moving logic from mac80211 to the driver but it should be the opposite, the driver should be dumb and mac80211 should control everything. Didn't we have a similar discussion last year when talking about controlling dynamic power save? I would have expected to see a user space interface for controlling these parameters, but not a driver interface. -- 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
