Merge "Check key before edit value in countFlushCompleteEventsLocked" into oc-dr1-dev
am: 8669acc97d

Change-Id: I5fa61c3ffa5f41a5093dba86ad255687d29466b4

Merge "Check key before edit value in countFlushCompleteEventsLocked" into oc-dr1-dev

Merge "Add DVR method to get native display metrics" into oc-dr1-dev
am: f412b2b9fc

Change-Id: I4cfecf4ccc8bd5a8c3b9c3c11d885ac0233b71d3

Merge "Add DVR method to get native display metrics" into oc-dr1-dev

Merge "Avoid SELinux violation at vndservicemanager" into oc-dr1-dev
am: 5e6abb3f0a

Change-Id: Id757814f2d04cdbbe66f5dcc30fcf34f14338aa2

Merge "Avoid SELinux violation at vndservicemanager" into oc-dr1-dev

dumpstate: Add the 2nd fd for binary file
am: 9fbfad0b6d

Change-Id: I6c8111188e7fdb8b4b6282aa3511a4d7187cd69e

Merge "Move include/binder to libs/binder/include" am: d46d9f1243 am: b212140bbf  -s ours
am: 7f683a8b8f  -s ours

Change-Id: I0a02efc1cac647857ba32235fac2c160d7dcb8db

Merge "Move include/binder to libs/binder/include" am: d46d9f1243
am: b212140bbf  -s ours

Change-Id: If344d9c14b3cf767956d55e0dd5d13b5a700c7a4

Merge "Move include/binder to libs/binder/include"
am: d46d9f1243

Change-Id: I41744e18ca3731b3027cb459b97945270a4d94ef

Merge "Move include/binder to libs/binder/include"

dumpstate: Add the 2nd fd for binary file

Bug: 33820081
Test: Verify binary file is added in bugreport
Change-Id: Ibe1c219ca4bf288adcb76996c6178426a4d04333

Avoid SELinux violation at vndservicemanager

Avoid following SELinux violation.

avc: denied { read } for comm="vndservicemanag"
name="nonplat_service_contexts" dev="rootfs" ino=17045
scontext=u:r:vndservicemanager:s0
tcontext=u:object_r:service_contexts_file:s0 tclass=file permissive=0
ppid=1 pcomm="init" pgid=1 pgcomm="init"

This violation caused by vndservicemanager reading service_contexts at
svcmgr_handler(). In main() loading as well, processing is divided by
the VENDORSERVICEMANAGER flag. Therefore, even in svcmgr_handler(),
processing is divided by flags like main().

Bug: 62562415
Test: mma
Change-Id: I06b0308a80fc6ea1ca57cd10d9555dd269b8e12d

Merge "Merge "surfaceflinger: fix a potential child layer leak" into oc-dev am: 1b02b6329e am: 989ed5c4c1" into oc-dr1-dev-plus-aosp

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev am: 1b02b6329e
am: 989ed5c4c1

Change-Id: I1255ff4ef6234b61996e6f7c8dfb86c890e4268e

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev am: 1b02b6329e
am: 4d5cc90559

Change-Id: Ib956b914044fee1941ef966bbf96c151bbbcf6a4

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev
am: 1b02b6329e

Change-Id: I3548aaa0a071f5b1009c5b1070405a09d7841095

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev
am: 1b02b6329e

Change-Id: Ie2bd5a6d1d37bb5b7c7b679b8c472389ff619188

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev

Check key before edit value in countFlushCompleteEventsLocked

Fix an unchecked editValueFor that potentially causes log fatal
when sensor is removed from connection shortly before arrival of
flush complete message.

Bug: 62806749
Test: compiles, flush() still working.
Change-Id: I3684251b7bb746b01d0d8849024217c80ad52f9d

Add DVR method to get native display metrics

This seems like the kind of infomration we should be able to communicate
to vrcore.

Bug: 62494565
Test: Builds and loads, still need to test with vrcore
Change-Id: I2342fabcd2bcb8a34c0b783391b60defc381f3c6
(cherry picked from commit 6c5a92f142b5fa1907ff91a7dce4d09ddcb554c6)

