Merge branch 'aosp/honeycomb-LTE-release' into honeycomb-mr2-x86

Fixed bug in x86-atom fast interpreter. Adds guard for potential null pointer.
Adds guard for potential null pointer when checking for debugger active on return from method.

Change-Id: Iaef2b6815fe6c64450cc0cc36f8920e4c38f40e6

Update to allow x86-atom build for FI on Gingerbread

Also handles most TODO.txt items and contains some rewriting of the dispatch code.
Also adds a newline to the end of files that did not previously end with a newline.

Change-Id: I0cf3e77dc7bbf5ed7ec8538ef4d7d8740e296640

am a028d87f: Always grow the limit address of the a heap along with its size.

* commit 'a028d87fb310eae8cac821183aa91567ec8cb0f9':
  Always grow the limit address of the a heap along with its size.

Always grow the limit address of the a heap along with its size.

Previously, clearing the growth limit only updated the maximumSize of
the unlimited heap without updating the limit value.  This caused any
operations iterated over the extent of the heap to ignore allocations
beyond the original heap size.  With this change the limit is always
kept in agreement with the maximumSize.

Bug: 4539441
Change-Id: I08c8004e724549d196e1fd5d6ad22efc85b0c8d2

Fix a type error in the allocation of non-moving arrays.

Originally, non-moving arrays were allocated with dvmAllocObjectArray.
However, dvmAllocObjectArray does not respect the width of the user
supplied element class and substitutes a hard coded value of 4 bytes.
This change replaces dvmAllocObjectArray with dvmAllocArrayByClass.
dvmAllocArrayByClass consults the element class for its width when
computing the storage size for the array being allocated.

Bug: 4309030

am af242824: Revert "Fix a type error in the allocation of non-moving arrays." per Ryan\'s request

* commit 'af242824dcedbbe39c92b49d148e7ff50267e47f':
  Revert "Fix a type error in the allocation of non-moving arrays." per Ryan's request

Revert "Fix a type error in the allocation of non-moving arrays." per Ryan's request

This reverts commit bdd06aadab6214d2378b7d5078f92bfb14e7e3dc.

Change-Id: Ifbb4bd7941fe4f2e9d02ca2b7d77186edae4dd55

am bdd06aad: Fix a type error in the allocation of non-moving arrays.

* commit 'bdd06aadab6214d2378b7d5078f92bfb14e7e3dc':
  Fix a type error in the allocation of non-moving arrays.

Fix a type error in the allocation of non-moving arrays.

Originally, non-moving arrays were allocated with dvmAllocObjectArray.
However, dvmAllocObjectArray does not respect the width of the user
supplied element class and substitutes a hard coded value of 4 bytes.
This change replaces dvmAllocObjectArray with dvmAllocArrayByClass.
dvmAllocArrayByClass consults the element class for its width when
computing the storage size for the array being allocated.

Bug: 4309030
Change-Id: I9cfe1b62ea61f03e46f44ad2be0d1cce1bef7cdd

am 486ac3c0: Removing inaccurate and unmaintained docs.

* commit '486ac3c0a90b05be5af4b82801ef325e83241057':
  Removing inaccurate and unmaintained docs.

Removing inaccurate and unmaintained docs.

Change-Id: Ifef37063e844e4eb9ccc8111a1f1263fe26bd517

Fix build.

The build server is stuck waiting for the O_BINARY change
from system/core. This will unblock it.

Change-Id: Iedd5f7a6cb140ac07b5084948a69eb5d94e2070f

am 7027ab61: am 88041260: am dd19f5b9: Merge "Fix for a crash when GC weak references."

* commit '7027ab6198f9d27b376e467971d55f14b160e40f':
  Fix for a crash when GC weak references.

am 0c91e9d5: am 98e0347a: am 2b3d7e8e: Merge "Open dexdump files in binary mode."

* commit '0c91e9d5a4078bc1c6da2c225bde5fb703fd9d58':
  Open dexdump files in binary mode.

