Commit graph

72671 commits

Author SHA1 Message Date
Christopher Ferris
f67a8e617f Merge "Disable scudo when using svelte config." am: be17619ccc
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1836204

Change-Id: I219350e1f5b3e255919653746f4f96d5f3a90c2a
2021-09-25 01:54:53 +00:00
Christopher Ferris
be17619ccc Merge "Disable scudo when using svelte config." 2021-09-25 01:36:48 +00:00
Treehugger Robot
8355803863 Merge "libsnapshot: Add function to verify merge sequence" am: 5125c62cf8
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1828972

Change-Id: I1fa3eba99bb5ffa119e0d8560179d98b00ac3312
2021-09-25 01:07:09 +00:00
Treehugger Robot
5125c62cf8 Merge "libsnapshot: Add function to verify merge sequence" 2021-09-25 00:50:49 +00:00
Daniel Rosenberg
1e3fa94657 libsnapshot: Add function to verify merge sequence
This adds a function that verifies that the merge sequence does not
attempt to use blocks that have already been overwritten. It prints the
first such conflict to the logs.

Additionally, this adds a forwards merge iterator, and the ability to
ignore current progress in a merge for the purposes of verifying the
whole sequence.

This can be accessed via the VerifyMergeOps Function in Cow Reader, or
by using inspect_cow -v [COWFILE]

Include merged ops in inspect_cow listing with -a, and view ops in merge
order with -n

Bug: 200076590
Test: cow_api_test.InvalidMergeOrderTets
Change-Id: I893b9f8a8803cb6dd53225ec34224167b9fe2fda
2021-09-24 16:03:17 -07:00
Marco Nelissen
437597dd98 Merge "tipc_test: allow connecting to arbitrary services" am: d394f74700
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1832713

Change-Id: I11eb2196a2020c94cdde49d2a098acd27db3819c
2021-09-24 21:56:15 +00:00
Marco Nelissen
d394f74700 Merge "tipc_test: allow connecting to arbitrary services" 2021-09-24 21:42:06 +00:00
Marco Nelissen
0aae5be54f tipc_test: allow connecting to arbitrary services
Add an option to specify a service name. Currently only the "connect"
test supports this, and will try to connect to the named service
instead of the echo and datasink services.
Also fix the built-in help text, as it was missing the "burst" option,
and didn't mention that the "connect" test connects to both the echo
and datasink services.

Bug: 200034376
Test: run

Change-Id: I3903f94c16320f8e07ac4bfb69e3bb45329138c0
2021-09-24 20:14:24 +00:00
Tri Vo
0b71744db0 Merge "storageproxyd: Fix inverted conditional in error checking" am: 6aaac7cad9
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1836533

Change-Id: I74f2c86019eafded7a8d8893a60dbb27cbc36654
2021-09-24 19:15:32 +00:00
Tri Vo
6aaac7cad9 Merge "storageproxyd: Fix inverted conditional in error checking" 2021-09-24 19:05:05 +00:00
Christopher Ferris
7aad2567ee Disable scudo when using svelte config.
This code was added, but a svelte config still tries to use scudo
related code that doesn't exist.

Bug: 201007100

Test: Ran unit tests on normal config.
Test: Ran unit tests on svelte config.
Change-Id: Ic84bae37717d213121aef182bac2f82dbee25213
2021-09-24 11:55:36 -07:00
Stephen Crane
bdb2eba4a4 storageproxyd: Fix inverted conditional in error checking
check_scci_sense (now renamed unexpected_scsi_sense) was supposed to
return false iff the sense data was valid and did not contain an error.
This return value was inverted. Invert the return values and rename the
function to clarify its role.

Test: m storageproxyd
Bug: 195544379
Change-Id: I3b84188aabe58345a83d2fd57bb8103e730cf365
2021-09-24 10:20:58 -07:00
Yi-Yo Chiang
623fb83cb2 Merge "Reland: Add /system_ext/etc/selinux/ to the debug policy search path for GSI" am: 711b56e2a0
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1834756

Change-Id: I6f9544179fea2a9685d13f067df58ad2046d1acb
2021-09-24 03:25:31 +00:00
Yi-Yo Chiang
711b56e2a0 Merge "Reland: Add /system_ext/etc/selinux/ to the debug policy search path for GSI" 2021-09-24 03:10:34 +00:00
David Anderson
8005743be4 Merge "libdm, libsnapshot: Add a virtual interface for DeviceMapper." am: 5cd741583f
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1825107

