Merge "libbinder: vendor_available" into oc-dev
am: 80a50bdc3b

Change-Id: If99137a1ea7e4214b1ba705a38b894692e2778fc

Merge "Move include/binder to libs/binder/include" into oc-dev
am: 9451d0b8ce

Change-Id: I7384f2b19143e30ca524549fcd81b95a5aaa7beb

Merge "libbinder: vendor_available" into oc-dev

Merge "Move include/binder to libs/binder/include" into oc-dev

Merge "Update optics" into oc-dev
am: 0a9e585fe2

Change-Id: I95636e962b9994bac5486f1b4f4706405e08ed79

Merge "Update optics" into oc-dev

Merge "binder: allow repeated invocations of initWithDriver" into oc-dev
am: 11c1acc33a

Change-Id: I44a4f3cb656c12eba9f506a5cb1299ca3faa985c

Merge "binder: allow repeated invocations of initWithDriver" into oc-dev

binder: allow repeated invocations of initWithDriver

As long as the driver name is the same, calls to
ProcessState::initWithDriver() will succeed, returning the
already-created object.

Test: marlin

Change-Id: I87d5ca2a105becd8d4fa6fa85c00b879360229b3
Signed-off-by: Iliyan Malchev <malchev@google.com>

Merge "Refine the binder latency test" into oc-dev
am: f18f8fe8cb

Change-Id: I0bc0cac5fd11bb197addc382699b57d0b17d22b6

Merge "Refine the binder latency test" into oc-dev

get rid of IGraphicBufferAlloc
am: 0556d79eac

Change-Id: I9b76eac770fc7585ea73a8497ae04e22ad5b9eec

get rid of IGraphicBufferAlloc

Buffers can now be allocated directly through
the graphic allocator HAL.

Test: marlin: run full camera cts
Test: angler: take screenshot, take photo w/ and w/o HDR, video, panorama, refocus, slo-mo
Bug: 36462585
Bug: 36333314
Change-Id: Ie5222c53c3b9462e0ac7a41568718aad131eb328

libbinder: vendor_available

By setting vendor_available, the following may become true:

* a prebuilt library from this release may be used at runtime by
  in a later releasse (by vendor code compiled against this release).
  so this library shouldn't depend on runtime state that may change
  in the future.
* this library may be loaded twice into a single process (potentially
  an old version and a newer version). The symbols will be isolated
  using linker namespaces, but this may break assumptions about 1
  library in 1 process (your singletons will run twice).

Background:

This means that these modules may be built and installed twice --
once for the system partition and once for the vendor partition. The
system version will build just like today, and will be used by the
framework components on /system. The vendor version will build
against a reduced set of exports and libraries -- similar to, but
separate from, the NDK. This means that all your dependencies must
also mark vendor_available.

At runtime, /system binaries will load libraries from /system/lib*,
while /vendor binaries will load libraries from /vendor/lib*. There
are some exceptions in both directions -- bionic(libc,etc) and liblog
are always loaded from /system. And SP-HALs (OpenGL, etc) may load
/vendor code into /system processes, but the dependencies of those
libraries will load from /vendor until it reaches a library that's
always on /system. In the SP-HAL case, if both framework and vendor
libraries depend on a library of the same name, both versions will be
loaded, but they will be isolated from each other.

It's possible to compile differently -- reducing your source files,
exporting different include directories, etc. For details see:

https://android-review.googlesource.com/368372

None of this is enabled unless the device opts into the system/vendor
split with BOARD_VNDK_VERSION := current.

Bug: 36426473
Bug: 36079834
Test: m -j libbinder
Test: attempt to compile with BOARD_VNDK_VERSION := current
Change-Id: Iec94a66adbc852f7f23352307cf36f49a79ee3a6

Update optics

* Specify optics via system properties

Bug: 36039976
Test: Ran on device

Change-Id: I7a38e5a9eb3b8a861f1997d5011ec109f5e79cca

Refine the binder latency test

* limit the output line length be be under 80.
* fix the target service when pair > 2

Bug:36705188
Test: sailfish with prebuilt kernel on oc-dev
Change-Id: I5ce2754418a669a61f9913a30dbeb7176796e230

