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.

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

* commit 'efb37a324e037b9dfbeee7627d7b0417e1b9aed8':
  Remove some pointless tests.

Only generate debugging LIRs in verbose mode.

This should reduce memory usage and JIT time a bit.
Affected opcodes: kArmPseudoSSARep and kArmPseudoDalvikByteCodeBoundary.

Change-Id: I18ce9338b8d258270df51a66f9dc98cd2d9dd0e8

am 9c6e857f: Merge "Minor cleanup of some initialization code." into honeycomb

* commit '9c6e857fb13cad874a2f430438babe44bcddf5d0':
  Minor cleanup of some initialization code.

Merge "Minor cleanup of some initialization code." into honeycomb

Minor cleanup of some initialization code.

This change removes the "dvm" prefix from static functions and makes
the memory option processor use the potentially wider size_t type for
manipulating memory sizes instead of the unsigned int type.

Change-Id: Ia85b01d2a852080e56e4da7c9393fe4e6c2bdf00

am d5dae2a4: Merge "Fix invoke-interface verification failures" into honeycomb

* commit 'd5dae2a47d5730887408feed993d8e9006953749':
  Fix invoke-interface verification failures

Merge "Fix invoke-interface verification failures" into honeycomb

am a4f35892: Merge "Remove a stray empty line." into honeycomb

* commit 'a4f3589250f6d786654131e44291c9c19c80333c':
  Remove a stray empty line.

Merge "Remove a stray empty line." into honeycomb

Remove a stray empty line.

Change-Id: Ie954aa547b5cbfd0058407c0139e057c08ad9337

am 61ed6e5e: Merge "Fix implementation of volatile stores" into honeycomb

* commit '61ed6e5ead6361a0589fee6bd42c56e44e3fd52c':
  Fix implementation of volatile stores

Fix invoke-interface verification failures

A regression introduced in bug 3329492 was causing exceptions to be
thrown immediately (during class initialization) rather than when the
problem was actually hit.

Bug 3345814.

Change-Id: I0a6b36a226b70f7a7da4384fe46095b45f004300

Merge "Fix implementation of volatile stores" into honeycomb

am 7ed3c87b: Add missing length modifiers for printing size_t values.

* commit '7ed3c87b8481abaf339e4844a400fd29dc7e5f56':
  Add missing length modifiers for printing size_t values.

Add missing length modifiers for printing size_t values.

Change-Id: Ie82f5c4d06e8cdd6b7ce3562a4e7bb28bf2ea858

Fix implementation of volatile stores

Volatile stores are synchronization actions, not merely releasing
stores, so an additional barrier is required.  We also have the
option of using atomic operations, but that's not recommended (except
for 64-bit fields, where we don't have a choice).

Bug 3338450

Change-Id: Ibfa3486eb89af7769c81bcf892b3fee50d2028d4

am a0f4e182: Remove stale enumerators for external allocation tracking.

* commit 'a0f4e1824b95c846cd4f04e411abb0862b26b733':
  Remove stale enumerators for external allocation tracking.

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

* commit 'f741b8e112834cb3c140ca81890dc41126649dee':
  Remove some pointless tests.

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

* commit '80a30431fa6c92433d2602c159bc71dd81c62c8f':
  Remove some pointless tests.

Remove stale enumerators for external allocation tracking.

Change-Id: Iab9c718f6bdee71512c2d5996244d46f944a455f

Remove some pointless tests.

Change-Id: Ibaaf74fbdffd6c08af48cac50ef62a05e8473899

am eeb834a5: Remove enumerators for stale GC reasons.

* commit 'eeb834a53da2d70bea754233469e2ab7a7957d84':
  Remove enumerators for stale GC reasons.

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

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

Remove enumerators for stale GC reasons.

Change-Id: I297e3ce5ba75e308cc994a7303d6bfc70e819737

am 0560c6e2: Remove the allocation limits interfaces.