Change-Id: I5b6aa6445809c808cf4d4cac8769f7d42cfe1bac
2021-09-23 22:39:48 +00:00
David Anderson
5cd741583f Merge "libdm, libsnapshot: Add a virtual interface for DeviceMapper." 2021-09-23 22:25:46 +00:00
Elliott Hughes
c75d25d172 Merge "libdebuggerd: fix process uptime." am: c00bfdcc9a
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1832953

Change-Id: I2898dd4105171edc5d73e75e355973135f036044
2021-09-23 15:25:41 +00:00
Elliott Hughes
c00bfdcc9a Merge "libdebuggerd: fix process uptime." 2021-09-23 15:16:10 +00:00
Yi-Yo Chiang
bb77c544e5 Reland: Add /system_ext/etc/selinux/ to the debug policy search path for GSI
This change only *adds* /system_ext/etc/selinux to the debug policy
search path, and does not change any preconditions to load the debug
policy. The device still needs to be bootloader-unlocked and has the
debug ramdisk flashed to be able to use the debug policy. The only thing
changed is that now the debug policy can be loaded from /system_ext or
/debug_ramdisk when system partition is compliance testing GSI.

The debug policy in the boot ramdisk may be outdated if the system
partition is flashed with a image built from a different revision.
This happens frequently when running the compliance testing VTS, where
the device is flashed with (A) GSI and (B) device vendor image & debug
boot image, and (A) and (B) are built from different git revisions.

To address this, we install a copy of the debug policy under
/system_ext, so that the version desync between (A) & (B) wouldn't be a
problem anymore because (A) no longer relies on the debug policy file
from (B).

Bug: 188067818
Test: Flash RQ2A.201207.001 bramble-user with debug ramdisk & flash
  gsi_arm64-user from master, device can boot and `adb root` works
Change-Id: Ibe17eb1ecb0e813309c9596d1f152b7455686b26
2021-09-23 14:26:27 +00:00
Santiago Seifert
f60946a242 Merge "Revert "Add /system_ext/etc/selinux/ to the debug policy search ..."" am: 2b324f5642
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1834618

Change-Id: I8b3e938430736bbe1a42c173d167d66c397f88de
2021-09-23 13:33:47 +00:00
Santiago Seifert
2b324f5642 Merge "Revert "Add /system_ext/etc/selinux/ to the debug policy search ..."" 2021-09-23 13:25:55 +00:00
Santiago Seifert
be09a1e643 Revert "Add /system_ext/etc/selinux/ to the debug policy search ..."
Revert "Add a copy of debug policy to GSI system image"

Revert "Add PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT"

Revert "Add system_ext_userdebug_plat_sepolicy.cil for GSI"

Revert submission 1824717-gsi_debug_policy

Reason for revert: Breaks the build (see b/200933187).
Reverted Changes:
I37ef02628:Add a copy of debug policy to GSI system image
I9c3dad8bb:Add PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT
I43adc6ada:Add system_ext_userdebug_plat_sepolicy.cil for GSI...
I4d6235c73:Add /system_ext/etc/selinux/ to the debug policy s...

Change-Id: I2eb3b00abb981c25514b75b2e7b4b7b203653390
2021-09-23 13:07:25 +00:00
Yi-Yo Chiang
1e27e4b5da Merge "Add /system_ext/etc/selinux/ to the debug policy search path for GSI" am: d7f8cf483b
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1824634

Change-Id: I6107412518294026723339a5de215593c1e6a1e7
2021-09-23 10:15:17 +00:00
Yi-Yo Chiang
d7f8cf483b Merge "Add /system_ext/etc/selinux/ to the debug policy search path for GSI" 2021-09-23 10:07:10 +00:00
Marco Nelissen
dd538a6ec9 Merge "trusty_apploader: log to stderr" am: e0e160c54b
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1833093

