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()"

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"

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""

Revert "dumpstate: Add security log dump"

This reverts commit c7ad8cb82f035576c28707a8cbeb16b7ead2a4e7.

Bug: 26029733
Change-Id: I0c61f4d2c0aa9e07c6cc7b190d803e98c3a98d40

Merge "Implementation of HCE for NFC-F."

Implementation of HCE for NFC-F.

This impementation includes HCE(Host-based Card Emulation) for NFC-F
which emulates NFC Forum Type 3 Tag on a smartphone.

Signed-off-by: Daisuke Ito <DaisukeE.Ito@jp.sony.com>
               Yoshinobu Ito <Yoshinobu.Ito@jp.sony.com>

Change-Id: I1618a7fba801e0705ff3cc078206a1446b3dd56d

Merge "Add PersistableBundle.aidl into binder includes"

Add PersistableBundle.aidl into binder includes

Add the PersistableBundle.aidl into the new /aidl/binder/
subdirectory (along with the necessary 'android' and 'os'
subdirectories) so that the aidl file will live alongside
its native implementation in libbinder. This change will
allow native-only Android checkouts (e.g. Brillo) to use
PersistableBundle.

The /aidl/ subdirectory should henceforth contain any
aidl files that live with native code.

BUG: 26292234
Change-Id: I07b49dad8afd679f33e41fa9044dffc02ec25e3e

Merge "libinstalld: add dependency on libselinux"

Merge "More O_CLOEXEC"

More O_CLOEXEC

Change-Id: I31991ed089f395174857e86e509b8190ad98f92a

libinstalld: add dependency on libselinux

libinstalld uses headers from libselinux, add a LOCAL_SHARED_LIBRARIES
dependency on libselinux.

Change-Id: Ic12c55547526638810bcc1ac952d205f2f163fd3

Merge "Fix format specifiers for size_t types"

Fix format specifiers for size_t types

Use %zu instead of %u for logging size_t types, which
are of the unsigned long type, not the unsigned int type.

BUG: None
Change-Id: I5af6c2ea544a5f11b283bc9aebe8b5f0cfc605ec

Merge "Add support for PersistableBundle in C++"

Merge "atrace.rc: tracing should be disabled by default."

atrace.rc: tracing should be disabled by default.

Bug: 26217098
Change-Id: I6066c433d19c2d1c220669a9a78f5085af8c1d4b

Merge "Allow null values for Parcel readers and writers"

Allow null values for Parcel readers and writers

Test: AIDL integration tests pass
Bug: 25969194
Change-Id: Ib7023c8e02af49e08c9921f1d097e5967c1d378e
Signed-off-by: Casey Dahlin <sadmac@google.com>

Merge "Don't redefine B_PACK_CHARS"

Don't redefine B_PACK_CHARS

Change-Id: I5dfaca405f0f866e697d5ae546ead8a8eb6de7ca
Test: make checkbuild succeeds
Bug: 25868260
Signed-off-by: Casey Dahlin <sadmac@google.com>

Merge "Migrated dumpstate to C++."

Migrated dumpstate to C++.

Changes required:

- Explicity casting function pointers.
- Moving variables initialization before 'goto' statements.
- Changing string references from 'char *' to 'const char *'.
- Using 'extern "C"' so device-dependent implementation of
  'dumpstate_board()' can be written in C or C++.

BUG: 25563823

Change-Id: Ibeb70fd23e64e9b530736b554a372cb232fe1a44

Add support for PersistableBundle in C++

Add support for PersistableBundle, a mapping from
String values to various types, in C++.

BUG: 25815410
Change-Id: If609b294a1709314bb4220afc4f2269b556babb8

Merge "atrace: add database tag"

Merge "dumpstate: Add security log dump"

dumpstate: Add security log dump

Bug: 26029733
Change-Id: I2e5b44d642bbac7f62f63388dd8a3123f2abeafe

Merge "dumpstate: build breakage @2482293"

dumpstate: build breakage @2482293

aosp is in C, internal is C++, struct must be explicit

Bug: 24200279
Change-Id: I1c9d8156a86db6de73ee5b70063e63b38170b052

