History log of /frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
23216ca333dc411e6ce0829f777ca29992388443 13-Jun-2017 Roshan Pius <rpius@google.com> WifiScanningService: Cache only results of full single scans

Change the single scan results caching logic inside WifiScanner:
1. Only cache scan results if they were for full scans. We've expanded
the meaning of "full scans" to include both 2g + 5g and 2g + 5g + dfs.
2. Filter out any scan results from the cache if they're over 3 minutes
old.

The motivation of this CL is to solve these problems:
1. Do not overwrite full scan results requested by an external app (ex.
Settings) with results from a partial scan
(ex. from WifiConnectivityManager).
This avoids the race condition where the settings app performs
a scan and gets a partial list of results back because it was ovewritten
by an internal partial scan request in between.
2. Scan results retrieved by apps will not contain stale scan
results. In N, wpa_supplicant evicted any scan results which were older
than 3 minutes from the cache. So, this CL restores that behavior.

Bug: 38212080
Test: Verified that both the settings and wifi wake is working properly
with the fix.
Test: Will send for full regression tests (b/62711875).
Test: New unit tests to verify that the partial scan results don't
override the cache.
Test: Modified existing unit tests to account for the change in
behavior. The broadcast (and cache) is now only updated on full scans.
Some of the tests were modified to produce full scan results, while the
other tests expectations were modified for partial scan results.

Change-Id: I4d6fb994d50995434f76218386272aa2562476fe
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
0bb87ee68b2e35f7f442c009d81263f63ba9a426 07-Jun-2017 Ningyuan Wang <nywang@google.com> Remove WTF log for WificondScannerImpl

This changes the log type from wtf to warning for
inconsistent scan event log because we already have
a fix for this.

Bug: 62019591
Test: compile
Change-Id: Ib0bd86f8d591905fc9d87c2de291cc3664bff7b7
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
e7ec898b26f04fca3390b2219965f365036b15ce 01-Jun-2017 Ningyuan Wang <nywang@google.com> Report single scan failure on inconsistent event

When mLastScanSettings is set for single scan but a
pno scan event comes, we should report single scan failure
event, so WifiSingleScanStateMachine won't stick in
ScanningState forever.

Bug: 62019591
Test: compile, unittests, manual tests, integration test
Change-Id: I96bcc3fedf1eeb47e662225724ead1713e1a7ce2
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
9dc9a8750ecd1ab25c5b4c7d17c8930ca2ffb6c3 23-May-2017 Ningyuan Wang <nywang@google.com> Rename ChannelHelper.getSupplicantScanFreqs()

We no longer use supplicant for scanning, so we need
to rename this helper function.
Since this helper function is not specifically for supplicant,
this CL just removes the 'Supplicant' part in its name.

This also fixes a few comments related to supplicant.

Bug: 36031295
Test: compile, unit tests, manual tests
Change-Id: Iabbf20e07b52ea015db227984511affaa055f205
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
2739f73dec628d2493e5447b71a83fe59c143fbd 11-May-2017 Ningyuan Wang <nywang@google.com> Do not cancel scan timeout alarm on PNO scan result

mScanTimeoutListener is only used for single scan timeout.
Canceling this time out alarm on PNO scan result may leave
WificondScannerImpl in the state where it is waiting for a
single scan result event forever.
This causes problems because it prevents processPendingScans()
to accept any new single scan requests.

Bug: 38139817
Test: compile, unit tests, manual tests, integration tests
Change-Id: I7a4ab04a7702986cd2de53f595dd958704cef95f
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
b47a0de674df76df087f0a0d642aea2603bbf6ab 27-Apr-2017 Ningyuan Wang <nywang@google.com> Remove networks from supplicant before starting PNO scan

We only start PNO scan when network is disconnected and screen
is off.
In the case we get disconnected from an AP, supplicant will keep
triggering scan and reconnect to that network.
This behavior is useful for immediate reconnection. However, when
PNO scan is on, the supplicant's single scan and PNO scan would
usually fail each other because they share the same radio resource.
This is causing power regression.
With this patch, we remove networks from supplicant so supplicant
will not keep scanning for reconnection when PNO scan is on.

Bug: 37744179
Test: unit tests, manual test, integration test

Change-Id: I03d4dea3b7f1acb51d716126f1057c13cd3ee9be
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
bb9016d1e5f7c2b27502e7d7db13b55460bca88b 24-Mar-2017 Michael Plass <mplass@google.com> Cherry-picl: Remove bssid hotlist from framework

Bug: 30147676
Test: Relevant unit tests pass, wifi functions
Change-Id: Iac6a4c3d4a9e842e73b6122218fe0842126e43e7
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
83e248aeeea818b3c4267c80fdd5d4eeb5197b63 28-Mar-2017 Ningyuan Wang <nywang@google.com> Remove temporary logging from WificondScannerImpl.java

Bug: 36276738
Test: compile, wifi works
Change-Id: I502463bf7f9bbcd3e899a39e5a1853b7003af482
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
51493e199ebf1e1191ccf652f0081d229018a11f 21-Mar-2017 Ningyuan Wang <nywang@google.com> Separate single scan and sched scan result handling

Bug: 36479602
Test: compile, unit tests, integration test

Change-Id: I066619ca7c53cf7e8e53ce776c1b419201d522f1
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
4bae3afd6041218b1eb2f0947d06c7839e6063e5 16-Mar-2017 Ningyuan Wang <nywang@google.com> Add temporary logs in WificondScannerImpl

Bug: 36276738
Test: compile, manual test
Change-Id: Ib94df5326ced497b7d32820d053763ceeda1800a
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java
05d51a2d8eb67ea2eaf4cae43280d1cf9fe92645 02-Mar-2017 Ningyuan Wang <nywang@google.com> Rename SupplicantWifiScannerImpl to WificondScannerImpl

This also fixes a few formatting issues.

Bug: 35707137
Test: compile, unit tests, integration test

Change-Id: I1b6642b723c5676620d65136cc6c773b49e28559
/frameworks/opt/net/wifi/service/java/com/android/server/wifi/scanner/WificondScannerImpl.java