Merge changes I1ed97f98,I32449355,I5c84d6f2,Ib9652278,I73deaa44 into oc-dev
am: ba55e0e5c8

Change-Id: I975c28ce923c101952ca40719473d56f19ac4856

Merge "dumpstate: add dropbox crashes to bugreport" into oc-dev
am: 77e270d0ff

Change-Id: I5ae4acc8377c87c4a581929f81899a1693f602d7

Merge changes I1ed97f98,I32449355,I5c84d6f2,Ib9652278,I73deaa44 into oc-dev

* changes:
  libgui: Make IConsumerListener a SafeInterface
  libgui: Format IConsumerListener
  libbinder: Support Flattenable in SafeInterface
  libgui: Add missing FenceTime header to GLConsumer
  libgui: Fix naming/enums in ISurfaceComposerClient

Merge "dumpstate: add dropbox crashes to bugreport" into oc-dev

Merge "Use Configstore to test getWideColor function" into oc-dev
am: 224306b0ac

Change-Id: Ie50093ad0177e6e8642b096bb05739ffaeaa6492

Merge "Use Configstore to test getWideColor function" into oc-dev

dumpstate: add dropbox crashes to bugreport

Adds system crashes from the dropbox to the end of the bugreport
to allow diagnosing issues even after the device has rebooted.

Test: adb bugreport
Bug: 36852229
Bug: 26849505
Change-Id: I11589b81208e916d25d543c43c1283bdbffda949

Make init process create PDX sockets for services
am: 409c6eecdc

Change-Id: I338a691df317f0d57b5fd5c78b5442fc96f95b3f

Make init process create PDX sockets for services

To help us control the creation of PDX sockets and properly labeling
them for SELinux, let the init process create sockets for us based on
the settings specified in .rc files for service processes.

For (test) services that are meant to be started manually from command
line (e.g. test services), keep the old functionality as an option so
that UDS endpoint can be created in a way that it automatically creates
the socket in the service itself.

Bug: 35220925
Test: `m -j32` succeeds. Ran sailfish in VR mode and made sure all the
      services (surfaceflinger, performanced, sensord, bufferhub).
      `m -j32 checkbuild` succeeds as well.

Change-Id: Ief733b41b534cea19b1bea31de76b06051aa50ab

Use Configstore to test getWideColor function

Test: make tests in libs/gui/tests/
Test: adb sync
Test: adb shell /data/nativetest/libgui_test/libgui_test --gtest_filter="SurfaceTest.GetWideColorSupport"
Bug: 29940137

Change-Id: I618c300b1882d2ebd7474586d72f4d469e2d41bf

import configstore-utils as a shared library.
am: 585e32facc

Change-Id: I48218e405e225b99dfd51d2e26b0d5f26893e646

import configstore-utils as a shared library.
am: 3e85cbed6a

Change-Id: I9cf1591e31a14d6cfe65baffb3c752e0d17f8cae

import configstore-utils as a shared library.

android.hardware.configstore-utils is now built as a shared library.

Bug: 36275627
Test: Built and check the boot log messages
Change-Id: I44b0f6a69e267f670380e18a37a519bb2ef1ee84

import configstore-utils as a shared library.

android.hardware.configstore-utils is now built as a shared library.

Bug: 36275627
Test: Built and check the boot log messages
Change-Id: I83ccebcad8005f39ca8534477d7cd3f69a2041aa

Merge "Remove useless GraphicBuffer() constructor" into oc-dev
am: d934111402

Change-Id: I3462ff83e6aeced6591a99bf7391138a5438d3aa

Merge "Remove useless GraphicBuffer() constructor" into oc-dev

Merge "vulkan: update Vulkan headers to 1.0.46" am: 4cc7755c7c am: a867102e0c
am: 84558da59d  -s ours

Change-Id: I4fbaf5cbb078693968061996a0e73a65e195e9da

