am f9d0ce5f: Reconcile with ics-mr1-release

* commit 'f9d0ce5f917adc46d1940425eff9d0f5b8459ff6':

Reconcile with ics-mr1-release

Change-Id: Iaa3411a4a3a03f433d2a841b3b48bb6371bd9fba

am 7c9027e2: am 1c7e1e14: Fix to dex verifier to handle float types that look like Boolean.

* commit '7c9027e23fd7096d82964961811103469d21b296':
  Fix to dex verifier to handle float types that look like Boolean.

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

am 31856263: Reconcile with ics-mr1-release

* commit '318562634a59d1c027dae121799fe6b1b9984500':

am f9568520: am 232c385b: Bug fix for control-flow construction involving nested loops.

* commit 'f9568520b2196693a41b8703fff0d2ddba31bc8e':
  Bug fix for control-flow construction involving nested loops.

Merge "Update and extend Dalvik tests"

Tweak UnsatisfiedLinkError

Change this:

  java.lang.UnsatisfiedLinkError: getSuperclass

to this:

  java.lang.UnsatisfiedLinkError: Native method not found:
    java.lang.Class.getSuperclass:()Ljava/lang/Class;

Change-Id: I23bd4350caf743ad9ba3524da0a10107c63af56a

Update and extend Dalvik tests

029 - exercise setDefaultAssertionStatus
031 - enable some tests that were disabled back when our generics
  support didn't exist.  Added new FancyClass test.
034 - print success/fail rather than matching exact exception text
038 - (same as 034)
042 - added new test for bug 5861201, currently disabled
045 - added a bunch of new tests, including one that fails
046 - added new tests, including some for bug 5861201 (the latter
  are disabled)
068 - loadClass() wants class names with dots, not slashes
086 - (same as 068)
087 - (same as 068)
092 - new test, exercises some locale-specific classes
093 - new test, exercises object serialization
094 - new test, exercises regex code

Test 045 now fails, because it's exercising a feature that should
work but doesn't.  All others pass.

Change-Id: I6481bfa88681c4ac8af629cb60797b2708bdeba4

am 1c7e1e14: Fix to dex verifier to handle float types that look like Boolean.

* commit '1c7e1e149d3dcf3949c76ae594ca9c1ca20392f9':
  Fix to dex verifier to handle float types that look like Boolean.

Fix to dex verifier to handle float types that look like Boolean.

Addresses http://code.google.com/p/android/issues/detail?id=23465, where
a Float value is potentially 0x0 or Float.MIN_VALUE (0x1), which combines
into a Boolean type, which should be allowed to be promoted to Float.

(cherry picked from commit 2a249c50903d669d2a9296f0fe407b67646626c8)

Change-Id: I1b81fad5e1e166246535bdfa4b7a98db3b2531bf

Merge "Fix to dex verifier to handle float types that look like Boolean."

Reconcile with ics-mr1-release

Change-Id: I69e3e58260f9a81452ac12d0365c8be2a17edc93

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

am 232c385b: Bug fix for control-flow construction involving nested loops.

* commit '232c385b9ece322d677a48947ab1e052c1e2427c':
  Bug fix for control-flow construction involving nested loops.

Bug fix for control-flow construction involving nested loops.

Bug: 5884080
Change-Id: Ic2f42abae3e71f9c0729984902396f92902184a3

Merge "Bug fix for control-flow construction involving nested loops."

Bug fix for control-flow construction involving nested loops.

Bug: 5884080
Change-Id: Ic2f42abae3e71f9c0729984902396f92902184a3

Remove stale assert

Delete a stale assert that can cause problems when running with
the portable interpreter when debugging or profiling.

Change-Id: I354a1fbe280e1baeb885cb51fed1f3e2c8eef159

am 9e8b05e4: am a669206d: Merge "cleanup redundant interfaces from iftable to avoid excessive LinearAlloc use"

* commit '9e8b05e4529c4f9bf66d3db21ae79ccbbe97a1dc':
  cleanup redundant interfaces from iftable to avoid excessive LinearAlloc use

Fix to dex verifier to handle float types that look like Boolean.

Addresses http://code.google.com/p/android/issues/detail?id=23465, where
a Float value is potentially 0x0 or Float.MIN_VALUE (0x1), which combines
into a Boolean type, which should be allowed to be promoted to Float.

Change-Id: I1718820dc05719772aa939f3f7830bb86a161d56

am a669206d: Merge "cleanup redundant interfaces from iftable to avoid excessive LinearAlloc use"

