Revert "Revert "Skip DumpForSigQuit if debugger is active""

Bug: 26118154

This reverts commit 539f7fb0964a184b24bf40c729577be76dd7749a.

Revert "Skip DumpForSigQuit if debugger is active"

Bug: 26118154

This reverts commit 0b8f1bfdfc721a41d98d13e12c4c67f62f698dfc.

(cherry picked from commit 6017782485d30c4e440e53107dde03dc20d56a9a)

Change-Id: I783dac75b97a424c2d770e0d69e6f346c915a754

Merge "Use collector specific helper classes"

Use collector specific helper classes

Changed to use inner classes. Also changed some visitors to lambdas.

Bug: 29413717
Bug: 19534862

Change-Id: I631c8bfe5f795eda4623c5bb4f357f2dd12358e2

Merge "ARM: Improve String.getCharsNoCheck intrinsic."

ARM: Improve String.getCharsNoCheck intrinsic.

- Optimize for short retrievals.
- Improve performance of longer retrievals with unrolled LDR/STR.

Change-Id: I24ec52067104b2f68799c04aebb61d20fff94020

Merge "Change IMT size from 64 to 43"

Change IMT size from 64 to 43

Data shows that this change can save around 2MB with an estimated
less than 2% regression in interface method invokation speed.

Further investigation is needed to measure the performance impact
in practice, particularly for much smaller IMT sizes. We are making
this initial change to 43 to see if it affects runtime perf benchmarks.

Change-Id: Ic0e036091c5da0e6b93f519fc695e9b48ab5ec46

Merge "For deadlock test, print out any unexpected exceptions."

For deadlock test, print out any unexpected exceptions.

Bug: 28663029
Change-Id: Icd10cabdde282f292fd997c3a8ad9dd6d1a14e0e

Merge changes I7d3ba051,I2dc96535

* changes:
  ART: Fix ImageSpace::Init
  Revert "Reduce ImageSpace::Init frame size"

Merge "Avoid duplicate class for mark compact"

Avoid duplicate class for mark compact

Renamed BitmapSetSlowPathVisitor to MCBitmapSetSlowPathVisitor for
the mark compact case.

Bug: 29413717
Change-Id: Ibb85acc8e796c0b8208b13c468b5510b0cec47e1

Merge "Ensure we get a ProfilingInfo object before compiling."

Ensure we get a ProfilingInfo object before compiling.

Otherwise, the compiler could decide not to JIT compile
the method.

Change-Id: I6e92224748861317c5ac629395ea180df4e51f8b

Merge "MIPS32: Assembler tests for MIPS32R6"

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

ART: Fix ImageSpace::Init

Refactor ImageSpace::Init. Fix a case of using nullptr for error_msg
where it was unsafe. Instead fix the remaining case of verbose map
dumping.

Also fixes the stack frame size issue on x86-64.

Change-Id: I7d3ba05100592b5a44a29b34349edd8a44b4a72f

Merge "Propagate flags to ShouldHaveEmbeddedVTable"

Propagate flags to ShouldHaveEmbeddedVTable

Pass kVerifyFlags and kReadBarrierrOption to ShouldHaveEmbeddedVTable
from ShouldHaveImt.

Bug: 29188168

Change-Id: Ibddd6e744a2f313b4048719987c41f2da04a4e0a

Revert "Reduce ImageSpace::Init frame size"

This reverts commit a67a49a9758c2055cb544bfecdbb50dba2c502d1.

Change is unnecessary, regular cleanup gives us the improvement.

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

MIPS32: Assembler tests for MIPS32R6

Change-Id: Iee3f4447a6182a769490b3235abeea9551111193

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