Search Linux Wireless

Re: [PATCH] zd1211rw-mac80211: debug output for mac80211 ops

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 31 Jul 2007 23:37:01 +0200, Michael Buesch wrote:
> On Tuesday 31 July 2007 23:00:43 Jiri Benc wrote:
> > It depends on the IEEE80211_HW_MONITOR_DURING_OPER flag in hw.flags. If you
> > set the flag in the driver, then monitor interfaces work in the exactly
> > same way as STA or IBSS interfaces - the add_interface callback is called
> > (with conf.type equal to IEEE80211_IF_TYPE_MNTR) whenever user brings up a
> > monitor interface. You are responsible for switching the card to a monitor
> > mode (e.g. turning off hardware packet filtering).
> 
> I think you don't want to completely turn off packet filtering
> (promisc) if a monitor interface is present. The promisc bit is to be
> honoured seperately.
> The only thing we do in bcm43xx is enable passing of ctl frames (ACKs, etc),
> if we have a monitor interface.

Hm, yes, you're right. I didn't realize that, thanks!

> > As Michael correctly said, don't care about the value, just store it and
> > hand it back to the stack when the stack wants it.
> 
> That might raise the question, if it's required to store if_ids of monitor
> interfaces. I think it's not, as you don't need any mac80211 callback,
> that requires an if_id, for a monitor interface.

Sure. If you don't need to call any callback which requires if_id (and
currently there is no such callback you can call for a monitor interface),
there is no point in storing the value. But it's also no harm.

Thanks,

 Jiri

-- 
Jiri Benc
SUSE Labs
-
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

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux