Freeze HALs for Android Q

Bug: 119518846
Test: none
Change-Id: Ia036f9da6dcb1380e5117fa7e90e5d95257fc6ec

system_suspend: update OWNERS

Bug: n/a
Test: n/a
Change-Id: I725607ee0b94c8a549ef5b0ac842abfa28ad416e

system_suspend: add missing types.hal dependency

Bug: n/a
Test: m android.system.suspend@1.0
Change-Id: If98be39f29e7cea740c413ece582e854cd6ef0c3

Migrating suspend functionality to SuspendControlService

Move most API methods (other than acquireWakeLock) from SystemSuspend to
SuspendControlService for interactions that happend solely between
suspend HAL and the system server. This allows us to tighten security
SEPolicy around methods that only system-server should be using.

Bug: 121210355
Test: SystemSuspendV1_0UnitTest
Change-Id: Ia92c42ecf069d606916854430e81f2e232d788c7
Merged-In: Ia92c42ecf069d606916854430e81f2e232d788c7

system_suspend: add TEST_MAPPING

Bug: n/a
Test: presubmit
Change-Id: I5e708c03fd649a336703a732e6a9cecfca28a1de

Add VTS for Wifi Keystore HAL's getPublicKey()

This exercises paths through the Wifi Keystore HAL's
getPublicKey() method.

Test: atest system/hardware/interfaces/wifi/keystore/1.0/
            vts/functional/VtsHalWifiKeystoreV1_0TargetTest.cpp

Change-Id: Ie549ea1c1cd6c767c013c340ae297ae8dfacca69

Add VTS for Wifi Keystore HAL's getBlob()

This exercises paths through the Wifi Keystore HAL's getBlob()
method.

Test: atest system/hardware/interfaces/wifi/keystore/1.0/
            vts/functional/VtsHalWifiKeystoreV1_0TargetTest.cpp

Change-Id: I93114c8de00049478b9bf57124d09b1f4f2ebdd6

Deploy VtsHalWifiKeystoreV1_0Target in automated vts tests

Bug: 120182820
Test: vts-tradefed run commandAndExit vts-hal --module VtsHalWifiKeystoreV1_0Target
Change-Id: I95d7ce91571014b7c639bf0628a3d3f3c565a27a

Add VTS for Wifi Keystore HAL's sign()

This generates a signing key and attempts to use it
in the Wifi Keystore HAL's sign() call. This reproduces
the SIGSEGV in 120182820, and with the fix shows that
data can be signed.

Bug: 120182820
Test: atest system/hardware/interfaces/wifi/keystore/1.0/
            vts/functional/VtsHalWifiKeystoreV1_0TargetTest.cpp

Change-Id: I98afa26e7a54cacff8935a3420d0ea5a2ca5ac50

Merge "Add OWNERS for Wifi Keystore HAL and VTS tests"

Add OWNERS for Wifi Keystore HAL and VTS tests

