Merge "Update tests for new build target"

Update tests for new build target

Change-Id: Ia1740d1b2c0b611c4559c5c846b12fb5c3e81b07

Merge "Return back-end result from eglDestroyImageKHR"

Merge "libagl: Transform the vertex if using eye space lighting with point lights"

Merge "Allow disable of dumpstate vibrate"

Allow disable of dumpstate vibrate

Change-Id: I747b757f4b5e2d6a472b7b2a19f8c1ca8a4b7fdd

Return back-end result from eglDestroyImageKHR

Change-Id: I0e972b778f9802c28f52092bb9af087285833e0b

Merge "Forward compatibility patch."

Forward compatibility patch.

Change-Id: I8e8af0c6035aaac5e5097f1cfb198250475627ee

Merge "Fixed clang build error for libgui"

Fixed clang build error for libgui

Fixed the order of the statements in ANDROID_SINGLETON_STATIC_INSTANCE
macro so that the templated static member variable initialization
comes before the instantiation of the Singleton class. This
fixes the clang compile error.

Change-Id: Ic47d17e152b657f2dff3191ccc3770753fdf002b
Author: Tareq A. Siraj <tareq.a.siraj@intel.com>
Reviewed-by: Edwin Vane <edwin.vane@intel.com>

Merge "Fix error trap in SurfaceTexture Client"

Merge "EGL: do not use sparse files for shader"

EGL: do not use sparse files for shader

- Process is killed by system with SIGBUS signal if it writes
  data to mapped sparse file on full filesystem.
- Allocate space using write() function instead of ftruncate()
  to avoid creation of sparse files on full filesystem.
  Catch write() errors to handle out-of-space case during allocation.

Bug: http://code.google.com/p/android/issues/detail?id=35376
Change-Id: Ifc366454f34e71a43a0973eda4f591a920ea3a14
Signed-off-by: Kirill Artamonov <kartamonov@nvidia.com>

libagl: Transform the vertex if using eye space lighting with point lights

This fixes lighting when using point lights, when eye space
lighting is used (which is the default).

Change-Id: I0cd0d2329893d6b5f8af3b1e595274c2076fc322

Fix error trap in SurfaceTexture Client

There was an issue in Surface::lock where failure to lock a surface
resulted in two bad things happening:
- success was returned to the caller (it was apparently locked).
- an uninitialised pointer was returned as the buffer.

Change-Id: I8b0df81400e0fa0542a8bb993d76923ac96b686e

Fix shutdown sequence to avoid SIGSEGV when running am command

When the app_process is shutting down the main thread will close the
binder fd while pool threads are executing an ioctl (in
IPCThreadState::stopProcess called by AppRuntime::onStarted in
app_main.c).

The binder driver will then return all pending calls in ioctl
without any error and with a command. One of the threads gets a
BR_SPAWN_LOOPER which will create a new thread (the other thread
gets a BR_NOOP). This new thread then calls
vm->AttachCurrentThread. Usually this results in a log entry with
"AndroidRuntime: NOTE: attach of thread 'Binder Thread #3' failed",
but sometimes it also causes a SIGSEGV. This depends on the timing
between the new thread an the main thread that calls DestroyJavaVM
(in AndroidRuntime::start).

If IPCThreadState.cpp is compiled with "#define LOG_NDEBUG 0" the
pool thread will loop and hit the
ALOG_ASSERT(mProcess->mDriverFD >= 0) in
IPCThreadState::talkWithDriver.

Crashes like this has been seen when running the am command and
other commands that use the app_process.

This fix makes sure that any command that is received when the driver
fd is closed are ignored and IPCThreadState::talkWithDriver instead
returns an error which will cause the pool thread to exit and detach
itself from the vm. A check to avoid calling ioctl to a fd with -1
was also added in IPCThreadState::threadDestructor.

Another solution might be to change the binder driver so that it
returns an error when the fd is closed (or atleast not a
BR_SPAWN_LOOPER command). It might also be possible to call exit(0)
which is done when System.exit(0) is called from java.

Change-Id: I3d1f0ff64896c44be2a5994b3a90f7a06d27f429

Reconcile with jb-release

Change-Id: Ic18750e72ccec1b5aad558da8959c9c4031b27f8

merge in jb-release history after reset to jb-dev

am a67e418e: Exit boot animation cleanly.

* commit 'a67e418e1fda219f6cc0a7e420bcf5cc4f9fe710':
  Exit boot animation cleanly.

Exit boot animation cleanly.

The desc.txt file can now mark parts as 'must finish cleanly' by using
'c' as the part line prefix rather than 'p'.  If so indicated, if the
bootanimation is asked to quit it will do so only after waiting to
finish that part.

I considered either making init.c service killing smarter or promoting
bootanim to be a bindable service with a requestExit method.  However,
these changes are probably too big/risky given our ship date.  So
I used a property as a mailbox between SurfaceFlinger and bootanim.

Bug: 6679877
Change-Id: Id7dca22caa50b450fff25ca94f7242d971034f41

Reconcile with jb-release

Change-Id: I6a74f5247794a5dddd6fbb75b2edbaba06674ab7

am 7c24b1d4: Merge "SF could get stuck waiting for vsync when turning the screen off" into jb-dev

* commit '7c24b1d4da5e13329d2105cb2f8285715d920787':
  SF could get stuck waiting for vsync when turning the screen off

merge in jb-release history after reset to jb-dev

SF could get stuck waiting for vsync when turning the screen off

When turning the screen off we could have 2 waiters on the
vsync condition: The main vsync waiter as well as one in
onScreenReleased(). We were only signaling the condition though,
so it it would be possible to wake onScreenReleased() without waking
the main vsync thread which would then be stuck in .wait().

We fix this by just using broadcast() when receiving a vsync event.

We also add a broadcast() to signal when the state of
mUseSoftwareVSync changes.  This is important particularly for
the transition from hardware to software vsync because the main
vsync waiter might have observed mUseSoftwareVSync == false
and decided to block indefinitely pending a hardware vsync
signal that will never arrive.

Removed a potentially deadlocking wait for a signal in
onScreenReleased().  The function was trying to wait for the last
vsync event from the hardware to be delivered to clients but there
was no guarantee that another thread would signal it to wake up
again afterwards.  (As far as I can tell, the only other other
thread that might wake it up at this point would be a client
application issuing a vsync request.)  We don't really need to wait
here anyhow.  It's enough to set the mUseSoftwareVSync flag,
wake up the thread loop and go.  If there was a pending vsync
timestamp from the hardware, then the thread loop will grab
it and use it then start software vsync on the next iteration.

Bug: 6672102
Change-Id: I7c6abc23bb021d1dfc94f101bd3ce18e3a81a73e

Merge "SF could get stuck waiting for vsync when turning the screen off" into jb-dev

SF could get stuck waiting for vsync when turning the screen off

When turning the screen off we could have 2 waiters on the
vsync condition: The main vsync waiter as well as one in
onScreenReleased(). We were only signaling the condition though,
so it it would be possible to wake onScreenReleased() without waking
the main vsync thread which would then be stuck in .wait().

We fix this by just using broadcast() when receiving a vsync event.

We also add a broadcast() to signal when the state of
mUseSoftwareVSync changes.  This is important particularly for
the transition from hardware to software vsync because the main
vsync waiter might have observed mUseSoftwareVSync == false
and decided to block indefinitely pending a hardware vsync
signal that will never arrive.

Removed a potentially deadlocking wait for a signal in
onScreenReleased().  The function was trying to wait for the last
vsync event from the hardware to be delivered to clients but there
was no guarantee that another thread would signal it to wake up
again afterwards.  (As far as I can tell, the only other other
thread that might wake it up at this point would be a client
application issuing a vsync request.)  We don't really need to wait
here anyhow.  It's enough to set the mUseSoftwareVSync flag,
wake up the thread loop and go.  If there was a pending vsync
timestamp from the hardware, then the thread loop will grab
it and use it then start software vsync on the next iteration.

Bug: 6672102
Change-Id: I7c6abc23bb021d1dfc94f101bd3ce18e3a81a73e

am 16272efb: Add ASSIST keycode.

