Fix ARM & ARM64 UnsafeGetObject intrinsics with read barriers.

The implementation was incorrectly interpreting the 'offset'
input as an index in a (4-byte) object reference array,
whereas it is a (1-byte) offset to an object reference field
within the 'base' (object) input.

Bug: 29516905
Change-Id: I4da5be0193217965f25e5d141c242592dea6ffe8
Test: Covered by test/004-UnsafeTest.

Fix x86 & x86-64 UnsafeGetObject intrinsics with read barriers.

The implementation was incorrectly interpreting the 'offset'
input as an index in a (4-byte) object reference array,
whereas it is a (1-byte) offset to an object reference field
within the 'base' (object) input.

Bug: 29516905
Change-Id: Idfbead8289222b55069816a81284401eff791e85
Test: Covered by test/004-UnsafeTest.

Merge "Use a barrier instead of sleep for flaky deadlock test."

Merge "Propagate flags to ShouldHaveEmbeddedVTable"

Propagate flags to ShouldHaveEmbeddedVTable

Pass kVerifyFlags and kReadBarrierrOption to ShouldHaveEmbeddedVTable
from ShouldHaveImt.

Bug: 29188168

Change-Id: Ibddd6e744a2f313b4048719987c41f2da04a4e0a

Use a barrier instead of sleep for flaky deadlock test.

Bug: 28663029
Change-Id: Ia94372c719f8014b4ee739ebc6a93c4f3548f717

Merge "Refactor GetIMTIndex"

Merge "Reduce ImageSpace::Init frame size"

Reduce ImageSpace::Init frame size

Fixes x86_64 build.

Change-Id: Ib636a1274e255a60bda47607570fedbe92a5e2cd

Merge "Optimize IMT"

Refactor GetIMTIndex

This allows us to more easily maintain and experiment with
interface method table indexing and hashing.

Change-Id: I719920fae7490dcedcda7c1c36db225c2b8b16df

Merge "Also compile libnetd_client for target testing."

Merge "Revert "Revert "Disable 149-suspend-all-stress, its output is flaky."""

Revert "Revert "Disable 149-suspend-all-stress, its output is flaky.""

Still flaky.

bug:28988206

This reverts commit 06cb9b25e09c10530ba4f3659b54161b1c76c932.

Change-Id: I13626a8959db27fd566f6db56fab469312422e70

Merge "ARM: Use GOT_PREL references for Runtime::Current()."

Merge "Visit invalid roots of only suspended threads"

Merge "ART: Empty out sa_mask in ArmInstructionSetFeatures"

Visit invalid roots of only suspended threads

Since this always happens with suspended threads or self, you can
just visit these threads and do not require a suspend all. This
will not miss any roots if the caller was marking a thread root.

Fixes issues like transitioning to suspended and back blocking on a
thread suspension request from another thread. This could cause
deadlocks previously.

Bug: 29062271

Change-Id: I2fef149387aacf0cdc9a773d4f172c42fa53e4dc

ART: Empty out sa_mask in ArmInstructionSetFeatures

For sdiv hardware detection by signal, empty out sa_mask.

Bug: 29282211
Change-Id: I7c85273f95089fcce75319843e5921842ea6df4d

Merge "Change test 149 to run for 10 seconds"

Merge "ART: Add setenv to Valgrind target suppressions"

ART: Add setenv to Valgrind target suppressions

Setenv is known-leaking when overwriting existing keys. Ignore.

Bug: 29282211
Change-Id: I4af0bf4d33b9e3d8352f7219710882215def09f1

Merge "ART: Workaround for ARM valgrind stack issues"

Merge "Revert "Revert "Revert "Revert "Revert "Revert some flaky unloading""""""

Change test 149 to run for 10 seconds

Run for a fixed 10 seconds to prevent timeouts on overloaded
machines.

Bug: 28988206

Change-Id: Ia40b8e61445ca0a6331668dc605401d1f69e4d17

ARM: Use GOT_PREL references for Runtime::Current().

Bug: 29259539
Change-Id: I5039ff016403d438f496a55d4bb1775c3d67c09c