Merge "dumpstate: Dump eMMC 5.0 stats"

dumpstate: Dump eMMC 5.0 stats

Extended csd EXT_CSD_REV, PRE_EOL_INFO, DEVICE_LIFE_TIME_EST_TYP_A,
and DEVICE_LIFE_TIME_EST_TYP_B fields printed if available. Allow
dumpstate access to /sys/kernel/debug/mmc0/mmc0:0001/ext_csd.

Bug: 24200279
Change-Id: I310bdcf546a9307cf927f109359ad5a6d24b70f8

Merge "Fix allocation count"

Merge "libbinder: Enable service specific error codes"

libbinder: Enable service specific error codes

Add another factory method that takes a message and service
specific error code.

Bug: 25800533
Test: system/tools/aidl integration tests pass

Change-Id: I592cb7def0538576965d14c200ab58548b3bef32

Fix allocation count

The realloc case in continueWrite should not increment the counter,
the pointer passed to realloc is guaranteed to be non-NULL so the total
number of allocations will not have changed.

When realloc is called in restartWrite mData has not been checked
against NULL, increment the counter if it was NULL.

Bug: 26086286
Change-Id: I4c8af450cca1868b91793c0c5f0d8c4b4b5badbe

Merge "Fix the execution point of onFrameAvailable/onFrameReplaced callbacks"
am: 789ede379d

* commit '789ede379d2f19ce284866c72bcaad7e8255b1cc':
  Fix the execution point of onFrameAvailable/onFrameReplaced callbacks

Merge "Fix the execution point of onFrameAvailable/onFrameReplaced callbacks"

Merge "Replace libGLESv3 symlink with shared library"
am: 18ff655766

* commit '18ff6557667f3d130ad1f83e04f3b175f8b8cf6d':
  Replace libGLESv3 symlink with shared library

Merge "Replace libGLESv3 symlink with shared library"

Fix the execution point of onFrameAvailable/onFrameReplaced callbacks

In a4650a5 the concept of a maximum frame number allowance for the consumer was
introduced. A call to acquireBuffers will only return buffers when their frame
number is less-than-or-equal-to this maximum frame number. When SurfaceFlinger
is the consumer, this maximum  frame number is calculated in the
onFrameAvailable/onFrameReplaced callbacks. These callbacks are called when a
new buffer is dequeued by the application. The problem is that these callbacks
are called _after_ the fence wait which is used to throttle the frame
production of client apps. When the previous frame needs a long time to draw,
those waits can potentially be a long time. As a result SurfaceFlinger won't do
any composition with the new frame until the wait is over.

Normally this isn't a big problem because there is a queue of buffers for
SurfaceFlinger to work with. However, this changes massively when a client app
is using a swap interval of zero. In this case, a new frame will instantly
replace the previous queued frame. However, SurfaceFlinger doesn't know this
until the onFrameReplaced callback gets called - which is delayed by the fence
wait. If the timing is bad, SurfaceFlinger never gets a chance to pick up a new
frame to do the composition with.

We see this behaviour on our TC development system (slow GPU) with legacy
on-screen benchmarks. Such apps are using a swap interval of zero and sometimes
frames don't get updated for several seconds. This behaviour can be also seen
on a Nexus5, although it isn't as obvious as on our TC.

The fix in this cl is to move the EGL throttling to the end of the queueBuffers
function. This ensures that if a frame gets replaced in the queue, all
consumers who installed the callbacks, get called in a timely fashion.

Change-Id: I36e9ecda162150f41e97d4fb7437963a3d86b371
Signed-off-by: Christian Poetzsch <christian.potzsch@imgtec.com>

Merge "Revert "libbinder: Don\'t redefine B_PACK_CHARS""
am: 3f46d82d7b

* commit '3f46d82d7b8ee0ae6295723311e5c986b649d727':
  Revert "libbinder: Don't redefine B_PACK_CHARS"

Merge "Revert "libbinder: Don't redefine B_PACK_CHARS""

Revert "libbinder: Don't redefine B_PACK_CHARS"

This reverts commit 3eab1b40acbcc2e82b3e90da26706129019eccf9.

