Merge "Decrease target utilization for foreground apps."

Decrease target utilization for foreground apps.

GC time in FormulaEvaluationActions.EvaluateAndApplyChanges goes from
26.1s to 23.2s. Benchmark score goes down ~50 in
FormulaEvaluationActions.EvaluateAndApplyChanges, and up ~50 in
GenericCalcActions.MemAllocTest.

Bug: 8788501
Change-Id: I412af1205f8b67e70a12237c990231ea62167bc0

Merge "Fix thread join hangs"

Merge "Fix typo in ISA centralization"

Fix typo in ISA centralization

X86 needs sign-extension.

Change-Id: I1253fbc9aa0e732e37b976a0723ec89155e7e2e7

Merge "Centralize instruction-set pointer-size, alignment, 64b-width code in instruction_set.h/cc"

Merge "Long support in optimizing compiler."

Long support in optimizing compiler.

- Add stack locations to the Location class.
- Change logic of parameter passing/setup by setting the
  location of such instructions the ones for the calling
  convention.

Change-Id: I4730ad58732813dcb9c238f44f55dfc0baa18799

Fix thread join hangs

Current implementation notifies only one of joining threads on thread exit.
This causes the other joining threads to hang.

This patch notifies all joining threads: just call NotifyAll() instead of Notify().

Change-Id: If75f101d3d3d876f1aa7dcf537e56a37cc347803
Signed-off-by: Yevgeny Rouban <yevgeny.y.rouban@intel.com>

Merge "Disable JDWP field watchpoint capabilities"

Disable JDWP field watchpoint capabilities

We do not fully support field watchpoint yet. We can only enable these
capabilities when we do report these events.

Bug: 8267708
Change-Id: I07ccf7ddffdd26e143b7fb0f13ebe030d75e8491

Merge "Allow use of instance filter for JDWP events"

Merge "Fix JDWP count event modifier"

Merge "Less confusing heap transition message."

Less confusing heap transition message.

The heap transition message now just says how long the transition
took and up to how much memory was saved (it can still be negative).

Change-Id: If421dcc2aa9cc40740b858dcc2ff1b8e595f9a40

Merge "Don't call SetFootprintLimit in CreateZygoteSpace."

Don't call SetFootprintLimit in CreateZygoteSpace.

Since SetFootprintLimit uses the allocators accounting it can set a
word inside of the zygote space mem map. This was causing issues
since it was occasionally clobbering one of the object classes. This
occured since we compact objects into the space which was used by the
allocator's internal accounting during zygote bin packing.

Also addressed some TODO.

Bug: 14056343
Change-Id: I9fc5a61174fa141c0f44d0ed0ae0cc6d77cb4e0a

Allow use of instance filter for JDWP events

Bug: 13922844
Change-Id: Ic4e6d7af7c80928c25417db5f43049614d225c97

Fix JDWP count event modifier

Bug: 14053614
Change-Id: I736f6d6802e58e7a4fb29d0558bdd2abd3b2a48e

Merge "Prepare field watchpoint support"

Prepare field watchpoint support

Adds field read/write events in the instrumentation. The debugger now registers
as a listener for these events so JDWP field access and field modification
events can be reported.

This CL will be followed by another one to report these events from the
interpreter. Therefore no JDWP field access and field modification events
can be sent for now.

Bug: 8267708
Change-Id: If2a93eb590805567d69015c83cce9cd2ab712cbd

Merge "Clear unused roots from debugger"

Merge "Deduplicate code starting JDWP thread"

Merge "Fix clang build."

Fix clang build.

Change-Id: Iaf0b76cfc02bac5f49ab98d494db9a774211f453

Merge "art_quick_unlock_object uses comparison incorrectly"

Merge "Refactor space bitmap to support different alignments."

Merge "Reduce bump pointer space capacity."

Reduce bump pointer space capacity.

Bug: 14058912
Change-Id: I7ae53e667539530d56ca2f5d10214e69c9c994e2

Merge "Recognize and ignore options '-ea' and '-da'"

Centralize instruction-set pointer-size, alignment, 64b-width code
in instruction_set.h/cc

This allows to clean up some places that currently make explicit
comparisons.

Change-Id: I0dcc924c52fa53306f706aceea93a2d4a655c5df

Recognize and ignore options '-ea' and '-da'

