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. - 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