ART: Workaround for ARM valgrind stack issues

The reports from pthreads and the builtins don't match up (compiled
code and thus actual stack pointer values haven't been tested, yet).
As a temporary workaround, implicitly disable the checks by storing
0x0 as the stack base.

Bug: 19127663
Bug: 29282211
Change-Id: Ib4170e222ca7e2ef92bf125b0a7409770ded7822

Merge "Pass the right CFLAGS to libarttest(d)."

Optimize IMT

* Remove IMT for classes which do not implement interfaces
* Remove IMT for array classes
* Share same IMT

Saved memory (measured on hammerhead):
boot.art:
Total number of classes: 3854
Number of affected classes: 1637
Saved memory: 409kB

Chrome (excluding classes in boot.art):
Total number of classes: 2409
Number of affected classes: 1259
Saved memory: 314kB

Google Maps (excluding classes in boot.art):
Total number of classes: 6988
Number of affected classes: 2574
Saved memory: 643kB

Performance regression on benchmarks/InvokeInterface.java benchmark
(measured timeCall10Interface)
1st launch: 9.6%
2nd launch: 6.8%

Change-Id: If07e45390014a6ee8f3c1c4ca095b43046f0871f

Merge "ARM64: Improve String.getCharsNoCheck intrinsic."

Merge "Revert "Disable 149-suspend-all-stress, its output is flaky.""

Merge "Support unlimited pending checkpoints"

Support unlimited pending checkpoints

Prevents the spinning that used to happen if RunCheckpoint was called
with 3 pending checkpoints. This spinning was done when holding
thread_list_lock_ and thread_suspend_count_lock_ and could deadlock
if any of the pending checkpoints required any of these locks.

The fix is to use an overflow list instead of having a fixed limit of
3.

Changed suspend stress test to have more threads and only compare last
line since there may be libbacktrace spam like:
"+E/libbacktrace(69891): void SignalHandler(int, siginfo_t *, void *):
Timed out waiting for unwind thread to indicate it completed."

Bug: 28988206

Change-Id: I2ae611506147d5199d59a08eee0395f7fa35d448

Merge "Propagate the read barrier option to Class::VisitNativeRoots."

Propagate the read barrier option to Class::VisitNativeRoots.

Propagate the read barrier option from Object::VisitReferences to
Class::VisitNativeRoots.

This is a step toward the GC thread avoiding graying objects (and
reducing dirty pages) in the immune spaces.

Bug: 12687968

Change-Id: I29c4126a4ad4c40e63a934e62451fb3fb36aad43

Revert "Disable 149-suspend-all-stress, its output is flaky."

This reverts commit e7aa9dd3c2f7156a4ad186dae93a3103f6eace81.

Merge "MIPS32: Improve offset calculations in loads and stores"

Merge "ARM64: Use VIXL's conditional select helper."

