Merge "Playback activity monitoring: add player types" into oc-dev
am: 8caf56ef7c

Change-Id: Iae04c321f57e33019861ebf60ea29256b0d63e29

Merge "Playback activity monitoring: add player types" into oc-dev

Merge "Delete vdex file in delete_odex." am: 99dedeb690 am: f6b79dd073
am: 6dc664460d

Change-Id: I0116a003cd1985e15d2beb9a60a4262549647774

Merge "Delete vdex file in delete_odex." am: 99dedeb690
am: f6b79dd073

Change-Id: I28ec2908c795bf3e88960732d69e2945b1eac422

Merge "Delete vdex file in delete_odex."
am: 99dedeb690

Change-Id: Idccc7fa241cdbb72b0ac5672978350a269ac8378

Merge "Delete vdex file in delete_odex."

Delete vdex file in delete_odex.

bug: 38513395
Test: unfortunately no unit test here, so after discussing
      with agampe@ just relied on build.

Change-Id: I305ddaadc813673fb55473f5e8ca357f39cf98f3

Merge "Add thread usage output to lshal." am: 4a3e0a9767 am: 9a130151a2
am: de314f2df2

Change-Id: I321f393d2fc7252ea6d009c743fe797b083d718a

Merge "Add thread usage output to lshal." am: 4a3e0a9767
am: 9a130151a2

Change-Id: I8045623cc47804816df7304cabea1633a18aeada

Merge "Add thread usage output to lshal."
am: 4a3e0a9767

Change-Id: Ie0a5eb6b3f1c502e9cf6c70e10354ce133985348

Merge "Add thread usage output to lshal."

Playback activity monitoring: add player types

Add player types for playback activity monitoring:
 - AAudio
 - hardware sources
 - proxy for external players

Test: n/a
Bug: 62027849
Change-Id: I17f4879dd79cc89eb7db373754a79e2e38c97457

Merge "Newly create idmap only when it is stale" into oc-dev
am: 4bf5dcce19

Change-Id: Ibafdbb1406b822a7e7f50ca229b8aefe9981b547

Merge "Newly create idmap only when it is stale" into oc-dev

libgui: fix BQ buffer allocation error checking
am: feec3b1a49

Change-Id: Ie63b629a8eb0f4c08eadbcf3a40ceaf8346a71ba

libgui: fix BQ buffer allocation error checking

The error path was broken since we moved away from
IGraphicBufferAlloc.

Bug: 37956614
Test: simulate buffer allocation failures
Change-Id: I1e1ec840fafd3075b84126c6dbf6194568175a42

Merge "SF: Call makeCurrent after presenting each display" am: 7bb8779a79 am: 5079756e8e
am: 53df8fac8b

Change-Id: I9860424ffa54807eccc6ef7ab4dbba99ed18d46a

Merge "SF: Call makeCurrent after presenting each display" am: 7bb8779a79
am: 5079756e8e

Change-Id: I6075c26cb0cd73de62f1607712c720e2d2a6248c

Merge "SF: Call makeCurrent after presenting each display"
am: 7bb8779a79

Change-Id: If22c0369b968a72dcbac8b97061f5555c18e8432

Merge "SF: Call makeCurrent after presenting each display"

Merge "Delete SurfaceFlinger BYPASS codepath" into oc-dev
am: d8ac4fd17a

Change-Id: Ieae1107f4da7b4a3eaa489175bda4bdb62e9d245

Merge "Delete SurfaceFlinger BYPASS codepath" into oc-dev

Add thread usage output to lshal.

This output shows how many threads are in use at a give time.

Test: lshal, manual
Bug: 35099601
Change-Id: I3a22bb131b828cdd77e73e7810229d9c68a496d3

Merge "libgui: Add test for OOB slot number in dequeue" into oc-dev
am: c1225bdafc

Change-Id: Ia6a3f61406928e3293423a4d8981a4ef594779ca

Merge "libgui: Add test for OOB slot number in dequeue" into oc-dev

SF: Call makeCurrent after presenting each display

HWC can change EGLContext in each present. Restore to
default EGLContext after commiting each display.

Test: see https://android-review.googlesource.com/#/c/372703/
Change-Id: I4abc84fb3d4b74964fabcad5a2568ff01a6c5dbb

Merge "Fix a anr bug caused by sendFinishedSignal logical error" am: 77bee3b429 am: 9d5bddcded
am: 251baedfaf

Change-Id: I34210fcd813c2118f0fa4814884b4adadacc415f

