Merge "Revert "Remove NativeWindow and unused DVR functions""

Merge "Allow "android" package name."

Merge "Fix the usage of EIGEN_STATIC_ASSERT_SAME_MATRIX_SIZE"

Merge "A helper header for matrices."

Revert "Remove NativeWindow and unused DVR functions"

This reverts commit a001af5ae07e156f9c31dcdb24fb7ddead4d9f1e.

Change-Id: Ieb60bc8e8e7caefee30b0fbd13cda15298c32075

Fix the usage of EIGEN_STATIC_ASSERT_SAME_MATRIX_SIZE

  - Eigen::internal::traits is not specialized for Eigen::MatrixBase, it
  should only be used with the Derived types. Just as Eigen's own tests
  do.

Bug: 34161771
Test: mm
Change-Id: I997b26b4a470c4ba851afe6edc5c602ae450c222

Allow "android" package name.

Test: /data/nativetest64/installd_utils_test/installd_utils_test
Bug: 36062649
Change-Id: I9b41a34822a1a194d256f5b99d5955e51716029c

Merge "Remove NativeWindow and unused DVR functions"

Merge "Interface for multiple virtual touchpads."

Merge "libbinder: use sysconf(_SC_PAGESIZE) to get pagesize" am: b638bf8d3f am: f83403fdd2
am: 391becb8a7

Change-Id: I11546e02a736bd6c6ece606831462f7ef85c4ec4

Merge "libbinder: use sysconf(_SC_PAGESIZE) to get pagesize" am: b638bf8d3f
am: f83403fdd2

Change-Id: If136ef494a22fb750321272302db7a0d8144eb95

Merge "libbinder: use sysconf(_SC_PAGESIZE) to get pagesize"
am: b638bf8d3f

Change-Id: I90088d0769cfe337b2c2c5c308f5fa7e832ca4df

Merge "libbinder: use sysconf(_SC_PAGESIZE) to get pagesize"

Merge "Update is_valid_package_name() to match framework."

Update is_valid_package_name() to match framework.

The framework has a new format for /data/app directory names in OC
to protect user privacy.  This change updates the validation logic
in installd to support the new format.

For consistency the logic now mirrors the package name validation
logic used by PackageParser.

Test: /data/nativetest64/installd_utils_test/installd_utils_test
Bug: 33705143
Change-Id: I41d270942479c52e063c29a2cb9552d7b238b86e

Merge "Fix enum ordering in sensor NDK"

Merge "Temporarily disable vr_wm visibility while we figure out what triggers it to come up at all time. This is to quickly unblock everyone."

Fix enum ordering in sensor NDK

Test: compiles
Change-Id: Ie186270af2f51d723855cdc62f0b09ffe1ab6ecf

Merge "Sensor direct report with HardwareBuffer channel"

Remove NativeWindow and unused DVR functions

1/ Remove the NativeWindow-backed DVR DisplaySurface implementation.
2/ Unused API such as: dvrCreateDisplaySurface and friends are removed.

Bug: 36033302
Test: Built system, ran basicvr, ran particles on Marlin. Also updated
graphics_app_tests to stop testing deprecated API.

Change-Id: I6ea6063c56fabacddcc301bc91567cc1b3f5d67e

Merge "Disable Eigen static assertion for tests/numeric_test.cpp"

Temporarily disable vr_wm visibility while we figure out what triggers it
to come up at all time. This is to quickly unblock everyone.

Bug: 36036583
Test: Compiled
Change-Id: Ibf045e7354ee96b7609af1c47b1e64b63659feb9

Merge "Add ASensorEventQueue_registerSensor and default error value"

Disable Eigen static assertion for tests/numeric_test.cpp

  - Clang complains with the Eigen static assertions in the tests, after
  Eigen 3.3.3 rebase.
  - Disable the static assertion until it is fixed in Eigen / Clang.

Bug: 34161771
Test: mm
Change-Id: I25056cf9df9bbb32f979390e02df1f23ac760bab

A helper header for matrices.

Bug: 34720474
Test: No functional change.
Change-Id: Ibcb8f11feeeadf6f484396d66c0b0cd1ef751f94
(cherry picked from commit 6fff8f0580e352fd81ecd7a0b6b58a6604bcbed7)

Merge "Add MiniFence to drop HWC2on1Adapter libui dep"

Merge "[SurfaceFlinger] Comment SurfaceFlinger HWC2"

