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.

am 6210c50d: am eef67346: Merge "Fix proxy expectation." into honeycomb

* commit '6210c50d31f45c28c2cddca405a4b946f7fbfb39':
  Fix proxy expectation.

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

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

am eef67346: Merge "Fix proxy expectation." into honeycomb

* commit 'eef673461bdbfbd8fde35c302ee5df5330f643a5':
  Fix proxy expectation.

Merge "Implement growth limits to support multiple heap configurations." into honeycomb

Implement growth limits to support multiple heap configurations.

When a growth limit is in effect, allocations will be limited to
number of bytes specified by the growth limit instead of the maximum
heap size.  Growth limits are specified on the command line with the
new parameter -XX:HeapGrowthLimit.  A growth limit can be removed at
runtime by calling the new clearGrowthLimit method.

This is a work around until we can adjust the maximum heap size at
runtime.

Change-Id: Ic01e32823b5ca8cf29c0948fb6cd2df10967c1fb

Merge "Fix proxy expectation." into honeycomb

Fix proxy expectation.

Change-Id: I70589302c686387a1f89e8f79eb58c4b6b8214a6

am 88b5885e: am 241cec80: Remove unused heap dumping code.

* commit '88b5885e8ba78898eadf6d201cebe97794fe578d':
  Remove unused heap dumping code.

am 241cec80: Remove unused heap dumping code.

* commit '241cec80a79551730122fb9dbc92a3527392b1de':
  Remove unused heap dumping code.

am 56750efd: am 0f403d5f: Remove some inline declarations to improve debugging.

* commit '56750efd3a5fffaef6306df89fc9a2054d5e5861':
  Remove some inline declarations to improve debugging.

am 548d70b0: am 23966773: Rename absoluteMaxSize to maximumSize.

* commit '548d70b045b434be5343fb1549a6d538c5c12ff7':
  Rename absoluteMaxSize to maximumSize.

am c5ec711d: am ac8f1125: Merge "Make the soft limited predicate look more like a predicate." into honeycomb

* commit 'c5ec711d856778b6fd108b15a0a77f68995d44af':
  Make the soft limited predicate look more like a predicate.

Remove unused heap dumping code.

Change-Id: Id8774848a870003c60be2ebcc505ce30795a4fcf

am 0f403d5f: Remove some inline declarations to improve debugging.

* commit '0f403d5f9286b42c3596004eb8015718269d9092':
  Remove some inline declarations to improve debugging.

Remove some inline declarations to improve debugging.

Change-Id: I41b044fb9ae66c1d531f4d61c73ce6fcd597102f

am 23966773: Rename absoluteMaxSize to maximumSize.

* commit '23966773259b828001634a71cafdc4fa98904bea':
  Rename absoluteMaxSize to maximumSize.

Rename absoluteMaxSize to maximumSize.

Change-Id: I3ec30b35f3689ad76a805af3b261cdaa7adbc3d7

am ac8f1125: Merge "Make the soft limited predicate look more like a predicate." into honeycomb

* commit 'ac8f112513e3481e7295eaeaf5843a55fa9b1b63':
  Make the soft limited predicate look more like a predicate.

Merge "Make the soft limited predicate look more like a predicate." into honeycomb

Make the soft limited predicate look more like a predicate.

Change-Id: I8edfafdfa8d73ebe1e3288bc0c8df2c6acbce2af

am 5918b804: am 22acd2ae: Merge "Fix JNI invocation of constructors." into honeycomb

* commit '5918b80472fe57aea34446ea25f2a8ca7a5290b0':
  Fix JNI invocation of constructors.

am 22acd2ae: Merge "Fix JNI invocation of constructors." into honeycomb

* commit '22acd2ae4b1f0f4f3aab4f55026c2e1d0d409dad':
  Fix JNI invocation of constructors.

Merge "Fix JNI invocation of constructors." into honeycomb

Fix JNI invocation of constructors.

The code that "virtualizes" method invocations was attempting to
virtualize a call to a constructor.  Constructors are direct methods,
so this resulted in a spurious CloneNotSupportedException.

Bug 3354578

(cherry-pick from dalvik-dev)

Change-Id: Ib158ab7fc6b2874eba1fc2afa45fd5436c1f2a8a

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

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

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

* commit '602b8ffb35fb555fa77081446efa598206e0c3bf':
  Clean up/out some comments. DO NOT MERGE.

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

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

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

* commit '70b130f5398c43418b6eacba2cc09f71d9febdc4':
  Clean up/out some comments. DO NOT MERGE.

Clean up/out some comments. DO NOT MERGE.

Change-Id: I16063cf9132e0f2d6556ce06e1ebfb90ecff05aa

am 80211d2b: Only generate debugging LIRs in verbose mode.

* commit '80211d2b18daa79c7b4c9b04f0cb366660c86f73':
  Only generate debugging LIRs in verbose mode.