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

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

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

ART: Fix computation of frame size for direct proxy methods

Proxy method has only one direct method and it is a constructor which
is cloned from java.lang.reflect.Proxy class together with code.
As a result its body is a compiled quick code and frame size should be
computed accordingly in contrast with other virtual methods which are
invoked through stub.

Change-Id: I0ac99b2f567b281c9342152bad1149dd6cd39480
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>
(cherry picked from commit 805bab1738549b2477b3ad4d9d57fd7c681451b9)

am e1a86627: Merge "Fix native allocation watermark clamping." into lmp-dev

* commit 'e1a8662792260953efe05fd991d945e508298aa8':
  Fix native allocation watermark clamping.

Merge "Fix native allocation watermark clamping." into lmp-dev

Fix native allocation watermark clamping.

The main issue causing the test to fail is that
native_footprint_gc_watermark_ becoming > growth_limit_ due to no
clamping.

Temporary runFinalization fix is calling runFinalization 2x.

Bug: 17371542
Change-Id: I188cb530a44dd109e066a22091f12f8d2d4350c3

am c4978258: Merge "Add exception check to AllocateInternalWithGc" into lmp-dev

* commit 'c4978258172beb35c3c8f0c31a5f7eb06b9a1de2':
  Add exception check to AllocateInternalWithGc

Merge "Add exception check to AllocateInternalWithGc" into lmp-dev

am 6b28a456: Fix stack overflow and duplicate methods while tracing.

* commit '6b28a456b075fe53dfb7e924a44dbf35d0d41eb3':
  Fix stack overflow and duplicate methods while tracing.

Fix stack overflow and duplicate methods while tracing.

Bug: 16386215
Change-Id: I0d0ae0113a3a00013ce84a1f5a110e2c52f19b86

Add exception check to AllocateInternalWithGc

Trying to figure out if the pending exception comes before or during
AllocateInternalWithGc.

Bug: 17164348

Change-Id: Id46d82bfc0f84d16e5ffdafe41658aee4cc2c702

am ff4cfe07: Fix SetupResourceMasks to pass correct parameters

* commit 'ff4cfe07009ac0b0f07475ef2aa91d9aaf9ab291':
  Fix SetupResourceMasks to pass correct parameters

Fix SetupResourceMasks to pass correct parameters

Calling SetupTargetResourceMasks function with correct parameter.
SetupTargetResourceMasks(lir, flags, use_mask, def_mask) is correct order.

b/17369806

(cherry-picked from commit sha 2a09504334a3a3b4c47100197df0827cc6740433)

Change-Id: I49118c3f2d3d06fac83fa3d5014b7fdaff6a2b00

am f72fd02c: Merge "Quick compiler: disable GVN DO NOT MERGE" into lmp-dev

* commit 'f72fd02c4ea0f1aa610b8a4f9f51ef86c285b1cf':
  Quick compiler: disable GVN DO NOT MERGE

Merge "Quick compiler: disable GVN DO NOT MERGE" into lmp-dev

Quick compiler: disable GVN DO NOT MERGE

Disable global value numbering optimization for lmp (but
keep on for master).

b/16398693

Change-Id: I125c588c987a81db54a15da1eec9dee2b009956f

am c60bdf66: Fix native allocation test.

* commit 'c60bdf667b5c6fa767548942f0115547523593b4':
  Fix native allocation test.

am 4d05d931: Merge "ART: Fix read-out-of-bounds in the compiler" into lmp-dev

* commit '4d05d931d3d608f5047e0d9dfb422df018ace716':
  ART: Fix read-out-of-bounds in the compiler

am 635531a2: Merge "Change native allocations to use growth limit." into lmp-dev

* commit '635531a2d99f413339187ce39b08affdf1b31d9f':
  Change native allocations to use growth limit.

am 37bb8184: Merge "Pre-allocate the NoClassDefFoundError to be thrown for boot classes." into lmp-dev

* commit '37bb8184e091e8d9199cfc2fc888dfb40a5c0f20':
  Pre-allocate the NoClassDefFoundError to be thrown for boot classes.

am 9db18e16: Merge "Reduce and speed-up class def searches." into lmp-dev

