Merge "Moved bugreport intents to the com.android.internal namespace."

Merge changes from topic 'cfg_nsfbuf'

* changes:
  configstore: removing BoardConfig variables (NUM_FRAMEBUFFER_SURFACE_BUFFERS)
  configstore: use utility functions defined in ConfigStoreUtils.h

Fix vr_wm's importing of buffers. Register needs to be first now

Test: Use VrModeTester to launch into vr mode, verify vr_wm gets buffers
Bug: None
Change-Id: I17a65dc822454386f392cfdd9ffb401055363515

Merge "Fix the ownership of compiler artifacts for secondary dex files"

Merge "EGL: Enable eglGetFrameTimestamps."

Moved bugreport intents to the com.android.internal namespace.

Change-Id: I5b19c119b9f845007b37dc955120d2a4f1187e63
Fixes: 35275802
Test: manual verification

configstore: removing BoardConfig variables (NUM_FRAMEBUFFER_SURFACE_BUFFERS)

This change removes the use of NUM_FRAMEBUFFER_SURFACE_BUFFERS from
SurfaceFlinger.

Instead of using it, the implementation is changed so that it retrieves
corresponding configuration value (useTripleFramebuffer) from
configstore.

Design doc: go/design-confighal
Bug: 34723491
Test: build, run, log message "SurfaceFlinger FrameBuffer: ?" should be seen.

Change-Id: If6f6d13738d894b4880f42576f4ce8220b4a8bef

Merge "VolumeShaper: Add to native IPlayer interface"

configstore: use utility functions defined in ConfigStoreUtils.h

This change modifies existing implementation to get config values via
configstore by using utility functions defined in ConfigStoreUtils.h.

Specifically, as an example, the utility function getInt64() is used
for vsyncEventPhaseoffsetNs defined in ISurfaceFlingerConfigs.

Design doc: go/design-confighal
Bug: 34724435
Test: build, run

Change-Id: I82a9371a84902c1e185d619d77e4c2e3272f84db

Merge "Revert "Temporarily disable enumeration of VK_KHR_incremental_present""

Fix the ownership of compiler artifacts for secondary dex files

odex, vdex and art files should be owned by the app and not by system.

Test: verify odex and vdex owner:
      adb shell cmd package compile -m speed --secondary-dex -f
com.google.android.gms
      adb shell ls
/data/user/0/com.google.android.gms/app_chimera/m/00000004/oat/arm64/
-al
      -rw-r--r-- 1 u0_a17 u0_a17       5640832 2017-02-02 18:00
DynamiteModulesA_GmsCore_prodmnc_alldpi_release.odex
      -rw-r--r-- 1 u0_a17 u0_a17       3016818 2017-02-15 12:42
DynamiteModulesA_GmsCore_prodmnc_alldpi_release.vdex

Bug: 35389574
Bug: 32871170
Change-Id: I0650265cedcc572d43bb1fc0d0758ae43ea890b2

Merge "EGL: Add eglGetCompositorTimingANDROID."

Revert "Temporarily disable enumeration of VK_KHR_incremental_present"

This reverts commit 15b1cb2e57435514b4810f27e5ee01a2e480b338.

Change-Id: Ia4d6a75a6fc71107145e7273e891e1efb0f3eb9a

Merge "sensorservice: Start using HIDL sensor HAL."

Merge "Remove TARGET_USES_QCOM_BSP from vr/sensord"

VolumeShaper: Add to native IPlayer interface

Test: Check proper ducking
Bug: 31015569
Change-Id: I679b32a3adbf8ec56bafa20c8479c7645b3d59fa

Merge "surfaceflinger: skip buffer cache when BYPASS_IHWC"

Merge "Fix unitialized field in HWC2on1Adapter"

Merge "Remove magic number usage in SurfaceFlinger"

Merge "Ignore VK_KHR_incremental_present hint for MAILBOX."

sensorservice: Start using HIDL sensor HAL.

Framework now use the HIDL sensor HAL only. Loading the legacy
HAL library is removed.

Bug: 32978887
Test: sensors stream data in both passthrough and binderized mode.
Change-Id: I1d700a5a23cd9d209559570439f306aa22c613c1

Merge "EGL: Use frameId instead of framesAgo for frame events."

Merge "Sensors: Add LOW_LATENCY_OFFBODY_DETECT sensor API"

Fix unitialized field in HWC2on1Adapter

Bug: 33487853
Test: AUPT
Change-Id: If271eb6b9d357047b41fed19a2079325970132a7

