On 16 July 2014 17:28, Ben Greear <greearb@xxxxxxxxxxxxxxx> wrote: > On 07/16/2014 06:21 AM, Michal Kazior wrote: >> >> ath10k doesn't apply any extra rx filters so >> there's no need to start monitor vdev for >> promiscuous mode. >> >> This fixes crashes with 4addr station interface >> bridging and some very rare crashes of AP >> interfaces with bridging as well. > > > Ahh, I was just working on some related hack-arounds to make sure > I left a vdev slot open for the monitor interface in case some poor > person started a sniffer on an interface and made it go promisc... > > With this patch, I can be sure monitor interfaces will not > be automatically created without explicit user request? I'm not sure if I understand you correctly, but: With this patch monitor vdev will be created and started only if there is at least one monitor interface up and running or CAC is in progress. Promiscuous mode does not change driver state at all. IOW if you start tcpdump on a wlan0 (station iftype) or add wlan0 (ap iftype, sta iftype) to a bridge then monitor vdev will not be created. You have to iw wlan0 interface add mon type monitor && ip link set mon up to start monitor vdev. From then on you can tcpdump or tcpdump -p on mon to sniff everything that the card sees on the air. > Any idea if it will be a problem to apply this to what is > effectively a 3.15 kernel? Hmm.. I guess it should apply. This depends on some earlier monitor clean up patches but I can't remember when those were merged. Michał -- 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
