Commit graph

77805 commits

Author SHA1 Message Date
Benergy Meenan Ravuri
e272fa6102 Check userspace snapshots only for API level >= T am: 9591090a17 am: 6a7cb89655
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/19263116

Change-Id: I1dc5b724348367df871482ad8612ff78c32f4a33
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-07-13 17:42:02 +00:00
Akilesh Kailash
355141f9cd libsnapshot: Fix vts_libsnapshot_test.SnapshotUpdateTest#FullUpdateFlow am: 299717115e am: 4327290387
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/19263115

Change-Id: Ie0f8bf637ebfbedce589c96d7ed6916423eefa4b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-07-13 17:42:00 +00:00
Benergy Meenan Ravuri
6a7cb89655 Check userspace snapshots only for API level >= T am: 9591090a17
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/19263116

Change-Id: I750b36be179054818ba7f5f724e01f17fa7e41a9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-07-13 17:25:41 +00:00
Akilesh Kailash
4327290387 libsnapshot: Fix vts_libsnapshot_test.SnapshotUpdateTest#FullUpdateFlow am: 299717115e
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/19263115

Change-Id: I487a3706561233cb3834593b570c80a9abfa6b3d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-07-13 17:25:40 +00:00
Benergy Meenan Ravuri
9591090a17 Check userspace snapshots only for API level >= T
Skip checking for userspace snapshots enabled property
for API level < T as this feature is not applicable for
GRF targets.

Bug: 236450435
Test: vts_ota_config_test
Change-Id: Ib5083f6237cdf4962aae06f166811d67cf6c385e
Ignore-AOSP-First: already present in aosp/master
2022-07-12 17:57:59 +00:00
Akilesh Kailash
299717115e libsnapshot: Fix vts_libsnapshot_test.SnapshotUpdateTest#FullUpdateFlow
Check if compression feature is enabled.

Bug: 238143011
Test: OTA
Signed-off-by: Akilesh Kailash <akailash@google.com>
Change-Id: I77663799a5371f1de8b4e29d7c777a24249d1b7b
Ignore-AOSP-First: already present in aosp/master
2022-07-12 17:57:25 +00:00
Ray Ye
f4c4bc7be3 Updating async trace for track trace symbols from T/U to G/H am: 621c74c063
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/19037614

Change-Id: I9de52855f93a49cffee97145bb79f4f5475d9456
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-06-23 19:59:52 +00:00
Akilesh Kailash
e30539f6b7 libsnapshot: Fix vts_libsnapshot_test for GRF with Vendor on S am: ae04e7ede1
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18917076

Change-Id: I82dd63f4f07154f4fe2b32f82e0afc171769a5f1
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-06-23 19:59:18 +00:00
Ray Ye
621c74c063 Updating async trace for track trace symbols from T/U to G/H
Bug: 230399626
Test: atest TraceDevTest
Ignore-AOSP-First: Change from aosp/2078467 without ABI change
Change-Id: Ia927c19f544536f78c20ccef5830bd1be0cebf9b
Merged-In: Ia1b39f55c96011a9672f4d5c8763044f849d0bb2
2022-06-22 19:09:06 +00:00
Akilesh Kailash
ae04e7ede1 libsnapshot: Fix vts_libsnapshot_test for GRF with Vendor on S
If the vendor partition is on S and system partition is on T,
certain tests in vts_libsnapshot_test used to fail. This is primarily
because of inconsistent check between daemon and vts test.

vts test checks the userspace.snapshots.enabled property which is true on T
but never checks if the underlying vendor partition is on S. Hence,
vts test will enable userspace snapshots. However, daemon checks
the vendor partition and disables userspace snapshots thereby
leading to inconsistency.

This is only a problem on vts tests. The underlying OTA on devices
works fine as we have the vendor partition check.

Bug: 236311008
Test: vts_libsnapshot_test on S vendor and T system
      vts_libsnapshot_test on T vendor and T system
Ignore-AOSP-First: cherry-pick from aosp
Signed-off-by: Akilesh Kailash <akailash@google.com>
Change-Id: Iad4f299bd2e07c9c01f5fbee6a20e2f01bf1778a
2022-06-16 22:56:14 +00:00
Shawn Willden
89a6887893 Implement GetRootOfTrust am: 18cf62712c
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18868385

Change-Id: Icf12c572faf7b17a0207218a2859d013c96d3346
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-06-14 16:49:07 +00:00
Shawn Willden
18cf62712c Implement GetRootOfTrust
Bug: 219076736
Test: VtsAidlKeyMintTargetTest
Change-Id: I8485360f253ca20f008c5df5090e79c1781fbb4f
Merged-In: I8485360f253ca20f008c5df5090e79c1781fbb4f
Ignore-AOSP-First: already present in aosp/master
2022-06-14 10:22:43 +01:00
Akilesh Kailash
d31b8a9678 Merge "init: Set oom_score_adj to snapuserd process" into tm-dev am: ae2837d80c
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18720435