Sensor direct report with HardwareBuffer channel

Implement creation of sensor direct channel in sensor
service for sensor direct report feature.

Bug: 30985702
Test: cts-tradefed run cts --module CtsSensorTestCases
      --test android.hardware.cts.SensorDirectReportTest
Test: cts-tradefed run cts --module CtsSensorTestCases
      --test android.hardware.cts.SensorNativeTest

Change-Id: I52350811df538986cea7f5e1fb336fad53234c86

Add ASensorEventQueue_registerSensor and default error value

ASensorEventQueue_registerSensor is new function created in 2016 but
is never exported at NDK level. In this CL the function signature is
formalized and the function name is added to NDK function export
table. Internal class SensorEventQueue is also changed to adapt.

Default error values are also defined.

Test: cts-tradefed run cts --module CtsSensorTestCases \
          --test android.hardware.cts.SensorNativeTest
Change-Id: I74dc72e50ea4bae6d2d8944ce6d2e191ed2f495e

Add MiniFence to drop HWC2on1Adapter libui dep

This class will soon become a library used by devices with no support
for HWC2. They will link against libhwc2on1adapter. Drivers should have
no depedencies on libui.

Test:Manual
Change-Id: Iabd2aa89fc3b737a999632a16c4f6c30464787c4

Merge "split libsensor our of libgui "

Interface for multiple virtual touchpads.

This passes the touchpad selection across binder, but does not yet
actually implement multiple touchpad devices.

Bug: b/35992608
Test: primary only in VirtualTouchpad_test
Change-Id: I7529735362e0e38805b9c0e52e07b5efc86bd535

split libsensor our of libgui

Test: compile & run
Bug: treble cleanup
Change-Id: I5b2c2a14f7d3b364d25695cf0901c5b08cfb0fc9

Merge "Installd: Clean up code related to foreign dex use" am: 62b34b57c4 am: 9d4819ff2e
am: 9181bdb57f

Change-Id: I2257654b3b73e74a92a179e1068f771e2b6af186

Merge "Installd: Clean up code related to foreign dex use" am: 62b34b57c4
am: 9d4819ff2e

Change-Id: I21299c36def0a5aa5e202d381abd4ad184c2e7b6

Merge "Installd: Clean up code related to foreign dex use"
am: 62b34b57c4

Change-Id: Id7beb09ed3d3b79b3b818dd5192b457605f5a54c

Merge "Installd: Clean up code related to foreign dex use"

Merge "Fix the compiler error for the ternary operator, caused by Eigen rebase."

Merge "Disable HDR extension for now"

libbinder: use sysconf(_SC_PAGESIZE) to get pagesize

pagesize is not always 4KB, use sysconf(_SC_PAGESIZE) to get
real pagesize.

Change-Id: Ib2c82c3a842257601a5c304da3a1f0b07c6ab8c0
Signed-off-by: Ganesh Mahendran <opensource.ganesh@gmail.com>

Merge "InputDevice warning fixes and make warnigs into Error"

InputDevice warning fixes and make warnigs into Error

use const char * for string literals in InputDevice
Remove unused variable in Keyboard

Bug: 35993509
Test: mm in frameworks/native/libs/input builds without the warning
Change-Id: I11242835eae855386c47e56d6c36e5a2f912a28f

Disable HDR extension for now

Test: adb shell vkinfo
Test: Should not see VK_EXT_hdr_metadata in extension list.

bug: 36007564

Change-Id: I96a365d75ddec75856af2fda834d3a4f8368ebb0

Fix the compiler error for the ternary operator, caused by Eigen rebase.

Bug: 34161771
Test: build
Change-Id: I3c0c1b3667a3d1f56052b247c94c6255e9e12004

[SurfaceFlinger] Document layerStack

Test: None, these are only comments.
Change-Id: Id5d45a5847d253bf92263091e5b322a1577fabe1

Merge "Double check hidl service at init + meaningful abort when hidl crash"

Merge "Make the AHardwareBuffer extension compatible with C."

Merge "lshal: update descriptions in output." am: 074c1cd157 am: 105e9391ac
am: 76ccc8cbd1

Change-Id: Iadb1ea22c087d643d0664919411a3e2408d12fd1

Merge "lshal: update descriptions in output." am: 074c1cd157
am: 105e9391ac