Move include/binder to libs/binder/include

Also moved include/private/binder to libs/binder/include/private/binder.
This allows libbinder to be compiled without depending on global
headers.

Bug: 33241851
Test: Build, flash, and run sailfish.
Test: Try building with BOARD_VNDK_VERSION := current
Change-Id: I0a82b733353add589de26eb2db7af5a246e5c97f

Merge "Update to allow multiple HALs in a manifest." into oc-dev
am: 1011062e1a

Change-Id: I3ed6dfa6daa5750a1c5262431fb876fa5203a824

Merge "Import broadcast_ring.h from GVR (reland)" into oc-dev
am: afe7df1a3f

Change-Id: I6b136b5d754a7fda78140137db1c282495179d58

Merge "Update to allow multiple HALs in a manifest." into oc-dev

Merge "Import broadcast_ring.h from GVR (reland)" into oc-dev

Merge "libpdx_uds: Handle EACCES error when connecting to PDX service" into oc-dev
am: 4df4202551

Change-Id: I02d1f307717c1628f6a6d54130d0bd5a2551bf84

Merge "libpdx_uds: Handle EACCES error when connecting to PDX service" into oc-dev

Merge "Use a separate heap size to receive buffer" into oc-dev
am: ca0b8ccfc5

Change-Id: I7d4b6c6f6103d19d0ae12b9850eca33d4d956411

Merge "Use a separate heap size to receive buffer" into oc-dev

Merge "Revert "Avoid a potential race condition on mDisplays"" into oc-dev
am: c90e469393

Change-Id: I12fa7cabdc29dcc707756c8e849682b93ceee199

Merge "Revert "Avoid a potential race condition on mDisplays"" into oc-dev

Merge changes Ica5f5f5c,Ifb7d2cec,Ia402b4bc,I9c3b6b7e,I36ba19c7, ... into oc-dev
am: 1b47dfd749

Change-Id: I9617b0192d649ac63bc7f1d587590000bae3cf86

Merge changes Ica5f5f5c,Ifb7d2cec,Ia402b4bc,I9c3b6b7e,I36ba19c7, ... into oc-dev

* changes:
  libgui: Make IGraphicBufferConsumer a SafeInterface
  libbinder: Add vector<Parcelable> to SafeInterface
  libbinder: Add sp<NativeHandle> to SafeInterface
  libbinder: Support enums in SafeInterface
  libbinder: Support sp<Flattenable> in SafeInterface
  libbinder: Add int64_t/uint64_t to SafeInterface
  libgui: Normalize IGraphicBufferConsumer methods
  libgui: Format IGraphicBufferConsumer

Revert "Avoid a potential race condition on mDisplays"

This reverts commit 8d6c16dc3dc8b88a0046f53668a4e3be074507ff.

Bug: b/37282502
Change-Id: Ibf64607f9e14ede201510e2c1b502c49a31e9f2a

Merge "Add autofill feature API" into oc-dev
am: 3a717993fa

Change-Id: I0bcd259aa5a3f688d29689585a3b5f60d5e0e7f8

Merge "Add autofill feature API" into oc-dev

Add autofill feature API

bug:35956220

Test: manual (requires a custom build)

Change-Id: I5d42919d9419ac34825709cd80affa705cce32ed

Merge "Fix installd test" into oc-dev
am: 22651020b5

Change-Id: I6ce8ae2c3550786fe876ce59beb5f29214b05bba

Merge "libvrhwc: Fix dependency order because Soong." into oc-dev
am: aaa1e1621e

Change-Id: If2d518091fe2e6b25bd7ae3f673890f942370b40

Import broadcast_ring.h from GVR (reland)

This will be used as a transport by the pose plumbing.

Remove host_supported=true from the blueprint. Although this library is
portable C++11, some of the tests aren't, and I don't know how to test
the various SDK targets in order to properly #ifdef them.

Bug: 37102512
Test: m -j32 broadcast_ring_tests; broadcast_ring_tests

Change-Id: Ib91387f2feba9030d2e892dc5145e353b88bad45

Use a separate heap size to receive buffer

