Merge "Log when the native trace is unavailable"

Log when the native trace is unavailable

This should allow us to differentiate between "couldn't get the
stack" and "didn't try to get the stack".  Also show the thread's
state (e.g. 'R' for running, 'D' for uninterruptible syscall).

Bug 7053953

(cherry-pick of b3667a19f5c573b7785876979af4781292d27327.)

Change-Id: I0a40cb3d3cdd9aef8589a39586cccd9c229aa8cb

Merge "Use 0xde as the poison value after JIT code cache reset."

Use 0xde as the poison value after JIT code cache reset.

(cherry-pick of 60497bbcee68d98b2bdcb3fb64682b00d61b942d.)

Change-Id: I4f8c1bac2ac3bdc1d931e91f3e2e8038fe4c2b5a

Merge "Optimize those StrictMath routines that are identical to Math routines."

Optimize those StrictMath routines that are identical to Math routines.

We can just use the existing Math intinsics.

Bug: 7146208
Change-Id: I9e78d33cf65a5dcc5a7c0133e67bd9c3c1e43f23

Merge "Remove unnecessary compiler pragma."

Merge "Fix an ugly bug where try/catch offsets weren't being mapped properly."

Merge "Fix a bug where the max blowup of annotations was incorrect."

Fix a bug where the max blowup of annotations was incorrect.

I'm not sure where the 1.34 number comes from but it's incorrect.
From the spec, the encoded_annotation is made up of a single byte
plus an unlimited number of uleb128 values. Each of these values
can double in width in the worst case. I received (personal) email
from one user who'd run into a case worse than the incorrect 1.34
limit.

Change-Id: I3b676e6d2b274aaa538ca61ce23945b3d49aff04

Fix an ugly bug where try/catch offsets weren't being mapped properly.

In dex files, there are several places where one object refers
to another object by its position in the file. The dex merger
is generally very careful to adjust these mappings when combining
dex files.

Unfortunately one of these cases was broken. Each try_item refers
to a corresponding encoded_catch_handler by its byte offset in a
list. Most of the time this byte offset is the same in the input
dex file and the output dex file. But encoded_catch_handlers are
variable-length because they use a variable-length uleb128 encoding
to address the type_idx being caught. When dex files are merged,
some exception types may go from having a small index to having
a large index, increasing the number of bytes required to encode
that index. This breaks our ability to directly copy over offsets
as we were doing previously.

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

Merge "Don't crash in dvmQuasiAtomicsShutdown if we never ran dvmQuasiAtomicsStartup."

Don't crash in dvmQuasiAtomicsShutdown if we never ran dvmQuasiAtomicsStartup.

dalvikvm --help would fail here:

  #0  __pthread_mutex_destroy (mutex=0x0) at pthread_mutex_destroy.c:28
  #1  0xf7b36e13 in dvmDestroyMutex (pMutex=0x0) at dalvik/vm/Thread.h:465
  #2  dvmQuasiAtomicsShutdown () at dalvik/vm/Atomic.cpp:188
  #3  0xf7b65781 in dvmShutdown () at dalvik/vm/Init.cpp:1889
  #4  0xf7b65f31 in ~ScopedShutdown (this=<synthetic pointer>, __in_chrg=<optimized out>) at dalvik/vm/Init.cpp:1317
  #5  dvmStartup (argc=11, argv=0x804b360, ignoreUnrecognized=false, pEnv=0x804b398) at dalvik/vm/Init.cpp:1548
  #6  0xf7b790e2 in JNI_CreateJavaVM (p_vm=0xffffc758, p_env=0xffffc75c, vm_args=0xffffc760) at dalvik/vm/Jni.cpp:3502
  #7  0x08048958 in main (argc=13, argv=0xffffc8b8) at dalvik/dalvikvm/Main.cpp:212

Change-Id: I8c833ca5ea454d9b2d1714a569fb728562088659

Remove unnecessary compiler pragma.

With dlmalloc 2.8.6 the compiler pragmas to suppress warnings are not
necessary.
Remove unnecessary dlmalloc dl prefix #define.

