Merge "Fix dalvik tests post-conscrypt."

Fix dalvik tests post-conscrypt.

Change-Id: I36b942916330a3d6a06fad93dd9aa0533b47ff85

Merge "Fix some leaks in VfyBasicBlock & BitVector."

Fix some leaks in VfyBasicBlock & BitVector.

Change-Id: I20050135588fc96268911b8ab690b04c8ad949bc

Merge "Fix dx to match the new SDK structure."

Fix dx to match the new SDK structure.

(cherry picked from commit cb9dfa123c6fcc43be629666837c4e6f86b476ea)

Change-Id: I071d73adc4a2eb812cdbc204d27884f4a527043d

Merge "Fix dx.bat to match the new SDK structure."

Merge "Tracking changes to *-host for host shared libraries"

Tracking changes to *-host for host shared libraries

Change-Id: I764889011cc259cdca306e2145bcc4c8c75eced5

Merge "Fix JIT initialization race"

Fix JIT initialization race

See b/8767735 Slow thread in thread spin test

A race exists in thread creation/JIT initialization that can
leave the JIT permanantly disabled for a thread (typically
the first interpreter thread).

(cherry-pick of 67985d960f5c877f86d0bba251d2d55c7a062b8d.)

Change-Id: Ic7e3a1cde2833ac78455fa5d465156b5e0896fc1

Merge "Update dalvik host script for new conscrypt and okhttp classpath entries"

Update dalvik host script for new conscrypt and okhttp classpath entries

Change-Id: I9af7dfeb7387544bff98af55716f3db9808e39cb

Merge changes I3ef1ad8c,I0bbbba1a,Ife107fb6,Ia420b766

* changes:
  Switch Dex from byte[] to ByteBuffer for backing store. Part 2.
  Move dex utilities from dalvik/dx to libcore/dex
  Mechanical refactoring of dx into two parts.
  Change com.android.dx.io.Annotation to use EncodedValue

Merge "Teach dex how to parse encoded values from .dex files."

Switch Dex from byte[] to ByteBuffer for backing store. Part 2.

Change-Id: Ifd3672027d075b6c9f8df208a04bc33122c883af
(cherry picked from commit f8357511efd16d76ce4410459c15dbb08b0140b6)

Conflicts:
dx/src/com/android/dx/merge/DexMerger.java

Move dex utilities from dalvik/dx to libcore/dex

Change-Id: If0022a069ba3ac51ae5cdc4b0eabb6b913cd8afe
(cherry picked from commit 7b13a4d41243f81fe80a76e2ff2f877b53610446)

Conflicts:
dx/src/com/android/dex/Code.java
dx/src/com/android/dex/Dex.java
dx/src/com/android/dex/SizeOf.java
dx/src/com/android/dex/TableOfContents.java

Mechanical refactoring of dx into two parts.

This splits off a new package, com.android.dex that contains
code for parsing and modelling dex files. This code is usable
both at build time (for compilation and analysis) and at runtime
(for introspection).

The original package, com.android.dx is the tool that compiles
.class files into .dex files. That package also includes utilities
for merging and querying dex files, all visible to the dx command
line application.

In a follow up change I'll move the new com.android.dex package
into the libcore/ project, and configure build rules so that it's
included in both dx.jar and core.jar. The core.jar will then be
able to use it for annotation processing.

Change-Id: I3c58cf87e728e4dda1925a0992c1ee7b8130e81a
(cherry picked from commit bab4abb07335d162ecdb8091fc395cf84803a580)

Conflicts:
dx/junit-tests/com/android/dx/util/BitIntSetTest.java
dx/junit-tests/com/android/dx/util/BitsTest.java
dx/junit-tests/com/android/dx/util/IntListTest.java
dx/junit-tests/com/android/dx/util/ListIntSetTest.java
dx/src/com/android/dex/Dex.java
dx/src/com/android/dex/TableOfContents.java
dx/src/com/android/dx/annotations/AnnotationAccess.java
dx/src/com/android/dx/command/Main.java
dx/src/com/android/dx/dex/code/OutputFinisher.java
dx/src/com/android/dx/dex/code/form/Form32s.java
dx/src/com/android/dx/dex/code/form/Form33x.java
dx/src/com/android/dx/dex/code/form/Form41c.java
dx/src/com/android/dx/dex/code/form/Form52c.java
dx/src/com/android/dx/gen/DexGenerator.java
dx/src/com/android/dx/merge/DexMerger.java

Change com.android.dx.io.Annotation to use EncodedValue

(cherry picked from commit 4ec022b0b6c84f994ab2fc78934110ad96d0bdec)

Change-Id: Ia420b7661f372c59af8fe62bdab7982a1d2b008f