Change-Id: Iedc99961c6014029722f814e621970c1732f9092

Merge "Track rename from base/ to android-base/."
am: f31cf3ab0f

* commit 'f31cf3ab0f53394200c0f22a4591db741ba1ccb0':
  Track rename from base/ to android-base/.

Merge "Track rename from base/ to android-base/."

Merge "libbinder: Don\'t redefine B_PACK_CHARS"
am: 7166e7823c

* commit '7166e7823cedb817f0278f569940aa86f424aa3b':
  libbinder: Don't redefine B_PACK_CHARS

Merge "libbinder: Don't redefine B_PACK_CHARS"

libbinder: Don't redefine B_PACK_CHARS

This macro is defined in linux/binder.h which is *sometimes* included
before binder/IBinder.h depending on codepath.

Bug: 25868260
Test: Compiles

Change-Id: Ied1e6658936668c16cbdacab3a3205107c907655

Track rename from base/ to android-base/.

Change-Id: I62c18891ffe319d20ff084f5ca2aa0c038c23cd3

Merge "libbinder: Handle transaction failures correctly"
am: 823fa55aec

* commit '823fa55aec55684605e6f4c764ec20901066553c':
  libbinder: Handle transaction failures correctly

Merge "libbinder: Handle transaction failures correctly"

libbinder: Handle transaction failures correctly

Java code expects status_t != OK to be caught at the JNI level in
android_util_Binder.cpp (see signalExceptionForError).  We were
incorrectly mapping this kind of failure to a special exception type
and writing that exception type to parcels.

Instead, refuse to write EX_TRANSACTION_FAILED to a parcel and return
the status value instead.

While here, remove non-trivial constructors to push authors toward the
more explicit factory methods.  Remove getException() and push authors
toward using the simpler getter methods. Fix minor camelCase issues.

Bug: 25615695
Test: system/tools/aidl integration tests still pass

Change-Id: I7cad3ac8ae8300b5ac0b466606f4934d01e503c5

Merge "bug#460672 framebuffer surface can\'t keep triple buffer if define NUM_FRAMEBUFFER_SURFACE_BUFFERS := 3"
am: b2e3095a60

* commit 'b2e3095a604e16d5d80d1e3d38bb92a6c37bd127':
  bug#460672 framebuffer surface can't keep triple buffer if define NUM_FRAMEBUFFER_SURFACE_BUFFERS := 3

Merge "bug#460672 framebuffer surface can't keep triple buffer if define NUM_FRAMEBUFFER_SURFACE_BUFFERS := 3"

bug#460672 framebuffer surface can't keep triple buffer if define NUM_FRAMEBUFFER_SURFACE_BUFFERS := 3

[root cause  ] framebuffer surface is in the same process of surface flinger
[changes     ] call IGraphicBufferProducer::allocateBuffers on producer inside the DisplayDevice constructor to force-allocate all three buffers.
[side effects] none
[self test   ]<tshark, enter and exit mainmenu>: ok
[reviewers   ] zhongjun.chen
[change_type ] AOB --> google_original
[tag_product ] common

Change-Id: Ideb75d3aa1ed23b2fa4d263a120ea3ca39d01361

Merge "Atrace: support streaming data to stdout."
am: 83a98b101c

* commit '83a98b101c2850a936c0e612e4eb16d815db025e':
  Atrace: support streaming data to stdout.

Merge "Atrace: support streaming data to stdout."

Atrace: support streaming data to stdout.

Change-Id: I8534a9c69f25ecdd82230a5d8b4b8d1c4d58439e

Merge "Add support for file descriptors and vectors thereof"
am: 8f237272e0

* commit '8f237272e0630b8f279db7f90aebfff73a1a9c00':
  Add support for file descriptors and vectors thereof

Merge "Add support for file descriptors and vectors thereof"

Add support for file descriptors and vectors thereof

We use ScopedFd from nativehelper to manage the lifetime.

Test: Passes AIDL unit and integration tests. Full build completes and
      runs.
Bug: 25242023
Signed-off-by: Casey Dahlin <sadmac@google.com>
Change-Id: I22c87ed1cb3bd4bf667b372ea4a2e9fcd4dd986a