Merge "vulkan: update Vulkan headers to 1.0.46" am: 4cc7755c7c
am: a867102e0c

Change-Id: I0769a81a5c324253886ee07ccd51fc0d8af15daa

libgui: Make IConsumerListener a SafeInterface

Converts IConsumerListener to be a SafeInterface such that all
parceling/unparceling is done automatically.

Test: libgui_tests + manual testing
Change-Id: I1ed97f9802e320662cd29e181539ce839ffe0f3f

Merge "vulkan: update Vulkan headers to 1.0.46"
am: 4cc7755c7c

Change-Id: Id6997396c7a42517338bb36e24db1c50c600e248

libgui: Format IConsumerListener

Applies the framework default .clang-format and does a bit of tidying
up (reflowing comments to 100 characters, reordering includes, etc.).

Test: libgui_tests + manual testing
Change-Id: I324493551bca5a6c6a011cbe773021236e11e992

libbinder: Support Flattenable in SafeInterface

Adds support for sending and receiving Flattenable parameters as part
of a SafeInterface.

Test: New test in binderSafeInterfaceTest
Change-Id: I5c84d6f27ac1f8c7ad37210e836f390e02b92959

libgui: Add missing FenceTime header to GLConsumer

GLConsumer had a missing dependency on FenceTime which was hidden by
coincidental includes in other files. This adds the proper include.

Test: GLConsumer compiles
Change-Id: Ib96522787fa22746e4aeb2cf047bfd667869add6

libgui: Fix naming/enums in ISurfaceComposerClient

Moves the Tag enum out of the ISurfaceComposerClient header and into an
anonymous namespace in the .cpp file, changes it to UPPER_UNDERSCORE
naming, and makes its LAST entry actually point to the last value.

Test: m -j + manual testing
Change-Id: I73deaa446a87709548c2069c3a1f29605967ab97

Merge "vulkan: update Vulkan headers to 1.0.46"

Merge "fix race condition that can cause a use after free" into oc-dev
am: 2d45cf2a40

Change-Id: Ib574c8c9887fecd4ca542d92d61671d6c8a3625e

Merge "fix race condition that can cause a use after free" into oc-dev

Merge "Harden BufferHubQueueProducerTest" into oc-dev
am: 8e5deb6b96

Change-Id: If03751b511b12e043aee734b8b7b401a7dd378d4

Remove useless GraphicBuffer() constructor

Test: compiled & run
Bug: 36869708

Change-Id: I83d80a1cde28671135b7716a434c503aeefcec46

Merge "Harden BufferHubQueueProducerTest" into oc-dev

Merge "Offer to "fixup" GIDs used for app data." into oc-dev
am: e7353ae5fc

Change-Id: I7b0b11c6b47fdd09f3f36bd77714065f917b87cd

Merge "Offer to "fixup" GIDs used for app data." into oc-dev

fix race condition that can cause a use after free

Bug: 33004354
Test: manual
Change-Id: I9b38ee644b02268c9b995a330db758aa2e568399

Merge "Fix pose predictor jank." into oc-dev
am: 53ed7bfca1

Change-Id: I1b23c28c71e1c463d80a7a5810a8dbe843c7befd

Merge "Fix pose predictor jank." into oc-dev

Merge "vulkan: update Vulkan headers to 1.0.46" into oc-dev
am: 05601b602c

Change-Id: I0eea0f3e19bc346f8e539c9614eb863a087fff7e

Merge "vulkan: update Vulkan headers to 1.0.46" into oc-dev

vulkan: update Vulkan headers to 1.0.46

- Add VK_EXT_swapchain_colorspace
- Add VK_KHR_incremental_present

Test: dEQP-VK.api.smoke.*
      dEQP-VK.wsi.android.incremental_present.*
Bug: 36886707
Change-Id: I70ffd56a63da39301cd9706b1b91cf3643aab916

Fix pose predictor jank.