Teach dex how to parse encoded values from .dex files.

Previously primitive values were treated as opaque byte arrays.
Now the encoded values can be interpreted as their proper types:
integers, indices, floats, chars, etc. This gets complicated
pretty fast due to the dense packing dx performs when encoding
the values.

The encoding code is moved from ValueEncoder. The decoding code
is ported from C++ to Java from Annotation.cpp.

There's a bunch of new tests to make sure the decoding is correct;
porting from C++ to Java was tricky because of the '>>' shifts on
unsigned values. The test input data is generated by dex!

Change-Id: I83b2fc3e16115d667fa94b3dab782d1a9687f3ad
(cherry picked from commit 27847605b9255358f0577ffec28886c450263898)

Merge "Fix JIT bug related to immediate doubles"

Fix JIT bug related to immediate doubles

Change 256211 (JIT: Performance Fix for const doubles) introduced a
defect that can cause the JIT to use the wrong floating point
double constant in traces in which the following conditions hold:

   o Two (or more) different 64-bit floating point constants are used.
   o The physical register holding the first constant is still live
     at the time the second constant is used.
   o The low 32 bits of the two constants are identical.

In this situation, the load/copy optimization pass will incorrectly
determine that the two constants are the same, delete the load of
the second constant and re-use the first constant value.

Note: this problem only occurs with 64-bit floating point literals.
64-bit long literals are unaffected.

This CL works around the problem, and a subsequent CL will rework
disambiguation of 64-bit immediates in a somewhat cleaner fashion.

(cherry-pick of c1757a6deab0ca0bfd42c38612d92b2f26c41dbe.)

Change-Id: I795b4b753550d2745cbbdd83ae25f4a7088990f6

Merge "Don't duplicate annotation set ref lists when merging."

Fix dx.bat to match the new SDK structure.

(Cherry-picked from c26c1291743ce7877cf11f3dc73d6f0c240229f7)

Change-Id: I57a7678178fc58e2a101c9d86da5009c6ec4f6a0

Merge "Remove spammy logging from 8470684 investigation."

Remove spammy logging from 8470684 investigation.

This has been tracked down to a kernel bug, so we no longer need
the extra diagnostics.

Bug: 8470684
Change-Id: Ib170d1f7b94488ed4acc763f8dddc44c81807aed

Merge "Work around a dalvik JDWP/GC deadlock."

Work around a dalvik JDWP/GC deadlock.

Also add a few missing TEMP_FAILURE_RETRYs that can cause dalvik to
incorrectly decide that the debugger connection has been broken.

Bug: 8191824
Change-Id: I791b514c6bec1ca634a65d9f87c5fa06da01b5ed

Merge "More native stack dump hardening."

More native stack dump hardening.

Threads just starting up or shutting down might not have any managed
stack frames, leading to a NULL "currFrame" frame pointer in the
interpreter stack.

Bug: 8596028
Change-Id: Ie24c8d5f8e78a5abe882a9e639046c03abb91649

Merge "Fix flakiness in test 080-oom-throw."

Fix flakiness in test 080-oom-throw.

The problem was that the instance test could potentially fill the
heap close enough to the edge that you'd hit a follow-on OOME in
the attempt to print the success message, causing the test to fail.

I've also made the code a bit clearer while I'm here.

Bug: https://code.google.com/p/android/issues/detail?id=54114
Change-Id: I39127c3ea6a99940fa66ad8763bbbf6c01d308ef

Merge "Harden the native stack dumping decision."

Harden the native stack dumping decision.

We've seen monkey crashes in this code, though I haven't been
able to reproduce them in my own SIGQUIT stress tests. Address
the two most likely causes of trouble: dumping the signal catcher's
own thread (which will always be runnable), and assuming that the
Method* pulled from the save area is non-NULL.

Bug: 8596028
Change-Id: I59d1dcb2264a774d8416d50a5f77a06c70d37c59

Merge "Dump native stacks for all threads in native code."

Dump native stacks for all threads in native code.

Bug: 7432159
Change-Id: I83cb530155edfc35ae3be0f7a2a044245223d2d5

Don't duplicate annotation set ref lists when merging.

See exception reported on this thread:
https://groups.google.com/d/msg/android-contrib/h0sDMo5DVBI/MAMiBv5XR44J

Change-Id: I867a8d2328b32625e5baa0b029f55e373fe11271

Merge "Tiny optimization for complier templates for arm."

Merge "Fix a leak in Dalvik_dalvik_system_DexFile_openDexFile"

Merge "Fix a minor leaks caused by failed initializations.(typo)"

Merge "Fix a minor leak in handleVM_CreateString"

Fix a minor leak in handleVM_CreateString

Change-Id: I3273e5d2eb4621202837c43270a466fe056d6894

