On Thu, Jan 14, 2016 at 12:07:12AM +0800, Eli Cooper wrote:
> Since commit df1404650ccbfeb76a84f301f22316be0d00a864 monitor mode for rt2x00
> has been made effectively useless because the hardware filter is configured to
> drop packets whose intended recipient is not the device, regardless of the
> presence of monitor mode interfaces.
>
> This patch fixes this regression by adding explicit monitor mode support, and
> configuring the hardware filter accordingly.
>
> Signed-off-by: Eli Cooper <elicooper@xxxxxxx>
[snip]
> unsigned int intf_associated;
> unsigned int intf_beaconing;
> + bool is_monitoring;
Could you change to use rt2x00dev->flags instead of new bool
variable i.e. introduce CONFIG_MONITORING in rt2x00_state_flags ?
> + if (ieee80211_flags & IEEE80211_CONF_CHANGE_MONITOR) {
> + if (conf->flags & IEEE80211_CONF_MONITOR) {
> + rt2x00_dbg(rt2x00dev, "Monitor mode is enabled\n");
I don't think those debug messages are actually needed.
Thanks
Stanislaw
--
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