-ea:<package or class> and -da:<package or class> options are ignored.

This patch just adds -ea and -da options without any package or
class specified.

Change-Id: Ide04b8eacb1b2edb415328130bc12e05dc3ddc7b
Signed-off-by: Yevgeny Rouban <yevgeny.y.rouban@intel.com>

Refactor space bitmap to support different alignments.

Required for:
Using space bitmaps instead of std::set in mod union table +
remembered set.
Using a bitmap instead of set for large object marking.

Bug: 13571028

Change-Id: Id024e9563d4ca4278f79607cdb2f81895121b113

Deduplicate code starting JDWP thread

Also waits for thread_start_cond_ in a loop and removes needless mutex lock in
JdwpState::Run when checking thread state.

Change-Id: I6fca2151b5343b2906e9fa3b439029e6ad5b086d

Clear unused roots from debugger

SingleStepControl and DebugInvokeReq hold roots which are only used during a
limited amount of time. Clear these roots once we finish using them.

Also cleans some code around single-step.

Change-Id: Ica2f65e04a4d60272199490cd40ad1803d3b9f02

Merge "Fix hammerhead phone boot on master."

Fix hammerhead phone boot on master.

Wasn't failing on AOSP.

Change-Id: I8e531bf7493d36dc2667afeaf3133ce8560a2ce1

Merge "Fix heap verification with SS/GSS."

Fix heap verification with SS/GSS.

Since I had removed the SwapSemiSpaces it was trying to verify
references in the from space of the semispace collector, which isn't
necessarily valid.

Bug: 13989162

Change-Id: I8e0b7119a1a53d63f2c71ecdb276290eda5bae07

Merge "Fix an occasional ThreadStress crash."

Merge "Clean-up heap spaces."

Fix an occasional ThreadStress crash.

Also, fix the NativeAllocations test for the case with the GSS
collector as the default GC. Since kGcCauseForAlloc (incorrect) was
being passed into the collector instead of kGcCauseForNativeAlloc
(correct) from Heap::RegisterNativeAllocation(), the GCC collector
never invoked a whole heap collection which was necessary to do
sufficent finalizations to pass the test.

Bug: 13988451
Bug: 11650816
Change-Id: Ib02f061751cd777e0d3bfa81b29e04a874dc58a0

Clean-up heap spaces.

We now use the CMS collector instead of the semispace collector when
the phone is booting. We still perform compaction during the zygote
space creation. This reduces time spent in GC by ~2s during boot
and doesn't affect zygote space size.

Changed the space creation logic to create the temp space when a
background transition occurs.

Added a flag to each space which is true if you are allowed to
move objects that are within this space.

Removed SwapSemiSpaces call from the semi space collector, it is now
the job of the caller to do this with threads suspended. This
simplifies the logic in the zygote compaction / heap transition code
since these do not copy from one semispace to another.

Added Space::Clear to RosAllocSpace and DlMallocSpace. This greatly
simplifies the code used for collector transitions.

Time spent in GC creating zygote space:
Before: 3.4s, After: 1.28s
No change in zygote space size.

Bug: 13878055

Change-Id: I700348ab7d5bf3aa537c0cd70c0fed09aa4b0623

Merge "AArch64: List arm64 as instruction set option"

AArch64: List arm64 as instruction set option

When arm64 was added to dex2oat, it wasn't added to the list of
instruction sets in the help text. This adds it.

Change-Id: I9ffecb67a053b24ad624effd1853ff47331d2e91
Signed-off-by: Stuart Monteith <stuart.monteith@arm.com>

Merge "Prevent deadlocks in Runtime::Abort."

Prevent deadlocks in Runtime::Abort.

If we have the threads suspended, attempting to use a
ScopedObjectAccess causes a deadlock. We now specifically avoid this
to prevent deadlocks.

Bug: 13747880
Change-Id: I45fd3fff917da98b22970e5351a9e25b143a4eed

Merge "AArch64: Jni compiler fixes"

AArch64: Jni compiler fixes

This patch fixes some of the issues with the ARM64 assembler and JNI
compiler.

The JNI compiler is not enabled by default, yet. To enable, change
line 1884 in compiler/driver/compiler_driver.cc, removing kArm64 from
the GenericJNI list.

The compiler passes all tests in jni_compiler_test.