Bug: 37216168
Bug: 37245884
Test: cts-tradefed run cts -m CtsNativeHardwareTestCases -t AHardwareBufferTest#AHardwareBuffer_SendAndRecv_Succeeds
Change-Id: I3bdf9c6a24ee1b07a14499f0987b0ac028bbae09

Merge "Add android.hardware.nfc.any to NFC features." into oc-dev
am: e7def4e9e0

Change-Id: I4c40be1c19abbac0c0a3808126821ee35b0bf307

libpdx_uds: Handle EACCES error when connecting to PDX service

There is a race condition in the way init process creates socket
files before it forks into a child (see system/core/init/util.cpp,
function create_socket()). It first creates a socket, then calls
bind() which creates a file entry, then calls chown/chmod to change
ownership and access permissions of that file object. If a client
process connects to the socket just after bind() was called but before
chmod(), the socket file will have incorrect permissions and the
connect call will be aborted with EACCESS (Permission denied) error.

We should retry connection in case we get EACCESS. A proper fix would
be is to find a way for init to create the file entry with the proper
permissions to start with, and eliminate the race condition altogether.

Bug: 37171113
Test: `lunch sailfish-eng && m -j32` succeeds
      Device boots correctly.
      Error recovery is handled correctly even after adding sleep(5)
      to init process just before calling chmod on the socket.

Change-Id: If97c316daa0c0be5ff7b5aa302a69aa60e9fb237

Merge "Fix installd test" into oc-dev

Merge "libvrhwc: Fix dependency order because Soong." into oc-dev

Merge "Add android.hardware.nfc.any to NFC features." into oc-dev

Merge "Cleanup VR HWC type names in libdvr" into oc-dev
am: dc87165eb5

Change-Id: I1829d325df18c783b8ac7c7871c9bb68f4cfbed4

Merge "Cleanup VR HWC type names in libdvr" into oc-dev

Merge "Only enumerate display_timing when present timestamp supported" into oc-dev
am: 430f8eae67

Change-Id: I97f8e31b73d5428b7b70409702286bb292ff9657

Merge "Only enumerate display_timing when present timestamp supported" into oc-dev

Merge "Revert "Import broadcast_ring.h from GVR"" into oc-dev
am: 5bf90e1d47

Change-Id: I57c43fbeb716cb788bc098a2ff1a58aba426c773

Merge "Revert "Import broadcast_ring.h from GVR"" into oc-dev

Revert "Import broadcast_ring.h from GVR"

This reverts commit e5c34974f2dd8a8e9ec61e68799d55f6df343224.

Reason for revert: Failing SDK target, see

https://android-build.googleplex.com/builds/quarterdeck?branch=git_master&target=sdk_phone_armv7-sdk_mac&lkgb=3906845&fkbb=3908362

Seems fix should be something like this:
https://android-review.googlesource.com/#/c/316195/1/src/code-buffer-vixl.cc

Anyway, I am not native code expert, so going to revert this :)

Test: treehugger only

Change-Id: Iba00bec4c2c8667dffd96f90a57948c827c4a6a3

Merge "Remove UID requirement for set sched_fifo for sensor thread" into oc-dev
am: 62dd04b4f2

Change-Id: Iff60e27ad92631e130795b3fe1fb820af9e41046

libvrhwc: Fix dependency order because Soong.

Static library dependencies must (usually) be in reverse order of
dependency. Use the correct dependency order to fix the build.

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

Merge "Remove UID requirement for set sched_fifo for sensor thread" into oc-dev

Update to allow multiple HALs in a manifest.

Test: libvintf_test
Test: lshal --init_vintf

Bug: 37294247
Change-Id: I202787d7856db6e06e595bd52c644f6a0eefcb66

Merge "SurfaceFlinger: Add parent-less relative layering." into oc-dev
am: 7dcda8011d

Change-Id: Ib6cd65edeeb2d1cbd577d8dade2d02ba0b25b4bd

Merge "SurfaceFlinger: Add parent-less relative layering." into oc-dev

Merge "MediaCas: combine openSession methods" into oc-dev
am: a30e1cbfcc

Change-Id: I90d7496fc25ff3b2bd98883253cec44d9e5779bb

Merge "MediaCas: combine openSession methods" into oc-dev

Merge "libhwc2on1adapter: mark as vendor_available" into oc-dev
am: 4a5c8a3c97

