)]}' { "log": [ { "commit": "cc1b5357f83f0b787d51fbfde3fe870c8a2fa050", "tree": "253d23dba63e1003dc8e953002b994d01c97f181", "parents": [ "5d0c3009fc56afdb4aeae5ee6dd98c3d1d2e7711" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 01 16:58:38 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Dec 02 14:00:22 2016 -0800" }, "message": "ART: Clean up ClassLinker\n\nTry to clean up and simplify ClassLinker.\n\nMove dex_lock to Locks. Remove dead code. Move single-use\ncode to user. Hide implementation details from header.\n\nTest: m test-art-host\nChange-Id: I58150fa9c2a9524f8304370270c2197d655cb3a8\n" }, { "commit": "063fc772b5b8aed7d769cd7cccb6ddc7619326ee", "tree": "bc165781989087a998721991504e589a7d5b0926", "parents": [ "48d08a4233ee4450b0d5073d41445f9dd1f17191" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Tue Aug 02 11:02:54 2016 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Dec 01 11:15:47 2016 -0800" }, "message": "Class Hierarchy Analysis (CHA)\n\nThe class linker now tracks whether a method has a single implementation\nand if so, the JIT compiler will try to devirtualize a virtual call for\nthe method into a direct call. If the single-implementation assumption\nis violated due to additional class linking, compiled code that makes the\nassumption is invalidated. Deoptimization is triggered for compiled code\nlive on stack. Instead of patching return pc\u0027s on stack, a CHA guard is\nadded which checks a hidden should_deoptimize flag for deoptimization.\nThis approach limits the number of deoptimization points.\n\nThis CL does not devirtualize abstract/interface method invocation.\n\nSlides on CHA:\nhttps://docs.google.com/a/google.com/presentation/d/1Ax6cabP1vM44aLOaJU3B26n5fTE9w5YU-1CRevIDsBc/edit?usp\u003dsharing\n\nChange-Id: I18bf716a601b6413b46312e925a6ad9e4008efa4\nTest: ART_TEST_JIT\u003dtrue m test-art-host/target-run-test test-art-host-gtest\n" }, { "commit": "f05ccaf2cc9882e8b2d0add30072fd2648bea88d", "tree": "c92854218ee522a99c298d6f32a45327c14626dc", "parents": [ "c394d0558de9afff6ab8b87ae9706497e01f37ff", "6f4d102e0c3029b3c74af13d3846a14e1c878a12" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 22 19:26:09 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Nov 22 19:26:10 2016 +0000" }, "message": "Merge \"Correctly handle EINTR return from futex\"" }, { "commit": "340dafabc8e88378e395cda9027cf17726910e91", "tree": "f742cfc9b9eb3fdf0245a66491d39fb841da7c01", "parents": [ "71d15102b52af67e8fe1193192aa2b4cd1956ae0" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Nov 18 16:03:10 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 22 13:08:29 2016 +0000" }, "message": "Use a per-thread VerifierDeps.\n\nAvoid lock contention on a singleton VerifierDeps by allocating\ntemporary per-thread VerifierDeps that get merged after verification.\n\nThis saves around ~35% compile-times on interpret-only.\n\nOnly the creation of extra strings is guarded by a lock, for simplicity.\n\nTest: test-art-host, test-art-target\nbug: 32641252\nbug: 30937355\n\nChange-Id: I11a2367da882b58e39afa7b42cba2e74a209b75d\n" }, { "commit": "6f4d102e0c3029b3c74af13d3846a14e1c878a12", "tree": "18ad351a86cabf94c2bb7f4f8b9dff9d0ec7c59a", "parents": [ "da2d78ba97d177b80e0e9528362bcebc1fd31f0f" ], "author": { "name": "Daniel Colascione", "email": "dancol@google.com", "time": "Mon Nov 21 14:35:42 2016 -0800" }, "committer": { "name": "Daniel Colascione", "email": "dancol@google.com", "time": "Mon Nov 21 15:44:06 2016 -0800" }, "message": "Correctly handle EINTR return from futex\n\nWe should handle EINTR the same way we do EAGAIN: they\u0027re both\ntransient failures. Either way, higher-level code (see\nReaderWriterMutex::SharedLock) will do the right thing, loop around,\nand try again.\n\nTest: code inspection\nChange-Id: I1eb7fb6851cc1a65acc9a0cf7cbeef787b6b28b6\n" }, { "commit": "ca47a17f90bb0be9b98685fbf46e4e4f53ea18c6", "tree": "2f0c28349de25f3e1feb5d31c70ae61f857cfc82", "parents": [ "6adc5a8cac4eab3b847f81c422def4b4ae510332" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Nov 04 16:39:27 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Nov 04 16:39:27 2016 -0700" }, "message": "ART: Clean up old mutex\n\nRemove obsolete mutex.\n\nTest: m test-art-host\nChange-Id: I476792cac4d5d680192e35055abed66772cf8081\n" }, { "commit": "f42eb2c7801dbb45a6ba20a372d5ba4712ebefba", "tree": "62939df36e0d113aab0307f75ab728de72b80953", "parents": [ "fc5fbfc13fd6873d42fe82c308ee68a685258b4c" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Oct 19 13:20:45 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 20 13:40:56 2016 -0700" }, "message": "Remove mutex dependency on art::Runtime\n\nBreaks the cyclic dependency between mutex and the runtime. This allows\nthe use of mutexes without instantiating a runtime.\n\nBug: 22322814\nTest: test-art\nChange-Id: Ia642e515937068d385e5bb1e10bbd3e50a6e36d2\n" }, { "commit": "05a364c8d8271ceeca307d04736f53e92d03de9d", "tree": "70e166900c99564c0ab3596ef3c8d5d85f14e767", "parents": [ "4aed412037e6e7bad2cf589645449df960b88200" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 14 13:27:12 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 14 14:48:26 2016 -0700" }, "message": "ART: Make JNI global and weak global locks global\n\nAdd them to Locks. Give the JNI global table lock\na lock level, close to the weak global JNI references.\n\nBug: 31684578\nTest: m test-art-host\nChange-Id: I3857a3b0be69b16811d9999096b2c42e7a25d227\n" }, { "commit": "0795f23920ee9aabf28e45c63cd592dcccf00216", "tree": "ff3f880c5e84f3316532b47d0e9a7729ade848ac", "parents": [ "d1224dce59eb0019507e41da5e10f12dda66bee4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 18:43:30 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 29 17:31:09 2016 -0700" }, "message": "Clean up ScopedThreadStateChange to use ObjPtr\n\nAlso fixed inclusion of -inl.h files in .h files by adding\nscoped_object_access-inl.h and scoped_fast_natvie_object_access-inl.h\n\nChanged AddLocalReference / Decode to use ObjPtr.\n\nChanged libartbenchmark to be debug to avoid linkage errors.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I4d2e160483a29d21e1e0e440585ed328b9811483\n" }, { "commit": "3fec9ac0d5af1358d216eb2fdc2000ec0205f3f0", "tree": "f38d8d8aae51f53e7ee6b474f47597b784fc2316", "parents": [ "0cfe19af3b7395658210ea6044a65c9811962a7a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 13 10:47:28 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 26 10:59:22 2016 -0700" }, "message": "ART: Use libbase logging\n\nMove most of our logging infrastructure over to system/core/base.\nRetain VLOG.\n\nUsing unified Android infrastructure has two main advantages. First,\nit reduces the complexity/maintenance burden in ART. Second, it\nallows to detach logging for the cases where we do not want or need\na runtime, e.g., dexdump, the disassembler, etc. As a part of the\nlatter, libbase is also supported for all hosts (including Windows).\n\nFrom a developer viewpoint, there are minor behavior changes for the\nLOG statements (see above), but otherwise usage is the same. Explicit\nseverity enum items are in the android::base namespace now.\n\nBug: 31338270\nTest: m test-art-host\nChange-Id: I5abcb2f45f5b03d49951874c48544f72a283a91b\n" }, { "commit": "ca3c8c33501bf199d6fd0a5db30a27d8e010cb23", "tree": "12a6c3a4266bd17c6b9fe7066fd0b25cb8d9b2d6", "parents": [ "0764877e21f1230e718923d5b84b048cd4346afa" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Sep 06 14:04:48 2016 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Sep 16 10:08:18 2016 +0100" }, "message": "Collect verifier dependencies\n\nMethodVerifier tests whether a DEX method is valid w.r.t. the classes\nin class path. Since the APK does not change across OTA updates, it\nis not necessary to analyze the bytecode again with MethodVerifier,\nas long as its dependencies on the class path (which may have changed)\nare satisfied.\n\nThis patch introduces VerifierDeps, a class path dependency collector,\nand adds hooks into MethodVerifier where classes/methods/fields are\nresolved and where assignability of types is tested.\n\nTest: m test-art-host-gtest-verifier_deps_test\nBug: 30937355\nChange-Id: Iee0b321d772a5c7d1cb471aaa6e13918310b7e2f\n" }, { "commit": "9231f4e55d6e79e37880071e222c28141bf65f49", "tree": "929c75752d7dd5d667b5f2cdaa044ad83d0522eb", "parents": [ "e99331f7a430b0b72142dd0b8b0689421489dcf6" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 23 17:35:19 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 23 18:20:12 2016 -0700" }, "message": "ART: Remove vestiges of GCC\n\nWe no longer support GCC.\n\nChange-Id: I869ac6e5dcb166747e45236a0d313c7672bbf2de\nTest: m test-art-host\n" }, { "commit": "14832efeb92334c562ebedef34e920d30e3cee69", "tree": "57d10c24966a46ae1c01861ee579ac6000326c45", "parents": [ "c984725d0035e1925371757c38fed339b409e525" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Fri Aug 05 11:44:32 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Aug 09 13:47:49 2016 +0100" }, "message": "Revert experimental lambda feature.\n\nThis is a revert of the following changes :\n\n30c475a2046951a81769c2db0b2dad66cd71e189.\nlambda: Minor capture-variable/liberate-variable clean-up after post-merge reviews.\n\n6918bf13eb855b3aa8ccdddda2d27ae8c60cec56.\nlambda: Experimental support for capture-variable and liberate-variable\n\nfc1ccd740b7c8e96dfac675cfc580122cd1b40a6.\nlambda: Infrastructure to support capture/liberate-variable dex opcodes\n\ne2facc5b18cd756a8b5500fb3d90da69c9ee0fb7.\nruntime: Add lambda box/unbox object equality\n\n2ee54e249ad21c74f29a161e248bebe7d22fddf1.\nruntime: Partially implement box-lambda and unbox-lambda experimental opcodes\n\n158f35c98e2ec0d40d2c032b8cdce5fb60944a7f.\ninterpreter: Add experimental lambda opcodes for invoke/create-lambda\n\na3bb72036f5454e410467f7151dc89f725ae1151.\nAdded format 25x to dexdump(2).\n\nPlus surrounding cleanups.\n\nTest: make test-art\nChange-Id: Ic6f999ad17385ef933f763641049cf721510b202\n" }, { "commit": "a206c745dbb64b14f05c87891d425475c2f6d63a", "tree": "b73c6dfa1f78d6605e71fab8a3e43fb484715647", "parents": [ "d53208c8e7ca4b2e2d4070d8e67ec1ee078cee74" ], "author": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Tue May 24 15:04:22 2016 -0700" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Wed May 25 11:26:43 2016 -0700" }, "message": "Don\u0027t use dlopen on host for already loaded oat files.\n\nBecause the behavior of dlopen on the host is different then the\ntarget in that case, and it causes tests to fail.\n\nBug: 28826195\n\nChange-Id: Id202bbac3318bade89a4133a9bcb1ee01e8b6182\n" }, { "commit": "fba39972d99701c80bf3beb7451aca508d67593c", "tree": "0d80ecb6997290140503926b08a72e7418915526", "parents": [ "718d4e269810c17d03df909c84b2f7bbd4f61fb9" ], "author": { "name": "Chih-Hung Hsieh", "email": "chh@google.com", "time": "Wed May 11 11:26:48 2016 -0700" }, "committer": { "name": "Chih-Hung Hsieh", "email": "chh@google.com", "time": "Fri May 13 10:38:56 2016 -0700" }, "message": "Fix misc-macro-parentheses warnings.\n\n* Add parentheses to fix warnings.\n* Use NOLINT to suppress wrong clang-tidy warnings.\n\nBug: 28705665\nChange-Id: Icc8bc9b59583dee0ea17ab83e0ff0383b8599c3e\n" }, { "commit": "32ce2adefb8a3d0eda59a29f5e87c1eb43eef796", "tree": "50aaa40b31ede355359765259edc7989485cc7d5", "parents": [ "c71dc3852edfa8e0cd2083d12845ff8607e189ff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 04 14:58:03 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 04 18:37:19 2016 -0800" }, "message": "Add more systracing everywhere\n\nAdded to:\nJIT\nDexFile functions\nOat file manager\n\nAdded helper ScopedTrace to prevent errors and reduce excess code.\n\nBug: 27502458\n\n(cherry picked from commit dabdc0fe183d4684f3cf4d70cb09d318cff81b42)\n\nChange-Id: Ifaeff8913d79eefc797380987d13cc00456266f8\n" }, { "commit": "98e6ce44c700abd9375fe17f0aa31fea1e1e938b", "tree": "aa15b4398290918e5eb5227781b5aef5ecff9e0b", "parents": [ "a1f65135cd2315159ac302f904ba0c5ba0d7fd0e" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 16 18:42:15 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 09:51:40 2016 +0000" }, "message": "Remove string init map.\n\nPartial revert of the String init change.\n\n- Make Quick bailout in the presence of String allocation.\n- Rely on the compiler for knowing when dex registers alias.\n\nbug:27173201\n\nChange-Id: I0bf58ba3825c71cef110b53f3a0a6f567cb2ef9a\n" }, { "commit": "4d87df607a0b86cdf4b2c04f61d72a60d8975ce0", "tree": "922e882c15633e6c700091c99864868d69822d7d", "parents": [ "97f2ca08c3d9a2b1694419aea07cd64f477c0af2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 07 15:14:19 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 08 15:46:52 2016 -0800" }, "message": "Improve handling of daemon threads after runtime shutdown\n\nThe main issue comes from the fact that user daemon threads are\nallowed to continue running after the runtime has shutdown. They may\nstill have a JNI env pointer. To prevent crashing if they call into\nthe env, we replace the function pointers with functions that sleep\nforever.\n\nThe other issue is that user daemon threads that are blocked in an\nART condition variable may get woken up by another user daemon inside\nof Monitor::Notify or by a spurious wakeup (i.e. SIGQUIT). To deal\nwith this issue, we check the JNI env for shutdown runtime when we\nare woken up from a condition variable wait. This check fixes test\n132 with --host --gdb --interpreter. Previously this test crashed\nsince dlclose was somehow causing a spurious futex wakeup.\n\nTODO: Investigate adding a unit test.\n\nBug: 18577101\nChange-Id: I479b38968ee9fbc4ee4b252ee2528787279972cc\n" }, { "commit": "3a0909248e04b22c3981cbf617bc2502ed5b6380", "tree": "b0b7013ea78b10f23fae75ef145e53a696ff797c", "parents": [ "457e874459ae638145cab6d572e34d48480e39d2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "message": "Revert \"lambda: Add support for invoke-interface for boxed innate lambdas\"\n\n955-lambda is flaky\n\nBug: 24618608\nBug: 25107649\n\nThis reverts commit 457e874459ae638145cab6d572e34d48480e39d2.\n\nChange-Id: I24884344d21d7a4262e53e3f5dba57032687ddb7\n" }, { "commit": "457e874459ae638145cab6d572e34d48480e39d2", "tree": "b192c4698c4889ae90db6abe2652199802213fe6", "parents": [ "3944f7175dcf60316ba58a42698ccf23c65ac57c" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Oct 22 17:37:50 2015 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Nov 20 16:33:06 2015 -0800" }, "message": "lambda: Add support for invoke-interface for boxed innate lambdas\n\nLambda closures created with the \u0027create-lambda\u0027 instruction\n(termed \"innate lambdas\") can be turned into an object with \u0027box-lambda\u0027.\n\nThis CL enables support for those kinds of lambdas to work with\n\u0027invoke-interface\u0027 by generating a proxy class for the lambda.\n\nNote: MIPS32/64 support not included.\n\nBug: 24618608\nBug: 25107649\nChange-Id: Ic8f1bb66ebeaed4097e758a50becf1cff6ccaefb\n" }, { "commit": "e58991b3b2282b5761f1a6023a16c803e1c4eb45", "tree": "5a6fac6c5d3cb6a463463b83b8a34a53b07a8ede", "parents": [ "fb11bab9bc96ff05dcb12f43abf58df256b7c7aa" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 13 07:59:34 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 13 13:17:50 2015 -0700" }, "message": "Revert \"Revert \"Unload oat files\"\"\n\nFixed a race where two threads calling OatFile::Open could both use\ndlopen on the host.\n\nBug: 22720414\n\nThis reverts commit 72da5e7461fec3b1e116050f2e6f233efb9c54f3.\n\nChange-Id: I1636045b724944d2a09417527280784967957095\n" }, { "commit": "f9c6fc610b27887f832e453a0da1789187293408", "tree": "3f45a15327e5bc7cb71e6c238ba75e87ae2e6058", "parents": [ "793e6fbdefb092d1dab50bca5618aed110c7e037" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 07 11:44:05 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 09 12:57:53 2015 -0700" }, "message": "Add OatFileManager\n\nTakes over a large amount of functionality from the class linker.\n\nChanged OatFile to loading the same OatFile multiple times. This is\nrequired for unloading OatFiles and moving dex caches to BSS since\nthese require a different OatFile for each dex cache and class\nloader.\n\nBug: 22720414\n\nChange-Id: I0321096723a294dc72949f21e66da82727b512fc\n" }, { "commit": "a2c38644d96cbad4106c0165811d0f670d6cec8f", "tree": "4ad15bed693ced09d99bb034175bdfe8cc3bb1c2", "parents": [ "6766eae2d91e894b4ceab9f29cc983900e7bc0c7" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Sep 17 17:29:01 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Sep 17 18:33:16 2015 -0700" }, "message": "Add lock around interpreter string init reg map.\n\nFixes intermittent test failures. Note that locks can\u0027t be held\nwhile the verifier is generating the map.\n\nChange-Id: Ie38d6bf0d9c5e81d66b8c167fde0e75a0565cea5\n" }, { "commit": "5bdb6550ab0341e3c9fc633866e51e08786bd589", "tree": "7cd631f9ebfb5fa2e4c1fc26155222172e0b8b48", "parents": [ "f64914d7392b7d35c7f35bc917b0797b9c4605ac" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 22 23:44:55 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 22 23:44:55 2015 -0700" }, "message": "ART: Fix missing role definition\n\nAdd missing static member definition. Follow-up to\n4e2cb098017bf073335ebb02b1bc0a36828cd720\n\nBug: 20072211\nChange-Id: I392da3724ded7c455b9b57add6fceb70191f9ff6\n" }, { "commit": "e2facc5b18cd756a8b5500fb3d90da69c9ee0fb7", "tree": "595a3c807e1cbaa4b8c22b93685d70faa4ff0f5a", "parents": [ "e07300b712f44397ecbfed1ef7f880686e46274a" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Jul 10 13:49:08 2015 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Mon Jul 20 15:11:59 2015 -0700" }, "message": "runtime: Add lambda box/unbox object equality\n\nA lambda that is boxed with box-lambda is now stored as a weak reference\nin a global runtime table (lambda::BoxTable). Repeatedly boxing the same\nlambda closure value will always return the same java.lang.Object back.\n\nSince there is no way to observe the address of an object, a GC can\nhappen and clean up the table of any dead boxed lambdas, which can also\nshrink the table to prevent the memory use from growing too much.\n\n(Note that a lambda closure is immutable, so hashing over it is\nguaranteed safe.)\n\nChange-Id: I786c1323ff14eed937936b303d511875f9642524\n" }, { "commit": "eac4424b3420c280f97ff2f815b5dedd8dac9801", "tree": "4e7c31b4312aa000bd3d2250df6a05daf61523d7", "parents": [ "ec97825f8553b3a99b6dfbbb90a50fe65301ea94" ], "author": { "name": "Yu Li", "email": "yu.l.li@intel.com", "time": "Mon Jun 29 10:50:03 2015 +0800" }, "committer": { "name": "Man Cao", "email": "manc@google.com", "time": "Mon Jul 13 15:56:19 2015 -0700" }, "message": "ART: JNI thread state transition optimization\n\nThis patch improves the JNI performance by removing the explicit acquiring and\nreleasing the mutator lock when a thread state transits between suspended and\nrunnable states.\n\nThe functions responsible for changing the state were found to be the costliest\npart of the JNI. Originally, a thread needs to acquire a shared mutator lock by\na CAS instruction when entering the runnable state and also needs to release\nthe lock by a CAS when entering the native state from runnable. This patch\nremoves these CAS operations when a thread state transits between suspended and\nrunnable. A thread in the runnable state is considered to have shared ownership\nof the mutator lock and therefore transitions in and out of the runnable state\nhave associated implication on the mutator lock ownership. Meanwhile, a barrier\nis added to control suspending all threads from running.\n\nJNI transition overhead was reduced by 25% on IA platform and by 17% on ARM\nplatform by this patch, while it has little impact on GC pause time (measured\nwith \"suspend all histogram\").\n\nChange-Id: Icee95d8ffff1bbfc95309a41cc48836536fec689\nSigned-off-by: Yu, Li \u003cyu.l.li@intel.com\u003e\nSigned-off-by: Haitao, Feng \u003chaitao.feng@intel.com\u003e\nSigned-off-by: Lei, Li \u003clei.l.li@intel.com\u003e\n" }, { "commit": "80afd02024d20e60b197d3adfbb43cc303cf29e0", "tree": "ef054c7b4f2a739f7cf063e0bc4c501c2c7e41b5", "parents": [ "559b178e34c5d92e7932f92e5d8a981ac334606f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 19 18:08:00 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 26 15:59:02 2015 +0100" }, "message": "ART: Clean up arm64 kNumberOfXRegisters usage.\n\nAvoid undefined behavior for arm64 stemming from 1u \u003c\u003c 32 in\nloops with upper bound kNumberOfXRegisters.\n\nCreate iterators for enumerating bits in an integer either\nfrom high to low or from low to high and use them for\n\u003carch\u003eContext::FillCalleeSaves() on all architectures.\n\nRefactor runtime/utils.{h,cc} by moving all bit-fiddling\nfunctions to runtime/base/bit_utils.{h,cc} (together with\nthe new bit iterators) and all time-related functions to\nruntime/base/time_utils.{h,cc}. Improve test coverage and\nfix some corner cases for the bit-fiddling functions.\n\nBug: 13925192\nChange-Id: I704884dab15b41ecf7a1c47d397ab1c3fc7ee0f7\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": "12d625f87bcd6c4059a205bb39007a255f57f382", "tree": "b69f6f80f56977c176e0c441869d2f1de06ede70", "parents": [ "b871a15f1f662c8efe10d41199180bd00b6decd1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 13 11:33:37 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 13 16:38:25 2015 -0700" }, "message": "Change method verifiers to be thread local roots\n\nBug: 10921004\n\nChange-Id: I3a1425d76cd380587ba67ef6d5ec1227564e55bf\n" }, { "commit": "2435a43f6c851c23922d8508fb17c6079248201c", "tree": "a0eea938dfbafc8f8e1546528295005d3a55ecee", "parents": [ "c685bce4e6cfec075cb5b468d06b2fcdeeda2005" ], "author": { "name": "bowen_lai", "email": "bowen_lai@htc.com", "time": "Fri Mar 13 14:34:40 2015 +0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 13 10:20:40 2015 -0700" }, "message": "ART: Add entries_ lock for race condition\n\nGCDaemon thread would visit incorrect RegType content when there is\nanother thread initializing classes.\nAdd a lock to protect entries_.\n\nhttps://code.google.com/p/android/issues/detail?id\u003d159849\n\nChange-Id: Iabaa1c7f5cc5106b60a6e3856152e0797e8a5d6d\n" }, { "commit": "91e56692c6bd9fa1d41951ee7dc311f19461f4be", "tree": "8e7600204eed1509b7ec436ae789f3d8195a8755", "parents": [ "dc47e986941b1a3754447fabea272485f3f0f382" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 03 13:51:04 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 03 16:31:49 2015 -0800" }, "message": "Wait for threads to finish unregistering\n\nThere was a race where Thread::join would return before the thread was\nunregistered. This caused a problem with Daemons.stop since the thread\nlist could get deleted before the daemon thread was removed from list_.\nThis caused occasional \"Request to unregister unattached thread\"\nerrors and warnings.\n\nThe fix is to wait until threads finish registering before destroying\nthe thread list. The only threads which can be unregistering at this\npoint are the daemons we stopped earlier during the runtime shutdown\nprocess. The issue is that thread join finishes before we remove the\nthread from the thread list.\n\nAlso some cleanup.\n\nBug: 18713034\nChange-Id: I8921122fe8462643a6b814b5f00632481e3831fb\n" }, { "commit": "4c10110b144eafa47333dc9d30da9220a03e7c08", "tree": "e373871d3b56a161eacd7a1d25846c2a39b0ad62", "parents": [ "763abfd0d803f8169e97d3da944043c2464aac0a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 27 17:14:16 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 27 18:20:41 2015 -0800" }, "message": "Print lock level in ExclusiveUnlock\n\nBug: 18713034\nChange-Id: I83e1bf1e2023b595d8ddf20008935c75b2b0aa0c\n" }, { "commit": "8f1fa100ee037131976c616ec72a6608dccb51e2", "tree": "4df001bf5f7bc441b81c750b5b858f5c912cd6eb", "parents": [ "604e2828896fbb8663897d1e75112da7305ead4c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 22 19:48:51 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jan 23 15:16:33 2015 -0800" }, "message": "ART: On shutdown, only warn on mutex contention\n\nDo not abort, as daemon threads may still be active.\n\nBug: 17894429\n\n(cherry picked from commit c0440f69ebf051ff2ffdc00de51005a040014462)\n\nChange-Id: I7c1d50ff8d4a5e150279e703a69c8f2f1d423e6b\n" }, { "commit": "071e48ecfc95b1c67b07c975190d51f646ac4276", "tree": "c873eaedf521aebaabe92d61720b81b12343f9bb", "parents": [ "c40a4350daac81ddbfc5f6ceab934f2180dc4ec6" ], "author": { "name": "tony.ys_liu", "email": "tony.ys_liu@htc.com", "time": "Wed Jan 14 18:28:03 2015 +0800" }, "committer": { "name": "tony.ys_liu", "email": "tony.ys_liu@htc.com", "time": "Wed Jan 14 18:28:03 2015 +0800" }, "message": "Fix false alarm on thread suspend timeout\n\nRoot cause:\n CLOCK_REALTIME will jump backward/forward when system time-of-day clock is changed.\n It implies now_abs_ts will jump a lot across end_abs_ts.\n Then, it makes a false alarm (process crash) on thread suspend timeout when doing ComputeRelativeTimeSpec.\n\n if (ComputeRelativeTimeSpec(\u0026rel_ts, end_abs_ts, now_abs_ts)) {\n return false; // Timed out.\n }\n\nSolution:\n Use CLOCK_MONOTONIC instead\n\nChange-Id: I768af52b05ee1548bb291f7d5e2f389ec85e0e71\n" }, { "commit": "eb0a179508f3c0533dd7db86ec7ab9dfa3773256", "tree": "51f4f2a694627cace4b65c7018a47c994db44bfd", "parents": [ "5b6912e455da798a79a4bb3235e276351b38ab58" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Dec 15 17:23:45 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Dec 15 17:31:17 2014 -0800" }, "message": "Add more logging to Mutex::ExclusiveUnlock\n\nBug: 18713034\nChange-Id: I9db5ca0a25592714ac9e292a2b8d46ef73779c8b\n" }, { "commit": "cef50f0487929f72183623bc2a5819cf3a63bff6", "tree": "b37c68730d982b1bff28d0faa8072df0354b5dd9", "parents": [ "b1a38e246cfbfb21100d9c9e57f33970e824f075" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Dec 09 17:38:52 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Dec 10 09:43:29 2014 -0800" }, "message": "Stop daemon threads in runtime shutdown\n\nEnsure that daemons are stopped and joins before bringing down the\nruntime. This fixes bugs related to native code still running when\nwe shutdown the runtime.\n\nAlso changed the mutex destructor to allow contenders if we are\ndeleting a monitor lock level mutex.\n\nBug: 18577101\nChange-Id: I6457b35fd69c6997b9003b5f15f39861749843a9\n" }, { "commit": "db978719dbcb73fc6acfd193561445c4462786b8", "tree": "be75911bfbe29a50fff685217c8ee155fb99ac10", "parents": [ "015b137efb434528173779bc3ec8d72494456254" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 09 13:33:38 2014 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 09 13:33:38 2014 +0000" }, "message": "Revert \"Tidy gAborting.\"\n\nCreates infinite loop: b/18674776.\n\nThis reverts commit 015b137efb434528173779bc3ec8d72494456254.\n\nChange-Id: I67fe310d2e95ee2ec37bec842be06fb1123b6f4e\n" }, { "commit": "015b137efb434528173779bc3ec8d72494456254", "tree": "2ccaba9d7465bd9995082a7251d34cf76b1ab1ea", "parents": [ "a21fdd0432aa38b63d144ee4b92b6a567aab61d2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Dec 04 19:53:52 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Dec 04 20:04:45 2014 -0800" }, "message": "Tidy gAborting.\n\nReduce scope to Runtime::Abort and short-cut recursive case earlier. gAborting\nremains global to avoid two fatal errors in thread and the verifier.\n\nChange-Id: Ibc893f891ffee9a763c65cde9507d99083d47b3f\n" }, { "commit": "90675a77fa03850ebd46280e60b0ff54b8d5eb79", "tree": "9a891b47e6b7e4e5f1c123a477ec2876b8183b52", "parents": [ "4a9ac63ba407edc09e13b4e095e2105cf13f603b" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 24 00:21:06 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 24 00:21:06 2014 -0800" }, "message": "ART: Avoid recursive abort\n\nBug: 18469797\nChange-Id: Ided50bec3377034dd9a995f9f0700a795adb3940\n" }, { "commit": "4ad5cd3e7d519484559ef778d96fb3f0be8919fa", "tree": "9870938a0552b4fe472d9994a55a3bf761fc69d5", "parents": [ "741e287b60136db49ecf8da72f2b5ca48b0a39bd" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 11 23:08:07 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Nov 12 15:29:12 2014 -0800" }, "message": "Modify the behavior of thread suspend shootouts.\n\nThe thread doing the suspension doesn\u0027t attempt to suspend the other thread\nunless it knows another thread isn\u0027t trying to suspend it. Use the suspend\ncount, and its lock, for this purpose.\nRe-enable ThreadStress test.\nBug: 15446488\n\nChange-Id: Idd34410c7b89d8abd6973e5699a15ca699472c78\n" }, { "commit": "51d212ef31945743abe8a469707aaa25bab95357", "tree": "a93e362ca61c7d5a6dc8521551241106ff1248b2", "parents": [ "e8bd8ac7fb41545ef342c8120c2e83d2fef20a50" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 23 17:48:20 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 23 17:49:51 2014 -0700" }, "message": "Make out-of-line mutex contention dependent on ART_USE_FUTEXES.\n\nFix Mac build. Also fix Linux compilation if ART_USE_FUTEXES is disabled.\n\nChange-Id: I51cb1d70b5548ea6121ff7567b9546bad0894e01\n" }, { "commit": "cf7f19135f0e273f7b0136315633c2abfc715343", "tree": "ffa4d9efd9c45f4b6789acc1f534bb9327052b7e", "parents": [ "aea6888b056be21adf762e066c7f33b8939b8a06" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 22:06:39 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 22:21:57 2014 -0700" }, "message": "C++11 related clean-up of DISALLOW_..\n\nMove DISALLOW_COPY_AND_ASSIGN to delete functions. By no having declarations\nwith no definitions this prompts better warning messages so deal with these\nby correcting the code.\nAdd a DISALLOW_ALLOCATION and use for ValueObject and mirror::Object.\nMake X86 assembly operand types ValueObjects to fix compilation errors.\nTidy the use of iostream and ostream.\nAvoid making cutils a dependency via mutex-inl.h for tests that link against\nlibart. Push tracing dependencies into appropriate files and mutex.cc.\nx86 32-bit host symbols size is increased for libarttest, avoid copying this\nin run-test 115 by using symlinks and remove this test\u0027s higher than normal\nulimit.\nFix the RunningOnValgrind test in RosAllocSpace to not use GetHeap as it\nreturns NULL when the heap is under construction by Runtime.\n\nChange-Id: Ia246f7ac0c11f73072b30d70566a196e9b78472b\n" }, { "commit": "c7dd295a4e0cc1d15c0c96088e55a85389bade74", "tree": "0c08a2236bc9ba5d9a4dc75d4dd0ed2d76f8f1c6", "parents": [ "94e5af8602150efa95bde35cc9be9891ddf30135" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 21 23:31:19 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 12:06:23 2014 -0700" }, "message": "Tidy up logging.\n\nMove gVerboseMethods to CompilerOptions. Now \"--verbose-methods\u003d\" option to\ndex2oat rather than runtime argument \"-verbose-methods:\".\nMove ToStr and Dumpable out of logging.h, move LogMessageData into logging.cc\nexcept for a forward declaration.\nRemove ConstDumpable as Dump methods are all const (and make this so if not\ncurrently true).\nMake LogSeverity an enum and improve compile time assertions and type checking.\nRemove log_severity.h that\u0027s only used in logging.h.\nWith system headers gone from logging.h, go add to .cc files missing system\nheader includes.\nAlso, make operator new in ValueObject private for compile time instantiation\nchecking.\n\nChange-Id: I3228f614500ccc9b14b49c72b9821c8b0db3d641\n" }, { "commit": "6f3dbbadf4ce66982eb3d400e0a74cb73eb034f3", "tree": "f7a20779e4d665f948c5fbcd26dac0071dafb8d4", "parents": [ "2df6840f68dd18d7dd8dbf53f8b6181bbfdc4fc4" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 14 17:41:57 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 16 19:27:28 2014 -0700" }, "message": "Make ART compile with GCC -O0 again.\n\nTidy up InstructionSetFeatures so that it has a type hierarchy dependent on\narchitecture.\nAdd to instruction_set_test to warn when InstructionSetFeatures don\u0027t agree\nwith ones from system properties, AT_HWCAP and /proc/cpuinfo.\nClean-up class linker entry point logic to not return entry points but to\ntest whether the passed code is the particular entrypoint. This works around\nimage trampolines that replicate entrypoints.\nBug: 17993736\n\nChange-Id: I5f4b49e88c3b02a79f9bee04f83395146ed7be23\n" }, { "commit": "5869a2c27ee0dbd7b420614c76ff14a706f0c5fb", "tree": "39bb17a7ae4a4d6c1db99d1a5e07ede899da0aec", "parents": [ "f1f05d303988a5c071c87b760056be8358276c94" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 08 14:26:23 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 08 14:26:23 2014 -0700" }, "message": "Dump mutator lock for thread suspend timeout\n\nShould help us see if thread suspend timeouts are due to someone\nholding on the mutator lock for too long or if it is a missed\nfutex wake.\n\n(cherry picked from commit f924d2381a5ea53967ba1e279766d601b9be05ea)\n\nBug: 17837911\nChange-Id: I3417156a78f574076d5da17c86836c5a7c76084a\n" }, { "commit": "9ef78b59da51080882e47505896b420977fd79ae", "tree": "6c568756e4e16e68d5d3346261009350969d5b77", "parents": [ "95f03e6a4737f90685fab86e98709f1c4393d5ef" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 25 17:03:12 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 26 16:04:56 2014 -0700" }, "message": "Fix broken runtime SetStatsEnabled logic\n\nPreviously, Runtime::SetStatsEnabled wouldn\u0027t take stats_enabled_\ninto account when deciding whether or not to increment / decrement\nteh stats enabled counter. This resulted in counter underflows and\nother errors which caused some CTS tests to fail.\n\nAlso added some locking to prevent race conditions.\n\nBug: 17360878\n\n(cherry picked from commit a98ffd745bbecb2e84a492194950c0b94966546b)\n\nChange-Id: I21d241a58d35bd6a607aa2305c6da81720bd0886\n" }, { "commit": "69dbec6d9d55eeb2867949c2791d01dc9aa916c8", "tree": "be4eca8161eafb4a8466f5adf28a2c8f27cd37d1", "parents": [ "926d8e99aa69151a271180299df68963688010d8" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Sep 15 18:03:41 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Sep 17 14:31:55 2014 -0700" }, "message": "Avoid suspending for alloc trace enabling when already suspended.\n\nBug: 17499772\n\n(cherry picked from commit 1d6ee090fddd4bfd35c304d6ceb929d5c529dfcc)\n\nChange-Id: Id09809c476c685f0a197ee75bb08638931364efd\n" }, { "commit": "a5a53efea976af505f4f849b5925d5e14c4f8e5c", "tree": "455a536a9394908c17ec1ede04ab7ec49f9d3420", "parents": [ "91bbd5347b391278b56440b2dbb0b77526bae4e8" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 12 12:58:05 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Sep 14 14:48:24 2014 -0700" }, "message": "Add native support for FinalizerList.makeCircularListIfUnenqueued\n\nCalled from FinalizerReference.enqueueSentinelReference to prevent\na race where the GC updates pendingNext of the sentinel reference\nbefore enqueueSentinelReference.\n\nBug: 17462553\n\n(cherry picked from commit 3256166df40981f1f1997a5f00303712277c963f)\n\nChange-Id: I7ad2fd250c2715d1aeb919bd548ef9aab24f30a2\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": "306db81aba41eb244a4e8299cf58ac18ae9999c7", "tree": "a069a9dda1b246466cbeee2736f46f18ef56714e", "parents": [ "b14339904c9cacc4af74260c7325e4eb32947f95" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Sep 05 13:01:41 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Sep 05 13:24:24 2014 -0700" }, "message": "Fix numerous issues with DdmVmInternal allocation tracking\n\nIssues addressed:\n- Using without JDWP attached caused native crash.\n- When buffer is full (64k entries), number of entries reported was 0.\n- Disabling tracking after disabling tracking caused native crash.\n- Asking for allocations after disabled caused native crash.\n- Lock ordering issues between mutator lock and alloc tracker lock.\n\nAdding 098-ddmc test to cover these cases.\n\nBug: 17392248\n\n(cherry picked from commit a5815065ac0877add9c0db3605d27b4d6c426e61)\n\nChange-Id: Ib0bc18dfcdafcc050ab9dceed3d167dd878d1d7a\n" }, { "commit": "ed2be1725fb79075892b1a9103487c9d9a95b350", "tree": "8adf1d95f401c2c7e7df1c659c0dfcd4dd27a68f", "parents": [ "645f9871115e82ff836397f6470cca825befd245" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 19 15:33:43 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 02 19:28:34 2014 +0200" }, "message": "Reduce lock contention when debugging\n\nUses a ReaderWriterMutex for the breakpoint lock to reduce contention during\ndebugging session.\n\nAlso adds missing thread safety annotations on fields and methods related to\ninstrumentation and debugging.\n\nBug: 16814665\nBug: 11667502\n\n(cherry picked from commit 59d9d668d4f4286813afe2b4e7c6db839222ce96)\n\nChange-Id: I5f1156da8c6cc8316d6db16e6cfb2470fe289ad3\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": "f3d874c60ee3ada19ce26a5c4e532312b6f3a9e9", "tree": "de14ab8b610f43a2c2c4c02f4ac67d614919fde2", "parents": [ "031ddea20cb311dfdb3bd16a13750f9cb426b299" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 18:52:42 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 18:59:40 2014 -0700" }, "message": "Avoid race in single thread suspension.\n\nDon\u0027t allow more than one concurrent single thread suspension to avoid\npotential cycles and deadlocks where threads try to suspend each other.\nBug: 16364458, 16354227\n\nChange-Id: I907f1d5591a6aa5c241d37d6b4a34f968f98df77\n" }, { "commit": "37f3c968ecd04e77802fe17bb82dabc07de21ca1", "tree": "90699a532f8c6f964a342c0f68b2afa605d7a335", "parents": [ "1817981832a531580c83f7d71f745da4e06889c3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 11:25:30 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 11:25:30 2014 -0700" }, "message": "Avoid direct use of QuasiAtomic by using Atomic.\n\nChange-Id: I92d0f822cddb2085de7cbb4c44c3a429081892a2\n" }, { "commit": "c4f72ec44660f804b595bfaf2b959f46fd2ff00d", "tree": "df8764e4ec382b4e4810b8d0309823a40ef47d5c", "parents": [ "af4cf5d72a2bf77f0e442bedb208f9227f262c89", "74240819ae09e29b2753ef38f4eb4be1c2762e2e" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 10 07:54:06 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jul 09 14:45:53 2014 +0000" }, "message": "Merge \"Use memory chunks for monitors on LP64\"" }, { "commit": "74240819ae09e29b2753ef38f4eb4be1c2762e2e", "tree": "61e2d3aa7268ce49fe77715593896f59feb92fe6", "parents": [ "32710dd4a0232149002a5ae7bde1c640cdffd564" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Apr 17 10:35:09 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 10 00:51:04 2014 -0700" }, "message": "Use memory chunks for monitors on LP64\n\nMonitor IDs in lock words are only 30b. On a 32b system that works\nfine, as memory is usually aligned enough that shifting works out.\nOn 64b systems, the virtual memory space is too large for that.\nThis adds memory chunks into which we allocate the monitors so that\nwe have base_addr + offset and can use the offset as the monitor ID.\nTo allow for relatively compact but growable storage, we use a list\nof chunks.\n\nAdded a global lock for the monitor pool.\n\nChange-Id: I0e290c4914a2556e0b2eef9902422d7c4dcf536d\n" }, { "commit": "c7190697f8665e706f6ebb4ae36fa63c46a32cd5", "tree": "f9281913f17e953143acd6c1a0f34dd3074bad56", "parents": [ "bcb3b29095817ce8987d8310d4db87271f5114ad" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 08 23:50:26 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 10 01:37:08 2014 +0000" }, "message": "Remove legacy CAS implementations from mutex.\n\nRemoves the use of __sync_bool_compare_and_swap and android_atomic_cas and uses\nintention revealing atomic operations from art::Atomic (which will eventually\ngive way to std::atomic).\n\nChange-Id: Iea44e1923f6706ec04b5459fe25427282c189a7e\n" }, { "commit": "c5f17732d8144491c642776b6b48c85dfadf4b52", "tree": "811daa488ae5ee5dfd9b3b73bd210bc1506e5ca1", "parents": [ "08654d40cdd256f6a6c8619bf06d04d4c819714a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jun 05 20:48:42 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jun 10 23:19:29 2014 -0700" }, "message": "Remove deprecated WITH_HOST_DALVIK.\n\nBug: 13751317\nFix the Mac build:\n - disable x86 selector removal that causes OS/X 10.9 kernel panics,\n - madvise don\u0027t need does zero memory on the Mac, factor into MemMap\n routine,\n - switch to the elf.h in elfutils to avoid Linux kernel dependencies,\n - we can\u0027t rely on exclusive_owner_ being available from other pthread\n libraries so maintain our own when futexes aren\u0027t available (we\n can\u0027t rely on the OS/X 10.8 hack any more),\n - fix symbol naming in assembly code,\n - work around C library differences,\n - disable backtrace in DumpNativeStack to avoid a broken libbacktrace\n dependency,\n - disable main thread signal handling logic,\n - align the stack in stub_test,\n - use $(HOST_SHLIB_SUFFIX) rather than .so in host make file variables.\n\nNot all host tests are passing on the Mac with this change. dex2oat\nworks as does running HelloWorld.\nChange-Id: I5a232aedfb2028524d49daa6397a8e60f3ee40d3\n" }, { "commit": "3035961cb41865b80b927546be0c708b6389cec6", "tree": "27ed39f92c013381d08c7ad51d311cb29be230dc", "parents": [ "e21bf0099117b82963cdf7f95a1b8dabfcf29397" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Wed May 21 17:46:23 2014 -0700" }, "committer": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Fri Jun 06 16:13:42 2014 -0700" }, "message": "Clean up and augment Atomic class. Replace QuasiAtomic MemBars.\n\nAdd a number of missing C++11 operations to Atomic class.\nInvoke the 64 bit routines in QuasiAtomic when necessary.\nReplace QuasiAtomic membars with fences that correspond to C++11 fences.\n\nQuasiAtomic was moved to the top of the file. Only fence implementations\nactually changed.\n\nThis replaces some buggy uses of MembarStoreStore, as reported\nin b/14685856 .\n\nAvoid some redundant fences for long volatile operations.\n\nIncompletely converts low-level memory access operations to Atomic.\n\nChange-Id: Iea828431a0cea46540eb74fcaa02071cab6fdcda\n" }, { "commit": "3eed93dd5be03e5539827bebf0f414251a12e15e", "tree": "186fa74420e2595c9b6fba9ea8de94756af9cef5", "parents": [ "bbdc5bc5fd5141711879a6c85d80ac45b7aad5d0" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Jun 04 11:43:59 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Jun 04 15:50:46 2014 -0700" }, "message": "Verify there\u0027s no mem map gap for immune region not to break.\n\nThis adds code that verifies that there\u0027s no memory map gap between\nthe image space and the main space so that the immune region\nfunctionality won\u0027t silently break. For example, if there\u0027s a gap and\na large object is allocated in that gap, the large object is\nincorrectly part of the immune region and the marking breaks.\n\nBug: 14059466\nChange-Id: Ie6ed82988d74b6d0562ebbbaac96ee43c15b14a6\n" }, { "commit": "7f0a6d6a699111e254a91c3980eb0522d4f8f0cd", "tree": "1cbd1b5c2f4d61b73eeb1a2e780235f6ca70fc68", "parents": [ "ddb1a1604d5070828e857902c8596de27e2fe5f5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 26 14:01:46 2014 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 26 14:01:46 2014 +0100" }, "message": "Fix x86_64 test failures.\n\nChange-Id: I97b3d1755d2e205ca7423248a2af37f1875d5116\n" }, { "commit": "9e36931cc79ca665908db9575126881d1cfdea5a", "tree": "69099016576581e3e4d6c44ee01a7a6cc7db4b2c", "parents": [ "5da6e2b88258733dd3856543af27ca73b395ef60" ], "author": { "name": "Chao-ying Fu", "email": "chao-ying.fu@intel.com", "time": "Wed May 21 11:20:52 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 22 10:25:57 2014 -0700" }, "message": "Move modify_ldt_lock into global lock order.\n\nMutex modify_ldt_lock was being removed during runtime shutdown while\ndaemons thread may still detach. Avoid this by placing in global lock\norder.\nThis fixes cts dalvik vm-tests-tf that hang on some x86 devices.\n\nBy irogers: also, tidy global locks to agree with enum constants and\nadd extra verification that the global annotalysis order agrees with\nthe LockLevel order. Bumped the oat version and moved the locks as\nLockLevel additions previously caused entrypoints to be moved. Make\nunattached lock not handle the default mutex level case by moving the\nallocated thread ids lock into the global order.\n\nChange-Id: I9d03f19d44ea254accf0ceae8022563c77f7a02f\nSigned-off-by: Chao-ying Fu \u003cchao-ying.fu@intel.com\u003e\n" }, { "commit": "3e5cf305db800b2989ad57b7cde8fb3cc9fa1b9e", "tree": "52a737323ebd505cf37ca0e4b2dcee6524fba07f", "parents": [ "27a2b70f612af9afc0fb5392fb10059f6a0a3569" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue May 20 16:40:37 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue May 20 21:17:03 2014 -0700" }, "message": "Begin migration of art::Atomic to std::atomic.\n\nChange-Id: I4858d9cbed95e5ca560956b9dabd976cebe68333\n" }, { "commit": "73d1e17b3afc7d5e56184f90bf819dc64956448a", "tree": "6fee546dcebc0c8979a62be7e75c337f79b2ac2e", "parents": [ "e8256e7773a230337c3d137cbf0365f737820405" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 11 17:53:48 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 25 14:08:49 2014 -0700" }, "message": "Enable reading page map without lock in RosAlloc::BulkFree\n\nEnabling this flag greatly reduces how much time was spent in the GC.\nIt was not done previously since it was regressing MemAllocTest. With\nthese RosAlloc changes, the benchmark score no longer regresses after\nwe enable the flag.\n\nChanged Run::AllocSlot to only have one mode of allocation. The new\nmode is finding the first free bit in the bitmap. This was\npreviously the slow path but is now the fast path. Some optimizations\nwhich enabled this include always having the alloc bitmap bits which\ncorrespond to invalid slots be set to 1. This prevents us from needing\na bound check since we will never end up allocating there.\n\nChanged revoking thread local buffer to point to an invalid run. The\ninvalid run is just a run which always has all the allocation bits set\nto 1. When a thread attempts to do a thread local allocation from here\nit will always fail and go slow path. This eliminates the need for a\nnull check for revoked runs.\n\nChanged zeroing of memory to happen during free, AllocPages should\nalways return zeroed memory. Added prefetching which happens when we\nallocate a run.\n\nSome refactoring to reduce duplicated code.\n\nErgonomics changes: Changed kStickyGcThroughputAdjustment to 1.0,\nthis helps reduce GC time.\n\nMeasurements (3 samples per benchmark):\nBefore: MemAllocTest scores: 3463, 3445, 3431\nEvaluateAndApplyChanges score | total GC time\nIter 1: 3485, 23.602436s\nIter 2: 3434, 22.499882s\nIter 3: 3483, 23.253274s\n\nAfter: MemAllocTest scores: 3495, 3417, 3409\nEvaluateAndApplyChanges score | total GC time:\nIter 1: 3375, 17.463462s\nIter 2: 3358, 16.185188s\nIter 3: 3367, 15.822312s\n\nBug: 8788501\nBug: 11790317\nBug: 9986565\nChange-Id: Ifd273a054824028dabed27c07c081dde1816f93c\n" }, { "commit": "8732bf96433bbc177de509390d285b2d5477c83d", "tree": "3c8974efdff51311910e155b226a7e64060c7e7b", "parents": [ "7f466c08888129a9923cb973a4dc73ee4a71574e", "51b71028661092e8860cca4f8ca79848e03cdc2c" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Mar 31 10:03:07 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Mar 31 10:03:08 2014 +0000" }, "message": "Merge \"Remove use of pthread_cond_timedwait_monotonic.\"" }, { "commit": "4d25df3f76f864b7629ac8c0046d46997f293d8d", "tree": "31840831d8c81d06ffd575fdb0adc8403cd8dbb1", "parents": [ "909f133bc938928a2403baccc983701cb9ebb17f" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Mar 21 17:44:46 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Mar 26 11:46:43 2014 +0100" }, "message": "Refactor deoptimization support in debugger\n\nThis CL prepares breakpoint support for inlined methods where we\u0027ll have to\ndeoptimize everything.\n\nWe move deoptimization-related information to Dbg class only (deoptimization\nrequest queue, full deoptimization event count and deoptimization lock). We\nreplace MethodInstrumentionRequest by DeoptimizationRequest. This is used to\nknow which kind of deoptimization is required for a particular event.\n\nIt also simplifies lock ordering a bit during event setup: we no longer need to\nhold the deoptimization lock while holding the breakpoint lock. Moreover, the\ndeoptimization lock should be held only after the event list lock.\n\nBug: 12187616\nChange-Id: Iff13f004adaeb25e5d609238bacce0b9720510e6\n" }, { "commit": "51b71028661092e8860cca4f8ca79848e03cdc2c", "tree": "ebb78fc3f6af61ba5d015ddaf0d28d6c3cbbbac1", "parents": [ "c5db77cce4138ac1f473872a06d230d876374337" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Mar 04 11:57:09 2014 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Mar 20 10:11:13 2014 +0000" }, "message": "Remove use of pthread_cond_timedwait_monotonic.\n\nUse posix compliant pthread_condattr_setclock instead.\nAlso, remove usage of HAVE_TIMEDWAIT_MONOTONIC and replace\nit with a specific reference to the only supported platform\nthat doesn\u0027t have it.\n\nChange-Id: I933f05c5b4965bab88ccd8e3e26c91549ed4184d\n" }, { "commit": "719d1a33f6569864f529e5a3fff59e7bca97aad0", "tree": "fcd84efd7b9806b93ec1a44e2317e6f882e7fe0e", "parents": [ "5365eea9940269b662cfbe103caa348816ff1558" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 06 12:13:39 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 06 19:16:01 2014 -0800" }, "message": "Enable annotalysis on clang ART builds.\n\nFix clang build errors aswell as restructure locking/mutex code for correct\nthread safety analysis support.\nReorder make dependencies so that host builds build first as they should\nprovide better compilation errors than target.\nRemove host\u0027s use of -fno-omit-frame-pointer as it has no value with correct\nuse of CFI, which we should have.\n\nChange-Id: I72cea8da9a3757b1a0b3acb4081feccb7c6cef90\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": "b122a4bbed34ab22b4c1541ee25e5cf22f12a926", "tree": "624f16271f4481a8fd5aa2f607385f490dc7b3ae", "parents": [ "e40687d053b89c495b6fbeb7a766b01c9c7e039c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 19 18:00:50 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Dec 20 08:01:57 2013 -0800" }, "message": "Tidy up memory barriers.\n\nChange-Id: I937ea93e6df1835ecfe2d4bb7d84c24fe7fc097b\n" }, { "commit": "34e82934546bd470283346907bd7b74990797c56", "tree": "578a6cb08be336c52609f3acb832cd213b9e187a", "parents": [ "46bc778f1feed02b20d25e3d03470c93ca2c0506" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 12 18:22:47 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 12 18:22:47 2013 -0800" }, "message": "Fix remaining mips build issues.\n\nMissed a few things in mutex.cc.\n\nChange-Id: I1c3acfc5faa2511490170199c03ab74c1f23022a\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": "d9c4fc94fa618617f94e1de9af5f034549100753", "tree": "1305efbbc3d4bc306c0947bb6d4b01553667f98e", "parents": [ "7ef126ce0593929bcf8fb73d8b1119ce3b95b3f2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 01 19:45:43 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 02 09:31:55 2013 -0700" }, "message": "Inflate contended lock word by suspending owner.\n\nBug 6961405.\nDon\u0027t inflate monitors for Notify and NotifyAll.\nTidy lock word, handle recursive lock case alongside unlocked case and move\nassembly out of line (except for ARM quick). Also handle null in out-of-line\nassembly as the test is quick and the enter/exit code is already a safepoint.\nTo gain ownership of a monitor on behalf of another thread, monitor contenders\nmust not hold the monitor_lock_, so they wait on a condition variable.\nReduce size of per mutex contention log.\nBe consistent in calling thin lock thread ids just thread ids.\nFix potential thread death races caused by the use of FindThreadByThreadId,\nmake it invariant that returned threads are either self or suspended now.\n\nCode size reduction on ARM boot.oat 0.2%.\nOld nexus 7 speedup 0.25%, new nexus 7 speedup 1.4%, nexus 10 speedup 2.24%,\nnexus 4 speedup 2.09% on DeltaBlue.\n\nChange-Id: Id52558b914f160d9c8578fdd7fc8199a9598576a\n" }, { "commit": "967a0adf8b93a23d2a8fef82e06bd913db94ac19", "tree": "35b7fbf6eb1899736213e57e11c85d16c1b4853e", "parents": [ "af3994db1b42929666b6a50c3c4b237a25c99951" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Sep 10 16:24:21 2013 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Sep 11 15:38:23 2013 -0700" }, "message": "More allocation code optimizations.\n\n- Inline Class::AllocObject() and Array::Alloc().\n- Inline some short Mutex functions and add LIKELY/UNLIKELY to some\n Mutex functions.\n- This change improves the Ritz MemAllocTest by ~6% on Nexus 4 and\n ~10% on host.\n\nBug: 9986565\nChange-Id: I1606c74ddb21676cbc1de1a40e9b076fc23eaea4\n" }, { "commit": "b3733086ab415088b97fac20b3eea24433a7d2c5", "tree": "ad853d385e5c113e5d6248d0f7d198d7d9136865", "parents": [ "4baa8083ae0820f71cf0f03b94fd7f8d3b9bfa46" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Aug 12 17:28:49 2013 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Aug 13 13:53:12 2013 -0700" }, "message": "Add a systrace support for lock contention logging.\n\n- Now several ART executables like oatdump need to link with libcutils\n as the mutex code is shared among them.\n- The blocking thread ID and lock owner thread ID are passed to\n ScopedContentionRecorder in the correct order.\n\nBug: 9986464\nChange-Id: Id766de23fbc4af1d8ba2de051595e365b04f5ae7\n" }, { "commit": "1afde13b36cc1d67528104c2b1395495f669cd3f", "tree": "5bcf953cb76033d812f06c096908344e8b6e9cd9", "parents": [ "c9f9444a0d58f395fee5ddd9f55ff32f92cb7b57" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Aug 06 17:09:30 2013 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Aug 09 12:45:36 2013 -0700" }, "message": "Polish the lock contention logging.\n\n- Make the code compilable.\n- Surround the code with kLogLockContentions instead of #ifdef CONTENTION_LOGGING.\n- Dump contended locks before never-contended locks for better log readability.\n- Change the wait time unit from ms to us for better precision.\n\nBug: 9986464\nChange-Id: I121c6ccf4424d3e0339b0dcd25e18976b41fe4f3\n" }, { "commit": "7934ac288acfb2552bb0b06ec1f61e5820d924a4", "tree": "43f3acd8af7fd34d4ae7b64f6e06bb8429d74bb8", "parents": [ "fb331d7ca004f39608fcfdae49d38df90c702ea9" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 26 10:54:15 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 26 11:55:10 2013 -0700" }, "message": "Fix cpplint whitespace/comments issues\n\nChange-Id: Iae286862c85fb8fd8901eae1204cd6d271d69496\n" }, { "commit": "0de7985b29257bb60be511db774a4f0119a81f20", "tree": "9b5e5f96f9e9e3801118093d97814e0edd78ba63", "parents": [ "58b42b986742565cccfde4366495ad23d0ba57d1" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 25 22:29:58 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 26 00:00:52 2013 -0700" }, "message": "Consistently handle EINTR on FUTEX_WAIT\n\nChange-Id: I848f33b51f7087232275a125d7ae414909233437\n" }, { "commit": "6eb5288264d68276085855bd041fa74fbca6827c", "tree": "8ac3d0a3667adaa260d70a14a4935bd22b1e7f97", "parents": [ "f52935278fca8c7aa220543eef4544e3d1105d91" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 19 00:31:07 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 19 00:31:07 2013 -0700" }, "message": "Conditionally define non-Android pthread structures\n\nChange-Id: I2258b521db20dcc6940cb5540ca24ea5bbaf006f\n" }, { "commit": "fb6996fe2b1bb28526751650441d7ac29dcdb4e6", "tree": "aba94b367c89719413d4be8fa7fd445ce3bee743", "parents": [ "3e79aadc447dcdb9e52368baced9405667e68e6a" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 18:21:14 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 21:55:45 2013 -0700" }, "message": "Fixing cpplint runtime/arrays, runtime/int, runtime/virtual issues\n\nChange-Id: Ia2ff94d2fb69465df26aaf83df82614a483b26e0\n" }, { "commit": "02c8cc6d1312a2b55533f02f6369dc7c94672f90", "tree": "4e4a268e8cb82dcd95f8e29a8169b0396e20b0cc", "parents": [ "6f485c62b9cfce3ab71020c646ab9f48d9d29d6d" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 15:54:44 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 16:02:33 2013 -0700" }, "message": "Fixing cpplint whitespace/blank_line, whitespace/end_of_line, whitespace/labels, whitespace/semicolon issues\n\nChange-Id: Ide4f8ea608338b3fed528de7582cfeb2011997b6\n" }, { "commit": "df62950e7a32031b82360c407d46a37b94188fbb", "tree": "038bf95a2ce296ae3e0c30a131ac22c0986f0f52", "parents": [ "0cd7ec2dcd8d7ba30bf3ca420b40dac52849876c" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 22:39:56 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 00:13:18 2013 -0700" }, "message": "Fix cpplint whitespace/parens issues\n\nChange-Id: Ifc678d59a8bed24ffddde5a0e543620b17b0aba9\n" }, { "commit": "93ba893c20532990a430741e0a97212900094e8c", "tree": "019990a1957562c1f573c77297df5923cce98f47", "parents": [ "b1eba213afaf7fa6445de863ddc9680ab99762ea" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 21:31:49 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 21:46:12 2013 -0700" }, "message": "Fix cpplint runtime/explicit issues\n\nChange-Id: I352ba0b427f1ff9b22887693952b180eae0839ba\n" }, { "commit": "2ce745c06271d5223d57dbf08117b20d5b60694a", "tree": "08ca327d1f1c8266fafb11422cfad6982f21e0bb", "parents": [ "fc0e3219edc9a5bf81b166e82fd5db2796eb6a0d" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 17:44:30 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 21:46:11 2013 -0700" }, "message": "Fix cpplint whitespace/braces issues\n\nChange-Id: Ide80939faf8e8690d8842dde8133902ac725ed1a\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": "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": "c4ee12e73fc844f28a9812a9141985cf05143a57", "tree": "59ea9f105b39b05beea9ff17cda83f1122461629", "parents": [ "d8274bcbc565fff1254574b3cbf46dc4e7b52812" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 16 11:19:53 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 16 11:26:24 2013 -0700" }, "message": "Avoid recursion if logging lock is corrupt.\n\nIf the logging lock is corrupt then using LOG fails and recursive death ensues.\nMake Mutex a friend of LogMessage so that it can do raw logging in this special\nsituation.\nOpportunistically also make fields of LogMessage and LogMessageData that can be\nconst, const.\n\nChange-Id: I9e0d07c3224096bcf03d6410cd64bb8b5c831fac\n" }, { "commit": "23055dc5d7a90c4a12e259fd0ed7cd4d04d89182", "tree": "ef7ae39a1c99fd7c57cf93473c27ba92c92299a0", "parents": [ "d914eb2a839f7b40156ff0299a60e5cb80080b73" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Apr 18 16:29:16 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Apr 18 16:29:16 2013 -0700" }, "message": "Mutex tidy up.\n\nCondition variable names needn\u0027t be strings - the use was leading to valgrind\nwarnings.\nDon\u0027t fail AssertSharedHeld when self isn\u0027t known.\nAllow gtest thread chance to merge to avoid memory leak.\nMake barrier test log output more human readable.\n\nChange-Id: If1923c69a2965d933036c496dc5b1d64ec887db2\n" }, { "commit": "04d7aa92bc5548bc4d272b9480614f06248194cc", "tree": "2b399853daab91995488c7d19a694063262f6531", "parents": [ "aed0716b9592bb3095cdfc4b111011f9ed74877d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 16 14:29:17 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 16 14:29:17 2013 -0700" }, "message": "Fix object verification.\n\nRefactor VERIFY_OBJECT_ENABLED to become less brittle to change enum and global\nconstant.\n\nChange-Id: Ie405106be81dce9a913730c7f46a5659582fa18b\n" }, { "commit": "0f827169742aad6209d830db773a101849c32a83", "tree": "df06bd3d9fa91757d2c87fbe891007184f823e9d", "parents": [ "eb5cb608fcb0d636961653ad561b2c7ee40de1ad" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Feb 26 12:12:58 2013 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Feb 26 12:12:58 2013 -0800" }, "message": "Fix JDWP ObjectRegistry lock ordering.\n\nAlso make debugging lock ordering violations nicer by using our\nLockLevel operator\u003c\u003c.\n\nChange-Id: Ic15ebe70363a90a09f6491bd5c336a604e9d6c48\n" }, { "commit": "b6c31ea69973b41d57146ab0c319613213a9a85e", "tree": "14bed22d29a40de9658f1370689610dc6fc9491d", "parents": [ "7cb0f9cf9c7798ae9c4d6cad47e7861f7a511f7e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 04 18:11:33 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 04 18:12:41 2013 -0800" }, "message": "Move hot routine out of mutex.cc.\n\nChange-Id: I06c48481f0072e93dc2c23d4fd5e562f13003673\n" }, { "commit": "693ff61274cd2c9b8eb7e68c370f84a911b8ca52", "tree": "11f1f6b3e5302b8a58c5ca44c3a1bec1b16d90a4", "parents": [ "12a2bdd245ea866b665d4800b493397ec412892a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 01 10:56:12 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 01 21:29:49 2013 -0800" }, "message": "Performance improvements by removing a DMB and inlining.\n\nCorrect the version of CAS used by Mutex::Lock to be acquire and not release.\nDon\u0027t do a memory barrier in thread transitions when there is already a\nbarrier associated with the mutator lock.\nForce inlining of the hot thread and shared lock code, heavily used by down\ncalls and JNI.\nForce inlining of mirror routines that are used by runtime support and hot.\n\nPerformance was measured and improved using perf and maps.\n\nChange-Id: I012580e337143236d8b6d06c1e270183ae51083c\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": "56edc432fa914f7ccfa87ce443e64f5ef475666d", "tree": "4e05c32f3524043614a2f368a6d49f7f3bc24616", "parents": [ "d75afb99f0c8b8a27ad84e2e31010383ecf72120" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jan 18 16:51:51 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jan 22 08:47:33 2013 -0800" }, "message": "Support for contention logging with ART futexes.\n\nRemove dangerous postfix operators on AtomicInteger. Clean up the atomic stack.\nFactor nanosleep into a useful shared utils.h routine.\n\nChange-Id: I417a73007c23fe247f410f41b2fa41a717c22139\n" }, { "commit": "07ed66b5ae659c452cbe1ab20c3dbf1d6f546461", "tree": "2350745da33df6fcb9fb0c9059e55ea5d5ea8f67", "parents": [ "76b6167407c2b6f5d40ad895b2793a6b037f54b2" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Dec 12 18:34:25 2012 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Dec 12 18:35:05 2012 -0800" }, "message": "Move logging.h into base/logging.h.\n\nChange-Id: Id68f85f7c3a71b156cb40dec63f94d4fb827f279\n" }, { "commit": "76b6167407c2b6f5d40ad895b2793a6b037f54b2", "tree": "5c433a0907ddf401f4078b6703253b1ae75e723e", "parents": [ "f66330a747eff155eb571b634fc9d901c55a78a7" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Dec 12 17:47:30 2012 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Dec 12 18:11:44 2012 -0800" }, "message": "Move mutex.h into base/mutex.h.\n\nWe don\u0027t actually need any hackery to get the right mutex.h any\nmore, but since we planned on having the google3-like stuff\nin google3-like locations, this is an easy one to fix.\n\nChange-Id: Ie27464ebbc208a6f4e694a97cf26b1cee0737009\n" }, { "commit": "760c943b1f7ed0838d5909f49d70584f776add20", "tree": "6055abbe3a6e45d52cf3370857d65942d640c190", "parents": [ "076539644368f61e679a9e6afca0b025440ee5b7" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Nov 29 16:46:27 2012 -0800" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Nov 29 16:50:40 2012 -0800" }, "message": "After ART_USE_FUTEXES was enabled, we need some different gAborting protection\n\nChange-Id: Ic5baa5678317e08d3082d36ebb0a484667f94e13\n" }, { "commit": "d45f201e9bd43490e30a35710865789b8d70e249", "tree": "cbd318570af04229cf44fa5ab62d8b240f300b39", "parents": [ "bc43860b7e5326264ada842c35a840dd4cda8120" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Nov 28 11:46:23 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Nov 28 11:55:23 2012 -0800" }, "message": "Simplify ConditionVariable and avoid potential race.\n\nMake waiters maintain mutex contenders rather than signal/broadcast\n(eliminates awoken count). Avoids race where a spurious Signal wake\ncould remove a Broadcast contender from the mutex.\n\nChange-Id: I5c3e36878c9fa2db09f5dc31d24a0a6222a61731\n" }, { "commit": "6a07d3c16402280560835ebd99ec2df8936506c5", "tree": "acc4bd1e06db8c764835ccedff48248d69fb6905", "parents": [ "725de295024c9e5adf5e316f498c39d48158e19d", "5bd97c483c1de1eb97afe76123b1b9ab53095edf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 27 13:20:47 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Tue Nov 27 13:20:47 2012 -0800" }, "message": "Merge \"Fix and enable ART futexes.\" into dalvik-dev" }, { "commit": "9e419ca03c674b913135b87bcf2291a12708260b", "tree": "a6f941c14eb4d2f1eaf3a020ec3f3b5432e6ee21", "parents": [ "a2424c27352351766512f39d3095f496dbb27ae7" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Nov 27 11:31:49 2012 -0800" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Nov 27 11:31:49 2012 -0800" }, "message": "Disable another CHECK when gAborting\n\nChange-Id: I9a53d200c93ddb2dd95820e03040761fb2d5a752\n" } ], "next": "5bd97c483c1de1eb97afe76123b1b9ab53095edf" }