Fix permission for public secondary dex files
am: ebc8a79bbc

Change-Id: Iee64f192d00ab1a2e106493ba827e11f85748959

Merge "vndservicemanager: use separate SELinux class." into oc-dev
am: 8385d60c69

Change-Id: Iccac8bab23f6f2ad081d2661e68dc5b81cbb294f

Merge "vndservicemanager: use separate SELinux class." into oc-dev

Merge changes I2f531131,Ibaaca2a1,Ib0cba4c7,I5ea748b8,I046e3f07, ... into oc-dev
am: 42b5469646

Change-Id: Ifc5e39570cbe69eb26820ba8abc482fbe5744306

Merge changes I2f531131,Ibaaca2a1,Ib0cba4c7,I5ea748b8,I046e3f07, ... into oc-dev

* changes:
  libgui: Make IDisplayEventConn... a SafeInterface
  libgui: Remove RefBase from BitTube
  libgui: Make BitTube Parcelable and use unique_fd
  libgui: Format BitTube and move into gui namespace
  libgui: Move BitTube out of system include dir
  libgui: Normalize IDisplayEventConnection methods
  libgui: Format IDisplayEventConnection

vndservicemanager: use separate SELinux class.

For doing access control.

Bug: 36052864
Test: device boots
Change-Id: I263679ed4532ec52b1e64e5e60bdc064b7484073

libgui: Make IDisplayEventConn... a SafeInterface

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

Test: libgui_test + SurfaceFlinger_test + manual testing
Change-Id: I2f5311315dc4fadbf3599f2b28f150097f53de57

libgui: Remove RefBase from BitTube

Removes RefBase from BitTube, since because it is not a Binder object,
it doesn't need to be reference-counted in this way.

In the process, we rename IDisplayEventConnection::getDataChannel to
IDEC::stealReceiveChannel to make it clearer that this is a non-const
operation on the remote end that removes its access to the receive
channel.

This also adds a couple of methods for moving the receive file
descriptor out of one BitTube and into another, since this is the
essence of the IDisplayEventConnection::stealReceiveChannel method,
and now with C++11 move semantics, we can do this without needing to
return an sp<> from EventThread's implementation of stealReceiveChannel.

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

libgui: Make BitTube Parcelable and use unique_fd

This change completes the Parcelable interface for BitTube (it was
semi-Parcelable before as it implemented writeToParcel, but this adds
the complementary readFromParcel).

It also changes the send and receive file descriptors from ints to
android::base::unique_fds, which simplifies some of their lifecycle
management.

