Merge "libgui should re-export libbinder include paths"

libgui should re-export libbinder include paths

libgui headers include headres from libbinder.  Therefor, consumers
need the complete include paths of both libries.

Bug: 27804373
Change-Id: I4c355f32fb58298059abf7c546b48e8b24b4dc3c
Test: Compiles when libbinder include path changes.

Merge "Revert "libbinder: Replace ScopedFd with base::unique_fd""

Revert "libbinder: Replace ScopedFd with base::unique_fd"

This reverts commit b12c5a67f9cf5a55f67c08bef898c48cefa9d03d.

Change-Id: I13ab8151376bc3bda7f96b92db7934720dc62215

Merge "libbinder: Replace ScopedFd with base::unique_fd"

Merge "Run 'nativeperms', remove Brillo #ifdef."

Merge "Add missing include to choreographer header."

Add missing include to choreographer header.

Change-Id: Id497f3a505211309a08e100b369300e81f84b13b

Merge "Add NDK Choreographer API headers"

Merge "libbinder: Remove templated binder::Status stream operator."

Run 'nativeperms', remove Brillo #ifdef.

Bug: 25663983
Change-Id: I5b9f9747695d75f84bdfb61f90d28e0e5fee612c

Merge "Implement a stub permission server for Brillo tier 1."

Implement a stub permission server for Brillo tier 1.

This will allow removing some existing hardcoded checks, like
https://cs.corp.google.com/android/frameworks/native/libs/binder/IServiceManager.cpp?l=70

Also add a .clang-format file for Android Framework C++ style.

Bug: 25663983
Change-Id: Icc4473a6f7b8c7395cdb167dc3fdbb2f076e5960

libbinder: Remove templated binder::Status stream operator.

The templated binder::Status stream operator doesn't work with gmock
because of ambiguous operator overloading. Limiting the stream operator
to just std::stringstream allows it to be used with gmock.

BUG=28171901
TEST=Compiles with gmock and CameraBinderTests

Change-Id: Ia674b68cbff4911b3f5cc3d8ee57d04a1d6cf6bf

libbinder: Replace ScopedFd with base::unique_fd

unique_fd is a more canonical construct going forward.

Bug: 27804373
Test: checkbuilds pass on aosp x86_64-eng

Change-Id: I8c1f202f1a6d646b5468f8c08fdcab7ea7f3178b

Merge "Don't include <sys/sysconf.h>."

Add NDK Choreographer API headers

Change-Id: Ib75d3341db4e48b328b7b92497a303c01c53fc90

Don't include <sys/sysconf.h>.

It doesn't exist in any standard, and is hence being moved. <unistd.h>
exposes everything in it already, anyway.

Bug: http://b/28067717
Change-Id: I258c607ef8799ec267151f424be27c72e295e3b6

Merge "Split increments to silence a compiler warning."

Split increments to silence a compiler warning.

Upstream clang complains that this line may be a possible misuse of the
comma operator, since the result from "p++" is being discarded. Rather
than overcomplicate things with a static_cast<void>(), it is easiest to
just move this to a separate statement.

Change-Id: I5d005eb82a02430f691f99a2f1e8b7237320f84a

Merge "Address const issues in preparation for libcxx rebase."

Merge "Switch GLES wrappers over to using Clang (and fix inline assembly)."

Address const issues in preparation for libcxx rebase.

Change-Id: I25a1e2946217049e9205589d6493e06362d2173b

Switch GLES wrappers over to using Clang (and fix inline assembly).

Bug: http://b/18003438

The GLES wrapper implementations make direct calls to their underlying
function without specifying that they are retaining the original
argument register(s). This change makes the retention of these argument
registers more explicit, and then removes the LOCAL_CLANG override,
since Clang no longer optimizes the code into a bad state.

We also switch to "naked" functions, so that we control all stack layout
information. This prevents us from accidentally still corrupting
arguments on x86 (and for other functions with many parameters).

Change-Id: I37f2ef4a697373388950b41f0a292393c9fcbac7

Merge "Release resources when repeating dlopen-dlcose."

Merge "libbinder: Make null handling explicit for strong binders"

Release resources when repeating dlopen-dlcose.

When the libbinder.so is dlopen and dlclose many
times, we need to delete the pthread_key and
munmap the memory mapping.

Otherwise, it will crash as the leakage causes
the pthread_key or memory mapping surpass their
limit.