Change-Id: I9dfa8644300c91bc2149e9f7f4ba5c978b85ecf5

Merge "lshal: update descriptions in output."
am: 074c1cd157

Change-Id: Iaa60e243566133d1674b2591becdbc9ead403754

Double check hidl service at init + meaningful abort when hidl crash

Sensor hidl service will kill itself when it detects
double-connection(*) from framework sensor service in order to avoid
potential deadlock.  This cl adds a no-op call to hidl service at
initialization of framework service to allow early detection on hidl
service side. It also adds retry mechanism in this case to avoid
another system server crash.

Log messages is added when hidl call transport failure to help better
identify problem. There is no sane recovery at this point and fatal
log is preferred.

* Double-connection to hidl service happens when framework sensor
service crash but leave a lingering connection in sensor hidl service.
Auto restarted framework service will attempt to connect hidl service
again which result in a double connection.

Test: manually killing sensor hidl service and/or restart runtime,
      with and without sensor service in restricted mode, observe
      system recover to good state.

      Pid of sensor hidl service can be found by lshal
          lshal | grep sensor
      switch to restrict mode
          dumpsys sensorservice restrict a.non-exist.package

Bug: 35727090
Bug: 35843387
Bug: 35702298
Bug: 35919167
Bug: 35848670
Bug: 35903635
Change-Id: Ie5c24300503e125d9de031a70948e2bda1eb6dc6

Merge "Fix <vulkan/vulkan.h> compatibility with C."

Merge "Whitelist EGL_KHR_no_config_context"

Merge "Provide a library interface to the virtual touchpad service."

Make the AHardwareBuffer extension compatible with C.

The eglext.h file must be C-compatible, and is included by vendor C
code. C does not support magicking up the 'struct' keyword.

Fixes the change made in I0da1173688cd27b8dd9e907e535100b9a0605290

Bug: 34050596
Test: built, ran demo app using extension
Change-Id: Iaa3028c3ae2087f7368549565a778ce75110d3cb

Merge "lshal: update descriptions in output."

Provide a library interface to the virtual touchpad service.

And make VrWindowManager pay for^H^H^H^H^H^H^H^H use it. The library
interface is a step toward moving 2D embedding into VrCore because it
hides the binder connection that isn't directly accessible to VrCore.

As a matter of implementation, the touchpad-service client library
and the low-level evdev touchpad are derived from the same interface.

Bug: b/35990873
Test: VirtualTouchpad_test; integration TBD.
Change-Id: Ic922ff223ddd5a44f6fb4433a271f8341e93a0e7

Whitelist EGL_KHR_no_config_context

This change allows driver to expose EGL_KHR_no_config_context
functionality.

Test: dEQP-EGL.info.extensions, dEQP-EGL.functional.create_context.no_config
Bug: 22093220
Change-Id: I4dee8577f2fa7378c1f2512cbdc568670d1c036e

Use valid libvulkan error results

Several of the error results were not valid according to spec.
Use values that are listed as possible results.

Bug: 35752097
Test: dEQP-VK.wsi.android.incremental_present.*
Change-Id: Ie7830e29b523265d62cd82638af7f6e34a36565b

Merge "Small change for testing." am: f88834e30d am: e89136bbe2
am: f67181001f

Change-Id: I3e86e4996e59266076fa6a0dfcc07a1302b34b9a

Merge "Small change for testing." am: f88834e30d
am: e89136bbe2

Change-Id: I489c109cffb55becae53ac957645422c2d9830c9

Merge "Small change for testing."
am: f88834e30d

Change-Id: I8ca5fd99ae37093e912ab699e976c05d446e2c81

Merge "Small change for testing."

Small change for testing.

Bug: 35970955, 35969439, 35968280
Change-Id: I56c6626221b456e13b1371b69e9c980edaddfa91

Installd: Clean up code related to foreign dex use

We simplified the way we track whether or not a dex file is used by
other apps. DexManager in the framework keeps track of the data and we
no longer need file markers on disk.

Also, fix the getUserSize by considering the size of the actual profiles
not of the set of empty markers.

Test: device boots, foreign dex markers are not created anymore

Bug: 32871170
Change-Id: I9250e816a1bfa95cb10155e513ae28ebfda7ce4b

[SurfaceFlinger] Comment SurfaceFlinger HWC2

Document decision to clear SurfaceFlinger HWC2 state upon sleep and
awake.

