linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] cfg80211/mac80211: implementation of scheduled scan
@ 2011-05-05 13:00 Luciano Coelho
  2011-05-05 13:00 ` [PATCH 1/3] cfg80211/nl80211: add support for scheduled scans Luciano Coelho
                   ` (2 more replies)
  0 siblings, 3 replies; 23+ messages in thread
From: Luciano Coelho @ 2011-05-05 13:00 UTC (permalink / raw)
  To: linux-wireless; +Cc: johannes, linville

Hi,

Here's another version of my scheduled scan patches.  I think these are now
good enough.  I applied all comments by Johannes and added some improvements.
These are the main differences from RFCv2:

mac80211:
---------
* mac80211 checks if the driver has an op->sched_scan_start and sets
  the WIPHY_FLAG_SUPPORTS_SCHED_SCAN flag accordingly;
* fixed a typo;
* moved drv_sched_scan_results to api_sched_scan_results;
* removed SCAN_SCHED_SCANNING as a scan_attribute and moved it to
  a separate local boolean, so that it doesn't interfere with
  normal scan scenarios;
* there was on comment that the sched_scan was "eating" beacons for
  the whole time when it was scanning, but this doesn't apply anymore
  because of commit b23b025fe246f3acc2988eb6d400df34c27cb8ae changed
  the code so that beacons are passed up if they're on the operating
  channel (see ieee80211_scan_rx());
* fixed allocation failure bug when allocating sched_scan_ies;
* removed lock in ieee80211_sched_scan_results();
* don't return a value in sched_scan_stop;

cfg80211:
---------
* fixed documentation in nl80211 so that it's clear that if an empty
  SSID is passed (broadcast), we do broadcast probe_reqs and if no
  SSID is passed, we do a passive scan;
* removed max_sched_scan_ssids, now we use max_scan_ssids instead;
* added __cfg80211_stop_sched_scan() and call that from
  nl80211_stop_sched_scan(), wdev_cleanup_work() and
  NETDEV_GOING_DOWN;
* removed unnecessary dev_hold() in nl80211_start_sched_scan();
* added the NL80211_CMD_SCHED_SCAN_STOPPED event and an interface for
  the driver to stop the scan by itself;
* send a NL80211_CMD_SCHED_SCAN_STOPPED event instead of a
  NL80211_CMD_STOP_SCHED_SCAN when the scan is stopped by a
  NL80211_CMD_STOP_SCHED_SCAN command;
* don't return a value in sched_scan_stop;
* added a new patch to include the scan interval;
* a few minor cleanups.

There are still more things on my TODO list, but they're mostly improvements
and it will be much easier to add them step-by-step after this part is
included.  This version is already functional and ready to be used. ;)

Please review and let me know if there are still some problems for inclusion.

Cheers,
Luca.

Luciano Coelho (3):
  cfg80211/nl80211: add support for scheduled scans
  mac80211: add support for HW scheduled scan
  cfg80211/nl80211: add interval attribute for scheduled scans

 include/linux/nl80211.h     |   31 +++++
 include/net/cfg80211.h      |   59 ++++++++++
 include/net/mac80211.h      |   50 ++++++++
 net/mac80211/cfg.c          |   27 +++++
 net/mac80211/driver-ops.h   |   27 +++++
 net/mac80211/driver-trace.h |   81 +++++++++++++
 net/mac80211/ieee80211_i.h  |    9 ++
 net/mac80211/main.c         |    8 +-
 net/mac80211/rx.c           |    6 +-
 net/mac80211/scan.c         |   99 ++++++++++++++++
 net/wireless/core.c         |   12 ++-
 net/wireless/core.h         |    7 +
 net/wireless/nl80211.c      |  261 +++++++++++++++++++++++++++++++++++++++++++
 net/wireless/nl80211.h      |    4 +
 net/wireless/scan.c         |   62 ++++++++++
 15 files changed, 738 insertions(+), 5 deletions(-)


^ permalink raw reply	[flat|nested] 23+ messages in thread

end of thread, other threads:[~2011-05-09 13:59 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-05 13:00 [PATCH 0/3] cfg80211/mac80211: implementation of scheduled scan Luciano Coelho
2011-05-05 13:00 ` [PATCH 1/3] cfg80211/nl80211: add support for scheduled scans Luciano Coelho
2011-05-09  8:12   ` Johannes Berg
2011-05-09  8:17     ` Luciano Coelho
2011-05-09  8:23       ` Johannes Berg
2011-05-09  8:25         ` Luciano Coelho
2011-05-09  9:50   ` Eliad Peller
2011-05-09 13:59     ` Luciano Coelho
2011-05-05 13:00 ` [PATCH 2/3] mac80211: add support for HW scheduled scan Luciano Coelho
2011-05-09  8:22   ` Johannes Berg
2011-05-09  8:22     ` Johannes Berg
2011-05-09 13:38     ` Luciano Coelho
2011-05-05 13:00 ` [PATCH 3/3] cfg80211/nl80211: add interval attribute for scheduled scans Luciano Coelho
2011-05-05 14:43   ` Ben Greear
2011-05-05 14:51     ` Johannes Berg
2011-05-05 14:50       ` Ben Greear
2011-05-05 14:58         ` Johannes Berg
2011-05-05 14:59           ` Johannes Berg
2011-05-05 15:48             ` Luciano Coelho
2011-05-05 15:51               ` Luciano Coelho
2011-05-05 16:13                 ` Johannes Berg
2011-05-05 16:22                   ` Ben Greear
2011-05-05 16:29                     ` Luciano Coelho

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).