mac80211: fix propagation of failed hardware reconfigurations
authorLuis R. Rodriguez <lrodriguez@atheros.com>
Thu, 24 Dec 2009 20:38:22 +0000 (15:38 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 6 Jan 2010 23:04:04 +0000 (15:04 -0800)
commit330b9373f9b62fcad200f470be7f64c99322d7ef
tree00d5861c7cacaf5258a99ebc9986132edb412aee
parent38cf2a039fec0233c1ab81b09125e56311574016
mac80211: fix propagation of failed hardware reconfigurations

commit 24feda0084722189468a65e20019cdd8ef99702b upstream.

mac80211 does not propagate failed hardware reconfiguration
requests. For suspend and resume this is important due to all
the possible issues that can come out of the suspend <-> resume
cycle. Not propagating the error means cfg80211 will assume
the resume for the device went through fine and mac80211 will
continue on trying to poke at the hardware, enable timers,
queue work, and so on for a device which is completley
unfunctional.

The least we can do is to propagate device start issues and
warn when this occurs upon resume. A side effect of this patch
is we also now propagate the start errors upon harware
reconfigurations (non-suspend), but this should also be desirable
anyway, there is not point in continuing to reconfigure a
device if mac80211 was unable to start the device.

For further details refer to the thread:

http://marc.info/?t=126151038700001&r=1&w=2

Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
net/mac80211/util.c