There were two issues:
- A math error in angular velocity computation.
  This has unit tests and they were failing.
  Need to make sure these units tests get executed.

- There is an inherent sensor latency that we need to account for.

Bug: 36693596, 36196200
Test: Run VrHome, see less hanky result.
Change-Id: Ifcf191f16de9ee64aa14bcbb74c9126db18ebaa8

Merge "libui: clean up GraphicBuffer interface" into oc-dev
am: c428826fdf

Change-Id: I5f594029e94b1a363e2f07dbdc5aeb32c2d58098

Merge "VR: Move VR HWC DVR API to libdvr" into oc-dev
am: 5ce4bb2cbe

Change-Id: I533b4634d8f6e25d06967092840cc30a9a3cf4c9

Merge "libui: clean up GraphicBuffer interface" into oc-dev

Merge "VR: Move VR HWC DVR API to libdvr" into oc-dev

Merge "pdx: Rework error reporting when transfering file and channel handles" into oc-dev
am: 93e3900ad3

Change-Id: I47723338f73266858e51e36090a2a2d4afbd6a2e

Merge "pdx: Rework error reporting when transfering file and channel handles" into oc-dev

vulkan: update Vulkan headers to 1.0.46

- Add VK_EXT_swapchain_colorspace
- Add VK_KHR_incremental_present

Test: build aosp_arm64-eng
Change-Id: I6cd81baa2870bf436abab6191a05ebfd60ccbc46
Merged-In: I70ffd56a63da39301cd9706b1b91cf3643aab916

Merge "Remove driver workaround" into oc-dev
am: ac81f70b43

Change-Id: I24da8479b8e2a84425fe7d3860cf3c0ed73a288f

Merge "Remove driver workaround" into oc-dev

VR: Move VR HWC DVR API to libdvr

Moved VR HWC API into a common API library.

Bug: 36050795
Test: Compiled
Change-Id: I1af32f4dedb7a6d4bb77a18d3c48cfbc8ea00a66
(cherry picked from commit 3d3fb3960f9c3956074496fd7fa1beb230ac1043)

Merge "Fix typo causing GL_EXT_debug_marker to disappear" into oc-dev
am: 8348f6d6d0

Change-Id: Ic4918d74375965dc18b8bd8e14c08090a4845d8c

Merge "Fix typo causing GL_EXT_debug_marker to disappear" into oc-dev

Merge changes from topic 'sensors_framework_hal' into oc-dev
am: 2d3349c9aa

Change-Id: I06a5e6a43d8841b464e794c7e9414de61b54e185

android.frameworks.sensorservice@1.0: ashmem direct channel check sizes.
am: 8a420ed627

Change-Id: I95c0352822944f1859896a5aaccebd88518e4437

android.frameworks.sensorservice@1.0: fix configureDirectChannel return positive integer.
am: 05fc8afc08

Change-Id: I79b7c4d2132693750ad7522eacd059e1d1dd0a15

Implement android.frameworks.sensorservice@1.0::IEventQueue.
am: 95c7a06367

Change-Id: I09ceb59fc4fd6e44b64306766ac712ccbf27b5b7

Renamed HIDL SensorManager::mManager -> mInternalManager.
am: 57d427735f

Change-Id: I8c2966e83f12715695d1fdd6ed5e1bd49854b047

Merge changes from topic 'sensors_framework_hal' into oc-dev

* changes:
  android.frameworks.sensorservice@1.0: make classes final
  android.frameworks.sensorservice@1.0: ashmem direct channel check sizes.
  android.frameworks.sensorservice@1.0: fix configureDirectChannel return positive integer.
  Implement android.frameworks.sensorservice@1.0::IEventQueue.
  Renamed HIDL SensorManager::mManager -> mInternalManager.

Merge "ANativeWindow_toSurface header" into oc-dev
am: a44aa44dd2

Change-Id: I127309a6bf09f5437f37713e967f876a8256a796

Merge "ANativeWindow_toSurface header" into oc-dev