Change-Id: I339b0434d908d841c5a57b3fabb5a76f01e8877c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-06-02 23:39:25 +00:00
Akilesh Kailash
ae2837d80c Merge "init: Set oom_score_adj to snapuserd process" into tm-dev 2022-06-02 23:18:52 +00:00
Akilesh Kailash
1a2f870cb3 init: Set oom_score_adj to snapuserd process
When a process is started as a native service,
oom_score_adj is set to -1000 so that processes
are unkillable by lmkd.

During boot, snapuserd process is not started as a service;
hence, we need to set the oom_score_adj explicitly else in
the event of low memory situation, lmkd can kill the
process thereby device can never boot.

Bug: 234691483
Test: th and OTA on Pixel
Ignore-AOSP-First: cherry-pick from AOSP
Signed-off-by: Akilesh Kailash <akailash@google.com>
Change-Id: Ic2c85aa470522b4bc847a16b4f5cebfc528ed3cf
2022-06-02 21:19:46 +00:00
Stephen Crane
577b216b27 [trusty] Bump number of ACVP arguments to match boringssl am: 71100bbc44
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18674342

Change-Id: Ieb1d049f0740003586d85854aa6a0444ab9445b7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-06-02 12:08:25 +00:00
Stephen Crane
71100bbc44 [trusty] Bump number of ACVP arguments to match boringssl
Cherry-pick note: Needed for the BoringSSL change in the same topic.  No functional change to Android devices, this change affects ACVP testing only.

Boringssl added a new argument to the ACVP modulewrapper, so we need to
sync this change to the Trusty modulewrapper.

Test: m trusty_acvp_modulewrapper
Bug: 233873228
Change-Id: Ia79705d9fe32a07afc09cf8e0231db300073b504
(cherry picked from commit 08d8557718)
Merged-In: Ia79705d9fe32a07afc09cf8e0231db300073b504
2022-05-31 19:04:19 +00:00
TreeHugger Robot
bc988a9c90 Merge "storaged: Avoid divide-by-zero in performance measurement" into tm-dev am: 190e2bc6e1
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18620177

Change-Id: Id2399bdc6294e269a755f60ab666ac1cb54c3a95
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-31 17:05:11 +00:00
TreeHugger Robot
190e2bc6e1 Merge "storaged: Avoid divide-by-zero in performance measurement" into tm-dev 2022-05-31 16:47:14 +00:00
David Anderson
2fe3b61913 vts_fs_test: Require EROFS sysfs nodes. am: 5b227f3451
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18636032

Change-Id: Ibf35f0f67fab62560fb50ca2624e1c0e544a8ade
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-27 21:44:05 +00:00
David Anderson
ac5172c916 Merge "Relax filesystem requirements in vts_fs_test." into tm-dev am: 0d3b8d557f
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18625783

Change-Id: I381eb347e23d2998217b2152847837086cc671aa
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-27 20:51:27 +00:00
Hsin-Yi Chen
f665fa95f5 Merge "RESTRICT AUTOMERGE Ignore weak symbol difference in libutils ABI check" into tm-dev am: 3f2a2bf298
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18603830

Change-Id: I95d489efe2373df6ee0353eb6efc7e72f76927de
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-27 20:50:46 +00:00
Hsin-Yi Chen
3ecc3962f3 RESTRICT AUTOMERGE Ignore weak symbol difference in libutils ABI check am: c7d9320ca4
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18603830

Change-Id: Ib7aac03aacd9007449e1e33115e0d4b91918a66c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-27 20:50:42 +00:00
David Anderson
5b227f3451 vts_fs_test: Require EROFS sysfs nodes.
These were backported to android13-5.10 and should be present in
T-launch kernels.

Bug: 233926292
Test: vts_fs_test
Change-Id: Ifb5ff6a200b081fe8696d5803d4a128740eb8e21
Merged-In: Ifb5ff6a200b081fe8696d5803d4a128740eb8e21
Ignore-AOSP-First: cherry-pick
2022-05-27 19:01:59 +00:00
David Anderson
0d3b8d557f Merge "Relax filesystem requirements in vts_fs_test." into tm-dev 2022-05-27 04:55:28 +00:00
Hsin-Yi Chen
3f2a2bf298 Merge "RESTRICT AUTOMERGE Ignore weak symbol difference in libutils ABI check" into tm-dev 2022-05-27 02:59:01 +00:00
Matthew Duggan
84b7ca8d81 storaged: Avoid divide-by-zero in performance measurement
The code previously assumed that write() took a non-zero number of clock
ticks.  This is mostly true, but can't be guaranteed.  Add a check to
avoid crashing if it's 0.

