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

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

Revert "libbinder: Don't redefine B_PACK_CHARS"

This reverts commit 3eab1b40acbcc2e82b3e90da26706129019eccf9.

Change-Id: Iedc99961c6014029722f814e621970c1732f9092

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

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"

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"

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

Atrace: support streaming data to stdout.

Change-Id: I8534a9c69f25ecdd82230a5d8b4b8d1c4d58439e

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

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

Merge "libbinder: Move vector writing templates to header"

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

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"

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"

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"

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

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

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

Change-Id: Ibc26b51e92425b2de7870a7a2890fc3566909cda

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"

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"

Remove obsolete profile handling from installd.

Bug: 24698874
Bug: 17173268
Change-Id: I140d70262f8f8fa7afd52c3cf63b4abaf401a479

Merge "Add ipi tracing to irq tracing."

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

Reduce duplication in bugreport ps output.

PROCESSES AND THREADS and PROCESSES (SELINUX LABELS) can be combined.
I suspect we can just remove PROCESSES too, since it's a strict
subset of the other information, but one step at a time. (The worry
is that someone's actually parsing this output, but it would be good
to find out who's doing that as soon as possible so we can work with
them as we transition to a more complete ps implementation.)

Change-Id: I5f1085f8853f7a988f7edcaeefdd7b63175c1473

Merge "Add readStrongBinder that takes an interface"
am: 71b70a244c

* commit '71b70a244c56719d30654b7123a944d971eb8d02':
  Add readStrongBinder that takes an interface

Merge "Add readStrongBinder that takes an interface"

Add readStrongBinder that takes an interface

We use a template function to let us pass arbitrary interfaces, all of which
can be cast to IBinder.

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

Merge "libbinder: Fix out of bounds in readBoolVector"
am: 1f76049c95

* commit '1f76049c955c605b61ef0d923582c740c325b1a0':
  libbinder: Fix out of bounds in readBoolVector

Merge "libbinder: Fix out of bounds in readBoolVector"

libbinder: Fix out of bounds in readBoolVector

Bug: 25012838
Test: This code now works at runtime.

Change-Id: I3d6b2677f11276e6be4c51451ca36572a4ee573d

Merge "surfaceflinger: Set max acquired buffer count"
am: af5c3b1d6a

* commit 'af5c3b1d6a9cc1d36b526291391009b2a4e097ff':
  surfaceflinger: Set max acquired buffer count

Merge "surfaceflinger: Set max acquired buffer count"

Merge "libbinder: Stub out checkPermissions() on Brillo."
am: 0d6d2917cb

* commit '0d6d2917cb22ba09da632552c7074d3aa110fbdb':
  libbinder: Stub out checkPermissions() on Brillo.

Merge "libbinder: Stub out checkPermissions() on Brillo."

libbinder: Stub out checkPermissions() on Brillo.

When BRILLO is defined at build-time, avoid checking
permissions with ActivityManager (which isn't built for
Brillo).

Bug: 24907738
Change-Id: I8e277fe45d73d551e3a8ab41bd9dfa2535967522

Merge "Fix benign integer overflow in printHexDump"
am: b36c7d0b6e

* commit 'b36c7d0b6e793fae2d27b43d0022981c0d366d55':
  Fix benign integer overflow in printHexDump

Merge "Fix benign integer overflow in printHexDump"

Fix benign integer overflow in printHexDump

There is an instance where an unsigned integer is multiplied by -1.
Refactor the code to eliminate this operation.

Bug: 25085348
Change-Id: I9c5dc1cc22a27dc998ae8eeacc9f889373d993cd

Merge "Fix build breakage due to extraneous dereference"
am: a763da3f1f

* commit 'a763da3f1fc010576b4807fa9b050c0720f81d00':
  Fix build breakage due to extraneous dereference

Merge "Fix build breakage due to extraneous dereference"

Fix build breakage due to extraneous dereference

Change-Id: If84d8b68a6af1e04a6085351655e588ebef65664

Merge "Add read/write methods for vectors"
am: d83a34d8c7

* commit 'd83a34d8c7fa4be9bd2fa3da4d3ca2c16febd651':
  Add read/write methods for vectors

Merge "Add read/write methods for vectors"

Add read/write methods for vectors

Also a readString16 with the expected semantics.

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

am 4e6d8c5b: Merge "Add methods to Parcel for bool and char and byte"

* commit '4e6d8c5b9d51f11db7e8dbe91971c8b46beae5c0':
  Add methods to Parcel for bool and char and byte

Merge "Add methods to Parcel for bool and char and byte"

Add methods to Parcel for bool and char and byte

We lift these to int, but handling the details of that here will be better
than having AIDL generate casting code all the time.

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

am 8e0c88ea: Merge "servicemanager: log pid and uid on selinux denial"