Depends upon: https://android-review.googlesource.com/42351

Change-Id: Idc45cee9342efec0aab63d5172b3308a0fca3c4f

Merge "Correctly handle switches on values in a high register"

Merge "Fix a bug where debugInfos' size wasn't being computed."

Merge "Fix for 6994917 GC is slower since JOO33B per FrameworkPerf test"

Merge "Fix build warning."

Fix build warning.

Change-Id: I1f9813c9dc1dfc388f82f37584fe60256b6a8b12

Merge "Trim pre-Zygote fork, fix under-estimate of heap limit."

Fix for 6994917 GC is slower since JOO33B per FrameworkPerf test

Fixes the GC aspect of the regression in FrameworkTest for Gc.
Large bitmap test seems to be unaffected.

Changes are:
We now have minimum sleep of 1ms instead of 10ms in dvmSuspendAll.
Some heuristics to avoid doing extra GCs if an allocation waited for a concurrent Gc.

Change-Id: I3797c7dc813e26bd1f7bb0a9dbe55eb70dd1c9c9

Fix a bug where debugInfos' size wasn't being computed.

Previously debugInfos' size of 0 caused a malformed dex file
because debug data was present but not mentioned in the table
of contents.

Change-Id: I07171aaee12fef9f303fc505909f44ef1a714114

Trim pre-Zygote fork, fix under-estimate of heap limit.

Card scanning uses dvmHeapSourceGetValue to scan from the heap base to
the total foot print of the heap. Trimming the Zygote may lower the
footprint of the heap by creating a hole in the middle. This hole is
unaccounted for in the card scanning leading to missed dirty cards.

This was a regression introduced by:
https://android-review.googlesource.com/#/c/41728/
as it didn't #define MORECORE_CANNOT_TRIM 1 that was previously defined
here:
https://android-review.googlesource.com/#/c/41731/4/libcutils/mspace.c

Introduce a notion of dvmHeapSourceGetLimit so that card scanning
doesn't rely on footprint. Trim the heap before forking the Zygote. This
change continues to enable morecore trimming of mspaces.

Change-Id: I3f0b12371097f50a1f920cac25846ca08685e7ed

Correctly handle switches on values in a high register

When dx generates a switch on a value in a high register, it prepends a move
instruction, to move the value to a low register. However, this causes the
switch data pseudo-instruction to use address of the move instruction as the
base address of the switch targets, rather than the address of the switch
instruction

Change-Id: I0708d35ec3059c30221cd0a64f426244411d31ea

Merge "Upgrade to dlmalloc 2.8.5."

Upgrade to dlmalloc 2.8.5.

Switch to using dlmalloc 2.8.5. Define mspace functionality directly
using dlmalloc rather than taking from libcutils.
Remove growth limit check in tryMalloc that only checks initial growth
limit.
Implement trimming at the end of the mspace.

Depends upon: https://android-review.googlesource.com/41717

Change-Id: Ia2c6b50bdb0b0d5aae4b18deefbd1bf50dfa49d5

Merge "hardening: eliminate all text relocations from lidbvm"

hardening: eliminate all text relocations from lidbvm

This patch consists of:
- changes to mterp/ that turn all literals from absolute
  to PC relative, so the relocations can be resolved at
  (build) link time
- changes to compiler/template/ that result in the
  compiler templates to live in the non-executable
  .data.rel.ro section (this code is never executed
  directly, only from the jit heap, so there is no
  reason to put it in the .text section)

Change-Id: I2dc97bd4720b393a74b7277a188f0c7b681fc932
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@gmail.com>

Merge "Fix ALOGV bit rot."

Fix ALOGV bit rot.

Change-Id: Ia182af8f5ca3967a21003b3d1513145f8e6e5db0

Merge "Don't crash on invalid string/type ids."

Don't crash on invalid string/type ids.

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

Merge "RETURN_VOID_BARRIER is also a return."

RETURN_VOID_BARRIER is also a return.

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

Merge "Change dalvik script to use exec"

Change dalvik script to use exec

Change-Id: I0a82ced8ce31ae0e6916509a1c79b738c475ae18