Test: This is just a comment.
Change-Id: I0034d8d79e6e0d5b6287579d23f941e291c89bc0

Merge "fallout of cleaning-up libutils includes"

fallout of cleaning-up libutils includes

Test: run & compile
Bug: n/a

Change-Id: I99e462d2db6560742fa3f065d5de50cd60111130

Merge changes I121df86d,Ia692a5fd

* changes:
  Add Vulkan to native helpers
  Update color space names to match extension

Merge changes from topic 'colorspace_support'

* changes:
  Use Surface::GetWideColorSupport in VK_EXT_swapchain_colorspace
  Add plumbing for Surface::getHdrSupport
  Add plumbing for Surface::getWideColorSupport

Merge "[surfaceflinger] Set blend mode correctly when using HWC2."

Merge "Fix black screen upon awake (reused after free)"

Merge "Delete unused variable in setUpHWComposer"

Merge "Add the missing Parcel.writeInterfaceToken call."

Add the missing Parcel.writeInterfaceToken call.

Bug: 35950090
Test: Verified that it fixes the following error.
      Parcel  : **** enforceInterface() expected 'android.app.IActivityManager' but read ''
Change-Id: Id70da2d2ba7198e4d5d14aeabe00328c4eb6ef14

Fix black screen upon awake (reused after free)

Under a specific timing, the state of the HWC can be left populated
with "old" layers. If a Refresh Message comes before the WM sends
a Transaction resetting the layer, the old layers are sent to the
HWC. These old layers features buffer_handle which had been freed,
resulting in a frame never signaling and a locked screen due to
backpressure triggering constantly.

Detailed explanation:

When the device power off is pressed, the WM plays the fadeout
animation, set the layer "fadeOut" to displayId -1 and calls
setPowerMode(Off). This sequence of events leads to a state where
no layers are deemed visible in surface flinger but the HWC2 still
has one.

Upon awakening the device, if a Refresh message is received before
the WM has the time to reset fadeOut layer displayId to its original
value, the HWC2 will use its last known state.

Sequence of events (with and without crash):

Put device to sleep.
Awake device.
PowerOn
Refresh <- crash

Put device to sleep.
Awake device.
PowerOn
Transac
Refresh <- Ok

Bug: 35712353
Test: AUPT (100 cycles sleep/awake without black screen lock)
Change-Id: If9177ccc0627f3b733030ed5e462dc465e63e963

Delete unused variable in setUpHWComposer

Test: AUPT and Manual
Change-Id: I53fbd7f9fa83b8729554a021be526a5fb71e2b13

lshal: update descriptions in output.

Test: pass
Change-Id: I912e2153e531a35ecdf1e76fd6bafa68880b8721

Merge "Tie VR mode support to a property"

Merge "libvulkan : Handle VK_PRESENT_MODE_MAILBOX_KHR CASES correctly"

Merge changes from topic 'lshal_32' am: 2eaf6b8eae am: 407ec93f4a
am: 8810351ddc

Change-Id: Id15799972ddead9152ac4f8bd9f1ddc071f16f89

Merge changes from topic 'lshal_32' am: 2eaf6b8eae
am: 407ec93f4a

Change-Id: I222d3eb23b358ff2dda442027eeb3e7944b027d9

Merge changes from topic 'lshal_32'
am: 2eaf6b8eae

Change-Id: Id18cdd5d27d2624c7ae0f7a60b178d5bf1c71c05

Merge changes from topic 'lshal_32'

* changes:
  lshal: Put more description to output
  lshal: also list libraries in 32-bit.

Add Vulkan to native helpers

Add helpers to convert Vulkan colorspace and pixel format
to native formats. Also add more error checking when
those conversions fail.
Test: TBD CTS test.

Change-Id: I121df86dfddc3c10cbf5da1d9448d9ad01091f3d

Use Surface::GetWideColorSupport in VK_EXT_swapchain_colorspace

Use the newly added Surface::GetWideColorSupport to
add wide color SurfaceFormats when appropriate.

Test: compile Vulkan CTS test TBD
Change-Id: I85dbe9617b8eba8b89c3ad788d5ffc9f0d66f935

Update color space names to match extension

Test: make
Change-Id: Ia692a5fdc466a28c55cf0898cb0ea0bcca06aece

Add plumbing for Surface::getHdrSupport

