am db00fdd3: am 09eae10c: am 23971e39: Merge "Added a new JNI helper for logging exceptions. Plugged some JNI reference leaks in existing JNI helpers." into kraken

Merge commit 'db00fdd383809700a01b2a19386f1ca81e94c6f0' into dalvik-dev

* commit 'db00fdd383809700a01b2a19386f1ca81e94c6f0':
  Added a new JNI helper for logging exceptions.

Merge "Pin objects in the debug registry hash table." into dalvik-dev

Add a callback driven visitor routine and make verify use it.

This is the first step along the way to ending the proliferation of
one-off object walking routines.  This code should supersede similar
code in the mark sweep collector, the copying collector, the hprof
tool, etc..  Once all of its uses have been debugged the routines in
the .c file should be hoisted into the .h file and marked as inline.

Soon, the signature of the visitor will probably have to be converted
to a functor with members specialized to reference type and, in
addition, given an extra parameter for a context structure.

Change-Id: Ib00850f22374c5aca51fa3708304774da90d8bc4

Pin objects in the debug registry hash table.

Evidently, these objects are presumed to have stable addresses.

Change-Id: I20ba2adc9b78ac157ba210a8a4d5b2b7355c33fb

Merge "Check the mprotect() result when initializing the JIT code cache." into dalvik-dev

am 09eae10c: am 23971e39: Merge "Added a new JNI helper for logging exceptions. Plugged some JNI reference leaks in existing JNI helpers." into kraken

am 23971e39: Merge "Added a new JNI helper for logging exceptions. Plugged some JNI reference leaks in existing JNI helpers." into kraken

Merge "Added a new JNI helper for logging exceptions. Plugged some JNI reference leaks in existing JNI helpers." into kraken

Check the mprotect() result when initializing the JIT code cache.

Bug: 2690371
Change-Id: I48cb6498d1cdddcbf8386649d02467db5a1306e8

am 46eba1c4: am db5e758f: am 97e34d0d: am c023ae6a: am 962f896e: Add /proc/pid/task/tid/schedstat info to thread stack dumps

Merge commit '46eba1c4a6104cefe138c1c39a46275214690217' into dalvik-dev

* commit '46eba1c4a6104cefe138c1c39a46275214690217':
  Add /proc/pid/task/tid/schedstat info to thread stack dumps

am db5e758f: am 97e34d0d: am c023ae6a: am 962f896e: Add /proc/pid/task/tid/schedstat info to thread stack dumps

am 97e34d0d: am c023ae6a: am 962f896e: Add /proc/pid/task/tid/schedstat info to thread stack dumps

am c023ae6a: am 962f896e: Add /proc/pid/task/tid/schedstat info to thread stack dumps

Merge commit 'c023ae6afcdd37b26dd16be43b023a93acc90905' into kraken

* commit 'c023ae6afcdd37b26dd16be43b023a93acc90905':
  Add /proc/pid/task/tid/schedstat info to thread stack dumps

am 962f896e: Add /proc/pid/task/tid/schedstat info to thread stack dumps

Merge commit '962f896e1eeb159a6a2ac7a560708939cbb15575' into froyo-plus-aosp

* commit '962f896e1eeb159a6a2ac7a560708939cbb15575':
  Add /proc/pid/task/tid/schedstat info to thread stack dumps

Add /proc/pid/task/tid/schedstat info to thread stack dumps

This lets us see whether a usually-blocked thread is actually making
progress in between stack snapshots.

Change-Id: If191627e4572457579d5f330d31bde86b8ce4ec5

Added a new JNI helper for logging exceptions.
Plugged some JNI reference leaks in existing JNI helpers.

Change-Id: I62213cdae375235f6ee9304ecd8dc3d2b7f58c85

Include move-result into the same trace as the invoke.

In preparation for method inlining implementation. Example trace:

D/dalvikvm(  708): Compiler: Building trace for fibonacci, offset 0x10
D/dalvikvm(  708): 0x426b6fa8: 0x0016 const-wide/16 v0, (#2), (#0)
D/dalvikvm(  708): 0x426b6fac: 0x009c sub-long v0, v7, v0
D/dalvikvm(  708): 0x426b6fb0: 0x0070 invoke-direct v6, v0, v1
D/dalvikvm(  708): 0x426b6fb6: 0x000b move-result-wide v0, (#0), (#0)
D/dalvikvm(  708): TRACEINFO (3): 0x426b6f88 Lcom/android/unit_tests/PerformanceTests$FibonacciSlow;fibonacci 0x10 8 of 32, 7 blocks
D/dalvikvm(  708): 7 blocks in total
D/dalvikvm(  708): Block 0 (insn 0010 - 0010 empty)
D/dalvikvm(  708):   Fallthrough : block 1 (0010)
D/dalvikvm(  708): Block 1 (insn 0010 - 0014)
D/dalvikvm(  708):   Taken branch: block 3 (0000)
D/dalvikvm(  708):   Fallthrough : block 2 (0017)
D/dalvikvm(  708): Block 2 (insn 0017 - 0017)
D/dalvikvm(  708):   Fallthrough : block 4 (0018)
D/dalvikvm(  708): Block 3 (insn 0000 - 0000 empty)
D/dalvikvm(  708): Block 4 (insn 0018 - 0018 empty)
D/dalvikvm(  708): Block 5 (insn 0000 - 0000 empty)
D/dalvikvm(  708): Block 6 (insn 0000 - 0000 empty)

Once implemented the inliner will consume the invoke-direct and
move-result-wide instructions altogether.

Change-Id: I4e0e6283989a468d9edf01cf26f644d2d8d7ec64

merge from master

Change-Id: I99ef0c5f5fcaee5e2cea55449488bae2ce330fb9

Merge "Merge branch 'dalvik-dev'"

Merge branch 'dalvik-dev'

am ead3d06b: am 67e82fb1: Merge "Atomic/SMP update, part 2." into kraken

Added EXPORT_PC to "throw" instruction.

For bug 2700761.

Change-Id: I889e59ea35d9cadd99fc884e5b1301a4cf103f93

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

Change-Id: I0c0edb3ebf0d5e040d6bbbf60269fab0deb70ef9

Merge "Clean up warnings detected by gcc." into dalvik-dev

Clean up warnings detected by gcc.

Also re-enabled the JIT for the ARMv5te target.

Change-Id: I89fd229205e30e6ee92a4933290a7d8dca001232

Merge "Remove unused mark tracking code." into dalvik-dev

Atomic/SMP update, part 2.  (manual to dalvik-dev)

Updated "generic" quasiatomic implementation to use atomic release store
instead of atomic swap when releasing the lock.

Track change to name of memory barrier.

Change-Id: I229fc2aec8996db07b84f2d1bcfbe4346a810d85

am 67e82fb1: Merge "Atomic/SMP update, part 2." into kraken

Merge "Atomic/SMP update, part 2." into kraken

Branch-to-self is a backward branch.

Teach the verifier to identify branch-to-self as a backward branch,
which makes the instruction a GC point and worthy of a register map.

For bug 2702514.

Change-Id: Id5b26220db8008adbb2b5b7f69894a7704a0b559

am 48064d18: am b1293374: am a02fe785: am 4d24105a: am 2b469f82: Jit: Fix for  2717275 First JIT bug reported by external developers

Merge commit '48064d18355ed8845d00707c63007c1292ea69a5' into dalvik-dev

* commit '48064d18355ed8845d00707c63007c1292ea69a5':
  Jit: Fix for  2717275 First JIT bug reported by external developers

am 466ad118: am 9e38f2e4: am a85f449b: am c0ebbcd5: Merge commit \'720d1e962e248a30f81c1493081ff4c01e35c839\' into froyo-plus-aosp

Merge commit '466ad118ba758ca09f67adb9984ce7838f1baac2' into dalvik-dev

* commit '466ad118ba758ca09f67adb9984ce7838f1baac2':

am b1293374: am a02fe785: am 4d24105a: am 2b469f82: Jit: Fix for  2717275 First JIT bug reported by external developers

am 9e38f2e4: am a85f449b: am c0ebbcd5: Merge commit \'720d1e962e248a30f81c1493081ff4c01e35c839\' into froyo-plus-aosp

Merge "Remove the write permission for the JIT code cache when not needed" into dalvik-dev

Remove the write permission for the JIT code cache when not needed

To support the feature, redesigned the predicted chaining mechanism so that the
profile count is shared globally in InterpState.

Bug: 2690371
Change-Id: Ifed427e8b1fa4f6c670f19e0761e45e2d4afdbb6

Merge "Fix two dex file structural verification issues." into dalvik-dev

Fix two dex file structural verification issues.

First, structural verification failed to check for inconsistencies with
the number of arguments to methods compared to the number of registers
declared by those methods.

Second, neither the dexdump nor dexlist tools would run structural
verification.

As an added bonus, I renamed the function that performs byte swapping
and structural verification to be more descriptive about what it
does.

Bug: 2716693
Change-Id: I58794713967f5bea95010084c85efe1f929ce7d1

am a02fe785: am 4d24105a: am 2b469f82: Jit: Fix for  2717275 First JIT bug reported by external developers

am 4d24105a: am 2b469f82: Jit: Fix for  2717275 First JIT bug reported by external developers

Merge commit '4d24105a25ae285c324103843b9859c75e3f1a4c' into kraken

* commit '4d24105a25ae285c324103843b9859c75e3f1a4c':
  Jit: Fix for  2717275 First JIT bug reported by external developers

am a85f449b: am c0ebbcd5: Merge commit \'720d1e962e248a30f81c1493081ff4c01e35c839\' into froyo-plus-aosp

am 2b469f82: Jit: Fix for  2717275 First JIT bug reported by external developers

Merge commit '2b469f82df14749c67da43f073fd7a9b6b1ccd74' into froyo-plus-aosp

* commit '2b469f82df14749c67da43f073fd7a9b6b1ccd74':
  Jit: Fix for  2717275 First JIT bug reported by external developers

am c0ebbcd5: Merge commit \'720d1e962e248a30f81c1493081ff4c01e35c839\' into froyo-plus-aosp

Merge commit 'c0ebbcd5341625fbbba594d45ecffdfb83637f41' into kraken

* commit 'c0ebbcd5341625fbbba594d45ecffdfb83637f41':
  Disable SSL Session Ticket extension for OpenSSLSocket

Merge commit '720d1e962e248a30f81c1493081ff4c01e35c839' into froyo-plus-aosp

Merge "Remove unused code from the thread scanner." into dalvik-dev

Atomic/SMP update, part 2.

Updated "generic" quasiatomic implementation to use atomic release store
instead of atomic swap when releasing the lock.

Track change to name of memory barrier.

Change-Id: Ie6ec37bca6984553f665bd25385210ea6c8e0943

Remove unused mark tracking code.

Change-Id: I5164f0a357b3823db8fb3ee33da953c1bf293674

Merge "Fix three minor issues that prevent the phone from booting." into dalvik-dev

Fix three minor issues that prevent the phone from booting.

* Implement dvmHeapSourceGetLiveBits so we can pre- and post-verify
  the heap.

* Add do-nothing methods for manipulating the heap threshold.

* Use the CLZ abstraction for open code on the ARM.

Change-Id: I988a207fba4a61572a3b4896b0e254078f95c333

Remove unused code from the thread scanner.

The affected cases have been migrated to the copying collector where
they are required.  The non-moving collector does not need to visit
these roots.

Change-Id: I8b35d63cd8154b7c9f85320959559a718749976b

Jit: Fix for  2717275 First JIT bug reported by external developers

The JIT was incorrectly keeping a short value in a floating point
register rather than copying it to a core register before storing.
There was an assert to catch this case, but asserts don't fire in
production builds.

The fix is safe and simple - just exclude this case from the "optimization".

Change-Id: I33767c8a202b6fa36a19d918ac5b914a5e4e4de3

Jit: Fix for  2717275 First JIT bug reported by external developers

The JIT was incorrectly keeping a short value in a floating point
register rather than copying it to a core register before storing.
There was an assert to catch this case, but asserts don't fire in
production builds.

The fix is safe and simple - just exclude this case from the "optimization".

Change-Id: I38f1865da11ef3f78213f84967d4737e066b9c68

Remove an unused variable introduced in the last commit.

Change-Id: I699dc158b7c3a84e9ee3a3cc636f8bb122d69a79

Eliminate more unused variables and compiler warnings.

This change also introduces wrappers for condition variable operations
similar to what we have already for mutex operations.

Almost all the remaining warnings are now in the compiler or non-debug
uses of the CHECK_JIT macro.

Change-Id: I9f492f1582a06065e3a52287c7834adddfbefff9

Replace the scanObject routines with one derived from Verify.
It seems to be slightly faster this way, as well as more readable.

Change-Id: I92b9c54289c484cb4606f054f258da4a4bef8658

Merge "Check the number of slots in the Class.java definition against the number assumed by the VM, and abort if the Class has too many." into dalvik-dev

Merge "Fix dalvik/tests/062-character-encodings." into dalvik-dev

Check the number of slots in the Class.java definition against the number assumed by the VM, and abort if the Class has too many.

Change-Id: Ibcf4539ab98639c1c92e5d910c38ee1feb9c8c43

Timeout the finalizer throw test if no finalization occurs.

This test will never complete if the objects intended for finalization
are not ready to be collected.  This condition occurs with the copying
collector.  Adding the timeout forces the test to fail and allows the
test suite to make forward progress.

Change-Id: I2abb0a2530755dd32a14ecf94b24a546fd1327a9

Fix dalvik/tests/062-character-encodings.

The dalvik charset test should just check that the standard charsets are
available, as guaranteed by the java.nio.charset.Charset documentation.

Bug: 2709552
Change-Id: I7d2b6210936f01736034222ce26cdf81c47ef984

Remove unused labels, variables, and functions.  Enable warnings.

Change-Id: Icbe24eaf1ad499f28b68b6a5f05368271a0a7e86

Merge commit '720d1e962e248a30f81c1493081ff4c01e35c839' into mm

Conflicts:
libcore/x-net/src/main/native/org_apache_harmony_xnet_provider_jsse_NativeCrypto.cpp

Change-Id: I765343d83fe9bd6610f630790fbbf034467834d1

resolved conflicts for merge of fc0dcfa2 to dalvik-dev

Change-Id: I54a5b9996252a22e8dd0214f4a4ccc8914f876ea

am f00de1ba: am 6a877083: Atomic/SMP update.

am 6a877083: Atomic/SMP update.

Merge "Improvements to setThreadSelf use." into dalvik-dev

Atomic/SMP update.

Moved quasiatomic 64-bit operations in here.  They still need work.

Use the new libcutils atomic-inline header for our memory barriers.
Adjust makefiles to set SMP definition appropriately.

Change-Id: Id2ab2123724bb0baeb32f862b5568392aba35a59

Verifiy threads, stacks, and heap worker structures.

Change-Id: I33caecfc53c8399b023675b5559be8a4b0696b97

Remove unused local variables and local functions.

Change-Id: Iacb14a83f59ac20ef5e0616acbf0c6f683b947d5

Properly implement conservative scanning.

Even when precise GC is enabled the register map may be unavailable
and registers must be treated as ambiguous roots.  We now pin these
roots, along with native arguments, just prior to scavenging.

Change-Id: Ifbd799a34385a87960af28174128e2220e599c73

Improvements to setThreadSelf use.

Made all calls to assignThreadId and prepareThread aquire
threadListLock, even in initialization, where it previously relied on
being single-threaded.

Change-Id: Ibffc454c4c6fd78efa6f22b955b955cd33998a4c

Instead of aborting, just ignore tracked allocation calls.

This fixes a crash in the 070-nio-buffer dalvik test.

Change-Id: If58895498d573408ed69b6b580c16b9e75e9c291

JIT: Fix for [Issue 2675245] FRF40 monkey crash in jit-cache

The JIT's chaining mechanism suffered from a narrow window that
could result in i-cache inconsistency.  One of the forms of chaining
cell consisted of a two 16-bit thumb instruction sequence.  If a thread were
interrupted between the execution of those two instructions *and*
another thread picked that moment to convert that cell's
chained/unchained state, then bad things happen.

This CL alters the chain/unchain model somewhat to avoid this case.
Chainable chaining cells grow by 4 bytes each, and instead of rewriting
a 32-bit cell to chain/unchain, we switch between chained and unchained
state by [re]writing the first 16-bits of the cell as either a 16-bit
Thumb unconditional branch (unchained mode) or the first half of a
32-bit Thumb branch. The 2nd 16-bits of the cell will never change once
the cell moves from its inital state - thus avoiding the possibility of it
becoming inconsistent.

This adds a trivial execution penalty on the slow path, but will add
about a kByte of memory usage to a typical process.

Change-Id: Id8b99802e11386cfbab23da6abae10e2d9fc4065

Disable SSL Session Ticket extension for OpenSSLSocket

Due to compatability issues with some sites, disable this SSL extension which wasn't present in Eclair. See also:

    b/2682876 Some ssl sites cause "A secure connection could not be established" error

Change-Id: Ib42fb249eb0f8d645ae3f983c77cb278ca54f946

Merge "Added a quick test." into dalvik-dev

Added a quick test.

Somebody on android-developers was having trouble with
Method.isAnnotationPresent().  This adds a pair of calls as a quick
sanity check.

Change-Id: I4907d7b1dbf9593be53c25149e1e6ec29cd4b222

Merge "Add counters to track JIT inline cache hit rate and code cache patch counts." into dalvik-dev

Add counters to track JIT inline cache hit rate and code cache patch counts.

Also did some WITH_JIT_TUNING cleanup.

Change-Id: I8bb2d681a06b0f2af1f976a007326825a88cea38

Resolve a few errors with the new collector and the dalvik tests.

* First, Scavenge class object instance fields.  This fixes a crash in
  the 067-preemptive-unpark test.

* Assert that allocations to back the transport of gray objects
  succeeds.  Fragmentation caused by pinning can cause a blow-up in
  the size of to-space.  Until we can perform emergency promotions
  this leads to more diagnosable test failures.

* Zero dvmHeapSweepUnmarkObjects's out parameters.  This silences some
  valgrind errors.

Change-Id: Iafbd915f2020418d584477b58c512b9ff4637213

Add a few global root verifiers to the common verification code.

Change-Id: I32427bc10b08b35e9be8851bfa9ef52408a07708

Plug leak in class initialization.

Don't add static fields and methods to the resolved-items table if the
defining class is still being initialized.  If we do, other threads can
then access those fields and methods without first performing an "is the
class initialized" test.

We will end up performing static field and method resolution more often
now than before.  On a Nexus One, zygote class preload time went from
2227ms to 2239ms (i.e. the performance difference is in the noise).

Bug 2655384.

(Re-applying this after the JIT fix.)

Change-Id: I14846865313730f705b819c5893456e342316e2e

Two fixes to referencing processing.

First, ensure the transporter allocates from blocks on the scavenger
work queue.  Previously, the transporter could have objects allocated
in the free space of dequeued to-space blocks.  When this condition
occurs we unconditionally enqueue the block for re-scavenging.

Second, finish the implementation of large heap table scavenging and
add a verification routine for this root.

Change-Id: I5a9caf4b2824673a1f532c4e215c53f64a6a4627

am 2725c3a4: am 7b30673d: am e54a4c58: am ff11012d: am f3377c64: Merge "Remove test runner tool vogar from Froyo." into froyo

Merge commit '2725c3a4d7a33d2c8d9723079a7ec7522ceaf628' into dalvik-dev

* commit '2725c3a4d7a33d2c8d9723079a7ec7522ceaf628':
  Remove test runner tool vogar from Froyo.

am 447fe997: am 9b0df189: am 46a56d05: am 677fead0: Merge commit \'264952afbf3986a2af4c2fa6b8daaed04f34fee8\' into froyo-plus-aosp

Merge commit '447fe9973744f685fdd83d14ba2103d2a3e13553' into dalvik-dev

* commit '447fe9973744f685fdd83d14ba2103d2a3e13553':
  b/2663512: "java.io.IOException: SSL shutdown failed.: Ok" shouldn't dump a stacktrace

am 7b30673d: am e54a4c58: am ff11012d: am f3377c64: Merge "Remove test runner tool vogar from Froyo." into froyo

am e54a4c58: am ff11012d: am f3377c64: Merge "Remove test runner tool vogar from Froyo." into froyo

am 9b0df189: am 46a56d05: am 677fead0: Merge commit \'264952afbf3986a2af4c2fa6b8daaed04f34fee8\' into froyo-plus-aosp

am ff11012d: am f3377c64: Merge "Remove test runner tool vogar from Froyo." into froyo

Merge commit 'ff11012d99f01ba9c0b9c66f7ae089df8d93f813' into kraken

* commit 'ff11012d99f01ba9c0b9c66f7ae089df8d93f813':
  Remove test runner tool vogar from Froyo.

am 46a56d05: am 677fead0: Merge commit \'264952afbf3986a2af4c2fa6b8daaed04f34fee8\' into froyo-plus-aosp

am 677fead0: Merge commit \'264952afbf3986a2af4c2fa6b8daaed04f34fee8\' into froyo-plus-aosp

Merge commit '677fead05d1df2457c1f9a92d193b809b91319c0' into kraken

* commit '677fead05d1df2457c1f9a92d193b809b91319c0':
  b/2663512: "java.io.IOException: SSL shutdown failed.: Ok" shouldn't dump a stacktrace

am f3377c64: Merge "Remove test runner tool vogar from Froyo." into froyo

Merge commit 'f3377c64212d5ebda0799d42659bde3e8f488b29' into froyo-plus-aosp

* commit 'f3377c64212d5ebda0799d42659bde3e8f488b29':
  Remove test runner tool vogar from Froyo.

Merge commit '264952afbf3986a2af4c2fa6b8daaed04f34fee8' into froyo-plus-aosp

Add the dummy CHECK_JIT and ABORT_JIT_TSELECT macros even when JIT is off.

Change-Id: I5d9886dcb8359dd4336d83b2a2bddad383f5cc36

Merge "Abandon a JIT trace if it contains static fields that are not fully initialized" into dalvik-dev

Correct and add additional LOGV statements.

Change-Id: I28db8a3b4c0bb9eb25a075f46b89280c7f8f466c

Abandon a JIT trace if it contains static fields that are not fully initialized

Also turn some asserts into aborts to capture future contract changes between
the VM and the JIT.

Bug: 2655384
Change-Id: I8bb0226c7ae26fedf6b4ad25a1cd1aa7013b60d4

Merge "Remove test runner tool vogar from Froyo." into froyo

Remove test runner tool vogar from Froyo.

This tool has been open sourced independently and it will be quite
awkward for external contributors who attempt to use or contribute
to this dead copy. This code is test only and not used by the
production runtime.
http://code.google.com/p/vogar/

Change-Id: Ib53bdfdb600330e1a2f200b5747305fc527a78a6

Rationalize logging.

Replacing stray uses of LOGI them with macros specific to functional
aspects of the collector.  Replace (void *) cast for with (void) cast
in the expansion of unused macros.  While these are both no-ops, the
latter cast does not emit a warning message.  Lastly, eliminate some
logging statements I never found useful.  More logging statements need
to be culled.

Change-Id: Id5603c3a816c016a91e00e296ff5c16dddfe7e1a

Merge "Implement reference processing for the copying collector." into dalvik-dev