am 88041260: am dd19f5b9: Merge "Fix for a crash when GC weak references."

* commit '88041260251335cc497db4708f0057d5367da8b4':
  Fix for a crash when GC weak references.

am 98e0347a: am 2b3d7e8e: Merge "Open dexdump files in binary mode."

* commit '98e0347a7f016764403384567267db6e9dcf8edf':
  Open dexdump files in binary mode.

am dd19f5b9: Merge "Fix for a crash when GC weak references."

* commit 'dd19f5b91a1032e60871febb143850fd81000c5a':
  Fix for a crash when GC weak references.

am 2b3d7e8e: Merge "Open dexdump files in binary mode."

* commit '2b3d7e8eccd7cbf64b2ac41c05073a9191d0d3b3':
  Open dexdump files in binary mode.

Add notice file for dx.jar and dexdump

Change-Id: I0094d382e866b8767383516abd367c98712da594

Merge "Fix for a crash when GC weak references."

Add new comparator function.

This one compares a DexProto with a list of method parameter types,
and it is immediately useful in the new getMethod() reflection code,
to avoid doing a bunch of extra string manipulation.

While I was in the territory, I removed some comment redundancy.

Change-Id: I5049e2ac4606e44a65c3c426ac2f880a891c51f9

Merge "Optimize Class.getMethod() by loading only one method."

Optimize Class.getMethod() by loading only one method.

Change-Id: Ia2cd776c0a14914264e8d94e978d55854ed75623
http://b/3431686

Merge "Open dexdump files in binary mode."

Bump Dalvik version.

Minor increase to differentiate HC MR1 from HC.

Change-Id: I2ccd57d4952c19c2df2fe948ce60c476819450d4

Fix prelink for libdvm.

Change-Id: If91f46f80c1b1947683720e8187cecb0f7077b94
Bug: 3470019

Open dexdump files in binary mode.

Requires change Ia5c0a59e from system/core
(which defines O_BINARY globally.)

Change-Id: If814608ee9662190a7cf80aae2ad2451d88364ae

Clean up use of HAVE_ANDROID_OS

HAVE_ANDROID_OS was defined as "1" for targets, but never defined as "0"
for non-targets. Changing them to #ifdef should be safe and matches all
the other uses of HAVE_ANDROID_OS throughout the system.

Change-Id: I077aee39b08ce82327a571d9aca0cae422a0ebb5

Fix "all stubs" interpreter

The "all stubs" interpreter is the recommended starting point for
anyone creating a "fast" interpreter on a new CPU.  This cleans
up some minor bit rot in the code and documentation.

Bug 3452689

Change-Id: I6bc3b0b5da11955d842d5fc5f16cb1a2209d4a89

Merge "Make Release(JNI_COMMIT) faster"

Make Release(JNI_COMMIT) faster

The Release<primitive>ArrayElements and ReleasePrimitiveArrayCritical
calls take a "mode" argument.  If you pass in JNI_COMMIT, the storage
isn't actually released; instead, the buffer contents are flushed to
the backing storage.  If the VM has handed you a pointer to the raw
data rather than a copy, the call is a no-op.

In its present incarnation, it's a no-op that changes the thread state,
adding overhead and the possibility of suspension.  This change moves
the test for JNI_COMMIT outside the enter/exit macros.

Bug 3430203

Change-Id: I8426647ee02b2336c5c90146028fc207cfbacc3a

Remove JNI_FORCE_C

This hasn't been useful in a really long time.

Change-Id: Ieea01f5d051bc8fc671f7414af3bed626ece75f1

Merge "Added JNI hack to support JNI hack"

Added JNI hack to support JNI hack

Some of our graphics code wants to get a pointer, release it, and then
continue to use it.  The "forcecopy" mode of CheckJNI was designed
to find any code that does this.  It succeeded.

To support the behavior, we provide a JNI helper function that does
the dirty work.  It passes a magic value into the Get and Release calls
that causes "forcecopy" to skip the copy.  When forcecopy is not
enabled, the values are simply ignored.

