am 0f79299e: Merge "Add fast path to VMClassLoader.findLoadedClass" into lmp-dev

* commit '0f79299e78cb9e304f2a723ce7691f35ecaa4bab':
  Add fast path to VMClassLoader.findLoadedClass

am 6b9f31a7: Merge "Delete pin table" into lmp-dev

* commit '6b9f31a7cbdc6d4b590c8c2119ffddbf5f8c0720':
  Delete pin table

am 776ee723: Merge "Print the command line and ABI in stack dumps like debuggerd." into lmp-dev

* commit '776ee7236f3d95ad5b8a1e6af693e636266f8131':
  Print the command line and ABI in stack dumps like debuggerd.

am 53f7ac1d: Merge "Revert "Print the command line and ABI in stack dumps like debuggerd."" into lmp-dev

* commit '53f7ac1d6b96621e9fd0225d7579a0183f6f179e':
  Revert "Print the command line and ABI in stack dumps like debuggerd."

am d1ae5004: Merge "Print the command line and ABI in stack dumps like debuggerd." into lmp-dev

* commit 'd1ae50047c5da787e722e182992bd242540b502a':
  Print the command line and ABI in stack dumps like debuggerd.

Merge "Add fast path to VMClassLoader.findLoadedClass" into lmp-dev

Merge "Delete pin table" into lmp-dev

Merge "Print the command line and ABI in stack dumps like debuggerd." into lmp-dev

Print the command line and ABI in stack dumps like debuggerd.

Ensure the heading is "Cmd line:" just like debuggerd and
as it used to be in Dalvik.
Fix a missing newline.
Trim all training nulls from the command line.
Don't bother printing the original command line if unset.
Add the ABI to the dump to help the native stack symbol tool.

Bug: 17474152
Change-Id: Ie1da1bd4f38c92b13ea08c7122f4573d8ec8fc73

Merge "Revert "Print the command line and ABI in stack dumps like debuggerd."" into lmp-dev

Revert "Print the command line and ABI in stack dumps like debuggerd."

This reverts commit 38ac77b6fb1fc626299022fe0f16d5b6c365cb9b.

Change-Id: I8e4952b80d03ad61c3ff2afaf8a96b8c2e36f0c7

Merge "Print the command line and ABI in stack dumps like debuggerd." into lmp-dev

Add fast path to VMClassLoader.findLoadedClass

VMClassLoader.findLoadedClass now calls FindClassInPathClassLoader
as a fast path. Exclusive time results (trace view maps launch):

Before:
nativeFillInStackTrace 1.4%
defineClassNative 1.2%
findLoadedClass 0.2%

After:
nativeFillInStackTrace 0.5%
defineClassNative 0.0%
findLoadedClass 0.9%

Bug: 16828525

Change-Id: I1bde48effcd28529778c00ec0fa0dda4e32026a3

Print the command line and ABI in stack dumps like debuggerd.

Ensure the heading is "Cmd line:" just like debuggerd and
as it used to be in Dalvik.
Fix a missing newline.
Trim all training nulls from the command line.
Don't bother printing the original command line if unset.
Add the ABI to the dump to help the native stack symbol tool.

Bug: 17474152
Change-Id: I19c585dbd7d31ef6c98270e469445cae40cbd518

Delete pin table

The pin table was brought over from dalvik but not really needed
since ART doesn't support pinning in movable spaces. The only
thing it did was hold objects live for JNI functions.
This shouldn't be necessary since people keep jni references to
these objects or else they could never release the elements.

Bug: 17456946

Change-Id: Ibed0d029157ffb9e75ecd80d4d544d690986c090

am 3892cf8d: ART: Fix preverified setting in VerifyClass

* commit '3892cf8da7d5e76c0dee585fc8f69df773680525':
  ART: Fix preverified setting in VerifyClass

ART: Fix preverified setting in VerifyClass

Make sure soft-failed classes cannot set methods to pre-verified.

Bug: 16828525, 17465185
Change-Id: I09c0a68ca722978459741311148eae7614f9ca49

am 0a40828e: Merge "Improve dex location canonicalization-related performance." into lmp-dev

* commit '0a40828e6f4914968048642e8c0e5fdf6bc3b1af':
  Improve dex location canonicalization-related performance.

am 11db17c4: Merge "Fix GetDexCanonicalLocation test for relative paths." into lmp-dev

* commit '11db17c44cdbd71e97a677cdc8fb1e18907ab93d':
  Fix GetDexCanonicalLocation test for relative paths.