Change-Id: Ice64d4ef685c1f665350286f9e5ddf790beac236
Signed-off-by: zhongjie <zhongjie.shi@intel.com>

libbinder: Make null handling explicit for strong binders

Bug: 27156495
Test: Compiles

Change-Id: I62cade1761377a9ec2921a758307582552e4c2ac

Merge ""#warning" conflict with -Werror"
am: e6b6843c5f

* commit 'e6b6843c5fc19976677706518c4875a78cc736bb':
  "#warning" conflict with -Werror

Merge ""#warning" conflict with -Werror"

"#warning" conflict with -Werror

if code run #warning, it will have compile error.

Change-Id: Ic7f05cd722c14f3c5da89e2c3792615dd15cb798
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

Merge commit 'db5c5bb0259fa024517d9a151726785facba4f02' into HEAD

Merge "Allocate buffers locally in systems without SurfaceFlinger"
am: afcf267cbd

* commit 'afcf267cbd86316cd98c96bc82400479c965d8d6':
  Allocate buffers locally in systems without SurfaceFlinger

Merge "Allocate buffers locally in systems without SurfaceFlinger"

Allocate buffers locally in systems without SurfaceFlinger

In Brillo systems, we have no applications and configure
SELinux policy statically.  In this model, we do not need
a special service to allocate buffers, since any process
that needs buffers can be appropriately marked at build time.

Bug: 26936651
Test: A test program on brilloemulator can allocate buffers for
      the camera with this patch.

Change-Id: Ifcdb7a4b878f9a26f4899c7b75a12447dcdb0ac9

Merge "libbinder: add getters for PersistableBundle keys"
am: 4f321d1629

* commit '4f321d1629e2b848b4e9a5304ab4a9262bc81569':
  libbinder: add getters for PersistableBundle keys

Merge "libbinder: add getters for PersistableBundle keys"

Merge "Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp"
am: f4475023e7

* commit 'f4475023e728db1456a72b0e9a1292e11fb38245':
  Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp

Merge "Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp"

Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp

Bug: http://b/27068438
Change-Id: I3064950f761411fe5271c6259867ad437c71e66d

Merge "Replace AppOpsManager locking with std::lock_guard"
am: d3a9228d1d

* commit 'd3a9228d1d2388f21b4e8013cf2fad7b4c91ace7':
  Replace AppOpsManager locking with std::lock_guard

Merge "Replace AppOpsManager locking with std::lock_guard"

Merge "Add missing liblog dependency"
am: 39f1abac3c

* commit '39f1abac3c21e2f5a4a450b21d1c52ba5f099ddb':
  Add missing liblog dependency

Merge "Add missing liblog dependency"

Add missing liblog dependency

Bug: http://b/27171986
Change-Id: Ifbe01c6b325e7198a95ea865f557b0c9acb65809

Merge "Don\'t rely on AppOpsManager in systems without applications"
am: 46c32a1b1c

* commit '46c32a1b1c895ec7630686aa8228785479f8fe92':
  Don't rely on AppOpsManager in systems without applications

Merge "Fix race in AppOpsManager client library"
am: 712fe62ab2

* commit '712fe62ab282b3897fc1ee98e35b12d95663be52':
  Fix race in AppOpsManager client library

Replace AppOpsManager locking with std::lock_guard

Change-Id: Ice4c101659837cea594194cfb335f9f3f6837f38

Merge "Don't rely on AppOpsManager in systems without applications"

Merge "Fix race in AppOpsManager client library"

libbinder: add getters for PersistableBundle keys

Add getters for the keys in each map stored in
PersistableBundle. This will allow users of
PersistableBundle to iterate over entries of
a specific type in PersistableBundle.

Also, add "using" statements to PersistableBundle
for std containers to reduce verbosity.

BUG: 27171235
Change-Id: Idd4b3bf713071ca8d8d023f1f83bdaf3ccbdd46f
TEST: libbinder compiles successfully.

Merge "libbinder: Allow readByteVector to take uint8_t"
am: dda39f2a22

* commit 'dda39f2a2229ea74cc4738daa1c6a1c758284d7d':
  libbinder: Allow readByteVector to take uint8_t

Merge "libbinder: Allow readByteVector to take uint8_t"

Merge "dumpstate: Continued build breakage"
am: 16072e7e17

* commit '16072e7e1717bcf1ea8be99eb7af90974dbbeeb7':
  dumpstate: Continued build breakage

Merge "dumpstate: Continued build breakage"

dumpstate: Continued build breakage