* commit '16272efb7af0692266fecdc53b2c6d995bf397b7':
  Add ASSIST keycode.

Add ASSIST keycode.

Bug: 6594275
Change-Id: I032b055207d16bfff93ee8a350c0dc52b9102926

am 2d15fcab: Merge "reduce PB size from 2MB to 512KB" into jb-dev

* commit '2d15fcab3e3588cfddb6c7b180faecd3eccce2e5':
  reduce PB size from 2MB to 512KB

merge in jb-release history after reset to jb-dev

Merge "reduce PB size from 2MB to 512KB" into jb-dev

Reconcile with jb-release

Change-Id: Iac1c88f2833e67e52af2bc22a17fa93d30cb44cc

merge in jb-release history after reset to jb-dev

am 9be7caf3: Merge "Include stack traces for certain native processes in bugreport." into jb-dev

* commit '9be7caf380b0e2fb29b8813fb8762c2ae417ce95':
  Include stack traces for certain native processes in bugreport.

Merge "Include stack traces for certain native processes in bugreport." into jb-dev

am 49b97e20: I am having second thoughts about 512m for the large heap size.

* commit '49b97e20f17af0ba98cdece2b7b93ab0a2199af4':
  I am having second thoughts about 512m for the large heap size.

I am having second thoughts about 512m for the large heap size.

Let's go with 384 megs, half way between the large heap size on
Xoom and 512.

Change-Id: I4a7f2e5a8b2920b49fa53777725e24811145f5f2

Include stack traces for certain native processes in bugreport.

Bug: 6615693
Change-Id: I64c3b3ce0bba62d9c332a795f7d979fb753dc27b

Aligning native Parcel implementation to Java.

The Java implementation of writing the RPC response header
calculates the length of the header including the 4 bytes
specifying the header length but the native implementation
excludes the 4 bytes specifying the length from the header
length.
The native implementation has been aligned to the Java impl.

Change-Id: I325bf272a63152d8fded4cf4e51a906b5a9bfe19

merge in jb-release history after reset to jb-dev

am 0cd545f1: sometimes we would incorrectly scale the content of a surface

* commit '0cd545f14261d829513e0d6e8fa5e4e4f3372b3d':
  sometimes we would incorrectly scale the content of a surface

sometimes we would incorrectly scale the content of a surface

this would happen when a resize was pending (ie: we have received
and processed a resize transaction but have not received a buffer
with the right size) and a new transaction came in that didn't
involve a resize, for instance a translate-only transaction.

in this case, we would incorrectly update the drawing state
with the pending size, eventhough we still don't have a buffer
for it.

the solution is quite simple, we never allow the size to propagate
from current to drawing state during the regular transaction processing
(unless we are in fixed-size mode -- meaning we don't need to have
a matching size buffer), this propagation happens later once we
receive the buffer.

Bug: 6624163
Change-Id: I11a97e4b88a7f3a0571ddcfe99c86cb04ce01a4d

Reconcile with jb-release

Change-Id: I73f561b3fb9c76aa30ff7de8eab378a1ba5963cc

merge in jb-release history after reset to jb-dev

am ba43e0a1: Merge "SurfaceFlinger: remove all GLES scissor calls." into jb-dev

* commit 'ba43e0a1faee9629ca2d0beb53dd6c44bb9bfd05':
  SurfaceFlinger: remove all GLES scissor calls.

Merge "SurfaceFlinger: remove all GLES scissor calls." into jb-dev

am be502a02: Add new Dalvik memory limit definition.

* commit 'be502a02c8e0ea232e7339ed60b1754c929ecec1':
  Add new Dalvik memory limit definition.

reduce PB size from 2MB to 512KB

this allows us to enable h/w acceleration on low-end
devices while keeping memory usage down.

Bug: 6557760
Change-Id: I8af2de3038dc2579360b8b73aa452cb7a0e506a9

SurfaceFlinger: remove all GLES scissor calls.

Bug: 6576505
Change-Id: I494b7627f2e271a234706bf49a9490f8ac56c77a

Add new Dalvik memory limit definition.

This is for a 7in hdpi/tvdpi tablet with 1G of RAM.

