Merge "Fix a typo in an error message of dexopt."

Merge "dumpstate: Log duration only if significant"

Fix a typo in an error message of dexopt.

Test: m
Change-Id: I881f772afa41ea0389e99d3c972cacdc5e93ed1a

Merge "Rename data/pkg_staging to data/app-staging"

dumpstate: Log duration only if significant

Ignore durations shorter than 500ms. Also adjust
the precision of the duration logged accordingly.

This is to avoid log spam.

Test: verified logcat on bugreport
Change-Id: Ibcef6b107de4327f769242c0c43d7f5317eee7f6

Rename data/pkg_staging to data/app-staging

Test: atest apex_e2e_tests
Bug: 126330086

(cherry picked from commit 950d5b4b8a55bda5874bfa75f3bcadf7160446a0)

Change-Id: Id98ecbad4f9452bc40a628ceb8c50ce20453d7ad
Merged-In: Id98ecbad4f9452bc40a628ceb8c50ce20453d7ad

Merge "Revert "libbinder: Choose the binder driver at runtime based on system/vendor process.""

Revert "libbinder: Choose the binder driver at runtime based on system/vendor process."

This reverts commit 46f95536bec334f8472ea92a7697c37602337398.

Reason for revert: appears to break (some?) native tests, which start looking for binder services in /dev/vndbinder

Change-Id: Id27d90e484d23a5975869729fa274213a9b1e1e4

Merge "DO NOT MERGE - Merge PPRL.190305.001 into master"

Merge "libbinder: Choose the binder driver at runtime based on system/vendor process."

libbinder: Choose the binder driver at runtime based on system/vendor process.

This is needed since llndk libraries may cause the system variant of libbinder (and therefore
compile time checks are not enough) to be loaded in a vendor process. In that case, it should not be
initing the driver with /dev/binder. It should use /dev/vndbinder instead.

Bug: 124128212

Test: Device boots
Test: play YouTube videos, use camera to take pictures / record video (sanity) on devices supporting
      legacy vndk(sailfish) and also devices supporting current vndk.

Change-Id: Ia5581efa04c8d4adc6af39668b0aa98e84324a27
Signed-off-by: Jayant Chowdhary <jchowdhary@google.com>

Merge "Dumpstate: Refactor DumpHals"

Merge "Add new emails to OWNER file"

Dumpstate: Refactor DumpHals

Prefactoring to handle user consent denial sooner.

BUG: 128270426
Test: "adb shell bugreport" for flat file bugreports works
Change-Id: Ifba9ec600ab7fe23b046414fd718220cd07a39d2

Add new emails to OWNER file

Test: builds successfully
Change-Id: I080d4cd20290f3662087ccb5034d09c43311ecfb

DO NOT MERGE - Merge PPRL.190305.001 into master

Bug: 127812889
Change-Id: I1cb1c38609ee26bb7c3e7343a9ed4c85fb4fb0a6

Merge "Log duration to logcat always."

Merge "dumpstate: move dump_traces to Dumpstate"

Log duration to logcat always.

Remove TODO and make the logging behavior intentional.

Rename field to make the intent clearer.

BUG:128270426

Test: took bugreport and verified log lines: adb logcat | grep "Duration was"
Change-Id: I6a9379af9ee6a03587c40c35f3f242d84ddf3bc3

dumpstate: move dump_traces to Dumpstate

Prefactoring to handle user consent denial sooner.

BUG: 128270426

Test: adb shell
/data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test &&
adb shell /data/nativetest64/dumpstate_test/dumpstate_test

Change-Id: Ifbace4391d22901480e41d3125d8a89508ebd52d

Merge changes from topic "b127336441"

* changes:
  libbinder_ndk: Fix 'isRemote' value for Bn*.
  libbinder_ndk: define userdata class in headers

Snap for 5240760 from 3ce2b66f92ea8c50005b38595bb1aed4c80135ed to pi-platform-release

Change-Id: I35689fd57b90f25332bd2e7219b3a72f3d386cab

libbinder_ndk: Fix 'isRemote' value for Bn*.