* commit '8e0c88ea489a7b395a3d349f81000fb2639bf448':
  servicemanager: log pid and uid on selinux denial

Merge "servicemanager: log pid and uid on selinux denial"

am 52b25375: Merge "dumpstate: add -v printable to logcat"

* commit '52b2537550a6fc43587980887858c7e7150255b0':
  dumpstate: add -v printable to logcat

Merge "dumpstate: add -v printable to logcat"

dumpstate: add -v printable to logcat

In the case where binary data lands in an utf-8 log buffer,
ensure that the content is made printable.

Bug: 23517551
Change-Id: I41696372d07228875d01a19f0c7ab2cb7494b7ef

am ae0405d6: Merge "dumpstate: execute procrank using su"

* commit 'ae0405d607f9ed08d708ee990716a7881a6cfd4d':
  dumpstate: execute procrank using su

Merge "dumpstate: execute procrank using su"

dumpstate: execute procrank using su

procrank is a setuid binary which only exists on userdebug/eng
builds. Instead of executing the setuid binary, run the binary using
the su command. This eliminates one more setuid binary, and allows
the tightening of the SELinux policy.

Bug: 18342188
Change-Id: I81d06196a29489feaefb2fea0cf5c4ff1e273d2f

servicemanager: log pid and uid on selinux denial

The audit logs for servicemanager were missing the pid and uid of the source.
This is useful for debugging.

Before:
... SELinux : avc:  denied  { find } for service=android.security.keystore scontext=u:r:system_app:s0 ...
After:
... SELinux : avc:  denied  { find } for service=android.security.keystore pid=1252 uid=1000 scontext=u:r:system_app:s0 ...

Change-Id: Id1a6f38b99f11a31315439620ead2f01108b18b2
Signed-off-by: William Roberts <william.c.roberts@intel.com>

am ddf5f250: am 05c86225: do not dumpstate raft BUG: 24159112

* commit 'ddf5f2509b48ee85dc352ee245f34a9d6307be39':
  do not dumpstate raft BUG: 24159112

am 05c86225: do not dumpstate raft BUG: 24159112

* commit '05c862252d5021f55a69281473c67d76e7097422':
  do not dumpstate raft BUG: 24159112

do not dumpstate raft
BUG: 24159112

Change-Id: I344aa73808992b5f84cb0536201a3ab2eb6f2b3c

am c9b39239: am 6c51a07d: am de18f6c3: InputResampling: Don\'t extrapolate for very low frame rates.

* commit 'c9b392395f3eada767cf9d1dce1eb40930ca1b95':
  InputResampling: Don't extrapolate for very low frame rates.

am 6c51a07d: am de18f6c3: InputResampling: Don\'t extrapolate for very low frame rates.

* commit '6c51a07d30a3c4bed9675da88c5f6f24a741d9a2':
  InputResampling: Don't extrapolate for very low frame rates.

am de18f6c3: InputResampling: Don\'t extrapolate for very low frame rates.

* commit 'de18f6c32add6fb22065807a00ddc88b363df527':
  InputResampling: Don't extrapolate for very low frame rates.

InputResampling: Don't extrapolate for very low frame rates.

In very low framerate situations, extrapolation is generally going to
either cause no benefit or make a mistake. We can safely turn it off
with no user-visible negative impact.

BUG=https://buganizer.corp.google.com/u/0/issues/24550942
TEST=Scrolled very slowly and saw mispredictions on Angler. With change,
saw the log message that the mispredictions were suppressed.

Change-Id: Ic9747d3ff098d7918047ada2ed1c2d21282c65b0

am caf1bb7f: am bd55551d: Merge "Add more methods to IPowerManager."

* commit 'caf1bb7f2bcfc9183208a360e51f9d54268265b0':
  Add more methods to IPowerManager.

am bd55551d: Merge "Add more methods to IPowerManager."

* commit 'bd55551d80832f62931ccb2d54d9a239822eeb20':
  Add more methods to IPowerManager.

Merge "Add more methods to IPowerManager."

Add more methods to IPowerManager.

Add the full list of transaction IDs from IPowerManager.aidl
to IPowerManager.h and make BpPowerManager support calling
goToSleep, reboot, shutdown, and crash. These are currently
needed by or likely to be needed by Brillo.

Bug: 22122485
Change-Id: I19abd3587c9d53b28ec150210e07f97517ee4ff4

am b450d9d8: am 77f147ff: Merge "frameworks/native: remove superfluous mention of SharedBuffer in Region.h"

* commit 'b450d9d8940f7406def0b1ccd7c139bb7203115f':
  frameworks/native: remove superfluous mention of SharedBuffer in Region.h

am 77f147ff: Merge "frameworks/native: remove superfluous mention of SharedBuffer in Region.h"

* commit '77f147ff566d350dc23c3a9807ea1262f056ff2f':
  frameworks/native: remove superfluous mention of SharedBuffer in Region.h