Rename (IF_)LOGI(_IF) to (IF_)ALOGI(_IF)  DO NOT MERGE

See https://android-git.corp.google.com/g/156801

Bug: 5449033
Change-Id: Ic558031c75b3702d90eb78bd730501ae5d3c077b

Remove unsupported experimental opcodes.

External developers were starting to try to get themselves into trouble with
this stuff...

Change-Id: I2b03bfeaa8c98b6a994bc7924fc8dcf4e4d4f6cb

Merge "Remove dexmaker code from dx."

Remove dexmaker code from dx.

This now lives as a standalone project:
http://code.google.com/p/dexmaker/

Change-Id: I4f7abff9399d6827082c9af78a015562fdbcdbb3

Rename (IF_)LOGD(_IF) to (IF_)ALOGD(_IF)  DO NOT MERGE

See https://android-git.corp.google.com/g/156016

Bug: 5449033
Change-Id: Ic663376d1ad6a6cb14bf81405ad9afd247cf2f60

Introduces ProxyBuilder and tests.

- ProxyBuilder is to concrete classes what java.lang.reflect.Proxy is to
  interfaces.
- Uses a builder pattern to make specifying of the various (optional)
  parameters easier.
- Creates a concrete subclass of the supplied input class whose
  implementation delegates to the given invocation handler.
- Also provides a fix for the Code#loadConstant method to allow loading
  null values.

Change-Id: I3ca6a98b91c64466df03120bc85f095365250aca

Merge "DexGenerator: allow specifying dex cache dir."

DexGenerator: allow specifying dex cache dir.

- Switches DexGenerator to not use extended opcodes.  This allows us to
  write suitable files for earlier versions of Android.
- Allows caller to specify optimised dex directory.  This allows us to
  get around /data/dalvik-cache not being world-writeable.
- Allows caller to specify optimised dex files cache dir.  This allows
  us to skip the requirement of being able to write to /sdcard.

Change-Id: I2cae2e187ccf5b20b98763cb7eb791383a7d5a59

DexOptions should default targetApiLevel to API_NO_EXTENDED_OPCODES

Bug: 5738782

(cherry picked from commit b14b69728337cf5808eed4d5652fb9d5be33dff0)

Change-Id: I51991c85c74b41fa51556b369a3724ba9dd0c090

Merge "Don't throw an AssertionError on invalid input."

Don't throw an AssertionError on invalid input.

Throwing an AssertionError prevents tools like FindUsages from
recovering.

Bug: http://b/5470168
Change-Id: I6ae99bd29e25f868a739548266cedf85d0b26e90

Merge "Remove dx's copy of JUnit."

am 2ca72c70: Reconcile with ics-mr1-release

* commit '2ca72c704e7462c56f8c4489369b618872b595a0':

Remove dx's copy of JUnit.

Instead, use JUnit the same way other code uses JUnit: by depending
on the copy in external. There is no longer a top-level option in
dx to run JUnit tests on itself.

There are two failing tests in dx/junit-tests. Both of these are in
the recently created DexGeneratorTest; I intend to fix these in another
change.

Bug: http://b/2286423
Change-Id: I3a173a0302c7ccc81be3b5a2a4d766000d1c242b

Admit to not supporting over 65,536 methods.

This was previously a work in progress, but that work wasn't
completed and the test shouldn't assume the work is complete.

Bug: http://code.google.com/p/android/issues/detail?id=22545

Change-Id: I44a9532d96ab548283278ca3218773ea449f8318

Reconcile with ics-mr1-release

Change-Id: I61fc74707db482c4600aaa34b751a665c75ffcec

Reconcile with ics-mr1-release

Change-Id: I63967a8c63fb1d2f5af167b3a23c848829b01938

merge in ics-mr1-release history after reset to ics-mr1

am a18e6d13: Initialize the vC field of inlined getters/setters.

* commit 'a18e6d135a869f9c62f9ec8bac8b9e78d92c697f':
  Initialize the vC field of inlined getters/setters.

Initialize the vC field of inlined getters/setters.

To eliminate a benign Valgrind warning.