Change-Id: I6a479aebdae6d13aecf4ebabdc592d095ca7bdba
2021-09-22 17:03:48 +00:00
Marco Nelissen
e0e160c54b Merge "trusty_apploader: log to stderr" 2021-09-22 17:01:30 +00:00
Mohammad Samiul Islam
954ab6ec55 Merge "libbinder: split out PackageManagerNative aidl" into stage-aosp-master 2021-09-22 10:28:10 +00:00
Elliott Hughes
32d3cdda22 libdebuggerd: fix process uptime.
I was here because we have a case where timeout(1) kills logcat, but
debuggerd alleges that the process that was killed had started less than
a second ago. I'm not sure this is the problem there, but I did notice
that far too many tombstones were claiming improbably short process
uptimes. It turns out that the code was measuring the *thread* uptime,
not the *process* uptime.

Also simplify the code a bit by switching to sysinfo(2) rather than
reading a file.

Test: manual, plus the existing unit test
Change-Id: Ie2810b1d5777ad9182be92bfb3f60795dc978b24
2021-09-21 17:30:30 -07:00
Marco Nelissen
92cd8891ff trusty_apploader: log to stderr
Print an error message to stderr on connection failure, so that the
command doesn't exit silently in case of the common user error of
forgetting to run it as root.

Bug: None
Test: Run as regular user, see "permission denied" message.
Change-Id: Ia213dedccab27e6152b5eb969fbee06d82d3d4ff
2021-09-21 15:51:58 -07:00
Marco Nelissen
b87e035e6a Merge "re-tab and clang-format tipc_test" am: 51fd280b8e
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1831680

Change-Id: Iba910dc3e2481b6c06a089fdf2a4f4368d4f2202
2021-09-21 14:56:59 +00:00
Marco Nelissen
51fd280b8e Merge "re-tab and clang-format tipc_test" 2021-09-21 14:42:52 +00:00
David Anderson
efec0abf4a libdm, libsnapshot: Add a virtual interface for DeviceMapper.
It is difficult to test libsnapshot failures because there is no way to
inject failure scenarios into the kernel's device-mapper. Exposing a
virtual interface will allow this kind of testing via dependency
injection.

Bug: 199785546
Test: builds, tests pass
Change-Id: I5add0c7e24f7fa2853114eaeda7c479eaa4e0552
2021-09-20 22:13:20 -07:00
Marco Nelissen
7637d25ae0 re-tab and clang-format tipc_test
tipc_test.c used a mix of tabs and spaces for indentation and other
things that clang-format doesn't like, so retab and format it.

Bug: None
Test: build
Change-Id: I9b3a64bae54697d43e7fb39fe7d6e1dabe0be380
2021-09-20 17:10:05 -07:00
Treehugger Robot
a3d7000475 Merge "bootstat: add more bootreasons" am: c91c8f88e6
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1813399

Change-Id: I7cb659bc1571000cf53bb6a74e8f54999baab759
2021-09-20 22:02:20 +00:00
Treehugger Robot
c91c8f88e6 Merge "bootstat: add more bootreasons" 2021-09-20 21:48:20 +00:00
Howie Chang
d5f24c5bb9 bootstat: add more bootreasons
add more bootreasons for new projects

Bug: 184012295
Test: trigger apc watchdog then "adb shell bootstat -p"
Change-Id: I7166969dde3675cdbe8433dbfba634a192f6be33
2021-09-19 19:11:06 +00:00
Yi-Yo Chiang
650b29d234 Add /system_ext/etc/selinux/ to the debug policy search path for GSI
This change only *adds* /system_ext/etc/selinux to the debug policy
search path, and does not change any preconditions to load the debug
policy. The device still needs to be bootloader-unlocked and has the
debug ramdisk flashed to be able to use the debug policy. The only thing
changed is that now the debug policy can be loaded from /system_ext or
/debug_ramdisk when system partition is compliance testing GSI.

The debug policy in the boot ramdisk may be outdated if the system
partition is flashed with a image built from a different revision.
This happens frequently when running the compliance testing VTS, where
the device is flashed with (A) GSI and (B) device vendor image & debug
boot image, and (A) and (B) are built from different git revisions.

To address this, we install a copy of the debug policy under
/system_ext, so that the version desync between (A) & (B) wouldn't be a
problem anymore because (A) no longer relies on the debug policy file
from (B).

Bug: 188067818
Test: Flash RQ2A.201207.001 bramble-user with debug ramdisk & flash
  gsi_arm64-user from master, device can boot and `adb root` works
