Merge "Use AP bssid for GetPacketCounters()" into oc-mr1-dev

Use AP bssid for GetPacketCounters()

we should use AP bssid instead of local interface mac address
for NL80211_CMD_GET_STATION. Local interface mac address only
works for some devices.

Bug: 66218880
Test: compile, unit tests
Test: run the following CTS test:
cts-tradefed run cts -m CtsNetTestCases -t
    android.net.wifi.cts.WifiManagerTest#testWifiWatchdog

Change-Id: I8852ab5085d186ee3f1e5b81abbf4c1d2500b9fa

Cherry-pick: Fail RSSI polling if wifi is not associated

Bug: 65025500
Test: compile, unit tests
Merged-In: Id4fe741d4a66329302cea0961d1bc17a35aa9382
Change-Id: Id4fe741d4a66329302cea0961d1bc17a35aa9382

Merge "Wificond: Remove mannual memory management in OffloadScanManager" into oc-mr1-dev

Count failures on fallback to netlink for pno metrics

Call OnPnoScanFailed callback if failed to fallback to netlink pno

Bug: 32842314
Test: runtests.sh
Change-Id: I6aab3c7d95ee0cfd7adc4f5519fcd80caab60759

Wificond: Remove mannual memory management in OffloadScanManager

Bug: 64853991
Test: Unit tests and on-device testing for finding APs in Offloaded PNO
      scans

Change-Id: I88fcbaed00627a7d282b6214f93419a761764da6

Wificond: Provide a way to enable Offload scans

Enable use of adb shell setprop <property_string> <value> command
to enable using offload HAL for disconnected mode PNO scans.

Bug: 64944391
Test: Unit test and on device to test to ensure the command works
Change-Id: I911a15177f98be6040d161093bf4c97089973b9c

Merge "Remove RTT related code from wificond" into oc-mr1-dev

Remove RTT related code from wificond

Bug: 64809697
Test: compile, unit tests
Change-Id: I4457dee48b7af6e4d2d709b636e4ddecbd8d68cd

Conditional compile flags not allowed in BoardConfig

The assumption that the Boardconfig.mk will export a flag that will
determine support for a feature is incorrect. Remove using of this
assumed flag from wificond.

Bug: 32842314
Test: Unit tests, On device test
Change-Id: I3aa33318a5e220a30f9950a7bb9488058f1421b9

Wificond ScannerImpl Unit testing Offload HAL

Wificond ScannerImpl interfaces with Offload HAL to invoke APIs to
perform disconnected mode PNO scans. This CL tests behavior of
ScannerImpl.

Bug: 32842314
Test: Unit test and VTS
Change-Id: I070d17c98ce90a9aeded80eac7135d02e36f0231

Wificond: Address bugs in handling Offload scans

This CL addresses the following issues in handling of Offload HAL scan
results from the Offload HAL service
- Make cached scan results member variable a pointer so that the memory
for the vector can be allocated on the heap
- When an Async Error occurs in the Offload HAL service, switch to
performing PNO scans over netlink instead of letting it decide again.
- OffloadScanUtils convertToNativeScanResults() now needs to take in a
pointer to the vector that will store the scan results for retrieval.
- Populate tsf field fo the scanResult so that it doesn't get filtered
out by the framework
- Add logging to scan stats

Bug: 63148974
Test: Unit tests, on-device testing for ensuring we connect to an
available access point from screen off disconnected mode.
Change-Id: Ida507d857faa8ea6dbee362cf0116f8ca858963f

Wificond: Create mocks of Offload Scan objects

Create objects for Offload HAL to enable testing of Scanner Impl

Test: Unit tests
Bug: 32842314
Change-Id: Iea881dbbef0f6dc53b87f8ea24acd3c3e347f902

Wifficond: Implement API to get PNO scan results

Implement getPnoScanResults() API to return scan results from the
interface that completed the scheduled PNO scans in disconnected mode
most recently.