To avoid any possibility of the function getting published in the NDK,
the function is not described in JNIHelp.h.

Bug 3409356

Change-Id: Ibd20d12ba6d3d3236ebf5760f7ccaa8c557e3774

Free some strings in error paths

Change-Id: I54629ca65c30960f930ce6886e0915e360e20237

Avoid double-free on error.

In certain circumstances on a -user build we could end up trying to
free a buffer twice.

Bug 3431477

Change-Id: I1cce1da0e32aabc85d5c5c69346224c6db49b8ee

Ensure word atomicity in System.arraycopy

The VM needs to ensure that updates to 16-bit and 32-bit fields
and array elements happen atomically.  System.arraycopy was calling
memmmove(), which in bionic happens to copy a byte at a time.

The new plan is to have bionic provide a platform-optimized memmove
variant that makes guarantees about word sizes.  If it is not present
(either because it hasn't been written for the current platform,
or we're not linking against bionic), we will use a trivial copy loop.

We assume that the memmove-by-word implementation does whatever
it needs to for best performance.  For example, if the function
determines that memcpy() is word-safe and will be faster, it can
call that.  The VM no longer makes assumptions about the internal
workings of libc functions.

I also rearranged the code to avoid making indirect calls, reading
function addresses from memory, and using integer multiplication
instructions when a simple shift will do.  (gcc now generates the
whole thing as one function.)

Also, added some primitive array copy tests to 011-array-copy.

Bug 3398352

Change-Id: I4677ee1c87998697a93e61f39a48b3f391e8e11b

am 168ae921: am e7bf56c5: Use the global maximum to compute the size of a new mspace.

* commit '168ae921fb68f2af7fbb62619232ec27bb741bea':
  Use the global maximum to compute the size of a new mspace.

am e7bf56c5: Use the global maximum to compute the size of a new mspace.

* commit 'e7bf56c584b8b2c3d60f79a27a4a2e9a6b1e2f22':
  Use the global maximum to compute the size of a new mspace.

Use the global maximum to compute the size of a new mspace.

The global maximum was used for the initial specificaion but a local
maximum was used for post-Zygote mspace creations.  This prevented a
clearing of the growth limit from having the intended effect.

Bug: 3414804
Change-Id: I47e0c1510f2212849e3b738c5578bbd7b1aff27d

Update doc with actual version number.

Honeycomb is 3.0.

Change-Id: Ic7f15a9e2345344acdc787db153f8b8c559fe15f

am d820da20: Reconcile gingerbread-plus-aosp and honeycomb

* commit 'd820da20e28c0468324125a8a2bf75c3c4c01c27':

Reconcile gingerbread-plus-aosp and honeycomb

Change-Id: Ic8c04a257aaf0ecf5348d176def5bedc6c640ee0

Mass merge from gingerbread - do not merge

Change-Id: I03cce63a718a19c27445efd2c48a7886f22f7ed1

am 81058aaf: (-s ours) Use bcopy() to move object refs around within an array. DO NOT MERGE.

* commit '81058aaf92c188cc20273edd0275329fc304aca2':
  Use bcopy() to move object refs around within an array. DO NOT MERGE.

am b46125f2: (-s ours) am 08e54499: (-s ours) am 81058aaf: Use bcopy() to move object refs around within an array. DO NOT MERGE.

* commit 'b46125f27e83da6b9844a502b11ffff48c59f729':
  Use bcopy() to move object refs around within an array. DO NOT MERGE.

am 08e54499: (-s ours) am 81058aaf: Use bcopy() to move object refs around within an array. DO NOT MERGE.

* commit '08e5449920e616cab1895eda309c20e109e5752e':
  Use bcopy() to move object refs around within an array. DO NOT MERGE.

am 81058aaf: Use bcopy() to move object refs around within an array. DO NOT MERGE.

* commit '81058aaf92c188cc20273edd0275329fc304aca2':
  Use bcopy() to move object refs around within an array. DO NOT MERGE.

am 99d4ea09: am 3207edfa: Use bcopy() to move object refs around within an array.

* commit '99d4ea09cb64768d0e1f2149f7e3af7d37172be7':
  Use bcopy() to move object refs around within an array.

am 3207edfa: Use bcopy() to move object refs around within an array.

* commit '3207edfa2de55be1207ff3e10409e7b291e33ab1':
  Use bcopy() to move object refs around within an array.

Use bcopy() to move object refs around within an array. DO NOT MERGE.

bcopy() guarantees that pointer-size words are stored atomically, and
so this prevents the gc from seeing a pointer in mid-change.

Bug: 3399673
Change-Id: Ic7d795e5d397548e9d6d8c11a585d86838cf8a44

Use bcopy() to move object refs around within an array.

bcopy() guarantees that pointer-size words are stored atomically, and
so this prevents the gc from seeing a pointer in mid-change.

Change-Id: I7ba426c5b54a3459504211d88cca1b07383168bb

am 6ad1992b: (-s ours) am 70b130f5: (-s ours) Clean up/out some comments. DO NOT MERGE.

* commit '6ad1992b311937963d44be68f3ec0255a87d82b8':
  Clean up/out some comments. DO NOT MERGE.

am f741b8e1: am 80a30431: (-s ours) Remove some pointless tests.

* commit 'f741b8e112834cb3c140ca81890dc41126649dee':
  Remove some pointless tests.

am 76675cb5: am 9b6881c2: Fix accidental transposition of partial and concurrent values.

* commit '76675cb52e5d5ddeaefd81ec83841e0c503d3411':
  Fix accidental transposition of partial and concurrent values.

am 9b6881c2: Fix accidental transposition of partial and concurrent values.

* commit '9b6881c26d58a46e6478294d83554fb9e3a85625':
  Fix accidental transposition of partial and concurrent values.

Fix accidental transposition of partial and concurrent values.

In the event of an allocation failure a full, concurrent collection was
initiated instead of a partial, non-concurrent collection.  This change
assigns the correct boolean values to that settings.

These values should be made enums to ensure this does not happen again.

Bug: 3379352

Change-Id: I9336e8f832cf02e4a74745cd475914a0c312ea4e

am a4c8f3fa: am d6b23e5c: Merge "Package garbage collection policy parameters into a single structure." into honeycomb

* commit 'a4c8f3fae538b56a003e0546b8c001d68873a650':
  Package garbage collection policy parameters into a single structure.

am d6b23e5c: Merge "Package garbage collection policy parameters into a single structure." into honeycomb

* commit 'd6b23e5ca90e011b6ede0d3592922616df3e33ec':
  Package garbage collection policy parameters into a single structure.

Merge "Package garbage collection policy parameters into a single structure." into honeycomb

Package garbage collection policy parameters into a single structure.

The garbage collection policy had been determined informally.  This change
defines the garbage collection behavior in a centralized location making it
clear how the system will behave when requesting a garbage collection in one
of the four common situations.  This change is required to allow us to change
the policy for an explicit garbage collection.

In addition, this change hoists out the thread priority toggle from the
main garbage collector routine into two simple subroutines making this
it much easier to read and spot behavior differences.

Bug: 3379352
Change-Id: If0d7d9ef337819598fb6e268f7a127f2bae23580

am 4d7dd569: Handle the case of referent clearing during tracing.

* commit '4d7dd56902150ab8db4c20a191bad112928839e9':
  Handle the case of referent clearing during tracing.

am 6159ef45: (-s ours) Parameterize instance counting to include subclasses.

* commit '6159ef4520073ae8e7ce7b7d1f7648b161a33302':
  Parameterize instance counting to include subclasses.

am ee875bb3: am 80e8d7f9: Increase default stack size

* commit 'ee875bb3c019e278500c000cdd99b34e629e42eb':
  Increase default stack size

am 80e8d7f9: Increase default stack size

* commit '80e8d7f9ca9b49437bec6224162d7bb7f42392da':
  Increase default stack size

Increase default stack size

This increases the default stack size from 12KB to 16KB.

Bug 3388513

Change-Id: I7f7c3f75b749ca2145a96434e89e6500f79973e4

am 704c48da: am a4313feb: Remove unprotected reads in the instance counting routines.

* commit '704c48daff6bd4f1e824eccb7171c08d32b86ea1':
  Remove unprotected reads in the instance counting routines.

am a4313feb: Remove unprotected reads in the instance counting routines.

* commit 'a4313feb23dd454905a3297bdc9ba745e6066a16':
  Remove unprotected reads in the instance counting routines.

Remove unprotected reads in the instance counting routines.

In the past, the instance counters would read the live bitmap, acquire
the heap lock, and then scan the live bitmap.  Reading the live bitmap
ahead of acquiring the heap lock gives the holder of the heap lock an
opportunity to invalidate the copy of the live bitmap read by the
instance counter routine.  For example, the garbage collector could
swap the mark and live bitmaps before releasing the heap lock.  This
change makes the read of the live bitmap safe by moving it into the
critical section.

Bug: 3271510
Change-Id: Ic02171e9ec3b4d24e83773199f00a18a75707427

am e1a5956b: am 0eac12e0: Merge "Replace collect with clear for the SoftReference policy." into honeycomb

* commit 'e1a5956bd9d3d7b89a09b554804c967a30c13932':
  Replace collect with clear for the SoftReference policy.

am e1034238: am b387ce8a: Remove references to android_atomic_swap.

* commit 'e10342385c9845cd82ac73c3b0d9b26c4a3dcd15':
  Remove references to android_atomic_swap.

am 0eac12e0: Merge "Replace collect with clear for the SoftReference policy." into honeycomb

* commit '0eac12e0b26c3cdfa3d16e73cc8d7ebd889c846b':
  Replace collect with clear for the SoftReference policy.

Merge "Replace collect with clear for the SoftReference policy." into honeycomb

Replace collect with clear for the SoftReference policy.

The garbage collector is invoked with a parameter that changes the
SoftReference clearing policy.  This parameter is normally false,
meaning the garbage collector will attempt to preserve some
softly-reachable referents of SoftReference instances.  When true, the
garbage collector will treat SoftReference instances as WeakReference
instances and always clearly referent fields with softly-reachable
referents.

The code refers to this action as collecting soft references which is
misleading.  The change to the garbage collector policy has an effect
on the referent field of SoftReference instances and any SoftReference
instance which is unreachable is subject collection like any other
object.  With this change we now use the description clearing soft
refernces to describe the policy.

Bug: 3381480

Change-Id: Ie179514e68d4621237b08658c1c55811a49f1210

am b387ce8a: Remove references to android_atomic_swap.

* commit 'b387ce8ae1ffa0861be1ec94ba6c313203931a1b':
  Remove references to android_atomic_swap.

Remove references to android_atomic_swap.

Bug: 3381237

Change-Id: Iabca4be285082d804b0b206025bd4ca8e008ee42

am d50993ff: am 6ff6ec6b: Fix dalvik test 084.

* commit 'd50993ff6ec834801d1442b0872de33f6d930c4d':
  Fix dalvik test 084.

am 6ff6ec6b: Fix dalvik test 084.

* commit '6ff6ec6b6361db707e4b0318075fc38dd319a49a':
  Fix dalvik test 084.

Fix dalvik test 084.

Two threads were waiting for initialization of a class to complete.
There was nothing encouraging one thread to resume execution first,
but on single-core devices it always worked out the same way.

Bug 3366697

Change-Id: I29430d398dfbd9509d7db2b7db2800d340b29d19

am 94918db6: am 84547e16: Merge "Remove logging of memory management statistics to the event log." into honeycomb

* commit '94918db6ac491d361f7d2fea63e88c67c16e8103':
  Remove logging of memory management statistics to the event log.

am 84547e16: Merge "Remove logging of memory management statistics to the event log." into honeycomb

* commit '84547e16cfef6e2f1ebe377a466e858a61b59e45':
  Remove logging of memory management statistics to the event log.

Merge "Remove logging of memory management statistics to the event log." into honeycomb

Remove logging of memory management statistics to the event log.

While the event log used to report better information than the logcat
log messages, this is no longer the case.

Change-Id: Iaf7bef5252c284c1a159bc8c0447247c5b5f47e4

am 98ce90f3: am e39e28a8: Merge "Include -XX:DisableExplicitGC in the usage output." into honeycomb

* commit '98ce90f3d007d77b8f7b65c987cc14dddb8cbfd6':
  Include -XX:DisableExplicitGC in the usage output.

am e39e28a8: Merge "Include -XX:DisableExplicitGC in the usage output." into honeycomb

* commit 'e39e28a8451222714120406d998a6d776e62bc68':
  Include -XX:DisableExplicitGC in the usage output.

Merge "Include -XX:DisableExplicitGC in the usage output." into honeycomb

Include -XX:DisableExplicitGC in the usage output.

Change-Id: I290d7b5762b616b02f59a653ac5a83388c299a93

am 9cef8218: am 836741f0: Merge "Change suspend order in GC" into honeycomb

* commit '9cef82185def11b5d712961dabc66adf56913963':
  Change suspend order in GC

am 836741f0: Merge "Change suspend order in GC" into honeycomb

* commit '836741f0984579c415f63b39efae6ce5aa695bcd':
  Change suspend order in GC

Merge "Change suspend order in GC" into honeycomb

Change suspend order in GC

The GC currently does this:

 1. acquire heapLock
 2. suspend all threads
 3. acquire heapWorkerLock

When the HeapWorker thread is suspended in #2, it might be holding
the lock we want in step #3, leading to VM deadlock.  This change
reverses the order of #2 and #3, which should allow the HeapWorker
thread to progress to a point where it releases the lock before
the GC requests the suspension.

If futexes are being unfair, the GC might have to wait a bit
longer while the HeapWorker does stuff (it might unlock, do work,
and re-lock without the scheduler giving time to the GC thread;
with the old scheme the HeapWorker would see the pending thread
suspension immediately and stop).  A better fix is planned.

Bug 3340837

Change-Id: Ib9b37c130903a2800f8f28ae61540a428dbfc5be

am 0bd1ebdb: am 461d57dc: Merge "Add an option to disable explicit GC requests." into honeycomb

* commit '0bd1ebdbeb876acfca801e337fe58ffe184edc1a':
  Add an option to disable explicit GC requests.

am 461d57dc: Merge "Add an option to disable explicit GC requests." into honeycomb

* commit '461d57dc62af82c9095a0ce3feb8fc0a4f88ddc9':
  Add an option to disable explicit GC requests.

Merge "Add an option to disable explicit GC requests." into honeycomb

Add an option to disable explicit GC requests.

Change-Id: Ibb2ca9d99fccfba61492a8059201777f59c81274

am 4b2dc337: am 6009938d: Increase LinearAlloc limit

* commit '4b2dc33744c00da2a89b075e8e631f3917f83d7f':
  Increase LinearAlloc limit

am 6009938d: Increase LinearAlloc limit

* commit '6009938d52c3ba5ae1fdf0074ab618d1166750bf':
  Increase LinearAlloc limit

Increase LinearAlloc limit

When run on framework.jar, dexopt needs about 4MB of LinearAlloc
space.  We were capped at 5MB.  Bump up the limit to 8MB to give
us some additional head room.

Bug 3362346

Change-Id: I61e06e4cd7d24d45e426026a5adddd7badc3b839

am 9622ef8d: am b2e15681: Merge "Implement growth limits to support multiple heap configurations." into honeycomb

* commit '9622ef8d66f3a2c1780166a84485e6130727fcc1':
  Implement growth limits to support multiple heap configurations.