Bn* is not remote (it's local).

This wasn't noticed before since the AIDL generated method 'associate'
always wrapped a proxy, and since a wrapped Bn object will always have
a null remoteBinder, it would properly show that it was local even
though it was wrapped.

Bug: 127336441
Test: atest android.binder.cts
Change-Id: Ic4d96822a355e88bc8f3dbbc8fa602835653139e

libbinder_ndk: define userdata class in headers

Rather than auto-generating it for each class.

From 'struct AIBinder' comment:
                         (process boundary)
                                 |
 MyInterface ---> AIBinder_Weak  |  ProxyForMyInterface
      ^                .         |          |
      |                .         |          |
      |                v         |          v
   UserData  <---   AIBinder   <-|-      AIBinder

ICInterfaceData represents 'UserData' here.

Bug: 127336441
Test: atest android.binder.cts
Change-Id: I2ffd7533dd81f06126547e4c82ec7e1e1cb0b0d0

Merge "libbinder_ndk: support dump"

libbinder_ndk: support dump

At the NDK layer:
AIBinder_Class_setOnDump (to implement dump)
AIBinder_dump (to call dump)

At the AIDL layer:
ICInterface::dump(...) to implement and call dump

Bug: 125733384
Test: atest android.binder.cts
Change-Id: I0873ad574bb7a8b9d645c7301fdfa0797e62c834

Merge "Move PublicFormat from libandroid_runtime to libui"

Merge "ip-up-vpn: Fix resource leaks in error cases"

ip-up-vpn: Fix resource leaks in error cases

These leaks aren't really consequential since this is a standalone
command, and these "leaks" are in main().  But by putting in simple
fixes for these, we reduce the noise when using static analysis to
find leaks in Android code.

Test: TreeHugger
Change-Id: I2cbb4585c6c8d59a0eb89a72f17dfc4fbc532249

Merge "dumpstate: handle errors gracefully"

Move PublicFormat from libandroid_runtime to libui

PublicFormat is defined in android_view_Surface.
However this and related functions are also used in libmediandk
which is both NDK and LL-NDK. As a LL-NDK it should run without
libandroid_runtime.

By moving PublicFormat definition and its utility functions
into libui, LL-NDK can use it without cost of libandroid_runtime.
And also PublicFormat will be deprecated(b/126594675).

This is a part cutting dependency from libmediandk to
libandroid_runtime.

Note: applied clang-format

Bug: 124268753
Test: m -j
Change-Id: I7215dde607b425f7a0d47a6541b83fbe3a0deaf4
Merged-In: I7215dde607b425f7a0d47a6541b83fbe3a0deaf4

Merge "Ensure we select the right execution binary for all dexopt commands"

Ensure we select the right execution binary for all dexopt commands

Some devices may not have the debug version of the binaries, so even if we
are asked to use them we can't...

Unify the logic which decides what execution binary to use and apply it to
all commands (dex2oat, profman, dexoptanalyzer).

(follow up fixes to 67a14f6584d6108c29b923ce6a77c0ed5cf347dc)

Test: installd tests
Bug: 124009579
Change-Id: Iec439c0ed63e709cdf35b7f718533eafd429b9c2

Merge "Update fence event path for kernel 4.10+"

Update fence event path for kernel 4.10+

The sysfs path for controlling dma fence events changed yet again in
Linux 4.10, see kernel commit f54d1867005c3.

Test: adb shell atrace --list_categories | grep sync
Test: external/chromium-trace/systrace.py sync, check for dma_fence
      events in trace.html's event list
Change-Id: I45905cf199d5de3d5679151f19f8aa693adea5e1

dumpstate: handle errors gracefully

Return error codes instead of exiting.

Fixes: 123584708
Test: adb shell
/data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test &&
adb shell /data/nativetest64/dumpstate_test/dumpstate_test

Change-Id: I2b5fcad4e7a7369e02f0c8f5532e0fc69d45480b

Merge "Use DEX2OATBOOTCLASSPATH in installd."

Merge "Query the enable_apex_image flag from native experiment."

Use DEX2OATBOOTCLASSPATH in installd.

Bug: 119800099
Bug: 122937705
Test: dexopt_test
Change-Id: Ie31762910a51ddb1211eed6a24456f0a70379d9c

Merge "Whitelist the staging directory in installd"

Query the enable_apex_image flag from native experiment.

Bug: 119800099
Test: m
Change-Id: Ibcd90840b4342fae14307ef7f30eec96d58a698b

Merge "Rename IDumpstateListener errorcode"

Merge "Log duration reporter to logcat"

Merge "installd: parameterize user data snapshots with snapshot_id"

Rename IDumpstateListener errorcode

BUG:123571915
Test: adb shell /data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test --gtest_filter=DumpstateBinderTest.*
Change-Id: I5533f3bfed647a4068893b61dee371f5a4678650

Log duration reporter to logcat

This is to help debug bugreport generation latency
BUG: 124089395
BUG: 123315882
Test: manual

Change-Id: I65b40054d48d36c7b51b3b78feac2295266a4720

installd: parameterize user data snapshots with snapshot_id

- snapshot_id will be provided by RollbackManagerService and for a given
  rollback will be equal to its rollback_id.
- snapshotAppData now creates a folder to snapshot data to, in case it
  does not exist.
- removed ceDataInode from restoreAppDataSnapshot API, since it's only
  called if users phone is unlocked, meaning that CE data is available and
  doesn't require inode to be accessed.
- refactored installd_service_test to reduce test set up boilerplate.
- added a test to cover case when snapshots for different packages have
  the same snapshot_id.

Bug: 124029909
Test: installd_utils_test installd_service_test, RollbackTest
Change-Id: I4177a44a60cdfdc5429a28c613ce661b24482b8b
Merged-In: I4177a44a60cdfdc5429a28c613ce661b24482b8b
(cherry picked from commit b68f0d14931905472b93ad08a753a01a33bd315b)

Merge "dumpstate: Fix IO Errors during zip writing."

Merge "DO NOT MERGE - Merge PPRL.190205.001 into master"

Merge "Installd: Add more compilation-related presubmits"

Merge "Revert^2: dexopt: Fix target-sdk-version argument creation"

Whitelist the staging directory in installd

Make the path /data/pkg_staging a valid path for APK and APEX files, inorder
to allow the deletion of APEX files after their corresponding sessions
reach a final state.

Bug: 123624108
Test: manual - trigger APEX deletion from the mentioned directory
Change-Id: I9bf03c924080e8c75a6c0e9d163460d365061487

Merge changes I33f0b3a0,I772b7981

* changes:
  Make dumpstate listener callbacks synchronous
  dumpstate: enforce oneshot

Revert^2: dexopt: Fix target-sdk-version argument creation

Refactor CL I8fdf96cfe0343042d1a868ca09e6c6b9a629b967 has a typo
which leaves the target-sdk-version argument of dex2oat always empty.
Fix the issue.

This reverts commit 142836515437bec4545d982387895190b8e5c678.

Test: manual inspection of command line arguments
Test: adb shell pm compile -m speed <package>
Change-Id: I6b92e8c595e3c00d71ab13d976ee43f58d1a0013

Make dumpstate listener callbacks synchronous

Dumpstate binder service is oneshot and needs to exit in error
conditions, often right after calling onError. This can make the event
handling on client complex since there could be a race between death
recipient and onError. To make things simpler make onError synchronous.
To keep it consistent make the other callbacks synchronous as well.

Also add a new error code to signal another bugreport is running.

BUG: 123571915
Test: adb shell /data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test --gtest_filter=DumpstateBinderTest.*

Change-Id: I33f0b3a080ba493dba3521439daa6a46354a8470
(cherry picked from commit e64c3aa814b8cbd62907b8026899533a0f24c8f6)

dumpstate: enforce oneshot

Ensure the service exits after errors as well as after successful
finish as a oneshot service should.

While at it also move remove(file) to unlink(file) and fix an error
in checking the return value of unlink.

BUG: 123571915
Test: adb shell /data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test --gtest_filter=DumpstateBinderTest.*
Change-Id: I772b7981cd3b2f7c285ab980495d5539d57ebf46

Merge "Add Winscope report to the bugreport when using continuous mode"

Add Winscope report to the bugreport when using continuous mode

When logging window traces in continuous mode the window manager doesn't
continuously save data to disk.

The window traces (continuous or not) have to be present in the bug
report.

Add instruction to dump the window trace during bugreporting.

To activate continuous trace:
- adb shell cmd window tracing continuous true
- adb shell cmd window tracing start

To manually dump window trace:
 - adb shell dumpsys window trace

Test: Flash a device. Set continuous mode. Start the trace. Trigger a
bugreport. Check if the zip file contains the a wm_trace.pb under
FS/data/misc/wmtrace

Change-Id: I1af0b3b37e48ba026fa64515d7e330a2ba7663b6

Merge "Revert "dexopt: Fix target-sdk-version argument creation""

Installd: Add more compilation-related presubmits

Bug: 124771423
Test: TH
Change-Id: I878040cfe32c790b7adf0d7865cf41d60d421547

Revert "dexopt: Fix target-sdk-version argument creation"

This reverts commit d458a4f956193afdb52138606f6e61fc00b363bf.

Reason for revert: Breaks tests and manual compile

Bug: 124771423
Change-Id: Icb0fba371ffdc7464f8df21c6b92c7d7faf568f8
Test: N/A

dumpstate: Fix IO Errors during zip writing.

We need to always call FinishEntry, even when the poll times
out for whatever unrelated reason.

Bug: 124089395
Test: manual
Test: adb shell /data/nativetest64/dumpstate_test/dumpstate_test
Test: adb shell /data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test

Change-Id: I039f3b049042988e5e520771b45d0b511d11c461

Merge "Add a dalvik.vm.boot-image option for zygote and installd."

Add a dalvik.vm.boot-image option for zygote and installd.

To experiment with a non-fully AOT'ed boot image.

Bug: 119800099
Test: adb shell setprop dalvik.vm.boot-image "/system/framework/boot.art"
Change-Id: I7c5bb0bf38ffc988734e691dff90866aead3a5b7

Merge "dexopt: Fix target-sdk-version argument creation"

DO NOT MERGE - Merge PPRL.190205.001 into master

Bug: 124234733
Change-Id: Ie986006f23ae05d80cc1c69eb9fbf20c5bd5d1a7

dexopt: Fix target-sdk-version argument creation

Refactor CL I8fdf96cfe0343042d1a868ca09e6c6b9a629b967 has a typo
which leaves the target-sdk-version argument of dex2oat always empty.
Fix the issue.

Test: manual inspection of command line arguments
Change-Id: Ibeff04758dabacf4ea04fd9e54976798fcba10f1

Merge "dumpstate: format readme"
am: 0119e28e9e

Change-Id: I4d736a374a5c3405f1ecc0e85a4965c1e1f7ddd3

Merge "dumpstate: format readme"

Merge "Fix dumpstate smoke test"
am: 917b4d4392

Change-Id: Idaaf794a1a5bc487cee938c11ee913c147f0ac1a

Merge "Fix dumpstate smoke test"

Merge "libbinder_ndk: AServiceManager_checkService # apex"
am: 7d3f8168e0

Change-Id: I13c6895098e3ed908563207860dbe304c63ee681

Merge "libbinder_ndk: AServiceManager_checkService # apex"

Merge "Add xfrm stats to dumpstate"
am: 661d91e832

Change-Id: Id935abb239175464161d44f08dcd1c668a10d13e

libbinder_ndk: AServiceManager_checkService # apex

Feature parity, for faster loading times, etc..

Bug: N/A (requested for use in libnetd_resolv)
Test: ./runtests.sh, noting specifically:
[ RUN      ] NdkBinder.CheckServiceThatDoesntExist
[       OK ] NdkBinder.CheckServiceThatDoesntExist (0 ms)
[ RUN      ] NdkBinder.CheckServiceThatDoesExist
[       OK ] NdkBinder.CheckServiceThatDoesExist (0 ms)

Change-Id: I36b2a7420f5dd374e6b60898a778bbcedb9bd406

Merge "Add xfrm stats to dumpstate"

Fix dumpstate smoke test

run_main exits when finished, which is suitable if we want to run
assertions on bugreport file. Instead call an intermediate function.

Test: adb shell /data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test
Change-Id: I43708325dcaaf8f0f1ca59f181c216bfb25d28bf

Merge "dumpstate: Update readme"
am: afff99c55a

Change-Id: Id8061b2a2088d4b0d39a8c178bf1da2271991760

Merge "dumpstate: Update readme"

dumpstate: format readme

Test: n/a
Change-Id: I89359eca7a822098fd06326130ab63f6c5eb88ff

dumpstate: Update readme

Add commands to update the shared libraries for incremental development.

Test: n/a
Change-Id: I6118db5cab4ffa06dfa59e05bf7b2ca87865aeb0

Merge "DO NOT MERGE - Merge pi-platform-release (PPRL.190205.001) into stage-aosp-master" into stage-aosp-master

Merge "Call dmabuf_dump tool in dumpstate"
am: 8678bdeb41

Change-Id: I7449d290feb58fd00d61a7aa2d59c04e4d0f36de

Merge "Call dmabuf_dump tool in dumpstate"

Merge changes from topic "cut_dependency"
am: 1e29946a3e

Change-Id: Id38b33ef756ba41f51c161ef247952c387512b2c

Cut the static deps from libvulkan to libnativeloader and libnativebridge
am: 2d81f3c409

Change-Id: Ibe74e66eef1ac6fb90c9b0ead672ec65e595506c

Merge changes from topic "cut_dependency"

* changes:
  Revert "Delay start of surfaceflinger until after the Runtime APEX is mounted."
  Cut the static deps from libvulkan to libnativeloader and libnativebridge

Call dmabuf_dump tool in dumpstate

This tool will dump shared buffer information on bug reports. Since it
is being rolled out on specific products, look for the binary in
/product/bin and execute it if found.

Bug: 63860998

Change-Id: If6e4ff48d3bc62e1fa7a88f16cc6ab0d5b05176f
Signed-off-by: Erick Reyes <erickreyes@google.com>

DO NOT MERGE - Merge pi-platform-release (PPRL.190205.001) into stage-aosp-master

Bug: 124234733
Change-Id: Iff213d95a3da69f58cd95afd129323ad7613c8e4

Merge "libbinder_ndk: vendor available"
am: 658187ca59

Change-Id: I59fdae284484900e7df197c985af881edc46b108

Merge "libbinder_ndk: vendor available"

Merge "Clear BOOTCLASSPATH for APP AOT compilation."
am: 96602ece01

Change-Id: I4dfd48704a338117507c2b433ee62e2f240e0104

Merge "Clear BOOTCLASSPATH for APP AOT compilation."

Cut the static deps from libvulkan to libnativeloader and libnativebridge

The dependencies are replaced with *_lazy libraries which dynamically
load and links to the real libraries at runtime using dlopen and dlsym.

This allows libvulkan to be used by early processes that are executed
before the activation of the runtime APEX where libnativeloader and
libnativebridge are provided.

Bug: 123403798
Test: m
Test: device boots to the UI
Change-Id: Id55e7b8b0269b1f39917cf9f5c57f55218a67fc4

Revert "Delay start of surfaceflinger until after the Runtime APEX is mounted."

This reverts commit a1ce9e7a833aa4a98ab5530a3a713838df7a4803.

Reason: static deps to libnativeloader and libnativebridge no longer
exist

Bug: 123403798
Test: device boots to the UI
Test: bootanimation is shown from the very beginning

Change-Id: I40fc87e0189e410e8501837517481f87786d5503

Merge "Add missing error checks in tests."
am: 7771604bd2

Change-Id: I39eb243582146e206391da01e849f903f8a75395

Merge "Add missing error checks in tests."

libbinder_ndk: vendor available

Bug: 121279086
Test: N/A
Change-Id: Ief1a52d2b51e071728a6379713c3e694f865cec5

Merge "Convert libGLES_android to Android.bp"
am: e2c9396283

Change-Id: I12db81b790db898ab0adb4aa6c17293961397902

Merge "Convert libGLES_android to Android.bp"