Bug: 63148974
Test: Unit tests, sanity test and on-device tests
Change-Id: Ife9f890faa7cdbc34da83708f59eae7398e76109

Wificond: Make Offload HAL service nonblocking

Offload HAL getService() is a blocking call which can cause a deadlock.
Replace it with non blocking call tryGetService() which returns null if
the service is not running instead of waiting on the service to start.
This implies that the client needs to be able to get a reference to the
service at a later time. The design choice is to use tryGetService() API
to get the handle when a disconnected mode PNO scan is requested if this
was not available before.

Bug: 62807929
Test: Unit test, sanity test
Change-Id: I344fd93c4afe2381c356e626e83311af49a5ad49

Wificond: Async Errors from Offload  HAL service

Asynchronous errors from the Offload HAL service must be forwarded to
Wificond scanner implementation which can then switch over to the other
interfaces for completing disconnected PNO scans. This requires
specification of an interface from Offload HAL client to Wificond's
scanner implementation. Unit tetsts are updated accordingly.

Bug: 32842314
Test: Unit test and VTS
Change-Id: Ie56a52caf161c851d1945584a2bf93734008598a

Wificond: AIDL APIs for Offload HAL API metrics

Add new APIs to IPnoScanEvent AIDL itnerface that can be used to report
Offload HAL events for purposes of Offload  HAL scan metric collection
by framework

Bug: 32842314
Test: Unit tests, on-device test for connecting to a network found by
Offload HAL during PNO scans

Change-Id: Iae97389a35a728e3c0fc67f4f2c9937a8fa4acee

Wificond: Converting Offload scan results update

When converting scan results from Offload HAL to NativeScanResult
format, fix updating the BSSID field.

Bug: 63148974
Test: Unit test, sanity test - bring up the phone and connect with
access points, test disconnected mode PNO scans with Offload HAL as the
interface of choice

Change-Id: I60ae8b4e6422727c2cab2d2fca4c5a0730558035

Support scan plan attribute for scheduled scan
am: dfbe7e4e12

Change-Id: Ie95cfb7746db96a64dab898a55edccabb34d75cc

Retrieve scan plan capabilities from kernel
am: 514a184b8e

Change-Id: I542e1a15d38bbb34449d7d5c98b75fb04580ddee

Support scan plan attribute for scheduled scan

This adds the support of the scan plan attribute.
With kernel updates, driver can no longer diffrentiate interval
attribute from single scan plan attribute, so driver's pno scan
back off logic cannot be triggered in that case.
To solve the PNO back-off problem, this CL allows wificond to explicitly
specify scan plans (if the driver supports it). With explicit scan
plans, the driver will have the information it needs to back-off PNO
scans.

Bug: 63837760
Test: compile, unit tests
Test: Manually test that pno scan works on multiple devices
Test: Integration sanity test

Change-Id: Ic7e38caa482a2d72f45bf3c8177fa656e534db08

Retrieve scan plan capabilities from kernel

Bug: 63837760
Test: compile, unit test
Test: manually inserting logs to test it retrieves the correct
values.

Change-Id: I750bd287b4f1cb8df44b8d1367c69f7963a3d8dc

Do not crash on netlink manager start error

For some device(emulator) kernel netlink interface is not configured.
NetlinkManager.Start() will fail and crash wificond.
Wificond is included in base.mk so it is not feasible to remove
it from emulator image.
This patch changes the error level from FATAL to ERROR to avoid
crashing.

Bug: 63837784
Test: compile, manual test wificond works
Change-Id: I0cf3f6a1ea73e455f8362870e4c96c76b59dd41c

Merge "Use new netlink attribute for scan result timestamp" into oc-dr1-dev am: 30f40a8855
am: ce5cdd8492

Change-Id: Ifdea96308eed891801a42d6c282c7ac40e04aa61

Merge "Use new netlink attribute for scan result timestamp" into oc-dr1-dev
am: 30f40a8855

Change-Id: I595fa798b0602ed7365476d358dcfdfdff0d4993