These are the owners as listed in hardware/interfaces/wifi/**

Test: none necessary

Change-Id: I247dea0a97029ce002db726d0c800c9835f89d07

Merge "Assign a new future before calling update()"

Assign a new future before calling update()

An empty future was being used, and when the update() call
completed the value in the empty future was grabbed
resulting in a nullptr de-reference.

Bug: 120182820
Test: atest system/hardware/interfaces/wifi/keystore/1.0/
            vts/functional/VtsHalWifiKeystoreV1_0TargetTest.cpp

Change-Id: I02968a5b93f2e59ce5801676ab3bae437e2c53d8

Handle /sys/power/* interfaces not being available.

If either /sys/power/wakeup_count or /sys/power/state fail to open, we
construct SystemSuspend with blocking fds. This way this process will
keep running, handle wake lock requests, collect stats, but won't
suspend the device.

We want this behavior on devices (hosts) where system suspend should not
be handles by Android platform e.g. ARC++, Android virtual devices.

Bug: 118637369
Test: Remove system.suspend access to /sys/power/{ wakeup_count state }
using SELinux. Device still boots and doesn't suspend.
Change-Id: I068a584ada6968520219ee5288f598a53aa48c68

system suspend: more information about wake locks

/sys/power/{wake_lock wake_unlock} can give a list of all currently
active and inactive wake lock names. Bring system suspend debug
functionality up to par by keeping track of all used wake lock names,
client pids, current state and last used timestamp.

To prevent collected stats from growing unboundedly we evict least
recently used stats entry after a given threshold.

Bug: 78888165
Test: SystemSuspendV1_0UnitTest
Change-Id: I26eb94e61d4656ec1bba45b4a70a0715fae15d05

Fix inverted error code.

During the Multithreaded keystore refactor the error code returned by
getKeyCharacteristics was mistakenly inverted.

Test: yes please!
Bug: 120182820
Bug: 111443219
Change-Id: I5f5c07f30e0ee264ce75a5ffd2986186b9d66ce3

Use stream operator to report result code

The cast operator for KeyStoreNativeReturnCode and
KeyStoreNativeReturnCode will be removed later.
There already exists a stream operator to get
at the underlying value. Use that instead.

Bug: 119771891
Test: Built for walleye successfully.
Change-Id: I784d3c54d7b2d612201f045fe2862cf2529f4e1f

Merge "Add trong@ to OWNERS for suspend HAL."

Merge "Multi-threaded Keystore"

Merge "Revert "Start suspend HAL early.""

Add trong@ to OWNERS for suspend HAL.

Bug: n/a
Test: n/a
Change-Id: I4c763478d86c68735edbf1355a8c25a8f9e22b7d

Revert "Start suspend HAL early."

This reverts commit ec7772299db6bc76f1c2aa1d471447cfddce04ae.

Fixes: 119325488
Reason for revert: regressed boot time

Change-Id: I91e2300b1f0625a5787d0af02db06b3079331787

Multi-threaded Keystore

This patch changes the calling code in the wifi interface to use the new
asynchronous keystore api model.

Test: would be nice
Bug: 111443219
Change-Id: Iee6a6fede4670dfe503dc7b87a34b88c1c8f01f4

Merge "Remove legacy functions form IKeystoreService.aidl"

Start suspend HAL early.

Requests to acquire wake locks are blocked on suspend HAL being started.
So ideally it should be available to the system as early as possible.

Bug: 78888165
Test: boot walleye
Change-Id: If38836ce561f06ce9d3ee3f6e03885aaccbd48c0

Remove legacy functions form IKeystoreService.aidl

In preparation for making the keystore service asynchronous we remove
redundant legacy functionality from the protocol.

This patch removes the functions get_pubkey, sign, verify, generate,
and import_key. Which have long been superseded by exportKey
(get_pubkey), begin/update/finish (sign, verify), generateKey
(generate), and importKey (import_key).

Test: Not yet
Bug: 109769728
Change-Id: Ia59b30377684c013e9b4496ca9e365f8c2710ec1

Merge pie-platform-release to aosp-master - DO NOT MERGE

Change-Id: Iae3aab5abbf9ec60db33a9184d76210c345641ee

Deploy SystemSuspendV1_0UnitTest in automated VTS runs.

Bug: 117176837
Test: vts-tradefed run commandAndExit vts-systems
--skip-all-system-status-check --primary-abi-only --skip-preconditions
--skip-device-info --module SystemSuspendV1_0TargetUnitTest
Change-Id: Icfef15e6fbbe86f7ca2a0a7e47897f1c837e7695

Fix deadlock when callback modifies callback list.
am: c0578b9128

Change-Id: Ifa436f31dbed547f3d3df9664dd8c92ff1496f34

Fix deadlock when callback modifies callback list.

Bug: 117555757
Test: SystemSuspendUnitTest
Change-Id: I40e7ffcc1bbc744a5b9e6d739166f1eebb8ed151

Implement exponential backoff analogous to libsuspend
am: 5d524a1d61

Change-Id: Ica9313242603fa9f2c8bc3ebaa0a473645ffd5dd

Implement exponential backoff analogous to libsuspend

Suspends attempts are not free w.r.t. power consumption. This change prevents
rapid battery drain on devices with broken system suspend (e.g. due to faulty
driver)

Originally implemented in libususpend in commit
fb896944b6a78d951c8442b444af860a28117e5c

Bug: 78888165
Test: SystemSuspendV1_0UnitTest
Test: left device overnight, ~50% fewer failed attempts to suspend with this
change vs without.
Change-Id: Icff1135b9c316fa9de1558ecea42db9ed0d10d9a

Passing partial vs full wake lock info for ARC++
am: 365b75f157

Change-Id: Ie783b4bed8cd0624d7320f79d7e043f772900618

Passing partial vs full wake lock info for ARC++

We pass an enum to acquireWakeLock(), symmetric to how it's done in libpower
interface.

The screen could be managed above (android) or below (ARC++) this HAL depending
on the device, so we don't mandate that full wake locks keep the screen alive.
Only requirement is that all wake locks block system suspend.

Bug: 115947180
Test: SystemSuspendV1_0UnitTest
Change-Id: I92c65371852ffb2f71f5d529b7020d256cb1ba76

Add ISystemSuspendCallback.
am: b53742607a

Change-Id: I87cb696e83bf7d52f0a7fe35e12cf62a0c69f68f

Add ISystemSuspendCallback.

The callback will be used to synchronously report system suspend events.

Current usage: BatteryStatsService needs to be notified about
each system suspend attempt and whether it was successful.

Bug: 78888165
Test: SystemSuspendV1_0UnitTest
Change-Id: I234f74f6f3c35e5d6bb33ceda569e11520850ccf

Add IWakeLock::release() method.
am: d502490bac

Change-Id: Id5e96044930c08c01589afdf1d213c98f39efb92

Add debugName parameter to acquireWakeLock().
am: a3573abb32

Change-Id: Ic8523357af3e14f5ffa4f461cf50eb8fcb74c00e

Add IWakeLock::release() method.

Motivation:
-This should help migrate code that relied on old
acquire/release_wake_lock() methods to new API.
-Being able to release a lock before the end of the scope is consistent
with other "lock" constructs (e.g. unique_lock, shared_lock).

Bug: 78888165
Test: SystemSuspendV1_0UnitTest
Change-Id: Ibb654f523b9c255df68d93407b593deac006920b

Add debugName parameter to acquireWakeLock().

Bug: 78888165
Test: SystemSuspendV1_0UnitTest
Change-Id: Ia6570af63eb1eb01145bd7870c196a0eed91883a

System suspend HAL implementation.
am: 180813bea6

Change-Id: I262ff63599f32cabbac892f799f9573c21aba1d1

System suspend HAL implementation.

Bug: 78888165
Test: SystemSuspendV1_0UnitTest
Change-Id: I4ba1075ce51684e431098635c72aece67351644a

System suspend HAL interface.
am: 3b083e7c94

Change-Id: I0119c0323a7fff2ed1afe0a474b2aed33c06664c

System suspend HAL interface.

Bug: 78888165
Test: SystemSuspendV1_0UnitTest
Change-Id: I3c3537a72dac65d316aeb77fd7b2d7cff4fb67e5
Merged-In: I3c3537a72dac65d316aeb77fd7b2d7cff4fb67e5
(cherry picked from commit 2fbf967c4f47b7f3977893fd4a2988d32eff4c27)

Merge Android Pie into master am: 69c3fc690d
am: 6c9a5982be

Change-Id: I3e1a3ad2624dd3e052483036d5a0029561a6a4ae

Merge Android Pie into master
am: 69c3fc690d

Change-Id: I399108f32aa466b8554b5fec1b517630b879f063

Merge Android Pie into master

Bug: 112104996
Change-Id: I7a2d421f293726a9e9db797f137601d73c9223eb

Remove path from hidl_package_root. am: 9720abcd5a
am: 184aad2a8f

Change-Id: I08ec3b53f7ddbd800eede7f8d6a251f04e365c1d

Remove path from hidl_package_root.
am: 9720abcd5a

Change-Id: I1f7924c1a00cfaadab8796b61ffd3162b66fac4b

Remove path from hidl_package_root.

This now defaults to the current path making references
to it relocatable.

Bug: 111366989
Test: make hidl interfaces under this package root
    (this exercises the new path)

Change-Id: Id0629f4e8fa066b5cddae71016f52dcd3d5e129a

Merge pi-dev-plus-aosp-without-vendor into stage-aosp-master
am: c2e1ca1aac

Change-Id: I66031f4bd7b288e4ab6564939a238bcfc31ab9e7

Merge pi-dev-plus-aosp-without-vendor into stage-aosp-master

Bug: 79597307
Change-Id: Id45caa0bfe6e942d9bdb835dc87345a87071bba3

Fix NetdHidlTest.TestAddRemoveInterfaces() am: 84777b06df
am: 073f5088c5

Change-Id: I1af658298b2fc10c857de01cff3a31a5e5c158e6

Fix NetdHidlTest.TestAddRemoveInterfaces()
am: 84777b06df

Change-Id: I8aea1d82b2bf3b55505f22a4f79d8d257a4341ea

Fix NetdHidlTest.TestAddRemoveInterfaces()

The regex in countRulesForFwmark() overeagerly matched rules ending
in the same hex digit. For instance, fwmark=3 would match both these
lines:

  from all fwmark 0xd0003/0xdffff iif lo lookup netd13474_27190
  from all fwmark 0x10063/0x1ffff iif lo lookup local_network

This fix consists in matching either just 0x3, or the zero-padded 0003.

Bug: 74922133
Test: as follows:
    m VtsHalNetNetdV1_1TargetTest && \
    adb push out/.../VtsHalNetNetdV1_1TargetTest /data/ && \
    adb shell /data/VtsHalNetNetdV1_1TargetTest
Change-Id: If6df9fa6cf843a5593ce61d87366c23383bb4d81
(cherry picked from commit 64848957ba49275896c80790ceeeb6219930f687)

Snap for 4751833 from 64848957ba49275896c80790ceeeb6219930f687 to pi-release

Change-Id: Ic167b25d177db6ad0c9204f66ed148d33e99a74a

Fix NetdHidlTest.TestAddRemoveInterfaces()
am: 64848957ba

Change-Id: I6915ed661784663ae10cb593e1081ea9c8c688f3

Fix NetdHidlTest.TestAddRemoveInterfaces()

The regex in countRulesForFwmark() overeagerly matched rules ending
in the same hex digit. For instance, fwmark=3 would match both these
lines:

  from all fwmark 0xd0003/0xdffff iif lo lookup netd13474_27190
  from all fwmark 0x10063/0x1ffff iif lo lookup local_network

This fix consists in matching either just 0x3, or the zero-padded 0003.

Bug: 74922133
Test: as follows:
    m VtsHalNetNetdV1_1TargetTest && \
    adb push out/.../VtsHalNetNetdV1_1TargetTest /data/ && \
    adb shell /data/VtsHalNetNetdV1_1TargetTest
Change-Id: If6df9fa6cf843a5593ce61d87366c23383bb4d81

Snap for 4693621 from 9a9a0772bc04427f3cc23289501a6b56463d1db1 to pi-release

Change-Id: I24dc0eba050216b702dc2e831055251fc87c0e52

Merge "update OWNERS for VTS" into pi-dev
am: 9a9a0772bc

Change-Id: I49a76fb5f97b5dc40880e740ab775bb92b3e83aa

Merge "update OWNERS for VTS" into pi-dev

[automerger skipped] Add tests for INetd 1.1.
am: f95d0c28b6  -s ours

Change-Id: If776ebfdd2f0791ac63581f5c4bd9db4f1dc647d

[automerger skipped] Make the VTS tests for the INetd HAL 1.0 more realistic.
am: 65ae996fca  -s ours

Change-Id: I95315fe8cf41fbae8b99b250eb72a5748577958b

update OWNERS for VTS

Test: inspection
Bug: 75325573
Change-Id: I61c0d7fd4c4ca49e9370c8dc2bcfc5e00812aec1

Add tests for INetd 1.1.

Bug: 73032258
Test: VtsHalNetNetdV1_0TargetTest and VtsHalNetNetdV1_1TargetTest passes on marlin
Change-Id: Id4d49f08563982353a9a7db79b569e933ea03c46
Merged-In: Id4d49f08563982353a9a7db79b569e933ea03c46
(cherry picked from commit 94832f94dc902b130bf3b2814e9addc2f27a2840)

Make the VTS tests for the INetd HAL 1.0 more realistic.

The VTS test for the netd HAL creates an OEM network, but it
doesn't check whether the network exists. Fix that by using
the native multinetwork API calls to select the network.
Also, check that when an OEM network is created, there are no IP
rules that select for its fwmark. There should be none because
a network that has just been created has no interfaces.

Put utility code into a new VtsHalNetNetdTestUtils static library
to avoid code duplication when we add new tests for future HALversions.

Bug: 73032258
Test: VtsHalNetNetdV1_0TargetTest passes on marlin
Change-Id: I345b9b27d7c2c8bdd6ce6d60430490dac8c3a989
Merged-In: Ie574990f7640fdc7d83ccd58baf6dfe1069a3654
(cherry picked from commit 79f693702c35dca81f7390d1619736a204e61a70)

Snap for 4662252 from 61323db76114d7115ad44281eba81801a562828c to pi-release

Change-Id: Iabd6928174603f91e1d4cf3790d338ee0156a169

Freeze HALs for Android P
am: 61323db761

Change-Id: I3899796c588e5254a2b6ce4ca5beb6556ebc50dd

Freeze HALs for Android P

Bug: 68112778
Test: none
Change-Id: Ic3617e937a4589f5ef6a40b4831c9a77328215e0

Add tests for INetd 1.1. am: 94832f94dc
am: 2cf1f13200

Change-Id: Ib42934f33261172bd8e9884fabbb814491067b59

Add tests for INetd 1.1.
am: 94832f94dc

Change-Id: Ie8e6da55140ff1f1e42731e9ce41ca08f864ca96

Make the VTS tests for the INetd HAL 1.0 more realistic. am: 79f693702c
am: 065b170e64

Change-Id: Ie574990f7640fdc7d83ccd58baf6dfe1069a3654

Make the VTS tests for the INetd HAL 1.0 more realistic.
am: 79f693702c

Change-Id: Id951ad6982142a423ffb881979d0ad9e660b4afe

Add tests for INetd 1.1.

Bug: 73032258
Test: VtsHalNetNetdV1_0TargetTest and VtsHalNetNetdV1_1TargetTest passes on marlin
Change-Id: Id4d49f08563982353a9a7db79b569e933ea03c46

Make the VTS tests for the INetd HAL 1.0 more realistic.

The VTS test for the netd HAL creates an OEM network, but it
doesn't check whether the network exists. Fix that by using
the native multinetwork API calls to select the network.
Also, check that when an OEM network is created, there are no IP
rules that select for its fwmark. There should be none because
a network that has just been created has no interfaces.

Put utility code into a new VtsHalNetNetdTestUtils static library
to avoid code duplication when we add new tests for future HALversions.

Bug: 73032258
Test: VtsHalNetNetdV1_0TargetTest passes on marlin
Change-Id: I345b9b27d7c2c8bdd6ce6d60430490dac8c3a989

Snap for 4647811 from 8f840890816dce4ce04cade229d5ab80b850dcea to pi-release

Change-Id: I9cd5167037720d715ca705a297dcd55214b07f90

Introduce netd 1.1 am: f5ef707452
am: 4d206db225

Change-Id: I11a56940d924ce0ecdfdd773fdec19ddefa7764c

Introduce netd 1.1
am: f5ef707452

Change-Id: Ia3ef4bf91623d9f2e71560e0bbd36b2320e60042

Introduce netd 1.1

Additional OEM Network APIs

 > add/remove interface
 > add/remove route
 > set forwarding

(cherry picked from commit 8f840890816dce4ce04cade229d5ab80b850dcea)

Bug: 73032258
Test: marlin builds, boots
Test: VtsHalNetNetdV1_1TargetTest, VtsHalNetNetdV1_1TargetTest pass on whole topic
Change-Id: I40b9a8023560aa6db86012ae013e8f6175472162

Introduce netd 1.1
am: 8f84089081

Change-Id: I74b587614e3c9bd81d4ffb3ad374ca0dbecfa6ce

Introduce netd 1.1

Additional OEM Network APIs

 > add/remove interface
 > add/remove route
 > set forwarding

Bug: 73032258
Test: marlin builds, boots
Test: VtsHalNetNetdV1_1TargetTest, VtsHalNetNetdV1_1TargetTest pass on whole topic
Change-Id: I40b9a8023560aa6db86012ae013e8f6175472162

Snap for 4496165 from fee9f9dd5c74219f725efe8d168f0497745cc137 to pi-release

Change-Id: Idb20a650b242ab9f046973d34e226939955450b8

DO NOT MERGE: Merge Oreo MR1 into master am: a5d45aa1dd  -s ours am: d10f572af0
am: 9b699fdb99  -s ours

Change-Id: I2db04878a0b2b54344b960f35502cbc08c25deda

DO NOT MERGE: Merge Oreo MR1 into master am: a5d45aa1dd  -s ours
am: d10f572af0

Change-Id: I43324aefa900fe2a70aec620ab22b9b5b0737e36

DO NOT MERGE: Merge Oreo MR1 into master
am: a5d45aa1dd  -s ours

Change-Id: Iabd8d07c4007f802c690fbeaee9edc24dae51842

DO NOT MERGE: Merge Oreo MR1 into master

Exempt-From-Owner-Approval: Changes already landed internally
Change-Id: I88cfd3f7ac04c338e456245f011eb7b0ca55248c

Snap for 4476081 from cedc03360070d3176c0c5c9551e5edc905111eaa to pi-release

Change-Id: I34c952fed4f3848fb8d18231c9c1526e4b230e2e

Make libkeystore_aidl shared and export to PDK.

Bug: 69539820
Test: manual
Change-Id: I280905a5572bab4fd3aeba9e82c0030d50c5890e

Snap for 4459973 from 7ebf2fb767add88b679dde17ba730319037b755d to pi-release

Change-Id: Ibe50f3cf1c6115852d76a481d6e21d9b3017e356

Merge "Get rid of manually generated IKeystoreService in wifi/keystore"

Snap for 4453288 from ec2384b75efccd1631d5c359d823be1e9f98f888 to pi-release

Change-Id: I48f50f9a44a22ac9a515645bc531aa6e435faa79

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD
am: 666ee70f8f

Change-Id: Iaa269e313fe7a0ca5128e04692fcd5119d7c4376

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD

Change-Id: I17f553bded1573f2e19a7096f18364ce638702d0

Merge commit '222107507d126c33bd48647fb92e0b4021fb861f' from
oc-mr1-dev-plus-aosp into stage-aosp-master

Change-Id: Ibd45fbd904abfbb986d8f3adfc415ddd0e35b1e7

Snap for 4450504 from 2e5f221b5aeb236e9d200cbb413b668cf3e2464e to pi-release

Change-Id: I86ab9c105da24caec9aba1ca36fefbdcabe12e10

Update makefiles for hidl_interface. am: c15b482924 am: 1e9c7e125f
am: 751427f1a8

Change-Id: Ia9c4a85147f189987946ef1bcbcb95abd2e0477f

Update make file for vndk enabled. am: 4b8f02e7aa am: e612a2ec6e
am: 47c864541e

Change-Id: I71486d2c7a502fa155359ab7731bf3093dda2d53

Update makefiles for hidl_interface. am: c15b482924
am: 1e9c7e125f

Change-Id: Ie2e32c3983cd9f2fbfc266b094355c43e72cbe99

Update make file for vndk enabled. am: 4b8f02e7aa
am: e612a2ec6e

Change-Id: Id9d5682551c8aef5e3f95db211fa28fd0621347f

Update makefiles for hidl_interface.
am: c15b482924

Change-Id: I75d2ed14f3660e2c8e84b728f56430cf361268bb