Merge "No more ashmem equivalent checker false alarm" into oc-dev am: d6d91e5833
am: a8e13ef021

Change-Id: Ia83ac2bda90969378009ce9beda7acc3d576b511

Merge "No more ashmem equivalent checker false alarm" into oc-dev
am: d6d91e5833

Change-Id: I754b5d0d1e9426efa07821eba8a6ca6c826914cf

Merge "No more ashmem equivalent checker false alarm" into oc-dev

Merge "DO NOT MERGE: Eliminate redundant changes to hardware vsync state." into cw-f-dev am: 9e5585930a am: f613f073b2  -s ours am: 4c0bb0a09a
am: 9a41a3483a

Change-Id: I65d061197561958cce50bc4a34b0123a25d17687

Merge "DO NOT MERGE: Eliminate redundant changes to hardware vsync state." into cw-f-dev am: 9e5585930a am: f613f073b2  -s ours am: 4c0bb0a09a
am: 5b4bcd21ac

Change-Id: If79fbe8a291e8e14eff877c01def5bd78a0aaedd

Merge "DO NOT MERGE: Eliminate redundant changes to hardware vsync state." into cw-f-dev am: 9e5585930a am: f613f073b2  -s ours
am: 4c0bb0a09a

Change-Id: I579772728ba0f1c12ac6a611c53b0bba6e23c09b

Merge "DO NOT MERGE: Eliminate redundant changes to hardware vsync state." into cw-f-dev am: 9e5585930a am: f613f073b2  -s ours
am: 4c0bb0a09a

Change-Id: I7872b8894502b9286206f7d73563634a3ac34c3b

Merge "DO NOT MERGE: Eliminate redundant changes to hardware vsync state." into cw-f-dev am: 9e5585930a
am: f613f073b2  -s ours

Change-Id: I8a6e0a9b7e7810cfc33db9de074c2cdac139d233

Merge "DO NOT MERGE: Eliminate redundant changes to hardware vsync state." into cw-f-dev
am: 9e5585930a

Change-Id: I3b534f26c60bd8d6cf0736da7de807cf339a1cdd

Merge "DO NOT MERGE: Eliminate redundant changes to hardware vsync state." into cw-f-dev

DO NOT MERGE: Eliminate redundant changes to hardware vsync state.

When transitioning between NORMAL and DOZE power modes
we were redundantly calling resyncHardwareVsync.

Similarly, when transitioning from DOZE_SUSPEND to OFF
we were redundantly calling disableHardwareVsync.

This change eliminates those redundant calls.

Fixes bug 62235417

Change-Id: I513bbf94a7ab973ab258efe16436441ac2379b70

No crash from AHardwareBuffer_unlock on NULL fence am: ae6955fda8
am: 71b22bee5b

Change-Id: I749958788a77eba8879acdd61dcd302f60122572

No crash from AHardwareBuffer_unlock on NULL fence am: ae6955fda8
am: c197b62e97

Change-Id: Ida8d12b6ff13655683dd604d112d47048f378be3

No crash from AHardwareBuffer_unlock on NULL fence
am: ae6955fda8

Change-Id: I884ab8f0f15ad2e9e838572be638bfeee076686f

No crash from AHardwareBuffer_unlock on NULL fence
am: ae6955fda8

Change-Id: Iaedc2bbf55cf369b9c273678a0787572826b8f72

No crash from AHardwareBuffer_unlock on NULL fence

The API document states: "If fence is not NULL then it will be set to a
file descriptor that is signaled when all pending work on the buffer is
completed." As it's too late to change the document, we should probably
update the implement and use GraphicsBuffer::unlock under the hood.

Bug: 62302267
Test: Build, AHardware CTS
Change-Id: Ic0fac5d7addfba67baaa1b5d7c221fc25d02df56
(cherry picked from commit 6cdbb537105b663572e71afe06bde1164fc215f3)

Merge changes from topic 'vrflinger-cleanup'

* changes:
  libbufferhub: Fix native buffer.
  libvrflinger: Move display request to avoid duplicate calls.
  libvrflinger: Remove superfluous update to display manager.
  libvrflinger: Move some tracing to a more verbose level.
  libvrflinger: Add additional info to PDX dump.

Merge "Support custom permission, slight adjust of dumpsys print"

Merge "Read saturation boost from a persistent property"

libbufferhub: Fix native buffer.

This class was broken during a clean up. Fix the code in preparation
for adding tools that will use it again.

Bug: None
Test: build; flash
Change-Id: I8e60c7ba316080a24ff8914807972886e21ebce9

libvrflinger: Move display request to avoid duplicate calls.

Move the request display call to avoid duplicate calls when direct
surfaces change but the current VrFlinger mode does not change.

Bug: 62389162
Test: build; flash; VrFlinger mode works.
Change-Id: I1348ada93e2992306cba7e319f9b20726fb97cb7

libvrflinger: Remove superfluous update to display manager.

Setting visibility attributes already triggers an update to the
display manager. Avoid the double trigger.

Bug: None
Test: build; run
Change-Id: I4c8a7c814143ccd1a4fc22fe068ba9b05d95cdd6

libvrflinger: Move some tracing to a more verbose level.

Quiet the logs when using TRACE=1.

Bug: None
Test: Set TRACE=1; build; observe traces do not appear.
Change-Id: I521fa5cf4b0ab18e73f89a1898bf2d58fc2d1f88

libvrflinger: Add additional info to PDX dump.

- Add logging for surface and layers.
- Minor cleanup of AcquiredBuffer code.
- Add additional debug tracing.

Bug: None
Test: servicetool --dump /dev/socket/pdx/system/vr/display/client
Change-Id: Ie7cfe46978139e0eef2184b03acb4b9bab428a33

Merge "buffer_hub_queue_client: Batch allocate buffers"

Do not use default initializer for union
am: 584dfc7688

Change-Id: I328296e07f5b6e5c3f880a2b243a7fe2e297252e

Do not use default initializer for union

The next Clang update complains about this code pattern. Rewrite default
constructor to explicitly initialize the union field instead of using
the default initializer.

Test: Build
Bug: 37752547
Change-Id: I22a2aa392d7f4803282baed832b0fa2f852016ac
(cherry picked from commit 4fdbdd1692a549a31382f33f5d529fd141453ea1)

atrace: setup saved_cmdlines_size and write only if available am: 9351f72049
am: 485d70cfa4

Change-Id: I5b0d49466edc2d0824cef0f6b3cef3a5bcf66f09

atrace: setup saved_cmdlines_size and write only if available
am: 9351f72049

Change-Id: I6ac54d96cda205bc5d4e4d569b03b2064140e310

Merge "Revert "Disable in-place vdex update"" into oc-dev am: 7258088ea5
am: 0ef3b0eca2

Change-Id: I54f6eea644b392f6aefa9c07e035a1e2478ba311

atrace: setup saved_cmdlines_size and write only if available

Older kernels (bullhead etc) don't have saved_cmdlines_size, check and
write only if available. Also make work on non-root by setting
permissions in rc file (thanks John Reck for suggestion).

Bug: 62375794

Change-Id: Iefffc3203afd092ffb6cf420a144b38844aaee39
Signed-off-by: Joel Fernandes <joelaf@google.com>

Merge "Revert "Disable in-place vdex update"" into oc-dev
am: 7258088ea5

Change-Id: I3e36f53541ccac1cccd2361d4d6b752e8e881668

Merge "Revert "Disable in-place vdex update"" into oc-dev

atrace: setup saved_cmdlines_size and write only if available

Older kernels (bullhead etc) don't have saved_cmdlines_size, check and
write only if available. Also make work on non-root by setting
permissions in rc file (thanks John Reck for suggestion).

Bug: 62375794

Change-Id: Iefffc3203afd092ffb6cf420a144b38844aaee39
Signed-off-by: Joel Fernandes <joelaf@google.com>

buffer_hub_queue_client: Batch allocate buffers

Bug: 36147743
Test: buffer_hub_queue-test, buffer_hub_queue_producer-test, dvr_api-test
Change-Id: I40a9babfa8d28d4496e27a7ccecb7ae1b9bc7bd5

Merge "Add DVR API for scroll event injection."

fix race condition that can cause a use after free am: 59485525a6 am: 5da64c91d0 am: 46361c6c2b am: ee464e00d3 am: 21e9397776 am: 7b8472157a  -s ours
am: 66bc85165a

Change-Id: I977e8a53b3ff189b1740dfd612ec19edf0acc79a

fix race condition that can cause a use after free am: 59485525a6 am: 5da64c91d0 am: 46361c6c2b am: ee464e00d3 am: 21e9397776  -s ours am: 1b81145396
am: 390786a204

Change-Id: I606f740ad26e0b415149a657adfe6c2dd879c139

fix race condition that can cause a use after free am: 59485525a6 am: 5da64c91d0 am: 46361c6c2b am: ee464e00d3 am: 21e9397776
am: 7b8472157a  -s ours

Change-Id: I5289fea87abce43cb9fa7cf4cb11041b5201800d

fix race condition that can cause a use after free am: 59485525a6 am: 5da64c91d0 am: 46361c6c2b am: ee464e00d3 am: 21e9397776  -s ours
am: 1b81145396

Change-Id: I3038e41c1e4b64e397162fea3ddb80eab6374614

fix race condition that can cause a use after free am: 59485525a6 am: 5da64c91d0 am: 46361c6c2b am: ee464e00d3
am: 21e9397776  -s ours

Change-Id: If0f229ba54f2c9978ba83beca331896d1d303db4

fix race condition that can cause a use after free am: 59485525a6 am: 5da64c91d0 am: 46361c6c2b am: ee464e00d3
am: 21e9397776

Change-Id: Ifffa0ed1d960232754976ebb29b891ec99d62b75

fix race condition that can cause a use after free am: 59485525a6 am: 5da64c91d0 am: 46361c6c2b
am: ee464e00d3

Change-Id: Ifb22956eadae06f2ff6b0445d5e92e450c947942

fix race condition that can cause a use after free am: 59485525a6 am: 5da64c91d0
am: 46361c6c2b

Change-Id: I33758163baac3c96525751d313091a5bce0d5377

fix race condition that can cause a use after free am: 59485525a6
am: 5da64c91d0

Change-Id: I6358492afbd410f20a96ceccc7957a67112c58bc

fix race condition that can cause a use after free
am: 59485525a6

Change-Id: I42d7ee49dc3dcbd9d5d4a86fc9d018ff6c29534d

Add DVR API for scroll event injection.

Bug: 62226675
Test: manual, vrcore build
Change-Id: Ic7d329eba8003b0e294b3a11a83ed5f868b801f8

Merge "surfaceflinger: fix layer count" into oc-dev am: 39cf8b4944
am: 317b493667

Change-Id: Ibc092a475c8733a7a70359516e869a83573c6f5c

Merge "surfaceflinger: fix layer count" into oc-dev
am: 39cf8b4944

Change-Id: I6a7c3ebc27831792be6329ac6083c86c61f01499

Merge "surfaceflinger: fix layer count" into oc-dev

Revert "Disable in-place vdex update"

Reverting to only apply this when doing dexlayout.
PS1 is a pure revert. PS2 contains the fix.

This reverts commit e446fede9258359b61d4c8753d7e96477fa29388.

Bug: 38113240
Test: OTA and dexopt

(cherry picked from commit b03814f39559c824e097bf2d869c7a56d8b41b67)

Change-Id: Ibabb1ec3af1101c6c44fbea498f8625c9515af8f

Merge "Revert "Disable in-place vdex update"" am: 52ca987db6 am: deb23917a9 am: 3470e3e3f7
am: fd256c977a

Change-Id: I59679c67151f61644662229c0106add720299dd1

Merge "Revert "Disable in-place vdex update"" am: 52ca987db6 am: deb23917a9
am: 3470e3e3f7

Change-Id: I9b6b0b3fecb25e01b10d126e9d236658acd1e99c

Merge "Revert "Disable in-place vdex update"" am: 52ca987db6
am: deb23917a9

Change-Id: I194665350fb6416acc1158fdfc125d6867cbf588

Merge "Revert "Disable in-place vdex update""
am: 52ca987db6

Change-Id: Id4060442d160fb91b8245388c1a45f442ac20a89

Merge "Revert "Disable in-place vdex update""

Revert "Disable in-place vdex update"

Reverting to only apply this when doing dexlayout.
PS1 is a pure revert. PS2 contains the fix.

This reverts commit e446fede9258359b61d4c8753d7e96477fa29388.

Change-Id: I04a7dbde142d59e8dcdf4e93ab0456c4aa3d02ab

Support custom permission, slight adjust of dumpsys print

Test: tested permission positive and negative case
Test: dumpsys print as expected
Change-Id: I916222fe4cd05ac5268b190fc250e8d51943114d

Read saturation boost from a persistent property

We use a persisten property because Android's various settings
stores are not available early enough to avoid a color change
during the boot animation.

Bug: 62238038
Test: Manual
Change-Id: I905435efe89b5e1c85fc8a396b7888de5c318a18

fix race condition that can cause a use after free

Backported from 12a0ccd6f7201bac706d903ac3f436c4358fe203.

Bug: 33004354
Test: manual
Change-Id: I9b38ee644b02268c9b995a330db758aa2e568399

Merge "Let the system process set the saturation boost"

Merge "Clockwork: add feature android.software.input_methods" into oc-dev am: eed865e61b
am: 73931bf207

Change-Id: Iabce8207d6ef63f2097c9a18fbafe26b85473f99

Merge "Clockwork: add feature android.software.input_methods" into oc-dev
am: eed865e61b

Change-Id: I8698f6f42fcbe2239493c6e03d8f850687c1de8c

Merge "Clockwork: add feature android.software.input_methods" into oc-dev

Merge "atrace: Enable sched_waking if available" into oc-dev am: 4b37107002
am: 5d2e06c8a7

Change-Id: I6f9dd1f9299cbb7c92108e2964f3d19f3dc71f84

atrace: Set saved_cmdline_size to 8192 am: efb73a96b5
am: be83a34f7d

Change-Id: I062fccf47b3e549a125810e40bf7d28dab19f061

Let the system process set the saturation boost

The display manager service will need to call this code early
in the boot process, but before the hardware test calling permission
check can pass (it depends on the order of initalization of other
services like sensors).

Bug: 62238038
Test: Manual (modified display manager service)
Change-Id: I46a673ba971e566a0b9e023005c41fa54603cd1f

Merge "atrace: Enable sched_waking if available" into oc-dev
am: 4b37107002

Change-Id: I961c608a96040cfd87bd55249617c715cb6d6488

atrace: Set saved_cmdline_size to 8192
am: efb73a96b5

Change-Id: I96758a6d56d5478b3287ead6c3d4e43fb7e61524

Merge "atrace: Enable sched_waking if available" into oc-dev

atrace: Set saved_cmdline_size to 8192

This increases the size of the table tracking comms and pids and
prevents overwriting them easily.

Bug: 38167231

Change-Id: I69b9000e4f9d0c2c7969efd38e02d8c1bffdad5c
Signed-off-by: Joel Fernandes <joelaf@google.com>
(cherry picked from commit ed80bd04cc3bf80082b1d733ef06d90a29c6270a)

atrace: Enable sched_waking if available

sched_waking helps to know when a wakeup happens at the earliest.
wakeups can originate on different CPUs which makes it impossible to
know just from sched_wakeup what the source was. Thankfully sched_waking
is just for that, enable it if its available.

Bug: 36415990

Change-Id: I489ca0112d5e0ed781888866a9efa57f17ca8a53
Signed-off-by: Joel Fernandes <joelaf@google.com>

atrace: Enable sched_waking if available

sched_waking helps to know when a wakeup happens at the earliest.
wakeups can originate on different CPUs which makes it impossible to
know just from sched_wakeup what the source was. Thankfully sched_waking
is just for that, enable it if its available.

Bug: 36415990

Change-Id: I489ca0112d5e0ed781888866a9efa57f17ca8a53
Signed-off-by: Joel Fernandes <joelaf@google.com>

surfaceflinger: fix layer count

When a layer is removed, mLayerCount is always decremented by 1 even
though the layer has children.  This commit fixes that by making
sure mLayerCount is properly decremented, and makes sure that a
parent layer must be a valid layer in mCurrentState.

Bug: 37121786
Test: manually open and close Camera app
Change-Id: I5a3e18be41646b167b4d361dfe08db4d06e1c2d0
(cherry picked from commit 98f1c108b016dda77c808f3d099e7a45bdd70768)

Merge "Add media.codec process to native stack dump list" into oc-dev am: eb30998a33
am: edde8a79f0

Change-Id: I944e3e094aac18ebb859cc2e900ccf8b162357b4

Merge "Setting queue metadata size via dvrSurface API"

Merge "Add media.codec process to native stack dump list" into oc-dev
am: eb30998a33

Change-Id: I8de97da46705f876848bfa939f05dc31f82389f2

Merge "Add media.codec process to native stack dump list" into oc-dev

Merge "No crash from AHardwareBuffer_unlock on NULL fence"

Add media.codec process to native stack dump list

This is needed for non-treble devices.

Test: do 'adb shell bugreport' on angler, and check that
'Cmd line: media.codec' section is present and stacks are there.

bug: 62297059
Change-Id: I2573bb48d021e7750adf44eb00f06f5b9efb3a85

Setting queue metadata size via dvrSurface API

This allows VRCore to use metadata feature on each individual DvrBufferQueue.

Bug: 62301995
Test: build, dvr_api-test
Change-Id: I8e0eb8ca1a2c197426c4853cae99a4ed63caeffe

No crash from AHardwareBuffer_unlock on NULL fence

The API document states: "If fence is not NULL then it will be set to a
file descriptor that is signaled when all pending work on the buffer is
completed." As it's too late to change the document, we should probably
update the implement and use GraphicsBuffer::unlock under the hood.

Bug: 62302267
Test: Build, AHardware CTS
Change-Id: Ic0fac5d7addfba67baaa1b5d7c221fc25d02df56

Merge "Refactor how legacy dataspaces are decoded"

Merge changes I87a57825,I3721d163 into oc-dev am: e44ea13766
am: 25bc953ae3

Change-Id: I9234764805e70a2e812520f0b02fa63fd4e70580

Merge changes I87a57825,I3721d163 into oc-dev
am: e44ea13766

Change-Id: I258fd8dd5fd03975a28c606dcd022ef506554fb6

Merge changes I87a57825,I3721d163 into oc-dev

* changes:
  SurfaceFlinger: Fix final cropping of child layers.
  SurfaceFlinger: Update parent pointer while performing transaction.

Merge "libpdx: Fix bug in Variant type."

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY" am: 9e94328e22 am: ce9203d628 am: d396aa6869
am: 5885883145

Change-Id: I89569cf2f3602cba8bb73dfca9a02e99d49fa33b

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY" am: 9e94328e22 am: ce9203d628
am: d396aa6869

Change-Id: Ibffb945b0db31ac1a2ff053595e82476cc8a3fa4

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY" am: 9e94328e22
am: ce9203d628

Change-Id: I51e54f6e11287c98ff684616be91efcbfc7b2e1e

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY"
am: 9e94328e22