- drop assignment length in scanf
- drop -std=gnu99 as invalid for C++

Change-Id: I924d2561069b3c289cb9deb475f732c46bb5a67e

libbinder: Allow readByteVector to take uint8_t

Change-Id: I2f0d1f3b4c04f04e7a866e9a4cf76c899b4b4980
Test: AIDL integration tests pass
Bug: 27078230

Merge changes I6b499f53,If821ed21
am: c658aee7d6

* commit 'c658aee7d6dc56b1b3b1722412029830e966ba71':
  dumpstate: report process times
  Add option to #define _DUMPSTATE_DRY_RUN_ to skip the actual dumps.

Merge changes I6b499f53,If821ed21

* changes:
  dumpstate: report process times
  Add option to #define _DUMPSTATE_DRY_RUN_ to skip the actual dumps.

dumpstate: report process times

- Fix per-pid helper to pick up comm if cmdline is empty
  and assume kernel thread.
- Add per pid report command line user time if not kernel thread,
  system time and I/O time, plus percentage if non zero.

Bug: 26966375
Change-Id: I6b499f535cc3687eb468b32ae39e7d151dffb11f

Add option to #define _DUMPSTATE_DRY_RUN_ to skip the actual dumps.

(cherry picked from commit 93d705b6d742e8f00167ff2384e35c2205a6f9da)

Such option is useful when debugging dumpstate itself, since it will
finish much sooner and its output will be much smaller.

Change-Id: If821ed21715461bf82eea0b2be4b926239ad69da

Merge "Enable wildcard in the list of kernel functions"
am: fda23631b9

* commit 'fda23631b978bec793b67037dc2841a6a15531fb':
  Enable wildcard in the list of kernel functions

Merge "Enable wildcard in the list of kernel functions"

Merge "dumpstate: refine calculated eMMC performance values"
am: 088e7586ea

* commit '088e7586ea7d0103d1b5395172a4594eddbf5b4a':
  dumpstate: refine calculated eMMC performance values

Merge "dumpstate: refine calculated eMMC performance values"

dumpstate: refine calculated eMMC performance values

Correct the raw performance with the average queue depth. Add a
report for I/O per second, and the average queue depth. Round
values up.

Bug: 26966375
Change-Id: I2a8d233872d5e5ff79d433abba3c34d5945701f4

Don't rely on AppOpsManager in systems without applications

Brillo has no applications, and doesn't run AppOpsManager.  Instead,
services are granted statically configured SELinux permissions at
build time.  Rely on that configuration rather than Android's
dynamically configurable permission model.

Bug: 26936651
Test: Test code on Brillo system is able to connect to the camera.

Change-Id: I84b72a762c2f534c2e1cc6f99ef2003388fb1265

Merge "system_server BINDER_TYPE_FD driver ashmem accessors"
am: b23a388c64

* commit 'b23a388c64d2fef51482feaaf89c345488f3cabc':
  system_server BINDER_TYPE_FD driver ashmem accessors

Merge "system_server BINDER_TYPE_FD driver ashmem accessors"

Fix race in AppOpsManager client library

If the service stays down for more than 10 seconds, client threads
will time out attempting to obtain a reference, and never release
the mutex.  This causes deadlock on the next attempt to speak
with the AppOpsManager.

Test: System without AppOpsManager service no longer hangs in camera
      code.

Change-Id: I5a8ac2f0f35345f72c3d2a45b9d1fc5a78d28a51

Enable wildcard in the list of kernel functions

The handling of the -k option is too restrictive
as it stricly checks the names of the kernel functions
after it wrote them in /d/tracing/set_ftrace_filter.

However, a common usage of that function filtering with ftrace
is to use a wildcard character (*) that the ftrace tracer
automatically expands to all the matching kernel functions.

This enables the support of the wildcard (*) character
in the -k option to trace kernel functions.

Change-Id: Ifffae975c20e1c253157a3a6b44a14b4f342b9d0
Signed-off-by: Thomas Buhot <thomas.buhot@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

Merge "Binder::Status: Add operator<< to help with gtest logging"
am: 259eb56171

* commit '259eb56171420f62be325eadc38673d967527ab6':
  Binder::Status: Add operator<< to help with gtest logging

Merge "Binder::Status: Add operator<< to help with gtest logging"

Binder::Status: Add operator<< to help with gtest logging

Change-Id: I22fcf7c6314451701ba947af2d69d4e45b22345a

