Merge "Remove unused include."

Remove unused include.

BUG: 29824203

Change-Id: Ied8ccb8c30632b41f6c6c3d848dc94d7b1e731e0

Merge "Start converting Android.mk to Android.bp"

Start converting Android.mk to Android.bp

See build/soong/README.md for more information.

Merged-In: Ifbe9b0072a63b4467c72b3a52148a606e5365f78
Change-Id: Ifbe9b0072a63b4467c72b3a52148a606e5365f78

Merge "libsigchain: intercept bsd_signal on lp32"

libsigchain: intercept bsd_signal on lp32

bsd_signal is a deprecated synonym for the signal. Nevertheless
there are apps and libraries calling this function.

libsigchain needs to intercept these calls and handle them the same
way it handles signal() call.

Bug: http://b/30562229
Test: readelf --dyn-sym surfaceflinger and check that bsd_signal is exported for lp32
      and is not exported for lp64 binary version
Change-Id: If80e45502275b26c3ee4026ab12cadd77f8fbd3b

Merge "egl: Call loseCurrent after eglReleaseThread"

Merge "egl: Enable EGL_KHR_GL_COLORSPACE"

Merge "Fix a typo in the atrace help text."

Fix a typo in the atrace help text.

Change-Id: I1b9d04d3d17e851d08a8ae39e26483e106516b88

Merge "Clean up memory allocation in dumpTrace."

Clean up memory allocation in dumpTrace.

Bug: http://b/20674092
Change-Id: Id8ba2b0efc1641929790770714e48cde545fb3b0

Merge "Add |readString8| method which returns failure"

Add |readString8| method which returns failure

Add a new method to return |String8| read failures from Parcels.

BUG: 30189542
Change-Id: I65b59887dd9a15068abe95d24d390aa577bc49a6

Merge "Change to include linux/android/binder.h."

Merge "libbinder: Tighten compiler warnings/errors"

egl: Enable EGL_KHR_GL_COLORSPACE

Add EGL_KHR_GL_COLORSPACE extension to EGL header.
Enable EGL_KHR_GL_COLORSPACE extension handling in meta EGL layer.

Change-Id: If6d3fe26c411ddc83d8b23bf16e93862cf68c1b6

Merge "Fix broken readNullableStrongBinder<T> implementation"

libbinder: Tighten compiler warnings/errors

Use all of -Wall -Wextra -Werror and fix the minor errors that result.

Change-Id: Id56e9c454dd3570f713d161577eed6aa8e8074aa

Fix broken readNullableStrongBinder<T> implementation

Added a missing return statement.  Omitting this statement
caused the compiler to silently cast a pointer to a status_t,
which in general wasn't the expected value of OK.

Only return an error when the type casted return value is null and
the IBinder we got back was not null, indicating a type error, rather
than the expected null binder.

Bug: 30160959
Change-Id: Ief3c924b902a607335eff2fcd4d860cd8523593a
Test: Integration tests in aidl that exercise this method now pass.

Change to include linux/android/binder.h.

The upstream kernels have binder.h as linux/android/binder.h, so
match that.

Bug: 30072483
Change-Id: Iacdc96629dd09c1f4be0a5767666ab64b3df8f15

Merge "Workaround alloc-dealloc-mismatch error on ASan device boot."

Workaround alloc-dealloc-mismatch error on ASan device boot.

/proc/self/environ inaccessible in SurfaceFlinger
=> ASan fails to read ASAN_OPTIONS => alloc-dealloc-mismatch bug is not
suppressed and prevents the device from booting.

This is not perfect because, by enabling it in asan build only, the first several attempts to start
surfaceflinger fail while the /data partition is being encrypted. On the other hand, this does not affect
user builds, and device eventually gets up anyway.

BUG=b/30067360

Change-Id: Ieaa37ff7768a11450ea33fee1f7d70a41b246c6a

Merge "[mips] Resolve issue with .align directive, when using clang IAS"

[mips] Resolve issue with .align directive, when using clang IAS

Clang IAS does not support .align without args, concrete value is needed.

Change-Id: I03a9daca0be380daa744307b3510446f8d6c9b7e

Merge "Fix misc-macro-parentheses warnings in opengl."

Fix misc-macro-parentheses warnings in opengl.

Add parentheses around negative literals.
Bug: 28705665
Test: build with clang-tidy

Change-Id: I342f1d04ac08bbc28ee3ff4ebbbccbfa9f754a94

Merge "dumpstate: Also pick up /sys/fs/pstore/console-ramoops-0 for LAST KMSG"

dumpstate: Also pick up /sys/fs/pstore/console-ramoops-0 for LAST KMSG