Merge "Use new netlink attribute for scan result timestamp" into oc-dr1-dev

ScannerImpl: Add unit tests for scheduled scans am: 7a3d8ddff0
am: 62d5572e2f

Change-Id: I5e0710f98bce725c3101d763478893f94a902246

More unit tests for ScannerImpl am: c658fa0ae1
am: c13c2e23c9

Change-Id: I7754fa9a5030894355ded96e1c8fc14306e759c2

ScannerImpl: Add unit tests for scheduled scans
am: 7a3d8ddff0

Change-Id: I05a37a1fe3b942f9a70c4d95354a7ee10a3e5fb9

More unit tests for ScannerImpl
am: c658fa0ae1

Change-Id: I445180bbfe03a5328e021c5261107b4611453150

Use new netlink attribute for scan result timestamp

Bug: 37758502
Test: compile, manual test
Test: integration test
Test: manually test that on new and old devices, there should
be only a small portion of scan results filtered out.

Change-Id: I2cbcb6f4c5b58b6a56a746de97afe491a9e338ac

ScannerImpl: Add unit tests for scheduled scans

This patch adds unit tests for scheduled scans.
In order to do that, this also moves the isOffloadScanSupprted()
out of OffloadScanManager to OffloadServiceUtils. By injecting
MockOffloadServiceUtils to ScannerImpl, we can test either offload
supported or not supported cases.

Bug: 33452931
Test: compile, unit tests
Test: manual test that offload scan is not enabled by mistake.
This is done by checking manually inserted logs.

Change-Id: I6c78df78605321013676e5f4578026f21d953e3b

More unit tests for ScannerImpl

This adds unit tests for abortScan() and getScanResults().

Bug: 33452931
Test: compile, unit tests

Change-Id: I9920022c02ac18bdd0b1cd045682c63763361065

Merge "WifiCond: Setup polling on HW binder FD" into oc-dr1-dev am: b7a65e9a63
am: 8151c5232c

Change-Id: I5797739fb6c0f0b3b01b1592e1fc5a9a5ad4d062

Merge "WifiCond: Setup polling on HW binder FD" into oc-dr1-dev
am: b7a65e9a63

Change-Id: Idb9e7c9308b262c0ffccd56aef6baa3e4a87799a

Merge "WifiCond: Setup polling on HW binder FD" into oc-dr1-dev

Create unittest for ScannerImpl am: 3c023a0e38
am: 7f5e3669fe

Change-Id: I22496dbb846798b2a8a7690bd421c144d387af8d

Create unittest for ScannerImpl
am: 3c023a0e38

Change-Id: Ic38f5a5ac88d7b176a63f23a1f7351ee3a204307

WifiCond: Setup polling on HW binder FD

To keep wificond running in single threaded mode, set it up to poll on
file descriptor of HW Binder that will receive Offload HAL callback
events

Bug: 62807929
Test: Unit tests, sanity tests, ensure Wificond has only one thread
Change-Id: Id1e977d1a62fecd9b9432d7726aee2cb5f168574

Restart wificond on ENODEV scan failure am: 793978cb7f
am: 3715880b1d

Change-Id: I3168afd2d930f52cd603a46a199a327bbda958e9

Create unittest for ScannerImpl

This creates the unit test suite for class ScannerImpl and
add a few test cases for single scan.

Bug: 33452931
Test: compile, unit tests

Change-Id: Ib0584068aa886769b50860309224e670f3415fc8

Restart wificond on ENODEV scan failure
am: 793978cb7f

Change-Id: Iec003a60e0c3abf0ad3fd7909a1cc74d14555cec

Restart wificond on ENODEV scan failure

ENODEV scan failure implies that driver entered a bad state.
In order to recover wificond, we should treat it as an fatal
error. The crashing of wificond will trigger the restarting
of framework, bringing everything back to work.

Bug: 36894921
Test: compile, unit tests
Test: manual test:
Use "iw wlan0 del" to remove kernel interface.
Observe that wificond restarts and wifi continues to work.

