Merge "Properly recover vr_wm from surface vr_flinger crashes."

Merge "Update native ColorSpace API for libhwui"

Update native ColorSpace API for libhwui

Test: colorspace_test
Bug: 32984164
Change-Id: Id0728bfa90192173eb45c2d26a8b81ce970398d7

Properly recover vr_wm from surface vr_flinger crashes.

This is addressing a number of issues:

1) surface_flinger_view_ was incorrectly reset
on de-allocation, which renders vr_wm incapable of seeing
new frames after a crash.

2) dvrGraphicsWaitNextFrame will return -1 if vr_flinger
crashes, which means our graphics context is no longer valid.
We need to de-allocate resources and wait for SF to
reconnect.

3) Displays needed be be de-allocated on SF detach because
they will return incorrect fences on SF re-connect.

4) display_client_ needs to be allocated on demand instead
of at initialization time because vr_flinger crash
invalidates it.

Bug: 36589793
Test: Crash vr_flinger, observe vr_wm working after SF restarts.

Change-Id: I72a79690ee5224e3326c7c571ae4e23797103e6d

This test is used to check if the kernel scheduler

1. supports priority inheritance.
2. utilize the sync flag.

It also make statistic on average, best and worst latency
as well as number of deadline miss with a configurable
deadline specified in us.

The output is in json format hopefully to make it easy
to integrated with present test framework.

usage:

./schd-dbg                   # test and output json
./schd-dbg -i 1              # number of iterations
./schd-dbg -pair 4           # number of process pairs
./schd-dbg -deadline_us 2500 # deadline in us
./schd-dbg -v                # debug

Test: pixel
Change-Id: Idccf0d402dcd7b81c639020364b5ee32075e18cb

Merge "libdvrcommon: Remove revision code."

libdvrcommon: Remove revision code.

Move revision to libdvrplatform.

Bug: 36040117
Test: N/A.
Change-Id: I3f70509f9a694383bd7eb94173d5230bd8c66e51

Checks hidl service and remove hidl call at dump

Two things:
  * check for hidl service availability before calling any hidl
    functions.
  * remove hidl dependency on dump to ensure dump will not cause
    crash due to hidl failure.

Bug: 36073404
Test: check sensor running after reboot
Test: check dumpsys sensorservice works normally.
Change-Id: I8626635a143a9f07f263f85d95ab16749ea1e452

Merge "Revert "Quick fix to correctly handle empty file/channel handles.""

Revert "Quick fix to correctly handle empty file/channel handles."

This reverts commit e71fb7d2ab1118d30c5bec8a85114ac610844dc6.

This CL broke PDX unit tests

Bug: 36587839
Change-Id: I5b3cb857aacff337cb9a268a0415f8398303abe8

Merge "media: mechanism to set/get vendor-extension configs for OMX component"

Merge "Remove more EGL_ANDROID_create_native_client_buffer references"

Remove more EGL_ANDROID_create_native_client_buffer references

Bug: 35101209
Test: build
Change-Id: I8709b9881f0133756e2cfcfad3819d4032cfb03e

Merge "Implement an API for vrwm to query if the current vr app is active"

binder: remove cgroup interactions [ DO NOT MERGE ] am: 05f4679d03
am: 400e9a72fd  -s ours

Change-Id: Ic31695b9c7a981341ab601638b68f0bb7ef99f28

binder: remove cgroup interactions [ DO NOT MERGE ]
am: 05f4679d03

Change-Id: Idba2f747151fc50866fb470681e50ba64b28b2d0

Merge "Revert "atrace: Make atrace notify VR services of new trace sessions""

Merge "Fix typo in Region to avoid unexpected values" am: 2cbfb0cda6 am: 7fc294e10e
am: 5288e22d00

Change-Id: Ib8095bd01becfacbbcecd0f4ec369ed933eff173

Merge "Dumpstate: dump HAL native traces processes."

Merge "Fix typo in Region to avoid unexpected values" am: 2cbfb0cda6
am: 7fc294e10e

Change-Id: I7feff39e1be80c69a744c4eefad835ba50796f4c

Merge "Fix typo in Region to avoid unexpected values"
am: 2cbfb0cda6

Change-Id: I92a204fa492374c4c69a5d794d974b3cb8f454b3

Merge "Fix typo in Region to avoid unexpected values"

Implement an API for vrwm to query if the current vr app is active