Ignore VK_KHR_incremental_present hint for MAILBOX.

Test: Using instrumented cube demo, and tmp ALOGI comments.

This code may eventually be replaced with code that constructs, stores,
and uses regions that spans multiple vkQueuePresentKHR() calls.  With
VK_PRESENT_MODE_MAILBOX_KHR, any given present may not be sent to the
display.  In such a case, the correct behavior is for the swapchain code
(a.k.a. acting for the "presentation engine") should build up a list of
regions across multiple vkQueuePresentKHR() calls, so that when an image
is presented, all of the updated regions will be sent to the display.
In the mean time, this code is correct (i.e. ignore the hint for
MAILBOX).

Change-Id: I413091871a9b9b7464ec941a857eea56e68f4189

surfaceflinger: skip buffer cache when BYPASS_IHWC

When SF talks to hwcomposer2 directly, we should skip HWC buffer
cache.  The cache works somehow similar to
IGraphicBufferConsumer::acquireBuffer, where buffer is set to
nullptr when it is already seen previously.  That confuses
hwcomposer2.

Bug: 35325794
Test: manual
Change-Id: I5306ecc9e7290bc6734091a9884024bf38667628

Merge "Revert "dumpsys: add option to list hardware services."" am: 9555a5d497 am: d2390bea79 am: 8d3069503e
am: 36ec1bc76f

Change-Id: I5b16be2dae542e0becc114f36a2f75418e9911d5

Merge "Revert "dumpsys: add option to list hardware services."" am: 9555a5d497 am: d2390bea79
am: 8d3069503e

Change-Id: I6f4d5148e41181a4498fe1bcc3b2e9e0551785fb

Merge "Revert "dumpsys: add option to list hardware services."" am: 9555a5d497
am: d2390bea79

Change-Id: Ib70ddc0635bc3a502dbc9f7eb31c5716860e9833

Merge "Revert "dumpsys: add option to list hardware services.""
am: 9555a5d497

Change-Id: I022d2e803e169907c32fc7e263090ec2896d94c5

Merge "Revert "dumpsys: add option to list hardware services.""

Revert "dumpsys: add option to list hardware services."

This reverts commit 6270dd1bf7f4a6eaa0e36b2cbb0f9bdd4790b5d8.

Reason for revert: obsolete, removed in favor of lshal
Test: dumpsys_test

Change-Id: I06e256a5bd0995727a3d6f3f88f79a623b1474f3

Merge "Ignore callbacks from the non-active hardware composer"

Merge "Temporarily disable enumeration of VK_KHR_incremental_present"

EGL: Enable eglGetFrameTimestamps.

Also clean up the specification description:
1) Reverse order of revision history notes.
2) Specifically mention polling from another thread.
3) Make it explicit that eglQueryTimestampSupportedANDROID
   works for the timestamps of both eglGetCompositorTimingANDROID
   and eglGetFrameTimestampsANDROID.

Test: adb shell /data/nativetest/libgui_test/libgui_test
    --gtest_filter=*GetFrameTimestamps*

Change-Id: I26ff1ea97654d8953a18afaad1455813191abe7d

EGL: Add eglGetCompositorTimingANDROID.

Exposes the composite deadline, composite interval, and
the composite to present latency.

A history of composite and present fences are stored.
When the present fence's timestamp becomes known,
the composite to present latency is updated with
sampling jitter removed.

The values are updated in the producer when timestamps
are enabled and on queue and dequeue.

The deadline is snapped to the next expected deadline
based on the current systemTime().

Test: adb shell /data/nativetest/libgui_test/libgui_test
    --gtest_filter=*GetFrameTimestamps*

Change-Id: I406814258613b984b56488236632494f2f61ff2e

Temporarily disable enumeration of VK_KHR_incremental_present

Test: Ran the instrumented cube demo, which showed the extension not
enumerated.

Since the extension is not yet ratified, we can't enable it in the
upcoming DP1 release.

Change-Id: I8234c2d9510a0d31c309fb2c25fe6126fb021a59

Merge "Remove TARGET_USES_QCOM_BSP from libvrflinger"

Merge "Vibrates using 'cmd vibrator' instead of HAL vibrator."

Ignore callbacks from the non-active hardware composer

Ignore callbacks from the non-active hardware composer so we don't get
e.g. duplicate vsync callbacks, one from each composer.

Bug: None

Test: Manually confirmed with logs we're now ignoring callbacks on the
      non-active composer.