Merge "Fix minor version bump reflected in lshal --init-vintf" am: ac6575097d am: 99065302ed
am: cf64ff00ea  -s ours

Change-Id: I9811de0500a5d5172744c962549524416c3183e3

Merge "Fix minor version bump reflected in lshal --init-vintf" am: ac6575097d
am: 99065302ed

Change-Id: Ic2a1f895312dee90e9e0eaf691edb48df4897c4e

Merge "Fix minor version bump reflected in lshal --init-vintf"
am: ac6575097d

Change-Id: I05dafdba2c7f302dc6200bd2d274e30a33f77de7

Merge "Fix minor version bump reflected in lshal --init-vintf"

Fix minor version bump reflected in lshal --init-vintf
am: e86f70b474

Change-Id: Id573ba5c3b436249a5f407e575a01a61e7985e62

Merge "Fix incorrect bitness for lshal --init-vintf" am: e162a97703 am: 126eee4d14
am: c77bc022f7  -s ours

Change-Id: I69a55e1df55604c5ec36a93fbd070fe333e01cc1

Merge "Fix incorrect bitness for lshal --init-vintf" am: e162a97703
am: 126eee4d14

Change-Id: I435466a45aa5217f99c73007d5f07c1754216995

Merge "Fix incorrect bitness for lshal --init-vintf"
am: e162a97703

Change-Id: Ib59a256b4491b2bf93e8eca1fa85d9a193719446

Merge "Fix incorrect bitness for lshal --init-vintf"

Fix minor version bump reflected in lshal --init-vintf

Test: lshal --init-vintf has no warnings, and power@1.1 / vibrator@1.1
      is shown in output.

Bug: 62675393
Change-Id: I1ffd8b6dc7fcadcf7228cf2b8838e918bb3afe6b
Merged-In: I1ffd8b6dc7fcadcf7228cf2b8838e918bb3afe6b

Fix minor version bump reflected in lshal --init-vintf

Test: lshal --init-vintf has no warnings, and power@1.1 / vibrator@1.1
      is shown in output.

Bug: 62675393
Change-Id: I1ffd8b6dc7fcadcf7228cf2b8838e918bb3afe6b
Merged-In: I1ffd8b6dc7fcadcf7228cf2b8838e918bb3afe6b

Merge "Fix incorrect bitness for lshal --init-vintf" into oc-dr1-dev
am: 5cf813eef0

Change-Id: I1267c96addf4c501fd695faddf5e4d59bd51f8a0

Merge "Fix incorrect bitness for lshal --init-vintf" into oc-dr1-dev

Fix incorrect bitness for lshal --init-vintf

Test: lshal --init-vintf, graphics.mapper has 32+64.

Change-Id: Ic586c8333064cc5636679ce8ad4e1231307191f4
Merged-In: Ic586c8333064cc5636679ce8ad4e1231307191f4
Fixes: 62675393

Fix incorrect bitness for lshal --init-vintf

Test: lshal --init-vintf, graphics.mapper has 32+64.

Change-Id: Ic586c8333064cc5636679ce8ad4e1231307191f4
Merged-In: Ic586c8333064cc5636679ce8ad4e1231307191f4
Fixes: 62675393

Merge "Merge changes from topic 'layer-wp-race' into oc-dev am: a17b14eb92 am: c35a5e29f3" into oc-dr1-dev-plus-aosp

Merge changes from topic 'layer-wp-race' into oc-dev am: a17b14eb92
am: c35a5e29f3

Change-Id: I590c8667049be99f407972953c41ff88e555bb2a

Merge changes from topic 'layer-wp-race' into oc-dev am: a17b14eb92
am: f39c08c906

Change-Id: I579504e93cc6c21dd13e5b8468d8b87037133a0c

Merge changes from topic 'layer-wp-race' into oc-dev
am: a17b14eb92

Change-Id: I1af51f57255024d36e9cae5c221361f36cb28da4

Merge changes from topic 'layer-wp-race' into oc-dev
am: a17b14eb92

Change-Id: I2fb898abae8b4602aa7257b45293f582f0641f24

Merge changes from topic 'layer-wp-race' into oc-dev