The API returns if the client_order 0 layer is visible or not. This
aids in 2D app detection

Bug: None
Test: Manual with permissionsgen & calculator

Change-Id: Id988e61a9f93885e93c28ed83ffaffd84f3bdf15

Revert "atrace: Make atrace notify VR services of new trace sessions"

This reverts commit 213feb32c76bb4ed4262dad8c71f0377ee710fac.

Reason for revert: tracing & perf teams don't want to support PDX at this point in time.

Change-Id: I28b8a7c55c9d31c8469236fb60a6e034b1ef8af7

Merge "Enable Universal Bandwidth Compression for textures"

Enable Universal Bandwidth Compression for textures

We removed texture compression for VR display surfaces, which introduced memory
bandwidth issues on some QC platforms.

This patch reintroduces texture compression for QC devices compatible with
both gralloc0/gralloc1 implementations.

Bug: 36370337
Test: Validate VR comes up
Change-Id: Ieaaa56f67522c85913476ed0d8da04f8eedbce63

Dumpstate: dump HAL native traces processes.

Test: hal service dumps in data/anr/bugreport-traces.txt after bugreport
Bug: 36414311
Change-Id: Ifca491d02f77bf4fb26450f06c2ea02769475786

media: mechanism to set/get vendor-extension configs for OMX component

Add OMX_CONFIG_ANDROID_VENDOR_EXTENSIONTYPE and
OMX_CONFIG_ANDROID_VENDOR_PARAMTYPE.

Android vendor extensions for OMX components have a name and one or
more parameter values - each with a string key - that are set together.
Support for not-set values is provided for vendor extensions in order
to not suddenly confuse applications with format keys that they were
not expecting.

Bug: 32746650
Change-Id: I21aad4a82d509fe2466764f65ee9e3b2d32f0842

Merge "Re-add access of DisplayClient from vr_rm"

Re-add access of DisplayClient from vr_rm

This was removed in a previous commit because it relied on an in-flight
CL that had not yet landed. Now that it has landed, the change is back
in.

Bug: 36139334
Test: Compile and run VR app, confirm that touch region is correct
      through hierarchy viewer.

Change-Id: I4ecc5e050fd896ba7442eac8b3a4e0b793915ed7

binder: remove cgroup interactions [ DO NOT MERGE ]

Having binder change cgroups on the fly is too expensive in the common case to be useful.

Test: builds

bug 34825105

Change-Id: Ia692c9d2ab261d852cb2fac8f27d6779d0e2b057

Merge "Check board config for wide-color color spaces"

Merge "VR: Add dvr API for service apps connecting to VR HWC"

Check board config for wide-color color spaces

Check that board config allows wide-color color
spaces.
Test: make tests in libs/gui/tests/
Test: adb sync
Test: adb shell /data/nativetest/libgui_test/libgui_test --gtest_filter="SurfaceTest.GetWideColorSupport"

bug #35996305

Change-Id: Ia5b8116bc25befb6491e72e67cd8ed3c90ca5698

Merge "Quick fix to correctly handle empty file/channel handles."

Merge "libpdx: Remove superfluous DispatchRemoteMethod overloads."

Add notification_title and notification_description to dumpstate

Initiator of dumpstate can set dumpstate.options.title and
dumpstate.options.description properties to notify dumpstate the details
that will be used to file a bug. dumpstate will send this information
along with the BUGREPORT_FINISHED intent.

Bug: 33561517
Test: Use the new API to invoke bugreport from TelephonyMonitor and
      ensure that the bugreport is taken and title and description are
      updated properly in the notification. Run dumpstate and
      BugreportReceiver unit tests

Change-Id: Ia23a3ef4d5751be14f347ac38ef7c8a18f79799d

Merge "Follow "atomic" to "group" refactoring."

Merge "lshal: --init-vintf exclude system hals" am: 8db626d2f3 am: 34206360f2
am: d1b6b40adb

Change-Id: Ie2d28a374a3fb18d969783394b80b3a0d6fe0869

Merge "lshal: --init-vintf exclude system hals" am: 8db626d2f3
am: 34206360f2

Change-Id: I21d0b37f6942204c748017e03e8c9a0e892a39dc

Merge "lshal: --init-vintf exclude system hals"
am: 8db626d2f3

Change-Id: Ibe4636eb13c9439d09f51948ca49921a1e209747