* commit '0560c6e2b5d3f6668f26858f86ce2cff0beffd86':
  Remove the allocation limits interfaces.

Remove the allocation limits interfaces.

Change-Id: I97a098fe385cc12991b8f1277683fb1146673570

am a05f6504: Remove support for setting the minimum size of the heap at runtime.

* commit 'a05f6504440ccf460477e9883c87cd70aca77b24':
  Remove support for setting the minimum size of the heap at runtime.

Remove support for setting the minimum size of the heap at runtime.

Change-Id: I1752c59184325b5929532b61f4fa0feaa03b8218

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

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

Handle the case of referent clearing during tracing.

Reference objects with non-null referent fields are collected during
tracing for processing after the trace has completed.  Before the
trace was made concurrent there was no way for a reference with a
non-null referent field to have its referent become null by any action
of the garbage collector after it was discovered.  Assertions were
placed in the reference processing code to check this invariant.

After the trace was made concurrent it became possible for a user to
clear the referent field of an already discovered reference.  This
violates the assertions in the reference processing code.

This change replaces the assertions in the reference processing code
with logic to deal with null referent fields.  The assert in the
SoftReference preservation code has been converted to a continue that
short circuits the preservation logic.  The assert in the white
reference clearing code short circuits the clearing and enqueueing.

Bug: 3342757
Change-Id: I967b011485e2691b2752500a3488fbcb54f129d3

am 15e4a632: Handle the case of referent clearing during tracing.

* commit '15e4a6325289d37ae9264f6c072d5071905c6929':
  Handle the case of referent clearing during tracing.

Handle the case of referent clearing during tracing.

Reference objects with non-null referent fields are collected during
tracing for processing after the trace has completed.  Before the
trace was made concurrent there was no way for a reference with a
non-null referent field to have its referent become null by any action
of the garbage collector after it was discovered.  Assertions were
placed in the reference processing code to check this invariant.

After the trace was made concurrent it became possible for a user to
clear the referent field of an already discovered reference.  This
violates the assertions in the reference processing code.

This change replaces the assertions in the reference processing code
with logic to deal with null referent fields.  The assert in the
SoftReference preservation code has been converted to a continue that
short circuits the preservation logic.  The assert in the white
reference clearing code short circuits the clearing and enqueueing.

Change-Id: I77bd18cd1ad52e9f457b0f9b1d4e1403e5de1a98

am 05bdc2f8: Merge "Check VMRuntime.addressOf is actually passed an array." into honeycomb

* commit '05bdc2f83cf5c64555c3acc94449e8baeb4d48f8':
  Check VMRuntime.addressOf is actually passed an array.

Merge "Check VMRuntime.addressOf is actually passed an array." into honeycomb

am b16add6a: Merge "Remove obsolete comments about small and large object heaps." into honeycomb

* commit 'b16add6aec5e35743f19ab540630128730263b7f':
  Remove obsolete comments about small and large object heaps.

Check VMRuntime.addressOf is actually passed an array.

Change-Id: I9571f13f397303963a99e380f9e1f8ef45fd504a

Merge "Remove obsolete comments about small and large object heaps." into honeycomb

Remove obsolete comments about small and large object heaps.

It looks like somebody intended to perform segregation of objects by
their size.  This was never implemented and the terminology is similar
to what is used by a forthcoming API.

Change-Id: Iff016781fedefd55961c5c5168569c83ecdf5258

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

* commit 'e012454580d01d6739864d303908d3699e3153e2':
  Parameterize instance counting to include subclasses.

Move nio direct byte buffers onto the Java heap.

Specifically, implement VMRuntime.newNonMovableArray and VMRuntime.addressOf.
For now these are pretty trivial because we don't have a copying collector,
but this (a) prevents code from allocating arbitrary amounts of direct byte
buffer without being charged for it, and (b) lets us move to a copying
collector in future.

Change-Id: I5de156dc8ac4ab5c997e18d447f635eb01ff08ab

am 6159ef45: Parameterize instance counting to include subclasses.

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

