Merge "Fix clang static anaylzer warnings." am: a97d58a21b am: 747d847ff1
am: 802529f46d

Change-Id: I497397026dafe63a912927cbd1d8a0ca327399f4

Merge "Reference nativehelper/jni.h as jni.h" am: 28c9177f66 am: 6ac4d2a42b
am: 9cd659c8dc

Change-Id: Iaf417e768c22335df1720b13e6583006a9a6c6c4

Merge "resolve merge conflicts of f8187f4e1 to oc-dr1-dev-plus-aosp" into oc-dr1-dev-plus-aosp

surfaceflinger: fix a nullptr dereference
am: 0a68b461d3

Change-Id: I4f449e515a8f18335d711b5e145537435dabe27f

resolve merge conflicts of f8187f4e1 to oc-dr1-dev-plus-aosp

Test: I solemnly swear I tested this conflict resolution.
Merged-In: I5f908f88923378e4d121dfd5fbc11e4f210cbb2a
Change-Id: I29912e8c2d61f7711b201deaf0e35cebb0bbf28a

Merge "Fix clang static anaylzer warnings." am: a97d58a21b
am: 747d847ff1

Change-Id: Ic74cc49d92fc6a661be461bca19b77b45e9a3a4e

Merge "Fix clang static anaylzer warnings."
am: a97d58a21b

Change-Id: Id66bd914d41190df5c3dc29968fab5b7f942851d

Merge "Reference nativehelper/jni.h as jni.h" am: 28c9177f66
am: 6ac4d2a42b

Change-Id: Id1fd8abd06cfb80ccae1304d2a3005b27ba95008

Merge "Reference nativehelper/jni.h as jni.h"
am: 28c9177f66

Change-Id: If460edb57fbd4678cfe932eb76b0931024f17f80

resolve merge conflicts of 8a094116b to stage-aosp-master
am: bb7e1b68e6

Change-Id: I453a7aaf28c20634337ac70ea6586d5a10d4736c

surfaceflinger: fix a nullptr dereference

When the child layer latched a buffer but the fixed-size parent
layer never did (for reasons such as no buffer queued or buffer got
rejected), we could end up with p->mActiveBuffer being nullptr.

Bug: 62996512
Test: manual (I could never repro anyway)
Change-Id: Id7e4c7037633b8a37039baa6e8a306e55170b894

Merge "Fix clang static anaylzer warnings."

Merge "Reference nativehelper/jni.h as jni.h"

Merge "Fix concurrent writes to the hardware composer" into oc-dr1-dev
am: 87cd11513f

Change-Id: I00d0a5267913c284eddd97bfe7eefb15b9ee403e

Merge "Fix concurrent writes to the hardware composer" into oc-dr1-dev

Fix concurrent writes to the hardware composer

Stale hardware composer layers were causing concurrent writes to the
Composer object from the surface flinger and vr flinger threads, a big
no-no. The concurrent writes would sometimes stomp on each other,
causing the hardware composer service to fail to read the command buffer
containing surface flinger's composer commands, leading to all sorts of
issues.

Bug: 62925812

Test: Locally added logs to surface flinger to catch the concurrent
writes, and confirmed the logs are no longer present with this patch
applied. Went through a bunch of sleep/wake cycles and confirmed the
device continues to function normally.

Change-Id: I70929c4a3c71142f5e9083cac294c122d127aa27

Merge "Fix memory leak." into oc-dr1-dev
am: 02b37f7594

Change-Id: If382353979a9a31c2d46fd7398ec59b086d97ac6

Merge "Fix memory leak." into oc-dr1-dev

resolve merge conflicts of 8a094116b to stage-aosp-master

Test: I solemnly swear I tested this conflict resolution.
Change-Id: I87e2955a7e44aa28eafacd44d404a23a1ae2e4a0

Merge "Update the Khronos EGL and GLES* header files."

Reference nativehelper/jni.h as jni.h

nativehelper will no longer export nativehelper/jni.h so that everywhere
can reference this file with the same name.

Bug: 63762847
Change-Id: Ifc3a3feb7578fd1352c25a3f55884ab75f337039

Update the Khronos EGL and GLES* header files.