* commit 'a669206dc91323e664a84b2a4ca170ec0332c926':
  cleanup redundant interfaces from iftable to avoid excessive LinearAlloc use

Merge "cleanup redundant interfaces from iftable to avoid excessive LinearAlloc use"

am 516c0968: am 390d7c08: fix personality call.

* commit '516c096823b3e1c06a9875334c2ec3cdcc00d125':
  fix personality call.

am 390d7c08: fix personality call.

* commit '390d7c08821750d0633435278b2d533fd6b1a7a5':
  fix personality call.

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

fix personality call.

Change-Id: I25845344494ac15b4a6876784d2b040779892459

Fix log macro name

Convert LOGW to ALOGW.

Change-Id: I4096541cfb6f76d6ab3af289288c840f87125dfa

am ca5d7a30: am 311886c6: Prevent memory fragmentation.

* commit 'ca5d7a300ecb8a96c5751796dff62b007c7095b1':
  Prevent memory fragmentation.

am 311886c6: Prevent memory fragmentation.

* commit '311886c6c6fcd3b531531f592d56caab5e2a259c':
  Prevent memory fragmentation.

am 6dea1575: Merge "fix dvmDbgOutputAllInterfaces to include only direct super-interfaces"

* commit '6dea1575fd44aeee1dbe4d2bb184357598581652':
  fix dvmDbgOutputAllInterfaces to include only direct super-interfaces

Prevent memory fragmentation.

Bug: 5817320

Prevent memory fragmentation and potential allocation failures. This
change is temporary.

Change-Id: Id1e8f9606687648235ea9e18861125a8c799d812

Merge "fix dvmDbgOutputAllInterfaces to include only direct super-interfaces"

cleanup redundant interfaces from iftable to avoid excessive LinearAlloc use

In deep interface hierarchies super-interfaces are recursively concatenated
to create the iftable. There was no checking for duplicated entries so that the
iftable could get pretty large with just a few layers of interfaces up to the
point where the LinearAlloc was exceeded completely.

This change scans the iftable linearly for existing entries before it adds another
one.

Issue: http://code.google.com/p/android/issues/detail?id=22586
Change-Id: Idb4a13ca7a52f390661629cf2539930242526876
Signed-off-by: Johannes Rudolph <johannes.rudolph@gmail.com>

Provide detail when methods or fields exceed the limit.

Here's what the new message looks like:

trouble writing output: Too many fields: 88304; max is 65536. By package:
    10 dalvik.annotation
   278 dalvik.bytecode
    81 dalvik.system
    94 dalvik.system.profiler
   111 java.awt.font
    12 java.beans
   322 java.io
   519 java.lang
    20 java.lang.annotation
    13 java.lang.ref
    72 java.lang.reflect
    72 java.math
      ...

Change-Id: Ieea7efb178522d9ac3cb10c5a6cfb453be3fd72d

fix dvmDbgOutputAllInterfaces to include only direct super-interfaces

As the javadoc and the spec say dvmDbgOutputAllInterfaces should output only
the directly implemented interfaces. This can be achieved by just iterating
over the `clazz->interfaces`.

Issue: http://code.google.com/p/android/issues/detail?id=21422
Change-Id: I1779e2cc2ada7afaaf4a1e2e5c7861d61d9ea014
Signed-off-by: Johannes Rudolph <johannes.rudolph@gmail.com>

Fix a small memory leak.

Found by someone reading the code, rather than because we saw a crash. This is
only a small leak, and you'd have to be dumping threads (because of an ANR) or
creating a new thread to provoke it.

Change-Id: I9c660d86056765bcbebbdfa634032885c3f91b82

Rename (IF_)LOGE(_IF) to (IF_)ALOGE(_IF)  DO NOT MERGE

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

Also fix an occurrence of LOGW missed in an earlier change.

Bug: 5449033
Change-Id: I2e3b23839e6dcd09015d6402280e9300c75e3406

Rename (IF_)LOGW(_IF) to (IF_)ALOGW(_IF)  DO NOT MERGE

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

Bug: 5449033
Change-Id: Ia5d301248024df26c2a29dabdfe738e39ec87c82

Remove misleading and outdated documentation.

Change-Id: Id118f2fc173fc37225a1fb36e8182f54030b1f5f

Remove documentation that's now in docs/source.android.com.

Change-Id: If6ea32e923a8c7fe99824760d9a09e7eee1ff71b

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