Change-Id: I8b475d6283d86c64ff96b41e78528bce8c6ff1d3

Merge changes from topic 'lshal' am: c9bc03729b am: d99d31d5d3 am: 5581e229a6
am: cd98373272

Change-Id: Ibea02dc9a1fc6e659992c746db3ef59ec5fc35aa

Merge changes from topic 'lshal' am: c9bc03729b am: d99d31d5d3
am: 5581e229a6

Change-Id: I526b604879c5c5103a7d0f9a66471534383c6175

Merge changes from topic 'lshal' am: c9bc03729b
am: d99d31d5d3

Change-Id: I615d885788739a2d3d2f11204ad23833a77d96d5

Merge changes from topic 'lshal'
am: c9bc03729b

Change-Id: I5051bdae873e53c21dcd6e949e7c47da4482a192

Merge changes from topic 'lshal'

* changes:
  Dump client PIDs for passthrough HALs.
  Fix permissions for lshal

libui: add notes to registerBuffer(buffer_handle_t)

The method is semi-broken on gralloc0 devices.  It calls down to
Gralloc1On0Adapter::managementHook, which will return
GRALLOC1_ERROR_BAD_HANDLE when the handle is received from a remote
process.

Test: make
Change-Id: Ia0ca2b51e38e8afe66a917ca7fe2282db3968f53

Remove TARGET_USES_QCOM_BSP from vr/sensord

Removing TARGET_USES_QCOM_BSP because 1) it is broken:
QCOM_B_FAMILY, QCOM_BSP macros are defined but never used. 2)
conditional compilation must be eliminated from the framework-side code
so that we can build a generic system.img.

Bug: 34115514
Test: build & run
Change-Id: Ic96c8d77bed9ce399fe6ffac2e57453861aa35a1

Remove magic number usage in SurfaceFlinger

Test: Manual
Change-Id: I9d4c7e4ec8546314864e0625a832e9a0bb49e95b

Remove TARGET_USES_QCOM_BSP from libvrflinger

Removing TARGET_USES_QCOM_BSP because 1) it is broken:
hardware/qcom/display/libgralloc,libqdutils do not exist and 2)
conditional compilation must be eliminated from the framework-side code
so that we can build a generic system.img.

Bug: 34115514
Test: build & run

Change-Id: I1fd643003426c52353b608bc6053a4ad80c1676a

Merge "Introduce an API to verify trusted caller apps by UID."

Dump client PIDs for passthrough HALs.

* This is done with registerPassthroughClient in IServiceManager; when a
  service is retrieved through the passthrough service manager, its PID
  will be recoreded by the binderized service manager.
* The action of dumping all available dynamic libraries is now done by
  dumpAllLibraries.
Bug: 34712252
Test: lshal

Change-Id: I7f08e3255ffed0f46435843504e0e225d0d9bbc9

Fix permissions for lshal

Instead of IServiceManager::debugDump, lshal now relies
on list and get. lshal itself will call getDebugInfo
on each interface, so that each interface can check if
the call is from a root user, then decide to leak
addresses or not.

Bug: 34899586

Test: lshal

Change-Id: I8eb9168c3b071b8b1f27dbeaa0ac27d2067200d7

Introduce an API to verify trusted caller apps by UID.

This CL introduces a new function to make sure only trusted packages
like VrCore can make use of APIs that are not intended for app use.

As a first example, this CL introduces a caller check for taking screenshots,
although any sensitive APIs should implement similar checks.

Package trust is defined by having the RESTRICTED_VR_ACCESS permission.

Bug: 34474022
Change-Id: Ib5a242d1a4e17f59b178fb1465064043613ac369

clean-ups, remove warnings, fix includes.

Test: built and run
Bug: none

Change-Id: I505df89e2263952d90fae432c5eaa7d00056c6f0

Vibrates using 'cmd vibrator' instead of HAL vibrator.

Change-Id: I7959f08bcb9623d1dfe26e941ccd2623e8cdf278
Fixes: 34346796
Test: manual verification
Test: dumpstate_test passes

Merge changes I5a8bb16f,I4555ba9d,Id1e3dd5e

* changes:
  vulkan: Temporarily hack around loader/driver mismatch
  vulkan: Add gralloc1 usage to VK_ANDROID_native_buffer
  vulkan: Tolerate missing GetSwapchainGrallocUsageANDROID function

Merge "Fix small child layer transparent region error"