system_server BINDER_TYPE_FD driver ashmem accessors

check if device matches the ashmem rdev, before calling
ashmem_get_size_region. This eliminates making this call
when associated with other driver file descriptors.

Bug: 26374183
Bug: 26918423
Bug: 26871259
Change-Id: I1f88c2c93ea35a73c8e14125f3d1a6c67fa4f15b

Merge "Fix typo from last commit."
am: ae8442856b

* commit 'ae8442856bfdcfd4daf287a880ce7ce4eadaa3e4':
  Fix typo from last commit.

Merge "Fix typo from last commit."

Fix typo from last commit.

Bad commit: https://android-review.googlesource.com/#/c/200449/1

Change-Id: I412e2740951f15cf843bd36a50476070c0cb1ebd

Merge "Clean up warnings in inputflinger."
am: f3606ef25f

* commit 'f3606ef25fb588283020c4df178076852eecc4e3':
  Clean up warnings in inputflinger.

Merge "Clean up warnings in EGL."
am: c312141adc

* commit 'c312141adc9a5f00b5d13d0980d132d2a8c6f3c9':
  Clean up warnings in EGL.

Merge "Clean up warnings in inputflinger."

Merge "Clean up warnings in EGL."

Clean up warnings in inputflinger.

Bug: http://b/26936282
Change-Id: Ie1fb281425c3b8c4b3e6e88dc5722d51b77ee093

Clean up warnings in EGL.

Bug: http://b/26936282
Change-Id: I059628332d638f080a5236eed62cf0afa1d4d6ba

Merge "Binder: Add log output in IPCThreadState::self()"
am: 2be113a248

* commit '2be113a2489e7d56b21f8ec340198959348012b3':
  Binder: Add log output in IPCThreadState::self()

Merge "Binder: Add log output in IPCThreadState::self()"

Merge "Add support for decoding UTF16 to UTF8"
am: ac4728f8ba

* commit 'ac4728f8ba54e5f25466efeaf65a1dbda179ca97':
  Add support for decoding UTF16 to UTF8

Merge "Add support for decoding UTF16 to UTF8"

Binder: Add log output in IPCThreadState::self()

In the failure case, and on shutdown, add log output. It's highly
likely the client will crash when a null value is returned. Displaying
a possible root cause helps diagnosing issues.

Bug: 26865304
Change-Id: Ie5255ac50d6dcdf9f918dcef785788ea795a4791

Add support for decoding UTF16 to UTF8

This is helpful because many native service developers use std::string
internally, and String16 doesn't work very well with many C++
primitives.

Bug: 26729450
Test: Integration test appears to show this is correct.

Change-Id: Ib75068881da28fd8515685bcbcbe864ba3b7ec23

Merge "Application image support for installd DO NOT MERGE"
am: 7f9c122ab9  -s ours

* commit '7f9c122ab98d84e54647541edf99e72dec86f6db':
  Application image support for installd DO NOT MERGE

Merge "Application image support for installd DO NOT MERGE"

Application image support for installd DO NOT MERGE

DO NOT MERGE

sort of
(cherry picked from commit edc8bc48fbf143e35578bf6cccf980dfab076196)

Bug: 22858531

Change-Id: I5a9dfcf67d2232e8755886085e9ba1bb37494d39

Merge "system_server BINDER_TYPE_FD sockets using ashmem accessors"

system_server BINDER_TYPE_FD sockets using ashmem accessors

check if device is a character device, before calling
ashmem_get_size_region. We do not check if the st_rdev
matches /dev/ashmem. So this at least eliminates making
this call when associated with a socket.

Bug: 26374183
Change-Id: I68ed9d1c2cd4c47228ed065e3e18eb4151f038f4

Merge changes I4ebccfd0,I3507bdfd,Iaf02f55c

* changes:
  Parcel: file descriptor leak
  Parcel: include Android coding standard
  Parcel: LOG_TAG cleanup

Parcel: file descriptor leak

Resolve a file descriptor leak when a request for
ashmem size adjustment is not filed.

Change-Id: I4ebccfd096ec5313725fd99dc3e025f9561d061f

Parcel: include Android coding standard

Change-Id: I3507bdfd04a06db85b46539338c787395ec652ff

Parcel: LOG_TAG cleanup

NB: several ALOG messages behind comments, recommend removal

Change-Id: Iaf02f55cbf7ef0ee7c4a6ee67c7f8b57cf97cd26

Merge "Revert "dumpstate: Add security log dump""