jniRegisterNativeMethods should abort on failure

The jniRegisterNativeMethods function currently behaves like the
RegisterNatives function that it wraps, returning 0 on success and
nonzero on failure.  However, this is an internal-only function used
by bits of infrastructure that should never fail to initialize.  By
aborting sooner we make the failure impossible to ignore.

This doesn't appear to break anything.

Bug 3323089

Change-Id: I48d1b02d90616d4c9ee01eb0e46cb61752727c5b

Merge "Fix JNI GetMethodID on interfaces" into honeycomb

Fix JNI GetMethodID on interfaces

The GetMethodID call was only searching through methods declared by
classes and superclasses.  If you passed it an interface class and
asked for a method declared in a superinterface, the call would fail.
We now have separate code for handling lookups on interfaces.

This also refactors some similar code in the interface method resolver.

Bug 3329492

(Cherry-pick from dalvik-dev)

Change-Id: Icaf744b9e75a1fd6d99f47281002cc6b3c36e368

Fix EABI call bridge.

Fix two bugs in the "slow" path, which is only exercised when calling a
native method with more than 28 words of arguments.  First, the code wasn't
skipping past the return type in the shorty signature, so the stack layout
was likely to be messed up if you had long/double args.  Second, the type
check on the value destined for r3 was broken, which would cause problems
if the method's first arg was 32-bit and the second arg was 64-bit.

Bug 3328925

(Cherry-pick from dalvik-dev)

Change-Id: I30a1972cf394dad993df574f17b8770ba9e840dd

More Jit-to-Interp entry point cleanup.