Merge "vndservicemanager: load vendor service_contexts." into oc-dev
am: 42aca5a0d3

Change-Id: Icd22fb964cb05f17232b7637eaf86e3120d88897

Merge "vndservicemanager: load vendor service_contexts." into oc-dev

ANativeWindow_toSurface header

This enables the conversion from an ANativeWindow (created by NDK API
such as: AImageReader_create) to a Java Surface, so that developers can
hookup a Java Producer to a native buffer consumer.

Bug: 36862948
Test: android.media.cts.NativeImageReaderTest
Change-Id: I11962bc791680a62579e51c32f2d51911cac48f8

android.frameworks.sensorservice@1.0: ashmem direct channel check sizes.

Test: VtsHalSensorManagerV1_0TargetTest
Change-Id: I18b87128da2607f82ffddb260bf440786c378010

android.frameworks.sensorservice@1.0: make classes final

Make SensorManager and DirectReportChannel final.

Test: pass
Change-Id: Idc7f7fcacf1ad1ad6282a603538529ff99f88184

android.frameworks.sensorservice@1.0: fix configureDirectChannel return positive integer.

SensorManager::configureDirectChannel can return a positive integer
as the token to indicate success. android.frameworks.sensorservice@1.0
doesn't care about the token.

Test: VtsHalSensorManagerV1_0TargetTest
Change-Id: I6bcbbe5dc747a0d93d2a0fd37f0b7187cb681101

Implement android.frameworks.sensorservice@1.0::IEventQueue.

Test: pass
Bug: 35219747
Change-Id: I52ddd64db500c23db22768fc0603bce0cc14f8c6

Merge "Add VNDK API to access AHardwareBuffer from ANativeWindowBuffer" into oc-dev
am: 31ee0e6a6a

Change-Id: I3b3f6869f4977b37fbdcbe56808d0f7aebf34c9e

VR: Update VR Window Manager to use the VR Hardware Composer service
am: 3db8412608

Change-Id: I3a9378c6612a5d2e698c0197d5813f06f96b32bb

Merge "Add VNDK API to access AHardwareBuffer from ANativeWindowBuffer" into oc-dev

VR: Update VR Window Manager to use the VR Hardware Composer service

Decouples VR HWC from VR Window Manager.

Bug: 36051907
Test: Compiled and ran on device in VR mode.
Change-Id: Ifd76e742148765b3cf3e71b0cf538b920814eb7b

Fix typo causing GL_EXT_debug_marker to disappear

Android EGL wrapper is supposed to always insert GL_EXT_debug_marker
implementation. A small typo in recent refactoring broke this logic.

Bug: 36865892
Test: dEQP-GLES2.functional.debug_marker.supported

Merge "Surface: add getRemovedBuffer API" into oc-dev
am: 0860434b11

Change-Id: I1ccd3818df2dff83c69be863603df768391be230

Merge "Surface: add getRemovedBuffer API" into oc-dev

Merge "add surfaceflinger and servicemanager to animation class" into oc-dev
am: f60ab4119e

Change-Id: Ib85f6365b77709576023afadd9b5465cfc755e99

Merge "add surfaceflinger and servicemanager to animation class" into oc-dev

pdx: Rework error reporting when transfering file and channel handles

There is a lot of confusion about reporting errors when passing file
and channel handles over PDX transport between client and service.

Methods like Message::PushFileHandle return an integer which means
both a file handle reference value (if positive) and a possible error
code (if negative). But file handles could contain negative values too
(when they are empty). This is used frequently when passing buffer
fences around (when a fence is not being used, its fd is set to -1).

This results in a special case of when PushFileHandle is called with
a file handle with value of -1, the return value is actually "-errno"
which becomes dependent on a global state (errno is not set by
PushFileHandle itself in case file handle value is negative) and results
in unpredicted behavior (sometimes errno is 0, sometimes its >0).