Merge "Improve dex location canonicalization-related performance." into lmp-dev

Merge "Fix GetDexCanonicalLocation test for relative paths." into lmp-dev

am 96cd6714: Fix crash when connected to DDMS

* commit '96cd6714d94b47512710776761dec98c8be68bef':
  Fix crash when connected to DDMS

Fix crash when connected to DDMS

Restore init of WellKnownClasses::org_apache_harmony_dalvik_ddmc_Chunk_data and
removed duplicated line.

Bug: 17467328
Bug: 17463118
Change-Id: I64638e23615b7c20061b49c41a9c0e9f35a4d537

am 5f0438a4: Merge "Fix deadlock in VirtualMachine.AllThreads" into lmp-dev

* commit '5f0438a4123aeaae55ee034d31c5b1465fcf69cd':
  Fix deadlock in VirtualMachine.AllThreads

Merge "Fix deadlock in VirtualMachine.AllThreads" into lmp-dev

am 94aa2482: Merge "Don\'t hold any lock when visiting classes from JDWP" into lmp-dev

* commit '94aa2482e8ba71e85a875c2afc85ad8b8d3fb0f3':
  Don't hold any lock when visiting classes from JDWP

Merge "Don't hold any lock when visiting classes from JDWP" into lmp-dev

am 59d0e8ac: Merge "Use WellKnownClasses in ThrowStackOverflowError to reduce changes of further stack overflow" into lmp-dev

* commit '59d0e8aca127088bc08586add9be38045ad420c0':
  Use WellKnownClasses in ThrowStackOverflowError to reduce changes of further stack overflow

Merge "Use WellKnownClasses in ThrowStackOverflowError to reduce changes of further stack overflow" into lmp-dev

Use WellKnownClasses in ThrowStackOverflowError to reduce changes of further stack overflow

Bug: 17463118
Change-Id: Ic05b07803d2d3112e0b67064f380cd46ba9e293f

am df1532b9: ART: Correctly make methods preverified

* commit 'df1532b9ba0cda2d00b78fbdef461f8a6cf8a737':
  ART: Correctly make methods preverified

ART: Correctly make methods preverified

Bug: 16828525
Change-Id: I66756348b2aa50e41dacca59769b6810a91c73b0

Improve dex location canonicalization-related performance.

Eagerly add canonical dex file locations to the OatFile's
primary lookup map in Setup(). This moves the boot.oat work
from every app startup to the zygote initialization. Since
we always ended up initializing the canonical location map
anyway due to the way that we're loading dex files, the lazy
initialization didn't save anything.

Clean up dex file name canonicalization to make sure we
free() the memory returned by realpath() rather than using
std::unique_ptr<> with the default deleter.

Avoid some unnecessary duplicate OatDexFile lookups.

Bug: 16828525
Bug: 17346103

(cherry picked from commit aa4497db59f1eeec954f2ba5da6d458fcdf9b3a4)

Change-Id: Icc4b14ebe903282ca91ce24e33a6d7c75dff991c

Fix GetDexCanonicalLocation test for relative paths.

If the reference file is created with a relative path the test will
fail.

(cherry picked from commit 61281dc0aaa22730fd4fee268cac40776b193042)

Change-Id: Ie82b9d381bd884111dfc91cc235b46ad9f3225be

am 827645f2: Merge "Ensure class is linked before resolution" into lmp-dev

* commit '827645f27237f3e77c87cb5e49e90389a22337a6':
  Ensure class is linked before resolution

Merge "Ensure class is linked before resolution" into lmp-dev

am 44332cc2: Merge "Add proper cfi directives." into lmp-dev

* commit '44332cc21494726c9ba90ac8f23af8acc2dc4536':
  Add proper cfi directives.

Ensure class is linked before resolution

Adds a missing call to ClassLinker::EnsureResolved to ensure we did link the
class and retired the temp class (placeholder) before doing the resolution.

Bug: 17435441

Change-Id: Ib1a7181d6e5e814ca9299d0504e739a2b69475ef

Merge "Add proper cfi directives." into lmp-dev

am 5875a488: Merge "VisitClassesWithoutClassesLock isn\'t safe if classes move." into lmp-dev

* commit '5875a4884efcfc7bf80b0bbb50148611286dbe30':
  VisitClassesWithoutClassesLock isn't safe if classes move.

Merge "VisitClassesWithoutClassesLock isn't safe if classes move." into lmp-dev

