Merge "Fix obsolete method change check."
am: 494505a9de

Change-Id: Ie3dac35eb222fb3662b2588576ca71c7a029495a

Merge "Fix obsolete method change check."

Fix obsolete method change check.

We were incorrectly checking for changes in the obsolete-method map.
This could cause issues if a class was redefined multiple times.

Bug: 37475600

Test: ./test.py --host -j40
Test: cts-tradefed run cts-dev --module CtsJvmtiRedefineClassesHostTestCases

Change-Id: Icf39d5154c0e48461405b700bf1fa20830195fc1

Merge "Make ART build/test configurations use the CC collector by default."
am: 741a81af44

Change-Id: Ica953a72f63c07fb551b75b2f0448bb1212d792f

Merge "Differentiate between native alloc and normal background GC"
am: e4e7d601ef

Change-Id: I26f9a501d9da8b5d9cdefc08ae1a4fa394a533b6

Merge "ART: Make less lock-level noise on abort"
am: 2888e85164

Change-Id: I58b17056982de8e306ab6e0dfef430f74b5ec11f

Merge "Make ART build/test configurations use the CC collector by default."

Merge "Differentiate between native alloc and normal background GC"

Merge "ART: Make less lock-level noise on abort"

Differentiate between native alloc and normal background GC

Added a new GC cause kGcCauseForNativeAllocBackground.

Bug: 35872915

Test: test-art-host

Change-Id: I94e17f8bd53af29f2862b9910bd8abd2df97e229

Merge "Use correct type for GetValueFromShadowFrame"
am: c36a970fcd

Change-Id: I7ae578e671658c4790aaaf67aac22adfded9b5df

Merge "Use WARN_UNUSED consistently in ART."
am: b2410c931a

Change-Id: I9bc56ed10c7c4b7d72b89f5cfc95ac964b3e4138

Merge "Use correct type for GetValueFromShadowFrame"

Merge "Use WARN_UNUSED consistently in ART."

Use correct type for GetValueFromShadowFrame

The field type is not necessarily the input type for boxed
primitives. If the field type is < 32 bits, it means there will be
partial object pointer in the JValue. If a conversion check is
later needed in GetUnboxedTypeAndValue, it will crash. The fix is
to use the PTypes.

Bug: 37446461
Test: test-art-host
Change-Id: I0c4b405f0c13910523b98a87ef12b9f302a5e241

Merge "JDWP: fix Dbg::ResumeThread"
am: 2203253254

Change-Id: Ie4f5a2ce448d182eb306b59f718a52a86b999bf7

Merge "JDWP: fix Dbg::ResumeThread"

Merge "ART: Call ThreadGroup.add in Thread::FinishStartup"
am: 2aac653df7

Change-Id: Ia457840ddc3c96eb3271dde5e210db26df404a58

Merge "ART: Call ThreadGroup.add in Thread::FinishStartup"

Merge "Fix ARM64 SystemArrayCopy intrinsic with large constant dest position."
am: 1cae72e6cf

Change-Id: Id814f38204ec28bb5d84e5ec772d5caf79a7e274

Merge "Fix ARM64 SystemArrayCopy intrinsic with large constant dest position."

Merge "Revert "Disable 160-read-barrier-stress temporarily""
am: e58b971b57

Change-Id: I288940985c066629dd5ab2e9312997b89055098f

Merge "Revert "Disable 160-read-barrier-stress temporarily""

Merge "Fix Arena allocation tracking."
am: 4cb99a13d2

Change-Id: I4353655032f22837c55d4d221619126210b8aa47

Merge "Fix Arena allocation tracking."

Use WARN_UNUSED consistently in ART.

Test: mmma art
Change-Id: I776840725f56258c45edfcfd165da61ab653f0b5

Fix ARM64 SystemArrayCopy intrinsic with large constant dest position.

Make sure we do not deplete the whole VIXL scratch register pool, so
that VIXL can still use IP0 as a temporary when emitting
macro-instructions.

Test: art/test/testrunner/testrunner.py --optimizing --target --64
Bug: 37256530
Change-Id: I5da22e552297fad87db5763e2dab60ae6a7a43af

Revert "Disable 160-read-barrier-stress temporarily"

And reduce the number of fields in class ManyFields from
10000 to 5000 to reduce memory pressure on Jack. Split
the ManyFields class in a simple hierarchy to keep files
under 64KiB and allow compilation with javac.

Test: testrunner.py --host -j 48
Test: run-test --host --jvm 160-read-barrier-stress
Bug: 37335480

This reverts commit 101074999bd339c276eeaaf0538930d90a96657e.

Change-Id: I52217fd2c9d00a5b045a7faa6e4dc1438c7f93b1

ART: Make less lock-level noise on abort

The lock-level violations with the abort lock aren't really all
that interesting.