Fix a minor leaks caused by failed initializations.(typo)

Change-Id: I62bc3ea48938a4f54e9f47218d4025e954a6d566

Fix a leak in Dalvik_dalvik_system_DexFile_openDexFile

Change-Id: I2b394c620af31914606763b35847b7f412774cf4

Tiny optimization for complier templates for arm.

1. Remove possible bubble in TEMPLATE_STRING_INDEXOF.S
2. Remove 1 instruction and reorder the opcodes
   TEMPLATE_MUL_LONG.S
3. Reorder ldr r2 instruction in TEMPLATE_RETURN.S

Change-Id: I571c1278aa72cb1e6dbc6efe433932d5e9591b80

Merge "[x86] Fix errors with WITH_JIT_TUNING defined"

Merge "[x86] Terminate trace if SGET/SPUT have unresolved fields in codegen"

Merge "[x86] FPU stack needs to be reset after double conversion"

[x86] FPU stack needs to be reset after double conversion

The x86 codegen uses the FPU stack for double/float to long conversions. We
need to clear out the FPU stack after done, to prevent an eventual stack
overflow.

Change-Id: I2f306d7c228ad3da2b84faf9f08326769a9417af
Signed-off-by: Udayan Banerji <udayan.banerji@intel.com>

Merge "[x86] Disable Method-JIT for x86 target"

Merge "Fix a cast in JniEnv::NewDirectByteBuffer."

Fix a cast in JniEnv::NewDirectByteBuffer.

Change-Id: I8692b31c6ce64efc68124df02117b6e4529dfe98

Merge "Track libcore's DirectByteBuffer <init> change."

Track libcore's DirectByteBuffer <init> change.

See 9ce6a2533b50ffcd24b8b10bae7f234042768819.

Change-Id: Ie736e69a3fa7bdf55709d8e2960761202a20e820

Merge "Extra log information when pthread_create fails."

Extra log information when pthread_create fails.

Bug: 8470684
Change-Id: I24d22d5ec4181614d173a582773540c8f712ab18

[x86] Fix errors with WITH_JIT_TUNING defined

This patch makes the necessary changes to pass on correct information to
dvmBumpNoChain, so that WITH_JIT_TUNING flag can be enabled for x86 codegen

Change-Id: Ia5e5c0406433bf645ef67143d0f1a11a28153a66
Signed-off-by: Udayan Banerji <udayan.banerji@intel.com>

[x86] Terminate trace if SGET/SPUT have unresolved fields in codegen

The interpreter doesn't allow SGET/SPUT bytecodes in a trace till the field
is resolved. However, exhaustTrace can pick up bytecodes beyond the trace
sent by the interpreter. Terminate the loop formation if this is seen.

Change-Id: I0f38d6690b3501111bd16103623fa545d0ec1873
Signed-off-by: Udayan Banerji <udayan.banerji@intel.com>

[x86] Disable Method-JIT for x86 target

Disable Method JIT when compiling for x86 target.

Change-Id: Ide0dbd1f602ffd955b901cc152de1e05771fd529
Signed-off-by: Udayan Banerji <udayan.banerji@intel.com>

Merge "Track 64-bit nio changes."

Track 64-bit nio changes.

Change-Id: Id7a11af5f94b8ba82d42e1e7045302aca533f31a

Merge "Track libcore DirectByteBuffer cleanup."

Track libcore DirectByteBuffer cleanup.

Bug: https://code.google.com/p/android/issues/detail?id=53637
Change-Id: I7acd990a2a48b41dacb5b09f6410b4634dd60438

Merge "pthread_create doesn't set errno."

pthread_create doesn't set errno.

Let's try to get slightly better logging and exception detail messages.

Also fix the "set but not read" warnings for the other pthread functions
in this file.

Bug: 8470684
Change-Id: I7f9dbb4d39b4e3022cd5f1a8a45c6426e68c858b

Merge "Print the locker thread when waiting on an object"

Merge "javac should have matching -source and -target flags."

javac should have matching -source and -target flags.

Change-Id: I2ac5f0685bd5844252cc334d4040d68e01d39429

Merge "Fix a Field.isAnnotationPresent crash."

Fix a Field.isAnnotationPresent crash.

For instances of Proxy, generated at runtime, there is no corresponding
.dex file.

Bug: https://code.google.com/p/android/issues/detail?id=24846
Change-Id: I36da43d5279a61b8cb71cdf2bbc3065b6a979830

Merge "Check JavaVMAttachArgs::version in CheckJNI."

Check JavaVMAttachArgs::version in CheckJNI.

Bug: 8363798
Change-Id: I314bf6d5d64a1e27ad2c1580532133859031c623