The latest Khronos header file contents are used.  Some
not-yet-registered Android extensions are added in, along with a few
data types that were accidentally deleted from "GLES/gl.h".  Note that
Khronos rearranged some of the content.

In addition, a repeat of a 2012 fix was required to avoid ANativeWindow
being declared twice during the compilation of source files which also
include "window.h".  Both "window.h" and "native_window.h" contain the
following line:  typedef struct ANativeWindow ANativeWindow;

I tested this by building Android as well as other source that includes
"window.h", and carefully compared diff's of the old and new content.

Bug: None
Test: Built Android

Merged-In: I5f908f88923378e4d121dfd5fbc11e4f210cbb2a
Change-Id: Ic40276fc8bc0b7120a11db34c3e59ebeec9c9f97

Merge changes I23e6f088,I800208e8,I55123a7a into oc-dr1-dev
am: 607b5d1b40

Change-Id: I51b25a25bae1517f64edf9ce072244addd1b3544

Merge changes I23e6f088,I800208e8,I55123a7a into oc-dr1-dev

* changes:
  SF: Move screenshot Surface ops off main thread
  SF: Add ImageHolder for screenshot code
  SF: Add WindowDisconnector for screenshot code

Merge "Add GraphicBufferMapper::preloadHal" into oc-dr1-dev
am: 2f83627449

Change-Id: I0462d321276c9315fba82d8cc97b9889c0e5a88c

Merge "Add GraphicBufferMapper::preloadHal" into oc-dr1-dev

Fix memory leak.

A missing destructor caused a memory leak in the UDS backend for PDX.
Add the destructor and use std namespace for realloc/malloc/free.

Bug: 63026993
Test: Ran pdx_benchmarks for 400M iterations before and after fix.
      Confirmed that heap grows before fix and is stable after fix.
      Confirmed that VR service reported in the bug now have stable
      memory use.

Change-Id: If870a5db50dfaf5f212285824a1803f34acdabc7

SF: Move screenshot Surface ops off main thread

This change rearranges the various operations that correspond to
capturing a screenshot such that all of the Surface-related ones
(connect, dequeue, queue, disconnect) are performed on the incoming
Binder thread rather than on SurfaceFlinger's main thread. This has two
major benefits:

  1) It reduces the amount of time that the SurfaceFlinger main thread
     is blocked while performing a screenshot, often by a considerable
     amount. This should reduce the risk of jank when screenshots are
     taken, such as for task snapshots during window transitions.
  2) It means that the SurfaceFlinger main thread is not susceptible to
     being blocked by a badly-performing BufferQueue consumer. This
     also enables us to remove the GraphicProducerWrapper class, which
     was previously performing a similar role.

Finally, this change also adds a mechanism that detects if the
screenshot would have been performed between the two phases of normal
SurfaceFlinger operation (invalidate and refresh), and defers it if
this condition is detected. This should further reduce the risk of jank
as a screenshot will only occur between frames rather than in the
middle of a frame.

Bug: 62257775
Test: SurfaceFlinger_test and manual verification that screenshots
      still work
Change-Id: I23e6f088b4d6e477472dfc2a6c36ef3dd930c047

Merge "Fix ALOGE_IF log in bufferhubqueue" into oc-dr1-dev
am: c3e0cdb28a

Change-Id: I5ed228cd0f98a6324b799b506377df750238e549

Merge "Fix ALOGE_IF log in bufferhubqueue" into oc-dr1-dev

Merge "Add sensor HAL process into dump list" into oc-dr1-dev
am: fec340ea0f

Change-Id: I942e3fc98bea55a826e81a90581a5e09ec825742

Merge "Add sensor HAL process into dump list" into oc-dr1-dev

Merge "Add getBinderKernelReferences" am: 4bd6f567d5  -s ours am: 6a505f7ce1  -s ours
am: 5e1532a8ea  -s ours

Change-Id: I443175be02ff43cdd241b264129441c800aa6af0

Merge "Add getBinderKernelReferences" am: 4bd6f567d5  -s ours
am: 6a505f7ce1  -s ours

Change-Id: I97b5a1471fbf262765869cdf0635dc3d04f723df