Bug: 29639467
Change-Id: Iba0b1754eaebfb2b0624dcbef522b7d8456eeb50

Merge "No need for dumpstate to explicitly reference 'toybox'..."

No need for dumpstate to explicitly reference 'toybox'...

...now we've switched the symlinks over to the toybox implementations.

Change-Id: I6301e3bc8de7210bf111889ee3b014802aecf15c

Merge "Move dumpstate over to toybox top."

Move dumpstate over to toybox top.

Change-Id: I3715b0d217aaf263687d6cab24ca470caf7c70a0

Merge "Fix misc-macro-parentheses warnings in openmax."

Merge "dumpstate: sscanf requires an asciiz string"

Merge "Fix misc-macro-parentheses warnings in libs/binder."

Merge "Fix misc-macro-parentheses warnings in libs/gui."

Merge "Fix misc-macro-parentheses warnings in sensorservice and CHECK_INTERFACE."

Merge "Fix misc-macro-parentheses warnings in test macros."

Merge "Fix misc-macro-parentheses warnings in opengl."

Fix misc-macro-parentheses warnings in libs/gui.

Bug: 28705665
Change-Id: I9fcebd23e0bcf89c41cb3cbc31cae00c8e623dbd

Fix misc-macro-parentheses warnings in openmax.

Bug: 28705665
Change-Id: Ifda977c1754b4bc87b0fe24bf3dab6890bdf344a

Fix misc-macro-parentheses warnings in sensorservice and CHECK_INTERFACE.

Bug: 28705665
Change-Id: I53ed01b67054327ad55a7e83143cc82283573fe1

Fix misc-macro-parentheses warnings in test macros.

Bug: 28705665
Change-Id: I6e64ccdae5869b66a3caa09d8f32b2fcc9c1b2b9

Fix misc-macro-parentheses warnings in libs/binder.

Bug: 28705665
Change-Id: I98a1dc5cc93e31609d3feedb6dd6a33685be69a6

Fix misc-macro-parentheses warnings in opengl.

Bug: 28705665
Change-Id: I761d310f0dff182c288cdbb5371272f4833aa282

dumpstate: sscanf requires an asciiz string

Bug: 28770668
Change-Id: Ice56b129a243296b2691d55e8f8c35b478b1f481

Merge "libbinder: Use char* message with binder::Status"

libbinder: Use char* message with binder::Status

Bug: None
Change-Id: Idcb8345cda2bf18960c1c437b7a6de4ec17b59fe

Merge "Fix google-explicit-constructor warnings."

Merge "Switch to more conventional ps arguments in dumpstate."

Switch to more conventional ps arguments in dumpstate.

I'll come back and remove the "toybox" once I've switched the symlink
from toolbox to toybox...

Change-Id: I00f848ab4dade8650726b054429a2926d3d72018

Fix google-explicit-constructor warnings.

Bug: 28341362
Change-Id: Icd42a3b0043647fd4c094e8fe195ca0f94445de9

Merge "Fix google-explicit-constructor warnings."

Fix google-explicit-constructor warnings.

Bug: 28341362
Change-Id: I83bb2282249ecd4e39347b90f5fb53788e1fff5b

Merge "Fix google-explicit-constructor warnings."

Merge "Atrace:use ReadFileToString to read file"

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

Merge "Remove HAVE_PTHREAD_COND_TIMEDWAIT_RELATIVE test."

Fix google-explicit-constructor warnings.

Bug: 28341362
Change-Id: I0ce850de85e0de6a5472dd142f738856142db9ec

Remove HAVE_PTHREAD_COND_TIMEDWAIT_RELATIVE test.

This is no longer defined anywhere, and has never existed on LP64.

Change-Id: Ic2c51410526f6adbdfe872d4c3b99cbe3badb8d1

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

Atrace:use ReadFileToString to read file

Since wildcard support was added to atrace, it can happen
very easily that file size is large than 4096
resulting in bad status that atrace cannot find kernel function
This patch use android::base::ReadFileToString to read file
which doesn't have size limitation.

Signed-off-by: Stephane Gasparini <stephane.gasparini@intel.com>
Change-Id: Ic24a9267053302a03ff04c8b2afeb143e5b94c84
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

Merge "Include libbase headers in libbinder include path"

Include libbase headers in libbinder include path

We will eventually switch to using libbase's unique_fd in favor of
ScopedFd.  Include the header by itself to check that all build targets
that need libbinder headers are picking up libbase as well.

Bug:  27804373
Test: Compiles

Change-Id: I2920b3d1a5357d81a54267febac3ab72fee88eb1

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