Squashed: gralloc1/binder/requestor

This is a combination of the following three commits:

libui: Convert Allocator and Mapper to gralloc1

Converts GraphicBufferAllocator and GraphicBufferMapper to speak
gralloc 1.0 (via the C++ shim and optionally the 1On0 adapter) instead
of gralloc 0.x.

-----

Create graphic buffer using binder interfaces

Using binder interfaces rather than directly allocating the buffer prevents
SELinux warnings.

-----

Add requestor name to GraphicBuffer alloc metadata

Adds a requestor name (usually the BufferQueue consumer's name) to the
metadata that GraphicBufferAllocator stores on allocation so that
`dumpsys SurfaceFlinger` can attempt to attribute buffer usage to the
correct client.

Bug: 28401203
Bug: 29402015
Bug: 30776557

libui: Add Gralloc1On0Adapter and C++ shim

Adds an adapter which provides the gralloc1 interface on top of a
gralloc 0.x device as well as a C++ shim which wraps a gralloc1 device
and provides a somewhat nicer interface to the rest of the system.

This has also been squashed with a later commit that added mutexes to
protect both the buffer list and the descriptor list from concurrent
access.

Bug: 28401203
Bug: 29420918
Change-Id: I0eeafc998b56e2e2fc39de6fad41e3ed2e19658a

libagl: Switch from gralloc to GraphicBufferMapper

Removes all direct references to the gralloc module from libagl and
uses the GraphicBufferMapper class instead when locking and unlocking
buffers.

Also a couple of minor code cleanups to eliminate warnings.

Bug: 28401203
Change-Id: Ie982d375b3152d5f677ab54c2067179b8d34c06d

Merge "redirect to call assignPointerIds if there is id < 0 case"

Merge "Fix google-explicit-constructor warnings in sensorservice."

Merge "installd: Simplify Dalvik cache path creation"

Merge "Revert "Don't use profiles with vm safe mode""

Revert "Don't use profiles with vm safe mode"

Reason for reverting is that it is no longer necessary since
https://android-review.googlesource.com/#/c/269293/
has bene merged.

Bug: 30733908

Test: adb install facebook
Test: adb shell cmd package compile -m speed-profile comb.BBClient

This reverts commit f513d77b6df0268408ccba654b7c7c978efefc4f.

Change-Id: Ic7eba70c8be4e1392bd0f9eece09b1b6de56540d

Merge changes from topic 'cherry_pms_installd'

* changes:
  Otapreopt: Allow A/B dexopt only when bootcomplete
  otapreopt: Collect and print size of artifacts
  Don't use profiles with vm safe mode
  Otapreopt: Try to mount vendor partition for A/B OTA
  Otapreopt: Add init script for A/B artifact move
  Otapreopt: Implement new A/B OTA path
  Installd: Add support for dex2oat very-large-app property
  Otapreopt: Send progress updates to update_engine
  Otapreopt: Filter preopted system apps
  Installd: Move app image A/B artifact
  Installd: Fix bad merge
  Installd: Change app image name computation
  Installd: Refactor dexopt to scoped file descriptor
  Installd: Remove string-helpers
  Otapreopt: Fix after shared-library work

Otapreopt: Allow A/B dexopt only when bootcomplete

Under certain circumstances cmd may not return because it is waiting
for a service manager. Avoid this situation by restricting the otapreopt
script to only do work when the system has completed boot (which
means that a service manager must exist).

Bug: 30797145
Change-Id: Ic1445e20f1ca06fb38e7707771bde818836c3adc
(cherry picked from commit 90ba9eb2f155a4123ec4c8b08863572360d55776)

otapreopt: Collect and print size of artifacts

Collect and print the size of artifacts when moving after a reboot.
This helps in diagnosing issues after the fact.

Bug: 30832951
Change-Id: Iee410adf59dcbb74fa4b49edb27d028025cd8bf9
(cherry picked from commit e7d555267725ca050ddd5e891267a61a55c9e16a)

Don't use profiles with vm safe mode

vm safe mode uses interpret only compiler filter, this filter will
generate a full app image if a profile is also passed down. App
image generation without a profile currently has a bug where uses
library is not pruned correctly.

Bug: 30688277

Change-Id: I60d87d698789583b8ab797eb47d031c96caf6d9b
Test: N5X boot, package compile speed-profile comb.BBClient
(cherry picked from commit d46d37b8d4f81e3ea60aeb5106d6d83dd3040775)

Otapreopt: Try to mount vendor partition for A/B OTA

Vendor apps are usually not preopted, so A/B dexopt should pick
them up. update_engine is not mounting the vendor partition, so
let otapreopt_chroot do the work.

Bug: 25612095
Bug: 29498238
Change-Id: I5a77bdb78a8e478ce10f6c1d0f911a8d6686becb
(cherry picked from commit 0ba073ce81fbe544592e0a3cd526b274e83fdf9f)

Otapreopt: Add init script for A/B artifact move

Move the /data/ota -> /data/dalvik-cache move from the zygote into
a minimal shell script. Add an init rc file to execute the script
once /data is mounted. Add Makefile rules to pull these files in
with the rest of the A/B OTA dexopt components.

The move is motivated by the new naming scheme that includes the
slot name. Passing the slot name to the zygote would complicate
the process, while the value is immediately available in the script
through a simple getprop call.

Bug: 25612095
Bug: 28069686
Change-Id: I63e718d488662e1b1de2ce857629cb90aa4f611d
(cherry picked from commit 99e2b263dd0df71b55e86f0aec6657eb38b334e1)

Otapreopt: Implement new A/B OTA path

Refactor otapreopt_script, otapreopt_chroot and otapreopt such
that the actual work will be done as a child of the script driven
by update_engine.

The script now uses the new OtaDexoptService command to get the
right parameters for a dexopt call in otapreopt. As we reach
otapreopt_chroot and otapreopt directly, we can add parameters
without tainting the regular installd path. Use this to add the
target slot suffix, and make naming of the A/B artifacts include
the suffix instead of the generic "b."

Otapreopt_chroot startup is slightly complicated because all file
descriptors must be closed. Otherwise, dex2oat needs selinux
permissions to inherit the descriptors from update_engine.

Bug: 25612095
Bug: 28069686
Change-Id: I9d7b86ac8ecfdf91af3800f7e96f41534c0afc0f
(cherry picked from commit 0354bd0eaf24de533e86885b811f816f1e4d15c8)

Installd: Add support for dex2oat very-large-app property

To support constrained devices, dex2oat supports a parameter that
can downgrade a compilation to verify-at-runtime. Expose this
through the dalvik.vm.dex2oat-very-large system property.

Bug: 29557002
Change-Id: Iace626a8ec18533d37f84a41c20729fe7e400386
(cherry picked from commit e21ac2193d4ed1fd9bc4d5328625d3868409fe4d)

Otapreopt: Send progress updates to update_engine

Use the new OtaDexoptService progress API to send updates back
to update_engine.

Bug: 25612095
Bug: 29223204
Change-Id: Ie8162946d18f6fa78649a40ad5d3949d31a181cd
(cherry picked from commit 6c05a734ce443cb5effd01120c5a83b93343b6b6)

Otapreopt: Filter preopted system apps

The tool will receive requests to compile preopted system apps, as
we can't detect changes right now and need to be conservative. As
some apps cannot be stripped (signed before being preopted), they
can be re-compiled on device, producing stale files that won't be
cleaned up.

Add code to detect this situation and avoid a compile.

Bug: 25612095
Change-Id: I5f489e61b8d4e5b619cbaa01f6c59880479a46cd
(cherry picked from commit 1409090c0a92751f9fe7406a6e043cdbce2ebe4d)

Installd: Move app image A/B artifact

In addition to odex, also move art file.

Bug: 25612095
Change-Id: I69f4c2b1e7283c9a9c361f57f5233e66679c5c28
(cherry picked from commit af8948d90605ad116c965c25e73e0fc9782112ab)

Installd: Fix bad merge

Don't cut off the leading dex location character.

Bug: 25612095
Change-Id: Iad556a0bb48c57f3c68ee75ee41f14e88d96b1a8
(cherry picked from commit f065032f5331e1739fe32a2db9df8c91905720f9)

Installd: Change app image name computation

Refactor computation to take A/B OTAs into account.

Fix a const issue.

Bug: 25612095
Change-Id: I5cf79def532d1eeec4b35bb80014376959083a83
(cherry picked from commit d01a02ce6980de0024333b4673cff9a77be629cf)

Installd: Refactor dexopt to scoped file descriptor

Refactor the goto-fail cleanup to a unique_fd-like wrapper that
includes a potential cleanup step. In preparation for changes for
A/B OTA.

Bug: 25612095
Change-Id: If6cca85c12e0951bc468cb4f212b2f2d288c6041
(cherry picked from commit afa58d175cc3e9adaa4f8df9bc7a8fac5798a50a)

Installd: Remove string-helpers

Replace string-helpers with equivalent functions from android::base.

Bug: 25612095
Change-Id: I186461b94dffe3590c16455d4dee0c2cbe921bf4
(cherry picked from commit 4769ccafd6e6b94ab73afcb755fe59764c228eae)

Otapreopt: Fix after shared-library work

Commit b63d91fd2737680351876406277b6c759f4db33c broke the installd
to otapreopt connection. Fix this. Also refactor a bit to make an
inadvertant break harder.

Bug: 25612095
Change-Id: I8e7b944ecbd5457e4ac4d6ffffdb0468618cf65f
(cherry picked from commit aef445d1c659698612c0e5a8f3d5f2b9f8a2b778)

installd: Simplify Dalvik cache path creation

Installd computes Dalvik cache path for a dex file by concatenating
'/classes.dex' to the APKs absolute path and then replacing all '/'
characters with '@'. OTA preopt does the same, only in reverse order,
i.e by concatenating '@classes.dex' to altered absolute path.

This patch unifies the two approaches so as to keep only one string
constant.

Change-Id: I69bb6bca831f45c873a0eb8580cf8d4b011f3b09

Merge "Fix google-explicit-constructor warnings in inputflinger."

Merge "Fix google-explicit-constructor warnings in vulkan."

Fix google-explicit-constructor warnings in vulkan.

* Add explicit keyword to conversion constructors.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I13c32bd5bc53fe897df58eda331ff70ebcf0b349

Fix google-explicit-constructor warnings in sensorservice.

* Add explicit keyword to conversion constructors,
  or use NOLINT for implicit converters.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: Ie7c4dfcdbd069809e103f0dd008ae2fb4b0502f4

Fix google-explicit-constructor warnings in inputflinger.

* Add explicit keyword to conversion constructors.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I8259317db74b177b6e62a0aa2ad21aead47f698e

redirect to call assignPointerIds if there is id < 0 case

Symptom: If assignPointerIds() is not called, the unknown value of
outPointer.id will lead to undefined things
Root Cause: mHavePointerIds be overwritten to true in last cycle
Solution: recover mHavePointerIds=false if any id < 0, to call assignPointerIds()

https://code.google.com/p/android/issues/detail?id=221273

Change-Id: I65ca0b82a6fceb495a0de17315b1f49cdab5fba0
Signed-off-by: gaoshang <gaoshang@xiaomi.com>

Merge "Remove FRAMEBUFFER_FORCE_FORMAT"

Remove FRAMEBUFFER_FORCE_FORMAT

It's no longer used, except for a debugging line.

Change-Id: I44e1c17129e71168b263e872e9aaab0f164d3832

Merge changes from topic 'utf'

* changes:
  Add bound checks to utf16_to_utf8
  Unicode: specify destination length in utf8_to_utf16 methods

Merge "binder: use ashmem_valid()"

Merge "Fix memory ordering issues; document IMemory peculiarities"

binder: use ashmem_valid()

Bug: 30310689
Change-Id: I38e2720843f1eebec39ce6c7d957ee9a1dafac59

Add bound checks to utf16_to_utf8

Bug: 29250543
Change-Id: I518e7b2fe10aaa3f1c1987586a09b1110aff7e1a
(cherry picked from commit 7e93b2ddcb49b5365fbe1dab134ffb38e6f1c719)

Unicode: specify destination length in utf8_to_utf16 methods

Change-Id: I5223caa7d42f4582a982609a898a02043265c6d3

Fix memory ordering issues; document IMemory peculiarities

Convert to standard atomics.

Correct mHeapId memory ordering. Required acquire ordering on loads
was missing in several places.

Remove atomic updates to count, since it is only updated with lock
held. (And would be missing fences if this were not true.)

Document the peculiar use of copy-in-write vectors in a context in
which copy-on-write is unsafe.

While we're here, consistently check dup() for errors.

Bug: 28816986

Merged-in: I05b9f96e3867fa2e0abe6f319be8c56b89624c41

Change-Id: I05b9f96e3867fa2e0abe6f319be8c56b89624c41

Merge "libbinder: Support reading/writing out T[] lengths"
am: b77d368d89

Change-Id: Ibc4639f6b1a4067b4c30ca294beb9e747736d3f8

Merge "libbinder: Support reading/writing out T[] lengths"

libbinder: Support reading/writing out T[] lengths

T[] types are special: the length of the out array is written
into the inbound parcel to signal the expected length to the
server.

Bug: 30836680
Change-Id: I4f2f39fd7b1a6b94a25bd7f6c3baed430fd567bc
Test: integration tests pass with this change.

Merge "Remove unused include."
am: 5242fb42fb

Change-Id: Id3ccb5f3dfb9d2188b6e570eaaa6ba23438ae392

Merge "Remove unused include."

Remove unused include.

BUG: 29824203

Change-Id: Ied8ccb8c30632b41f6c6c3d848dc94d7b1e731e0

resolve merge conflicts of 0783d5f to stage-aosp-master

Change-Id: I4fe9c0391d6d41bae5fbbb0df6e00e264e87649f

Merge "Start converting Android.mk to Android.bp"

Start converting Android.mk to Android.bp

See build/soong/README.md for more information.

Merged-In: Ifbe9b0072a63b4467c72b3a52148a606e5365f78
Change-Id: Ifbe9b0072a63b4467c72b3a52148a606e5365f78

Merge "libsigchain: intercept bsd_signal on lp32"
am: 1c59b4b4e8

Change-Id: Ib0b9789d7a75928f7433557936b9fac81c53b049

Merge "libsigchain: intercept bsd_signal on lp32"

libsigchain: intercept bsd_signal on lp32

bsd_signal is a deprecated synonym for the signal. Nevertheless
there are apps and libraries calling this function.

libsigchain needs to intercept these calls and handle them the same
way it handles signal() call.

Bug: http://b/30562229
Test: readelf --dyn-sym surfaceflinger and check that bsd_signal is exported for lp32
      and is not exported for lp64 binary version
Change-Id: If80e45502275b26c3ee4026ab12cadd77f8fbd3b

Merge "libvulkan: Fix for -Wcast-calling-convention" into stage-aosp-master

resolve merge conflicts of 75b79d3 to stage-aosp-master

Change-Id: Ia7201fb25389688231cbcfb1ac24aeef7dec79d8

Merge \"egl: Enable EGL_KHR_GL_COLORSPACE\"
am: 5271156353

Change-Id: I2c2538852782bef9026b354ad1c0aca1e0fedbeb

Merge "egl: Call loseCurrent after eglReleaseThread"

Merge "egl: Enable EGL_KHR_GL_COLORSPACE"

Merge \"Fix a typo in the atrace help text.\"
am: 133650e5b1

Change-Id: Idc66112bedc8f6a31bd73e479bd8905a7875d360

Merge "Fix a typo in the atrace help text."

Fix a typo in the atrace help text.

Change-Id: I1b9d04d3d17e851d08a8ae39e26483e106516b88

resolve merge conflicts of f5b35b6 to stage-aosp-master

Change-Id: Ie1ac182893214022a68746f563765f6a271f8d1d

Merge "Clean up memory allocation in dumpTrace."

libvulkan: Fix for -Wcast-calling-convention

It should be harmless, as NoOp does not use any argument and should never
be reached.

Bug: 30290462
Change-Id: I0592c3ce10c510713525b5aa0bd38a2f9adc5898

Clean up memory allocation in dumpTrace.

Bug: http://b/20674092
Change-Id: Id8ba2b0efc1641929790770714e48cde545fb3b0

Merge "Dumpstate: Fix long lines." into stage-aosp-master

Merge \"Add |readString8| method which returns failure\"
am: 115e6ec7e9

Change-Id: Iaf1ea3b66ed242dcd82a88ebc0556c08a5fdf85e

Merge "Add |readString8| method which returns failure"

Add |readString8| method which returns failure

Add a new method to return |String8| read failures from Parcels.

BUG: 30189542
Change-Id: I65b59887dd9a15068abe95d24d390aa577bc49a6

Merge \"Change to include linux/android/binder.h.\"
am: b34400ea94

Change-Id: I1aab76bd51c448410d1d9ff3ae288013c81c8d52

Merge "Change to include linux/android/binder.h."

Fix unused variable build errors

Change-Id: Ia4f5dc95110b75eefc41b2828990cdbbcb75c033

Merge \"libbinder: Tighten compiler warnings/errors\"
am: ead989c34d

Change-Id: I8d06de42b9ac861b3189db936656cf906ba3597f

Merge "libbinder: Tighten compiler warnings/errors"

Disable -Wcast-calling-convention for vulkan

The following warning is triggered by the upcoming Clang update to
r271374:

frameworks/native/vulkan/libvulkan/stubhal.cpp:114:12: error: cast
between incompatible calling conventions 'cdecl' and 'aapcs-vfp'; calls
through thispointer may abort at runtime
[-Werror,-Wcast-calling-convention]
    return reinterpret_cast<PFN_vkVoidFunction>(NoOp);

Change-Id: I8de75fd6a3b579a91598aebbdd23bf74b57c3593
(cherry picked from commit b4c0eb1d203f88f0ef9ef9c360c03bcbb3e90efc)

egl: Enable EGL_KHR_GL_COLORSPACE

Add EGL_KHR_GL_COLORSPACE extension to EGL header.
Enable EGL_KHR_GL_COLORSPACE extension handling in meta EGL layer.

Change-Id: If6d3fe26c411ddc83d8b23bf16e93862cf68c1b6

Dumpstate: Fix long lines.

Follow-up to commit aff684300a3b7d6984d3b3c1efddb810cd0205e7.

Change-Id: I3faa645b0da7ea27ce3a8df2dbd3b6a50bc1c2bd
(cherry picked from commit 27cd7b256eb08bf9dec7e4e8af8375711ab10225)

Dumpstate: Use unique_fd

ScopedFd has been deprecated (and removed) in favor of unique_fd.

Bug: 21192156
Change-Id: Ic46ca6d349bf0cc640b7d4008ac540d2c6d08568
(cherry picked from commit aff684300a3b7d6984d3b3c1efddb810cd0205e7)

Merge \"Fix broken readNullableStrongBinder<T> implementation\"
am: 6e81c9a4c6

Change-Id: I4342a65c92cbc70f716b21c730f355da3f98dbbb

Merge "Fix broken readNullableStrongBinder<T> implementation"

libbinder: Tighten compiler warnings/errors

Use all of -Wall -Wextra -Werror and fix the minor errors that result.

Change-Id: Id56e9c454dd3570f713d161577eed6aa8e8074aa

Fix broken readNullableStrongBinder<T> implementation

Added a missing return statement.  Omitting this statement
caused the compiler to silently cast a pointer to a status_t,
which in general wasn't the expected value of OK.

Only return an error when the type casted return value is null and
the IBinder we got back was not null, indicating a type error, rather
than the expected null binder.

Bug: 30160959
Change-Id: Ief3c924b902a607335eff2fcd4d860cd8523593a
Test: Integration tests in aidl that exercise this method now pass.

Change to include linux/android/binder.h.

The upstream kernels have binder.h as linux/android/binder.h, so
match that.

Bug: 30072483
Change-Id: Iacdc96629dd09c1f4be0a5767666ab64b3df8f15

Merge \"Workaround alloc-dealloc-mismatch error on ASan device boot.\"
am: 7d8f246415

Change-Id: Ic34e6263619b98739f935e0aa05bf50aa7399829

Merge "Workaround alloc-dealloc-mismatch error on ASan device boot."

Workaround alloc-dealloc-mismatch error on ASan device boot.

/proc/self/environ inaccessible in SurfaceFlinger
=> ASan fails to read ASAN_OPTIONS => alloc-dealloc-mismatch bug is not
suppressed and prevents the device from booting.

This is not perfect because, by enabling it in asan build only, the first several attempts to start
surfaceflinger fail while the /data partition is being encrypted. On the other hand, this does not affect
user builds, and device eventually gets up anyway.

BUG=b/30067360

Change-Id: Ieaa37ff7768a11450ea33fee1f7d70a41b246c6a

Merge \"[mips] Resolve issue with .align directive, when using clang IAS\"
am: febb7ea6a0

Change-Id: I38969dbcd1251355cf3b7bbbf2e25a20930c28df

Merge "[mips] Resolve issue with .align directive, when using clang IAS"

[mips] Resolve issue with .align directive, when using clang IAS

Clang IAS does not support .align without args, concrete value is needed.

Change-Id: I03a9daca0be380daa744307b3510446f8d6c9b7e

Merge \"Fix misc-macro-parentheses warnings in opengl.\"
am: d4ffb636d8

Change-Id: I93f0fca53e380f0272ef10c4d38ea0033cef7cc4

Merge "Fix misc-macro-parentheses warnings in opengl."

Fix misc-macro-parentheses warnings in opengl.

Add parentheses around negative literals.
Bug: 28705665
Test: build with clang-tidy

Change-Id: I342f1d04ac08bbc28ee3ff4ebbbccbfa9f754a94

dumpstate: Also pick up /sys/fs/pstore/console-ramoops-0 for LAST KMSG
am: ced6078b0e

Change-Id: I5972646a22b4a853d5e711d5574c6776a781a5bd

Merge \\"dumpstate: Also pick up /sys/fs/pstore/console-ramoops-0 for LAST KMSG\\" am: 00e3f2995f
am: db290501a5

Change-Id: Ib628ab8c43db208fa891d1e19f483a1f6bea21e0

Merge \"dumpstate: Also pick up /sys/fs/pstore/console-ramoops-0 for LAST KMSG\"
am: 00e3f2995f

Change-Id: Ie0eed7a01655af927304b148005e9a37f9c11883

dumpstate: Also pick up /sys/fs/pstore/console-ramoops-0 for LAST KMSG

(cherry pick from commit 7d0a762ecaf9f4d005f0f6db913034c2e084d362)

Bug: 29639467
Change-Id: Iba0b1754eaebfb2b0624dcbef522b7d8456eeb50

Merge "dumpstate: Also pick up /sys/fs/pstore/console-ramoops-0 for LAST KMSG"

dumpstate: Also pick up /sys/fs/pstore/console-ramoops-0 for LAST KMSG

Bug: 29639467
Change-Id: Iba0b1754eaebfb2b0624dcbef522b7d8456eeb50

Merge \"Sensor: Make getId() more varied\" into nyc-dev
am: c0ca0ceaa5

Change-Id: Ifa6208fea63d830d07447d5232b39f9e3ccf870b

Merge "Sensor: Make getId() more varied" into nyc-dev

Sensor: Make getId() more varied

Apps from different developers will now receive a different
ID for the same dynamic sensor.  Additionally, all apps
will now receive a different/new ID for the same dynamic
sensor after a factory reset.

Bug: 28775590, 29547335
Change-Id: I368655b0e4bdc16736a7fd163ea3f7dad2886b3b