Merge "lshal: --init-vintf exclude system hals"

Merge "Fix BlockUntilVSync() to actually block"

Follow "atomic" to "group" refactoring.

Remove noisy logging about UIDs that are relying on default cache
quota of 64MiB.

Move away from yucky old statfs() and use statvfs() instead.

Test: /data/nativetest/installd_cache_test/installd_cache_test
Bug: 35812899, 35684969, 36482620
Change-Id: I3d68da97eac2ebcda489bdf9d27061cac5b3f7cc

Merge "SurfaceFlinger: Missed changes in HWC1. Fix build error."

lshal: --init-vintf exclude system hals

--init-vintf is provided for convenience so that vendors can initialize
their vendor interface. Excluding system hals so that they aren't
included in the default vintf file that is generated. They are
maintained manually.

Test: Verified frameworks hals are no longer visible in init-vintf.
Bug: 34772739
Change-Id: Ie40c384334affcb3abee063d58000b808f04e234

Fix BlockUntilVSync() to actually block

In my refactoring in ag/2008740 I introduced a regression where blocking
until the next vsync would busy loop instead of actually blocking on an
event on the vsync fd. The problem was I changed the poll() call to
listen for POLLIN in addition to POLLPRI. While waiting for vsync events
we only want to listen for POLLPRI.

Bug: 36495351

Test: Confirmed with debug logs the busy loop bug and that we correctly
block with this patch applied.

Change-Id: Iec4951e014575a5d0ed3cfe3fc20ea91c67edf1b

SurfaceFlinger: Missed changes in HWC1. Fix build error.

Apparently HWC1 isn't in presubmit anymore.

Test: Manual
Change-Id: Id173d71be62b8a3c888943c57c0ae0ec511bc94b

Merge "dumpstate: Remove FWDUMP_bcmdhd support" am: 86388dbc9a am: 2d71f97dfb
am: b09149f6ec  -s ours

Change-Id: Iba7447b4f5eb32cbdd480b69fcc369d1ee1a3e3f

Merge "batteryservice: add scheduleUpdate binder method" am: e007716bc4 am: 799bb3501a
am: cd1d8c6b10

Change-Id: I6fcf1788701a7b3c9e481a2bdfbdd352cf24d598

Merge "dumpstate: Remove FWDUMP_bcmdhd support" am: 86388dbc9a
am: 2d71f97dfb

Change-Id: I1e3a7ae0e7855a8003f463586d368a81ad89be5f

Merge "batteryservice: add scheduleUpdate binder method" am: e007716bc4
am: 799bb3501a

Change-Id: Ia1ef3d769c453bc58cc52b93df5d6f03e601d23e

Merge "dumpstate: Remove FWDUMP_bcmdhd support"
am: 86388dbc9a

Change-Id: Iadf37acb012f762919567354a7aede95f81b9eff

Merge "batteryservice: add scheduleUpdate binder method"
am: e007716bc4

Change-Id: I8565c0502c3838d74fcd09ec75677e65cf85a0a9

Merge "dumpstate: Remove FWDUMP_bcmdhd support"

Merge "batteryservice: add scheduleUpdate binder method"

Merge "BQ: Resend queued buffer handles after disconnect"

Merge changes Ic8309518,If3199d7c,Ie4d32162

* changes:
  SurfaceFlinger: Inherit Alpha for child surfaces.
  Add SurfaceFlinger test filter for presubmit.
  Include finalCrop in setGeometryAppliesWithResize.

Merge "vrwm: Add 2D Mode + control app change"

Merge "dumpstate: Remove FWDUMP_bcmdhd support"

Merge changes from topic 'vendorbinder'

* changes:
  Add vndservicemanager.
  Allow ProcessState to use another binder dev node.

Merge "Reduce vr_wm logging."

vrwm: Add 2D Mode + control app change

2D mode is similar to debug mode in that it will always show the
application layer if it exists, but unlike debug mode will strip any
system layers from the output and will attempt to recognize the VR
layers. The virtual display is always in 2D mode as it can only show
2D apps. The primary display can be switched by vr_wm_ctl

Bug: None
Test: Manual with permissionsgen and launching calculator app
Change-Id: I38d025c457b1fca9aeb47cb94ed481ac9ff084f9

BQ: Resend queued buffer handles after disconnect