* changes:
  surfaceflinger: Layer::getParent requires state lock held
  surfaceflinger: distinguish mCurrentParent/mDrawingParent
  surfaceflinger: protect Client::mParentLayer with a lock

Merge "Add method to explicitly create Surface. am: 4e623e259c am: 84051d9c97" into oc-dr1-dev-plus-aosp

Add method to explicitly create Surface. am: 4e623e259c
am: 84051d9c97

Change-Id: I4368e31fdb9b7ec0912bddbe75399c7d778383b8

Add method to explicitly create Surface. am: 4e623e259c
am: 05e6335833

Change-Id: Id470d62303b9055522d1cb6ad456bcbe4d320cfd

surfaceflinger: fix a potential child layer leak

We should not remove a child layer from its already removed parent.
Call p->removeChild only after we've checked that the ancestor is
alive.

Apply e6b63e1ae12692327f7e46d5f10d6ade5a7bf192 and this fix to
SurfaceFlinger_hwc1.cpp as well.

Bug: 37121786
Test: manual stress test
Change-Id: I7b811450a998acc4ad9690bd4eda058ce6588e14

surfaceflinger: Layer::getParent requires state lock held

We rely on mStateLock to synchronize accesses to
Layer::mCurrentParent.

Bug: 38505866
Test: manual stress test
Change-Id: I5f8ec358ed7e35df28f8c6aec31ae6ee51cb5b93

Add method to explicitly create Surface.
am: 4e623e259c

Change-Id: I22ff0cd39e80478a74e4589084d5ccabfbe54443

Add method to explicitly create Surface.
am: 4e623e259c

Change-Id: Ie3755008a75dc67f1d01a0a5fc8bf4e953c8cf9d

Merge "libui: Load a.h.graphics.mapper passthrough service on library load" into oc-dr1-dev
am: 90254a8d57

Change-Id: I4e1b88ca22dc7ca4bb1be792f17b03cad5e75f10

Merge "libui: Load a.h.graphics.mapper passthrough service on library load" into oc-dr1-dev

Merge "Fix virtual touchpad scroll events." into oc-dr1-dev
am: 23970309e2

Change-Id: I89257f0a2b434016601aac06a9ed8cbbf5be1bd1

Merge "Fix virtual touchpad scroll events." into oc-dr1-dev

Add method to explicitly create Surface.

It is sometimes necessary for a SurfaceControl client to request the
associated Surface object be created rather than reuse the cached
value. This changelist adds an additional method that performs the
same creation operations as if the Surface object was created for the
first time.

Bug: 62108743
Test: go/wm-smoke
Change-Id: I2f460f20b2d16ed9ff81cb36842bcd2f8641e03c

surfaceflinger: distinguish mCurrentParent/mDrawingParent

Updates to wp<> is not atomic.  We cannot use/update it at the same
time from the main thread and a binder thread.  With this change,
binder threads use mCurrentParent with the external state lock held.
The main thread uses mDrawingParent.

This is also an alternative fix to bug 62099658 and allows us to
revert "SurfaceFlinger: Update parent pointer while performing
transaction."

Bug: 38505866
Bug: 62099658
Test: boots and no repro (but I can never repro)
Change-Id: Id286a437537daaeec5eee5de62b1d9df245ece53

surfaceflinger: protect Client::mParentLayer with a lock

Updates to wp<> is not atomic.  Use Client::mLock to protect
mParentLayer.

Bug: 38505866
Test: camera and youtube work
Change-Id: I2739382d5bb99961a47c1011963b6f676d34eec6

Fix virtual touchpad scroll events.

1. Correctly configure the uinput device to pass REL_*WHEEL events.

2. Configure the virtual touchpad as a 'rotary encoder', which is
   an inputflinger device class originally provided for the G1 scroll
   ball. This provides two advantages over the default mouse-wheel
   interpretation:
   - It works without an associated mouse pointer to determine the
     event target.
   - It allows for higher resolution through a scale factor (the
     scaling here is chosen by experiment to feel no worse than wheel
     mode, but might benefit from future adjustment).
   There is one accompanying disadvantage: inputflinger only handles
   vertical scrolling (REL_WHEEL) and ignores the horizontal axis
   (REL_HWHEEL).