Merge "Add getBinderKernelReferences"
am: 4bd6f567d5  -s ours

Change-Id: I42dcecfe9d49d60c1552bdb597a2f3c2001fd7ed

Merge "Add getBinderKernelReferences"

Add getBinderKernelReferences am: 9d45ccc057
am: e318eb1af8

Change-Id: Ia369a97ba24b80d81d81feb6c53646b9f4747424

Add getBinderKernelReferences
am: 9d45ccc057

Change-Id: I5710b544cb1ce2d07d396aa302d313324d4e3e3b

Add getBinderKernelReferences

Add a wrapper for the new BINDER_GET_NODE_DEBUG_INFO ioctl for use by
libmemunreachable.

Test: memunreachable_binder_test
Bug: 28275695
Change-Id: Ic112584fa05071bd336974b3a18869077a69389b
Merged-In: Ic112584fa05071bd336974b3a18869077a69389b
(cherry picked from commit b869cc94704d1a3d6226f471984eb33ff1bac7d5)

Add getBinderKernelReferences

Add a wrapper for the new BINDER_GET_NODE_DEBUG_INFO ioctl for use by
libmemunreachable.

Test: memunreachable_binder_test
Bug: 28275695
Change-Id: Ic112584fa05071bd336974b3a18869077a69389b
Merged-In: Ic112584fa05071bd336974b3a18869077a69389b
(cherry picked from commit b869cc94704d1a3d6226f471984eb33ff1bac7d5)

Fix ALOGE_IF log in bufferhubqueue

Bug: 63636350
Test: system builds when with TRACE=1 override
Change-Id: Icc1245191b90aafbb553a1d2b0bce3bae4d0a05a

SF: Add ImageHolder for screenshot code

Adds a simple ImageHolder class, which holds an EGLImage and destroys
it either when an explicit destroy() method is called or when the class
is destructed (whichever occurs first). This allows us to reduce the
nesting of, and otherwise simplify, the captureScreenImplLocked method
of SurfaceFlinger.

Bug: 62257775
Test: SurfaceFlinger_test and manually verified that screenshots still
      work
Change-Id: I800208e8f6bedd2ce66827268ecc64968dbe616d

SF: Add WindowDisconnector for screenshot code

Adds a simple WindowDisconnector class that holds an ANativeWindow* and
disconnects from it when it goes out of scope. This allows us to
drastically reduce the nesting of the captureScreenImplLocked method
of SurfaceFlinger.

Bug: 62257775
Test: SurfaceFlinger_test and manually verified that screenshots still
      work
Change-Id: I55123a7a6b2036158d0959328b0e6f8b206cce5d

Add sensor HAL process into dump list

Test: adb bugreport contains native stack trace for
      /vendor/bin/hw/android.hardware.sensors@1.0-service

Bug: 63096400
Change-Id: I2c24d4fd2fab7ed52dbd12b5324ae03aeaec3cda

Add GraphicBufferMapper::preloadHal

Plumbing to allow Zygote to explicitly preload the a.h.graphics.mapper
HAL. An earlier attempt did this implicitly with a static initializer in
libui.so, but the library is also used by non-Zygote processes that
don't have access to HALs. This way, the HIDL libraries and HAL are only
loaded in Zygote and its children, not random other processes.

Bug: 62353585
Test: check that a.h.graphics.mapper is in /proc/`pid zygote64`/maps
Change-Id: I4989c8089e3a4cfd627f8940f5552950c1c617fc
(cherry picked from commit 5dac7819ac9461c14f0b7c208597579c5c3fb814)

Fix clang static anaylzer warnings.

Initialize the values in readBool, readChar and readByte functions.
Silences the following warnings:
frameworks/native/libs/binder/Parcel.cpp:1856:18: warning: The left
operand of '!=' is a garbage value.
frameworks/native/libs/binder/Parcel.cpp:1869:11: warning: Assigned
value is garbage or undefined.
frameworks/native/libs/binder/Parcel.cpp:1882:11: warning: Assigned
value is garbage or undefined.

Bug: b/27101951
Test:Warnings are gone.
Change-Id: Iba8d242d4d9b1b48f7cafb901023133e21d9d732