Change-Id: I5f0ae53b8c86fe476f0b7ddfb6589191aea464f6

am ee155d4f: Initialize the JNI indirect ref table to make Valgrind happy.

* commit 'ee155d4fe47fa751262beb43437a339fde8eabe5':
  Initialize the JNI indirect ref table to make Valgrind happy.

Initialize the JNI indirect ref table to make Valgrind happy.

Specifically, this is to eliminate the benign warning about the
'serial' field being uninitialized.

BUG: 5542417
Change-Id: I8550972e70db94f24c47c4e5dea0b59cc5840b48

Use libcorkscrew to format the stack trace.

Change-Id: I06ddbb7a8035971b6e1a164adcd00208fae89b1a

Fix DexMerger to emit dex files without extended op codes.

Certain tools in our tool chain don't like these.

Change-Id: Iaaff85c1b634f647d0105c36f403b2e03aaf0ca7

Be humane when there are more methods or fields than Dalvik can handle.

Previously our errors would look like this:
  trouble writing output: opcode == null
Or this:
  trouble writing output: No expanded opcode for 7f9c1af3

Now our errors look like this:
  trouble writing output: Too many methods: 86922; max is 65536

Bug: http://code.google.com/p/android/issues/detail?id=20814

Change-Id: I2d9dc55e188a6ac1661b74af2194b18019859a29

Can't set variable to null with loadConstant.

Change-Id: I9349507b8af2b59e03f7542b7589f2dbcdb77711

This test fails with a class not found error.

Change-Id: I7a98cbc30d32a0f19fa3892b58775ba10536c6aa

Reconcile with ics-mr1-release

Change-Id: I63807d2f883382fa765e5eefe7fd17ac3cbcbe67

am cab0fdf8: (-s ours) DO NOT MERGE Further refinement of card table clear

* commit 'cab0fdf82a668e2a1a054cd6506835046218e419':
  DO NOT MERGE Further refinement of card table clear

merge in ics-mr1-release history after reset to ics-mr1

DO NOT MERGE Further refinement of card table clear

We're currently using memset() to clear cards out to the growth
limit, which on a device configured for 48MB/128MB process caps
means we're sitting on 384KB or 1MB of pages.  We can reduce that
substantially.

This change attempts to reduce the memset() length to the currently
active portion of the card table, based on the placement of objects
in the "live objects" bitmap.  This should avoid faulting in card
pages that we don't actually need.

