)]}' { "log": [ { "commit": "e401d146407d61eeb99f8d6176b2ac13c4df1e33", "tree": "17927f9bfe7d2041b5942c89832d55f9dedb24c5", "parents": [ "2006b7b9b8e32722bd0d640c62549d8a0ac624b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 13:56:20 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 29 18:45:49 2015 -0700" }, "message": "Move mirror::ArtMethod to native\n\nOptimizing + quick tests are passing, devices boot.\n\nTODO: Test and fix bugs in mips64.\n\nSaves 16 bytes per most ArtMethod, 7.5MB reduction in system PSS.\nSome of the savings are from removal of virtual methods and direct\nmethods object arrays.\n\nBug: 19264997\nChange-Id: I622469a0cfa0e7082a2119f3d6a9491eb61e3f3d\n" }, { "commit": "15efde2badd80b8b124f8e1444165e4afbac17f0", "tree": "043819e50737fc6c3ba619b0e708e7864c7a801a", "parents": [ "dc31313c4577bb4c00d90774159c84e738ed348d" ], "author": { "name": "Dmitriy Ivanov", "email": "dimitry@google.com", "time": "Fri May 22 18:03:34 2015 -0700" }, "committer": { "name": "Dmitriy Ivanov", "email": "dimitry@google.com", "time": "Wed May 27 14:10:16 2015 -0700" }, "message": "Reset libc targetSdkVersion\n\n This will let libc/linker know which sdk version\n is targeted by the app. This is needed to work around\n incorrect DT_NEEDED entries in old apps.\n\nBug: http://b/21364029\nChange-Id: I2f2907104106aac6c8831381ca171dc0302e6a9d\n" }, { "commit": "0ae98992d348b5bf3fc85a52d81b0e5f221f20d6", "tree": "4e699a63f743515133ed53ed52bb6e19c4596dfd", "parents": [ "1db3554cf9751d4328d5514b6bee15b6a92b82c4" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri May 01 14:33:19 2015 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri May 01 14:53:36 2015 -0700" }, "message": "Request full/partial GC type from VMRuntime.requestConcurrentGC().\n\nBug: 20554807\nChange-Id: I1cd855507c3f4e191035dbc2c429e1b8b344d81b\n" }, { "commit": "2cebb24bfc3247d3e9be138a3350106737455918", "tree": "d04d27d21b3c7733d784e303f01f873bb99e7770", "parents": [ "1f02f1a7b3073b8fef07770a67fbf94afad317f0" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 21 16:50:40 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 12:44:27 2015 -0700" }, "message": "Replace NULL with nullptr\n\nAlso fixed some lines that were too long, and a few other minor\ndetails.\n\nChange-Id: I6efba5fb6e03eb5d0a300fddb2a75bf8e2f175cb\n" }, { "commit": "d3ed9a320a89cb9b91b2361892c043ab7e112717", "tree": "94d2b646e8ff9b28e0bef735804ce17a6a8be729", "parents": [ "4b5673b7387804947a1605a906deee132ab28f14" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 10 14:23:35 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 13 16:15:22 2015 -0700" }, "message": "Fix DCHECK failures from Class::VisitFieldRoots\n\nWe now use GetDeclaringClassUnchecked when marking roots to fix\nflaky test failures. Fixed a race condition in root marking where\nwe could have non zero field array length with a null pointer.\nFixed a race condition where we could be marking roots before\nFixupTemporaryDeclaringClass had finished. The solution is to\nonly do the declaring class CHECK if we are at least resolved.\nFixed JDWP tests by changing FieldId / MethodId to be 64 bits.\n\nAlso some cleanup.\n\nChange-Id: Ibac09519860d93c3f68a5cc964bbc91dc10a279a\n" }, { "commit": "c785344b87221f5e4e6473e5b762e4e61fe65dcf", "tree": "cd32ad2c2604596a18926f04d4c313dab255ecfd", "parents": [ "a29d93b380c9aeb8270e281aefbdd0c77a430d43" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 27 14:35:38 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 10 12:57:27 2015 -0700" }, "message": "Move ArtField to native\n\nAdd linear alloc. Moved ArtField to be native object. Changed image\nwriter to put ArtFields after the mirror section.\n\nSavings:\n2MB on low ram devices\n4MB on normal devices\n\nTotal PSS measurements before (normal N5, 95s after shell start):\nImage size: 7729152 bytes\n23112 kB: .NonMoving\n23212 kB: .NonMoving\n22868 kB: .NonMoving\n23072 kB: .NonMoving\n22836 kB: .NonMoving\n19618 kB: .Zygote\n19850 kB: .Zygote\n19623 kB: .Zygote\n19924 kB: .Zygote\n19612 kB: .Zygote\nAvg: 42745.4 kB\n\nAfter:\nImage size: 7462912 bytes\n17440 kB: .NonMoving\n16776 kB: .NonMoving\n16804 kB: .NonMoving\n17812 kB: .NonMoving\n16820 kB: .NonMoving\n18788 kB: .Zygote\n18856 kB: .Zygote\n19064 kB: .Zygote\n18841 kB: .Zygote\n18629 kB: .Zygote\n3499 kB: .LinearAlloc\n3408 kB: .LinearAlloc\n3424 kB: .LinearAlloc\n3600 kB: .LinearAlloc\n3436 kB: .LinearAlloc\nAvg: 39439.4 kB\n\nNo reflection performance changes.\n\nBug: 19264997\nBug: 17643507\n\nChange-Id: I10c73a37913332080aeb978c7c94713bdfe4fe1c\n" }, { "commit": "bb87e0f1a52de656bc77cb01cb887e51a0e5198b", "tree": "113f014c6e20fab3e936a3ac05f9f738639541f6", "parents": [ "e57fc0f0260fcb1d08cbb720ec95c04c0f394b91" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 03 11:21:55 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 06 10:44:37 2015 -0700" }, "message": "Refactor and improve GC root handling\n\nChanged GcRoot to use compressed references. Changed root visiting to\nuse virtual functions instead of function pointers. Changed root visting\ninterface to be an array of roots instead of a single root at a time.\nAdded buffered root marking helper to avoid dispatch overhead.\n\nRoot marking seems a bit faster on EvaluateAndApplyChanges due to batch\nmarking. Pause times unaffected.\n\nMips64 is untested but might work, maybe.\n\nBefore:\nMarkConcurrentRoots: Sum: 67.678ms 99% C.I. 2us-664.999us Avg: 161.138us Max: 671us\n\nAfter:\nMarkConcurrentRoots: Sum: 54.806ms 99% C.I. 2us-499.986us Avg: 136.333us Max: 602us\n\nBug: 19264997\n\nChange-Id: I0a71ebb5928f205b9b3f7945b25db6489d5657ca\n" }, { "commit": "0aa50ce2fb75bfc2e815a0c33adf9b049561923b", "tree": "9a3f9603ab30d5cbc7fc21aee0ceb48bbb0dd25a", "parents": [ "e8e42f3548fd894f860912bb1b71ce6fa2d7daf3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 11:03:29 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 14:51:11 2015 +0000" }, "message": "Remove ThrowLocation.\n\nNote that this is a cleanup change, and has no functionality change.\nThe ThrowLocation had no use anymore.\n\nChange-Id: I3d2126af1dc673cec3a0453ff3d56a172663a5f6\n" }, { "commit": "e34fa1df67fbe0173b4ea9abddcc3ae3d0537037", "tree": "a5148f079b5671a95f60910c41981ebf91db3a02", "parents": [ "9f06b1946ae3ef1d2fd75bbf7f0a288bc611fe58" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 14 14:55:47 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 15 12:23:28 2015 -0800" }, "message": "Print more info in MarkSweep::VerifyRoot\n\nRefactored old root callback to use a new class called RootInfo.\nRootInfo contains all the relevant info related to the root\nassociated with the callback. The MarkSweep::VerifyRoot function\nnow uses this info to print the StackVisitor\u0027s described location\nif the GC root is of the type kRootJavaFrame.\n\nSome other cleanup.\n\nExample output:\nE/art (12167): Tried to mark 0x123 not contained by any spaces\nE/art (12167): Attempting see if it\u0027s a bad root\nE/art (12167): Found invalid root: 0x123 with type RootJavaFrame\nE/art (12167): Location\u003dVisiting method\n\u0027void java.lang.Runtime.gc()\u0027 at dex PC 0xffffffff (native PC 0x0)\nvreg\u003d0\n\n(cherry picked from commit 12f7423a2bb4bfab76700d84eb6d4338d211983a)\n\nBug: 18588862\nChange-Id: Ic5a2781f704e931265ffb3621c2eab4b2e25f60f\n" }, { "commit": "379d09fe3c3feb7c2a2fb5a3623689b5ace7e79b", "tree": "680b4173130057a1f70ad321eaf4cfbeba0e291c", "parents": [ "ca7d89d09294254f16db170a53b0f8dfbf0213ac" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 08 11:28:13 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 08 16:02:00 2015 -0800" }, "message": "Add clamp growth limit\n\nClamp growth limit shrinks the space memmaps to the current growth\nlimit. This reduces virtual memory usage for apps with small heaps.\n\nBug: 18387825\nBug: 17131630\n\nChange-Id: I4a8fdc335d2c40492e991708adabcc46299efb7d\n" }, { "commit": "a5eae69589ff562ad66c57665882cd16f237321c", "tree": "b80e50c050f5d32fc7b258ef1446a245a97c3df8", "parents": [ "6d1a047b4b3f9707d4ee1cc19e99717ee021ef48" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Dec 17 17:56:03 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 19 17:08:43 2014 -0800" }, "message": "Add heap task processor\n\nThe heap task processor processes async tasks which may be delayed.\nThe motivation for this change is preventing deadlocks which\ncan occur when the daemon threads get suspended by GetThreadStack.\n\nOther improvements, reduces daemon thread count by one.\nCleaner pending transition VS heap trimming logic.\n\nBug: 18739541\n\nChange-Id: Idab52b2d9661a6385cada74b93ff297ddc55fc78\n" }, { "commit": "079101a17575114622f6e1d5be5c9ba643630e9a", "tree": "bc9ff009cca8e649f24adb8f7626c95b7930be40", "parents": [ "81d035213b0426ee076cc0d017c895c7833d32e8" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Dec 15 14:23:10 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Dec 15 15:18:46 2014 -0800" }, "message": "Move GC daemon locking logic into heap\n\nFixes deadlock caused by acquirng the mutator lock while\nsynchronizing on the daemon thread.\n\nBug: 18739541\nChange-Id: I925b8f0f3b58178da6eff17b9c073f655c39597b\n" }, { "commit": "d582fa4ea62083a7598dded5b82dc2198b3daac7", "tree": "c76704c266ef4687eab425612ddf3fd24f93fe8d", "parents": [ "f20076ff813b8012096ff31af236d59db3c0f4e1" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Nov 05 23:46:43 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Nov 13 16:17:46 2014 -0800" }, "message": "Instruction set features for ARM64, MIPS and X86.\n\nAlso, refactor how feature strings are handled so they are additive or\nsubtractive.\nMake MIPS have features for FPU 32-bit and MIPS v2. Use in the quick compiler\nrather than #ifdefs that wouldn\u0027t have worked in cross-compilation.\nAdd SIMD features for x86/x86-64 proposed in:\n https://android-review.googlesource.com/#/c/112370/\n\nBug: 18056890\n\nChange-Id: Ic88ff84a714926bd277beb74a430c5c7d5ed7666\n" }, { "commit": "e7c9a8c2b8481aafbc6af4ce6229bd361ba24742", "tree": "f6d8fe8fd7aeae117a6547dc4f012cd4085cb4e8", "parents": [ "00b2da5c02339c36ffa4006f731f55203b09265d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Nov 06 16:35:45 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 07 11:45:06 2014 -0800" }, "message": "Add hash map, reduce excessive hashing\n\nChanged the class def index to use a HashMap instead of unordered_map\nso that we can use FindWithHash to reduce how often we need to compute\nhashes.\n\nFixed a bug in ClassLinker::UpdateClass where we didn\u0027t properly\nhandle classes with the same descriptor but different class loaders.\nIntroduced by previous CL.\n\nBefore (fb launch):\n1.74% art::ComputeModifiedUtf8Hash(char const*)\n\nAfter:\n0.95% art::ComputeModifiedUtf8Hash(char const*)\n\nBug: 18054905\nBug: 16828525\n\nChange-Id: Iba2ee37c9837289e0ea187800ba4af322225a994\n\n(cherry picked from commit 564ff985184737977aa26c485d0c1a413e530705)\n" }, { "commit": "277ccbd200ea43590dfc06a93ae184a765327ad0", "tree": "d89712e93da5fb2748989353c9ee071102cf3f33", "parents": [ "ad17d41841ba1fb177fb0bf175ec0e9f5e1412b3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 03 21:36:10 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 04 18:40:08 2014 -0800" }, "message": "ART: More warnings\n\nEnable -Wno-conversion-null, -Wredundant-decls and -Wshadow in general,\nand -Wunused-but-set-parameter for GCC builds.\n\nChange-Id: I81bbdd762213444673c65d85edae594a523836e5\n" }, { "commit": "6a3c1fcb4ba42ad4d5d142c17a3712a6ddd3866f", "tree": "9df58b57af13240a93a6da4eefcf03f70cce9ad9", "parents": [ "c6e0955737e15f7c0c3575d4e13789b3411f4993" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 31 00:33:20 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Nov 03 20:01:04 2014 -0800" }, "message": "Remove -Wno-unused-parameter and -Wno-sign-promo from base cflags.\n\nFix associated errors about unused paramenters and implict sign conversions.\nFor sign conversion this was largely in the area of enums, so add ostream\noperators for the effected enums and fix tools/generate-operator-out.py.\nTidy arena allocation code and arena allocated data types, rather than fixing\nnew and delete operators.\nRemove dead code.\n\nChange-Id: I5b433e722d2f75baacfacae4d32aef4a828bfe1b\n" }, { "commit": "2c4257be8191c5eefde744e8965fcefc80a0a97d", "tree": "9db3e1f1c60f2df29638ba3ce9d5d5bb8b26ca2c", "parents": [ "98c271d517bc4d25fc6879b4b8e35ea93885d9e2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 24 14:20:06 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 24 14:23:42 2014 -0700" }, "message": "Tidy logging code not using UNIMPLEMENTED.\n\nChange-Id: I7a79c1671a6ff8b2040887133b3e0925ef9a3cfe\n" }, { "commit": "13735955f39b3b304c37d2b2840663c131262c18", "tree": "0a731ac42b8230f9929172fa3e3d8051874e2b18", "parents": [ "25b18bbdaa36ff936eb44f228f0518d4223e9d52" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 08 12:43:28 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 16:05:58 2014 -0700" }, "message": "stdint types all the way!\n\nChange-Id: I4e4ef3a2002fc59ebd9097087f150eaf3f2a7e08\n" }, { "commit": "31d8f5295c24730a57cb36cbc41197f0b7e8397c", "tree": "e37953f590f924e06b9905f922407559a3661cb6", "parents": [ "8283e079a209e81caafd123a12a525922368fd34" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Sep 29 11:22:54 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Sep 30 21:49:43 2014 -0700" }, "message": "Add VMRuntime.isBootClassPathOnDisk\n\nBug: 17679443\n\n(cherry picked from commit 95a935415d44903b28326424beb4db5c013ef089)\n\nChange-Id: Iba40291dead3f0b6715903c986370fd0cf1e41e1\n" }, { "commit": "f0edfc355893d53d1104b05501c99ad5ccf305c4", "tree": "7e1fa49875759512f5d02b1c45435d3e3366b920", "parents": [ "1ed5b27ee329208fd8ae22b8a9a61d708e2c1ffb" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Sep 25 11:46:46 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Sep 25 15:53:54 2014 -0700" }, "message": "Some optimizations for the array alloc path.\n\n- Force Array::Alloc() to be inlined.\n- Simplify the array size overflow check.\n- Turn fill_usable into a template parameter.\n- Remove a branch in Array::DataOffset() and avoid\n Primitive::ComponentSize(), which has a switch, in the array alloc\n path.\n- Strength reductions in the array size computation by using component\n size shifts instead of component sizes. Store component size shift\n in the upper 16 bits of primitive_type field.\n- Speedup: ~4% (3435-\u003e3284) in MemAllocTest on N4.\n\nBug: 9986565\n\nChange-Id: I4b142ffac4ab8b5b915836f1660a949d6442344c\n" }, { "commit": "7b078e8c04f3e1451dbdd18543c8b9692b5b067e", "tree": "414229c6b87eb20ea24c40780752da5a3999a49a", "parents": [ "f79ba17defbd9342e44ab9f3de0807054673d3c9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 10 14:44:24 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Sep 12 14:57:53 2014 -0700" }, "message": "Compile time performance improvements focusing on interpret-only.\n\nReduce virtual method dispatch in the method verifier and make more code\ninline-able.\nAdd a StringPiece with const char* equality operator to avoid redundant\nStringPieces and strlens.\nRemove back link from register line to verifier and pass as argument to reduce\nsize of RegisterLine.\nRemove instruction length from instruction flags and compute from the\ninstruction, again to reduce size.\nAdd suspend checks to resolve and verify to allow for more easy monitor\ninflation and reduce contention on Locks::thread_list_suspend_thread_lock_.\nChange ThrowEarlierClassFailure to throw pre-allocated exception.\nAvoid calls to Thread::Current() by passing self.\nTemplate specialize IsValidClassName.\nMake ANR reporting with SIGQUIT run using checkpoints rather than suspending\nall threads. This makes the stack/lock analysis less lock error prone.\nExtra Barrier assertions and condition variable time out is now returned as a\nboolean both from Barrier and ConditionVariable::Wait.\n\n2 threaded host x86-64 interpret-only numbers from 341 samples:\nBefore change: Avg 176.137ms 99% CI 3.468ms to 1060.770ms\nAfter change: Avg 139.163% 99% CI 3.027ms to 838.257ms\nReduction in average compile time after change is 20.9%.\nSlow-down without change is 26.5%.\n\nBug: 17471626 - Fix bug where RegTypeCache::JavaLangObject/String/Class/Throwable\ncould return unresolved type when class loading is disabled.\nBug: 17398101\n\nChange-Id: Id59ce3cc520701c6ecf612f7152498107bc40684\n" }, { "commit": "8ec31f9835a8d1513be9be7f9fa236fe4715750d", "tree": "2b161b3683e559c66ae1926250583165499862e9", "parents": [ "badae981ada1fa279e54360df177ab6f40c299b9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 03 10:30:11 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 03 21:35:32 2014 -0700" }, "message": "Fix native allocation watermark clamping.\n\nThe main issue causing the test to fail is that\nnative_footprint_gc_watermark_ becoming \u003e growth_limit_ due to no\nclamping.\n\nTemporary runFinalization fix is calling runFinalization 2x.\n\nBug: 17371542\n\n(cherry picked from commit 4c7fc5950853b0c368e2148db77ced7c4d3c303c)\n\nChange-Id: I05b85e95560c32c33d53bc96abf87d5262007395\n" }, { "commit": "68d8b42ddec39ec0174162d90d4abaa004d1983e", "tree": "0bb6ccf3b996bb8363c10d07aa39cde221980602", "parents": [ "9c522c2cbbf50dc687728747b37ad59985750b65" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 11:09:10 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 08 08:29:53 2014 -0700" }, "message": "Wire up check JNI force copy mode.\n\nIncrease check JNI checks.\nBreak apart jni_internal.h in to jni_env_ext.h and java_vm_ext.h.\nFix the abuse of ScopedObjectAccess/annotalysis by ScopedCheck in the case\nof VM routines.\nMake class loader override and shared library class loader JNI global\nreferences rather than mirror pointers.\nClean-ups to native bridge.\n\nChange-Id: If7c6110b5aade7a402bfb67534af86a7b2cdeb55\n" }, { "commit": "22d5e735f403c57525fe868304c7123f0ce66399", "tree": "2458684efa56f0b800dd75a9dedd0449f76f581f", "parents": [ "fbde4dd1cb6db729e3f3ee5bdae0cdd824d73054" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 22:23:51 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jul 16 06:13:46 2014 -0700" }, "message": "Remove object_utils.h.\n\nBreak into object_lock, field_helper and method_helper.\nClean up header files following this.\nAlso tidy some of the Handle code in response to compiler errors when resolving\nthe changes in this CL.\n\nChange-Id: I73e63015a0f02a754d0866bfaf58208aebcaa295\n" }, { "commit": "20aa7cbe93d782e0e756a36e70d610fe84b4bbb4", "tree": "cfaaa9770d5ddbc1034b2c9770a12e6ff4d19b7b", "parents": [ "eab52e3318dd0ab0f357a8420f0a8ccea69d61f1", "c1b643cc6ac45dbd0eabdcd7425c7e86006c27d6" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Jun 06 11:15:25 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jun 06 11:15:25 2014 +0000" }, "message": "Merge \"Fixed and refactored profiler options handling\"" }, { "commit": "c1b643cc6ac45dbd0eabdcd7425c7e86006c27d6", "tree": "250455427da979409a075a2b3197bd43ccd40fe1", "parents": [ "bb0b53f58f11c628f077603b56077dfed1a18f11" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri May 30 23:44:11 2014 +0100" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Jun 06 12:14:01 2014 +0100" }, "message": "Fixed and refactored profiler options handling\n\n- extracted profiler options in a separate class\n- switched from system property reading to command line arguments\n- added profile based compilation options to CompilerOptions\n- removed no longer used kProfile compilation filter\n- optimize dex files only if the profiler is enabled\n- clean up unused arguments\n\nBug: 12877748\nBug: 15275634\nChange-Id: I37ff68e7694370950ce8db2360562e9058ecebb7\n" }, { "commit": "f00571c4e1ae202b3b4acb6b47cbe23a65178f7f", "tree": "484d129a8265d03eff56c796e92e302e23ad5e16", "parents": [ "f6b5378ba182c5c26c556ae34638afc38adef519" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu May 29 17:29:47 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Jun 02 09:59:22 2014 -0700" }, "message": "Added workaround passing empty arg array for proxy invocation.\n\nThis recreates old Dalvik behavior for older target sdk versions,\nbut will still pass null for newer ones.\n\nBug: 13247236\n\n(cherry picked from commit 01d5a146e20660bd06f026c16f19ec080f8fdd7b)\n\nChange-Id: I911889cf559ad8d9f37ea9be3929387c86446851\n" }, { "commit": "b74cd29802f364b4cec88f4913fa38ade26b8fab", "tree": "73ccdce1c5a6e57d5763cae1f889e53dca2b5e24", "parents": [ "11138c73e0c5f8a90d1eeed3c48a2b63b310671c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 29 14:31:33 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 30 13:58:22 2014 -0700" }, "message": "Compaction cleanup for FindArrayClass.\n\nWe now pass double pointer in to signify that it can cause thread\nsuspension, this double pointer gets wrapped by a handle if don\u0027t\nfind the array class in the cache.\n\nChange-Id: I43436b6c35597c7252b65d1180baddb5ac4caabb\n" }, { "commit": "0cd81352a7c06e381951cea1b104fd73516f4341", "tree": "fcc8ff9cdeb608d0913097aa1fb64d1879f11ffa", "parents": [ "dfd301f19800a4f168283cf2d0b15b2c09071955" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 22 16:48:55 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 23 12:55:45 2014 -0700" }, "message": "Revert \"Revert \"Fix an outstanding compaction bug in interpreter.\"\"\n\nFixed the generic trampoline to not use ToJObject when unnecessary.\n\nBug: 15167269\n\nThis reverts commit 3bdb873122964da7937eb070cbcf2ef638a8e459.\n\nChange-Id: I0525d0e0f3afb753c770e1572070a0fa22b02271\n" }, { "commit": "3bdb873122964da7937eb070cbcf2ef638a8e459", "tree": "3d29123f33853e11adb67bdd6ac56f19ee798a1f", "parents": [ "e09ae0920be57760fb390b6944bce420fa0b5582" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 22 22:06:51 2014 +0000" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 22 22:06:51 2014 +0000" }, "message": "Revert \"Fix an outstanding compaction bug in interpreter.\"\n\nThis reverts commit e09ae0920be57760fb390b6944bce420fa0b5582.\n\nChange-Id: I48036306130d5ccfec683d0dc3e9a642a02ee9c1\n" }, { "commit": "e09ae0920be57760fb390b6944bce420fa0b5582", "tree": "acc40266093df4289ffb6728c979cafd6b5114d2", "parents": [ "b8033db2a8dc6f7c7e29b1552177542964f56e44" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 15 12:39:19 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 22 10:47:44 2014 -0700" }, "message": "Fix an outstanding compaction bug in interpreter.\n\nFixed a bug in DoFieldPut where the FieldHelper GetType could cause\nthread suspension which would result in a stale obj.\n\nAdded more handles in the class linker to facilitate moving fiels\nand methods in the future.\n\nRemoved un-necessarly passing handle references since these are value\ntypes and don\u0027t need to be passed by reference.\n\nAdded a special NullHandle type which allows null handles without a\nhandle scope.\n\nChange-Id: I1b51723920a2e4f4f8b2907066f578a3e879fd5b\n" }, { "commit": "a705376e3355fd307bfbf7bf72d6944c63a9f2f7", "tree": "9973ff31bd7d51ea1922a874725e62f4b15f739d", "parents": [ "bf17a08fb48fa296acb09904d4c10ba42f63f55d" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri May 16 16:56:45 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu May 22 10:29:16 2014 +0200" }, "message": "Support new VMRuntime native methods\n\nImplements vmInstructionSet, is64Bit and isCheckJniEnabled methods.\n\nBug: 14888999\nBug: 14888124\nChange-Id: I8a2d3f22f84a093be2fbc74464af6aa7b2a2bebb\n" }, { "commit": "f832284dd847ff077577bb5712225430bbbb3b67", "tree": "44f6b91098639c6ebc438b4ec998d0dc128cef9a", "parents": [ "8f0776768712b2021aa8fb649b51017b9f0fc7a9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 16 10:59:25 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun May 18 12:50:33 2014 -0700" }, "message": "Delete ClassHelper and fix compaction bug in GetDirectInterface\n\nCleanup helps to prevent compaction bugs. Fixed a fairly serious\ncompaction error caused by calling ClassHelper::GetDirectInterface\nwithout handling the case where it causes thread suspension due to\nResolveType.\n\nBug: 8981901\n\nChange-Id: I82b3bb6dd48d21eb6ece7aae0733c4a23c2bc408\n" }, { "commit": "cbfb8b2f30de46632d3499a838661765451845c5", "tree": "1c1e2bacb0dd16a7bb22cf26cbd7ad84d6a7f370", "parents": [ "6099df8c0695a81a8f25c47ff801998b12d09d1b", "2d10b206f9d0b97396b7dadb9a6415cd39efd341" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed May 14 19:09:45 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 14 19:09:45 2014 +0000" }, "message": "Merge \"Ensure JNI primitive array type is appropriate.\"" }, { "commit": "2d10b206f9d0b97396b7dadb9a6415cd39efd341", "tree": "032625cf6175d1749739bca3c9d673b027ba92c1", "parents": [ "6fb66a2bc4e1c0b7931101153e58714991237af7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 12 19:15:18 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed May 14 12:03:16 2014 -0700" }, "message": "Ensure JNI primitive array type is appropriate.\n\nCheck the primitive array type for GetPrimitiveArray, ReleasePrimitiveArray,\nGetPrimitiveArrayRegion and SetPrimitiveArrayRegion matches the given array\ntype. Check the GetPrimitiveArrayCritical and ReleasePrimitiveArrayCritical are\ngiven a primitive array.\nAdd unit tests that null parameters lead to fatal errors, not crashes. Fix\nissues where CheckJNI assumed non-null arguments.\nTidy testing code via the use of nullptr. Add a few extra checks.\nEnsure arrays of void are not able to be created, use RI compatible\nNoClassDefError.\n\nBug: 14817823\n\nChange-Id: I9903bcd800d0da1988ced07f61fb97b783c5deab\n" }, { "commit": "eb8167a4f4d27fce0530f6724ab8032610cd146b", "tree": "bcfeaf13ad78f2dd68466bbd0e20c71944f7e854", "parents": [ "6fb66a2bc4e1c0b7931101153e58714991237af7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 07 15:43:14 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue May 13 14:45:54 2014 -0700" }, "message": "Add Handle/HandleScope and delete SirtRef.\n\nDelete SirtRef and replaced it with Handle. Handles are value types\nwhich wrap around StackReference*.\n\nRenamed StackIndirectReferenceTable to HandleScope.\n\nAdded a scoped handle wrapper which wraps around an Object** and\nrestores it in its destructor.\n\nRenamed Handle::get -\u003e Get.\n\nBug: 8473721\n\nChange-Id: Idbfebd4f35af629f0f43931b7c5184b334822c7a\n" }, { "commit": "987560fee798e48fb725c44b796d8ca7a5872ad6", "tree": "d61f35a129847f3f5fbed1436649e75acd940b3f", "parents": [ "44b0053fdb7ad8a30138d29f714172a7dc69efb8" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Apr 22 11:42:59 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Apr 22 14:52:19 2014 -0700" }, "message": "Remove support for app JNI workarounds.\n\nChange-Id: I4396df7e93fcace4b5b19c2c387e5c30089182a6\n" }, { "commit": "53b8b09fc80329539585dcf43657bc5f4ecefdff", "tree": "cac0f82fbb89bd907104e3fed6c36203e11a3de0", "parents": [ "0dea9872082bc3e576ed6cefed86b0d6c0c45ffd" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 13 23:45:53 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 11:28:10 2014 -0700" }, "message": "Refactor reflective method invocation.\n\nMove invocation code out of JNI internal into reflection, including ArgArray\ncode. Make reflective invocation use the ArgArray to build arguments rather\nthan allocating a jvalue[] and unboxing arguments into that.\nMove reflection part of jni_internal_test into reflection_test.\nMake greater use of fast JNI.\n\nChange-Id: Ib381372df5f9a83679e30e7275de24fa0e6b1057\n" }, { "commit": "39c3bfbd03d85c63cfbe69f17ce5800ccc7d6c13", "tree": "fa777039b3f7c34f3dd322d04307766246526080", "parents": [ "0918614b7434783477e8668df7850a7aaf8d5611" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Tue Jan 28 18:33:52 2014 -0800" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Fri Mar 07 13:42:48 2014 -0800" }, "message": "Make use of profiling information for dex2oat\n\nIf the profile file exists, the compiler driver will read it\nand store the data in an internal map. Then, when we want to work\nout whether to compile a method or not, the map is consulted and if\nthe method shows up with a high enough percentage of use we compile it.\n\nThe profile file itself is created by installd and is writeable by the\napp. The file is in /data/dalvik-cache/profiles and is named by\nthe package name.\n\nThis also modifies the profiler itself to:\n\n1. Only count runnable threads (not suspended threads) in the profile\n2. Use system properties to allow tuning of the profile parameters\n3. Merge profiles from multiple processes using file locking.\n\nBug: 12877748\nChange-Id: Iab2f3a327a2860db2a80d5724277d6c626227f2b\n\nConflicts:\n\tcompiler/dex/frontend.cc\n\tcompiler/dex/mir_analysis.cc\n\tcompiler/dex/verification_results.cc\n\tcompiler/driver/compiler_driver.cc\n\tdex2oat/dex2oat.cc\n\truntime/class_linker.cc\n\truntime/runtime.cc\n\truntime/runtime.h\n" }, { "commit": "a5f9de040a7a740b7b4979bc8d47f95eb05718ae", "tree": "a06560b27a6e2ec1e7c90b3b8f0f1f6344f09c81", "parents": [ "c7a8981098646b4e255d9ede1f3798023bac6e84" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Feb 28 16:48:42 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 06 15:38:06 2014 -0800" }, "message": "Defer heap transitions to heap trimming daemon.\n\nThis fixes the case where quick back and forth process state changes\nwould cause a lot of heap transitions.\n\nChange-Id: Ia5be792edcf26079b2aa23b9c115f6b0a9a39a1c\n" }, { "commit": "6dda898d47b3e8931e4404330e81b7110108e34f", "tree": "d8a0a20b8ac1ce3cd44273cf5c748d09980bd800", "parents": [ "8785d615122d4abbd22db702139584e8c472f502" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 06 11:11:48 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 06 11:43:06 2014 -0800" }, "message": "Disable compaction for jni workarounds.\n\nCompaction can\u0027t work when jni workarounds is enabled. Also some\nother refactoring.\n\nChange-Id: Ia7b0f2b39c79f5a0a5f50874d823b950ab02a0c3\n" }, { "commit": "893263b7d5bc2ca43a91ecb8071867f5134fc60a", "tree": "a99238843a9caad00122c8f7d13e031f5d81bc38", "parents": [ "2fece5941f12395a94e742313e7059a9e419994d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 04 11:07:42 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 04 14:21:58 2014 -0800" }, "message": "Avoid marking old class linker and intern table roots during pause.\n\nThe new root visiting logic has a concept of a root log which holds\nnew roots which were added since the start of the GC. This is an\noptimization since it lets us only mark these newly added roots\nduring the pause (or pre-cleaning) since the other roots intern table\nand class linker roots were marked concurrently at the start of the\nGC.\n\nBefore (EvaluateAndApplyChanges):\nMarkConcurrentRoots:\tSum: 605.193ms\nAfter:\nMarkConcurrentRoots:\tSum: 271.858ms\n\nThis should also reduce pathological GC pauses which used to be able\nto happen when the intern table or class linker became \"dirty\"\nduring the concurrent GC.\n\nChange-Id: I433fab021f2c339d50c35aaae7161a50a0901dec\n" }, { "commit": "491ca9e75fad381468dd7f5fdbff56d1a9738dd7", "tree": "4a88d05188aebbcbe868a27b58fc1a932f914ce2", "parents": [ "82e3a8ef9cd776d7026b45fe7fb6c72bcfd61394" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Mar 02 18:24:38 2014 -0800" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Mar 03 13:31:40 2014 -0800" }, "message": "Add ART -help and -showversion\n\nBug: https://code.google.com/p/android/issues/detail?id\u003d62149\nChange-Id: I5c21c0433f5f341f94e7d977e8d2d69cc46d6fe3\n" }, { "commit": "f9c612f2e657adb0764285256b228622e0789254", "tree": "c4a94053d07fc089703694ba1406a295333fecad", "parents": [ "95c2e440582f31f77a047797a5e2be4af53a8bfa" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Feb 27 10:58:17 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 28 17:13:09 2014 -0800" }, "message": "Support in VMRuntime for allocating \"unpadded\" arrays.\n\nBug: 13028925.\n\nChange-Id: I0a53645f0468aaf44d38aabe016cb610fcfefaf7\n" }, { "commit": "6fac447555dc94a935b78198479cce645c837b89", "tree": "bcf1449999084b1e1dec3dac287f6f3670d7eda0", "parents": [ "7f0ff7e7fff82566bca5f9353eaa2c4f81f0671a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 25 17:01:10 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 26 16:38:22 2014 -0800" }, "message": "Make allocations report usable size.\n\nWork-in-progress to allow arrays to fill usable size. Bug: 13028925.\nUse C++11\u0027s override keyword on GCC \u003e\u003d 2.7 to ensure that we override GC and\nallocator methods.\nMove initial mirror::Class set up into a Functor so that all allocated objects\nhave non-zero sizes. Use this property to assert that all objects are never\nlarger than their usable size.\nOther bits of GC related clean-up, missing initialization, missing use of\nconst, hot methods in .cc files, \"unimplemented\" functions that fail at\nruntime in header files, reducing header file includes, move valgrind\u0027s space\ninto its own files, reduce number of array allocation routines.\n\nChange-Id: Id5760041a2d7f94dcaf17ec760f6095ec75dadaa\n" }, { "commit": "9837939678bb5dcba178e5fb00ed59b5d14c8d9b", "tree": "00f0e6b54d7c4cac78a02752e268724157e50b6e", "parents": [ "3fcf18e25241253f23efbeebe77b2a4c4a7c54d3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 24 16:53:16 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 24 18:47:23 2014 -0800" }, "message": "Avoid std::string allocations for finding an array class.\n\nIntroduce ClassLinker::FindArrayClass which performs an array class lookup\ngiven the element/component class. This has a 16 element cache of recently\nlooked up arrays.\nPass the current thread to ClassLinker Find .. Class routines to avoid calls\nto Thread::Current().\nAvoid some uses of FindClass in the debugger where WellKnownClasses is a\nfaster and more compacting GC friendly alternative.\n\nChange-Id: I60e231820b349543a7edb3ceb9cf1ce92db3c843\n" }, { "commit": "815873ecc312b1d231acce71e1a16f42cdaf09f2", "tree": "18ba2fa951775e60b240271bfe975e6e2cfc654c", "parents": [ "2befd09cf4fe89a18a655f3e1dd310831bfa769f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Feb 13 18:02:13 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 18 10:45:12 2014 -0800" }, "message": "Change root visitor to use Object**.\n\nSimplifies code and improves the performance of root visiting since\nwe usually don\u0027t need to check to see if the object moved.\n\nChange-Id: Iba998f5a15ae1fa1b53ca5226dd2168a411196cf\n" }, { "commit": "83c8ee000d525017ead8753fce6bc1020249b96a", "tree": "d5167ed15dee2629905ac3640b6ea0578d4ae312", "parents": [ "7cba217ab0661d74deccbb97160cdf60b74d4ea3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 28 14:50:23 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 11 10:40:10 2014 -0800" }, "message": "Add root types and thread id to root visiting.\n\nEnables us to pass the root type and thread id to hprof.\n\nBug: 12680863\nChange-Id: I6a0f1f9e3aa8f9b4033d695818ae7ca3460d67cb\n" }, { "commit": "ef7d42fca18c16fbaf103822ad16f23246e2905d", "tree": "c67eea52a349c2ea7f2c3bdda8e73933c05531a8", "parents": [ "822115a225185d2896607eb08d70ce5c7099adef" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Jan 06 12:55:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Feb 06 23:20:27 2014 -0800" }, "message": "Object model changes to support 64bit.\n\nModify mirror objects so that references between them use an ObjectReference\nvalue type rather than an Object* so that functionality to compress larger\nreferences can be captured in the ObjectRefererence implementation.\nObjectReferences are 32bit and all other aspects of object layout remain as\nthey are currently.\n\nExpand fields in objects holding pointers so they can hold 64bit pointers. Its\nexpected the size of these will come down by improving where we hold compiler\nmeta-data.\nStub out x86_64 architecture specific runtime implementation.\nModify OutputStream so that reads and writes are of unsigned quantities.\nMake the use of portable or quick code more explicit.\nTemplatize AtomicInteger to support more than just int32_t as a type.\nAdd missing, and fix issues relating to, missing annotalysis information on the\nmutator lock.\nRefactor and share implementations for array copy between System and uses\nelsewhere in the runtime.\nFix numerous 64bit build issues.\n\nChange-Id: I1a5694c251a42c9eff71084dfdd4b51fff716822\n" }, { "commit": "0cb50e58bcdab321378df3b86f1e914d30992b5b", "tree": "560146dfd200432b3e0cb83827e40e08bb1b67a8", "parents": [ "4c863edfe99dd63550ee462d77820a007ec40a7d", "5ddb4104ac605d66693b55b79f26f8b8a5505e63" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 08 17:19:03 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 08 17:19:03 2014 +0000" }, "message": "Merge \"Remove intialized static storage from dex cache.\"" }, { "commit": "5ddb4104ac605d66693b55b79f26f8b8a5505e63", "tree": "9231f1e54e9a1aaca640c590d5b4f318720e263b", "parents": [ "5a2ced515a456f15dcf194843c024e835eda7dbe" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jan 07 08:58:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 08 09:15:46 2014 -0800" }, "message": "Remove intialized static storage from dex cache.\n\nThe initialized static storage array is used by compiled code to determine if\nfor a sget/sput class initialization is necessary. The compiled code typically\ndoesn\u0027t require this test as the class is pre-initialized or the class being\naccessed is the same as the current method.\n\nChange-Id: Icbc45e692b3d0ac61e559e69edb6c9b29439e571\n" }, { "commit": "0aded089f565008ba5908e395e5914ca4f91f2de", "tree": "3b1238d956e0181d00243604c83c3a0be368437d", "parents": [ "98ba6cebd22489c959d845884ee045dcbd958d41" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Thu Nov 07 13:15:11 2013 -0800" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Tue Jan 07 10:58:07 2014 -0800" }, "message": "Add sampling profiler\n\nThis adds a sampling profiler thread that runs every so often, gathering\nprofiling data and writing the results to a file in /data/data (specific to\napp running). The intention is to use these files as input to the compiler\nso that it can determine the best methods to compile.\n\nBug: 11539952\nChange-Id: I0bfbb4146fb7966673c792f017ffac8107b6272d\n" }, { "commit": "ca2a24da53869a04e1947aa46d06ccce5247d6f4", "tree": "724eccc02dec0f0ff3fc4e0b505511d691e795e6", "parents": [ "5da3778746375b73e7e77c5f1371f29684674776" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 25 15:12:12 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 25 17:25:15 2013 -0800" }, "message": "Add interface for updating process state.\n\nCalled from activity manager to let the heap know when it should\nperform compaction and trimming.\nBug: 8981901\n\nChange-Id: Ib8ea48d2dc9d6901c3f2e0554391721d6691e726\n" }, { "commit": "cbb2d20bea2861f244da2e2318d8c088300a3710", "tree": "9735d496716cf165ea0ee2d7e2f62d723ffc7734", "parents": [ "d31fb9718a6180304cd951619dc36be8e090a641" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Nov 14 17:45:16 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 20 11:14:11 2013 -0800" }, "message": "Refactor allocation entrypoints.\n\nAdds support for switching entrypoints during runtime. Enables\naddition of new allocators with out requiring significant copy\npaste. Slight speedup on ritzperf probably due to more inlining.\n\nTODO: Ensuring that the entire allocation path is inlined so\nthat the switch statement in the allocation code is optimized\nout.\n\nRosalloc measurements:\n4583\n4453\n4439\n4434\n4751\n\nAfter change:\n4184\n4287\n4131\n4335\n4097\n\nChange-Id: I1352a3cbcdf6dae93921582726324d91312df5c9\n" }, { "commit": "590fee9e8972f872301c2d16a575d579ee564bee", "tree": "b02db45c72f1911ec896b93379ada0276aea3199", "parents": [ "5b70680b8df6d8fa95bb8e1070d0107f3d388940" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 13 13:46:47 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 11 15:34:27 2013 -0800" }, "message": "Compacting collector.\n\nThe compacting collector is currently similar to semispace. It works by\ncopying objects back and forth between two bump pointer spaces. There\nare types of objects which are \"non-movable\" due to current runtime\nlimitations. These are Classes, Methods, and Fields.\n\nBump pointer spaces are a new type of continuous alloc space which have\nno lock in the allocation code path. When you allocate from these it uses\natomic operations to increase an index. Traversing the objects in the bump\npointer space relies on Object::SizeOf matching the allocated size exactly.\n\nRuntime changes:\nJNI::GetArrayElements returns copies objects if you attempt to get the\nbacking data of a movable array. For GetArrayElementsCritical, we return\ndirect backing storage for any types of arrays, but temporarily disable\nthe GC until the critical region is completed.\n\nAdded a new runtime call called VisitObjects, this is used in place of\nthe old pattern which was flushing the allocation stack and walking\nthe bitmaps.\n\nChanged image writer to be compaction safe and use object monitor word\nfor forwarding addresses.\n\nAdded a bunch of added SIRTs to ClassLinker, MethodLinker, etc..\n\nTODO: Enable switching allocators, compacting on background, etc..\n\nBug: 8981901\n\nChange-Id: I3c886fd322a6eef2b99388d19a765042ec26ab99\n" }, { "commit": "cf5077ac14f0922b6104a8a03fd66d97a490a3dd", "tree": "e5a4c42b87f06dcf5dd8bce1de992d3e2a2430fa", "parents": [ "1b8cb967143d5c46db1db8ac940a73d111be3628" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 31 12:37:54 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 31 12:37:54 2013 -0700" }, "message": "Remove unused length from DexFile GetString calls.\n\nAddress extra review comments from commit\ndfb325e0ddd746cd8f7c2e3723b3a573eb7cc111.\n\nChange-Id: If76e81e7af5870431901de0bf561e0f827435fe3\n" }, { "commit": "dfb325e0ddd746cd8f7c2e3723b3a573eb7cc111", "tree": "0b63738b04404f42015da7164dfe5ec506452b02", "parents": [ "eda6df7374bf61c933d4a36635e2a9a30c9892a0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 30 01:00:44 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 30 01:00:44 2013 -0700" }, "message": "Don\u0027t use UTF16 length as length for MUTF8.\n\nBug 11367555.\n\nChange-Id: Ia0b07072a1a49d435c3b71ed9a668b316b7ff5d8\n" }, { "commit": "e2179ebd18f9b3c7b0ad4c1ce1a2b76a5781064c", "tree": "06bbbb32153613666994a09474b1ef3d898c4f6a", "parents": [ "dbf39a79d17f226936430b025ce1f0beeac2f441", "881392f55e566a7c6407e7d321a31836d2d8865f" ], "author": { "name": "Alex Klyubin", "email": "klyubin@google.com", "time": "Fri Oct 25 13:09:49 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Fri Oct 25 13:09:49 2013 -0700" }, "message": "am 881392f5: Adjust the name of setTargetSdkVersion native method.\n\n* commit \u0027881392f55e566a7c6407e7d321a31836d2d8865f\u0027:\n Adjust the name of setTargetSdkVersion native method.\n" }, { "commit": "881392f55e566a7c6407e7d321a31836d2d8865f", "tree": "35043d24b73a5af6d90706aae9cffd96243c5b7e", "parents": [ "7510d4cc0111258b53ae74cf8292f31c6ca5f001" ], "author": { "name": "Alex Klyubin", "email": "klyubin@google.com", "time": "Thu Oct 24 16:31:56 2013 -0700" }, "committer": { "name": "Alex Klyubin", "email": "klyubin@google.com", "time": "Thu Oct 24 16:31:56 2013 -0700" }, "message": "Adjust the name of setTargetSdkVersion native method.\n\nChange-Id: I60f1febc1f8ddf2a97925a48ddf232508d5c44f9\n" }, { "commit": "57e6d8a99058e5c74d5244b68a5f4d53526fa108", "tree": "bca484bbb16e9222a30337fa1c094b5b985df23c", "parents": [ "4e6274692288d3e04b91e1319b0b48fc9d5b6deb" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Oct 20 22:53:26 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Oct 20 22:53:26 2013 -0700" }, "message": "Tracking change to InternTable::VisitRoots\n\nChange-Id: I260e8f07ae4c4dde048a1ad4bb6208624af965e9\n" }, { "commit": "4e6274692288d3e04b91e1319b0b48fc9d5b6deb", "tree": "ab356d25b69c9a87b9719a510770e2757174538e", "parents": [ "77a41a69201579792397ba33bc93f5f5a77da981", "5d79c5deba3fa27825a2d5a399713b9490f5ca28" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Oct 20 21:44:11 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Oct 20 21:44:11 2013 -0700" }, "message": "resolved conflicts for merge of 5d79c5de to dalvik-dev\n\nChange-Id: I0afb78246053c5adf4b5ed4a062c9e0cfda8d398\n" }, { "commit": "e810452722ac83b294d1f7aa80bdd88e547d5af0", "tree": "6000d7851d7e60936a2ec6e7ef6ca935bc374832", "parents": [ "67fe2b41a6afccf6ab1a58879eae3e0e8f3d6c7a" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Oct 15 21:56:36 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Oct 20 21:38:53 2013 -0700" }, "message": "Preload DexCaches\n\nBug: 11045348\nChange-Id: I6f9c0d11613b6b4933a04ae23dbf4bc7879cea65\n" }, { "commit": "1eb512d33f94d1dd7ea38263307ba0f7a0dfa653", "tree": "b4d4d9b16013ab90fb4b40d23013d7ef44bb5852", "parents": [ "b917ea1a62aa0ab8eca3f689ef64b5be34e11abb" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 18 15:42:20 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sun Oct 20 14:55:26 2013 -0700" }, "message": "Fast JNI support.\n\nUse a modifier to signal a native method is a fast JNI method. If the\nmodifier is set then don\u0027t perform runnable transitions.\n\nChange-Id: I7835b4d837bfdd1cb8e2d54b919c0d5e6cf90499\n" }, { "commit": "be031fff278799984166ec866c2dd202447e0f23", "tree": "48c60def3d4727cd5d8ba5a83b12ed27092bbb4b", "parents": [ "34633b22f74393344987a50b8aaee548a9dadc18" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Oct 08 16:42:37 2013 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Oct 09 17:29:11 2013 -0700" }, "message": "Remove four counter increments out of the allocation path.\n\n- This change removes the four allocation counters (the number of\nbytes allocated, the number of objects allocated, the number of bytes\nallocated ever, the number of objects allocated ever) from the\ndlmalloc space allocation path. Now those counter values are computed\non the fly based on a combination of the two new counters that are\nincremented by the GC side (the number of bytes freed ever and the\nnumber of objects freed ever) and mspace_inspect_all() calls.\n\n- This results in a 1-2% speedup (though with some noise) in Ritz\nMemAllocTest on Nexus 4.\n\nBug: 9986565\nChange-Id: Id9a8e05a745ac1e5ea7a2b5fd9319814a9d4af13\n" }, { "commit": "a436fde2762664a3ecdda5eefcadd20b2e104f59", "tree": "22431bc387483220728a3bc99ec7314abce3ad60", "parents": [ "10cc81056261625902a8a073bafb9f499eec4ff7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 27 23:34:06 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 27 23:37:33 2013 -0700" }, "message": "Handle OOMEs in class linker with grace.\n\nCheck for OOMEs and then fail due to them in class loading.\nMake the compiler driver spot OOMEs during resolution and abort compilation to\navoid needless GC thrash then eventual death.\nAllocate the pre-allocated OOME during Runtime::Init as Runtime::Start isn\u0027t\ncalled in the context of the compiler/tools.\n\nChange-Id: Id72199d0fe82001b5bf22758b3cdc9cc4b8efbb9\n" }, { "commit": "1a4d6d809b4c24580b4667e3075d545738d83ec7", "tree": "2b1ba966630ab935e85aa8519d3ed33b907ce107", "parents": [ "e2f77e7592842cad5a59e73535b8982c35e156ca" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 13 20:07:05 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 13 20:07:05 2013 -0700" }, "message": "If CheckJNI is enabled don\u0027t allow app workarounds\n\nChange-Id: I030231d2b1ebcabb5c3922ea91230b40998711ba\n" }, { "commit": "cc386f94631dd1d60b0b0b2d4e1e243dda363f32", "tree": "dab43dbcd94c1a92d32f88a90cf900c8c4d90da9", "parents": [ "36e58b8ab7f2cf8609ecebb09dd302db3f51f40c", "c39e342317d77e701b4cd01cd5b05902e6512f4b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 09 13:49:35 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Fri Aug 09 13:49:35 2013 -0700" }, "message": "am c39e3423: Use JNI to access activity manager process state.\n\n* commit \u0027c39e342317d77e701b4cd01cd5b05902e6512f4b\u0027:\n Use JNI to access activity manager process state.\n" }, { "commit": "c39e342317d77e701b4cd01cd5b05902e6512f4b", "tree": "b43a85e1c60377505d56d0d91fe4da704f4458fc", "parents": [ "f6a7406e9210449ddd23bff23ad208e1877b35b7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 07 16:41:36 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 09 11:03:47 2013 -0700" }, "message": "Use JNI to access activity manager process state.\n\nAfter a GC occurs, we use JNI to access activity manager process\nstate to decide whether or not we will trim.\n\nChange-Id: Iad981e3a7cdc694729d8792cc0f19e0262154388\n" }, { "commit": "3839f7a4d482049cfb8d5a0a9e955e7ede27e1d3", "tree": "423184963c9824ae908d5e7f516f59ded973a1ed", "parents": [ "d8de72f54f6076218373af4abf76dd2a548f13e3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Aug 07 23:26:58 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Aug 07 23:26:58 2013 -0700" }, "message": "Fix portable build.\n\nChange-Id: Icec79ae87b027e49e9d71404b127d61601bbb3d6\n" }, { "commit": "848871b4d8481229c32e0d048a9856e5a9a17ef9", "tree": "4be4602e3becc899f76a29a87618801458fe04b9", "parents": [ "423fb4d70f2ac36bf9f630146b4150771a8e7e76" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Aug 05 10:56:33 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Aug 07 22:17:00 2013 -0700" }, "message": "Entry point clean up.\n\nCreate set of entry points needed for image methods to avoid fix-up at load time:\n - interpreter - bridge to interpreter, bridge to compiled code\n - jni - dlsym lookup\n - quick - resolution and bridge to interpreter\n - portable - resolution and bridge to interpreter\n\nFix JNI work around to use JNI work around argument rewriting code that\u0027d been\naccidentally disabled.\nRemove abstact method error stub, use interpreter bridge instead.\nConsolidate trampoline (previously stub) generation in generic helper.\nSimplify trampolines to jump directly into assembly code, keeps stack crawlable.\nDex: replace use of int with ThreadOffset for values that are thread offsets.\nTidy entry point routines between interpreter, jni, quick and portable.\n\nChange-Id: I52a7c2bbb1b7e0ff8a3c3100b774212309d0828e\n" }, { "commit": "bbf8a09601f5379293b498b7d181ffc2a1f93b6a", "tree": "88ffc1d3de2b367154c4413d0471e7d96ede73b7", "parents": [ "1359a18a4b9b7c9442afcbb1bf7eec1398b85ae6", "3da08f4d2cef02bd0a3cc45ae3b7555fc547d29a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jul 19 09:55:14 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jul 19 09:55:14 2013 -0700" }, "message": "resolved conflicts for merge of 3da08f4d to dalvik-dev\n\nChange-Id: Ia237450dac762602bb159904a1032e6fccb2d7fd\n" }, { "commit": "82353313bbe00d033b6733535d0dcd6f56c36d8d", "tree": "7f154381b5288793aedaff313a0b2e7e800e8967", "parents": [ "0cd7ec2dcd8d7ba30bf3ca420b40dac52849876c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 18 10:47:51 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 18 14:33:24 2013 -0700" }, "message": "Add interface for updating the heap\u0027s process state\n\nIn the future, the heap\u0027s process state will be used\nto determine GC behavior.\n\nChange-Id: Iba4f038d28dbf483b6573d8feb25e4246ead1d50\n" }, { "commit": "1db9113bcc12368e405583804ceb8aa7c80cc0cd", "tree": "b826b626f901b6012adecf697cb979d371aca78f", "parents": [ "bba5dd55b7deda3a3271be502f1d3b0c30a759d6", "7940e44f4517de5e2634a7e07d58d0fb26160513" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 18:05:20 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 18:05:53 2013 -0700" }, "message": "resolved conflicts for merge of 7940e44f to dalvik-dev\n\nChange-Id: I6529b2fc27dfaedd2cb87b3697d049ccabed36ee\n" }, { "commit": "7940e44f4517de5e2634a7e07d58d0fb26160513", "tree": "ac90242d96229a6942f6e24ab137bc1f8f2e0025", "parents": [ "5cd9e3b122f276f610980cbaf0d2ad6ed4cd9088" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 13:46:57 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 17:49:01 2013 -0700" }, "message": "Create separate Android.mk for main build targets\n\nThe runtime, compiler, dex2oat, and oatdump now are in seperate trees\nto prevent dependency creep. They can now be individually built\nwithout rebuilding the rest of the art projects. dalvikvm and jdwpspy\nwere already this way. Builds in the art directory should behave as\nbefore, building everything including tests.\n\nChange-Id: Ic6b1151e5ed0f823c3dd301afd2b13eb2d8feb81\n" }, { "commit": "bba5dd55b7deda3a3271be502f1d3b0c30a759d6", "tree": "74bf9e72fbd66def253f317b9d8b6d7b8d67b86b", "parents": [], "author": { "name": "Anwar Ghuloum", "email": "anwarg@google.com", "time": "Fri Jul 12 22:57:56 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Fri Jul 12 22:57:56 2013 +0000" }, "message": "Merge \"Semi-pruned SSA support for sea-ir.\" into dalvik-dev" }, { "commit": "cef450c613d8324db7c418130596182b8e9a5cdd", "tree": "2355aca9a92bd8ff515c7b5f759f92220c396467", "parents": [ "55163b8c34f40c9107cc3aa72e0d2f542f7a164e" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jun 28 14:38:26 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Jun 30 13:17:06 2013 -0700" }, "message": "add dalvik.system.VMRuntime.vmLibrary implementation for ART\n\nChange-Id: Ibc1a5abb5a0ee49cb20e55dc2e169a224fcf9d8b\n" }, { "commit": "1d54e73444e017d3a65234e0f193846f3e27472b", "tree": "1de93661e95a0ce6fa78fdfc23d0cfd3dd2a06f7", "parents": [ "4c22e7eabef3f815841dfc6e0d5bbead96150752" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 02 21:10:01 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 21 11:10:42 2013 -0700" }, "message": "GC clean up.\n\nGreater use of directories and namespaces.\nFix bugs that cause verify options to fail.\nAddress numerous other issues:\n\nGC barrier wait occurring holding locks:\nGC barrier waits occur when we wait for threads to run the check point function\non themselves. This is happening with the heap bitmap and mutator lock held\nmeaning that a thread that tries to take either lock exclusively will block\nwaiting on a thread that is waiting. If this thread is the thread we\u0027re waiting\nto run the check point then the VM will deadlock.\nThis deadlock occurred unnoticed as the call to check for wait safety was\nremoved in: https://googleplex-android-review.googlesource.com/#/c/249423/1.\n\nNewTimingLogger:\nExisting timing log states when a split ends but not when it begins. This isn\u0027t\ngood for systrace, in the context of GC it means that races between mutators\nand the GC are hard to discover what phase the GC is in, we know what phase it\njust finished and derive but that\u0027s not ideal.\n\nSupport for only 1 discontinuous space:\nCode special cases continuous and large object space, rather than assuming we\ncan have a collection of both.\n\nSorted atomic stacks:\nUsed to improve verification performance. Simplify their use and add extra\nchecks.\n\nSimplify mod-union table abstractions.\n\nReduce use of std::strings and their associated overhead in hot code.\n\nMake time units of fields explicit.\n\nReduce confusion that IsAllocSpace is really IsDlMallocSpace.\n\nMake GetTotalMemory (exposed via System) equal to the footprint (as in Dalvik)\nrather than the max memory footprint.\n\nChange-Id: Ie87067140fa4499b15edab691fe6565d79599812\n" }, { "commit": "62d6c772205b8859f0ebf7ad105402ec4c3e2e01", "tree": "e2f2ba6d71ed5a39c9f6909e3f7c08e998053315", "parents": [ "c9b17c7ee96cd04fac9048aab624ed554fe260bf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 08:32:07 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Apr 08 14:24:13 2013 -0700" }, "message": "Interpreter entries and instrumentation as a listener.\n\nMake the instrumentation responsible for whether we want method entry/exit\nstubs, and allow it to use interpreter entry stubs when instruction by\ninstruction instrumentation is required. Improve deoptimization so more JDWP\ntest cases are passing.\n\nRefactor exception debug posting, in particular improve reporting in the\ninterpreter. Improve class linker exception throwing so that broken dex files\nare more likely to be reported. Fixes the performance issue Bug: 8410519.\n\nFix some error reporting lock level errors for the large object space. Make\nfast object verification faster.\n\nAdd some debug mode robustness to finding dex PCs in GC maps.\n\nAdd printf attributes to JniAbortF and fix errors.\n\nExpand run-test 044 to test return behaviors and fix issues with not throwing\nappropriate exceptions for proxies.\n\nEnsure causes are reported with a class linker NoClassDefFoundError and JNI\nNoSuchFieldError.\n\nRemove unused debugMe and updateDebuggerFromCode.\n\nThere\u0027s a minor sizing tweak to the arg array builder, and an extra reference\narray check in the interpreter.\n\nSome clean-up of trace code.\n\nFix reg type cache destructor if it is called after the reg type cache is\nshutdown (as is the case in oatdump).\n\nChange-Id: I6519c7b35df77f978d011999354c864f4918e8ce\n" }, { "commit": "4f6ad8ab428038129b2d0d6c40b7fd625cca15e1", "tree": "d1025ed9203d0cdf7aa959d9cecd8dc1f0c0bfeb", "parents": [ "aabe8adacc7bf3a8ab6f6f3dd37fc839b6fd762f" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 18 15:27:28 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 18 15:29:59 2013 -0700" }, "message": "Various performance improvements.\n\nPerformance had regressed due to verify object and method invocation changes.\nAvoid trampolines for static calls in same class.\nVarious inlining changes.\nMake verify object something that\u0027s only compiled-in in debug builds.\n\nChange-Id: Ia261a52232c3b10667c668f8adfadc0da3048bc5\n" }, { "commit": "c928de90ad22ecdf83c18a07008409595f13d3b1", "tree": "065a08f379960f7ba946318aaa742d7ddd0c94bd", "parents": [ "0f827169742aad6209d830db773a101849c32a83" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 14:30:44 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 14:30:44 2013 -0800" }, "message": "Remove Iceland.\n\nART_USE_LLVM_COMPILER is removed and when necessary ART_USE_PORTABLE_COMPILER\nis used in #ifdefs.\n\nChange-Id: Iffa9ce5b0246c7c427ccc4e67ecc134624632e55\n" }, { "commit": "2dd0e2cea360bc9206eb88ecc40d259e796c239d", "tree": "9d619dc9508cbe73e4793bf6f08cbc761abfb48a", "parents": [ "ac21b797b3a425975d656d6b84a7b24401d35f42" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jan 24 12:42:14 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 30 16:48:31 2013 -0800" }, "message": "Directory restructuring of object.h\n\nBreak object.h into constituent files.\nReduce number of #includes in other GC header files.\nIntroduce -inl.h files to avoid mirror files #include-ing each other.\nCheck invariants of verifier RegTypes for all constructors.\n\nChange-Id: Iecf1171c02910ac152d52947330ef456df4043bc\n" }, { "commit": "4893188d8ff9744f5d37b344e5a0bd582242eded", "tree": "dc5ac7a136b6b084fefb246ce2a2fde71bd06ebc", "parents": [ "28d76682d591034a5bcb610487c1f2db32d01c84" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jan 22 14:35:16 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jan 22 16:24:35 2013 -0800" }, "message": "Avoid heap trims in foreground and audio apps.\n\nHeap trimming the dlmalloc spaces holds their locks causing jank.\nImprove diagnostic message to break apart managed and native heap trims and\ndetail amount of heap advised back to the kernel.\n\nChange-Id: Ic08cf4749342cbd057b162b901d1676bb7e6418b\n" }, { "commit": "3056d0ca38acc3d829a13ec8f97d35a002ee354e", "tree": "395028dc5fbfaf00f52594fe2df34b57b88e65f4", "parents": [ "ec139de9e77663c13a907a5244267db8bed947b5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 19 10:49:56 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 19 11:33:07 2012 -0700" }, "message": "Improve accuracy of heap trim times\n\nBefore, WaitForConcurrentGc was being called inside of Heap::Trim.\nThis caused the printed trim times to be larger than they should be.\n\nChange-Id: Icc76b5ed7fb99350536d48a5215e7c1fdb8b4567\n" }, { "commit": "1c23e1edb7361bbaec6e57fca86d8d3797960ad2", "tree": "af8aa791777271fc5105f19a70aa972768493856", "parents": [ "ed6d5ed3494648780b9e91422c90d6bc22c16b79" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 12 14:14:11 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 12 17:34:08 2012 -0700" }, "message": "More space refactoring.\n\nAdd common interface, AllocSpace.\n\nRenamed the old AllocSpace to DLMallocSpace.\n\nAdded an new option enforce_target_size_, which when enabled, doesn\u0027t let\nthe heap grow past ideal heap size calculated during last Gc.\n\nRemoved redundant AllocationSize calls.\n\nMoved large object space to its own file instead of being in space.h/cc.\n\nChange-Id: I15e60531114bf213800599737cbd66ef44b46b15\n" }, { "commit": "7469ebf3888b8037421cb6834f37f946646265ec", "tree": "b1eb8cd8d1e3b6263c2de77616020de4b4b5019f", "parents": [ "d8195f19840911a73b1491dfc8e7c18139753731" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 24 16:28:36 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 08 18:02:42 2012 -0700" }, "message": "Refactor and remove copy mark bits.\n\nRefactor code GC realted code to be in a GC folder.\n\nRemove copy mark bits by using pointer changing instead.\n\nEnable concurrent sweeping of system weaks.\n\nFix non concurrent GC plan.\n\nChange-Id: I9c71478be27d21a75f8a4e6af6faabe896e5e263\n" }, { "commit": "50b35e2fd1a68cd1240e4a9d9f363e11764957d1", "tree": "4f3c58b7b293380f19e4f33357cb352e3330772e", "parents": [ "8e9a1497f0d7da4d55f6e7ed8a7d96ba6db7222d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 04 10:09:15 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 05 11:07:39 2012 -0700" }, "message": "Explicitly pass Thread::Current to MutexLock and Alloc.\n\nChange-Id: I8b75bc0617915465f102815b32306aa7760dcae4\n" }, { "commit": "2fde53367dbe721e5273c34b590e67112322cc9e", "tree": "c5e10aa7d9d41b29298a42a0b96807787e27c574", "parents": [ "2e447e0e8725503d975c50fbbd13df4c769ca85e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 14 14:51:54 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 04 10:14:03 2012 -0700" }, "message": "Refactor spaces and add free list large object space\n\nAdded some more abstraction for spaces, we now have ContinuousSpaces and DiscontinousSpaces.\n\nAdded a free list version of large object space.\n\nPerformance should be better than the memory map version since we avoid creating more than\none memory map.\n\nAdded a cause for Gc which prints with the Gc message, dalvik has this as well.\n\nChange-Id: Ie4aa6b204fbde7193e8305eb246158fae0444cc1\n" }, { "commit": "81d425b0b232962441616f8b14f73620bffef5e5", "tree": "cd1e46656269acf20e78817d675e56d9b9133e6c", "parents": [ "bfaf917edbb1de8d158c3615e0da8ac3143d10c8" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 27 16:03:43 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 27 17:19:25 2012 -0700" }, "message": "Pass self to lock methods.\n\nThis avoids frequent recomputation of\nThread::Current/pthread_getspecific.\n\nAlso add a futex based reader/writer mutex that is disabled.\n\nChange-Id: I118fdb99ef1d1c4bfda6446ba3a0d8b6ab31eaee\n" }, { "commit": "b726dcb581bf72da46527378ccb6889020f0e6e9", "tree": "c0383ed788652926e93147e50d659ed226519548", "parents": [ "fd678beb171a4686a4f2d53ca4188a4ade8fa54e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 08:57:23 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 09:06:19 2012 -0700" }, "message": "Rename GlobalSynchronization to Locks\n\nAlso address some review comments in common_throws relating to\nToStr\u003cInvokeType\u003e and exception detail messages.\n\nChange-Id: Ibf2c0f147689fa236d349bd7f01eed3c2522552b\n" }, { "commit": "fd678beb171a4686a4f2d53ca4188a4ade8fa54e", "tree": "ab932e92d683d8033241bb2248da98e76249d3eb", "parents": [ "5bfa60ffcc953340feb711ed05cf576ac821905e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Aug 30 14:50:54 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 04 14:58:14 2012 -0700" }, "message": "Additional heap verification for the Gc\n\nThere are now two verification booleans which can be enabled. When these get enabled, it verifies that each live object only references other live objects.\n\nChanged SetClass to use SetFieldPtr to avoid having an extra card mark. This is safe since all classes are held live by the class linker.\n\nChange-Id: I005bb59e5cc8153a79d3ccb3d7b5cabd29fb4051\n" }, { "commit": "15bf2d34efa2218e287b584fb3653d268b9edc8d", "tree": "4435664bd2c9a041d8c355940f2b5c95e03d38f7", "parents": [ "e4f0b2ab4abd8e942a099e9b9b4682fbdd9eb21c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 28 17:33:04 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Aug 30 17:22:52 2012 -0700" }, "message": "Use bionic\u0027s dlmalloc 2.8.5.\n\nAlso fix free space computations for DDMS.\n\nChange-Id: Icbc045b5461af89a0516f37f01acaaa815205348\n" }, { "commit": "b18e77abdb06a443744fbb6589e0932fa89f6073", "tree": "0e77099cfedbbb13a1fee3c1b93c34305b447d1c", "parents": [ "22ad71521f3006b9bf344738c0276a501934fc01" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Aug 21 14:20:03 2012 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Aug 21 14:20:21 2012 -0700" }, "message": "Switch art over to the new bionic dlmalloc 2.8.5 callback\n\nChange-Id: I1314e87a51553fd358dbf9c44f804a7eb2de3a7d\n" }, { "commit": "00f7d0eaa6bd93d33bf0c1429bf4ba0b3f28abac", "tree": "6a2172ece15c4699e6c2a67ce76f019db0a9a21d", "parents": [ "634ea28f78c4a138e6a1de54eae8696095422415" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 19 15:28:27 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 14 10:45:52 2012 -0700" }, "message": "Global lock levels.\n\nIntroduce the notion of the mutators/GC being a shared-exclusive (aka\nreader-writer) lock. Introduce globally ordered locks, analysable by\nannotalysis, statically at compile time. Add locking attributes to\nmethods.\n\nMore subtly, remove the heap_lock_ and split between various locks that\nare held for smaller periods (where work doesn\u0027t get blocked). Remove\nbuggy Dalvik style thread transitions. Make GC use CMS in all cases when\nconcurrent is enabled. Fix bug where suspend counts rather than debug\nsuspend counts were sent to JDWP. Move the PathClassLoader to\nWellKnownClasses. In debugger refactor calls to send request and\npossibly suspend. Break apart different VmWait thread states. Move\nidentity hash code to a shared method.\n\nChange-Id: Icdbfc3ce3fcccd14341860ac7305d8e97b51f5c6\n" }, { "commit": "b062fdd4cb097fbae69b4bcb479c34d83ecab8ca", "tree": "215ea8fb42828a0f753ac5bd424ca098ce748342", "parents": [ "ca314c6a1be1b4cc11f4d284da90af7dc8a4ce25" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jul 03 09:51:48 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 11 17:17:46 2012 -0700" }, "message": "Each space has its own bitmap(s)\n\nEach alloc space now has One mark+live bitmap. Each image space has only one live bitmap.\n\nChange-Id: I2e919d1bd7d9f4d35d0e95ed83a58df6f754df6e\n" }, { "commit": "365c10235438607541fa2259a5fec48061b90bd8", "tree": "26516a4985a8bbef4c8da36b99bf1e54af3ad24d", "parents": [ "147440427984816b6ed5235a9b1fb2cbf99496d0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 22 15:05:28 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jun 26 15:21:01 2012 -0700" }, "message": "Increase use of ScopedJniThreadState.\n\nMove the routines for changing Object* to jobject and vice-versa\n(AddLocalReference and Decode) to ScopedJniThreadState to enforce use of\nObject*s in the Runnable thread state. In the Runnable thread state\nsuspension is necessary before GC can take place.\n\nReduce use of const ClassLoader* as the code bottoms out in FindClass\nand with a field assignment where the const is cast away (ie if we\u0027re\nnot going to enforce the const-ness we shouldn\u0027t pretend it is).\n\nRefactor the Thread::Attach API so that we\u0027re not handling raw Objects on\nunattached threads.\n\nRemove some unreachable code.\n\nChange-Id: I0fa969f49ee6a8f10752af74a6b0e04d46b4cd97\n" }, { "commit": "7664f5cd118b355a5fe0c7536cb48ac991ed2b62", "tree": "0564c36316e2aa0a892ff9914258433f14641a07", "parents": [ "9eb5f03e0dd90ac25e01fcf81f832c50a1f09d2c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jun 08 18:15:32 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 11 11:17:13 2012 -0700" }, "message": "Fix GC performance regression\n\nEnable CMS and fix performance regression due to recursive marking image spaces. Dependent on my java change list.\n\nChange-Id: I4765792aa8226e811ac158f04ab88217db755573\n" }, { "commit": "9eebd3b36f1f04c430623d6a2cffff05d80513ba", "tree": "487a51f0f4e78c385e240ce7722d46d4ca220be9", "parents": [ "a9719eb4167b544438268d46692389761652fc5d" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Jun 08 13:56:31 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Jun 08 14:10:51 2012 -0700" }, "message": "Trim the native heap too, like dalvik did.\n\nChange-Id: I240f33937ffe7a3d3df2a66cf5b4ee5296bce16d\n" }, { "commit": "eac766769e3114a078c188ea26776a81f0edb3cf", "tree": "4b9b8dfc89725173431817baba4455c013e40577", "parents": [ "3f9ace8d90bfc48c8d7bf35af66de8ce8238de7f" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu May 24 21:56:51 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri May 25 12:04:43 2012 -0700" }, "message": "DO NOT MERGE Fix all unsafe caching to be like libcore.\n\nThis way, if a runtime is restarted within a process, we re-initialize all\nthe cached data.\n\nConflicts:\n\n\tsrc/native/java_lang_Runtime.cc -- nativeExit lost an argument in dalvik-dev\n\n(cherry picked from commit 7756d5473fa27ce7e6ac7c31770eef7030431da4)\n\nChange-Id: I6184fc20c2a9ec16c4b053584a4d1c3b64452d0f\n" }, { "commit": "fbe5b757cff040a455f93388d6eebdee0653a089", "tree": "1cc69bee1f70aa8adae7fb32add6c46f8a3a0ddb", "parents": [ "da83d972b1aa90ada23b592bcea00603d1ee78b7" ], "author": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Thu Apr 19 00:26:00 2012 -0700" }, "committer": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Thu Apr 19 00:33:43 2012 -0700" }, "message": "Fix LLVM build. Otherwise, function not used warning.\n\nNote that we use -Werror.\n\nChange-Id: I14c2d1d040d392cf5b2866298dc9c62a216f329b\n" }, { "commit": "264675757a1d1ba43f53634d1301b7d6b79fe02c", "tree": "de8ebe45f28c4c287d3be5e8cca7fee2fe5c587a", "parents": [ "69eafaae2c910ea6f704a3867f9b78647174a0df" ], "author": { "name": "TDYa127", "email": "tdy@google.com", "time": "Tue Apr 17 20:51:22 2012 -0700" }, "committer": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Wed Apr 18 18:44:23 2012 -0700" }, "message": "Don\u0027t use work_around_app_jni_bugs for llvm now.\n\nChange-Id: Id698e60c09dc863311445498d8813e38a48b8cc4\n" } ], "next": "0dae08ead9112adb81ad507b81187f00f77bc168" }