Only register entry points dispatched through [r6+#offset] in
JitToInterpEntries.

For ARM targets check the size of JitToInterpEntries explicitly to
make sure that its last entry is within 128 byte from InterpState
due to the Thumb codegen constraint.

Change-Id: I74184115cb3a3c89afc3a5fe53685671d9cb1027

Parameterize instance counting to include subclasses.

Don't treat dvmJitToPatchPredictedChain as a Jit-to-Interp entry point.

It is just a native callout helper function.

Change-Id: I6398b6876f5ba579b76e732107157a4c99337796

[Jit] Fix for 3311468 Maps crashed at handleFmt...

Change https://android-git.corp.google.com/g/#change,86452 eliminated unused
chaining cells for direct JNI calls.   However, a code path in CodegenDriver.c
assumed all similar invokes would have such cells.  Slightly re-arranged the
to avoid relying on the existance of the cell in cases in which it isn't
needed.

Change-Id: Ifc28acf559455a292b4b915ef1302085557e1d81

Merge "Restore a few external allocation constants for compatibility."

Eliminate the chaining cells for direct JNI calls.

The JNI code is dispatched through TEMPLATE_INVOKE_METHOD_NATIVE, so
having a chaining cell for a non-virtual native call simply wastes
space.

Change-Id: Iaf944f745a0723a7772ea168fb35f79e79b21fae

Restore a few external allocation constants for compatibility.

Aspects of the external allocation facility were exposed through the
VMDebug getAllocCount method.  In a previous change I removed all of
the references to external allocation from getAllocCount.  This had
the unfortunate side effect of breaking some CTS tests and causing the
VM to abort if the old constants were provided to getAllocCount on an
asserts enabled dalvik build.

The straight forward workaround seems to be to restore the special
treatment of these values in getAllocCount for as long as we support
the public interfaces of the external allocation facility.  An easier
way out may have been to make the failure case of getAllocCount return
0 instead of -1 and aborting on an asserts build.  Without some
analysis of API usage in market I would prefer to not change the -1
return value to 0 as it seems the thread counts currently return -1.

This change also eliminates the conditional export of the enum values
related to external allocation.  Those values are published API so it
makes no sense to maintain a way to guard their inclusion.

Change-Id: I49c173e0ec305536760c7aec15eebdc29213fc56

Revert "Remove inline natives for an unused performance test."

This reverts commit 7ecd89dc02ce00c425788bd4989bdb6cde9a618a.

Change-Id: I427635b7e3f7be45cfde78b8046dab3b23b64562

Update DalvikVersion due to removed NativeTestTarget

Change-Id: I891b63a676b6c3f12fdbc5858ef12f235bc1b9c1

Remove inline natives for an unused performance test.

Change-Id: I80cfb918bdf174aeb6de83909c840563f6b945dd

Simplify the interface for waiting for a concurrent GC to complete.

Previously, there was no guarantee that a GC would have completed when
the waiting routine returned.  This was an oversight, and the workaround
became to wait in a loop that tested the condition that the GC running
flag was not set.  The right fix was always to push the loop into the
waiting routine itself to keep the interface simple.  This change backs
out the hack, moving the waiting loop to where it belongs.

Change-Id: Idc6ab6e5d4379a5da27e9e115d23d4123c7cacc4

Remove the external allocation facility.

Change-Id: Iff508a9173382f29c67ca9e6eb6f65855dce0be4

Expose sysCopyFileToFile() and use it.

Moved from ZipArchive.c, with tweaks to for argument order and name. Also
tweaked a related function in ZipArchive to match.

Change-Id: I4d2e3c8b44386d87c5aa1c4565f72da87357fce8

Hook up the preexisting RawDexFile opener.

This one is similar in functionality to what's in JarFile.

I also made a couple minor (whitespace/comment/logging) changes to
JarFile where I spotted the opportunities.

This change makes it so that, if you happen to put a plain dex file on
your classpath (e.g., "dalvikvm -classpath /data/local/blort.dex
Blort"), it will actually be able to be used. This was documented as
working, but it was a lie...until now.

Change-Id: I2c93a8fdb40afa2d920904ab1235b7909e5837f8

Merge "Initialize my field."

Initialize my field.

<self-flagellation/>We can't count on malloc() to hand us zeroed
memory. I should know better by now.</self-flagellation>

Change-Id: If63c9d213982236e6e0362d03a4dd772b15bc73c

Merge "Never write an hprof record for a null root."

Never write an hprof record for a null root.

The old hprof implementation was never presented with null references
but with the new visitor based implementation null references are a
possibility.  With this change, the root visitor short circuits after
encountering a null reference and does write to the hprof file.

Change-Id: I38e8808a6ea43cb4361582ada4714f20ace7bea6

Plumbing for loading byte[]-based dex files.

This is obviously incomplete, but it's at least similarly incomplete
to the related code path it's right next to.

Change-Id: I06af6df3a4c2d648bf5b61b30a1ed5bbab4d170e

Add a new, optional "is sensitive thread?" hook.

The Android framework implements this hook in Ie6f58d130a29.

Bug: 3226270
Change-Id: I7342248e5699a22b71647e7cfeec52ec871cefaf

Fix a bug in hprof that corrupted the root set output.

The root visitor takes an Object ** but the hprof output assumed an
Object *.  As such, bad root set object ids were written to the hprof
dump and, by extension, objects reachable only from the roots either
looked like floating garbage or were wholly omitted from the output.

Change-Id: I58b31b6ad145fa9843ecbda630ed8b6ab951d931

Various improvements to dexdeps.

This includes:
* Adding an option to only produce lists of dependent classes (and not
  also fields and methods).
* Adding the ability to process multiple files on a single invocation.
* Adding a label at the head of the section for each file.
* Neatening up Output a bit by factoring "System.out" into a static field.

Change-Id: Id9a691d23afd18f82ab3790cb760cfd12e14bc86

Throw useful detail messages when Method.invoke fails.

Part of my general push for better exceptions.

(Cherry-pick from dalvik-dev of https://android-git.corp.google.com/g/83287.)

Change-Id: I0cd47b492fbdc52db2eef8e2a02f31bda2a086eb

Merge remote branch 'goog/dalvik-dev' into dalvik-dev-to-master