Change-Id: I89f4403bcb570d7f413fa4cc9ca50378de428d3d

Merge changes from topic '7_CLS_topic' into oc-dr1-dev am: b8ad0fd508
am: cabe9d5cc5

Change-Id: I9424612fea0526024e0a8a28812cc66cc334a421

Wificond: Offload HAL APIs return status am: e10ec9e1f1
am: 04559d12e9

Change-Id: I966018daa0bc6803dc63c6e7ee746921931c4c77

Merge changes from topic '7_CLS_topic' into oc-dr1-dev
am: b8ad0fd508

Change-Id: I861f95ceea5affe5d55bfe0b28fcfc349155b37d

Wificond: Offload HAL APIs return status
am: e10ec9e1f1

Change-Id: I7221e41b93d2ad1e1521d868694474e4b5fe70b7

Merge changes from topic '7_CLS_topic' into oc-dr1-dev

* changes:
  Offload HAL Wificond Client: Handle binder death
  Wificond: Offload HAL APIs return status

Merge "Revert "Use new netlink attribute for scan result timestamp" am: ecb6bb205f am: f4a2b51c0e" into oc-dr1-dev-plus-aosp
am: 0636688388

Change-Id: I769405ce2cfb8b35c5b049dac92e054a4b6bb453

Revert "Use new netlink attribute for scan result timestamp" am: ecb6bb205f am: 5bb4bd8518
am: 16c35d2940

Change-Id: I131885d262e52e0e75b60652d7601cbc2e881147

Merge "Revert "Use new netlink attribute for scan result timestamp" am: ecb6bb205f am: f4a2b51c0e" into oc-dr1-dev-plus-aosp

Revert "Use new netlink attribute for scan result timestamp" am: ecb6bb205f
am: f4a2b51c0e

Change-Id: Iec20d99a06c30ff0d5660c110114de2b1171b403

Revert "Use new netlink attribute for scan result timestamp" am: ecb6bb205f
am: 5bb4bd8518

Change-Id: Ib6762dacd462964516b648a191360a7e81ddcd86

Revert "Use new netlink attribute for scan result timestamp"
am: ecb6bb205f

Change-Id: Id0ad9f2c0c2ab434e921893b8d2b383b389170eb

Revert "Use new netlink attribute for scan result timestamp"
am: ecb6bb205f

Change-Id: I1915833acd8ed41b2f21e43a954338b593f5da02

Revert "Use new netlink attribute for scan result timestamp"

This reverts commit 74db77ecd5314fa28ac2b78ccc3ca18788eeef60.

Bug: 62818212
Bug: 37758502

Change-Id: Icd93d4caffab92a5bf79c8d6aa8fe4dceca3ec06

Use new netlink attribute for scan result timestamp am: 74db77ecd5 am: 11b94d5983
am: dd3262305c

Change-Id: I91848562b85ac3e6fc571df5485fde64480c1b41

Use new netlink attribute for scan result timestamp am: 74db77ecd5 am: de349be439
am: 8d40c19351

Change-Id: I311cb94740ed4b295333d95eb6a6a065b15dbdc7

Use new netlink attribute for scan result timestamp am: 74db77ecd5
am: 11b94d5983

Change-Id: I126fe893ff999fb5acfd3c4ca8be86e2ce57ffc8

Use new netlink attribute for scan result timestamp am: 74db77ecd5
am: de349be439

Change-Id: I04a2307e43c75ffa66797c19da3b60dd79e4c66a

Use new netlink attribute for scan result timestamp
am: 74db77ecd5

Change-Id: Ie2264901d0389448f891152d7044c041ab5b4b93

Use new netlink attribute for scan result timestamp
am: 74db77ecd5

Change-Id: Ifb6ae8a81c40b940194f7608a2edb67de42a61da

Use new netlink attribute for scan result timestamp

