Re: [PATCH] zd1211rw-mac80211: Fix for monitor mode bug

[Michael Buesch <mb@bu3sch.de>]

On Monday 26 March 2007 13:31, Johannes Berg wrote:
> On Mon, 2007-03-26 at 13:28 +0200, Michael Buesch wrote:
> 
> > We currently have two module parameters in bcm43xx-mac80211 for this:
> > 
> > static int modparam_mon_keep_bad;
> > module_param_named(mon_keep_bad, modparam_mon_keep_bad, int, 0444);
> > MODULE_PARM_DESC(mon_keep_bad, "Keep bad frames in monitor mode");
> > 
> > static int modparam_mon_keep_badplcp;
> > module_param_named(mon_keep_badplcp, modparam_mon_keep_bad, int, 0444);
> > MODULE_PARM_DESC(mon_keep_badplcp, "Keep frames with bad PLCP in monitor mode");
> 
> Out of curiosity, doesn't that break things when you have both a monitor
> and a non-monitor interface [1]? Or do you just disable it then?
> 
> johannes
> 
> [1] I think that mac80211 assumes CRC checks have passed, no?

Well, yeah. It does of course. If you have a STA and a MON interface,
the STA also receives the packs with bad CRCs. But that's how
mac80211 designed the virt-interface stuff.
Same goes for promisc. If you have a STA and a MON you can't have
the STA in non-promisc mode, so you also receive promisc packets on
the STA. That's how it's supposed to be. (I think mac80211 filters
them in the STA code somewhere).

The modparams actually are just a temporary hack and were never meant
to stay there. I wanted to have the knob in cfg80211, sooner or later.
So when the knob is there, mac80211 should take care that the STA
doesn't receive the bad packets.

-- 
Greetings Michael.