atrace: add database tag

Change-Id: I96acc1afa8d2eeac5585ee482d3f35716da49950
Signed-off-by: Greg Hackmann <ghackmann@google.com>

Merge "libbinder: Add support for C++ Parcelable"
am: 49b5443a49

* commit '49b5443a49d737d13c8d05dfd9b827b2c4198edc':
  libbinder: Add support for C++ Parcelable

Merge "libbinder: Add support for C++ Parcelable"

Merge "libbinder: Move vector writing templates to header"
am: e1aa1c7e13

* commit 'e1aa1c7e136e7372c17b8db7a6d8b307ba4cd571':
  libbinder: Move vector writing templates to header

Merge "libbinder: Move vector writing templates to header"

Merge "Add support for new sched kernel tracepoint"
am: 61c9506a09

* commit '61c9506a09959ae07832cc019818f869403d3f2a':
  Add support for new sched kernel tracepoint

Merge "Add support for new sched kernel tracepoint"

Add support for new sched kernel tracepoint

This tracepoint gives systrace the means to display the reason
for uninterruptible sleep.

libbinder: Add support for C++ Parcelable

Bug: 23600712
Test: compiles, integration tests for Java/C++ compatibility pass

Change-Id: I0919571919a3633350169ed5668bbb000da9691c

libbinder: Move vector writing templates to header

This allows us to reuse them to read and write vectors of parcelables.

Bug: 23600712
Test: Compiles, aidl integration tests pass

Change-Id: Ibbfc2f158b6b3000b13f35f3c09a0cd3741e8128

Replace libGLESv3 symlink with shared library

Because of the link to libGLESv2.so, libGLESv3.so had
incorrect dt_soname, which led to inability to use
in in linker public namespaces.

This change replaces symlink with shared library.

Bug: http://b/25754041
Change-Id: I3743c81f05bc6caf746e40ef7531723e97632a70

Merge "Revert "Add support for unique_fds and vectors of file descriptors""
am: 5cd09786eb

* commit '5cd09786eb4ff6e99351e8a317205a5efd12ac74':
  Revert "Add support for unique_fds and vectors of file descriptors"

Merge "Revert "Add support for unique_fds and vectors of file descriptors""

Revert "Add support for unique_fds and vectors of file descriptors"

This reverts commit 9626d88882972d45576dcceedc037d2e9843196b.

Change-Id: I6121f388d17c6f2d0cf6f31bc42f0804dd72aba2

Merge "Add support for unique_fds and vectors of file descriptors"
am: b1819fc295

* commit 'b1819fc295811ca3a005042a80149c8d0655211f':
  Add support for unique_fds and vectors of file descriptors

Merge "Add support for unique_fds and vectors of file descriptors"

Add support for unique_fds and vectors of file descriptors

Change-Id: Ic61e69d9db438b4686032459c896aed2a482c9a5
Test: Unit tests pass
Bug: 25242023
Signed-off-by: Casey Dahlin <sadmac@google.com>

Merge "libbinder: Add binder::Status type"
am: babed3ea52

* commit 'babed3ea523822f4f5084c3e8a951f2cdf924695':
  libbinder: Add binder::Status type

Merge "libbinder: Add binder::Status type"

libbinder: Add binder::Status type

This object implements equivalent functionality to the Java logic which
serializes and re-throws exceptions from services.

Bug: 25615695
Test: Integration test for generated AIDL code reveals this to work
      correctly.

Change-Id: Ic80c9def0dd232582ea9c49717ec50894af6bfc2

Merge "use negative mask"
am: 4e427f0aab

* commit '4e427f0aabf89b118b9a8f2b128f71a058755ba7':
  use negative mask

Merge "use negative mask"

use negative mask

the test should be for _invalid_ bits; as it's written,
it tests for valid bits

Change-Id: Id5e42e865449a7877fe58aca9fb47675fa7452c9

Merge "Remove obsolete profile handling from installd."
am: 8161da7a0e

* commit '8161da7a0e54bc4af36e49353e4caad02de384dc':
  Remove obsolete profile handling from installd.

Merge "Remove obsolete profile handling from installd."