lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 16 Apr 2012 10:41:06 -0700 From: Ben Greear <greearb@...delatech.com> To: Johannes Berg <johannes@...solutions.net> CC: linux-wireless@...r.kernel.org, netdev@...r.kernel.org Subject: Re: [PATCH v3] mac80211: Support on-channel scan option. On 04/16/2012 10:33 AM, Johannes Berg wrote: > Did you test it? Yes. I added my old can-scan-one patch on top of this so that as soon as one station is associated the rest of the hostap-generated scans can just scan the current channel. I then bring up 100 or so virtual stations and they all scan and do their thing... I also tried some manual scans on a fixed frequency. Those appeared to work, though I got scan results for APs on different channels. Does that seem right? I can't see how my patch would affect that sort of thing one way or another... If you have some suggestions for other ways to test this please let me know. >> + /* If we are scanning only on the current channel, then >> + * we do not need to stop normal activities >> + */ >> + unsigned long next_delay; >> + >> + __set_bit(SCAN_ONCHANNEL_SCANNING,&local->scanning); >> + ieee80211_recalc_idle(local); >> + /* Notify driver scan is starting, keep order of operations >> + * same as normal software scan, in case that matters. */ >> + drv_sw_scan_start(local); >> + ieee80211_configure_filter(local); /* accept probe-responses */ >> + /* We need to ensure power level is at max for scanning. */ >> + ieee80211_hw_config(local, 0); > > a few blank lines would be nice :) Sure, will do. >> + if ((req->channels[0]->flags& >> + IEEE80211_CHAN_PASSIVE_SCAN) || >> + !local->scan_req->n_ssids) { > > That formatting is a bit off it seems. Or is that just my mailer? Well, it looked OK to me..though when I replied my mailer certainly made it look funky. I'll re-run it through checkpatch just in case. Thanks, Ben -- Ben Greear <greearb@...delatech.com> Candela Technologies Inc http://www.candelatech.com -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists