Merge changes from topic 'sensor_looper_crash' into oc-dev am: 638055ef41
am: 91768a2bfe

Change-Id: Ib1eabf66f2658de1adca4133c20fb4b9fdc71526

SensorManager: attach background thread to JavaVM am: 09034f076e
am: fc045b9de4

Change-Id: I2a0ddc2040000e6c7cc27d70812f9429fcea6c61

Framework sensor manager init not depends on ::android::SensorManager. am: 542eace6bf  -s ours
am: 401b0f43ee  -s ours

Change-Id: Ib932c565af2ee3d968595b265a259f988fca57a0

Merge changes from topic 'sensor_looper_crash' into oc-dev
am: 638055ef41

Change-Id: I774229e9c84e7b390861638e478e2330b66c3d57

SensorManager: attach background thread to JavaVM
am: 09034f076e

Change-Id: I07364b649a58f9ca0891407540a607967155bf4b

Framework sensor manager init not depends on ::android::SensorManager.
am: 542eace6bf  -s ours

Change-Id: I846d806f4d135a8c353f10ea514a3cdedd22cd85

Merge changes from topic 'sensor_looper_crash' into oc-dev

* changes:
  Fix race condition for SensorManager looper.
  SensorManager: attach background thread to JavaVM
  Framework sensor manager init not depends on ::android::SensorManager.

Fix race condition for SensorManager looper.

Before the fix there could be two background threads started for two
loopers. Add a guard to prevent this from happening.

Test: pass
Bug: 62404135
Change-Id: I8230addaaa6b83670d7b83c96809b00fca95d352

SensorManager: attach background thread to JavaVM

Test: pass
Bug: 62404135

Change-Id: I63a128c3c933904211f5ff2483492674eaba93ed

Framework sensor manager init not depends on ::android::SensorManager.

Test: photo sphere
Change-Id: Ice37a758421827757226a9898bcf3e00857443ff
Merged-In: Ice37a758421827757226a9898bcf3e00857443ff
Fixes: 36448245

Merge "Resurrect the pose tool." into oc-dr1-dev
am: 5af190c517

Change-Id: I8af4bac4553638fb4a2a1994e9d590e96f796a51

Merge "Resurrect the pose tool." into oc-dr1-dev

Merge "atrace.rc: Correct path to saved_cmdlines_size when setting permission am: 922e418f45 am: 83e73f35d6" into oc-dr1-dev-plus-aosp

atrace.rc: Correct path to saved_cmdlines_size when setting permission am: 922e418f45
am: bf16598142

Change-Id: Ia489f46bc7be10fc877d709eedc70ca49be53c3b

atrace.rc: Correct path to saved_cmdlines_size when setting permission am: 922e418f45
am: 83e73f35d6

Change-Id: I0e081393abcb36b99ced7000a075cc7ef5e1825e

atrace.rc: Correct path to saved_cmdlines_size when setting permission
am: 922e418f45

Change-Id: If92363cbd1ad525aa6b65e255cc86d7d6005ad25

atrace.rc: Correct path to saved_cmdlines_size when setting permission
am: 922e418f45

Change-Id: I99862d8e16d1c10405a696b89553a855881c967a

Resurrect the pose tool.

Editing the tool to look into the same broadcast ring that
VrCore reads from.

Bug: 62355308
Test: Run pose tool
Change-Id: I270e3934c7df398048701a81851ee860a8acb712
(cherry picked from commit 2a4d63ce1737705fff6def88e68b2e537db9074c)

atrace.rc: Correct path to saved_cmdlines_size when setting permission

Bug: 62375794
Test: CTS test

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

dumpstate: Better format for dumping block stat
am: 1dc1ef5af5

Change-Id: Ida45ae1d195ace8aed23e98be6eaf50742e47c7f

dumpstate: Better format for dumping block stat

After CL the format looks like:

------ BLOCK STAT (/sys/block/mmcblk0) ------
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/stat                 :    11299    16093  1262592    31863     3895     2968    63232     6463        0    10260    38243
/mmcblk0/stat                 : perf(ios) rd: 75622KB/s(1322/s) wr: 18671KB/s(2246/s) q: 4
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p1/stat       :       37       29       66        6        0        0        0        0        0        6        6
/mmcblk0/mmcblk0p1/stat       : perf(ios) rd: 5632KB/s(6167/s) q: 1
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p24/stat      :       12       32      346        0        1        0        8        0        0        0        0
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p26/stat      :        0        0        0        0        3      381     3072      296        0      100      296
/mmcblk0/mmcblk0p26/stat      : perf(ios) rd: 0KB/s(0/s) wr: 15729KB/s(30/s) q: 3
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p30/stat      :       11       33      352       23        0        0        0        0        0       23       23
/mmcblk0/mmcblk0p30/stat      : perf(ios) rd: 7836KB/s(478/s) q: 1
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p38/stat      :        1        3       32        0        0        0        0        0        0        0        0
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p39/stat      :      816     1072    47000     3460        0        0        0        0        0      273     3460
/mmcblk0/mmcblk0p39/stat      : perf(ios) rd: 88147KB/s(2989/s) q: 13
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p40/stat      :       17       13      228        0        4        0       32        3        0        3        3
/mmcblk0/mmcblk0p40/stat      : perf(ios) rd: 0KB/s(0/s) wr: 5461KB/s(1333/s) q: 1
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p41/stat      :     4831    14319   453376    17336        0        0        0        0        0     3273    17330
/mmcblk0/mmcblk0p41/stat      : perf(ios) rd: 70898KB/s(1476/s) q: 5
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p42/stat      :        8      124     1056        3        0        0        0        0        0        3        3
/mmcblk0/mmcblk0p42/stat      : perf(ios) rd: 180224KB/s(2667/s) q: 1
Block-Dev                     :    R-IOs   R-merg   R-sect   R-wait    W-IOs   W-merg   W-sect   W-wait   in-fli    activ   T-wait
/mmcblk0/mmcblk0p45/stat      :     5560      468   760088    11033     3046     2587    60120     5490        0     6840    16453

Bug: 62319771
Test: adb bugreport and check the result
Change-Id: I9464c4758f03479d433bdb7a907dd4953f226e1e

Merge "dumpstate: Remove wpa_cli usage" into oc-dr1-dev
am: 516e0c3738

Change-Id: I7b51227046dd3af58d0b7008f4a3a49d72ad69d5

Merge "dumpstate: Remove wpa_cli usage" into oc-dr1-dev

Merge "Re-enable default display when switching to VR mode." into oc-dr1-dev
am: efa87ca42e

Change-Id: I149f9ea222d058b6b42405dabd2d27f44706a853

Merge "Add LambdaMessage to SurfaceFlinger" into oc-dr1-dev
am: 2f7af47f12

Change-Id: I9dcff900087c79debfd52e2462745e5312c58946

Merge "Re-enable default display when switching to VR mode." into oc-dr1-dev

Merge "Add LambdaMessage to SurfaceFlinger" into oc-dr1-dev

dumpstate: Dump all block device stat
am: 509bb5d176

Change-Id: I3ddf083f033b093e9304c8cf61e0f5b0ddb47a06

dumpstate: Dump all block device stat

We have "UPTIME MMC PERF" but it is limited to MMC and thus has
been broken for a while in recent devices.
This CL is to add dump for all block devices stat information.

Bug: 62319771
Test: adb bugreport and check the result
Change-Id: I74e0bf0c4bf40e268dffd0abb8922f0ffb5e9692

Merge "Move global buffer ops to VR display service." into oc-dr1-dev
am: 6c7f1196ea

Change-Id: I7d08e724cfebcb70e5d5ca23b576d2a878b604e3

Merge "Move global buffer ops to VR display service." into oc-dr1-dev

Merge "Add missing VNDK pixel formats to AHardwareBuffer" into oc-dev am: 2563c0f3b0
am: f86cfd6dd9

Change-Id: I7a4e930b62e9cdffa22a66b09509b0927a05beeb

Merge "Add missing VNDK pixel formats to AHardwareBuffer" into oc-dev am: 2563c0f3b0
am: b52ddcb252

Change-Id: I29100f8c59a63036606523991fb0bd03a1d32329

Merge "Add missing VNDK pixel formats to AHardwareBuffer" into oc-dev
am: 2563c0f3b0

Change-Id: Iba3dfbfcbaa778da25de68a16beee024ac1b6efe

Merge "Add missing VNDK pixel formats to AHardwareBuffer" into oc-dev
am: 2563c0f3b0

Change-Id: I10e623eebb70da1f4127c0eec2aa85867da0623c

Merge "Add missing VNDK pixel formats to AHardwareBuffer" into oc-dev

Merge "Floor height API changes." into oc-dr1-dev
am: 2717eaba05

Change-Id: Id5783b9713cda9a5b1c62499a29322af3e86630c

Merge "Floor height API changes." into oc-dr1-dev

Re-enable default display when switching to VR mode.

Previously, in an attempt to minimize the number of changes that got
reset during VR mode transitions, the primary display was not turned back
on.

Bug: 62215749
Test: Switch device into VR mode, observe primary display invalidations.

Change-Id: Idd883e1522f519a0db06fdd88b071007487b6e70
(cherry picked from commit 933ed5ceb9d70829333ae438985a1bfeec8e79a8)

Add LambdaMessage to SurfaceFlinger

LambdaMessage allows for a cleaner, more compact way to create and
dispatch messages inside SurfaceFlinger. A follow up CL uses this
method, but it was isolated here to keep added functionality and
helper facilities separate.

Bug: 62215749
Test: Compile
Change-Id: I9e13e04f1b67fd60f01bcab02fe4f19c91c10bd4
(cherry picked from commit 117698b8e51c839e4c3925d66af13f3de3b420ab)

Merge "Fix build am: aaef0983ba am: 2c410e0d5f" into oc-dr1-dev-plus-aosp

Fix build am: aaef0983ba
am: 2c410e0d5f

Change-Id: I10728186af370d9e47196eeda43dfeccc582bc74

Fix build am: aaef0983ba
am: 5a556bc71e

Change-Id: I59426f537ebe484b865a3b424e7e56608fa14112

Fix build
am: aaef0983ba

Change-Id: I087952f069da1ba624bef32d1ac913a242533321

Fix build
am: aaef0983ba

Change-Id: Ia4d360cb942c43180d6077cd01bd9e9eb5b598d5

Fix build

Test: builds
Bug: 36631902
Change-Id: Iec23f8ab2b16c923f07d1857aa3471cfefc057dd

Properly run window animations at vsync-sf (2/2) am: b1e2f8deb3
am: 5d37ae59e0

Change-Id: I286fbd09c2fe425a90fd2b93c4a83b03d55e8113

Properly run window animations at vsync-sf (2/2) am: b1e2f8deb3
am: d5ebce91c8

Change-Id: I30768aba500834a157290871b0970f20a8eebd06

Properly run window animations at vsync-sf (2/2)
am: b1e2f8deb3

Change-Id: I2cf08e016fcd177117d7135d211e2676a1945eba

Properly run window animations at vsync-sf (2/2)
am: b1e2f8deb3

Change-Id: I7fbd22c681688edce1eda7f2786c68e6cbe32a22

Add missing VNDK pixel formats to AHardwareBuffer

AHardwareBuffer is now the only API available to
vendors to create graphic buffers in their HAL
implementations, unfortunately AHardwareBuffer
was designed for NDK use and was missing all
HAL specific pixel formats.

This CL adds these missing format to the VNDK-only
header.

The implementation is changed to accept all formats
which means, VNDK formats are indirectly exposed
to the NDK.

Bug: 38466076
Test: manual
Change-Id: I1e752cd33641aa861d9d15432cde6bec5a682e9a

Properly run window animations at vsync-sf (2/2)

- Add new Choreographer instance that runs on vsync-sf
- Use this new Choreographer for WindowAnimator, and remove all
the hacks around it

Test: Open apps and close apps, notice no stutter
Test: Screen zoom animations
Test: go/wm-smoke
Bug: 36631902
Change-Id: I72a8b39709303a38fc077100229b8a81a153ba3e

Move global buffer ops to VR display service.

The original home for these ops was the VR display manager service,
which is a protected singleton service that may only have one client.
Since more than one service needs to create global buffers, move these
ops to the display service. They are already protected by permission
checks.

Bug: 62424911
Test: dvr_api-test passes
Change-Id: Ia2f57fdf8a5258b52a652935d160e90db0f1cf9e

Floor height API changes.

The platform does not populate this field yet. This CL is for making the
necessary API change in frameworks/native before we cut the DR branch.

Bug: 38212129
Test: No functional change. Must compile.
Change-Id: Iaa21bbc0cec866bf5325d760e13989e49bce502a

Merge "Merge "Don't double-count external cached app data." into oc-dev am: b5a1f50f5e am: c11cc1d365" into oc-dr1-dev-plus-aosp

Merge "Don't double-count external cached app data." into oc-dev am: b5a1f50f5e
am: 267b4fd7e7

Change-Id: Id940efe570f37a873df7a1b4a4b1dd1ee4ef32b0

Merge "Don't double-count external cached app data." into oc-dev am: b5a1f50f5e
am: c11cc1d365

Change-Id: I543c21f831b22ab5a74da370a8b18acfb5891d7e

Merge "Don't double-count external cached app data." into oc-dev
am: b5a1f50f5e

Change-Id: Ia991cb98e842a9a9f64e669435c860611e79ff72

Merge "Don't double-count external cached app data." into oc-dev
am: b5a1f50f5e

Change-Id: Ia2057fe4522596b4593465db1f5d6cdcbf39c063

Merge "Don't double-count external cached app data." into oc-dev

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

Change-Id: Ife1b6aa7c68d97fe1ba0a0b121852d2d2cbb1a80

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
am: d6d91e5833

Change-Id: Ia4dad911062490db360fec8a1dc470b5a7bce9d6

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

Don't double-count external cached app data.

Cached app data is already counted in extStats.dataSize, so we don't
need to count it twice.

Test: builds, boots, verified Settings UI before/after
Bug: 62343684
Change-Id: Ic02cbe3f5938949730f9bc8d911ed398436f84d7

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

dumpstate: Remove wpa_cli usage

List of saved networks is dumped out by WifiConfigManager during
bugreport collection. The 'wpa_cli` command in dumpstate.cpp is
non-functional since the saved networks are no longer saved in
wpa_supplicant.

Bug: 62068865
Test: Collected bugreport and ensured that there are no wpa_cli errors.
Change-Id: I0080bf10dafcbd187fcf583232ce64dd3868d818

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."