am 0bff4f37: Increase dex2oat timeouts for debug builds

* commit '0bff4f375d8013a4a59cf70b3427fb461a17e6de':
  Increase dex2oat timeouts for debug builds

Add proper cfi directives.

This function has a mix of arm and cfi directives. Since our unwinders
can work with both, remove the arm directives and fix the cfi directives.

Bug: 17392751
Change-Id: Ibd4859a4f0b8f2da751a0928a2f5f4afa3df7df8

Increase dex2oat timeouts for debug builds

Should fix the 056-const-string-jumbo test.

Change-Id: I04d98dfd59003b964c1feee6eee31079f9f4a476

am 4e67841e: Change Reference.get() intrinsic to Reference.getReferent().

* commit '4e67841e99e4a206133e7010653ccd132682296a':
  Change Reference.get() intrinsic to Reference.getReferent().

Change Reference.get() intrinsic to Reference.getReferent().

The reference intrinsic was incorrectly inlining
PhantomReference.get(). We now get around this by adding a
layer of indirection. Reference.get() now calls getReferent()
which is intrinsified and inlined.

Requires:
https://android-review.googlesource.com/#/c/107100/

Bug: 17429865

(cherry picked from commit cd48f2d86197d4fe87cc88077bc4af5ba66e5295)

Change-Id: Ie91e70abf43cedf3c707c7bb8a5059e19d2a2577

am f8aed38f: Merge "Change 099-vmdebug test to use File.createTempFile." into lmp-dev

* commit 'f8aed38fd66b14ca31638362050cbfaa64f6135e':
  Change 099-vmdebug test to use File.createTempFile.

Merge "Change 099-vmdebug test to use File.createTempFile." into lmp-dev

Change 099-vmdebug test to use File.createTempFile.

Avoids issues with concurrency.

Bug: 17439227
Change-Id: I4a5956d9f26fad416a244216c2c77d5e6f072c62

am 3298611e: Merge "Temporarily disable LOS on 64 bit" into lmp-dev

* commit '3298611e205a299612f2e403bfaeb955ec0a2bdd':
  Temporarily disable LOS on 64 bit

Merge "Temporarily disable LOS on 64 bit" into lmp-dev

Fix deadlock in VirtualMachine.AllThreads

We cannot add any object in the JDWP object registry while holding the
Locks::thread_list_lock. Indeed we may need to suspend a thread and take it,
causing a deadlock by waiting for ourself on this lock.

Bug: 17343664
Change-Id: I2335c63e418c0d485daf2c35f8debcd52f943140

am 1b2140c9: ART: Better error reporting on broken oat file

* commit '1b2140c9bf46388e4f59ecf3d62d99cd78fed12c':
  ART: Better error reporting on broken oat file

ART: Better error reporting on broken oat file

Do not blow up when in GetHashBucket if GetHashBucketNum is zero.
Instead back-step to ElfFileOpen, which prints which symbol was
missing from which file.

Bug: 17422404
Change-Id: I3b890dd1c31c08b1ccc0f7f668afcceee95f8d00

am 6b8f8688: Merge "Workaround Darwin ContainedWithinExistingMap issue" into lmp-dev

* commit '6b8f868862de9c8a191462ab2a2c979aea4857d3':
  Workaround Darwin ContainedWithinExistingMap issue

Merge "Workaround Darwin ContainedWithinExistingMap issue" into lmp-dev

Workaround Darwin ContainedWithinExistingMap issue

Bug: 16861075
Change-Id: I577a0c7ede919e610c356e7c40dd60d87914dd3a

am 77c318d5: Merge "Fix monitor visit for class initializer" into lmp-dev

* commit '77c318d5c22c5db02bd09f80c97bb117ce6f5e4c':
  Fix monitor visit for class initializer

am 5300d478: Merge "ART: Fix unchecked register index validity" into lmp-dev

* commit '5300d478fb60f3aa9277f29aef9a901d26f7469a':
  ART: Fix unchecked register index validity

Merge "Fix monitor visit for class initializer" into lmp-dev

Merge "ART: Fix unchecked register index validity" into lmp-dev

am 8a4064e2: Fix darwin run-test problems

* commit '8a4064e2c0987637b11afcc656478d3113d81618':
  Fix darwin run-test problems

Fix darwin run-test problems

Bug: 17428908
Change-Id: Id74a0383fdae6753558e27cd447c1ca349819c91