Change-Id: I0d1274767fe70358302f7a6a76aa6306eb04743c

Merge "libhwc2on1adapter: mark as vendor_available" into oc-dev

Merge "Import broadcast_ring.h from GVR" into oc-dev
am: bb724dd447

Change-Id: I59aa0537268e76b9b15ee22108619b2e94211dd8

Merge "Import broadcast_ring.h from GVR" into oc-dev

Remove UID requirement for set sched_fifo for sensor thread

Allow all requests to set priority.

Test: check error message in log, check thread prio using systrace
Bug: 35219747
Change-Id: I2b43c172ec0c982cb829764f484a057d096d82bc

Fix installd test

CreatePrimaryCurrentProfile was using the wrong userid for the expected
value.

Test: adb shell /data/nativetest/installd_utils_test/installd_utils_test
Bug: 36101738
Change-Id: I80b8212d09a0e4b600c00f85ac604a0e5fa1cc11

Merge "display_manager_client unused parameter warning fixes" into oc-dev
am: 4267ac38bd

Change-Id: Iee5a1e21ddd2bdf2f083c53134de8e55d9afcfa5

Merge "display_manager_client unused parameter warning fixes" into oc-dev

Merge "surfaceflinger: start grahpics allocator when requested" into oc-dev
am: 7da46d06f0

Change-Id: If687b478f82fd24d7fbfab430a51c217ba8f05a4

Merge "surfaceflinger: start grahpics allocator when requested" into oc-dev

Merge "Add PERMISSION_DENIED to convertResult." into oc-dev
am: d111a91cd1

Change-Id: I1860fc445ffe3bffd41162043dc99f5eafb5eda6

Merge "Add PERMISSION_DENIED to convertResult." into oc-dev

libgui: Make IGraphicBufferConsumer a SafeInterface

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

Test: libgui_tests + manual testing
Change-Id: Ica5f5f5c3e6b5735f0f1327a50cefe8f3f6f0feb

libbinder: Add vector<Parcelable> to SafeInterface

Adds support fo sending and receiving std::vector<Parcelable>
parameters (such as the vector of OccupancyTracker::Segments in
IGraphicBufferConsumer::getOccupancyHistory) as part of a SafeInterface.

Test: New test in binderSafeInterfaceTest
Change-Id: Ifb7d2cece79ee9fd52791f61d2488e6a490805f3

libbinder: Add sp<NativeHandle> to SafeInterface

Adds support for sending and receiving sp<NativeHandle> parameters as
part of a SafeInterface.

Test: New test in binderSafeInterfaceTest
Change-Id: Ia402b4bceedff358864669d201b4c1e22be6d0fa

Merge "Mark as vendor_available" into oc-dev
am: b120f2ead6

Change-Id: I21a67fee1ddb79fa2f1d5845d40a2a2cb04e73b7

Merge "Mark as vendor_available" into oc-dev

display_manager_client unused parameter warning fixes

Bug: 37206802
Test: builds locally with no warnings
Change-Id: I5755ee30d6ce4057297c367222d3cdd44904e23e
Edit

Merge "Correct configure direct report function document" into oc-dev
am: e502a9b2e2

Change-Id: Id1c535e5271da42656b6cce5ba6e4b64f906b07f

Only enumerate display_timing when present timestamp supported

Test: CTS and cube demo.
Bug: 36887025

Vulkan should only enumerate the VK_GOOGLE_display_timing extension if
the device supports reliably returning the present timestamp.  Vulkan
will determine this by reading a property that SurfaceFlinger will set
(based on the HWC2::Capability::PresentFenceIsNotReliable capability).

Change-Id: I6025be5f9120a8f02f228b8437b64be4ff8cfb17

Merge "Correct configure direct report function document" into oc-dev

Merge changes from topic 'presentFenceNotReliable' into oc-dev
am: 21d072ac4a

Change-Id: If70925313d46370103d1aa09a38348fb5731bee0

Merge "Link vndservicemanager against libselinux statically." into oc-dev
am: 94f2ea67c5

Change-Id: I824ad4f11704c5bf95f12e7b0ffeaa9ddc23f78d