This CL prioritize the new netlink attribute
NL80211_BSS_LAST_SEEN_BOOTTIME to the old NL80211_BSS_TSF
because if driver supports the new attribute, the new
attribute will carry more accurate data. If not, we still
can fall back to NL80211_BSS_TSF.

Bug: 37758502
Test: compile, unit tests, integration test
Test: manual tests:
1. Test devices which have new kernel/driver supporting this attribute.
Test that most APs still consistantly show up in scan result.
1. Test devices which don't support this attribute. Insert logs to test
that wificond can fall back to the old attributes.

Change-Id: I14226b77e8bf50f4e035679fff3313affb800e86

Offload HAL Wificond Client: Handle binder death

Binder death handling for Offload HAL and corresponding unit tests

Bug: 32842314
Test: Unit tests
Change-Id: I7eb52689070046ae8d2db15fd2dda80d3d398022

Wificond: Offload HAL APIs return status

Handle changes to the Offload HAL HIDL interface where the APIs now
return status of synchronous operations.

Bug: 32842314
Test: Unit Tests
Change-Id: If5cde06c3ad398774f33430ededda8f541540a31

Add OWNERS in system/connectivity/wificond am: 483bd8dcfd am: 1e593176ba am: 0ddca176d2
am: 9ac2756540

Change-Id: I11b77728134e65991d1768d6e98e213df2ce296f

Add OWNERS in system/connectivity/wificond am: 483bd8dcfd am: 1e593176ba
am: 0ddca176d2

Change-Id: I6337d77073e86923731b33ffed1a4eb38bbed7be

Add OWNERS in system/connectivity/wificond am: 483bd8dcfd
am: 1e593176ba

Change-Id: Ie0c2e1fce2187b0ca92dae53537efcc913638aad

Add OWNERS in system/connectivity/wificond
am: 483bd8dcfd

Change-Id: Id788cdfa3d08a0f2ca7cf2aaa79ea5f9cb98c062

Add OWNERS in system/connectivity/wificond

Owners are selected from top CL approvals or owners.
They will be suggested to review/approve future CLs.

Test: build/make/tools/checkowners.py -c -v OWNERS
Change-Id: Icce8798b3a57cac4180a50faed7ae811ab577a2c

Remove unused lambda captures

Test: build
Bug: 37752547
Change-Id: If39b0322aababd45868146fcde0c052e13acff85

Merge "Change abortScan() binder API to void" into oc-dev am: 7124c9f3f0
am: d96924d24c

Change-Id: I1fc5276a6e3a6907f8492785981cd3e333a92135

Merge "Change abortScan() binder API to void" into oc-dev
am: 7124c9f3f0

Change-Id: I5e518a611f34111c2d7db335d641ac8529049be9

Merge "Change abortScan() binder API to void" into oc-dev

Merge "Wificond: Offload HAL calling code formatting fixes"

Wificond: Offload HAL calling code formatting fixes

Clang formatting for all files introduced, Also move "using" to
source files from header files.

Bug: 32842314
Test: Unit tests
Change-Id: I913fced31568a61d4ae1a78ceeec5a43d300904b

Merge "WifiCond: Invoke OffloadScanManager APIs for PNO scans"

wificond should return hidden ssid scan results am: 2eee0d91bf
am: 420040b7cc

Change-Id: Ie0751d40ec1982c518b7ebfcdd307c3780272439

wificond should return hidden ssid scan results
am: 2eee0d91bf

Change-Id: Iee2410b74cab8c079387457c28c7d83d3197f77a

WifiCond: Invoke OffloadScanManager APIs for PNO scans

Offload Scan Manager APIs will be used if available. If scans over
Offload fail, scans will be performed over netlink as a fallback.

Bug: 32842314
Test: Unit tests
Change-Id: If48eb0bd1de38c29a2b3a7d461673b1e3502baa5

wificond should return hidden ssid scan results

Scan results with empty SSID might be useful for APPs.
Wificond should return them in the scan results list.

Bug: 38260073
Test: compile, unit tests, manual tests
Change-Id: If30338224a61109146c4cc6fbdbee1e70b23f962