Temporarily disable LOS on 64 bit

Problem is that MAP_32BIT simulation can take seconds on large object
allocations in the worst case.

Bug: 17414549

Change-Id: I463bb0b728eed14847a32a27a93640ec6080f6b3

am 1e25d938: Merge "Add fast path for FindClass using the type dex file." into lmp-dev

* commit '1e25d9386c7e94761f1eda38e2d699192a915e35':
  Add fast path for FindClass using the type dex file.

Merge "Add fast path for FindClass using the type dex file." into lmp-dev

am dd66899f: Merge "ART: Change ART for new native bridge sequence" into lmp-dev

* commit 'dd66899f8d5ca4385937770694d2f2d9fb61ce8e':
  ART: Change ART for new native bridge sequence

Add fast path for FindClass using the type dex file.

If we are using the PathClassLoader with a BootClassLoader
parent, we can handle the common case in the FindClass function
without needing to go back to java code.

Around 10% speedup measured of maps launch, could be noise due to
large variation of app launch times. Eliminates defineClassNative
from being anywhere near the top of sampling profiles.

Bug: 17397179
Bug: 16828525
Change-Id: Ide0db2b5f6cf5b96fc46e89178e0799de667cb88

Merge "ART: Change ART for new native bridge sequence" into lmp-dev

am c0e1fa73: Merge "Add test for hashCode and System.identityHashCode." into lmp-dev

* commit 'c0e1fa7382b2e8e64da07181aff316dbcc84c395':
  Add test for hashCode and System.identityHashCode.

Merge "Add test for hashCode and System.identityHashCode." into lmp-dev

am b087849f: Disable tracing if enabled by default for new tracing test.

* commit 'b087849f5bbbf25bf5b03cf6172d2a50c08119e9':
  Disable tracing if enabled by default for new tracing test.

Disable tracing if enabled by default for new tracing test.

Bug: 17412385
Change-Id: If963a9ba2214c72efbef673a840bbeb6fdbb1795

am afb7de60: Merge "Change when sampling thread is reset during shutdown to prevent races." into lmp-dev

* commit 'afb7de600066cc5daba0112eacbfdb43f1fb9f16':
  Change when sampling thread is reset during shutdown to prevent races.

Merge "Change when sampling thread is reset during shutdown to prevent races." into lmp-dev

Change when sampling thread is reset during shutdown to prevent races.

Also adds some extra argument checking and testing for tracing.

Bug: 17412385
Change-Id: I876378d5bd17d75ab346584713b04dc0656d2120

am f81bfa3e: Ignore heap trim requests if we are the zygote

* commit 'f81bfa3e21c8a4e031505470ea954c93949cb30b':
  Ignore heap trim requests if we are the zygote

am 4d3981c7: Revert "Fix heap trimmer daemon sleeping."

* commit '4d3981c72a546b87c8f5d391ff8b5135b8ca99a7':
  Revert "Fix heap trimmer daemon sleeping."

Add test for hashCode and System.identityHashCode.

(cherry picked from commit bda2836af470c45e30c567b14d6747a7450c4741)

Change-Id: Ieba8f8eb09441ec145a176df00c293695eae0f47

Ignore heap trim requests if we are the zygote

Done to prevent app launching lag due to sleep in heap trimmer
daemon.

Bug: 17310019

Change-Id: Ia593e7eced1c1583771985ec9e7b60ee0c0e7912

Revert "Fix heap trimmer daemon sleeping."

This reverts commit 698b44634396e8111e96e1116586fde89a838347.

Change-Id: Ifa8c2d554b82289cc31803ad6027735d2c3c9be8

ART: Fix unchecked register index validity

The static check of index validity is against the type given by
the instruction, e.g., boolean for SPUT_BOOLEAN, but the target_type
is the resolved field type and can differ. An additional check is
necessary to avoid a read out of bounds.

Bug: 17411109
Change-Id: Ie4ed8bbda79f3f6403a24e727450a943447aa71d

ART: Change ART for new native bridge sequence

Initialize or unload after a fork.

(cherry picked from commit 6be67eeedbe60afce42300ae3e7f0e7180a96efa)

Change-Id: I5a20de1cb68dd1802937b369b14c50c9c1031c67

VisitClassesWithoutClassesLock isn't safe if classes move.

Which they do, so avoid by doing an array allocation.
Also, tidy member variables to the end of ClassLinker.
Remove unnecessary mutable. Tidy and fix a locks required/excluded.