Bug: 233008287
Test: Compile
Change-Id: Idec0052f534dc4abb81a414ca76f6c11f4a4dded
(cherry picked from commit 5a02a9bab9)
Merged-In: Idec0052f534dc4abb81a414ca76f6c11f4a4dded
2022-05-27 01:51:34 +00:00
David Anderson
0895d63952 Relax filesystem requirements in vts_fs_test.
Bug: 233926292
Test: vts_fs_test
Change-Id: I9665acebd6ebfde14e0cd76e8044e7ced9ffbeb3
Ignore-AOSP-First: cherry-pick from AOSP
2022-05-27 00:21:30 +00:00
Rubin Xu
e460cb3763 Add AID_SECURITY_LOG_WRITER am: 2cf162748f
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18603801

Change-Id: I03b10a85651b8c44e8f752c330ab1f7b7391cc77
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-26 17:41:16 +00:00
Hsin-Yi Chen
c7d9320ca4 RESTRICT AUTOMERGE Ignore weak symbol difference in libutils ABI check
The diff_flags make the ABI checker ignore the weak symbol difference
caused by PGO.

Ignore-AOSP-First: This patch relaxes the ABI check for T only.
                   b/232982219 tracks the long-term solution.
Test: make libutils.vendor
Bug: 230076879
Change-Id: I8ec2c0f5a540263b4e8a0a4f9cf26f9c297593b5
2022-05-26 15:20:06 +08:00
Rubin Xu
2cf162748f Add AID_SECURITY_LOG_WRITER
A suppplementary group to mark which app can write to the security log
buffer. Granted via android.permission.WRITE_SECURITY_LOG and checked by
logd.

Bug: 232283779
Test: manual
Change-Id: I3ad66031ab1c9eef26db0f3e8968659dfa2855cb
(cherry picked from commit 7e14d138ae)
Merged-In: I3ad66031ab1c9eef26db0f3e8968659dfa2855cb
2022-05-25 20:54:35 +00:00
Christopher Ferris
1b5eb8d667 Fix check for thread unwind. am: 2d5d46ca85
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18596248

Change-Id: I8d23b262eb90576de05b5c625775d31cb0044044
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-25 18:32:27 +00:00
Christopher Ferris
2d5d46ca85 Fix check for thread unwind.
If a process requires executing fallback unwinder and the thread
crashing is not the main thread, the wrong unwinder is used.
Fix this case, and add a new unit test that causes an abort in
the non main thread.

Bug: 233721755

Test: New unit test passes with fix and fails without.
Test: Ran debuggerd on swcodec process and it still dumps all threads.
Change-Id: I70fffc5d680256ce867e7a1d427593b584259160
2022-05-24 21:31:03 -07:00
Steven Moreland
096e8598cc Merge "init: log services requested restart" into tm-dev am: b862059972
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18447265

Change-Id: I55af7b54fce846c7b90a94cfa9890804149fed4b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-23 18:53:13 +00:00
Steven Moreland
b862059972 Merge "init: log services requested restart" into tm-dev 2022-05-23 18:28:02 +00:00
Akilesh Kailash
9cd245279f libsnapshot: Initialize merge_op_start_ to zero. am: d36a68183c
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18512862

Change-Id: If91af36b16c229388fa13f9564d2c2d5638ac61e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-20 22:21:31 +00:00
Akilesh Kailash
d36a68183c libsnapshot: Initialize merge_op_start_ to zero.
merge_op_start_ is used to set the iterator for merge operations.
Uninitialized value can potentially lead to setting up
of bad iterator.

Bug: 233246309
Test: Full OTA
Signed-off-by: Akilesh Kailash <akailash@google.com>
Change-Id: I3cc48a66b532cfe8b2d87c8724d77ab3169a2ddb
Merged-In: I3cc48a66b532cfe8b2d87c8724d77ab3169a2ddb
2022-05-20 06:11:32 +00:00
David Drysdale
cb220c3806 KeyMint HAL: pass auth token on updateAad am: 986a2dd965
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18457422

Change-Id: Id55c71c2ff97695851c8e9772c087035b2cc08c2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-19 11:07:04 +00:00
David Drysdale
986a2dd965 KeyMint HAL: pass auth token on updateAad
Bug: 230716629
Test: CtsVerifier with aosp/2077898 included
Change-Id: I2ba275718ef7ce6cc701ac2eb4a122639d7a4351
Merged-In: I2ba275718ef7ce6cc701ac2eb4a122639d7a4351
Ignore-AOSP-First: cherry-pick of aosp/2101110
2022-05-18 12:31:32 +01:00
Steven Moreland
9e0aae3683 init: log services requested restart
We have a case where a service is requested to be started and does
not appear to be running, but we see no indication that it is
actually starting. This log should be enough information to see
if init is in a bad state.

