Improve the interface for reading Dex files.

I'm planning on making this code reusable for grepping
dex files.

Change-Id: Iebf545ed6d6a4eb347ccc4a39fe40c02d75d69e4

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

* commit '480de458e3735af6868dc5546f335aec3233cd6a':
  Fix JNI invocation of constructors.

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

Merge "Fix JNI invocation of constructors." into dalvik-dev

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

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

Change-Id: Icac922593f1d872fffaf3bce98c8e24e287bf7af

Merge "Read Dex files from a byte[] rather than a RandomAccessFile." into dalvik-dev

Read Dex files from a byte[] rather than a RandomAccessFile.

This improves time to merge a trivial file with core.dex from
3.8 seconds to 0.8 seconds.

Also fixing read method names to be consistent with the spec.

Change-Id: I9033bcb497afe7a0d73e00cca14fa046b53a62bf

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

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

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

Merge "New tool merge two dex files into one." into dalvik-dev

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

* commit '45e9a9908f8874b64294dbd3e4dcfb6b76c4b6e3':
  Only generate debugging LIRs in verbose mode.

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

* commit '660095cc310535c4874e953f5eea0abc2540f655':
  Remove some pointless tests.

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 c45c71a8: am 9c6e857f: Merge "Minor cleanup of some initialization code." into honeycomb

* commit 'c45c71a82832422d8d45b6bf7f236d51d77e04f8':
  Minor cleanup of some initialization code.

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 db3c9225: am d5dae2a4: Merge "Fix invoke-interface verification failures" into honeycomb

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

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 c3be0be1: am a4f35892: Merge "Remove a stray empty line." into honeycomb

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

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

resolved conflicts for merge of ae08dca2 to dalvik-dev

Change-Id: I5f35e811bf96f54b45dea7e330a0ecf6a842365f

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

* commit '6c7aadc2e5a5b761d815ce2c79452286b62d24a6':
  Add missing length modifiers for printing size_t values.

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 ca3f39e7: am a0f4e182: Remove stale enumerators for external allocation tracking.

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

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 0111aa23: am eeb834a5: Remove enumerators for stale GC reasons.

* commit '0111aa23f186122ca874d22e619467d22be564df':
  Remove enumerators for stale GC reasons.

am eeb834a5: Remove enumerators for stale GC reasons.

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

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

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

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 7f3e6d75: am 0560c6e2: Remove the allocation limits interfaces.

* commit '7f3e6d753b190e8c57888829a9f804f9e77ce01f':
  Remove the allocation limits interfaces.

am 0560c6e2: Remove the allocation limits interfaces.

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

Remove the allocation limits interfaces.

Change-Id: I97a098fe385cc12991b8f1277683fb1146673570

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

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

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 255cbef2: am 15e4a632: Handle the case of referent clearing during tracing.

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

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 e506a20f: am 05bdc2f8: Merge "Check VMRuntime.addressOf is actually passed an array." into honeycomb

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

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 943b3a18: am b16add6a: Merge "Remove obsolete comments about small and large object heaps." into honeycomb

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

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

Merge "[JIT] Fix obscure code cache reset bug" into dalvik-dev

[JIT] Fix obscure code cache reset bug

A recent change (dalvik-dev only) introduced a bug in translation
cache reinitialization.  In real life, this would be unlikely to
ever hit as it only shows up if you use a testing mode to force
a JIT translation cache flush when the JIT is disabled.

Change-Id: If4dbf2f959ac2f6a60505a816d17caca5f6b004e

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

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

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

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

am f5949168: Move nio direct byte buffers onto the Java heap.

* commit 'f5949168d4142cde1582117173b03c614b632938':
  Move nio direct byte buffers onto the Java heap.

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

Merge-conflict cleanup.

Change-Id: I252a0b9173e130020faa3b29ba34ec39dc18bb97

New tool merge two dex files into one.

This code isn't yet ready for general use, but I have booted a device
using a core.jar from the product of a merge. In particular, I still
need to better size the output dex.

The motivation is to support incremental builds in dx. Given a
"--incremental" flag, dx would create a dex file containing only those
classes newer than the target .dex. Then it would merge that small
.dex with the current .dex. Hopefully this yields a nice speedup
over dexing everything.

It would also be possible to use this to do parallel builds. We would
manually partition the input files, dx them, and then merge the result
together.

Change-Id: I8997fb3c8bfe73b64ef38a7cb3d6456fbe799d0f

am 6159ef45: Parameterize instance counting to include subclasses.

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

am 5e261cd1: jniRegisterNativeMethods should abort on failure

* commit '5e261cd107a923ffc4d4329aa9bd19da220981c8':
  jniRegisterNativeMethods should abort on failure

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

am 02558df0: Merge "Fix JNI GetMethodID on interfaces" into honeycomb

* commit '02558df0173befced077255dafddcc45afa4a009':
  Fix JNI GetMethodID on interfaces

am 175ce8a9: Fix EABI call bridge.

* commit '175ce8a9192e5c6dfde8493231939e30e6c40eb6':
  Fix EABI call bridge.

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

Merge "Fix JNI GetMethodID on interfaces" into dalvik-dev

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

Change-Id: I4db505231501ef089cd1c406b4654e897bd77d48

Added vm support for new jumbo opcodes.

This enables jumbo opcodes by default, and they will get used by the
current build without modification. Support has been added for arm, x86,
and the portable interpreter. x86-atom support is on the TODO list. This
commit also includes a test for the new jumbo opcodes.

Change-Id: Ic3f1b41b51645861c5196f76aaf0e96e727ea537

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

Change-Id: I6f243003458e38f3afb324b505225254240a75ff

Update field offsets in InterpState per Jit-to-Interp cleanup

Change-Id: Icf4aeac2fabbd0b6cd33f4d946055a9dc7e49f49

resolved conflicts for merge of 4fa9da7e to dalvik-dev

Change-Id: I56b52104f50d2e67115227e61e4b250e1116135d

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.

am af5aa1f4: Don\'t treat dvmJitToPatchPredictedChain as a Jit-to-Interp entry point.

* commit 'af5aa1f4ce7eecc1b47a4c038cebb67d33f08f18':
  Don't treat dvmJitToPatchPredictedChain as a Jit-to-Interp entry point.

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

It is just a native callout helper function.

Change-Id: I6398b6876f5ba579b76e732107157a4c99337796

am a8589335: [Jit] Fix for 3311468 Maps crashed at handleFmt...

* commit 'a85893356ac4d86ef7d7dd18807d7bef95d7dddb':
  [Jit] Fix for 3311468 Maps crashed at handleFmt...

[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

am 9679fddf: Merge "Restore a few external allocation constants for compatibility."

* commit '9679fddfac3aad7f4f7407ae4f285d71140f6168':
  Restore a few external allocation constants for compatibility.