Merge "WifiCond: getScanStats() API in OffloadSCanManager"

Change abortScan() binder API to void

Framework don't reley and can't rely on the success/failure
of abortScan(). In order to avoid confusion, this API should
be void instead returning bool.

Bug: 37743832
Test: compile, unit tests, manual test, integration test
Change-Id: I32075984b5e5f5be4822e9669076004a2f4a73c2

WifiCond: getScanStats() API in OffloadSCanManager

Introduce new API in OffloadScanManager getScanStats() to get scan stats
from Offload HAL. Created a parcelable NativeScanStats struct to export
this data and corresponding unit test coverage. Also included unit tests
to verify conversion from ScanStats in Offload HAL format to Wificond
format.

Test: Unit tests
Bug: 32842314
Change-Id: I82bec2cc42635588ef8cb3fd0ad3206242349645

Merge "Support abortScan" into oc-dev am: b1e5477375
am: 55784b05aa

Change-Id: I58af82cbd5dfe556c72e6347862fd942333d50e5

Merge "Support abortScan" into oc-dev
am: b1e5477375

Change-Id: Ide8aa03b1d380a2089e353ca7292ee58dfcefb41

Merge "Support abortScan" into oc-dev

Merge "Wificond: Introduce new APIS to OffloadScanManager"

Support abortScan

Support to abortScan instead of "ABORT_SCAN" command of
wpa_supplicant.
When STA scan triggered by wificond is ongoing, abortScan sends
NL80211_CMD_ABORT_SCAN to kernel to stop it.

Bug: 37227357
BUg: 37743832
Test: Manual test
Change-Id: I542088e7711bd7e32a22685397af8e7f3a8ccdf7

Link libwifi-system-iface for InterfaceTool

This commit adds libwifi-system-iface to LOCAL_SHARED_LIBRARIES so that
/system/bin/wificond and its tests can use InterfaceTool, which was
extracted from libwifi-system.so to a new library named
libwifi-system-iface.so.

Bug: 37429084
Test: Sailfish builds, boots, and wifi works.
Change-Id: I04ea9e0588c48e8ddca7f237d7061875435ad1a6

Wificond: Introduce new APIS to OffloadScanManager

The following APIs are now supported
- startScan() and stopScan()

Including unit tests to verify behavior of these APIs

Test: Unit tests
Bug: 32842314
Change-Id: I23ddd436139976b0aefcbd05b486c934b748e3c4

Revert "Support abortScan" am: e2cfd0257c am: 395d92eb80 am: 28f2c2c565
am: dfd0c5eda7

Change-Id: I9a9da2c629b26908d4a08de0199e13ba910f6e09

Merge "Add dump state for wificond interface classes" into oc-dev am: 0d676773e1
am: 9470002e4f

Change-Id: Ibecde3bc26cb43099d7d677f47fddfc21b448f02

Revert "Support abortScan" am: e2cfd0257c am: 395d92eb80
am: 28f2c2c565

Change-Id: I703d5e29a6852f0dee10e77ae7381265d94c504a

Revert "Support abortScan" am: e2cfd0257c
am: 395d92eb80

Change-Id: I77c0506b9e277e643784386fc9b44bd55dcc26dd

Revert "Support abortScan"
am: e2cfd0257c

Change-Id: I687f2de3d75badad774dfee5fe8d19a0a3124b7a

Merge "Add dump state for wificond interface classes" into oc-dev
am: 0d676773e1

Change-Id: I55d815f6c112d099f0a08645d146aa0559a22203

Merge "Add dump state for wificond interface classes" into oc-dev

Blacklist nan interfaces for wificond

Bug: 37644243
Test: manual tests
Change-Id: Ia08cbd50e67b9c0d4ee1b5837c81001e33049936

Revert "Support abortScan"

This reverts commit 77732670f190b8bebb496478c34f157b546bdbed.

Change-Id: I4f6304740fd6c4248ac7f56655bbc15ccf0a9dcb