Cleaned this all up by using Status<T> everywhere we used an int to
pass value payload along with possible error code.

Now the semantics of the calls are more clear.

Bug: 36866492
Test: `m -j32` for sailfish-eng succeeds
      Ran unit tests on device (pdx_tests, libpdx_uds_tests, bufferhub_tests,
      buffer_hub_queue-test, buffer_hub_queue_producer-test), all pass
      Ran CubeSea, NativeTreasureHunt and Ithaca on Sailfish with vrflinger
      enabled, was able to use controller and screen rendered correctly.

Change-Id: I0f40c3f356fcba8bc217d5219a0ddf9685e57fd7

Merge "Use gralloc1 usage in AHardwareBuffer_describe" into oc-dev
am: b2a1ac9ee4

Change-Id: If0617be00de4cbcb74ad2016475cc365ce6f4d5a

Remove driver workaround

Appears that the workaround for bug #34177594 is no longer
necessary, so removing.
Bug: 34177594
Test: Run Vulkan app

Change-Id: Id574760bdea775ac7541a9657b5a8124eb9064ef

Merge "Use gralloc1 usage in AHardwareBuffer_describe" into oc-dev

Offer to "fixup" GIDs used for app data.

We recently started tracking cached app data using a per-app GID for
the "cache" and "code_cache" directories and their contents.  For
upgraded devices, we ideally want to "fixup" the GIDs of any existing
data while the device is still showing the boot animation, instead of
blocking the user when they unlock the device.

Since all the information we need is available in metadata, we can
update GIDs before the user has unlocked data.  We're pretty paranoid
and we only pivot between the normal app GID and the cache GID; any
other GID values are ignored.

This "fixup" method can also be used in the future to ensure
consistency of the files on disk.  Also fix bug by always using
"fts_path" instead of "fts_accpath" which is based on racy chdir().

Test: /data/nativetest/installd_service_test/installd_service_test
Bug: 34201111, 35084485
Change-Id: Ia52694f3763cba09926082c08f0766477e03e39c

Add VNDK API to access AHardwareBuffer from ANativeWindowBuffer

this is to allow vendors to use AHardwareBuffer’s
lock/unlock on buffer dequeued from an
ANativeWindow.

Test: compiled
Bug: 36866217
Change-Id: If630105572ae0e57f6fda19c13e82150f415ed6b

Surface: add getRemovedBuffer API

This allows clients to optionally sign up to track
buffers going out of Surface.

Test: compile
Bug: 34461678
Change-Id: I8696b304c169099c872bd9ba783015c0f0dad73a

vndservicemanager: load vendor service_contexts.

Also, don't crash when /dev/vndbinder is not
accessible.

Bug: 36052864
Test: loads vendor service contexts
Test: doesn't crash without /dev/vndbinder, works with it
Change-Id: Ida9dd94791206e14246348a8b8e66be09ed8fca7

Harden BufferHubQueueProducerTest

This is a test that covers our implementation of bufferhubqueue-based
IGraphicBufferProducer, which eventually enables VrCore/VrFlinger to
access buffers produced from other Android compontents. Note that the
gtest itself is a (slightly) modified version of
IGraphicBufferProducer_test.cpp.

This also improves BufferHubQueue in the following way to support more
features of android::BufferQueue.
1/ Supports more buffer queue parameters that we query.
2/ BufferHubQueue (the PDX client) now supports default width, height,
and format configuration.
3/ Change default max_dequeue_buffer_count to 1 (which is the same
behavior current android::BufferQueue adopts).
4/ Fill in |QueueBufferOutput| during |enqueueBuffer|.

Bug: 34197998
Bug: 36266201
Test: build and run buffer_hub_queue_producer-test
Change-Id: I9d3e8bb66dbfb66e67ab7b0e5093e49a3f374e9c

Renamed HIDL SensorManager::mManager -> mInternalManager.

Use "Internal" to represent objects of the wrapped type.