Add plumbing for future Vulkan and EGL extensions
VK_EXT_swapchain_colorspace
EGL_EXT_colorspace_scrgb_linear
EGL_KHR_gl_colorspace

Test: TBD
Change-Id: Id1f28cfb36828adbdc6dd3694b5943eca038c356

[surfaceflinger] Set blend mode correctly when using HWC2.

HWC1 code will reset blend mode to None using setDefaultState() but
HWC2 just leaves the layer in the previous state. The result is that
once blend mode is set to something other than HWC2::BlendMode::None
we can never go back to HWC2::BlendMode::None and HWC2 implementations
are forced to use blending when not necessary.

Bug: 35915184
Test: manually verified using overdraw feedback on arc enabled chromebook
Change-Id: I86957439bad0a1be9faa56df491d8415c4cdd2c2

libvulkan : Handle VK_PRESENT_MODE_MAILBOX_KHR CASES correctly

The vulkan swapchain creation code set async mode for
VK_PRESENT_MODE_MAILBOX_KHR cases. However it goes so after
setting the buffer count for the native window being used.
This causes a mismatch between the minUndequeuedBuffers and
maxDequeuedBuffers count maintained by the libvulkan layer and
buffer queue code.
Fix this by setting the async mode correctly before the buffer
count for a native window is set.

Bug 200266758, b/32882690

Change-Id: I5d5e18e659b8d31f3d3fd9570b18a92f83a1516d
Test: dEQP-VK.wsi.android.incremental_present.*.mailbox.*

Add plumbing for Surface::getWideColorSupport

Add plumbing for future Vulkan and EGL extensions
VK_EXT_swapchain_colorspace
EGL_EXT_colorspace_scrgb_linear
EGL_KHR_gl_colorspace

Test: make tests in libs/gui/tests/
Test: adb sync
Test: adb shell /data/nativetest/libgui_test/libgui_test --gtest_filter="SurfaceTest.GetWideColorSupport"
Change-Id: Ibb182d02c468d8f24130545187096d1abe5fc30e
(cherry picked from commit d634f970fabe965a421f50011bb1167d6b7cac86)

Tie VR mode support to a property

Bug: 35854545
Test:
  - Validate VR mode is not supported when ro.boot.vr is unset
  - Validate VR mode is supported when ro.boot.vr is set

Change-Id: I26f7e851766eab0d8a5a2f9fb4d72bfcee0c42f3

Merge changes from topic 'secondary-dex' am: ad79651822 am: ba1d417eca
am: 431856d7a6

Change-Id: I3444fa1e87d54c67b1aa11a576f27e3a0447953d

DEXOPT_FORCE may be passed for primary apk as well. am: 2ab108c5be am: b77cb8b653
am: 64f3d11dd7

Change-Id: Ibce43134609ce8e188238f0666c05c7b174c3be4

[Installd] Clean up logic for secondary dex oat files am: c9eab382fa am: 1061d48614
am: c8bf9b2ff0  -s ours

Change-Id: Idc79cf49a6781bb7012db4202f66d78fd4d3e272

Add installd logic for compiling secondary dex files am: 80a2125b85 am: 51b4f988c9
am: f8bab369cc  -s ours

Change-Id: I726d685368c4d0eab8e41bb9c11d581de5fddd7a

lshal: Put more description to output

Split the output by three parts according to the source of information.
Describe each part before the table.

--sort will be applied to each table individually.

Bug: 35803917
Test: lshal
Change-Id: Ief0dae21fdeb58ebaed46d2aa68f298b8b75218d

lshal: also list libraries in 32-bit.

Add an "Arch" column (selected by -r) to lshal
to show whether the HAL runs in 32 bit or 64 bit.

* For binderized services, whether the process
  runs in 32bit or 64bit (__LP64__)
* For passthrough libraries (-impl.so), whether
  the library is in /{system,vendor,odm}/lib/hw
  or /{system,vendor,odm}lib64/hw

Bug: 35803184
Test: lshal -itrpc
Change-Id: I328da4ad9eacbf2959be4ac2e478c16535a89068

Merge changes from topic 'secondary-dex' am: ad79651822
am: ba1d417eca

Change-Id: Ie579b31a131b475449fec93f0bf5527677022edd

DEXOPT_FORCE may be passed for primary apk as well. am: 2ab108c5be
am: b77cb8b653

Change-Id: I83f8e20747b6f001a7ec44a1f708d71811a81f51