Merge "Fix an array out-of-bound read in the JIT compiler."

Fix an array out-of-bound read in the JIT compiler.

Without the fix the compiler is still safe since the offending memory access
is a read, though the hoisted distance is non-deterministic. The easiest and
safest fix is to unconditionally hoist a load when it can reach the
scheduling barrier.

Change-Id: I021161cb2a6e011301ab65ba62bc2a74af1cb552
Signed-off-by: Douglas Leung <douglas@mips.com>

Merge "Fix the MIPS build: 1. LOG->ALOG conversion 2. Remove Jumbo opcodes support"

Fix the MIPS build:
1. LOG->ALOG conversion
2. Remove Jumbo opcodes support

Change-Id: Ifbd1eae2f5e7a1cab5828ebfbdba380ad499abfd

Merge "Fix Method.getParameterAnnotations."

Fix Method.getParameterAnnotations.

The dex format represents parameter annotations using an
annotation_set_ref_item for each parameter. According to the spec,
the item's annotation_offset can be "0 if there are no annotations for this
element". The Dalvik VM accepts 0 values in the verification step,
but Method#getParameterAnnotations then crashes with a segmentation fault.

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

Merge "[X86] X86 trace JIT compiler support"

[X86] X86 trace JIT compiler support

This patch provides a fully functional x86 trace JIT compiler for Dalvik
VM. It is built on top of the existing x86 fast interpreter
with bug fixes and needed extension to support trace JIT interface. The
x86 trace JIT code generator was developed independent of the existing
template-based code generator and thus does not share exactly the same
infrastructure. Included in this patch are:

* Deprecated and removed the x86-atom fast interpreter that is no
  longer functional since ICS.
* Augmented x86 fast interpreter to provide interfaces for x86 trace JIT
  compiler.
* Added x86 trace JIT code generator with full JDWP debugging support.
* Method JIT and self-verification mode are not supported.

The x86 code generator uses the x86 instruction encoder/decoder library
from the Apache Harmony project. Additional wrapper extension and bug
fixes were added to support the x86 trace JIT code generator. The x86
instruction encoder/decoder is embedded inside the x86 code generator
under the libenc subdirectory.

Change-Id: I241113681963a16c13a3562390813cbaaa6eedf0
Signed-off-by: Dong-Yuan Chen <dong-yuan.chen@intel.com>
Signed-off-by: Yixin Shou <yixin.shou@intel.com>
Signed-off-by: Johnnie Birch <johnnie.l.birch.jr@intel.com>
Signed-off-by: Udayan <udayan.banerji@intel.com>
Signed-off-by: Sushma Kyasaralli Thimmappa <sushma.kyasaralli.thimmappa@intel.com>
Signed-off-by: Bijoy Jose <bijoy.a.jose@intel.com>
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>
Signed-off-by: Tim Hartley <timothy.d.hartley@intel.com>

Merge "Use uintptr_t for cast pointer to integer."

Use uintptr_t for cast pointer to integer.

On 64bit host, sizeof pointer is not equal to sizeof int/u4.
Need for host tools.

Change-Id: Id8d9418787e79523226b9c9e3f67277f9ac7c6aa

Merge "Remove untested SH "support"; bionic long since stopped supporting SH."

Remove untested SH "support"; bionic long since stopped supporting SH.

Hello world!

Change-Id: I4660c8905b2641f90d90987938498e8693dc33da

Merge "Avoid sign extension in packed-switch."

Avoid sign extension in packed-switch.

This code (at least in the ARM version) is trying to assign to r0 and r1
from C by returning a 64-bit result. The mistaken use of signed integers
for pointers can lead to sign extension if the JIT code cache is at a
sufficiently high address.

Bug: 6799823
Bug: 6703991

(cherry-pick from 2d0c1c2dbe44458ebb199c47ce1047f266db5349.)

Change-Id: I79e72228b60e195272d11899ac69bb4a76b7402f

Merge "Fix a small and unlikely memory leak."

Fix a small and unlikely memory leak.

Change-Id: I131b9f2f8ae4a4b2bb3bb00bc2cd1e607d979bb5