Test: m test-art-host
Change-Id: I8a5fc687009db914ec8f60d86068d87e71f8a894

ART: Call ThreadGroup.add in Thread::FinishStartup

ART should add the main thread to the main ThreadGroup. Behavior
of the Thread constructor changed.

Bug: 37444210
Test: art/test/testrunner/testrunner.py -b --host -t 051
Test: m test-art-host
Test: m build-art-host && art/tools/run-libcore-tests.sh --mode=host
Change-Id: I92cf2f9a6c5c3fdf385eb7925addc38b64fa4d98

Merge "sigchain: switch from __thread to pthread_setspecific."
am: 36831abc29

Change-Id: Ie5567b3889bf9b2af37ef2e88296f392d7f86073

Merge "sigchain: switch from __thread to pthread_setspecific."

sigchain: switch from __thread to pthread_setspecific.

__thread is implemented via emutls on Android, which will result in the
thread local variable being allocated again and leaked if it gets used
after it has been destructed already (e.g. by a later destructor
calling sigprocmask, or triggering a signal handler).

Switch to pthread_setspecific, which doesn't suffer from this problem.

Bug: http://b/36871013
Test: ran dalvikvm on a class that spins creating threads
Change-Id: Ie5deab453be387490ba30a0010e12f60d736c8ad

Merge "Enable some redefinition run-tests in CTS"
am: 66f43b988a

Change-Id: Ic0db769c4d17ed3996ac653c77fb136ed5278c57

Merge "ART: Change run-test 913 array testing"
am: 030b1066a6

Change-Id: I5848a08931e0ddb7124c0d11facdd581f7129b5d

Merge "Enable some redefinition run-tests in CTS"

Merge "ART: Change run-test 913 array testing"

Merge "Disable read barrier thunks for heap poisoning"
am: 3ace1f0ce2

Change-Id: I5f0e3606cb05968ef517ebf5f99d1d87e84babef

Merge "Print runtime isa when invoking 'dalvikvm --showversion'"
am: 445452cda0

Change-Id: I131eca067fb6e0bd8bb05c319785e005dc743e45

Merge "Disable read barrier thunks for heap poisoning"

Merge "Print runtime isa when invoking 'dalvikvm --showversion'"

Enable some redefinition run-tests in CTS

We added the tests to run-test-jvmti-java-library
target and made some small changes to tests to
ensure that everything works without linking to
libart.so.

Bug: 32072923
Test: ./test.py --host -j40
Change-Id: I8fdad180c77b125c22c043cd61305cee79801cc7

ART: Change run-test 913 array testing

Use an explicit array in the test. Ignore all arrays that are
untagged.

Bug: 32072923
Bug: 36727422
Test: m
Test: art/test.py --host -r -t 913
Change-Id: Idf7bc03ae81e3e645935c8809bf59636cfa3784a

Disable read barrier thunks for heap poisoning

Logic to unpoison is not yet implemented, this causes SIGSEGVs in
various places.

Bug: 29516974
Bug: 30126666
Bug: 36141117

Test: ./test/testrunner/testrunner.py -j4 --optimizing --debuggable --ndebuggable --target --verbose
Change-Id: I8317a142d07af36090d5f05ce46100dfa07c17e7

Merge "Use standard loading code in many tests"
am: 4bb9a2b0b2

Change-Id: Iecc67c50bb772624803149681bb89e8a6e672b7f

Merge "Use standard loading code in many tests"

Merge "Remove native code from test 944 for CTS"
am: 4a4e245a63

Change-Id: Ide22ef4ab749731763334593bdf923358cc0542d

Merge "Remove native code from test 944 for CTS"

Use standard loading code in many tests

In order to prepare for use in CTS tests make tests 902, 914, 915,
917, 919, 926, 930, 932, 940, 942, 945, 981 & 982 use standard OnLoad
functions. Final configuration is done during test initialization.

Bug: 32072923
Test: ./test/testrunner/testrunner.py --host -j40
Change-Id: I84b2e17d0e024255d2dc49452f3d74dfd64abd22

Merge "Fixed bug on pending environment use of termination condition. With regression test."
am: 27fb1dc467

Change-Id: I606f6dadc4705179e501cebb90e47582387ad3f4

Merge "Fixed bug on pending environment use of termination condition. With regression test."

Merge "Add art_defaults to dexlayout"
am: 35fc819cd9

Change-Id: Iaa201535063cac651911a0cf9650ddd8592e2915

Merge "Add art_defaults to dexlayout"

Remove native code from test 944 for CTS

Replaced native code in test 944 with java reflection based code.

Bug: 32072923
Test: ./test/testrunner/testrunner.py --host -j40 -t 944
Change-Id: I678d805936009a33ce8fe6b8570bfb788a8da72d

Add art_defaults to dexlayout