Merge "Fix a anr bug caused by sendFinishedSignal logical error" am: 77bee3b429
am: 9d5bddcded

Change-Id: I9ff24304c580bba54afe9b0f4bc4394d513c04cf

Merge "Fix a anr bug caused by sendFinishedSignal logical error"
am: 77bee3b429

Change-Id: If794d0ec576c91fc8554bf974d7409966b306cba

Merge "Fix a anr bug caused by sendFinishedSignal logical error"

Delete SurfaceFlinger BYPASS codepath

Test: AUPT, Manual
Change-Id: I8d6139fbb8a3ec113d7cb973420a5874d3e31030
Merged-Id: I8d6139fbb8a3ec113d7cb973420a5874d3e31030

Newly create idmap only when it is stale

For now, OverlayManagerService calls Installd.idmap() whenever a user
is changed, and then a idmap is re-generated even though there are no
changes on its target apk and overlay apk.
This CL is to avoid such unnecessary re-generation of idmap. Instead
only a group id will be updated when the idmap isn't outdated.

To correctly check staleness of idmap, "--verify" function of idmap is
invented as well.

Test: building succeeded and tested on sailfish.
Bug: 37179531
Change-Id: I874be9765d37bfa6c562c3f39a395040dc6a7d1f

Merge "Do not set the trace clock in atrace.rc." into oc-dev
am: 27ef38ca4c

Change-Id: I068e661eba9c7e668a6157dc4d704b4b73e8514c

Merge "Do not set the trace clock in atrace.rc." into oc-dev

Merge "Surface: Use buffer id to synchronize slots" into oc-dev
am: adf0ad2090

Change-Id: I01b8012c941001530678f944670a083e34b7ebc6

Merge "Surface: Use buffer id to synchronize slots" into oc-dev

Do not set the trace clock in atrace.rc.

This reverts only the atrace.rc change from commit ea826794184be368f65f099ceac4fc31234700f6.
"Set the atrace clock to the best available value: boot, mono, or global."

This change caused a 100+ms regression in boot time. We also confirm that
the clock is set to the correct value when we call atrace, so we will
still see correct behavior after reverting this change.

Bug: 38504158
Test: Manually tested that the trace clock is updated when atrace is
called.
Change-Id: Ifa302694964a4f9fc54db16e7125d5279067bbfd

Surface: Use buffer id to synchronize slots

For the same buffer, GraphicBuffer's buffer_handle_t* may change when
doing attach/detach. Use Id instead.

Test: Call attach/detach, and observe mRemovedBuffers is updated
Bug: 38238747
Change-Id: I6ab1ae5340565f98cb981b7e38c31a4f85190e56

Merge "SurfaceFlinger: setGeometryAppliesWithResize crop latching fixes." into oc-dev
am: e2fad8108e

Change-Id: I3fae02f6d1c03e69f55a05bfdf6229580ae1c4ee

Merge "SurfaceFlinger: setGeometryAppliesWithResize crop latching fixes." into oc-dev

Otapreopt: Add some static_asserts
am: ef21fd2b6b

Change-Id: I9f72d69a74a945f3c822401e466028d94330fdb3

Otapreopt: Add some static_asserts

Add some checks that indicate version issues.

Bug: 38501464
Test: m
Change-Id: Iec2043eda19126166139d7220618d11032e98af0

Fix a anr bug caused by sendFinishedSignal logical error

Because of eliminate multiple benign overflow conditions see:
https://android-review.googlesource.com/#/c/172237/
Changed the do while loop resulting in a logical difference.

When the chainIndex-- to 0, the loop is not run.
It is the most important cycle, it will push head seq to mSeqChains.
If not run, will lead to a batch of seq can not corrected finish.
Eventually leading to the occurrence of anr.

Signed-off-by: gaoshang <gaoshang@xiaomi.com>
Test: Input dispatche process
Bug: 38366215
Change-Id: I87e609dfcb00ac7b8e82c6de789df094e7c25efd

Merge "vulkan: Translate usage flags before passing to driver" into oc-dev
am: b667c05f03

Change-Id: I974356cbc64e3e52b52bb85defa2f548286ece63

Merge "vulkan: Translate usage flags before passing to driver" into oc-dev

Merge "Set the atrace clock to the best available value: boot, mono, or global." into oc-dev
am: bfe6bf6f31

Change-Id: I4b198e07cdf1742aff8a771721021ca6ba28f20a

Merge "Set the atrace clock to the best available value: boot, mono, or global." into oc-dev

vulkan: Translate usage flags before passing to driver