This is suboptimal when parts of the card table are used briefly
and then ignored for a while.  An alternative implementation, which
uses madvise(DONTNEED) aggressively, is also included (#ifdefed out).

Bug 5567332

(cherry-pick from master)

Change-Id: I6b181de20a0c8b6307acaebbc5d6dac20422f25d

Further refinement of card table clear

We're currently using memset() to clear cards out to the growth
limit, which on a device configured for 48MB/128MB process caps
means we're sitting on 384KB or 1MB of pages.  We can reduce that
substantially.

This change attempts to reduce the memset() length to the currently
active portion of the card table, based on the placement of objects
in the "live objects" bitmap.  This should avoid faulting in card
pages that we don't actually need.

This is suboptimal when parts of the card table are used briefly
and then ignored for a while.  An alternative implementation, which
uses madvise(DONTNEED) aggressively, is also included (#ifdefed out).

Bug 5567332

Change-Id: I7663ae7d15d0aaa8158deed5e331bf05333da6bb

Merge "Fix reporting of initial pause time"

Fix reporting of initial pause time

The "rootEnd" time was being captured before dvmClearCardTable()
was called.

Bug 5567332

Change-Id: I5499f1648c3e41296c18fa5d95d834c2d2853e8f

Reconcile with ics-mr1-release

Change-Id: I91ffc584e0d2d39aaec9b28ac3cde8b625234968

merge in ics-mr1-release history after reset to ics-mr1

am 6cd72bd0: Merge "Don\'t pay for filename/line number lookup unless you need to." into ics-mr1

* commit '6cd72bd0161459e21d496b927967bd0ca6fe0f94':
  Don't pay for filename/line number lookup unless you need to.

am 50e01501: Merge "Knock ::self() out of the ParseXml profile." into ics-mr1

* commit '50e015017f091118f0147d02d23140ff9f88daec':
  Knock ::self() out of the ParseXml profile.

am 4c83f214: Merge "Reduce Dalvik card table overhead" into ics-mr1

* commit '4c83f214932af008cf79aee528ca3f9e66572040':
  Reduce Dalvik card table overhead

am ccf4c9d9: Don\'t decr profiling count without incr

* commit 'ccf4c9d99fcffbc9bf65066004896745b9dcc340':
  Don't decr profiling count without incr

Merge "Slight change to unwinder API."

Merge "Don't pay for filename/line number lookup unless you need to." into ics-mr1

Merge "Knock ::self() out of the ParseXml profile." into ics-mr1

Merge "Reduce Dalvik card table overhead" into ics-mr1

Don't decr profiling count without incr

The failure cleanup code in dvmMethodTraceStart() was assuming
that profiling had already been started, which was true until my
previous change.  The code in the interpreter isn't too picky, but
updateActiveProfilers() is trying to keep an accurate acount, and
gets upset.  If you pass in a bad filename, we disable something that
was never enabled, and the VM gets confused and aborts out of spite.

Related to bug 5564440

Change-Id: I17fd8bf4654c29f49efb10142d222a91c190b6ed

Reduce Dalvik card table overhead

The VM has a "growth limit" (e.g. 48MB) that is used to cap normal
apps, and a "max heap" (e.g. 128MB) that is used for apps with unusual
high-memory requirements.  The Dalvik GC uses a 128:1 "card table"
that spans the entire managed heap, out to the "max heap" limit.

The table is erased during every concurrent GC.  The first time
we do that post-zygote, the copy-on-write behavior causes physical
pages to be allocated.  We're currently clearing out to the heap max,
not the growth limit, which means we have a bunch of physical memory
allocated for pages that will never be used.

This changes it so we only clear out to the growth limit.  If the
growth limit is removed by the app, we clear out to the heap max.
On devices with a 128MB max heap, this reduces the private/dirty
usage for most apps by 640KB.

Bug 5567332

Change-Id: Iac633017c128d63b284d48016e47d40cc46df1de

Reconcile with ics-mr1-release

Change-Id: Ibae283408ede0f3ff5a297e8d205e976455f3b95

Don't pay for filename/line number lookup unless you need to.

This wasn't a regression; the code's always been like this. But this shows up
in profiles of anything doing a lot of synchronized stuff, even when there's
no contention.

There are two awkward cases. One is that the old code used to offer a variety
of special-case messages for failures to find the current frame, save area,
and Method*. I assume this was just to help in debugging and shouldn't happen
in practice, so I don't think we've lost anything there. The other case was
that on unlocking, we used to explicitly say "unlocked". But I'm not sure we
could get into a situation where we'd be reporting contention on a monitor
that wasn't locked when we tried to lock it. So I think that's okay too.

Change-Id: Ib4401c771f717e9c8cc9c4e5346ec7a5f46a1636

Slight change to unwinder API.

Change-Id: If73f01fff9fb56ce9cba70a386d865957eaebe78

Knock ::self() out of the ParseXml profile.

New profile (>=0.6, which is where the original seems to have cut off):

47       11.0070  dvmCallJNIMethod(unsigned int const*, JValue*, Method const*, Thread*)
30        7.0258  dvmChangeStatus(Thread*, ThreadStatus)
29        6.7916  addLocalReference(Thread*, Object*)
26        6.0890  dexDecodeDebugInfo(DexFile const*, DexCode const*, char const*, unsigned int, unsigned int, int (*)(void*, unsigned int, unsigned int), void (*)(void*, unsigned short, unsigned int, unsigned int, char const*, char const*, char const*), void*)
22        5.1522  dalvik_inst
15        3.5129  lockMonitor(Thread*, Monitor*)
14        3.2787  dvmLineNumFromPC
13        3.0445  javaLangString_equals(unsigned int, unsigned int, unsigned int, unsigned int, JValue*)
13        3.0445  scanObject(Object const*, GcMarkContext*)
12        2.8103  ScopedJniThreadState::ScopedJniThreadState(_JNIEnv*)
12        2.8103  common_invokeMethodNoRange
12        2.8103  dvmDecodeIndirectRef(Thread*, _jobject*)
9         2.1077  IndirectRefTable::add(unsigned int, Object*)
9         2.1077  ReleasePrimitiveArrayCritical(_JNIEnv*, _jarray*, void*, int)
9         2.1077  markObjectNonNull(Object const*, GcMarkContext*, bool)
9         2.1077  unpinPrimitiveArray(ArrayObject*)
8         1.8735  getCodeAddrCommon(unsigned short const*, bool)
7         1.6393  dexStringByTypeIdx(DexFile const*, unsigned int)
7         1.6393  dvmHeapSourceAlloc(unsigned int)
6         1.4052  GetPrimitiveArrayCritical(_JNIEnv*, _jarray*, unsigned char*)
6         1.4052  dvmPlatformInvoke
6         1.4052  pinPrimitiveArray(ArrayObject*)
6         1.4052  readUnsignedLeb128(unsigned char const**)
6         1.4052  scanFields(Object const*, GcMarkContext*)
5         1.1710  IndirectRefTable::get(void*) const
5         1.1710  dvmFindInReferenceTable(ReferenceTable const*, Object**, Object*)
4         0.9368  common_returnFromMethod
4         0.9368  dvmAddToReferenceTable(ReferenceTable*, Object*)
4         0.9368  dvmHeapBitmapScanWalk(HeapBitmap*, void (*)(Object*, void*, void*), void*)
4         0.9368  dvmInterpret(Thread*, Method const*, JValue*)
4         0.9368  dvmLockObject
4         0.9368  dvmMalloc(unsigned int, int)
4         0.9368  findPackedSwitchIndex(unsigned short const*, int, int)
4         0.9368  readStringIdx(DexFile const*, unsigned char const**)
4         0.9368  unlockMonitor(Thread*, Monitor*)
3         0.7026  dvmSetFinalizable

Change-Id: Ic5c36859f6810413bd0b48aad1d99da7daa6e8ba

am ab97cd16: Start profiling after state prep

* commit 'ab97cd1634ed4e2c9e5c558a9f88ed9ca8623c8c':
  Start profiling after state prep

Start profiling after state prep

The updateActiveProfilers() call was happening before the profiler
state setup was completed, creating a race that could lead to a
native crash.

Bug 5564440

Change-Id: I8c030fc87f106aa8b2a67876d604a5bd79e05175

Merge "hprof-conv: Better error message when run on a 1.0.2 file."

hprof-conv: Better error message when run on a 1.0.2 file.

Change-Id: I36b14aadc11f6ba8a66543546043280da64e7605

Merge "Disable dumping native stacks of non-Dalvik threads. Bug: 5544153"

am c4102d66: (-s ours) Merge "DO NOT MERGE: Optimize IndirectRefTable." into ics-mr1

* commit 'c4102d6647043591bd89bba75454cc04d23753bd':
  DO NOT MERGE: Optimize IndirectRefTable.

Merge "DO NOT MERGE: Optimize IndirectRefTable." into ics-mr1

am 63cd4b08: Don\'t malloc between fork and exec

* commit '63cd4b08e623242066c63563bad3a9f414a2bd2d':
  Don't malloc between fork and exec

Disable dumping native stacks of non-Dalvik threads.
Bug: 5544153

We are seeing cases where dumping certain non-Dalvik threads
causes system instability.  Temporarily disable this feature.

Change-Id: I14d7907a90f152bcb15d066f8bd3fdedc578e722

Don't malloc between fork and exec

This was causing hangs in DEX file loading.

Bug 5329833

Change-Id: I19e9e0624f3823cc771c71a4b4ac177d57343c58

am 5269ef7b: am 3931f798: Fix cleanup (cherry-picked from ics-mr1)

* commit '5269ef7b87c6aa89d86e73f438363661f2f65a04':
  Fix cleanup (cherry-picked from ics-mr1)

am f51d43f0: am e3d9cde4: Disable JIT single-stepping resume mode.

* commit 'f51d43f0ebfa5739d1ee691d661097dbbad98041':
  Disable JIT single-stepping resume mode.

am a78b1255: Fix cleanup

* commit 'a78b1255ad1ce8dcacad21338d4f88db4231c4cd':
  Fix cleanup

merge in ics-mr1-release history after reset to ics-mr1

am 3931f798: Fix cleanup (cherry-picked from ics-mr1)

* commit '3931f79801b67577bf8ac12d0b3fb5b4aeaaf766':
  Fix cleanup (cherry-picked from ics-mr1)

Fix cleanup
(cherry-picked from ics-mr1)

The code was attempting to null out a struct member after freeing
the struct.

Also, changed the order of directory permission tests so that
writable comes first.  Somehow "dalvik-cache directory not
writable" seems more direct than "not readable", since the code
isn't generally interested in reading the directory.

Bug 5549907

Change-Id: If737ab822b356aae98e47292d21946e33a04342b

am e3d9cde4: Disable JIT single-stepping resume mode.

* commit 'e3d9cde40f96bf40dd4ad3d495949869bde5dd1e':
  Disable JIT single-stepping resume mode.

Disable JIT single-stepping resume mode.

BUG: 5551114
Change-Id: Id8ee75be5e71ee258994235952871ad181024e56

Fix cleanup

The code was attempting to null out a struct member after freeing
the struct.

Also, changed the order of directory permission tests so that
writable comes first.  Somehow "dalvik-cache directory not
writable" seems more direct than "not readable", since the code
isn't generally interested in reading the directory.

Bug 5549907

Change-Id: Ie412ff6c0870357b92e53606f70cb9407e04e7cf

DO NOT MERGE: Optimize IndirectRefTable.

Removed unused debugging code that maintained pointers to the
4 previous objects in each slot.

Promoted the serial number property of a slot to be first class.
This is a sufficiently cheap and useful check to perform that we
might as well always do it.

Changed get() and remove() so they check the validity of the
references at the same time as they retrieve them.  They're
also a bit smarter about checking the reference kind given
that they can assume that the table will only contain references
of one kind.  The checks are now much more consistent.
For example, remove() used to check whether an index was
stale only in the case where it was not removing the top entry
(now it always checks).

Made add() return NULL instead of dvmAbort()'ing in the case
where the table overflowed and ensure we don't blow away the
table in that cases.  This change resolves an issue with
TestIndirectRefTable.cpp which deliberately overflows the table
and expects add() to return NULL (as documented!).  As it
happens, the add() method is called in exactly 3 places in
Jni.cpp.  In each of those cases, the code was written to
handle a NULL result and only in the case of adding a weak
global reference did it not abort.  Fixed the weak global
reference case to be consistent with the others.

Changed the signature of contains() to take an Object* since
that's what we're actually looking for inside the table.

Added a couple of calls to dump() in TestIndirectRefTable.cpp
for visual inspection of its correctness.

Performance as measured by TestIndirectRefTable on same hardware.

Old implementation:
- Add/remove 100 objects FIFO order, 100000 iterations, 0.023ms / iteration
- Add/remove 100 objects LIFO order, 100000 iterations, 0.020ms / iteration
- Get 100 objects, 100000 iterations, 0.009ms / iteration

New implementation:
- Add/remove 100 objects FIFO order, 100000 iterations, 0.010ms / iteration
- Add/remove 100 objects LIFO order, 100000 iterations, 0.009ms / iteration
- Get 100 objects, 100000 iterations, 0.002ms / iteration

Cherry-pick from master.

Conflicts:

vm/IndirectRefTable.cpp

Change-Id: I157f3c1ba598137222878b8e6a5890efb744fe76

Reconcile with ics-mr1-release

Change-Id: Iaa5f9ff52792e1eff28cf9591a5efca18ac165e3

Fix single-stepping resume mode in the interpreter.

(cherry-picked from master)

When a resume attempt is cancelled due to other pending request make sure
the old native resume PC is cleared. Otherwise the JIT code cache may be
re-entered with mismatching Dalvik PC.

Also fix a code bloat problem where single-step count is not set properly
after executing a return instruction.

BUG: 5208786

Change-Id: I39db1633dcda1f464311c878ded4b5695a9b7ffd

am f4948cc1: am b1e2271b: Fix single-stepping resume mode in the interpreter.

* commit 'f4948cc1ddb41ed72e5843ffe3d4af4889e6c74c':
  Fix single-stepping resume mode in the interpreter.

am 8faec7e7: Fix single-stepping resume mode in the interpreter.

* commit '8faec7e782eca1fd6a2ad071c30b939c77c82662':
  Fix single-stepping resume mode in the interpreter.

am b1e2271b: Fix single-stepping resume mode in the interpreter.

* commit 'b1e2271b51cfb1f264b8966f1151652767259751':
  Fix single-stepping resume mode in the interpreter.

Fix single-stepping resume mode in the interpreter.

(cherry-picked from master)

When a resume attempt is cancelled due to other pending request make sure
the old native resume PC is cleared. Otherwise the JIT code cache may be
re-entered with mismatching Dalvik PC.

Also fix a code bloat problem where single-step count is not set properly
after executing a return instruction.

BUG: 5208786

Change-Id: I39db1633dcda1f464311c878ded4b5695a9b7ffd

Fix single-stepping resume mode in the interpreter.

(cherry-picked from master)

When a resume attempt is cancelled due to other pending request make sure
the old native resume PC is cleared. Otherwise the JIT code cache may be
re-entered with mismatching Dalvik PC.

Also fix a code bloat problem where single-step count is not set properly
after executing a return instruction.

BUG: 5208786

Change-Id: I54775215b11eae29ccdb6111dc0fdfa99e41e08d

Merge "Fix single-stepping resume mode in the interpreter."

Dump stacks for all native threads, not just those in JNI.

Change-Id: I276f5f448f22f8a926cdfc8c93935da687db5d9b

Fix single-stepping resume mode in the interpreter.

When a resume attempt is cancelled due to other pending request make sure
the old native resume PC is cleared. Otherwise the JIT code cache may be
re-entered with mismatching Dalvik PC.

Also fix a code bloat problem where single-step count is not set properly
after executing a return instruction.

Change-Id: Ia926bc3a87cae1296719a1069838060b181ce54d

Optimize IndirectRefTable.

Removed unused debugging code that maintained pointers to the
4 previous objects in each slot.

Promoted the serial number property of a slot to be first class.
This is a sufficiently cheap and useful check to perform that we
might as well always do it.

Changed get() and remove() so they check the validity of the
references at the same time as they retrieve them.  They're
also a bit smarter about checking the reference kind given
that they can assume that the table will only contain references
of one kind.  The checks are now much more consistent.
For example, remove() used to check whether an index was
stale only in the case where it was not removing the top entry
(now it always checks).

Made add() return NULL instead of dvmAbort()'ing in the case
where the table overflowed and ensure we don't blow away the
table in that cases.  This change resolves an issue with
TestIndirectRefTable.cpp which deliberately overflows the table
and expects add() to return NULL (as documented!).  As it
happens, the add() method is called in exactly 3 places in
Jni.cpp.  In each of those cases, the code was written to
handle a NULL result and only in the case of adding a weak
global reference did it not abort.  Fixed the weak global
reference case to be consistent with the others.

Changed the signature of contains() to take an Object* since
that's what we're actually looking for inside the table.

Added a couple of calls to dump() in TestIndirectRefTable.cpp
for visual inspection of its correctness.

Performance as measured by TestIndirectRefTable on same hardware.

Old implementation:
- Add/remove 100 objects FIFO order, 100000 iterations, 0.023ms / iteration
- Add/remove 100 objects LIFO order, 100000 iterations, 0.020ms / iteration
- Get 100 objects, 100000 iterations, 0.009ms / iteration

New implementation:
- Add/remove 100 objects FIFO order, 100000 iterations, 0.010ms / iteration
- Add/remove 100 objects LIFO order, 100000 iterations, 0.009ms / iteration
- Get 100 objects, 100000 iterations, 0.002ms / iteration

Change-Id: I1c904eb03bc8dd9822e6d9cce702c696e976974e

Add a simple performance test for IndirectRefTable.

Change-Id: Ib75354508fff1921d8d73eba5e3a3905a362b1e1

Fix reference to non-existent function in debug code.

The generated interpreter code has some debugging code
that call dvmIsHeapAddressObject, which doesn't exist.
This causes the build to fails when we build Dalvik with
DEBUG_DALVIK_VM=true.  Use dvmIsHeapAddress instead.

Change-Id: Ifc5eeb4bd2ea6b45046c606b744b1a70a5e56dfc

Rename (IF_)LOGV(_IF) to (IF_)ALOGV(_IF)  DO NOT MERGE

See https://android-git.corp.google.com/g/#/c/143865

Bug: 5449033
Change-Id: I8bd96961e369a08e86ff78b82d90f20f42787eb1

Rename (IF_)LOG() to (IF_)ALOG()  DO NOT MERGE

See https://android-git.corp.google.com/g/#/c/141576

Bug: 5449033
Change-Id: Ie3bc0d5436218ea05f98cb0373ecf5924f78db05

Dump native stack of JNI methods when handling SIGQUIT.

Change-Id: I7da7259f1350e853153ba4dea96797fc86284068

merge in ics-release history after reset to master

Merge "Provide debugging info should extended opcode lookup fail."

Provide debugging info should extended opcode lookup fail.

Bug: http://code.google.com/p/android/issues/detail?id=20814
Change-Id: Iedaf07721c29dd3d24827c3f7a953b7371c9735d

merge in ics-release history after reset to master

Avoid deadlock.

dvmIsHeapAddress is the pointer validity checker for use outside the GC. If
it doesn't take the heap lock, it risks looking at data structures that are
changing under its feet. If it does take the heap lock, it risks deadlock
between a thread doing an explicit GC and the signal catcher trying to dump
threads (causing the GC thread to be suspended while holding the heap lock).

Calling back into managed code while holding the heap lock and with other
threads resumed sounds like an inherently bad idea to me, but that's a battle
for another day.

With this change, we can handle SIGQUIT while we're in ReferenceQueue.add
as a result of an explicit GC in a system doing concurrent collections, so
the only known problem is fixed.

Bug: 5425802
Change-Id: I42d434d5ea3ffbcb77a4c544b81b08a4c7364a16

merge in ics-release history after reset to master

Merge "Leave up to 512 chars of gDvm.lastMessage on dvmAbort's stack frame."

Leave up to 512 chars of gDvm.lastMessage on dvmAbort's stack frame.

Bug: 5372634
Change-Id: I019a059e5a989de3dfc9e2f5e78f7caa7168cf75

Merge "Reordered some checks and added more to the verifier."

Merge "Revert "Use the card table to scan the immune region of the heap.""

Revert "Use the card table to scan the immune region of the heap."

This reverts commit 6c355e53332502314c3d82a7afcf898d66118f27.

Conflicts are resolved in:

vm/alloc/CardTable.cpp
vm/alloc/HeapBitmap.cpp
vm/alloc/HeapInternal.h
vm/alloc/HeapSource.cpp
vm/alloc/MarkSweep.cpp

Tested with overnight monkey runs + Delaunay.

Bug: 5037417
Change-Id: I400eff8542d0bddd865ce570a479f7750c1ba484

Merge "Don't say UTF-8 in Dalvik's CheckJNI when we mean Modified UTF-8."

Don't say UTF-8 in Dalvik's CheckJNI when we mean Modified UTF-8.

Change-Id: I4130e7f92275275a6d6f782e1ad979fbf9197a8e

Revert "GC experiment."

This reverts commit b3fb1d08137ead7b6be016e1bc147cb781e76111.