The build rule defaults for dexlayout targets didn't include defaults
from the ART project.  Because of this ART_TARGET_* weren't defined,
among other problems

Bug: 35800981
Test: make -j 40 test-art-host
Change-Id: Ic528bfc6b622fa9b4f5c0ddf883f15912641b720

Merge "ART: Tentative fix for stack count issue in test 913"
am: 8af5efdb1d

Change-Id: I0b902d209cb2381a3c3cee6508ff39cd0bd41983

Merge "ART: Tentative fix for stack count issue in test 913"

ART: Tentative fix for stack count issue in test 913

Use kCountTransition == kNo to also skip runtime methods in the
cur_depth_ count of a StackVisitor.

Currently this is only used in the VisitRoots function of Thread.
If the additional load is too expensive, we can make this dependent
on the requested precision (and thus revert "regular" visits to
the old scheme of also counting transitions between managed stack
frames and shadow frames, which is only an increment of a field).

Bug: 31385354
Bug: 32072923
Test: m test-art-host
Change-Id: Ief951585407dd6e03f8050774fe612e7b204247e

Merge "ART: Add debug code to run-test 913"
am: 39d562895a

Change-Id: I4a40720e1c97fd0e80c5945bf370bcaec908c652

Merge "ART: Add debug code to run-test 913"

Merge changes I5b05e23a,I9350b250,I1d66b42a
am: 393a1fd02f

Change-Id: If6e778443603c658483cf94d73ff9bec9bd4297f

Merge changes I5b05e23a,I9350b250,I1d66b42a

* changes:
  More refactor for JVMTI redefinition run tests (3/3)
  More refactor for JVMTI redefinition run tests (2/3)
  More refactor for JVMTI redefinition run tests (1/3)

Merge "Add check that TypeIndex is valid in StringByTypeIdx."
am: 1cda71a414

Change-Id: I505a522a49d67bd0bc42b1b2df2731da818f8700

Merge "Add check that TypeIndex is valid in StringByTypeIdx."

Add check that TypeIndex is valid in StringByTypeIdx.

StringByTypeIdx should fail gracefully if given a bad TypeIndex. This
adds a check that the TypeIndex is valid before getting its TypeId.

This fixes a regression that removed this check when it was refactored
in this CL: https://android-review.googlesource.com/#/c/243493/

Bug: 37287051
Test: mm -j31 test-art-host-gtest-dex_file_test
Change-Id: Ib68cb8135011f5f30335251583e181b089982754

Print runtime isa when invoking 'dalvikvm --showversion'

The runtime ISA is appended at the end, after the version number.

Test: dalvikvm --showversion
Bug: 36824842
Change-Id: I09bbe3bf06054fe237543b5f96e9480631ea0838

Merge "Remove test skips associated with b/37240685"
am: 452092b4c9

Change-Id: I91a4ac9f600e4cf6f648f816bce68a002b55c17d

Merge "Remove test skips associated with b/37240685"

More refactor for JVMTI redefinition run tests (3/3)

This adds in the Main.java files for the modified tests.

By doing the 3 CLs we keep git from getting confused about the history
of the files.

Bug: 32072923
Test: ./test.py --host -j40
Change-Id: I5b05e23aaf9e90101817ca01d1928a30f14cb7e5

ART: Add debug code to run-test 913

Try to figure out why we get wrong depth reporting by aborting in
a deep frame.

Bug: 32072923
Test: m
Test: art/test/testrunner/testrunner.py --host -t 913
Change-Id: Icea02ebdee0e5395a5935c58c80e6d7c82eb6c6d

More refactor for JVMTI redefinition run tests (2/3)

Move refactored tests into their final places.

NOTE: BROKEN WITHOUT A FOLLOW-UP CL TO ADD NEW MAIN.JAVA FILES.
Bug: 32072923
Test: None
Change-Id: I9350b2505815c5d21816df4189482aee4868c91c

More refactor for JVMTI redefinition run tests (1/3)

Does as much as possible without making git get confused about the
provenance of files. Follow up CLs will move the files into the
appropriate places.

NOTE: BROKEN WITHOUT A FOLLOW-UP CL TO CHANGE FILE NAMES!

Bug: 32072923
Test: None
Change-Id: I1d66b42a5041d4ea1a0e8eaa61d55422a507e26f

Fixed bug on pending environment use of termination condition.
With regression test.

Test: test-art-host
Bug: 37247891
Change-Id: I55b06939d465d3ddb736d1ba659b1df179a5c390

Merge "ART: Refactor run-test 913"
am: 8d7f78d63b

Change-Id: Ib122a0f2200efd554fadaeedef7eccff31e7cc99

Remove test skips associated with b/37240685

It's been fixed.

Test: ./test.py --host --jvmti-stress -j40
Change-Id: Ic1bb74dc62a080f54f93cec6fd75d748a9941653