Merge changes from topic 'presentFenceNotReliable' into oc-dev

* changes:
  egl: Use reserved values for eglGetFrameTimestamps
  egl: Differentiate pending vs invalid timestamps.
  egl: Avoid use of retire as present

Merge "Link vndservicemanager against libselinux statically." into oc-dev

Add android.hardware.nfc.any to NFC features.

So we can start the NFC process whenever one of
these is present.

Bug: 36873979
Bug: 37269025
Test: NFC is still started on devices with this feature
Change-Id: I879c084c1ed56e33a4249172c8fa3c04b1c5f236

surfaceflinger: start grahpics allocator when requested

When ISurfaceFlingerConfigs::startGraphicsAllocatorService returns
true, start graphics allocator service.

Bug: 36462585
Test: YouTube, Play Movies
Change-Id: Iac177e96ec89465398ec3eef46208bdb817934e3

Merge "atrace: don't use sendfile." am: c1c9354ad8 am: 86504eb9e6
am: 3ec7db90cc

Change-Id: I4f57aaf2b483a98631bb746b77cecfacf52298b1

Merge "atrace: don't use sendfile." am: c1c9354ad8
am: 86504eb9e6

Change-Id: I1cdcf014bc7d5b40d1ecd20649d6b2c63777aa7a

Merge "atrace: don't use sendfile."
am: c1c9354ad8

Change-Id: Ic42793ae6df11ac557e21cbd7a6d6bf758e2c3d8

SurfaceFlinger: Add parent-less relative layering.

This is a temporary functionality for the intermediate state where not
all child-surface like things in WM land are ported to use Child
Surfaces. In particular, we have ported SurfaceView to use child
surfaces and relative Z ordering. However the TV frameworks provide a
View framework component which overlays views over the SurfaceView but
below the main application window. Since we have not ported View
framework surfaces to use child layers, there is nothing the WM or
View Framework can do about this situation. Luckily the WM API's to
have requested this are @hide but we have the one media framework
component using it. In order to solve this issue we provide a method
to set Z ordering relative to another window without inheriting
its coordinate space as a child window would. This way the WM can
recognize these TYPE_APPLICATION_MEDIA_OVERLAY windows and Z-order
them at -1 with respect to the parents (and the SurfaceView can be at
-2).

Test: Included in transaction tests. Also manual test of bug repro steps with accomp frameworks/base CL
Bug: 36693738
Change-Id: I921852d3d34f67f79ec745b9703f9e679867e7a1

Merge "Mark as vendor_available" am: d03d569f06 am: b365ddfbf5
am: 1bc920ae16

Change-Id: I83a4e64b7237316b4aa8c4c68336fe246538e6e2

Merge "atrace: don't use sendfile."

Merge "Mark as vendor_available" am: d03d569f06
am: b365ddfbf5

Change-Id: I52bf41fd520c08904c24ec63353f57e30b2eafe3

Merge "Mark as vendor_available"
am: d03d569f06

Change-Id: I685b097e616aa605eaacc917d1de75e45555fc52

Merge "Mark as vendor_available"

Link vndservicemanager against libselinux statically.

Bug: 37189668
Test: vndservicemanager works
Change-Id: I8187fb86371d5a2073287beb6b17e263eedfb17b

Cleanup VR HWC type names in libdvr

Add DvrHwc prefix to enums and types defined in the API to avoid naming
conflicts.

Bug: 36050795
Test: Compiled
Change-Id: Ic5071ccd6f2b16863f0001009a4255f8279ce1e8

Merge "Add private helpers needed for job queue." into oc-dev
am: 68a2340243

Change-Id: I2cc3f89d6b8034d8cbebde8fdd42a9b94a33f0da

Merge "Add private helpers needed for job queue." into oc-dev

Merge "Otapreopt: Check dexopt return status before retry" into oc-dev
am: d3ef230cac

Change-Id: I581bf4250a6ed5f86b2f1c6c417cb6c337fda7c6

Merge "Otapreopt: Check dexopt return status before retry" into oc-dev

Merge changes Ia1fa4a32,I2a4c3d9f into oc-dev
am: 6d957ec373

Change-Id: I70a0a4b117013f5dbca339729be4799b49c135a1