Bug: 232297944
Test: doesn't add too much spam
    ~/android/aosp/system/core/init :) adb logcat -d | grep "requested start" | wc -l
    42
Change-Id: Ic07f250c98b200b9e5b4432200c3668c6ca0ff35
Merged-In: Ic07f250c98b200b9e5b4432200c3668c6ca0ff35
2022-05-18 00:08:44 +00:00
Victor Hsieh
f36a6ea720 Make VMCompilationPerformance an aggregated profile with default am: b40fb113e7
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18423662

Change-Id: I224f53b864c3daf9a2e8f8da63a11704aecb4881
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-17 22:54:04 +00:00
Victor Hsieh
b40fb113e7 Make VMCompilationPerformance an aggregated profile with default
The VMCompilationPerformance profile is used to run Isolated Compilation
in a Protected VM, normally during the device idle, where relevant
APEXes are staged.

The original VMCompilationPerformance introduced in aosp/2060891 does
not have any specific definition and requires vendors to customize it.
This change re-defines it as an aggregated profile with a default set of
existing profiles, so that performance can be reasonable by default.

This profile may be renamed to a more generic name later, e.g.
"SCHED_SP_COMPUTE".

Bug: 231437770
Test: Run `composd_cmd test-compile` on a local device.
      Before: 1m50s +/- 10s (with whatever that's default)
      After: 1m25s +/- 5s
Ignore-AOSP-First: cherry pick from aosp/2097065
Change-Id: Ib8cd65782c818474fb129efbd9ef9a3e23ad1eb3
Merged-In: Ib8cd65782c818474fb129efbd9ef9a3e23ad1eb3
2022-05-16 17:53:01 -07:00
TreeHugger Robot
8ce05188da Merge "Update the encryption rule of /data/bootanim to DeleteIfNecessary" into tm-dev am: 7e778c1350
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18364895

Change-Id: I95d701fb014bc26a8f822bea263be8d49bf83582
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-13 23:53:17 +00:00
TreeHugger Robot
7e778c1350 Merge "Update the encryption rule of /data/bootanim to DeleteIfNecessary" into tm-dev 2022-05-13 23:21:41 +00:00
Pete Bentley
35a32d0c58 Remove setenvs when running boringssl_self_test. am: 8a9b9820e5
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18375393

Change-Id: I7777f1564319d03da46c5da55acf33d11d1a6b04
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-13 21:57:28 +00:00
Josh Yang
86b08e5efd Update the encryption rule of /data/bootanim to DeleteIfNecessary
Directories should always be encrypted unless there is a specific reason
they can't be.  /data/bootanim is unencrypted without a specific reason,
so fix it to be encrypted.  It is too late to use encryption=Require.
However, the contents of this directory doesn't need to be preserved on
updates, so we can use encryption=DeleteIfNecessary instead of
encryption=Attempt.

Bug: 232299581
Test: build succeeds
Change-Id: Iae80043f3957a053d47ab4754c42b1fdc8e696b3
Merged-In: I17bcb901ad533cada4e0aa061196fc94d7b213ec
2022-05-13 17:03:22 +00:00
Pete Bentley
8a9b9820e5 Remove setenvs when running boringssl_self_test.
No longer needed as the code to generate flag files based on
this environment variable is removed in Android 13.

Bug: 231946889
Test: Build and boot,
Change-Id: I8ce57619aa4d1e6457f3f864bf5e403f727c040c
(cherry picked from commit ba830eb81b)
Merged-In: I8ce57619aa4d1e6457f3f864bf5e403f727c040c
2022-05-13 17:42:53 +01:00
William Escande
a0a289b2b2 Set the aptx as provided by system am: e41474eae5
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18337263

Change-Id: Ic90c52bc33e25d20bc0bd6207fc817c16bb6dbfa
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-12 23:37:55 +00:00
William Escande
e41474eae5 Set the aptx as provided by system
Aptx and aptx-Hd are both vendor libs in system_ext.
In order to load their dependencies from a compatible environement
Bluetooth need to setup the linker

Fix: 231967310
Test: atest net_test_stack_a2dp_native
Test: Log analyze after bluetooth boot
Tag: #refactor
Ignore-AOSP-First: Apex only on tm and below
Change-Id: I752e65889a42fe3378f51bd0821c2958ed9b5a7d
2022-05-11 21:14:41 -07:00
Suren Baghdasaryan
e26561d8e8 Merge changes from topic "228160715_fix" into tm-dev am: e31c7a17e5
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/core/+/18090771

Change-Id: I58af003e94c94c6e6926c9e276d67f1d33096455
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-04 16:31:38 +00:00