Merge "Split view::Surface out of Surface.cpp"

vulkan: Temporarily hack around loader/driver mismatch

A driver was built with an older declaration of
vkGetSwapchainGrallocUsage2KHR, and doesn't match what libvulkan
expects. This is a temporary workaround until the driver can be
rebuilt against the current header. When a new driver is available,
this change should be reverted.

Test: vulkanGears on affected device
Change-Id: I5a8bb16f0f9563e32314cff2a2a7a45760cfc356

vulkan: Add gralloc1 usage to VK_ANDROID_native_buffer

Replaces the gralloc0-style 'int' usage in
vkGetSwapchainGrallocUsage2ANDROID and VkNativeBufferANDROID with a
gralloc1-style pair of uint64_t usages. A later change will plumb this
to and from ANativeWindow; for now we just collapse and expand from
32-bit usage.

Test: vulkanGears
Change-Id: I4555ba9d6ca1d15ed0791dc0bda01a1443890d02

VR: Build VR Window Manager only on platforms supported by libgvr

Bug: None
Test: Verified build succeeds on aosp_mips-eng
lunch aosp_mips-eng
cd frameworks/native/services/vr/vr_window_manager
mma

Change-Id: Idf8c4cb842f5fd30f047e70fb1374bca6247eef9

Merge "Fix boolean logic in SurfaceFlinger property read"

vulkan: Tolerate missing GetSwapchainGrallocUsageANDROID function

Some drivers implementing ANDROID_native_buffer V6 have /removed/ the
old function. We shouldn't really mind if they do that -- we're never
going to call it anyway if they do expose the new
GetSwapchainGrallocUsage2ANDROID function.

V2: Fail at CreateDevice-time if neither function is exposed.

Change-Id: Id1e3dd5e2e71a9291dd610e908f9906acaaf0ad0
Related bugs: b/34177594
Test: build

Merge "Disable Virtual Display by default in SF"

Merge "Allow surfaceflinger to run vrflinger"

Split view::Surface out of Surface.cpp

it’s moved into a new view/Surface.cpp, header in
gui/view/Surface.h

their entanglement also meant that code that needs
only view::Surface ended up with all of Surface
dependencies.

Test: built and booted device
Bug: treble
Change-Id: I1a350d258f124fbe0a7bc5a2ca165311dcb0d8e3

Merge "Break dependency of libui on libbinder"

Merge "VR: Update VR HWC and VR WM to use GrallocMapper"

Merge "Revert "sensorservice: Remove ENABLE_TREBLE flag.""

Merge "Fix bug in isEquivalent()"

Allow surfaceflinger to run vrflinger

* Allows surface flinger to switch in and out of vr mode

Bug: None
Test: Manually ran various vr test programs.
Change-Id: I15fbba0eb67cbcffeef41be31429550448a1db9c

Disable Virtual Display by default in SF

These should be enabled by default once b/30022738 is resolved.

Change-Id: I70a48a0bcf98ca984a28aa130170270d16e708bf

Revert "sensorservice: Remove ENABLE_TREBLE flag."

This reverts commit d333511e94afbcc6462dd9c81405f4a3e30ecac9.

Change-Id: I6812643a7d0265f3a64a03a35e4d5a2359ba87cf

Merge "NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations. Test: Verified changes with romainguy@. Related bugs: b/26922523" am: 270fff6a7c am: f9c7e2a002 am: 3d2dbf4d2f
am: cc070b4ddd

Change-Id: Ia8e0c98670407a8218d1fe6522a46a5d51843a49

Merge "NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations. Test: Verified changes with romainguy@. Related bugs: b/26922523" am: 270fff6a7c am: f9c7e2a002
am: 3d2dbf4d2f

Change-Id: Ice33e4e60d90f303652d2a993e90e3c58f3e261f

Merge "NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations. Test: Verified changes with romainguy@. Related bugs: b/26922523" am: 270fff6a7c
am: f9c7e2a002

Change-Id: I228a0f7ed82d88e64ddade491f5aece700c9d9dd

Merge "NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations. Test: Verified changes with romainguy@. Related bugs: b/26922523"
am: 270fff6a7c

Change-Id: Ibfc593ff8a9d913e083b8fda4341ca5110ba87a1

Merge "NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations. Test: Verified changes with romainguy@. Related bugs: b/26922523"

Fix bug in isEquivalent()

Fix a bug in SensorService::SensorDirectConnection::isEquivalent()
that causes the check to unfairly judge two shared memory buffers.