Revert "Revert "Revert "Revert "Revert "Revert some flaky unloading"""""

With go/aog/238641 this should pass on the buildbots again.

Bug: 28406866

This reverts commit 7dbf20abe99eb6c1d9c137e592c2025af725fa06.

Change-Id: I1038fe454491488fd561f5fce2e51a27487dce20

ARM64: Use VIXL's conditional select helper.

Change-Id: Id6bb880e2fffb54cf1f480191fc734eaaf4cd293

Also compile libnetd_client for target testing.

The runtime linker will load the one on /system otherwise.

Change-Id: Id57e84d1b34e807437818d39bf6ef5f727a7d210

Pass the right CFLAGS to libarttest(d).

bug:28406866
Change-Id: If063bfda90786f58e2dba68fe1b96a668e7db177

Merge "Remove too aggressive DCHECKs."

ARM64: Improve String.getCharsNoCheck intrinsic.

- Optimize for < 8 characters.
- Improve performance of longer retrievals with LDP/STP.

Change-Id: Ib9a9507ae652a439c7fdcab5cc916e1930722228

Remove too aggressive DCHECKs.

A class can move from a state greater or equal than resolved
to erroneous concurrently to the verifier or the compiler.

bug:29239283
Change-Id: I89f3fe1c1d9556c6c99b8e005b3ec02de7f01b85
(cherry picked from commit f7d994622aabcc689f62253a9253e0c67d9e787e)

Merge "Disable 149-suspend-all-stress, its output is flaky."

Disable 149-suspend-all-stress, its output is flaky.

bug:28988206
Change-Id: I13cce14b72d753c9ba2ca9b4e256eb571620caa9

Merge "Revert "ART: Disable check under memory tools""

Revert "ART: Disable check under memory tools"

This reverts commit 820c13bea5795e04986bf617319517562de76514.

The check is valid, the whole stack setup is broken.

Merge "ART: Suppress memcpy overlap in jemalloc for valgrind"

ART: Suppress memcpy overlap in jemalloc for valgrind

Clang/LLVM uses memcpy for a self-assign, which is detected by
valgrind. Ignore.

Bug: 29279679
Change-Id: I47483e1a2a982b9f3642f98e4389633989885bb8

Merge "Use ScopedObjectAccess in ThreadList::Dump"

Use ScopedObjectAccess in ThreadList::Dump

Prevent deadlocks if two different threads are calling Dump at the
same time:

Thread 1: Requests thread 2 to suspend
Thread 2: Requests thread 1 to suspend
Both threads are suspended and blocked on ScopedObjectAccess in
thread dumping.

Added to suspend all stress, reduced thread count to 4 temporarily
since this is the maximum number of simultaneous checkpoints.

Bug: 28988206

Change-Id: I9e4b8391ebad0bca0e42a03819c58c99ddba6b35

Merge "ART: Disable check under memory tools"

Merge "Fix the region space name for dumpsys meminfo."

Merge "Disable the CC collector / read barrier checks in non-debug build."

ART: Disable check under memory tools

Disable a stack check under memory tools. It assumes a single
common stack, which seems violated at least under valgrind on
ARM.

Change-Id: Ida05adb0728cc46f3bbda4d34be7a1623bb3bc82

Fix the region space name for dumpsys meminfo.

Bug: 12687968
Change-Id: I42e3d4cd9afb9892eb1d173b18f8b3e9491429dd

MIPS32: Improve offset calculations in loads and stores

Change-Id: I6c3773e8bc1233bcda83d5b7254438ef69e9570d

Disable the CC collector / read barrier checks in non-debug build.

Bug: 12687968

Change-Id: Ia8295354b705018ffa864eb8101aa5c09528af13

Merge "ART: Fix swap usage determination in dex2oat"

ART: Fix swap usage determination in dex2oat

The decision needs to be postponed until we have access to the
dex files.

Bug: 27878043

(cherry picked from commit ce55f0d4da1db20676273032b3c79624f665b35b)

Change-Id: I55833c6ea827ec7a322711bd182fc723d1727f08

Merge "Fix dex file leak in oat file manager"

Fix dex file leak in oat file manager

Simplified ownership by having a vector of unique pointers own
all newly opened dex files.

Bug: 29246280

(cherry picked from commit fed316715faeec7bf34e3c4a878288c1342cb0e8)

Change-Id: I97db09ced76db8ffdbae371ff72977c4276a0494

Merge "ART: Add ATRACE section for collision check"

ART: Add ATRACE section for collision check

The collision check for duplicate classes is potentially expensive.
Add a ScopedTrace to mark it.

Bug: 26880306
Change-Id: Icd75bd0ccc15f0d1f321d85382a4d038fa8ab049

Merge "Add Valgrind target tests"

Add Valgrind target tests

Change-Id: I1251ac2122c60ed5fcabf184704bc439243ee85b

Merge "Fix some "possible" divide by 0"

Fix some "possible" divide by 0

Bug: 28529431
Change-Id: I61f638926b2ae63c5f883fc2cfdce19b00ce79c8

Merge "Add thread suspend stress"

Merge "Re-introduce an old version of 600-verifier-fails."

Re-introduce an old version of 600-verifier-fails.

The fugu buildbot had a crash on this version:
https://build.chromium.org/p/client.art/builders/fugu-ndebug/builds/3844

It got lost after updates to the tests.

Change-Id: Ib3748e6d6772e90bbc56b8139e374c7cf8953ce4

Merge "Ensure we have a profiling info object before trying to compile."

Ensure we have a profiling info object before trying to compile.

Otherwise we will never succeed compiling and infinite loop.

Change-Id: I7ff09c69e67f0df4cb450ffa10a529625eac67fd

Merge "Also catch OOME from the new byte[]."

Also catch OOME from the new byte[].

Change-Id: Ic756070ed4a3cb3760760fd5a2ebcad1ea0a8ca0

Merge "Fix FOUR_ARG_DOWNCALL assembly stubs on arm and x86."

Merge "Use usleep instead of sleep(0)."

Fix FOUR_ARG_DOWNCALL assembly stubs on arm and x86.

They were creating a stack that the runtime did not understand.

bug:28348339
Change-Id: Ic03663552209beda8ff1e79db58bedc8f34d9a0e

Use usleep instead of sleep(0).

sleep(0) is unfortunately not specified.

Change-Id: I5e9c626d21a562ca04329c0cc039a1f3a71dfc84

Merge "Bump frame size limit 8 bytes to fix x86_64 build with latest external/libcxx"

Bump frame size limit 8 bytes to fix x86_64 build with latest external/libcxx

Change-Id: Ib8964d18aa509ee40a91d92eea1011447d429ed7

Merge "Revert "Revert "Revert "Revert "Revert some flaky unloading"""""