* commit '9db18e1679800bc57c1e30f7c5ff786949ffe5cf':
  Reduce and speed-up class def searches.

am e610faf9: Merge "ART: Tighten verifier list reading and offsets" into lmp-dev

* commit 'e610faf917fd280fa0d9e02d5c2d0dbacbb037b7':
  ART: Tighten verifier list reading and offsets

Fix native allocation test.

Forgot to update with last CL.

Change-Id: Id2f57870b8d4f848cd93012d9da69beee5184be4

Merge "ART: Fix read-out-of-bounds in the compiler" into lmp-dev

Merge "Change native allocations to use growth limit." into lmp-dev

Change native allocations to use growth limit.

Previously native allocation tracking used a GC footprint limit
which would cause GC in the allocating thread. This prevented
excessive growth of the heap but could cause jank due to GC in
the allocating thread. The new behavior is using the growth_limit
instead of the native footprint limit.

Bug: 17006948

Change-Id: I40f30af09bb25596a9f57fa50e2a155fb947b5fe

Merge "Pre-allocate the NoClassDefFoundError to be thrown for boot classes." into lmp-dev

Merge "Reduce and speed-up class def searches." into lmp-dev

Merge "ART: Tighten verifier list reading and offsets" into lmp-dev

Pre-allocate the NoClassDefFoundError to be thrown for boot classes.

Bring over a Dalvik "optimization".
Bug: 12804658
Bug: 16853450
Change-Id: I6419de7bd2ba18d91479cb52489104954f5c4524

(cherry picked from commit 63557459a4098294a9ff44d035241de2966047c0)

Reduce and speed-up class def searches.