Merge "remove extraneous `using namespace android`" am: 8e661076b6 am: b765fc92bb
am: 7dfd2ba2d6

Change-Id: If53e428ad2e3ebd2c6df0468716eb30e27a83fe2

Merge "remove extraneous `using namespace android`" am: 8e661076b6
am: b765fc92bb

Change-Id: I56c5e1f53e9351ceb85521df00f6baca3c8fa49d

Merge "remove extraneous `using namespace android`"
am: 8e661076b6

Change-Id: Ie4c4191c448b78afddba31c712c6963238d92169

Merge "remove extraneous `using namespace android`"

Merge "Synchronous resource recover  mechanism for ISensorEventConnection" into oc-dr1-dev
am: 0db05d3448

Change-Id: I21f2f66a84de9d5737e9c41c2cfff7f6295c3c12

Merge "Synchronous resource recover  mechanism for ISensorEventConnection" into oc-dr1-dev

remove extraneous `using namespace android`

This namespace is not used by this file and its declaration is removed
in a separate CL from the only header that defines it, so remove this
extraneous line.

Test: Build
Change-Id: Ibec16b8889f4d886a9f9ec865d872e2437da034b

Merge changes from topic 'svcmgr_shutdown_critical' into oc-dr1-dev
am: 5ebcaf1bb0

Change-Id: I2e51acc6d0c83fd8d3865cdae676e33111d494ee

Merge changes from topic 'svcmgr_shutdown_critical' into oc-dr1-dev

* changes:
  servicemanager: Mark servicemanager as shutdown critical
  servicemanager: Mark vndservicemanager as shutdown critical

servicemanager: Mark servicemanager as shutdown critical

Bug: 62951929, 63514363, 62845700
Test: Build
Change-Id: Ib1b7aec4b6d3c590eb469daeed40b954d37efa18

Merge "Fix BufferHub state machine to return errors on invalid transitions." into oc-dr1-dev
am: 2b7b5c34d3

Change-Id: I311d3fa978ca1a7d74bc6403be59705dd9ea995e

servicemanager: Mark vndservicemanager as shutdown critical

vndservicemanager needs to be alive during shutdown to allow vndk
components which may run during shutdown to communicate. The system
ndk servicemanager keeps running throughout shutdown, so this brings
the vndservicemanager on par with that behavior.

Bug: 62951929, 63514363, 62845700
Test: Build
Change-Id: Iba154820ef69c6491a5cb05747b6b905763a825d

Merge "Fix BufferHub state machine to return errors on invalid transitions." into oc-dr1-dev

Synchronous resource recover  mechanism for ISensorEventConnection

Add synchronous destroy() function to recover resource used by
remote ISensorEventConnection object.

Bug: 63542033
Test: SensorDirectReportTest pass
Change-Id: If98782ee12c7b1a733eb15a2fd8d7c5dacde243b

Merge "Make default permission checks in performance service more restrictive." into oc-dr1-dev
am: 9481328acc

Change-Id: I813414e808bc1f7a04ad4f457f291bffddfad43f

Merge "Fix concurrent access bug in VrFlinger." into oc-dr1-dev
am: d90e5399a4

Change-Id: I73730c662401aded83ffcae6430937a2472ae071

Merge "Make default permission checks in performance service more restrictive." into oc-dr1-dev

Merge "Fix concurrent access bug in VrFlinger." into oc-dr1-dev

Merge "Clear callbacks when destroying DvrHwcClient" into oc-dr1-dev
am: 222b760089

Change-Id: Ic4344b56bf1ea766114bf6a0faaeb0b47c064651

Merge "Clear callbacks when destroying DvrHwcClient" into oc-dr1-dev

Merge "Validate the size of all dex paths eagerly" into oc-dr1-dev
am: c3be65b533

Change-Id: I0f37ecbe7f47fc011b62f366f88d38c3cffa7908

Merge "dvrReadBufferQueueDequeue: allow empty metadata" into oc-dr1-dev
am: 3a674573f3

Change-Id: Ib9ce46939355fdb46802fe83741a6c10f3c8a4df

Merge "Validate the size of all dex paths eagerly" into oc-dr1-dev