Finally, it changes the default constructor to leave the class
uninitialized, adding a BitTube(BitTube::DefaultSize) constructor to
replace that functionality (and paving the way for the ability to
default-construct a BitTube prior to readFromParcel'ing into it).

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

libgui: Format BitTube and move into gui namespace

Reformats BitTube using the framework default .clang-format and moves
it into the android::gui namespace.

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

libgui: Move BitTube out of system include dir

Moves the BitTube header out of the system include directory and into
libgui's include directory.

Test: m -j
Change-Id: I046e3f07ec4a58736c8de7ea7ba0f43a8e3e328b

libgui: Normalize IDisplayEventConnection methods

This change modifies the IDisplayEventConnection API such that every
synchronous method returns a status_t (to be able to return transport
errors). This required changing getDataChannel to return its channel by
output parameter rather than return type.

Currently no more error messages are checked than before, but this will
both enable calling code to check error messages if it desires and,
more importantly, allow the Bp/Bn code to be semi-automatically
generated using SafeInterface.

Test: libgui_tests + manual testing
Change-Id: I8d5bc5ef0475cee07b638a97079b234f0384c022

libgui: Format IDisplayEventConnection

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: I8b3515339a93dbe42ce4421cb561b78ed2fe9104

Merge "Use explicit .c_str() for hidl_string" into oc-dev
am: 0005f132ce

Change-Id: I16d601dd262aa1ac2c60f38d7d3658085ee0c7b1

Merge "dvr_api GetExternalSurface returns ANativeWindow" into oc-dev
am: 0c91fbbcfc

Change-Id: Ideed03fe6c311a843a71c85d27bd2b2a56961e9f

Merge "Use explicit .c_str() for hidl_string" into oc-dev

Merge "dvr_api GetExternalSurface returns ANativeWindow" into oc-dev

ifdef HWC2 functionality
am: 4f20f9cf32

Change-Id: I9a93704750627663668c768bfb3301e487b9b47c

ifdef HWC2 functionality

Fix build issue for devices that are not using HWC2.
Test: lunch hikey-userdebug; make

Change-Id: I47e9aa09b8db3f0dca6d733e2c8f3b59c45547b2

Fix permission for public secondary dex files

- make the oat directory accessible by others
- if the dex files is readable by others and we are not doing a speed-
profile compilation make the oat file readable by others.

Bug: 36896515
Test: adb shell cmd package compile -r bg-dexopt  --secondary-dex com.google.android.gms
      check that the oat directory and files have the right permissions

Change-Id: I1829217004353fb8c86e5a6b7eb570555e36a495

Merge "Check if a visible layer exists instead of an invisible layer exists" into oc-dev
am: 3fdeae8b2d

Change-Id: I0a49900b7451e21c5cf7c82ee84f534563afa34b

Merge "Check if a visible layer exists instead of an invisible layer exists" into oc-dev

Merge "Make sure all HWC2 functionality is ifdef'd" into oc-dev
am: f0f8a81d06

Change-Id: Ibf4cf46eb5326419ec13e9924fb16afcf22cf6ec

Merge "Make sure all HWC2 functionality is ifdef'd" into oc-dev

Make sure all HWC2 functionality is ifdef'd

Fix build issue for devices that are not using HWC2.
Test: lunch bat_land, make

Change-Id: If94222cd00f2c8b0b1f7a464ad0378224fc059e7

Merge "VR: Remove HIDL VR Composer from service/vr/vr_window_manager" into oc-dev
am: b324457171

Change-Id: Icba466ed6f03ef909aba8fade30645611e5f70c8

Merge "VR: Remove HIDL VR Composer from service/vr/vr_window_manager" into oc-dev

Merge "lshal --init-vintf stripped <impl> tag." am: b7003ce5d4 am: e0b65151d8
am: 44456bb92d

Change-Id: Idaa848f62fe52e04a364fb1d9fb22b36ef4bce2f

Merge "lshal --init-vintf stripped <impl> tag." am: b7003ce5d4
am: e0b65151d8

Change-Id: Ie7efd9f294713804a5210567d1a3a2e0ca14ce61

Merge "lshal --init-vintf stripped <impl> tag."
am: b7003ce5d4

Change-Id: I7f084b9041c6b6b4da9ed77798d570d4d7a4c176

Merge "lshal --init-vintf stripped <impl> tag." into oc-dev
am: 7ad6b13213

Change-Id: Ib463dd6572229c3e09a17482f318b36825ed6547

Merge "lshal --init-vintf stripped <impl> tag."

Merge "lshal --init-vintf stripped <impl> tag." into oc-dev

Merge "This test is used to check if the kernel scheduler" am: 8039adad34 am: 0145ac08f0
am: 16126749c6  -s ours

Change-Id: I7f3e0795ca620616d3214249133e4257c7550af7

Merge "This test is used to check if the kernel scheduler" am: 8039adad34
am: 0145ac08f0

Change-Id: Ib1e12e0ca70279127d89a336dbbdfcf1033694f6

Merge "This test is used to check if the kernel scheduler"
am: 8039adad34

Change-Id: I5d57ee904f1431384fabb89b893700e3a7731da0

Merge "This test is used to check if the kernel scheduler"

Fix build error with configstore
am: c221d640bc

Change-Id: I43e51d1177f78014675761a4dff1ee1eaa6d77cf

Fix build error with configstore

CL I2c0895f8afbbb2947b62164acaf62a491c451dc0 made it necessary
to include android.hardware.configstore-utils as a shared library.
Was causing build error that looked like:
ninja: error:
'out/target/product/generic_x86_64/obj/HEADER_LIBRARIES/android.hardware.configstore-utils_intermediates/export_includes',
needed by
'out/target/product/generic_x86_64/obj/NATIVE_TESTS/EGL_test_intermediates/import_includes',
missing and no known rule to make it
23:25:54 ninja failed with: exit status 1
make: *** [run_soong_ui] Error 1

Happened in tests directory which doesn't get built by default.

Test: build opengl/tests
Change-Id: I2c731c4e1b23888edd67fd9311b797fba61c011f

Merge "Set sensor operation parameter implementation" into oc-dev
am: 691d271b00

Change-Id: Ifb699a3a83ff7b3821932dfe86c5160d6adce654

Merge "Set sensor operation parameter implementation" into oc-dev

Fix soong dependency issue
am: 381019d9ce

Change-Id: Ibaed4bd3a446ecdbffeb16c072e5326422aeab13

lshal --init-vintf stripped <impl> tag.

<impl> tag is deprecated from the manifest.

Test: lshal --init-vintf
Change-Id: Ifa9336e3dfe7f1091ee61aa6a10df9629ad98b49

Fix soong dependency issue

Test: make

Change-Id: I3ea00a32f08395c81e44625603d878a2f1b6975b

Merge changes I9da985a7,I2bd642aa,I9358b3c9,I9d3c60b1 into oc-dev
am: 62dc1344e0

Change-Id: I473b1f54ff0322ecc2a99a732bd17a302a8967a2

Merge "vrwm: Also recenter the controller when should_recenter_ is set" into oc-dev
am: 674436272e

Change-Id: I7c02a3c2f9308445a2bf365fb386cc2ee0fba53b

Merge changes I9da985a7,I2bd642aa,I9358b3c9,I9d3c60b1 into oc-dev

* changes:
  Add test for FP16 surface support
  Dump info about wide-color
  Add wide-color support to SurfaceFlinger
  Add debug string functions for graphics.h enums

Check if a visible layer exists instead of an invisible layer exists

Multiple layers may exist at index 0 and what we really want to know
is if any of them are active, not if any aren't.

Bug: 36867186, 36036583
Test: Launch ithaca then youtube
Change-Id: I5881a838c7df8670fa42eece558717140d4cb021

Merge "vrwm: Also recenter the controller when should_recenter_ is set" into oc-dev

Merge "Strip VR daemon binaries." into oc-dev
am: 5db4c0c58e

Change-Id: I699fff7df1860d46a4f80630ccc914431ca8f45e

Merge "Strip VR daemon binaries." into oc-dev

Merge "dumpstate: don't read /proc/interrupts" into oc-dev
am: f806b87fe1

Change-Id: I1bd0811c428c2492e7f1a1a98e598ac675d6d00d

Merge "dumpstate: don't read /proc/interrupts" into oc-dev

Strip VR daemon binaries.

Stripping was suppressed to assist debugging, but this adds 24+ MB to
release builds.

Bug: none
Test: manual
Change-Id: Ic2a866930cebb67af052b6665af2631b131601ae

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

Add test for FP16 surface support

Test: adb shell /data/nativetest/EGL_test/EGL_test
      adb shell /system/bin/test-opengl-gl_basic # look for RED_SIZE: of 16.
Bug: 29940137

Change-Id: I9da985a7574d3582b06cf55bc02515db0869505a

Dump info about wide-color

Test: adb shell dumpsys SurfaceFlinger
      Look for section titled "Wide-Color information:"
Bug: 29940137

Change-Id: I2bd642aa026b49bbb9effd5cb1687c7a36dae684

Add wide-color support to SurfaceFlinger

Test: manual testing
Bug: 29940137

Change-Id: I9358b3c982e0205e598fd20cbf8d2e956591b3ac

Add debug string functions for graphics.h enums

Test: see dataspace and colormode details when running with debug
Bug: 29940137

Change-Id: I9d3c60b1f9c71c44aba475450c7f867dcd68627e

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

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

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

dvr_api GetExternalSurface returns ANativeWindow

Now that we have a public NDK API to convert an ANativeWindow to a Java
Surface, we no longer need to mess with Jni and libandroid_runtime in
libdvr.

See more information at:
http://ag/#/q/topic:ANativeWindow_toSurface+(status:open+OR+status:merged)

Also, as a side effect, we are now free to convert this library to use
Android.bp (though we probably want to hold off for a while).

Bug: 36266201
Test: Built and ran dvr_buffer_queue-test
Change-Id: Ia0f6a8735c7d508195c30e3c7d040791e13a6ce3

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"

lshal --init-vintf stripped <impl> tag.

<impl> tag is deprecated from the manifest.

Test: lshal --init-vintf
Change-Id: Ifa9336e3dfe7f1091ee61aa6a10df9629ad98b49

dumpstate: don't read /proc/interrupts

/proc/interrupts may be dumped by dumpstate HAL if required.
The framework dependency on this file is unnecessary, since the
interrupt names are hardware / kernel dependent anyway.

Bug: 36486169
Test: 'adb shell bugreport' on sailfish
Change-Id: I336318b24ce4bcc5b710645359b7c4ccda3f6989
Signed-off-by: Sandeep Patil <sspatil@google.com>

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

vrwm: Also recenter the controller when should_recenter_ is set

Bug: 36201389
Test: Manual, recenter controller with vrwm up
Change-Id: I776ac51578d692832baac5aad50993b5ed1b8eea

fix race condition that can cause a use after free

Bug: 33004354
Test: manual
Change-Id: I9b38ee644b02268c9b995a330db758aa2e568399

Set sensor operation parameter implementation

Set operation parameter to sensor HAL. Possible paramters include
local geomagnetic field, local gravity acceleration, device dock
state, whether device should run in high performance mode, and
if a magnetic field calibration guide is in progress.

Bug: 30958130
Test: Tested with marlin. Modified hal implementation can get local
      geomagnetic field.

Change-Id: Idaedd9e616d01383ba244cc64cf3e319ce02191f

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