When the producer disconnects from a BufferQueue, all of its slots are
cleared and buffers in the queue are marked as stale. When the slots
are cleared, we notify the consumer that buffers were released so that
it can clear its cache of slot-to-buffer mappings.

However, when we were marking queued buffers as stale, we were not
clearing the bit that dictates whether the BQ needs to send a buffer
handle upon acquire, so in the case where the producer queues a buffer
and disconnects, and then the consumer acquires that queued buffer,
the consumer doesn't get the buffer handle, nor can it retrieve it from
its cache, because it has (or should have) already cleared it.

This change clears that bit upon disconnect.

Bug: 36453246
Test: New test in libgui_tests + Vulkan CTS test from bug
Change-Id: Ic27f119a4164878257c3841b4e1d421694159774

SurfaceFlinger: Inherit Alpha for child surfaces.

Necessary for various animations to work correctly and
for the general goal of treating child surfaces like
pixels in the parent surface.

Test: Includes new test in SurfaceFlinger_test
Bug: None yet ;)
Change-Id: Ic8309518bd327bbba6ad2f79f265e24cf37898e7

Add SurfaceFlinger test filter for presubmit.

HWC2 tests and SurfaceInterceptor tests aren't yet passing. See
second attached bug for more context on test filters for pre-submit.

Test: Verify Surfaceflinger_test.filter appears in output folder after build.
Bug: 36117490
Bug: 36359778
Change-Id: If3199d7c005722c235f065766f2961e2ed2d6e0d

Include finalCrop in setGeometryAppliesWithResize.

A refresher on this archaic function. Normally window crop and
position apply immediately. This is so the window manager can always
crop surfaces (e.g. while docked resizing) even if the app is lagging
behind. In some cases though the apps position or cropping may depend
on whether the resize has occured or not. For example when an app
gains shadows, we need to move expand the Surface and move it by a
negative offset for the content to remain in the same place. This
movement needs to be synchronized with the buffer latching for the
expansion.

We implemented setGeometryAppliesWithResize to take care of this. At
the time it wasn't clear if it was needed for finalCrop so we let it
be. Now that we are exclusively using final crop in the pinned stack
however it is required. The same way we have an issue with position
and shadows, we have an issue with expanding the crop rect (we may
have actually been cropping out part of a larger surface that is now
supposed to be shadows).

Also includes a test suite for setGeometryAppliesWithResize behavior
which was previously untested.

Bug: 35396882
Test: SurfaceFlinger_test
Change-Id: Ie4d32162eb21154496bb231161692d18341a1e1e

Merge "Fix virtual touchpad re-attachment."

Merge "Fix janky pose"

Merge "VR: Add VR Hardware Composer Service"

Reduce vr_wm logging.

Controller events ALOGI -> ALOGV.

Change-Id: I942b6b88cc9f3fe35efcb1e3c1ce0c478d1281e1
Test: manual

dumpstate: Remove FWDUMP_bcmdhd support

Bug: 35983944
Test: Manual

Change-Id: I4fe1c5d755940d84967d0bd1270ab86638e0311a
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>

batteryservice: add scheduleUpdate binder method

Allows a caller to force an update to occur, instead of waiting
for an event or timer timeout.
Test: None (interface change)
Bug: 32645990

Change-Id: I1c94f1bed34d5f4cfe8e7d50cca4e5391dac2d62

Merge "Use max acquired buffer setting to configStore"

Quick fix to correctly handle empty file/channel handles.

Empty file/channel handles (value < 0) should result in a negative
FileReference/ChannelReference value. The original code incorrectly
interpreted the negative reference value as an error, causing errno
to be returned. In most cases errno is 0, leading to confusion
across the wire when the deserialization logic attempts to lookup an
invalid, but non-empty reference.

Bug: None
Test: bufferhub_tests passes
Change-Id: I664daf3cdc178fe2c4ca0c44eef35a8bb9c7310d

libpdx: Remove superfluous DispatchRemoteMethod overloads.

The introduction of RemoteMethodReturn<>() properly handles the
differences between most of these overloads. Remove the ones that
are now obsolete in favor of the generic version.

Bug: None
Test: make -j30
Change-Id: I23e63431e9f668c324b408707ce65e7d6aee69c8

Merge "Fully strip META state during event rewrite"

dumpstate: Remove FWDUMP_bcmdhd support

Bug: 35983944
Test: Manual