Merge "dvrReadBufferQueueDequeue: allow empty metadata" into oc-dr1-dev

Merge "Include tracking state bits to DvrPose." into oc-dr1-dev
am: 0d346a0cb1

Change-Id: I1fe407462296a93ba23cc1d00fe95379cae21e05

Merge "Include tracking state bits to DvrPose." into oc-dr1-dev

Merge "vulkan: update Vulkan headers to 1.0.53" am: 38bcc110b8 am: d0b439e33a
am: 06ac13f0e5

Change-Id: Ie5587cd9ad0dee432ae3e50eff347a4d7d8602cd

Merge "vulkan: update Vulkan headers to 1.0.53" am: 38bcc110b8
am: d0b439e33a

Change-Id: I6f5b93b8b6a56e90a51d8779c7d76a4ac321cf9a

Merge "vulkan: update Vulkan headers to 1.0.53"
am: 38bcc110b8

Change-Id: I986ac1445697fb93fd72ab196087145a6b9945a0

Merge "vulkan: update Vulkan headers to 1.0.53"

Merge "Add target binderDriverInterfaceTest_IPC_32" am: 86c1fa9527 am: 940286e38f
am: ceb942b99f

Change-Id: If0c39eda327dc49e08ab13dc6e0da2038c6ae815

Merge "Add target binderDriverInterfaceTest_IPC_32" am: 86c1fa9527
am: 940286e38f

Change-Id: Iaefd8b3a1e9c0f4bde0b3e04342233be93eb7b18

Merge "Add target binderDriverInterfaceTest_IPC_32"
am: 86c1fa9527

Change-Id: I86a28976f575c314bcd74cb8edc18334ced9c575

Merge "Add target binderDriverInterfaceTest_IPC_32"

Add target binderDriverInterfaceTest_IPC_32

Add the target binderDriverInterfaceTest_IPC_32 which tests 32-bit
kernel and is independent from product_variables.

Bug: 62124459
Test: vts-tradefed run commandAndExit vts -m VtsKernelBinderTest
Change-Id: Ifad9ef24c35060179650955e24d1f046af7b7385

vulkan: update Vulkan headers to 1.0.53

Key changes since 1.0.49:
* Rename some extension enumerants to conform to naming style
* Add VK_EXT_sampler_filter_minmax
* Add VK_EXT_blend_operation_advanced
* Add VK_AMD_texture_gather_bias_lod
* Add VK_AMD_gpu_shader_int16
* Add VK_NV_fragment_coverage_to_color
* Add VK_NV_framebuffer_mixed_samples
* Add VK_NV_fill_rectangle

Test: build
Change-Id: I6b7931f4b7dfe4cae4fbc85fa961249534ccc11e

Merge "vulkan: update Vulkan headers to 1.0.49" am: fa3c1590b9  -s ours am: 08fa90eaab  -s ours
am: 56d1521b02  -s ours

Change-Id: I355b8cb7488b6878ec49a8d09f85512d69ad72b8

Merge "vulkan: update Vulkan headers to 1.0.49" am: fa3c1590b9  -s ours
am: 08fa90eaab  -s ours

Change-Id: Ibd45109195a20a14435e9caceb8b5197f2265164

Merge "vulkan: update Vulkan headers to 1.0.49"
am: fa3c1590b9  -s ours

Change-Id: I1d22e6ffa33b6cd4479d6b09bb9d54c67f72ebfa

Merge "vulkan: update Vulkan headers to 1.0.49"

Validate the size of all dex paths eagerly

Also, increase the PKG_PATH_MAX which is used as the max path length to
1024. This should accommodate reasonably long dex path (to cover for
longer secondary dex files paths).

If a dex path  exceeds 1024 characters the dex file will not be compiled.

Bug: 63285397
Test: manual
1) create an artifical long paths under gmscore:
2) update /data/system/package-dex-usage.list to simulate the use of those
dex file
3) run `adb shell cmd package compile -r bg-dexopt --secondary-dex
com.google.android.gms`
4) observe that a 256+ characters dex path is ok while a 1024+ long one
fails.
5) run `adb shell cmd package reconcile-secondary-dex-files
com.google.android.gms` and check no errors are present.