Merge "ART: Refactor run-test 913"

Merge "Disable 160-read-barrier-stress temporarily"
am: c569d4a083

Change-Id: I900f2fd97f03f384f47ccff44adc3f06fc41f113

Merge "Disable 160-read-barrier-stress temporarily"

JDWP: fix Dbg::ResumeThread

A thread should be resumed if and only if it has been suspended by
the debugger. Therefore, we must check the debug suspend count, not
the suspend count (which includes debug suspend count). Otherwise we
could end up resuming a thread that has been suspended for another
reason (like a GC) but not by the debugger itself.

Bug: 27385848
Test: art/tools/run-jdwp-tests.sh --mode=host --variant=X64
Change-Id: I70ed6dfc233501e92332634650babf21ba911048

Disable 160-read-barrier-stress temporarily

This test is causing an OOME with Jack compiler. This CL disables it
temporarily until we find a fix.

Bug: 37335480
Test: make -j test-art-host-run-test
Change-Id: I580fc99fbec0b0c94c25256fe20b957e07106205

ART: Refactor run-test 913

Remove references to ART. Filter roots from the JIT. Canonicalize
some thread IDs. Move the test to its own thread, and filter
stack-locals of other threads.

These changes ensure that the test is less dependent on the main
thread and its environment, which is required to use it in CTS.

Also fix a reporting issue for roots.

Bug: 32072923
Test: art/test/testrunner/testrunner.py --host -t 913
Change-Id: I8480ba7751fb6420c256db87cba11b8a65e25ea5

Merge "ART: Use _exit in dex2oat"
am: c1be9f2b77

Change-Id: Icc4c5f98014866ffda5b513feb5ce40e4f43c52c

Merge "ART: Use _exit in dex2oat"

Merge "Remove native printing from test 980 and reliance on print internals"
am: 9fdafe8fe4

Change-Id: I2efab79eb983dae090de59d792ec875221d0fbec

Merge "Remove native printing from test 980 and reliance on print internals"

Merge "Add check that map offset is sane to DexFile object init."
am: d58d0841b4

Change-Id: I233766259a0dcecbe497e6a34e733066fdbd41d7

Merge "Various debug ART GC performance improvements"
am: 704f16f786

Change-Id: If496b54ff9dc9c487c10d3947ad75a603aa967ec

Merge "Add check that dexlayout can open output file."
am: 52e3e57488

Change-Id: I7e40bcd7b6fb3603d426f2d14cd467c05d47a6c5

Merge "Add check that map offset is sane to DexFile object init."

Merge "Various debug ART GC performance improvements"

Merge "Add check that dexlayout can open output file."

Remove native printing from test 980 and reliance on print internals

In preparation for a making a CTS test out of this make the test not
rely on internal allocation patterns of the 'System.out' object and
not make use of std::cout.

Test: ./test.py --host -j40
Change-Id: Ib7e874aaec71f93e834cf94ac5fe96663536691a

Add check that map offset is sane to DexFile object init.

The DexFile constructor calls InitializeSectionsFromMapList, which
uses the header's map_offset before any checks are done on the file.

Bug: 37235346
Test: mm test-art-host-gtest-dex_file_test
Change-Id: I4fb8fcb57f9ef7e0182965b7ce663424b953abcb

Various debug ART GC performance improvements

Removed unnecessary read barriers in SizeOf and a few other places.
Disabled disallow read barrier check.

Before:
GC time: 15.817s
Real 0m26.113s
user  1m16.780s
sys 0m3.152s

After:
GC time: 9.212s
real  0m19.875s
user  1m9.916s
sys 0m1.916s

Bug: 35644369
Test: test-art-host

Change-Id: I79a65259deff2a478a96e02ae69b14730b6dcbe6

Merge changes from topic 'art_run_test_910_cts'
am: e008fd53bf

Change-Id: Icdd80aeb36f47eb1b89b2e067f4ae8edbe517954

ART: Fix GetArgumentsSize
am: e1f79b62f3

Change-Id: Ifde959b97aefb64cac865aec5764f0a91251c066

Merge changes from topic 'art_run_test_910_cts'

* changes:
  ART: Refactor run-test 910
  ART: Fix GetArgumentsSize

Merge "MIPS32: java.lang.System.arraycopy() for copying char[] to char[]."
am: 99fde3c929

Change-Id: I3d42028292f9b1f46de5f23ac5b6ca5fac94afda

Merge "MIPS32: java.lang.System.arraycopy() for copying char[] to char[]."

ART: Use _exit in dex2oat

That's what we really wanted to call. Seems to save about 50ms when
compiling something small-ish (out of .8s wall-clock time).

Test: m test-art-host
Change-Id: I9a20451adeaf63f535a7e42b46b4db78ee1d2bea