Use the class linker for descriptor lookups from the compile driver so that
dex caches are populated.
Reduce the scope of functions for scanning class paths to just the class
linker where they are performed.
If we see more than a threshold number of find class def misses on a dex file
lazily compute an index, so that future lookups are constant time (part of the
collection code is taken from
https://android-review.googlesource.com/#/c/103865/3). Note that we take a lazy
approach so that we don't serialize on loading dex files, this avoids the
reason the index was removed in 8b2c0b9abc3f520495f4387ea040132ba85cae69.
Remove an implicit and unnecessary std::string creation for PrintableString.

Single threaded interpret-only dex2oat performance is improved by roughly 10%.

Bug: 16853450

Change-Id: Icf72df76b0a4328f2a24075e81f4ff267b9401f4
(cherry picked from commit 68b56858367e29461ae290fd797443a1ef6d8005)

am 23590fd0: Merge "Change intern table to unordered set." into lmp-dev

* commit '23590fd094df6a4a2360b722763249adf1689f15':
  Change intern table to unordered set.

Merge "Change intern table to unordered set." into lmp-dev

am 8bd2961b: Change heap transitions + hspace compation to use VLOG(heap)

* commit '8bd2961b385c870953e650c253a04ee24b76809b':
  Change heap transitions + hspace compation to use VLOG(heap)

Change heap transitions + hspace compation to use VLOG(heap)

VLOG(gc) is spammy since it is enabled.

Change-Id: I07d1d40e2f19134f885d6777b60533a9010ae164

ART: Tighten verifier list reading and offsets

Check offsets and sizes for header entries of a dex file.

Bug: 17347459
Change-Id: Ia1727c33dea51f7a8e345f3799f1ba414708239c

am b373ba1b: ART: Fix typo

* commit 'b373ba1bf7e67d4dd8286df08065ac76aa8ac1ee':
  ART: Fix typo

ART: Fix typo

Bug: 17327877
Change-Id: I6132a8afb4b69c8d22a01b477f6f324c789ed666

am 277a7c7b: ART: Fix dex file verifier type-list handling

* commit '277a7c7b4bb9c421380592fd3998d2e79e4035b3':
  ART: Fix dex file verifier type-list handling

ART: Fix dex file verifier type-list handling

It is rare, but valid, to have an empty type list.

Bug: 17327877
Change-Id: Ib3a8ff3e5ccd8fe7c04b1e97485bf3e6de72aa4d

Change intern table to unordered set.

Intern table active used bytes goes from 430k to 317k on system
server. Similar %wise savings on other apps.

Bug: 16238192

Change-Id: I92de72de3eae0b76629e9da83db2c2d0fa613052

am 20632daa: Merge "Add native memory accounting through custom allocator." into lmp-dev

* commit '20632daac36995947d2d1943f89f07ac277e61f0':
  Add native memory accounting through custom allocator.

Merge "Add native memory accounting through custom allocator." into lmp-dev

Add native memory accounting through custom allocator.

Added a custom allocator that lets you pass in a special tag which
specifices where the allocation came from. This is used when
dumping. The performance overhead is low since each allocation only
does a atomic add/sub for each allocation/free.

The measurements are dumped to traces.txt during SIGQUIT.

Example output:
I/art     (27274): AllocatorTagHeap active=120 max=120 total=168
I/art     (27274): AllocatorTagMonitorList active=1572 max=6240 total=11724
I/art     (27274): AllocatorTagClassTable active=185208 max=185208 total=268608
I/art     (27274): AllocatorTagInternTable active=430368 max=430368 total=436080
I/art     (27274): AllocatorTagMaps active=5616 max=6168 total=34392
I/art     (27274): AllocatorTagLOS active=1024 max=1536 total=2044
I/art     (27274): AllocatorTagSafeMap active=0 max=51936 total=533688
I/art     (27274): AllocatorTagLOSMaps active=144 max=1248 total=5760
I/art     (27274): AllocatorTagReferenceTable active=10944 max=11840 total=19136
I/art     (27274): AllocatorTagHeapBitmap active=32 max=40 total=56
I/art     (27274): AllocatorTagHeapBitmapLOS active=8 max=8 total=8
I/art     (27274): AllocatorTagVerifier active=0 max=18844 total=1073156
I/art     (27274): AllocatorTagModUnionCardSet active=5300 max=5920 total=56020
I/art     (27274): AllocatorTagModUnionReferenceArray active=24864 max=24864 total=24864
I/art     (27274): AllocatorTagJNILibrarires active=320 max=320 total=320
I/art     (27274): AllocatorTagOatFile active=1400 max=1400 total=5852

Bug: 16238192

Change-Id: Ibb470ef2e9c9a24563bb46422d46a55799704d82

am 9bf1b088: Merge "Fix mips build for nested signal handler" into lmp-dev

* commit '9bf1b088e4e12121037090dd9c92393fbe805cb6':
  Fix mips build for nested signal handler

Merge "Fix mips build for nested signal handler" into lmp-dev

am 496cd337: Fix segfault if running without image.

* commit '496cd337c19ca8386fec24633160f82c01993bbd':
  Fix segfault if running without image.

Fix mips build for nested signal handler

Bug: 17006816

Change-Id: If41c6d7a2f12f1822ca36d9e2f0f22b3ca854098

Fix segfault if running without image.

Bug: 17325091

(cherry picked from commit 7adb7ac3913364de8cc57b8934024dd12e1d3bea)

Change-Id: I343099543ce0abf02219da84d61d9ce2dfc47980

am 2cc42e30: Merge "Quick compiler: Fix MIPS build" into lmp-dev

* commit '2cc42e3029182ad6c6443376c1a4cbf9c1e6a784':
  Quick compiler: Fix MIPS build

Merge "Quick compiler: Fix MIPS build" into lmp-dev

Quick compiler: Fix MIPS build

b/16484538

In debug builds, the Quick compiler frequently runs a sanity checker
over the register pool.  Among other things, it attempts to verify
consistent representation of register pairs.  However, a register's
"wide" flag is meaningful only when the register pair is associated
with a Dalvik wide value (sreg != INVALID_SREG) rather than a temp
wide value.

The MIPS build was tripping over this bad assertion.  Fixed here.

Note related cl/105461

(cherry-picked from commit sha 3a658077af2a931e5d7f6cd22b777c57112c19df)

Change-Id: Id726ff1ea0f5cbcc8dba6fa3aacb3fd4fc043a63

am 8ac3356d: Avoid null pointer dereference when sending JDWP packets

* commit '8ac3356db74f5e38c70406cc5f5e65a49ff41fd7':
  Avoid null pointer dereference when sending JDWP packets