On Thu, 2010-12-16 at 03:17 -0800, Paul Stewart wrote: > > I thought this was solved in ath9k differently? Was that somehow > > inadequate? > > Unfortunately previous attempts to solve this problem failed. I thought ath9k was fixed to simply go idle when stopped? Why didn't that help? > Here's a slightly better rationale, as I'm getting to understand the problem. > ieee80211_do_stop() decrements "open_count" so early during the function > that if the device is not idle at this point (e.g, due to scanning), > drv_config() > will never be called on the the lower-level driver, since ieee80211_hw_config() > tests for "open_count > 0" and silently returns no-error. Yes, I realise that -- but also Luis said this change wasn't sufficient in his tests, he also needed something in scan code itself. Also, this is really quite intentional if you read the comments around the area you're modifying -- mac80211 assumes that after it shuts down the device with stop(), the device will be powered off under control of the driver until start(). I just don't think it'll be easy to actually make sure that we always go IDLE before calling stop(). This patch might solve part of the problem, but what about hardware scan, that might have to be stopped by the driver during stop()? Similar things might come up with other features. Therefore I'm not sure we should even try. If we stop() the hardware, what's preventing the driver from doing whatever it needs to put the device into a low power state? johannes -- 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