Change-Id: I993c5680157f45e1912e7a24ae5ee1d6cd652f92

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY"

Merge "atrace: Set saved_cmdline_size to 8192"

Merge "Add saturation boost feature to SurfaceFlinger"

SurfaceFlinger: Fix final cropping of child layers.

Straightforward mistake documented by the test case. Not sure how it
made it this long.

Test: Included in Transaction_test.cpp. Manual from bug. go/wm-smoke.
Bug: 37511473
Change-Id: I87a57825cca879c5519fe72fef199168f7c23479

SurfaceFlinger: Update parent pointer while performing transaction.

To understand the change at a high level, notice that all the usage
of getParent() is assosciated with the drawing state. We see in this way
that the parent is a part of the drawing state and should only be updated
when transactions occurs.

More specifically we can consider the following scenario:

1. Imagine that we have a visible surface, with a visible child surface
2. Now imagine we create a new surface, which is hidden, and post a buffer to it.
   It is configured such that it would be visible if it were not hidden.
3. We open a transaction
4. We reparent the child from the old surface to the new one
5. We show the new surface
6. We close the transaction

At this point we would expect the child to remain visible, as it is atomically
reparented with the visibility change of the parent. However prior to this CL
we see a flash, as the sequence can continue as follows:

7. Closing the transaction triggers setClientState, causing Layer::reparentChildren
   which would call setParent updating the parent pointer.
8. setClientState updates the current but not drawing visibility of the new parent and marks
   a transaction to occur.
9. We return to the main thread, MESSAGE_REFRESH occurs before MESSAGE_TRANSACTION
   and we render a frame
10. We observe the child window as invisible, as it has the new parent
    but it's new parent is not yet visible.

We simply have to ensure the parent pointer is updated at transaction time.
I chose a location in commitChildList where mDrawingChildren was copied
from mCurrentChildren as it seemed to express the intent well.

Test: Difficult to automatically test. Manually try resizing Chrome/Youtube/etc in docked stack before flicker should be ~1/10 after flicker should be <1/100 (I haven't seen one yet)
Bug: 62099658
Change-Id: I3721d16361b53128146510d3fda43e1c12f93223

libpdx: Fix bug in Variant type.

Fix a subtle bug in the Variant value destruction along the
EmptyVariant assignment path:

    some_variant = EmptyVariant{};

The problem arises from the private utility method Destruct(),
which does not set the type index to "empty" after destroying the
current sub-element. For most paths this is okay because the type
index is immediately set to the new sub-element type. However, the
EmptyVariant path does not assign a new type because the Variant
should become empty. Since the type is not set to "empty" the
Variant incorrectly double destroys the previous value when a new
value is assigned.

Also clean up a superfluous overload of Assign() that is skipped
due to the stronger binding of the universal reference overload
Assign(T&&).

Update tests to properly cover this case. In the process discovered
two incorrect tests related to this issue and updated them.

Bug: None
Test: pdx_tests passes.
Change-Id: I106f863b34f2719820d04d0e701968332f659c3e

Refactor how legacy dataspaces are decoded

Test: adb shell dumpsys SurfaceFlinger
Observe that (deprecated) sRGB is decoded similar to how
newer dataspaces are decoded. Previous decoding added
extra bits in description that weren't helping.

Change-Id: Ic95a3a766f164f978d778d999f47252b83f79016

Remove redundent layer setDataspace calls.

Cache the layer's dataspace in hwc2 and only send
to the HAL when it changes. Allows HAL to optimize
use of layer.

Bug: 38396128
Test: Manually checked systraces on Pixel
Change-Id: Ia94bebe7de9f79d01ae19b9917591837d865fea9

atrace: Set saved_cmdline_size to 8192

This increases the size of the table tracking comms and pids and
prevents overwriting them easily.

Bug: 38167231

Change-Id: I69b9000e4f9d0c2c7969efd38e02d8c1bffdad5c
Signed-off-by: Joel Fernandes <joelaf@google.com>