Merge "Don't crash if the dmtracedump -d file doesn't exist."

Don't crash if the dmtracedump -d file doesn't exist.

Change-Id: Ie4b504b43a8a61d09ca60d68b1a62694391ac223

Merge "Fix -Xjitthreshold (for real this time)."

Fix -Xjitthreshold (for real this time).

My previous "fix" (c89d83e1c05979b68037ad15413fa4460a88e36f) had the
conditions reversed, so you _had_ to use -Xjitthreshold to get a non-zero
threshold, but when you did, you'd get the default instead of what you
asked for!

This was spotted by the jank tests.

Bug: 8285558
Bug: https://code.google.com/p/android/issues/detail?id=52017
Change-Id: I28270f2573d46929eb10d30789fecf7d5a8cea75

Merge "Zygote: use proper header file."

Merge "Remove CAP_NET_RAW from zygote"

Remove CAP_NET_RAW from zygote

ping no longer needs CAP_NET_RAW. See:

* http://lwn.net/Articles/443051/
* https://android-review.googlesource.com/52090
* https://android-review.googlesource.com/52072

Eliminate the CAP_NET_RAW special case in zygote.

Change-Id: Ie4b1d2b65a27051bbd2825d288f9838762ae31c8

Zygote: use proper header file.

capset is defined in <sys/capability.h>

Change-Id: Ifb340cfb3506830242749456706d4eddb73fb2aa

Merge "Fix -Xjitthreshold."

Fix -Xjitthreshold.

Previously, we'd always overwrite the user-supplied value because
the architecture-specific default gets set so late.

Bug: https://code.google.com/p/android/issues/detail?id=52017
Change-Id: I469bf9ce599820f5ce3dea346aa8f680deffb0c5

Merge "Abstract methods aren't implementations."

Abstract methods aren't implementations.

We should only be checking the visibility of actual implementations
of interface methods. Intervening abstract methods are allowed to
be non-public.

Bug: https://code.google.com/p/android/issues/detail?id=42991
Change-Id: I510d0cc6c1f89d161d4ebad7ef058c03fa09e9b0

Merge "Zygote: replace set?id() with setres?id()"

Zygote: replace set?id() with setres?id()

setuid -> setresuid()
setgid -> setresgid()

Change-Id: I319a6a0ea63cf5bd3755ec8371064365be30872b

Merge "Expand test 004-annotations to cover package annotations."

Expand test 004-annotations to cover package annotations.

Bug: https://code.google.com/p/android/issues/detail?id=16149
Change-Id: I0fa26a5201fc80a6f049d61d34e403d4cb5c505a

Merge "Added missing initialization of modLock when DvmDex structures are allocated."

Added missing initialization of modLock when DvmDex structures are allocated.

The structure is zeroed with dvmAllocRegion (use to be calloc) which for most
implementations of pthreads behaves correctly as if initialized. However, an explicit
initialization is more correct.

Change-Id: I172fed73c2a8a6962d720f78c60436b4f58cc168

Merge "Move the arraycopy implementation out of bionic and into dalvik."

Move the arraycopy implementation out of bionic and into dalvik.

Change-Id: I5b7964e53eca6f2da949d6ef24d42a9c6ed3996c

Merge "Zygote: address comments from previous review."

Zygote: address comments from previous review.

Previous review at https://android-review.googlesource.com/51731

Change-Id: Idfe6c95976cc8f82ec473bcb630678fa9ec9486a

Merge "Zygote: limit the bounding capability set to CAP_NET_RAW"

Zygote: limit the bounding capability set to CAP_NET_RAW

Prevent a zygote spawned application from acquiring
capabilities other than CAP_NET_RAW.  The only Zygote
accessible program on Android which grants capabilities
is /system/bin/ping (CAP_NET_RAW), so we don't need to
keep the other capabilities in our bounding set.

If the kernel doesn't support file capabilities, we
end up printing approx 30 lines of warning messages. Hopefully
this will encourage kernel developers to upgrade. In a future
change, we can turn a prctl(PR_CAPBSET_DROP) failure into
a fatal error.

Change-Id: I8560fa5ad125bf31f0d13be513431697bc7d22bb

Merge "Remove stray references to non-existent opcodes."

Remove stray references to non-existent opcodes.

Change-Id: I83ca6ea12344a8df7ef14d600baf5060b76870f5

Merge "Revert "Zygote: limit the bounding capability set to CAP_NET_RAW""

Revert "Zygote: limit the bounding capability set to CAP_NET_RAW"

This reverts commit 921e9aea72bc2aa99a52ccdb90573bbd3bf3508f, which prevents the emulator from booting.

Change-Id: I73b7afba38d4087d552dd685c56d4e5ddb3103cf