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

Merge "Use templates for write*Vector methods"
am: 81960facb6

* commit '81960facb673740acd41b72d21febea16ba4986c':
  Use templates for write*Vector methods

Merge "Use templates for write*Vector methods"

Use templates for write*Vector methods

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

Merge "Move dumpstate to top -H rather than top -t."
am: 20986bf383

* commit '20986bf3830dabae74837660403c43f92c9131bd':
  Move dumpstate to top -H rather than top -t.

Merge "Move dumpstate to top -H rather than top -t."

Move dumpstate to top -H rather than top -t.

Change-Id: Ibc26b51e92425b2de7870a7a2890fc3566909cda

Merge "libbinder: Return UNEXPECTED_NULL when appropriate"
am: 41e50f99bc

* commit '41e50f99bc3b832df2b7249ce1d3fc92e718c98d':
  libbinder: Return UNEXPECTED_NULL when appropriate

Merge "libbinder: Return UNEXPECTED_NULL when appropriate"

libbinder: Return UNEXPECTED_NULL when appropriate

Return a meaningful and consistent error code when libbinder encounters
a null reference that it didn't expect.  This allows us to detect these
situations and cause a corresponding NullPointerException to be thrown
on the remote callsite.

While here, consolidate places we read vector<T> from a parcel into a
single templated version.  This dramatically reduces code duplication.

Bug: 25617251
Test: Integration tests asserting that vectors serialize correctly
      passes.  Integration test asserting that we throw
      NullPointerException on seeing a null reference also passes.

Change-Id: Ia59c264884ec89539c1b99cd7bd28ac201b5b7ac

Merge "Allow read access to /proc entries for other UIDs"
am: fd53598220

* commit 'fd53598220e19fbb91270f1f1831f6c69c669aaa':
  Allow read access to /proc entries for other UIDs

Merge "Allow read access to /proc entries for other UIDs"

Allow read access to /proc entries for other UIDs

Allow dumpstate (aka adb bugreport), servicemanager, and
surfaceflinger to access /proc entries associated with other UIDs.

Bug: 23310674
Change-Id: I385dcf0db3376ba979409cbe6fe1a468d36237ef

Merge "Add support for reading/writing a vector of binders"
am: fd6843236f

* commit 'fd6843236fe9094fc8868f1a58e6ba4364a485a7':
  Add support for reading/writing a vector of binders

Merge "Add support for reading/writing a vector of binders"

Remove obsolete profile handling from installd.

Bug: 24698874
Bug: 17173268
Change-Id: I140d70262f8f8fa7afd52c3cf63b4abaf401a479

Merge "Add ipi tracing to irq tracing."
am: c84da67d72

* commit 'c84da67d723ce4bafc3622cd269207ea3995ff1c':
  Add ipi tracing to irq tracing.

Merge "Add ipi tracing to irq tracing."

Merge commit '75c5de42b759bdcfa7c0e6103fb959c28d686116' into HEAD

Add support for reading/writing a vector of binders

Change-Id: Iaa8da704b2ae3c1ca5456177441a335991b40e8a
Test: unit tests pass
Bug: 24470786
Signed-off-by: Casey Dahlin <sadmac@google.com>

Add ipi tracing to irq tracing.

Merge "Fix bug in byte vector serialization"
am: fe98e8d6e5

* commit 'fe98e8d6e53ffef172c4571e6c408c4801c3e96c':
  Fix bug in byte vector serialization

Merge "Fix bug in byte vector serialization"

Merge "Further reduce duplication in bugreport ps output."
am: 670508d09a

* commit '670508d09acf64fad83d0c7ac90330d3db89c861':
  Further reduce duplication in bugreport ps output.

Merge "Further reduce duplication in bugreport ps output."

Further reduce duplication in bugreport ps output.

PROCESSES and PROCESSES AND THREADS can be combined. There's nothing in
the former that isn't in the latter.

Change-Id: I105a6d8d35488236ef6dbf4107496ea7e5ad70ea

Fix bug in byte vector serialization

Byte vectors are used by the generated C++ code as the representation
of Java byte[].  This type is serialized as a packed byte array on
the java side.

Bug: 25012838
Test: integration tests for byte[] show this type crossing language
      boundaries correctly.

Change-Id: I8bb1b8ffcb77ced44f99f6b370226a32694f7df1

Merge "Reduce duplication in bugreport ps output."
am: 8ebad13b34

* commit '8ebad13b340921a40a367c9f1808d0c26147e2a6':
  Reduce duplication in bugreport ps output.

Merge "Reduce duplication in bugreport ps output."