| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| | |
Change-Id: If28c8ec3d713c695d7d59cac13dd20ab0af18ef0
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
See:
https://clang.llvm.org/extra/clang-tidy/checks/bugprone-unhandled-self-assignment.html
Test: m apexd
Change-Id: I73e23a52c057f3a9973f32f566618f2b7ee6a51b
|
| |\ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Instead, now onBootstrap is responsible for finding and activating all
bootstrap apexes.
This is yet another step in cleaning interfaces to mount apexes, and
making them reusable outside of apexd.
Bug: 165948777
Test: atest ApexTestCases
Test: checked kernel logs during boot
Change-Id: Ib406c7dbae12e0fa145615044129e1d44ac00862
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Removing copy&move c-tors ensures that callers won't accidentally create
a local copy of a singletone.
Test: device boots
Test: atest ApexTestCases
Change-Id: Ib93566de8c587bb465588551d04cc5c7ebf9e9c7
|
| |\ \
| | |/
| |/| |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Test: adb logcat
Test: atest ApexTestCases
Change-Id: I2bf53a6c81e1c5698667f987e5e3264b474d8788
|
| |\ \
| | |/
| |/| |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This fixes the previous mistake that the APEX was disabled for
linux_bionic OS, which broke one of the ART builds. At the time it was
disabled with the hope that we don't build it for the new
linux_bionic_arm64 target. But we disabled it for all linux_bionic_*
targets. The ART build was using the linux_bionic_x86_64 target.
Enabling the linux_bionic_x86_64 target doesn't work because APEXes are
multilib:common modules. Enabling or disabling it for a specific
architecture is not possible.
In fact, the root cause of the problem was that the APEX build system
tried to include artifacts from all the host targets that are available.
That issue is fixed by skipping the host targets which can't be
executed natively on the host machine (i.e. HostCross == true).
Bug: 169454252
Test: HOST_CROSS_OS=linux_bionic HOST_CROSS_ARCH=arm64 m
Test: DIST_DIR=$(pwd)/out/dist \
./art/tools/dist_linux_bionic.sh com.android.support.apexer
Change-Id: If0a29fc2583883f2fb84e85fd25ae0200ee763e3
|
| |\ \
| | | |
| | | |
| | | |
| | | |
| | | | |
* changes:
Update --wait flag to --wait-for-staged-ready for staged install
Add StagedInstallInternalTest to system/apex/tests TEST_MAPPING
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 158314277
Test: atest --test-mapping system/apex/tests
Change-Id: Iafc13734ca27d1aef66096a33b7fad1a81b10a7a
Merged-In: Iafc13734ca27d1aef66096a33b7fad1a81b10a7a
(cherry picked from commit cd977532d560fd9f61844c814b2a9866284b5dde)
|
| | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Bug: 158314277
Test: atest --test-mapping system/apex/tests
Change-Id: Iafc28b6ba15287f64ffea7fa849f12073f0c9675
Merged-In: Iafc28b6ba15287f64ffea7fa849f12073f0c9675
(cherry picked from commit 6d58999c522d48f31ee8ef78a966d9b32c66e809)
|
| |\ \
| | | |
| | | |
| | | |
| | | |
| | | | |
* changes:
Refactor apexd boot logic to a new class
Handle boot completed calls from ApexManager
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This CL moves the logic to mark boot completed/ verify that the
boot has completed to a new class. The logic to check whether
apexd is booting or not is also moved to this class.
Bug: 160001168
Test: atest ApexServiceTest
Test: atest ApexRollbackTests
Change-Id: I608ca48fc5634cf3552229df913b0a2db1b586e0
Merged-In: I608ca48fc5634cf3552229df913b0a2db1b586e0
(cherry picked from commit a5ac8829ecb9d071056ded989da2b204774a8e92)
|
| | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Instead of querying the sys.boot_completed property, apexd will
instead receive a binder call upon boot completion. Upon
receiving this call, apexd will stop checking for crashing
native processes once the current checking window is completed, and
the boot completed cleanup logic will be performed.
Test: atest apexservice_test
Test: atest StagedInstallTest
Test: atest ApexRollbackTests
Bug: 158467962
Change-Id: I9f03bab40fc14ca0292ec5d619c5c5d55808cf0a
Merged-In: I9f03bab40fc14ca0292ec5d619c5c5d55808cf0a
(cherry picked from commit e35972631c57261177159c82b2534cf7fb192d12)
|
| | |
| | |
| | |
| | |
| | |
| | | |
Bug: 165948777
Test: atest ApexTestCases
Change-Id: I7af1cda8a7e1a64f262fc92b6a9b816fae081dbe
|
| |/
| |
| |
| |
| |
| | |
Test: atest ApexTestCases
Bug: 165948777
Change-Id: I20402391e132a72479f6a5947b90403c65670d8c
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
O_DIRECT does not work on squashfs file systems, and will return EINVAL. Try
without O_DIRECT.
On boards with `BOARD_SYSTEMIMAGE_FILE_SYSTEM_TYPE := squashfs` apexd will fail
to loop mount. Unfortunately O_DIRECT is not possible because image is
compressed on disk and we do need the buffering.
Bug: 167309455
Bug: 168746790
Test: tast run kukui-arc-r arc.Boot.vm
Change-Id: I9da1af535cb18cebed751ecab0e250780e2df377
|
| |\ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Android uses /dev/block/loop*, but let's also allow regular /dev/loop*.
Also, since configureReadAhead is supposed to work for all block devices,
we wouldn't need to check the prefix and simply rely on
/sys/block/${basename}/... to open successfully.
Bug: 165093975
Test: use the code to mount an apex with /dev/loop* outside of Android
Test: apex still mounts in Android
Change-Id: I1ff0b84f873c155f98bd08ded5514ef337388b9d
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
With the addition of linux_bionic_arm64 host target, this test apex
automatically gets enabled for the cross host target. However, some
dependencies of this apex like soong_zip, etc. are not available for the
cross-host target. As a result, we get the following errors:
error: system/apex/apexer/Android.bp:104:1: dependency "merge_zips" of
"com.android.support.apexer" missing variant:
os:linux_bionic, arch:arm64
available variants:
os:linux_glibc, arch:x86_64
error: system/apex/apexer/Android.bp:104:1: dependency "soong_zip"
of "com.android.support.apexer" missing variant:
os:linux_bionic, arch:arm64
available variants:
os:linux_glibc, arch:x86_64
Fix the problem by disabling this apex for targets where the tools are
not available.
Bug: 159685774
Test: HOST_CROSS_OS=linux_bionic HOST_CROSS_ARCH=arm64 m
Change-Id: Ie79b116438da3b08d3aba65391affa409a3dfa36
|
| |\ \ \
| | |/ /
| |/| | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Move temp mounting logic out of StageFnInstall, and instead
enforce that the caller must pass a vector of temp mount points,
and clean up these mount points.
Also move creation of activation points to run after unshare.
Test: atest ApexServiceTest
Test: atest ApexServicePrePostInstallTest
Bug: 158470432
Change-Id: Id824ff1a97c879efffeb3e179b0166bcba7d4a5c
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Move VerifyManifestMatches to apexd.cpp
* Move FindApexes/FindApexFilesByName to apexd_utils.h
Test: atest ApexTestCases
Bug: 165948777
Change-Id: I1983163f5170f238384d2dd1b215ce189b20ccf3
Merged-In: I1983163f5170f238384d2dd1b215ce189b20ccf3
(cherry picked from commit ac2318c4da86cf9bcffd957194ecc86016bddecb)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The only reason that dependency was there is to get a trusted public key
when validating vbmeta of an apex. This responsibility is now delegated
to the caller of ValidateApexVerity.
Test: atest ApexTestCases
Bug: 165948777
Change-Id: Ibfff81c4dfd078546d3dcf3167ad624e8028a52b
Merged-In: Ibfff81c4dfd078546d3dcf3167ad624e8028a52b
(cherry picked from commit 814ca221e8862ad9db72838a23fa673c4201d8a0)
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Storing state in a global variable inside a namespace is prone to
errors, and doesn't provide a clean api. Given that there are several
different entities that would like/already are to mount apexes, it's
time to add a cleaner api for them to use.
ApexPreinstalledData is now a non-copyable and non-moveable singletone,
with some documentation on how it's expected to be used. This is a first
cl in a sequence that tries to do a small cleanup of apexd codebase. :)
Test: presubmit
Test: atest ApexTestCases
Bug: 165948777
Change-Id: Icb26e4a7f6ed08c3702223a173ccfc399069b5d5
Merged-In: Icb26e4a7f6ed08c3702223a173ccfc399069b5d5
(cherry picked from commit 645d612e4c6fd2d3ac6e1acb6b71713cdaceb5f2)
|
| |\ \
| | |/
| |/|
| | |
| | |
| | | |
Bug: 168057903
Merged-In: Idf66ef4bfe1b40cbfe6ff8d29bcfe33b1ea2f643
Change-Id: I3e54dc92149b436d64560e20028d872d14b1b437
|
| | |\
| | |/
| |/|
| | |
| | |
| | | |
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1416210
Change-Id: Idf66ef4bfe1b40cbfe6ff8d29bcfe33b1ea2f643
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
If opening the `unique_fd` here fails, we'll call
`CloseArchive(garbage)`, which could be harmful. It looks like
OpenArchiveFd initializes `handle` to _something_ even on failure, so we
don't obviously have to move it below the `ret < 0` block.
Bug: None
Test: TreeHugger
Change-Id: I46ecfdc86264109e77699eb73474a515400d96cf
|
| | |\
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 166295507
Merged-In: Iae4263276a05739c2e3a0e691f5d3a830a07d5cf
Change-Id: I05f0b0683b0759d0f9d014e9c8cbc16967c88a26
|
| | | |\
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 51aa2b56b1 am: 839a20cff5
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1312193
Change-Id: Iae4263276a05739c2e3a0e691f5d3a830a07d5cf
|
| | | |\ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
ec5ba6c209 -s ours
am skip reason: Change-Id I813428d71674e7a8e19f2f2476b9bfa5546c7b72 with SHA-1 b29c0d3086 is in history
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1362619
Change-Id: Ice818aa80968ef4c8760f5e722f8269867945cab
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Test: presubmit
Merged-In: I813428d71674e7a8e19f2f2476b9bfa5546c7b72
Change-Id: I813428d71674e7a8e19f2f2476b9bfa5546c7b72
(cherry picked from commit 664429cc85e10450c000cf32f0062ecd0d656a58)
|
| | | |\ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1359432
Change-Id: Ib93e374aa9d84fd4e613e157cb2e5c47bf9b14c0
|
| | | |\ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
am: d17af32ff5
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1339997
Change-Id: Iffd0186513a759093bca1d61ac33ee6c1fd5df65
|
| | | |\ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1354515
Change-Id: I72a92b7b9f83f6e29ab516e927814e7f6fd80297
|
| | | |\ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1354505
Change-Id: Ieeb6e5210605993380c196ab229660bb64d4af5e
|
| | | |\ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1354504
Change-Id: I8fd4a46f46532b35f48e936056490d5226074bd5
|
| | | |\ \ \ \ \ \ \ \
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1354503
Change-Id: I30a7092524457de86f0c7e240e1a9c3c0041ea91
|
| | | |\ \ \ \ \ \ \ \ \
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1353403
Change-Id: Ia82e0f6219eada7733b01b90b0d81a09274ff2c8
|
| | | |\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
am: 490e99bac7 -s ours
am skip reason: Change-Id I77800cb121371b796b940b5792dd16ced41da0fa with SHA-1 08c223c405 is in history
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1353492
Change-Id: I60fd698d7b9d858cf27e1c997f2f81e5136b2e11
|
| | | |\ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/apex/+/12034262
Change-Id: I6b379b18d31e2008de7096376b5f72be747b7145
|
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
`deapexer extract` which uses `rdump` command of `debugfs` creates
`lost+found` directory as well. Because the directory is not necessary,
remove it after extraction.
Bug: 159711661
Test: deapexer extract <apex> <dir>
<dir> should not contain lost+found directory
Change-Id: I77800cb121371b796b940b5792dd16ced41da0fa
|
| | | |\ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
f6bd7b31e4 -s ours
am skip reason: Change-Id I522d38d0a922fd4f5cb244ba9d61bdfd3bd3cacf with SHA-1 023c507f4d is in history
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1351054
Change-Id: I78e11789361fd54b359a7b2c91c4d6df8c416d69
|
| | | |\ \ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | |
b126d6316e am: 84337bb687 -s ours
am skip reason: Change-Id I32f76ccf0b455cc1c62282223c0c23be64e71664 with SHA-1 0a45a9c212 is in history
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1351053
Change-Id: I1741422215ae23066d80c1a09f232d2d4f116e3f
|
| | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | |
am: 8c5d59b0a6 am: 7092f2fe3a -s ours
am skip reason: Change-Id I5a80da23cc90e8f8dfcb7c3b205af475a1786e99 with SHA-1 769deb1915 is in history
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1351052
Change-Id: I3b1615d7f6a89964cce586fbb22df721f1b73c3e
|
| | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | | |
5c09ecb23d -s ours
am skip reason: Change-Id I1900358743eb34282682608e04b6b9fe0f36f11c with SHA-1 8ebfc34fb8 is in history
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1351051
Change-Id: Idc8da465b9b49f4e3baf7a31f60fd4b540f820f2
|
| | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | | | |
am skip reason: Change-Id I35fd94ed65eedc5fd06479bd052099da8cef89d1 with SHA-1 e0f9f9a720 is in history
Original change: https://android-review.googlesource.com/c/platform/system/apex/+/1325818
Change-Id: I217e859fff7c08dc34bdafd46de8fbbf01fdfa02
|