release-request-05263112-375a-4b1f-a657-a14bb2a5c5a3-for-git_oc-mr1-release-4185249 snap-temp-L63000000082739046

Change-Id: I586e6d1e84d64c56e3ff5f7baf84d51d73fb9f16

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

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

release-request-5ee66a34-3e58-4de0-ba73-1f49074ee497-for-git_oc-mr1-release-4169335 snap-temp-L20600000081126177

Change-Id: I970aeaed65f18fc5311cbbff4a0325eaf8f25b86

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

release-request-d9dc98f7-19b2-484c-b4d1-f35dc43e9c05-for-git_oc-mr1-release-4152006 snap-temp-L91700000079405440

Change-Id: I31e5790ede6e7d8dc9a92ca18d9f704253b49c87

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

release-request-6db00e04-e62d-4e43-807a-a9cc1185dcbb-for-git_oc-mr1-release-4136929 snap-temp-L91700000077883673

Change-Id: Ic766ac3c45530254eeffec92269572fa32844e5d

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

release-request-7bfcab52-d1c0-4256-9d6b-5b5092bc78ca-for-git_oc-mr1-release-4133428 snap-temp-L95800000077479875

Change-Id: Ica00a413fa48609320a3ecd8ef0ab47029b1055a

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

release-request-d1418208-3b0b-4ef9-a2f0-a8f8ac6c24e8-for-git_oc-mr1-release-4120176 snap-temp-L55000000076156472

Change-Id: I2fa90a0ce92d88e766fc5775a0174118fe100a91

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

release-request-c9e3b153-009d-4386-b83e-337752602795-for-git_oc-mr1-release-4111654 snap-temp-L22900000075285650

Change-Id: I69f25d4490e9325d8d2169aae6fa320571025fe2

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

release-request-703268f3-975f-4a98-83d7-cbf58019eda0-for-git_oc-mr1-release-4067152 snap-temp-L75700000070444922

Change-Id: I72cbbd723c5ac108f9c7ac0ac7042eeec6f49a27

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

release-request-e81235f1-b2b4-4de1-b974-255760176f06-for-git_oc-mr1-release-4040405 snap-temp-L68200000067456736

Change-Id: Ic821254407a8fdef3ad9a4e6de2832d270b7ba62

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

release-request-91ef8626-1af7-4595-b2bd-2d85ee97e99f-for-git_oc-mr1-release-4029936 snap-temp-L27200000066301195

Change-Id: Ic7a4100f8151ea898e9ba85f2a11d28bfee6664c

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

release-request-323db86e-b638-4d24-8eb1-d2e3bf4a9d1a-for-git_oc-mr1-release-4017779 snap-temp-L47900000064949209

Change-Id: Ie6fa7fc3062460fdc8c420d4059f1dbd8351c3b1

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

Support abortScan am: 77732670f1 am: 36889eb2d7 am: 646644f254
am: e32024f6f9

Change-Id: I834720124d1cd167eda0ecc0023c37c435c4ffb0

Support abortScan am: 77732670f1 am: 36889eb2d7
am: 646644f254

Change-Id: I1679424b82608ca87679e29cfebdaacbb12d42a4

Support abortScan am: 77732670f1
am: 36889eb2d7

Change-Id: Ib1fdb3d216feb49c95b4f5acc89c2f97bb1ed7e5

Support abortScan
am: 77732670f1

Change-Id: Ia7d557ef77ca0636a187028fe5b7073e2e8f693c

Remove unused declaration in ipc_constants.h am: 6b9d3875d9
am: c381520ba9

Change-Id: Iac116da90918c1be19734a1f30fc52dcc499abf2

Remove unused declaration in ipc_constants.h
am: 6b9d3875d9

Change-Id: I26ceacb7d388d6c5d1e14a53c90405213b0648a8

Add dump state for wificond interface classes

This allows wificond's ClientInterfaceImpl and
ApInterfaceImpl to dump state.

The dump state includes scan capabilities, wiphy features, and
number of current associated stations for AP mode.

This also adds the missing const keyword for IsAssociated().

Bug: 31336376
Test: compile, manual test

Sample dump output:
Current wiphy index: 0
Cached interfaces list from kernel message:
Interface index: 23, name: p2p0, mac address: 66:bc:0c:47:df:00
Interface index: 22, name: wlan0, mac address: 64:bc:0c:47:df:00
------- Dump of client interface with index: 22 and name: wlan0-------
Max number of ssids for single shot scan: 10
Max number of ssids for scheduled scan: 16
Max number of match sets for scheduled scan: 16
Device supports random MAC for single shot scan: 1
Device supports random MAC for scheduled scan: 1
------- Dump End -------
--------- 0.004s was the duration of dumpsys wificond

Change-Id: I862974af743ca6ca303dfc1a4f6c773dd5b7a03e

Remove unused declaration in ipc_constants.h

Bug: 37517107
Test: compile
Change-Id: I00b3dc6da18e732611ab338843ca5c169e8b90a0

Wificond: Handle Error callback from Offload HAL

Client code changes to handle error can be reported by the Offload HAL
service and corresponding unit tests.

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

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
Test: Manual test

Change-Id: Ifc4b81de8ec1d75467edf497abfe348c4ce1b283

wificond: Dump interface list am: 099ee0c764
am: c1dca4b627

Change-Id: Idb122ca837389bc336f7d42360ce4f1ce5a69198

Allow wificond to dump logs am: b340bb5604
am: c576bbd5bf

Change-Id: I0cf4821cf68fd4f8b4d81501cb14675d60ba2816

wificond: Dump interface list
am: 099ee0c764

Change-Id: Id39b973e1e4de76d4a04877850939a62d2155518

Allow wificond to dump logs
am: b340bb5604

Change-Id: Iaac2866ca7c67d9a89e91a33ac34aa9503ac1fcb

wificond: Dump interface list

Bug: 31336376
Test: compile
Test: manual test, check bug report
For example, I can see the following dump state:
Cached interfaces list from kernel message:
Interface index: 23, name: p2p0, mac address: 66:bc:0c:47:df:00
Interface index: 22, name: wlan0, mac address: 64:bc:0c:47:df:00

Change-Id: Iae02bc85a73f211aa39fb16d77eb17d399651dce

Allow wificond to dump logs

This adds the dump() function for wificond to dump logs
to provided fd.

Bug: 31336376
Test: compile
Test: manual test, see wificond dump logs in bug report

Change-Id: Ia27cee9c14728095d9843dfc08994ec981283ee3