Previously we just cast the 32bit flags and jammed them in the two
gralloc1 slots. Use the helper which does the correct translation.

(Yes, all these gralloc1isms are going away in favor of a single 64bit
flags field, but Vulkan HALs expect us to implement our side of
ANDROID_native_buffer correctly.)

Test: boot sailfish, run vulkan demo
Bug: 37686587
Change-Id: I806011986e8140d7f608d281798605bd9ca9a86b

Merge "DO NOT MERGE Revert "Logging for catching memory issue in Parcel"" into oc-dev
am: 1967749cdc

Change-Id: I1ef9cd112030f317518c483364a9fb505a40d7bc

Merge "DO NOT MERGE Revert "Logging for catching memory issue in Parcel"" into oc-dev

Merge "vulkan: update Vulkan headers to 1.0.49" into oc-dev-plus-aosp

vulkan: update Vulkan headers to 1.0.49

Key changes since 1.0.46:
* Add VK_KHR_get_surface_capabilities2
* Add VK_KHR_shared_presentable_image
* Add VkObjectType enumeration to core API

The extensions had already been added manually in our internal tree,
so this just moves the existing declarations to the same place as in
the public header and fixes one small difference.

Test: dEQP-VK.wsi.android.surface.query_capabilities2
      dEQP-VK.wsi.android.shared_presentable_image.*
Change-Id: I1e4bd5d19f36af0d4e878b3971145a1ae1281dd6

Merge changes from topic 'move-hwc2on1' into oc-dev
am: df8ca52d13

Change-Id: I54fd4fe69160540a0f0eae3a9b6aa7a377448cd4

Merge changes from topic 'move-hwc2on1' into oc-dev

* changes:
  libhwc2on1adapter: Move to vendor partition
  Remove stale dependency to libhwc2on1adaptor.so.

Merge "Delete secondary dex profiles during reconciliation" into oc-dev
am: 74a812c587

Change-Id: Ia31c6372f66a4c1a0e574e1e85dfcc7af1307bfd

Merge "Delete secondary dex profiles during reconciliation" into oc-dev

Merge "Fix race condition in displayservice." into oc-dev
am: 02e3c25f95

Change-Id: I2497f6867fa92f1d746d24c920ba62762b11b49b

Merge "Fix race condition in displayservice." into oc-dev

Set the atrace clock to the best available value: boot, mono, or global.

This reverts commit dc34097448a76a4bb09636f25567113d7a6551dc.
"Revert "Set the atrace clock to boot when possible and mono otherwise.""

...and includes a fix for devices that break with that change. Write will
fail with an Invalid Argument exception if we programmatically write a
value to the trace_file that doesn't exist in the file. So, we'll check
for both potential values we might set. If neither of them exist, fall
back to setting to 'global', which should be safe since we were doing it
before without checking.

Bug: 32379831
Test: cts-tradefed run singleCommand cts-dev --module
CtsAtraceHostTestCases passed.
Test: Manually examining trace_clock before and after running atrace shows that the
trace_clock changes as expected (for this test I disabled atrace.rc and added an
additional debug print statement):
$ cat /d/tracing/trace_clock
[local] global counter uptime perf mono boot
$ atrace --async_start freq
capturing trace...marlin:/ $ cat /d/tracing/trace_clock
local global counter uptime perf mono [boot]
$ atrace --async_stop > /dev/null
$ cat /d/tracing/trace_clock
local global counter uptime perf mono [boot]
$ atrace --async_start freq
clock is already correct!
$ atrace --async_stop > /dev/null

Change-Id: I267056d19bcdbea58881ab2b32f093caac5f14c1

Delete secondary dex profiles during reconciliation

Test: manual inspections after adb shell dumpsys package dexopt
Bug: 38430319
Change-Id: I8401288b753a0c38ee3020d12e8e02c4a25723d2

SurfaceFlinger: setGeometryAppliesWithResize crop latching fixes.

The same sort of thing we had with setPosition...not sure why I didn't
realize we would need the fixes here too! In particular we need to ensure
the following scenarios work:

1. Additional calls to set(Final)Crop while in the setGeometryAppliesWithResize
   state are eventually applied.
2. Additional calls to set(Final)Crop while in the setGeometryAppliesWithResize
   state are not immediately applied.
3. In LayerRejector.cpp we have to be sure we are not just latching a buffer
   at the old size, which we still allow. This is the correct time to latch
   the transparentRegion as it is content dependent, but doesn't represent
   a size changing.