Also change the common_compiler_test instruction-set-features logic.
We allow tests when the build-time features are a subset of the
runtime features.

Dex2oat cross-compiling is now working. A 32b version of dex2oat should
be able to compile correctly.

Change-Id: I51d1c24f2c75d4397a11c54724a8b277ff3b3df8
Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>

Merge "Insert read barriers in more places."

Insert read barriers in more places.

Bug: 12687968
Change-Id: If3ffca4741e039f314ac848551c481d44cbcd3ca

Merge "Fix race condition in TransitionCollector."

Merge "Fix problem with empty profiles."

Fix problem with empty profiles.

This fixes an issue where a profile file was created by the installer
but never written by the runtime.  This happens when profiles are off.
This caused the compiler to think that there are no methods worthy
of compilation and skipped everything.

Bug: 13960166
Change-Id: I4c4598d4746218e21ca949112071dc1424d7d1f9

Fix race condition in TransitionCollector.

There was a race condition where if multiple threads were calling
TransitionCollector it could cause a crash due to an invalid
collector transition if another thread did the collector transition
before the SuspendAll.

Bug: 13929101
Change-Id: I8c162a83c1f53d0cbdefab62b0a5bcbb151d6c42

Merge "art: Use SIRT::GetAlignedSirtSizeTarget in calling convention"

art: Use SIRT::GetAlignedSirtSizeTarget in calling convention

Calculate frame size based on SIRT::GetAlignedSirtSizeTarget existing method.

Make offset functions pointer-size-dependent for cross-compiling. Add
a test to check whether our computations are correct.

Change-Id: Ic66daf6f9908890eda906bdcbbc4444c4fef614f
Signed-off-by: Dmitry Petrochenko <dmitry.petrochenko@intel.com>

art_quick_unlock_object uses comparison incorrectly

art_quick_unlock_object incrrectly checks the thin lock condition.
The patch fixes this issue.

Change-Id: I9e2fcf89c9e9cb34aca72ed001672fd83cd53377
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Merge "AArch64 code alignment is 4 for OatWriter"

AArch64 code alignment is 4 for OatWriter

OatWriter DCHECKs against ARM alignment, which is 8 and <= all
other architectures *except* AArch64, so that was fine before.

Change-Id: I55a11fe60cfbec889f2e8d8b0f489fe0930ebf6f

Merge "Change FindClass in GetQuickInvokedMethod to use reg_types_."

Merge "Revert "Revert "Revert "Use trampolines for calls to helpers""""

Revert "Revert "Revert "Use trampolines for calls to helpers"""

This reverts commit f9487c039efb4112616d438593a2ab02792e0304.

Change-Id: Id48a4aae4ecce73db468587967968a3f7618b700

Change FindClass in GetQuickInvokedMethod to use reg_types_.

FindClass can do ScopedThreadStateChange which causes check failures
during SIGQUIT.

Bug: 13929160
Change-Id: I00be4972423c04254a922de3aa731e8edb38ca02

Merge "Add mean GC duration to DumpGcPerformanceInfo."

Merge "Revert "Revert "Use trampolines for calls to helpers"""

Merge "Switch profiler off by default"

Switch profiler off by default

Change-Id: I40fdbfa1145a2e80644125932756b8ec7eafd4a2

Revert "Revert "Use trampolines for calls to helpers""

This reverts commit 081f73e888b3c246cf7635db37b7f1105cf1a2ff.

Change-Id: Ibd777f8ce73cf8ed6c4cb81d50bf6437ac28cb61

Conflicts:
compiler/dex/quick/mir_to_lir.h

Add mean GC duration to DumpGcPerformanceInfo.

Change-Id: Ibd68e9d8cae71d0ceb769fc975f1148287e9ca4a

Merge "ARM64: Fix array out of bounds access in DoLongJump"

Merge "Adjust sticky GC ergonomics."

ARM64: Fix array out of bounds access in DoLongJump

Only copy the "real" 32 registers, not the virtual 33rd XZR.

Change-Id: I911a56e3ae3a4e76fd29da5eb7caae5c000fdc28

Adjust sticky GC ergonomics.

Added an adjustment factor that causes sticky GC to occur more often.
Currently, we adjust the throughput by 5 / 4 comapred to the non
sticky GC. This improves the MemAllocTest score by 50-100. I believe
this happens since the sticky GC uses less memory bandwidth than
partial/full GC. No benchmark score regression on
EvaluateAndApplyChanges though total GC time is increased by ~0.5s.