Change-Id: I23b34013e32c5c64ca9c1381cc4d8d67e7121cc8

Fix BufferHub state machine to return errors on invalid transitions.

The consumer side of the BufferHub flow did not have an adequate
state machine to track transitions and return errors when Acquire
or Release are requested from the wrong state. This bug allowed
other buggy usage to go unnoticed.

- Fix the consumer state machine to correctly validate all requests.
- Add tests to verify correctness of the fix.

Tested BufferHub with the new test before and after the fix and
verified that the test catches the problem and that the fix solves
the problem.

Bug: 62886596
Bug: 63149525
Test: bufferhub_tests passes.
Change-Id: I802679ed74c7f505b9243ba4048824350d4e37be

Restore PDX support to atrace.
am: a6c0a721bd

Change-Id: I39bd28bde2cad630c8891727670724df3c234ab6

Restore PDX support to atrace.

Restore PDX service tracing in atrace. This time only enable poking PDX services
when the "pdx" trace category is included.

Bug: 62882199
Test: Run systrace with/without pdx category and observe correct poke behavior.

Change-Id: I478332a661c3b8bc6061fda43f64b75d8113b305

Fix concurrent access bug in VrFlinger.

VR surface queues have erroneous concurrent access from two threads
because of misplaced lock guards. Move the locking to cover the
queue accesses.

Bug: 62886596
Bug: 63149525
Test: Extensive logging shows that buffers are accessed in the
 correct order and that the consistency of the queues is now
 maintained.

Change-Id: I649e3d75a2230cf3aff50886df9958917d3aa4bf

Clear callbacks when destroying DvrHwcClient

When a DvrHwcClient is destroyed, the VrComposer retains a reference to its
observer. Because registerObserver returns a non-OK status if an observer
is already registered, this was causing subsequent DvrHwcClients to not be
created.

This was an issue with the CompatibilityUiOverlay app on standalone
headsets. CompatibilityUiOverlay was destroying its DvrHwcClient when turning
the screen off, but upon turning the screen back on, CompatibilityUiOverlay
was unable to create another DvrHwcClient.

Bug: 63139142
Test: Turn the screen off/on for a standalone headset, observe
CompatibilityUiOverlay running.

Change-Id: Ie45e961d2bf977ab9a661e23c558354d3a96ae20

dvrReadBufferQueueDequeue: allow empty metadata

Currently, when there is no metadata on the read buffer queue, we have
to pass in dummy out_metadata; otherwise the API always return
-EINVAL. This CL allows (nullptr, 0) as legit input and add test to
cover that.

Bug: 62958474
Test: Ran the updated DvrBufferQueueTest
Change-Id: I0ad666006a5d93b67a48b9c34f27e325dfa9a419

surfaceflinger: refresh after latching any buffer
am: a36bf92663

Change-Id: If437e00a4c8aeda42f26150066a4fd7baaad485e

surfaceflinger: refresh after latching any buffer

We should not skip refresh when a latched buffer happens to have an
empty dirty region.  Also, we should signal a layer update when we
skip refresh.

Bug: 62752640
Test: manual
Change-Id: Ia603e7eeb37491c6ece7ea08d5d1e3b7ba93e6fa

Make default permission checks in performance service more restrictive.

- Check sending process in unrestricted policies.
- Allow IsTrustedUid() to cache results for better runtime efficiency.
- Update tests to verify unrestricted policy behavior.

Bug: 62468109
Test: performance_service_tests passes.
Change-Id: I0614879c8bde35ffeda2f673a3da68092433dc1a

Merge "Fix potential access to invalid memory during shutdown" into oc-dr1-dev
am: 77e9eeba69

Change-Id: I5b5703ab3b70755d3f8b8e2674c7c32ce0e1c05b

Merge "Fix potential access to invalid memory during shutdown" into oc-dr1-dev

Merge "Add dvr api entries for sensor enable/disable" into oc-dr1-dev
am: 0a9bae53bc

Change-Id: I8ad0fedbac9cf950bf6917a524cb50839b41a362

Merge "Add dvr api entries for sensor enable/disable" into oc-dr1-dev