Change-Id: I2404a9e7a1ea997d68ab1206f97d2a20dffbda06
(cherry picked from commit dbf3be0f133c0bdf454f637fee2452dbb5f7c027)

Fix monitor visit for class initializer

We do not hold class object's lock when invoking class initializer. Therefore
it is wrong to visit it when looking for monitors owned by a thread.

Bug: 17377924

Change-Id: I7fddcdb9a554301e040c571971eec9be6cba5eda

am 2c63eee3: Merge "Fix numerous issues with DdmVmInternal allocation tracking" into lmp-dev

* commit '2c63eee334bca18536060eea2e4a2f7883ef8eb9':
  Fix numerous issues with DdmVmInternal allocation tracking

Merge "Fix numerous issues with DdmVmInternal allocation tracking" into lmp-dev

Fix numerous issues with DdmVmInternal allocation tracking

Issues addressed:
- Using without JDWP attached caused native crash.
- When buffer is full (64k entries), number of entries reported was 0.
- Disabling tracking after disabling tracking caused native crash.
- Asking for allocations after disabled caused native crash.
- Lock ordering issues between mutator lock and alloc tracker lock.

Adding 098-ddmc test to cover these cases.

Bug: 17392248
Change-Id: I45c0abbfb3dd7a7421aea3bbba2ea8d05225e48b

am b100ff4f: Merge "Avoid recomputing the dex checksum during class loading" into lmp-dev

* commit 'b100ff4f21e034172be9669b18851829bb51e567':
  Avoid recomputing the dex checksum during class loading

Merge "Avoid recomputing the dex checksum during class loading" into lmp-dev

Avoid recomputing the dex checksum during class loading

Thread the already computed checksum to VerifyOatAndDexFileChecksums and
LoadMultiDexFilesFromOatFile to avoid recomputing it.

Bug:17346103
Change-Id: Ifa0c1cad952853751e98cbb3c999631b9909a9f9

am cb7c0e9e: Ensure alternate signal stack is minimum size.

* commit 'cb7c0e9eb785a8d605da5d5b26ab265d4d635f1e':
  Ensure alternate signal stack is minimum size.

Ensure alternate signal stack is minimum size.

On Mac a sigaltstack may need to be 32KB, setting it to 16KB causes
sigaltstack to fail.

(cherry picked from commit c24a1e00b13a7ebf10dd896f6a18b1ad09617ec7)

Change-Id: Ic06d80f1acf5c4eb3d2dd1156bfa2805ab0df492

am bbcd0a75: Merge "Fix heap trimmer daemon sleeping." into lmp-dev

* commit 'bbcd0a75324dd9554109d91f2e9f7ae5d4c4a28a':
  Fix heap trimmer daemon sleeping.

Merge "Fix heap trimmer daemon sleeping." into lmp-dev

Fix heap trimmer daemon sleeping.

Problem:
Heap trimmer daemon sleeping caused app launch occasionally to take a
while due stopping the heap trimming thread taking a long time.

The heap trimmer thread now never sleeps, we prevent issues caused
by frequent back and forth process state changes by only
transitioning to background if it has been kCollectorTransitionWait
time since the last transition to background. Similar logic for heap
trimming.

Bug: 17310019

Change-Id: I23980421cc388f36b66a4c03ed15dd11d43f59b5

am 57f2bbda: ART: Fix x86_64 GenSelect case when destination is Ref

* commit '57f2bbda741d2d3d769a50268b4e0b9b54959684':
  ART: Fix x86_64 GenSelect case when destination is Ref

ART: Fix x86_64 GenSelect case when destination is Ref

Reference in x86_64 is a 64-bit solo register. As a result, the invocation
  of OpRegImm results in an error when Select opcode of the kind:
    ref = boolean ? null : null;
because opRegImm does not support 64-bit destination for OpMov.

The case above is only possible for ref because no one other constant except
  null is possible.

Bug: 17327895
Change-Id: I7541e744ec1c8619711712fd17be72764efcf3a8
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>
(cherry picked from commit 3f51e7d942c22edaab3a7e703a1e6a2dd6a26f77)

am 607dcd8a: Merge "ART: Fix computation of frame size for direct proxy methods" into lmp-dev

* commit '607dcd8a2aca4e44a564b7839c285315230545fd':
  ART: Fix computation of frame size for direct proxy methods

Merge "ART: Fix computation of frame size for direct proxy methods" into lmp-dev