Bug: 13818507
Change-Id: Ic721a1bff069ddaf8acc55e776b8bc26e171d46e

Merge "GSS: Fix the bump pointer space only collection."

Merge "Fix ARM64 invoke stubs, correct CFI directives"

GSS: Fix the bump pointer space only collection.

Fixes b/13912464 where the clear soft reference behavior accidentally
disabled the bump pointer space only collection.

Changed the collector name so that the GC logs would indicate the
generational mode and the collection mode.

Peformance enhancement: instead of triggering the whole heap
collection every 5 collections, count the bytes promoted since the
last whole heap collection and use it to decide when to trigger the
whole heap collection. This improves MemAllocTest by 5-10% (N4 and
host).

Bug: 13912464
Bug: 11650816
Bug: 9986565
Change-Id: I653a0dca62a8b54adf69abe2940a41eac70f809b

Fix ARM64 invoke stubs, correct CFI directives

The stubs do not advance over the arguments array when they cannot
place the parameter in a register. Fixed.

Changed the frame code for the invoke stubs so that CFI is easier
to apply. Also defined two macros to unite the parts that are
identical between static and dynamic invoke. With moving one statement
and accepting 12B increased (dead) code size one could almost fully
unite the implementations.

Corrected CFI directives for the CALLEE_SAVE macros.

Change-Id: Idf593fc46f0b6e1eb579010d0cdcf9c1a71730b1

Merge "Reset GC timings after SIGQUIT."

Merge "Better error reporting when loading dex files"

Better error reporting when loading dex files

Collect all partial error messages and return them as cause
exceptions for the top-level exception returned.

Change-Id: I9661b8aed2a571dc88bf0f06d447108eeaed1409

Reset GC timings after SIGQUIT.

We now reset the GC timings when a SIGQUIT happens, this is useful
for excluding GCs which happen during the initialization of an app
when measuring GC performance.

Change-Id: I68c79bdb279290c12ae588bc7e95ac24908c157e

Merge "ART: A workaround for a wrongly-read caller's frame"

Merge "Tweaked profile significant_difference."

Merge "Profile: made startImmediately settable"

Merge "Test control flow instruction with optimizing compiler."

Test control flow instruction with optimizing compiler.

Add support for basic instructions to implement these tests.

Change-Id: I3870bf9301599043b3511522bb49dc6364c9b4c0

Merge "Quick compiler: fix CmpLong pair handling"

Merge "Implement rosalloc page trimming without suspending threads."

Implement rosalloc page trimming without suspending threads.

Also, making it more efficient by not going through the chunks smaller
than the page size by not using InspectAll().

Change-Id: I79ceb0374cb8aba5f6b8dde1afbace9af98b6cff

Merge "Use LIRSlowPath for throwing div by zero exception."

Merge "Add monitor deflation."

Use LIRSlowPath for throwing div by zero exception.

Get rid of launchpads for throwing div by zero exception and
use LIRSlowPath instead. Add a CallRuntimeHelper that takes no
argument for the runtime function.

Bug: 13170824
Change-Id: I7e0563e736c6f92bd63e3fbdfe3a777ad333e338

Add monitor deflation.

We now deflate the monitors when we perform a heap trim. This causes
a pause but it shouldn't matter since we should be in a state where
we don't care about pauses. Memory savings are hard to measure.

Fixed integer overflow bug in GetEstimatedLastIterationThroughput.

Bug: 13733906
Change-Id: I4e0e68add02e7f43370b3a5ea763d6fe8a5b212c

Merge "AArch64: Fix Managed Register unit tests"

Quick compiler: fix CmpLong pair handling

OpCmpLong wasn't properly extracting the low register of a
pair.

Change-Id: I6d6cc3de1f543f4316e561648f371f793502fddb

Merge "Revert "Use trampolines for calls to helpers""

Revert "Use trampolines for calls to helpers"

This reverts commit 754ddad084ccb610d0cf486f6131bdc69bae5bc6.

Change-Id: Icd979adee1d8d781b40a5e75daf3719444cb72e8

Merge "Inlining synthetic accessors."

Merge "Use trampolines for calls to helpers"