The difference between this and the original CL which was reverted has to do with
point 3. The original CL tried to solve point 3 by moving the latching logic from
the LayerRejecter in to Layer::doTransaction. However, in general doTransaction
will not be called in between Latching the buffer and drawing the frame, so this
introduced errors. The new test "FinalCropLatchingBufferOldSize" encapsulates this.

Bug: 37621737
Bug: 37531386
Test: Included in Transaction_test.cpp
Change-Id: I14bd09d01ac6b85895caa1b707d6fa7dac962074

DO NOT MERGE Revert "Logging for catching memory issue in Parcel"

This reverts commit c65de59762cfc284bd203d97281f5a8f86aba877.

Bug: 37298089
Change-Id: Ib42d27f91bf1408d633a98e6ac7f316175f1e289

surfaceflinger: clear HWC layers in Layer::onRemoved v2
am: 3851225fdd

Change-Id: Id7a9f7883a43b71737efcb645c53b4e81f8ded02

surfaceflinger: clear HWC layers in Layer::onRemoved v2

Explicitly destroy HWC layers in Layer::onRemoved rather than doing
that implicitly in the destructor.  The layer may be destructed by
any thread that holds a sp<Layer>, but only the main thread should
talk to the composer.

v2 fixes build breakage for non-HWC2 targets.

Bug: 38151478
Bug: 37978067
Test: manual
Change-Id: I8858fc46de1fc6facded219740b931c6486e317e

Merge "Revert "surfaceflinger: clear HWC layers in Layer::onRemoved"" into oc-dev
am: fb8cedea27

Change-Id: I3c71f6ea2f15249486398c65d77ad1c9e161b118

Merge "Revert "surfaceflinger: clear HWC layers in Layer::onRemoved"" into oc-dev

Revert "surfaceflinger: clear HWC layers in Layer::onRemoved"

Causing failures due to lack of clearHwcLayers in git_master and few
other branches/builds. Looks like clearHwcLayers needs USE_HWC2
to be defined, seems to be not defined in few branches/builds?

This reverts commit 06837418fe9fb0c6deeab7d62ac5bf65c7bb2b98.

Change-Id: Ic0e3a99b45543503d5fa8a90f3569764ebe922e3

libhwc2on1adapter: Move to vendor partition

libhwc2on1adapter is a shim layer for legacy hwcomposer HALs.  This is
only used by android.hardware.graphics.composer@2.1-impl.so.  This must
not be accessed by framework modules.  Thus, this commit moves
libhwc2on1adapter to the vendor partition and removes vendor_available.

Bug: 38302528
Test: Sailfish builds and boots.
Change-Id: I425d090fb213abc767916db3f80cff8eb4ffb876
Merged-In: I43d66c8c62f3dc90e75805f6a459573c093f9f14

Remove stale dependency to libhwc2on1adaptor.so.

This commit removes stale dependency to libhwc2on1adaptor.  Since commit
831d48ed, mAdapter has become an unused private member.  In addition,
this unused unique_ptr leads to an unnecessary symbol reference to the
destructor of HWC2on1Adaptor.  This commit removes them completely.

Bug: 38302528
Test: Sailfish builds and boots.
Change-Id: I0208d7fb21ff2f5d59d367162bc5f982f1515982
Merged-In: I479f12ddbae91282a13464159f00513aa7cb3208

Merge "Support halt the trace when schd-dbg hit deadline" am: f232542c03 am: 9d57a08f8e
am: 79453d7171  -s ours

Change-Id: Id31db457d4469e55299c63196c2e15d2d6d9c405

Merge "surfaceflinger: clear HWC layers in Layer::onRemoved" into oc-dev
am: e256ecb511

Change-Id: I3d46760b868a0b1f4db95b8ba8de4f434292ebc9

Merge "surfaceflinger: clear HWC layers in Layer::onRemoved" into oc-dev

Merge "Support halt the trace when schd-dbg hit deadline" am: f232542c03
am: 9d57a08f8e

Change-Id: I4133c2f0eb21a282385cbb05656b115e9a5b4dfa

Merge "Support halt the trace when schd-dbg hit deadline"
am: f232542c03

Change-Id: I45b92a5340d8bbc6aa0898b0a1ef073223e49cf6

Merge "Support halt the trace when schd-dbg hit deadline"

Merge "libui: flush hwbinder commands in Allocator::allocate" into oc-dev
am: 06da1581fc

Change-Id: I23fb96fa38421ff8e1788b489d927736534ea458

Merge "libui: flush hwbinder commands in Allocator::allocate" into oc-dev