Change-Id: I4d6235c73472e4d97619b2230292e6a0bc4b3e05
2021-09-18 20:21:25 +08:00
Christopher Ferris
acd0e3f8cc Merge "Add a test to verify the dex_pc is correct." am: 3275d3eeec
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1827525

Change-Id: I68f82bcb614e874ab0b23e6595fb70511bbd847e
2021-09-16 16:46:44 +00:00
Christopher Ferris
3275d3eeec Merge "Add a test to verify the dex_pc is correct." 2021-09-16 16:32:14 +00:00
Samiul Islam
abacb514f9 libbinder: split out PackageManagerNative aidl
Very few clients of libbinder use PackageManagerNative service, as such
it's a waste to couple them together. Now, user of PackageManagerNative
service need to add the corresponding aidl files as shared library.

Bug: 183654927
Test: builds + presubmit
Ignore-AOSP-First: To avoid merge conflicts, uploading it internally
first. Will be cherry-picked to AOSP later.

Change-Id: Ieca32fc3c970f2b720d76071651e85459d082f02
Merged-In: Ieca32fc3c970f2b720d76071651e85459d082f02
2021-09-16 10:41:54 +00:00
Treehugger Robot
e1e29d75eb Merge "[storageproxyd] Dump binary buffers to log instead of stderr" am: 7ea42b96e6
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1826416

Change-Id: I961297b92b53a07dda4ab7e45c4a382a003b2f6e
2021-09-16 00:39:32 +00:00
Treehugger Robot
7ea42b96e6 Merge "[storageproxyd] Dump binary buffers to log instead of stderr" 2021-09-16 00:22:34 +00:00
Christopher Ferris
2038cc7633 Add a test to verify the dex_pc is correct.
The libunwindstack code will attempt to dlopen the libdexfile.so
when a dex pc is found. Unfortunately, this failed since that
library was not properly listed as a runtime library. To make
sure this doesn't happen again, add an end to end test that
will create a dex pc frame, and will verify the correct
dex function name is in that frame.

Bug: 199043576

Test: Unit test passes on arm/aarch64/x86/x86_64.
Test: Removed the runtime_libs of libdexfile from libunwindstack
Test: and verified the new test fails.
Change-Id: I3a11f9ee44e06e37a547d193b04f7fbb90ccfe0a
2021-09-15 22:14:28 +00:00
Stephen Crane
38003165b2 [storageproxyd] Dump binary buffers to log instead of stderr
Rewrites the buffer logging to print to the Android log instead of
stderr. The storageproxyd service is started by init and stderr is not
logged anywhere. Because we want to log sense data errors under normal
operation, we need to log to logcat instead of stderr.

Test: Define RPMB_DEBUG, build for qemu-trusty, run, inspect logcat
Bug: 195544379
Change-Id: Iecc0d4680420bb92c94be628789093888a2f1941
2021-09-15 14:17:38 -07:00
Treehugger Robot
6387a16f9a Merge "Add LOCAL_LICENSE_KINDS to system/core/fs_mgr" am: c305070746
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1827516

Change-Id: Ib3cade87344093a9e08ec86721756a442ba8dca6
2021-09-15 20:50:39 +00:00
Treehugger Robot
c305070746 Merge "Add LOCAL_LICENSE_KINDS to system/core/fs_mgr" 2021-09-15 20:31:38 +00:00
Bob Badour
fe9d02890c Add LOCAL_LICENSE_KINDS to system/core/fs_mgr
Added Android-Apache-2.0 to fs_mgr/libsnapshot/scripts/Android.bp

Bug: 68860345
Bug: 151177513
Bug: 151953481

Test: m all
Change-Id: If966409962d564c1cfec901af2f2b78e891b7fca
2021-09-15 10:57:11 -07:00
Akilesh Kailash
0725099e89 Merge "libsnapshot: Error handling after QuerySnapshotStatus" am: 6f9672291c
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1824992

Change-Id: I15cad82611d8ff880ae84c8d5d57a118b95dcc49
2021-09-15 16:44:49 +00:00
Akilesh Kailash
6f9672291c Merge "libsnapshot: Error handling after QuerySnapshotStatus" 2021-09-15 16:31:33 +00:00
Colin Cross
2efb060e2b Merge "Fix compiling libprocessgroup against musl" am: c92ae83adc
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1826815

Change-Id: Idae752020bef026600c89aa076a8e832484906eb
2021-09-14 23:44:23 +00:00