Test: pass
Change-Id: I4f86bc2c7c272c9abb0c5b4c2837baccfdc2a748

Merge "We really want f_frsize." into oc-dev
am: 623912183d

Change-Id: I735d29c3f09a3c8ebc44d11539aba836254502d6

libui: clean up GraphicBuffer interface

Other than transferring the ownership, some callers want to control
whether to register or to clone the handle when creating a
GraphicBuffer from a handle.  Add a new constructor with
HandleWrapMethod for finer controls, to replace the one that has
only a boolean, keepOwnership.

The motivation is to enable users to construct GraphicBuffer without
fighting with GraphicBufferMapper::registerBuffer first.  More
generally, GraphicBufferMapper::{registerBuffer,unregisterBuffer} is
tricky to use correctly.  They should really be considered private
by most.

While at it, deprecate constructors that do not distinguish
producer/consumer usage.  It turns out GraphicBuffer is used by some
proprietary libraries.  I have to maintain the ABI and cannot remove
the deprecated constructors.

Bug: 36355756
Test: boots
Change-Id: Ieea618d8495f06a6233299d076fb821127cd6bf8

Merge "We really want f_frsize." into oc-dev

Merge "Define upper-bound disk quotas for all apps." into oc-dev
am: c86462b7c0

Change-Id: I1ea5db7d8f556fbc04d300fcc4fe8532ea349610

Merge "Define upper-bound disk quotas for all apps." into oc-dev

We really want f_frsize.

It's confusing, but f_bsize is not the value you're looking for; the
real block size is f_frsize.  Fix all those bugs.

Test: builds, boots
Bug: 36840579
Change-Id: I2846b8998c27f3e9a71dbf573a0a22158215a3bd

Define upper-bound disk quotas for all apps.

Abusive or broken apps can go crazy and try allocating all of the
disk space on the device.  To mitigate the impact on system health,
set hard limits to block any given app from using more than 90% of
disk blocks, or 50% of disk inodes.

Also define the hard limit for AID_MEDIA_RW to avoid filling up the
device via the SD card.

Kick QUOTAON when scanning devices, since ext4 doesn't toggle
DQUOT_LIMITS_ENABLED during initial mount.

Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.StorageHostTest
Test: cts-tradefed run commandAndExit cts-dev -m CtsOsTestCases -t android.os.cts.EnvironmentTest#testSaneInodes
Bug: 36450358
Change-Id: Iaa2bc6a2f0bc9047ee54c1d1a49bbda92142457a

Merge "surfaceflinger: Generate unique layer names" into oc-dev
am: c3bae03442

Change-Id: Ic4eaacc2d01ff92403eadcf8e666b2892063bef7

Merge "surfaceflinger: Generate unique layer names" into oc-dev

Merge "[libadapter] Fix nullpointer in acceptchanges" into oc-dev
am: 041e0ad654

Change-Id: I2271e20a406d6278156437b9064efedcdb5ce315

Merge "[libadapter] Fix nullpointer in acceptchanges" into oc-dev

DO NOT MERGE: ConsumerBase: call freeBufferLocked when a slot is overwritten
am: 723c489872

Change-Id: I801da460d77c2d180bc92058db0c19b61171eb4a

[libadapter] Fix nullpointer in acceptchanges

Check the layer still exists upon accepting changes.

Bug: 36205238
Test: AUPT
Change-Id: I8443265ecc7b8b0a12347439fd75b138d14c7e3a

DO NOT MERGE: ConsumerBase: call freeBufferLocked when a slot is overwritten

Cherrypicked from master.

Test: the CL + camera service side fix pass ReprocessCaptureTest
Bug: 34461678
Change-Id: Ia7863321bcb46cba53532fc1570c5facd9280aaa

Merge "vulkan: Dequeue the shared buffer again after presenting" into oc-dev
am: f2e5430b95

Change-Id: I5ae7ff9a5ef0c090149240ccf2613238e48d6b93