Test: compiles
Change-Id: I8274b01f331dc958c8b0183aab32119f21a8c4ff

NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations.
Test: Verified changes with romainguy@.
Related bugs: b/26922523

Change-Id: Id51445b09896407e808b519f3ee2f4be90f5b7eb

VR: Update VR HWC and VR WM to use GrallocMapper

Removes the gralloc hacks to access buffer internals and re-enables
building of VR Window Manager.

Bug: b/34878847
Test: Compiled
Change-Id: I06aa5ec0428b2f282a36ee89dbc4da9838c25782

Move pdx system/ paths to system/vr/

If PDX is used for non-vr stuff in the future, it will be clear
that system/vr/display is for vr and not general display

Test: compile and make sure the vr systems still work
Change-Id: I7f4331b51011574598ea3636f675be68d1ff49a0

surfaceflinger: cache HWC client targets and buffers

Remember HWC client targets and buffers, and make sure we send each
unique slot/handle pair only once.  This allows the composer to
clone/register/retain each buffer only once.

Test: builds and boots
Change-Id: Ib485189043a9c132031e82d4d7380ace3bf9453d

Merge "sensorservice: Remove ENABLE_TREBLE flag."

Merge "Explicitly sets Shell component on broadcast."

Fix small child layer transparent region error

Just turned transparent region back on in my SurfaceView port
and saw things were pretty broken.

Test: Manual
Change-Id: I3ab5520ecd226c86ef903767f6a562af7d7eb16d

Explicitly sets Shell component on broadcast.

Change-Id: I11299885c300709d4b6c5a8b4d21bc31d96e2a62
Fixes: 34833915
Test: manual verification
Test: dumpstate_test passes

Break dependency of libui on libbinder

the dependency existed only because of
HdrCapabilities, which inherited from
Parcelable, instead of LightFlatenable<>.

An added benefit is that HdrCapabilities isn’t
virtual anymore.

Also, since the dtor is not trivial, move it to
the cpp, to avoid inlining.

Fix-up includes.

Test: built and booted device

Change-Id: Iaf9c4ef9dcc5cb28b29f79d9f7d8f21800223eb8

Merge "vulkan: Disable shared present modes for now"

Merge "Fix vr flinger deadlock and race condition"

vulkan: Disable shared present modes for now

This extension is not yet ratified, and we shouldn't really be
unconditionally exposing the new present modes anyway.

Test: build
Change-Id: I0a9631a5079648c0d8981f07a9f704b7f8758ce3

Merge "Remove BoardConfig variables - BOARD_EGL_CFG"

Merge "move GraphicEnv from libgui to libui"

Merge "Fix include dependencies"

Fix vr flinger deadlock and race condition

While investigating hangs when transitioning from 2d --> vr and back I
found a deadlock and race condition in the vr flinger pause/resume
handling. This CL should fix both issues.

Unfortunately there's still another deadlock related to multiple threads
trying to suspend/resume vr flinger, but considering how vr flinger is
currently used by surface flinger we shouldn't ever hit that scenario in
practice.

Bug: None

Test: I was able to reliably get a hang when starting/stopping vr
launcher a few times, but with this CL applied and another CL to remove
calls to SetPowerMode() applied (that CL will be submitted separately),
I no longer see any hangs.

Change-Id: Ie842bf9fb00e4e2937769ed7e1e2ec9cc47861f7
(cherry picked from commit cf921a3919d68d8c8d1b8be39e03a372f6346f57)

Stop calling setPowerMode in VrFlinger

Seems that SetPowerMode from VrFlinger is causing the HIDL HWC service
(android.hardware.graphics.composer@2.1-service) to hang.

Bug: None
Test: Start particles, stop particles, start CubeSea; stop CubeSea
Change-Id: I074568fa0b04041d94dd76768ff27d467e699682
(cherry picked from commit 7fba71a24630bdf757bd4e8a28193a3995c133aa)

Merge "Integrate binder service to vr window manager for controller input."

Merge "Discard invisible surfaces"

Merge "fix warning: dereference of a null pointer"

Update the GVR platform library.

This CL renames the target for the GVR platform library and introduces
a new method in it to determine if this is an all-in-one device or a
Daydream-ready smartphone.

Bug: 34713987
Change-Id: I38067cb02fd805f3fbe1925250bddebfe82cce55
(cherry picked from commit 904762bdddd2ea5d386b1cf5fb7a35899fbc7a1e)