Add thread suspend stress

Seems to be passing, aim to catch bugs related to thread suspension.

Bug: 28988206

Change-Id: I48190900ccd7a2545c6f09cf730e995043b0d9b7

Revert "Revert "Revert "Revert "Revert some flaky unloading""""

Fugu is still unhappy

Bug: 28406866

This reverts commit 340f486aa0126facb67494449b5c2ee46a1a75e6.

Change-Id: I45fc77f924d991669d27b99c1458b2def8692664

Merge "Temporarily disable test E"

Merge "Add a few more pending exception checks"

Add a few more pending exception checks

Bug: 28371539

Change-Id: Ibd2f7ccf7bd8cf7ef6c24534be1efbcc9b30014a

Temporarily disable test E

Rationale:
Interpreter does not seem to perform access checks
prior to null check, a feature that was just fixed
in the AOT compiler. Disabling the test until we
figure out who is right.....

BUG=29068831

Change-Id: I2cdce568727e631f48648f401806e7ecd6985038

Merge "Revert "Revert "Revert "Revert some flaky unloading""""

Revert "Revert "Revert "Revert some flaky unloading"""

Ensure that all threads are in the JNI code before destroying the
runtime.

Bug: 28406866

This reverts commit b2716bbf321293ddbd0b96f0ea999794b0e9b09b.

Change-Id: I7f6189a9d0d8a1bf7af5535026804bc2183466c9

Merge "Do not place null check on unresolved method calls."

Do not place null check on unresolved method calls.

Rationale:
These invokes drop through the runtime anyway where various
checks are done, including null check. A few of these
checks need to occur before the null check.

With fail-before/pass-after smali test.

BUG=29068831

Change-Id: I260715e742365433a323598d97f7fdab321e8512

Merge "Use sleep instead of sched_yield."

Use sleep instead of sched_yield.

The compiler thread has a different priority than the main thread,
so there's no guarantee sched_yield will actually pass the CPU to
the compiler thread.

Change-Id: I7e8d3cbf478c482a9ba9502f7bba7fb1397cd28e

Merge "Be pro-active into ensuring a method is JITted in test."

Be pro-active into ensuring a method is JITted in test.

Change-Id: I27d7bb76178478e6153e87a076bb39a4dc61f50b

Merge "Wrap certain exception types when loading an erroneous class."

Wrap certain exception types when loading an erroneous class.

Bug: 28787733

(cherry-picked from commit 7c8aa8357196781c811a73d2eb66aaaa1681ce36)

Change-Id: Iea55486c4b95ee16e1f19c8ba2d24c18b9100c97