That sounds kind-of familiar.  I don't know.  Have I seen
such a thing before?  Maybe.

Bug: 6576049
Change-Id: Iabc245692d5106feec9199eb2b5a3d06e27a9b83

merge in jb-release history after reset to jb-dev

am 4824d40a: sometimes SF would not process a surface resize

* commit '4824d40a35333182c2eb3593511b9bcbecd0a943':
  sometimes SF would not process a surface resize

sometimes SF would not process a surface resize

this would happen when a window started with size A, was
resized to B and immediately resized to A. In this situation
the erquested and active size would be the same, and SF
would think a transaction wasn't needed.

we fix this by always comparing the requested sizes.

Also, make sure to set mRefreshPending once we're sure
we have succesfully called updateTexImage().

Bug: 6580962
Change-Id: I2c48b4df7f05fd35c9e1d2dd82095b0f3d5a0b6a

merge in jb-release history after reset to jb-dev

am 4467bba7: Merge "Support looper callbacks based on smart pointers." into jb-dev

* commit '4467bba73a91161da01d5d969cf7ba3e2309d989':
  Support looper callbacks based on smart pointers.

am dad23789: Merge "Delete unused poll() code." into jb-dev

* commit 'dad2378911a244607afa3899928c429b340031cb':
  Delete unused poll() code.

am 9e2e781a: Merge "Remove unused statistics code." into jb-dev

* commit '9e2e781acaead54d0fb095d55a1c44b32563248f':
  Remove unused statistics code.

Merge "Support looper callbacks based on smart pointers." into jb-dev

Merge "Delete unused poll() code." into jb-dev

Merge "Remove unused statistics code." into jb-dev

Support looper callbacks based on smart pointers.

Bug: 6559630
Change-Id: I5a667f219f431838638acefbc9fa6afa610971bd

Delete unused poll() code.

We don't need this code anymore and it is just in the way.

Bug: 6559630
Change-Id: I1dc9decf85d5ea1feab159c2985da6c20baffdd5

Remove unused statistics code.

Bug: 6559630
Change-Id: Iacdf4bb4c1c125c09305cbd8cb443c7c80cfc010

Reconcile with jb-release

Change-Id: If0a430615dadb425b82aa27204e6c670f06ee099

merge in jb-release history after reset to jb-dev

am e31564d8: Fix a crasher is surfaceflinger.

* commit 'e31564d8eb0ab67e167a888eccce20f5b4e4ef45':
  Fix a crasher is surfaceflinger.

Fix a crasher is surfaceflinger.

this bug introduced recently would happen when the very first
buffer of a surface was rejected for not having the right size

Bug: 6577035
Change-Id: I9fabf20006019f2a6c308be7c7f5c05bdcfd5014

am 2c8207e9: add the ability to reject buffers in SurfaceTexture::updateTexImage

* commit '2c8207e9627fe6c7a90e31fae8d71ae49df56845':
  add the ability to reject buffers in SurfaceTexture::updateTexImage

am 702634a4: refactoring in preparation for bug:6498869 fix

* commit '702634a4dad85cfc292618ac91eda6c00f42b7c5':
  refactoring in preparation for bug:6498869 fix

am 05cec9d1: improve resize transactions

* commit '05cec9d1275fd939c2d1aec235dca2bdb8edef63':
  improve resize transactions

am 419e196e: Improve debug logs and minor clean-up

* commit '419e196e639c8adb875da2765abcef95017b6d4a':
  Improve debug logs and minor clean-up

Reconcile with nakasi-factoryrom-release

Change-Id: Ib47a2f4001bbf5ec4a2b889a16e41a1259eb6e1a

merge in nakasi-factoryrom-release history after reset to jb-dev

Reconcile with jb-release nakasi-factoryrom-release

Change-Id: I3bc06420f32fe0ce50b6943fe33eca293f375d05

add the ability to reject buffers in SurfaceTexture::updateTexImage

SurfaceFlinger is using this new feature to reject buffers that
don't have the right size.

Bug: 6498869
Change-Id: I8a7250a47db6c082a357b703feb3b9d0fc8d3443

refactoring in preparation for bug:6498869 fix