Bug: 62632827
Test: manual on sailfish
Change-Id: Ic8edad01796d75e8c94fa20f05a5badb0727a948

Set the vr hwc vsync rate to match the native vsync
am: 22c1462add

Change-Id: I3b4ebea0ad9e8249d6dec76e798e67724a84761d

Move include/binder to libs/binder/include

Also moved include/private/binder to libs/binder/include/private/binder.
This allows libbinder to be compiled without depending on global
headers.

Bug: 33241851
Test: Build, flash, and run sailfish.
Test: Try building with BOARD_VNDK_VERSION := current
Change-Id: I266d5338677a92dd0563b311a479e7ad27bbaf55
Merged-In: I0a82b733353add589de26eb2db7af5a246e5c97f

libui: Load a.h.graphics.mapper passthrough service on library load

Zygote loads libui, this helps complete the chain from
  zygote -> libui -> a.h.graphics.mapper -> gralloc
so that all of these libraries which are used by nearly every app are
loaded by zygote and shared.

Bug: 62353585
Test: boot to launcher, run calculator, confirm a.h.graphics.mapper
      implementation library is in /proc/`pid zygote64`/maps

Change-Id: Ib505c926a410191e3e2ad809bfe98a73b21fac90

Set the vr hwc vsync rate to match the native vsync

This simple modification reduces the motopho latency from 85ms to 65ms.
However, this is not a final solution, and we should continue to look
for how to address the issue of difference in vsync reporting.

Bug: 36493906
Test: Run motopho, latency is now much lower.
Change-Id: Id52689900a5fe58dfc9d7790b7c3746903380972

Merge "Fix vr flinger post thread to resume correctly" into oc-dr1-dev
am: 4bfb7550af

Change-Id: I2957b1b32bf8378f4b965d1f60dcafd4c48f938e

Merge "Fix vr flinger post thread to resume correctly" into oc-dr1-dev

Fix vr flinger post thread to resume correctly

After being paused, the vr flinger post thread remained paused until it
was enabled by requesting display ownership, but the request for display
ownership was made from the vr flinger post thread, causing the post
thread to get stuck in the paused state forever. I moved the display
ownership request to the vr flinger dispatcher thread, so the post
thread can be resumed.

Bug: 62666511

Test: Put a device through a sleep/wake cycle and confirmed the vr
flinger post thread resumes as expected.

Change-Id: Ic57adb7376c8f2b5760bfaa4a5a30b58894a7ab7

Merge "Use _exit for profile copy dexopt command" into oc-dr1-dev
am: de65cb8e1f

Change-Id: I1d53221347533d9d7b393141b4262b66cee53134

Merge "Use _exit for profile copy dexopt command" into oc-dr1-dev

Merge "Fix getDisplayInfo() for SurfaceFlinger in HWC2 mode. am: c175253b6d am: f86e12ba89" into oc-dr1-dev-plus-aosp

Fix getDisplayInfo() for SurfaceFlinger in HWC2 mode. am: c175253b6d
am: d8449e55a6

Change-Id: I400998a8226e19fb54f5cb981630841ee8d288ef

Fix getDisplayInfo() for SurfaceFlinger in HWC2 mode. am: c175253b6d
am: f86e12ba89

Change-Id: I56fcdd654db56912fe2a04457946eef468b1825f

Fix getDisplayInfo() for SurfaceFlinger in HWC2 mode.
am: c175253b6d

Change-Id: I55ca473facef90f00eb1a401d9bc8ad7a36e4878

Fix getDisplayInfo() for SurfaceFlinger in HWC2 mode.
am: c175253b6d

Change-Id: I8a1db090809ec0354dda54a6202ff92b76681bb4

Fix getDisplayInfo() for SurfaceFlinger in HWC2 mode.

When HWC2 mode is enabled in SurfaceFlinger, the getDisplayInfo()
function would fail to return the correct active mode in some cases.

This bug was only noticable if you had more than one mode registered
by the HWC2 backend.