Change-Id: Ia527bbe950b0f569a07e68805684d11ba2f71c66
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>

Add vndservicemanager.

vndservicemanager is the context manager for
binder services that are exclusively used from
the vendor binder domain, "/dev/vndbinder";
these services can only be registered and accessed
by vendor processes.

To facilitate code re-use, vndservicemanager
is just framework servicemanager accessing the
vendor binder devnode.

Bug: 36052864
Test: modified binder_test to use /dev/vndbinder

Change-Id: Iac19a1180321a5be78d7be5ceb4ba211c593dc47

Merge "Replace 'cpuset' init option with 'writepid'"

VR: Add dvr API for service apps connecting to VR HWC

Bug: 36050795
Test: Compiled
Change-Id: I447fe56f7585ed3774bdc3577389d1f9b8281e7c

VR: Add VR Hardware Composer Service

Create a separate service with serves the binderized HWComposer
implementation for VR.

Exposes a binder service which VrCore may connect in order to receive
SurfaceFlinger output.

Bug: 36051907
Test: Compiled and ran unittests
Change-Id: I16ec08d32ba0445ac982fc235e59bc3129e03096

Fix virtual touchpad re-attachment.

If the client crashes without detaching the virtual touchpad
and a new instance reconnects, it should take over the existing
device without attempting to configure it again.

Bug: 36051900
Test: no
Change-Id: I446f1bc7cb9c08f914b9bfc653857a97fdc7839c

Fix janky pose

It's introduced through ag/1980993, where render_buffer_index_ was not
managed after the refactor.

Bug: 36194745
Test: Build and flash, pose is no longer janky.
Change-Id: I6ef86625dbc1c1afd2725614062f5e40e1b7f60a

Merge "Remove mediacodec from native_processes_to_dump."

Remove mediacodec from native_processes_to_dump.

Test: Camera, Photos, YouTube, and Play Movies apps.
Bug: 35328855
Change-Id: I098fe262e414822419c1e08e5f891e8e2b13cdc9

Fully strip META state during event rewrite

Bug: 36288555
Test: Logged keys coming into PhoneWindowManager to verify that
      meta state is stripped.

Change-Id: If51210517845464de185997f7c71a3888ee6af56

Merge "Update re hwservicemanager debugDump semantics." am: ed2debbb47 am: eec11e4cf5
am: 1cf95ad88a

Change-Id: Ieb69025cf922fbcc4e838c88d8701c8a8a5abfb7

Merge "binder: remove cgroup interactions" am: c12c6dca4e am: 62f07e540f
am: bcb3d7fd03

Change-Id: I860e9a7565870681f2cf841a7f866b9ca22e3f22

Merge "Add system feature for Companion Device Manager"

Allow ProcessState to use another binder dev node.

Since we have different binder domains at different
/dev nodes now, processes using libbinder must be able
to select which /dev node they want to use. This is
achieved with a new ProcessState::initWithDriver() API,
which takes the /dev node to talk to as an argument.

Bug: 36052864
Test: system boots without issues
Change-Id: I5b3b13decf254ca0957665c91abac4b287e448c4

Use max acquired buffer setting to configStore

Change-Id: I48454243bcac12f5a83d080cd9115686f9f728ca

Merge "Update re hwservicemanager debugDump semantics." am: ed2debbb47
am: eec11e4cf5

Change-Id: Ib6813604432530b9e09c1848f89d12dabd348b20

Merge "Update re hwservicemanager debugDump semantics."
am: ed2debbb47

Change-Id: Ibcac4fabedc0f3a1b310d02510785e4c46261a3b

Merge "Update re hwservicemanager debugDump semantics."

Merge "binder: remove cgroup interactions" am: c12c6dca4e
am: 62f07e540f

Change-Id: I2d52ec71454f41500fb1fd2963b34984cfeb2e7d

Merge "Revert "Revert "Tie vr flinger to persistent vr mode"""

Merge "binder: remove cgroup interactions"
am: c12c6dca4e

Change-Id: I98cba352689749b2f5f4306453985dad83b98ed8

Add system feature for Companion Device Manager

Bug: 30932767
Test: Add just the feature gating first, ensure Context#getSystemSetrvice now
returns null.
Add feature in the configuration files, ensure companion device feature works again.
Change-Id: I4ad2d52f7877eae0e951f43ddbb23881d5de8d8b

Merge "binder: remove cgroup interactions"