here we just shuffle a bit the code inside lockPageFlip(),
we move the code that handles the buffer size closer to the call
to updateTexImage(). no functionality change.

Change-Id: Ie3193cd86cd32cf3c89532449fd747d145ca0ab6

improve resize transactions

use a flag instead of pre-committing the "requested" state
to prevent propagation or "requested" to "active", which makes
things a lot clearer when reading the code.

also avoid going through the "resized" code-path when requested
size is equal to "active" size.

Bug: 6498869
Change-Id: I24f893ba0ec5ca06aac5b8da9818989ae7ce4005

Improve debug logs and minor clean-up

Bug: 6498869
Change-Id: I14d1b4d6960b87b5a7c4d7e20b92538edd9331ff

merge in jb-release history after reset to jb-dev

am e1082496: Enable SurfaceTexture fence sync for the emulator

* commit 'e10824965c64663b8878b808206ee382c7ab6af8':
  Enable SurfaceTexture fence sync for the emulator

Enable SurfaceTexture fence sync for the emulator

The host-accelerated GL ES driver doesn't do the implicit
synchronization required when fence sync isn't used.

Bug: 6515813
Change-Id: I6a667f2db6b519f3557b5abda78775f767841dae

am 83e7c8c4: Implement the EGL_KHR_fence_sync in libagl

* commit '83e7c8c431ec8b578b6f778811837632e3798768':
  Implement the EGL_KHR_fence_sync in libagl

merge in nakasi-factoryrom-release history after reset to jb-dev

Reconcile with jb-release

Change-Id: I37477f9871208d227b42937c5aa7d27129bdecb1

Implement the EGL_KHR_fence_sync in libagl

Implementing this in libagl allows us to start using it for
SurfaceTexture in emulator builds, which is necessary to avoid
corruption in the Browser when using the host-accelerated GL path.

Bug: 6515813
Change-Id: Icafba8687cb5d010d8d42b3866b298d2be984fc9

merge in jb-release history after reset to jb-dev

am cc2b1560: fix EGL_CLIENT_STRING

* commit 'cc2b1560e87369676a2d13f17bd1ff4021a91819':
  fix EGL_CLIENT_STRING

fix EGL_CLIENT_STRING

it should read OpenGL_ES (with an underscore)

Bug: 6529643

Change-Id: I7caf07793bffe5a8dcceaff496a222077486fc87

merge in jb-release history after reset to jb-dev

Reconcile with jb-release nakasi-factoryrom-release

Change-Id: I42366ee22738eac90627669da9bb03ec8b598724

am 7e477bfe: make sure BufferQueue releases newly acquired buffers on failures

* commit '7e477bfe11f8bb31622353b64e85721df3cf9702':
  make sure BufferQueue releases newly acquired buffers on failures

make sure BufferQueue releases newly acquired buffers on failures

this prevents SurfaceTexture producer to hang in dequeueBuffer()
when something goes wrong in the consumer.
only the consumer gets an error and the current frame is kept
instead of the new one. the producer is unaware of the problem.

Bug: 6476587
Change-Id: Ie6db5526632aabc3e60229b93dfe29c19491ade4

am f899e411: Remove misleading and wrong ALOGW

* commit 'f899e4113a5b5545cd91b4625514d0ab0d33751c':
  Remove misleading and wrong ALOGW

am ed9807bd: we need to wait for vsync when doing the screen-off animation

* commit 'ed9807bd7dbd4863841e251138392c54755eb394':
  we need to wait for vsync when doing the screen-off animation

Remove misleading and wrong ALOGW

Bug: 6476587
Change-Id: I4345f1100db02786bb50ad83ca7b559cad301706

we need to wait for vsync when doing the screen-off animation

Bug: 6511421
Change-Id: I7a85a55e66a3a8d9937df575e98a5efec01a634f

am a66faaa6: Merge "Dump packages.xml as root." into jb-dev

* commit 'a66faaa65a235e24e78f75c79cc3bd209b67e1f8':
  Dump packages.xml as root.

Merge "Dump packages.xml as root." into jb-dev

merge in nakasi-factoryrom-release history after reset to jb-dev