The SurfaceComposerClient::getDisplayInfo() function works by calling
getDisplayConfigs(), and de-referencing the config at the index
returned by getActiveConfig(). The active config returned was correct,
but the getDisplayConfigs() array was not properly sorted.

Tracing this back, this problem occurs because the configId is stored
alongside the config in an unordered_map, so when this is converted
to a vector and the configId is discarded, the conversion must be
sorted correctly; it can't just be the hash order returned from
the unordered_map.

There are a few ways to fix this problem, but the easiest was to nip
the problem in the bud by not allowing an unsorted list of configs
to exist on the base HWC2 class. It may be better still to just get
rid of the map and go back to a vector, as this data is more often
used as an array than it is indexed by configId, so the use of an
unordered_map was probably overkill.

Change-Id: Ibfb015a6d9b49c870f37a1c892f15f90abbd1e2c
Bug: 62617569
Test: ./cts-tradefed run cts -m CtsViewTestCases -t android.view.cts.DisplayRefreshRateTest#testRefreshRate

Use _exit for profile copy dexopt command

Previously we used exit(0), but this called global destructors and
could cause problems depending on the state when the forking
happened.

Using _exit avoids calling hte global destructors in the child
process.

Test: Delete packages.xml and flash to simulate first boot
Bug: 62597429
Change-Id: I3a6dcd5f05ca85e1488df154ec283c2ec842e59f

Merge "Merge changes Ibc1444b8,I14c200d1 into oc-dev am: fd43dc6723 am: eb444d602c" into oc-dr1-dev-plus-aosp

Merge changes Ibc1444b8,I14c200d1 into oc-dev am: fd43dc6723
am: b209c12155

Change-Id: I9da639be63a43343d236c865aa810957290e2e49

Merge changes Ibc1444b8,I14c200d1 into oc-dev am: fd43dc6723
am: eb444d602c

Change-Id: I5c01a250bf9422b23210fc6860676c60ed33c172

Merge changes Ibc1444b8,I14c200d1 into oc-dev
am: fd43dc6723

Change-Id: Iae111bcc2770c1e75aba29021e31d07ff89243ab

Merge changes Ibc1444b8,I14c200d1 into oc-dev
am: fd43dc6723

Change-Id: I7a63dd4d785a3bc4e386cc10a427b12d678e7b28

Merge changes Ibc1444b8,I14c200d1 into oc-dev

* changes:
  ConsumerBase: discardFreeBuffers() also needs to dump its own cache
  BufferQueueProducer: Call onBuffersReleased() in detachNextBuffer

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

Change-Id: I5c5bacff22639e8e5457cc9bfdc9ac728a76473c

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

Change-Id: I9565d4edc4e56589f38fe297b19a4a5abb87c7ac

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

Change-Id: Ic92c39e375542d214273c7ba89dcb0020a34bd93

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

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

Change-Id: Ib48966c671e1cedb7ed1d26498b8389a62e30921

SensorManager: attach background thread to JavaVM
am: 09034f076e

Change-Id: I07364b649a58f9ca0891407540a607967155bf4b

SensorManager: attach background thread to JavaVM
am: 09034f076e

Change-Id: Ib32d5a33c7de35d9d1181fc737848ba63fe94171

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

Change-Id: I8228b3afec41260135c4123fe8cd9e9dc99f46e5

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

ConsumerBase: discardFreeBuffers() also needs to dump its own cache

ConsumerBase has its own cached slots with graphic buffer references,
so it's not enough to just ask the buffer queue consumer to free
buffers.

Add code equivalent to what happens in the onBuffersReleased callback.

Test:
Bug: 62593139
Change-Id: Ibc1444b868c6150aa2da1c209e06bdba42f1595d

BufferQueueProducer: Call onBuffersReleased() in detachNextBuffer

Like detachBuffer(), detachNextBuffer() needs to inform the consumer,
who generally has a cache of buffer items, that a buffer is no longer
owned by the buffer queue.

Otherwise the consumer layer can leak the buffer reference until
consumer teardown/disconnect.

Test: Camera CTS is fine, using a camera app shows correct memory
  behavior
Bug: 62591036

Change-Id: I14c200d13e60dbbe21261343941f84fb786db117

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