Merge "Remove some dead code in the MIPS stuff, and run rebuild.sh."

Remove some dead code in the MIPS stuff, and run rebuild.sh.

Change-Id: I7c3226efe97e6e5678b757eb2350d0eeb2efd8b5

Merge "Don't log "WAIT_FOR_CONCURRENT_GC blocked 0ms"."

Don't log "WAIT_FOR_CONCURRENT_GC blocked 0ms".

Stupid to spend more time logging than we spent waiting. (Highly visible
in the zygote's preloading phase.)

Change-Id: If2d6b6093b3bebae0887aa385745b7ed9225216b

Merge "Warn if we see a class whose behavior has changed because of https://android-review.googlesource.com/#/c/32480/."

Warn if we see a class whose behavior has changed because of https://android-review.googlesource.com/#/c/32480/.

Change-Id: Ic7ca229a92c8d1bd666ecf64209d061301dcb0c9

Merge "s/LOGE/ALOGE/ in HAVE_SELINUX blocks"

s/LOGE/ALOGE/ in HAVE_SELINUX blocks

The latest push changed LOGE to ALOGE but it was not updated in HAVE_SELINUX blocks.

Change-Id: If64c6beeb1d7c24e1b610acb8ccb195e00c777da
Signed-off-by: Joshua Brindle <jbrindle@tresys.com>

Merge "Avoid method overriding if its super method is inaccessible."

Avoid method overriding if its super method is inaccessible.

A call to package private method could be redirected to subclass which
was not in the same package.
Modified vtable to retain virtual super methods which cannot be accessed.

This change affects vtable index in optimized dex.

Change-Id: I9cc7e309c305bca12e5061009c4245fb70014681
Signed-off-by: Im Sooin <ciecet@gmail.com>

Merge "Missing zero-checks in JIT compiler"

Missing zero-checks in JIT compiler

Zero-checks were not generated by the JIT compiler for
some instructions. This caused crashes instead of
the expected ArithmeticException.

Change-Id: Ic8041741a7cccc1bd6b8c3c0723ba55a55af856b

Merge from AOSP

Change-Id: I8ac8ecf5d57b4d852957a0c4498e3e304e8e2e17

Merge "hprof generation, not writing the first record"

am 7798a614: Merge "Fix a possible off-by-one in the verifier."

* commit '7798a614753251adf7a77835792d78db2e93f5ff':
  Fix a possible off-by-one in the verifier.

hprof generation, not writing the first record

We miss some roots since the first record isn't initialized.

Change-Id: I50c80a701a4ee142c52602b9015f6e93be54da9f

Merge "Fix a possible off-by-one in the verifier."

Fix a possible off-by-one in the verifier.

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

resolved conflicts for merge of 7c03fb79 to jb-dev-plus-aosp

Change-Id: Icedbe5a6d7cbcedde53e05249f7e90a390f2f0ea

Merge "Extend forkAndSpecialize to take additional inputs and to set the SELinux security context."

am 44292346: Merge "Switched code to blx <reg>"

* commit '44292346f4935cd078ca7b931042809af80f9d95':
  Switched code to blx <reg>

Merge "Switched code to blx <reg>"

Switched code to blx <reg>

ldr ip,<addr> blx ip is preferred over mov lr,pc ldr pc,<addr> from armv5te,
and will typically perform better on later ARM processors.

Change-Id: I8f2e5e794c644faafd767037ad56579f2934de47

am effc95cb: Merge "Make dex merging stateless."

* commit 'effc95cb732d8d5453930e647240a1665b6dc71a':
  Make dex merging stateless.

am a9ac3a9d: Merge "Support debug info in dexmerge."

* commit 'a9ac3a9d1f8de71bcdc39d1f4827c04a952a0c29':
  Support debug info in dexmerge.

Merge "Make dex merging stateless."

Make dex merging stateless.

This is required to run dx in Eclipse which doesn't unload the dx
library after each run.

(cherry-pick of 7aa5ce7e990dc3766eba97cd0932b62e4de21503.)

Change-Id: I7a69a3015d448ddd5558c307cd01346156cbc739

Merge "Support debug info in dexmerge."

Support debug info in dexmerge.

Bug: 4090053

(cherry-pick of bc23c4f3ebaefebb3f1be7732767631f91e165ea.)

Change-Id: I1108933fc03330ff91be3a2edef8b4966977dcd7
Signed-off-by: Jesse Wilson <jesse@swank.ca>

am bf11122d: Merge "Revert "  Trim after explicit gc.""

* commit 'bf11122d175d1315783b313897ee8ecae63f0278':
  Revert "  Trim after explicit gc."

Merge "Revert "  Trim after explicit gc.""

Revert "  Trim after explicit gc."

This reverts commit ea4c6e57a48e08eacbf08520c64133175e7d5da0

Reconcile with jb-release

Change-Id: I3e7e63e5d19d73ece11951f9cc3f85589c72adbe

am 53bc3719: Merge "Fixing a crash when doing lock profiling"

* commit '53bc3719bab6f0a075dd3f968a6d9a630fb81476':
  Fixing a crash when doing lock profiling

Merge "Fixing a crash when doing lock profiling"

am 90d3e3b3: Merge "Add test cases for concurrent gc and System.arraycopy"

* commit '90d3e3b3999aa22d0cecc4f064e5686c555ad700':
  Add test cases for concurrent gc and System.arraycopy

Merge "Add test cases for concurrent gc and System.arraycopy"

Add test cases for concurrent gc and System.arraycopy

When System.arraycopy runs at the same time as a concurrent gc
the phone will sometimes crash since System.arraycopy was implemented
using memmove and memcpy. In current implementation of mememove bytes
are copied one at a time. If for instance only 3 out of 4 bytes
to an object reference were copied when the thread switch to the
gc thread occurred and the gc was scanning the marked objects the
gc read an invalid address. The parameters to dvmWriteBarrierArray
in one case was also corrected (they are currently not used).

The fix itself for this crash is made elsewhere, this commit just
adds test cases to verify that this works.

Change-Id: I0a8cfd43561b3d5de4bba818993bcf8b40413045

Fixing a crash when doing lock profiling

This is a fix for a crash that can happen when logging Contention
events. This logging is performed when lock profiling is enabled.
This is by default enabled on userdebug builds. The crash happened
when a thread was being destroyed. When a thread is being
destroyed it is normal that the frame depth is zero, and thus the
current frame is null. logContentionEvent() requires that the
current frame is not null, or it will crash.

The fix is to check if the current frame is null.

Change-Id: I4c2b9ad94b663398645497fdffa1ec6f7ea86a51

merge in jb-release history after reset to jb-dev

am 89cf125c: Report GC pause times more accurately.

* commit '89cf125cd95a7df3a713cb3d3d39a033304e474c':
  Report GC pause times more accurately.

Report GC pause times more accurately.

Include the time that it actually took to pause or suspend
threads in the reported time.

Also report the amount of time that other threads spend blocked
waiting for a concurrent GC to complete.

Change-Id: I80af351c2dc3171fc7db48dbbb361f3d92bbca80

am 07901d5e: Merge "Running rebuild.sh found a few changes to be updated to vm/mterp/out/InterpAsm-mips.S and vm/mterp/out/InterpC-mips.cpp"

* commit '07901d5e8d0be0130d92b626f0b92d177ba8f460':
  Running rebuild.sh found a few changes to be updated to vm/mterp/out/InterpAsm-mips.S and vm/mterp/out/InterpC-mips.cpp

Merge "Running rebuild.sh found a few changes to be updated to vm/mterp/out/InterpAsm-mips.S and vm/mterp/out/InterpC-mips.cpp"

Running rebuild.sh found a few changes to be updated to
vm/mterp/out/InterpAsm-mips.S and
vm/mterp/out/InterpC-mips.cpp

Reconcile with jb-release

Change-Id: I9be82c25a57356698a04628abbbb0a18f95a9ca2

merge in jb-release history after reset to jb-dev

merge in jb-release history after reset to jb-dev