Fix race condition in displayservice.

Also addressed a couple of format nits and added copyright headers.

Note, this race condition wouldn't have occured as is because the
service is only being served on one thread.

Test: no race condition still
Bug: 38311538
Change-Id: I7516c800eb89280a16468934677fa978daa5dcec

Merge "surfaceflinger: fix a release fence leak" into oc-dev
am: 5c580eaeb6

Change-Id: I84760b3e922c4a3b6ea221bd04e3d24a8d932e22

Merge "surfaceflinger: fix a release fence leak" into oc-dev

Merge "libvulkan: Enable GSC2 and SPI extensions" into oc-dev
am: d69b9d0e87

Change-Id: Ibf936d0fe5f3584a198623ecf669522b63aebd06

Merge "Surface: add unit test for GetAndFlushRemovedBuffers" into oc-dev
am: 52402901b3

Change-Id: I65921d1e49b099ecb010acd53d5310d5291fe23a

Merge "libvulkan: Enable GSC2 and SPI extensions" into oc-dev

surfaceflinger: clear HWC layers in Layer::onRemoved

Explicitly destroy HWC layers in Layer::onRemoved rather than doing
that implicitly in the destructor.  The layer may be destructed by
any thread that holds a sp<Layer>, but only the main thread should
talk to the composer.

Bug: 38151478
Bug: 37978067
Test: manual
Change-Id: If6fcfaa5a70d247b751f2b4d9aec9eca890e3aa5

Merge "Surface: add unit test for GetAndFlushRemovedBuffers" into oc-dev

surfaceflinger: fix a release fence leak

We leak release fence fds when we believe hwcomposer2 gives us any
invalid layer and return early.

Bug: 38151478
Test: boots
Change-Id: I299721eca8bc7927052170fe87ee51e9b91006aa

Merge "Add displayservice." into oc-dev
am: 4820c84197

Change-Id: I4817233aedd593dff62b4f8b4d41fa08cd4c5dd7

libgui: Add test for OOB slot number in dequeue

Adds a unit test that verifies that a malicious BnGraphicBufferProducer
cannot pass an invalid slot number to a Surface and thus cause the
Surface to segfault (or otherwise access out-of-bounds memory).

Bug: 36991414
Test: The new Malicious* tests pass after applying the security patch
Change-Id: I2083465a496cfcc8e8c3b3bc288f5f6904005ac3

Merge "Add displayservice." into oc-dev

Add displayservice.

This hidl service provides information about vsync and hotplug
to vendor services which is required by at least some camera
hal implementations.

Test: VtsFwkDisplayServiceV1_0TargetTest
Bug: 38311538
Change-Id: Ifc344c11ca4b3c8cebc6f0145f506d1aa774506d

Merge "Make the binder_lock category optional" am: 61e58d5a47 am: c4a8373ccd
am: f06f26bbb5

Change-Id: I6f9d043a8a2ac1a3a1d6a1e135fb061b94edccba

Merge "Make the binder_lock category optional" am: 61e58d5a47
am: c4a8373ccd

Change-Id: I1c8565bbc58b42522a12a2b43004fc765e20f605

Merge "Make the binder_lock category optional"
am: 61e58d5a47

Change-Id: I6fec33b45edd7665d26f4c5ad5e9c7f4f7ce7d21

Merge "Make the binder_lock category optional"

Make the binder_lock category optional

The binder_lock event type has been deprecated after binder
driver uses fine grained locking. This patch makes "binder_lock"
an optional category for atrace to support system w/ w/o FGL.

Bug: 37559846
Test: pixel phone with "atrace --async_start binder_lock"
Merged-In: I691efd355e903a26629f896a56a664502e5807f7
Change-Id: I691efd355e903a26629f896a56a664502e5807f7

Merge "Fix error BufferQueue attaching flow." into oc-dev
am: 4052245c53

Change-Id: Ifaead205c3a02ff346512af486e20212d6ffd2b1

Merge "Fix error BufferQueue attaching flow." into oc-dev

Merge "Revert "Newly create idmap only when it is outdated"" into oc-dev
am: b57d3d222a

Change-Id: Ia9e211300173e81c4d64785e19e31c3d0c7344d9

Merge "Revert "Newly create idmap only when it is outdated"" into oc-dev

Revert "Newly create idmap only when it is outdated"

This reverts commit 1d091465334784adeded4ef182cc6f732eb6f99e.

Reason for revert: Causes b/37992057

Change-Id: I7811483016b28a88d6a50bd445dd5d5e8d2e48d6