)]}' { "log": [ { "commit": "1c8f4ff04062407db3d2dd834dd6bc24f6662c4d", "tree": "a116c7d55fc1bcff2b451028f46a1196ee15d990", "parents": [ "c1be9f2b776b4900e0603013f7c4d84e534f0bb0" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Apr 14 15:05:12 2017 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Apr 18 18:06:54 2017 +0200" }, "message": "Always check result of Thread::ModifySuspendCount\n\nEnsures that we never ignore the result of ModifySuspendCount so that\nwe can react if the suspend count is not updated as expected.\n\nThis CL does the following:\n* Adds __attribute__((warn_unused_result)) on the method to raise an\n error at compilation time if the result is ignored.\n* Wraps calls with DCHECK where the result used to be ignored.\n\nBug: 27385848\nTest: make -j test-art-host\nTest: art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64\nChange-Id: I2d0e1ab7158c70ec8076c8bae6e4b814aee75af6\n" }, { "commit": "efd20cb2f63cf647c7d947d00e8987affefeb177", "tree": "666ea53832f1085378bcc75620f314ba1f46a330", "parents": [ "caa348cee1312150a957bd308ce77ad406f3eb64" ], "author": { "name": "Josh Gao", "email": "jmgao@google.com", "time": "Tue Feb 28 16:53:59 2017 -0800" }, "committer": { "name": "Josh Gao", "email": "jmgao@google.com", "time": "Wed Mar 08 15:07:49 2017 -0800" }, "message": "Catch signals that happen inside a fault handler.\n\nUnblock some signals (SIGABRT, SIGBUS, SIGSEGV) that could happen\ninside of the ART internal fault handlers, to report crashes inside of\nthe signal handler. Because we can\u0027t use sigaction to change the\nhandler when this happens, because it modifies global state, add a new\nmember variable in Thread to track whether a call to the fault handler\nis reentrant or not.\n\nRemove the old nested signal implementation that attempted to do this.\n\nBug: http://b/35853436\nTest: changed the #if 0 to #if 1, ran a dummy process that\n threw a NullPointerException, inspected logcat\nChange-Id: I04bb4a09433c6817933d64ec681ec433b528f2a5\n" }, { "commit": "45d942662883a0c5106b85b54e87be0f0610b394", "tree": "c5fc69b1d327ac7479eb2b6f90f9d45c4246689d", "parents": [ "3fb852a88d2a8ffaa87089752f4b1d5f9d6ce3c1", "bad529da18122c7044fbfd7d56153535d34367b1" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Feb 15 21:02:26 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Feb 15 21:02:27 2017 +0000" }, "message": "Merge \"ART: Add currentThread cutout to Unstarted Runtime\"" }, { "commit": "8dc12b1546c7409be19a7b5dc48932011db13067", "tree": "a157d2fb7bee08f7c0290e8a41f0c7c9c883794d", "parents": [ "e92ba687f9ab2a3cd9724f494b6899e8e6e472ec", "ffc8cad04135192a8d10e7f38bd97c48101294a8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 15 09:39:05 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Feb 15 09:39:06 2017 +0000" }, "message": "Merge \"Add a DCHECK in thread::GetPeer.\"" }, { "commit": "bad529da18122c7044fbfd7d56153535d34367b1", "tree": "44982bc9ec345be1e96ba6c66e22e5c08fcaed56", "parents": [ "1fb51268b78f55b345ea951a73eae5194ec71b91" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Feb 13 18:52:10 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Feb 14 17:43:52 2017 -0800" }, "message": "ART: Add currentThread cutout to Unstarted Runtime\n\nAdd currentThread() and getThreadState() cutouts to the unstarted\nruntime to allow further compile-time initialization. The cutouts\nare protected by call-stack checks.\n\nAdd tests.\n\nBug: 34956610\nTest: m test-art-host-gtest-unstarted_runtime_test\nChange-Id: I6335bccda8bedae90376fc7c47b303576f1ac78b\n" }, { "commit": "ffc8cad04135192a8d10e7f38bd97c48101294a8", "tree": "fb90432ea5e6d218891c0b3fcccc0fa77d202f7d", "parents": [ "4e4b62e21672dbacf5d5abb688a126aabad29269" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 10 10:59:22 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 14 15:42:29 2017 +0000" }, "message": "Add a DCHECK in thread::GetPeer.\n\nMotivated by https://android-review.googlesource.com/#/c/333205/.\n\nTest: test-art-host run-jdwp-test.sh\nChange-Id: I173c060324aa0dc39144db55e3a97e672c012ba8\n" }, { "commit": "a222404a5832ab16786931576d52825d08eed3ca", "tree": "7350e5efbb6ef72c33e82753d79f7f4bed92d9d4", "parents": [ "1561de49b382627ddd277b8ad7e5e8f4cec32f0b" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Feb 08 16:35:45 2017 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Feb 10 16:15:01 2017 -0800" }, "message": "Fix the empty checkpoint timeout.\n\nThe problem happens when a thread is indirectly blocked on a mutex\nthat another thread holds and is blocked on a weak ref access.\n\nAdd a way to do a dummy wakeup on a thread that\u0027s blocked on a mutex\nso that the thread will respond to the empty checkpoint request. Do\nthis for the mutexes that are expected to be held when a weak ref is\naccessed. Add a check that detects an unexpected case.\n\nBug: 33006388\nBug: 12687968\nTest: test-art-host.\nChange-Id: Iefec69b9a21aa25a928cb31fcf4fb872f867a8c2\n" }, { "commit": "23d7c8590bea7f24b301d1dff79e09a4a9a8a51e", "tree": "132e735766807d4b4f09a266b8c1184751f31a74", "parents": [ "6347ea975cd80e7344c8d04909e211685b9a32ad", "02f365f0fb74acf09ffb8f04a80d32ca1152ed8b" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Feb 07 07:10:59 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Feb 07 07:11:00 2017 +0000" }, "message": "Merge \"Force stack dump to diagnose empty checkpoint timeout (2).\"" }, { "commit": "02f365f0fb74acf09ffb8f04a80d32ca1152ed8b", "tree": "7658baab9ac5c3c86b974d8dd5238f3cf2d9f2f0", "parents": [ "7b220d60e3cb58c384a0d245106406080c6f3e37" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Feb 03 15:06:00 2017 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Feb 06 16:06:58 2017 -0800" }, "message": "Force stack dump to diagnose empty checkpoint timeout (2).\n\nBug: 33006388\nBug: 12687968\nTest: test-art-host\nTest: Thread dumping in a simulated empty checkpoint timeout.\n\nChange-Id: I06641396b8f3d7a1a98366a01807aab2e6f31bd5\n" }, { "commit": "202f85a54bfd1844664a18c18a4c0494763c6b82", "tree": "7f24ae09a96a91d4ad41a4cc8de18c9b9a79a6c0", "parents": [ "df04fda5638aa1fb9956808f1cd497b49f62088e" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Feb 06 10:23:26 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Feb 06 13:44:54 2017 -0800" }, "message": "ART: Add marking to thread peer gathering in TI\n\nWhen CC is marking, we may get stale references due to a missing\nmark.\n\nBug: 34760612\nTest: ART_TEST_JIT\u003dtrue ART_TEST_GC_STRESS\u003dtrue test-art-host-run-test-924-threads\nTest: ART_TEST_JIT\u003dtrue ART_TEST_GC_STRESS\u003dtrue test-art-host-run-test-925-threadgroups\nChange-Id: I1becfc188b59a3c99cc7eea07c63abaaf108fd15\n" }, { "commit": "13c1635e07b8ee09120e267e2abe860aee05db7c", "tree": "f9f8464c9ba62af15213afc91879751af6959045", "parents": [ "a0305a27fc2da2bff16415e2edf733db2adf202f" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Jan 31 10:15:08 2017 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Jan 31 10:37:14 2017 -0800" }, "message": "Force stack dump to diagnose empty checkpoint timeout.\n\nBug: 33006388\nBug: 12687968\nTest: test-art-host\nChange-Id: Ida149c98eeb05ca4ad8d0f7ef49228c9b45a6312\n" }, { "commit": "7cd7b964e28ba9985565e7c3154357ed7d153fdf", "tree": "f1dbee29af9bde819b913f52d63b69ef5b10165c", "parents": [ "03042d54563657ca4904cb35a6cd8092b37b2001", "ac30fa2151095d2b5fe42f169a02ed09fea312b4" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Jan 20 01:49:00 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jan 20 01:49:01 2017 +0000" }, "message": "Merge changes Ib5a3632c,I871f6b3c,I6ce0f71d\n\n* changes:\n ART: Make RuntimeCallbacks unique_ptr\n ART: Add ClassLoadCallback\n Revert \"Revert \"ART: Start RuntimeCallbacks\"\"\n" }, { "commit": "732b0aca4fb8e5ffbd518ca913cb82810b6b2061", "tree": "ee3b509a39400af3cac5be5f6a8974290374123d", "parents": [ "07e56f3e1d1d6d7cca3a8819cb8429a65f0507f2" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 18 15:23:39 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 19 17:13:21 2017 -0800" }, "message": "ART: Add Agent Thread API\n\nAdd support for RunAgentThread. Add test.\n\nBug: 31684593\nTest: m test-art-host-run-test-931-agent-thread\nChange-Id: I5deb213fb06eedc5ee78a340458cf0dff615d0ac\n" }, { "commit": "04bbb5be5b9c0f0b3a72116353d23ea63c8bc5e9", "tree": "51ee41e14d6bfccc88ccde91938cac901c0e39a9", "parents": [ "a58bc2e74441ac89cb706b49c5a6d096df97080b" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 19 17:49:03 2017 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 19 15:12:06 2017 -0800" }, "message": "Revert \"Revert \"ART: Start RuntimeCallbacks\"\"\n\nThis reverts commit bf6331a45f730e1f1044af2ce43bceda660ae6fc.\n\nSwitch to using the mutator lock.\n\nBug: 31684920\nTest: m test-art-host-gtest-runtime_callbacks_test\nTest: art/tools/run-jdwp-tests.sh --mode\u003dhost --debug\nChange-Id: I6ce0f71de2aa9f90cd23cfca7723a793b560b16b\n" }, { "commit": "bf6331a45f730e1f1044af2ce43bceda660ae6fc", "tree": "66013474dc85b2a868d5fad2506c8674a086fa1a", "parents": [ "13093d455b8266338fd713b04261c58e9dc2b164" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 19 09:02:50 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 19 09:02:50 2017 +0000" }, "message": "Revert \"ART: Start RuntimeCallbacks\"\n\njdwp tests failing with:\n STDERR\u003e dalvikvm32 E 01-18 23:14:50 12180 12198 thread-inl.h:137] holding \"runtime callbacks lock\" at point where thread suspension is expected\n\nBug: 31684920\n\nThis reverts commit 13093d455b8266338fd713b04261c58e9dc2b164.\n\nChange-Id: I94e4154e273d006eecdd485607dcfd96392d6a00\n" }, { "commit": "13093d455b8266338fd713b04261c58e9dc2b164", "tree": "b4c5e54ce8a74af2f8929df089133967ea7f7274", "parents": [ "3a5e34b65727d2c9e7e6ebe45c337dae0d0398a4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 17 21:40:35 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 18 19:48:37 2017 +0000" }, "message": "ART: Start RuntimeCallbacks\n\nAdd a central RuntimeCallbacks structure to handle certain interesting\nruntime events.\n\nIn a first iteration, add ThreadLifecycleCallback with ThreadStart and\nThreadStop. Move Dbg over to ThreadLifecycleCallback.\n\nAdd a test.\n\nBug: 31684920\nTest: m test-art-host-gtest-runtime_callbacks_test\nTest: art/tools/run-jdwp-tests.sh --mode\u003dhost\nChange-Id: Ie0f77739a563207bfb4f04374e72dc6935c40b4f\n" }, { "commit": "e71b35446985835363a4508646cf7b1121bd95a3", "tree": "bd40763b04ba2028f3383736b2a14808e407120c", "parents": [ "8bd59a0fd46db83616785168231e09fb95ed2ead" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Jan 16 14:58:23 2017 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Jan 17 11:55:07 2017 +0000" }, "message": "Move some fields in art::Thread to reduce maintenance burden.\n\nMove fields `thread_local_start`, `thread_local_pos`,\n`thread_local_end` and `thread_local_objects` before fields\n`jni_entrypoints` and `quick_entrypoints` within\nart::Thread, to avoid repetitive art::Thread field moves in\nfuture CLs caused by the addition or deletion of entry\npoints.\n\nTest: m test-art-host\ntest: m test-art-target (on ARM)\nChange-Id: Ib67842e44a7f21a871ca4d1bb95dc6f7cfedc829\n" }, { "commit": "f26bf2dd7d43716946ef8d4e046b9c0be36fb3a4", "tree": "8dc3514ac4d2c3bc45b2621e78448a5c169d350e", "parents": [ "4ca818c31cc38746332e68a89ca48be88aa9d7b6" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jan 13 16:47:14 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 16 08:20:13 2017 -0800" }, "message": "ART: Add Thread TLS support\n\nAdd support for GetThreadLocalStorage and SetThreadLocalStorage.\nAdd test.\n\nBug: 31684593\nTest: m test-art-host-run-test-924-threads\nChange-Id: Ia2e567a832716b076a61664809ab29290fffcb70\n" }, { "commit": "ac141397dc29189ad2b2df41f8d4312246beec60", "tree": "a2f481463a14695bf9327fd2f549878ecf30c77b", "parents": [ "5c9f90c5ecf2ff6f93ada0f7b18b46d866c59ea1" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jan 13 11:53:47 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Sun Jan 15 15:18:07 2017 +0000" }, "message": "Revert \"Revert \"ART: Compiler support for invoke-polymorphic.\"\"\n\nThis reverts commit 0fb5af1c8287b1ec85c55c306a1c43820c38a337.\n\nThis takes us back to the original change and attempts to fix the\nissues encountered:\n\n- Adds transition record push/pop around artInvokePolymorphic.\n- Changes X86/X64 relocations for MacSDK.\n- Implements MIPS entrypoint for art_quick_invoke_polymorphic.\n- Corrects size of returned reference in art_quick_invoke_polymorphic\n on ARM.\n\nBug: 30550796,33191393\nTest: art/test/run-test 953\nTest: m test-art-run-test\n\nChange-Id: Ib6b93e00b37b9d4ab743a3470ab3d77fe857cda8\n" }, { "commit": "0d3998b5ff619364acf47bec0b541e7a49bd6fe7", "tree": "a4763c0660372f6311b612c09267cbbc2fe71e89", "parents": [ "aa89a4c6fca095904521842c018399f1e3501a45" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 15:35:12 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 16:51:12 2017 +0000" }, "message": "Revert \"Revert \"Make object allocation entrypoints only take a class.\"\"\n\nThis reverts commit f7aaacd97881c6924b8212c7f8fe4a4c8721ef53.\n\nChange-Id: I6756cd1e6110bb45231f62f5e388f16c044cb145\n" }, { "commit": "c8144cdad955b77988a48777cfbdc6fd2e8c1916", "tree": "354e00610ec25279a8c4b77e8b685e380815813f", "parents": [ "d1a277954284c4dd4b5b14fd4e58f1854daed848", "f7aaacd97881c6924b8212c7f8fe4a4c8721ef53" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jan 12 06:19:22 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jan 12 06:19:23 2017 +0000" }, "message": "Merge \"Revert \"Make object allocation entrypoints only take a class.\"\"" }, { "commit": "f7aaacd97881c6924b8212c7f8fe4a4c8721ef53", "tree": "780209ac8e992fa63307062977f672aa5bb55d9e", "parents": [ "2b615ba29c4dfcf54aaf44955f2eac60f5080b2e" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Jan 12 02:58:38 2017 +0000" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Jan 12 02:58:38 2017 +0000" }, "message": "Revert \"Make object allocation entrypoints only take a class.\"\n\n960-default-smali64 is failing.\n\nThis reverts commit 2b615ba29c4dfcf54aaf44955f2eac60f5080b2e.\n\nChange-Id: Iebb8ee5a917fa84c5f01660ce432798524d078ef\n" }, { "commit": "dcf52765ab5886abdd85a4436fa0358b2a31341d", "tree": "b68ee975792a5bf488ed93cfbe09a37f80008288", "parents": [ "a28ddf5140cd1f4a2ae93dbf8be2f200b1552003", "0fb5af1c8287b1ec85c55c306a1c43820c38a337" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 19:18:51 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 11 19:18:51 2017 +0000" }, "message": "Merge \"Revert \"ART: Compiler support for invoke-polymorphic.\"\"" }, { "commit": "0fb5af1c8287b1ec85c55c306a1c43820c38a337", "tree": "66239e7f745fae54e1630e91fb44a859bff615d6", "parents": [ "02e3092f8d98f339588e48691db77f227b48ac1e" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 18:58:15 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 19:17:33 2017 +0000" }, "message": "Revert \"ART: Compiler support for invoke-polymorphic.\"\n\nThis reverts commit 02e3092f8d98f339588e48691db77f227b48ac1e.\n\nReasons for revert:\n\n- Breaks MIPS/MIPS64 build.\n- Fails under GCStress test on x64.\n- Different x64 build configuration doesn\u0027t like relocation.\n\nChange-Id: I512555b38165d05f8a07e8aed528f00302061001\n" }, { "commit": "79f9928fc9e0a88430f3329069bfb2f9a0d37f0c", "tree": "e3142e4829c808c3df1059f3b05c0b3a37193ce9", "parents": [ "716eb25353390f699778a79d69006a5b8d8289c2", "02e3092f8d98f339588e48691db77f227b48ac1e" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jan 11 18:08:03 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 11 18:08:04 2017 +0000" }, "message": "Merge \"ART: Compiler support for invoke-polymorphic.\"" }, { "commit": "716eb25353390f699778a79d69006a5b8d8289c2", "tree": "ba86b06bb9dc02deea0420d4432e00b219ea94f2", "parents": [ "07d2e7a760aafa54fb043917bf84dad0484dbd68", "f5769e15aa9f058a5af987f0d1354f16f4464a7d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 11 17:59:34 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 11 17:59:35 2017 +0000" }, "message": "Merge \"Move most of art::Thread to ObjPtr\"" }, { "commit": "02e3092f8d98f339588e48691db77f227b48ac1e", "tree": "127dd23346206b0547b7c6453a776253252b3c6e", "parents": [ "bc7d0deda4549f314e68ee3e0e6afd68c4a8fd06" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Dec 01 10:33:51 2016 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 16:26:43 2017 +0000" }, "message": "ART: Compiler support for invoke-polymorphic.\n\nAdds basic support to invoke method handles in compiled code.\n\nEnables method verification for methods containing invoke-polymorphic.\n\nAdds k45cc/k45rc output to Instruction::DumpString() which\nwas found to be missing when enabling verification.\n\nInclude stack traces in test 957-methodhandle-transforms for\nfailures so they can be easily identified.\n\nBug: 30550796,33191393\nTest: art/test/run-test 953\nTest: m test-art-run-test\nChange-Id: Ic9a96ea24906087597d96ad8159a5bc349d06950\n" }, { "commit": "2b615ba29c4dfcf54aaf44955f2eac60f5080b2e", "tree": "0a2fe5f9243645a054d4aa094bff5a69cc1abb88", "parents": [ "c9a060f2688599d4a402ee6234db46c2e9b7463f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 06 14:40:07 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 11 10:34:10 2017 +0000" }, "message": "Make object allocation entrypoints only take a class.\n\nChange motivated by:\n- Dex cache compression: having the allocation fast path do a\n dex cache lookup will be too expensive. So instead, rely on the\n compiler having direct access to the class (either through BSS for\n AOT, or JIT tables for JIT).\n- Inlining: the entrypoints relied on the caller of the allocation to\n have the same dex cache as the outer method (stored at the bottom of\n the stack). This meant we could not inline methods from a different\n dex file that do allocations. By avoiding the dex cache lookup in\n the entrypoint, we can now remove this restriction.\n\nCode expansion on average for Docs/Gms/FB/Framework (go/lem numbers):\n- Around 0.8% on arm64\n- Around 1% for x64, arm\n- Around 1.5% on x86\n\nTest: test-art-host, test-art-target, ART_USE_READ_BARRIER\u003dtrue/false\nTest: test-art-host, test-art-target, ART_DEFAULT_GC_TYPE\u003dSS ART_USE_TLAB\u003dtrue\n\nChange-Id: I41f3748bb4d251996aaf6a90fae4c50176f9295f\n" }, { "commit": "f5769e15aa9f058a5af987f0d1354f16f4464a7d", "tree": "8a55c514c70a701a06102d3cbda6e4fe8a84242c", "parents": [ "2e17788df086284845108c6b39954a5c74e93a69" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 15:54:41 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 15:57:11 2017 -0800" }, "message": "Move most of art::Thread to ObjPtr\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I73d0de3c875d3ec3d732034fdc961dae79ef4070\n" }, { "commit": "ccd56958eb46fbb00c1eb45c7a7b23d5bbfd7698", "tree": "8ff64df9415a91c4a1bc9ef100293de743fe48b4", "parents": [ "9538f9e2a5d03f1b1bc07ebfbd93b61dcf8ad604" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Dec 15 17:57:38 2016 +0000" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Dec 16 20:58:46 2016 +0000" }, "message": "Add Thread entry to signal if the thread can call into java\n\nCompiler threads (AOT or JIT) should not call into Java as they have no\npeers (which may lead to crashes, e.g. b/33067273)\n\nBug: 32602185\nBug: 33067273\n\nTest: m test-art-host-run-test; m test-art-host-gtest\nChange-Id: I97dda7a5444643db3c5d5318339a65a602f709e8\n" }, { "commit": "585da955bc8e5040705dcfd941b2131025ebcef8", "tree": "d22555ab3b33b446271fcbbd1676177d21d7221f", "parents": [ "ce29d1813914ee9a9e9b04b034968f09694dd557" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Dec 02 14:52:29 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Dec 12 11:47:02 2016 -0800" }, "message": "ART: Add precise root visiting\n\nAdd VisitRootFlags::kVisitRootFlagPrecise to signal a\nrequest for precise RootInfo objects.\n\nMove VisitRootFlags to gc_root.h. Refactor VisitRoot\nfunctions to pass flags. Add code in Thread visiting\nto decode vregs in compiled code.\n\nBug: 31385354\nTest: m test-art-host\nChange-Id: I8e7280f0ab682871f729f2a1c6b18670cf2dbf82\n" }, { "commit": "2c19f5b12cd4cd446fda6b4dd05397e9f8424dc3", "tree": "04032e98d1934fd8e4287698bbab86fe29173bf5", "parents": [ "0480523e01102e40a072d266e43a18a0ca4344e4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 28 08:10:18 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Dec 02 17:24:11 2016 -0800" }, "message": "ART: Clean up Thread\n\nMove debug code to .cc, simplify includes.\n\nTest: m test-art-host\nChange-Id: I5e66a54223a8f965a46fb14021d0e29aa2496ed1\n" }, { "commit": "5ace201d84adb7753680bf4c7877b3b71558da82", "tree": "4687d41f5a1c8358e8356066d1e1b43b231b6037", "parents": [ "304f2d3a1d090846a7a55a0016efc9fc8eb14078" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 30 10:15:41 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 30 11:01:52 2016 -0800" }, "message": "Revert \"Revert CC related changes.\"\n\nDisable entrypoint switching in ResetQuickAllocEntryPointsForThread\ninstead of callers. Fixes bug where instrumentation would switch\nto non CC entrypoints for non X86_64 architectures causing aborts.\n\nBug: 31018974\n\nTest: test-art-host\nTest: test/run-test 099\n\nThis reverts commit 96172e0172c5fca6e9a5ad4b857a24d8c7b064e5.\n\nChange-Id: If206694ae35ff4446c6a8a97bfbcbf2dac35e3f9\n" }, { "commit": "96172e0172c5fca6e9a5ad4b857a24d8c7b064e5", "tree": "04bb8fe644c9bc961c8ac33e609c1b162ec45222", "parents": [ "626b839f881f09a1481377a76712d08580c47a16" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 30 11:52:19 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 30 11:52:19 2016 +0000" }, "message": "Revert CC related changes.\n\nRevert: \"X86_64: Add allocation entrypoint switching for CC is_marking\"\nRevert: \"Fix mips build in InitEntryPoints\"\nRevert: \"Fix mac build in ResetQuickAllocEntryPoints\"\n\nTest: test-art-target-run-test\nChange-Id: If38d44edf8c5def5c4d8c9419e4af0cd8d3be724\n" }, { "commit": "ab191538a1d9eee6ec96bc3fa86dde36a007a6f5", "tree": "64c4ffaf96a8cf0aeb27bf8cbbbd5cfa42d1ff40", "parents": [ "8fb28dcf0e83f7153e76e176671cd4ad1f20205b", "f5de23265360e15fcfceb7d07bdadca0e5bb5f0a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 29 21:32:11 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Nov 29 21:32:12 2016 +0000" }, "message": "Merge \"X86_64: Add allocation entrypoint switching for CC is_marking\"" }, { "commit": "e424c93c767dae47aa9dc02c99ee0aa63b2c072b", "tree": "c8ef0d76ab0b093476a25254bb83c6d1c730abbe", "parents": [ "71601907b95a3527fc554a3a1724b3c12b872a7a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 23 12:52:01 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 23 12:52:01 2016 +0000" }, "message": "Update expectations in SetVerifierDeps/SetStackSample.\n\nSetStackSample might update an existing sample. So instead\ncheck that SetStackSample is only called in non-AOT mode, and\nSetVerifierDeps is only called in AOT mode.\n\nTest: 099-vmdebug\nChange-Id: I80dc3d61da651fb311364418802b0bb7e1ce9351\n" }, { "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": "f5de23265360e15fcfceb7d07bdadca0e5bb5f0a", "tree": "1bf893445a4c6fb35c430a68d79f66fa02fe4d2b", "parents": [ "e89667815b9d0a1eacb91678fed2a7518bb07cc2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 15 17:56:15 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 16 23:26:16 2016 +0000" }, "message": "X86_64: Add allocation entrypoint switching for CC is_marking\n\nOnly X86_64 done so far. Use normal TLAB allocators if GC is not\nmarking.\n\nAllocation speed goes up by ~8% based on perf sampling.\n\nWithout change:\n1.19%: art_quick_alloc_object_region_tlab\n\nWith change:\n0.63%: art_quick_alloc_object_tlab\n0.47%: art_quick_alloc_object_region_tlab\n\nBug: 31018974\nBug: 12687968\n\nTest: test-art-host-run-test\n\nChange-Id: I4c4d9eb229d4ad2f41b856ba5c2958a5eb3b7ffa\n" }, { "commit": "39ce7d2907edb978f1ab6cd75884be48ab11cd77", "tree": "7d2f0c56bd8a7dd483b5c1ede7047b623d0e35d7", "parents": [ "5302bd999afee6db5b0cfd63e6a49b0fcbf92dd4", "3049324f4ef71b5d7a3de49bd77c75f07dbf8f3a" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Nov 11 18:44:44 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Nov 11 18:44:44 2016 +0000" }, "message": "Merge \"Make empty checkpoint work while weak ref access is disabled.\"" }, { "commit": "fe814e89965ddf9a8b603863bd28259f8dd7be35", "tree": "32ecc282f176d0e2a98f41f6e4b116d26cf58bee", "parents": [ "977919fbcbfbf0c1c3fa97e51490bc678da646d2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 09 14:32:49 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Nov 10 14:41:28 2016 -0800" }, "message": "Use entrypoint switching to reduce code size of GcRoot read barrier\n\nSet the read barrier mark register entrypoints to null when the GC\nis not marking. The compiler uses this to avoid needing to load the\nis_gc_marking boolean.\n\nCode size results on ritzperf CC:\narm32: 13439400 -\u003e 13242792 (-1.5%)\narm64: 16380544 -\u003e 16208512 (-1.05%)\n\nImplemented for arm32 and arm64. TODO: Consider implementing on x86.\n\nBug: 32638713\nBug: 29516974\n\nTest: test-art-host + run ritzperf\nChange-Id: I527ca5dc4cd43950ba43b872d0ac81e1eb5791eb\n" }, { "commit": "3049324f4ef71b5d7a3de49bd77c75f07dbf8f3a", "tree": "19e2d9d5e8476bf526dd5924ef05b1d727b75f8b", "parents": [ "e7b46e22c7f4f6f503501b3b2ad99113289d142b" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Nov 03 13:06:52 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Nov 09 18:14:08 2016 -0800" }, "message": "Make empty checkpoint work while weak ref access is disabled.\n\nFix a potential race on PushOntoMarkStack for CC by running an empty\ncheckpoint (while weak ref access is disabled).\n\nBug: 32508093\nBug: 12687968\nTest: test-art-host with CC/CMS, libartd boot with N9, Ritz EAAC.\nChange-Id: I3749bb525e7734804307ee16262355f3fc730312\n" }, { "commit": "08883debd927d18c9ecf66683a2a11aa98165656", "tree": "a73e1025905c5d5a4a64f1f20fcc4fc5295c5f5e", "parents": [ "8bab69aafb3039f1dfe4aad4faf2a99986018abb" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 08 13:20:52 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 08 13:20:52 2016 -0800" }, "message": "ART: Refactor jfieldID handling\n\nArtField objects have been native since Marshmallow.\nRemove the dependency on being runnable. Refactor the\ncode into the jni_internal header.\n\nTest: m test-art-host\nChange-Id: I46708c70f9b4b566d7e26e4c5ffc3f0cbadc43fa\n" }, { "commit": "ea47ff8d26c831f1380f5f9f5c7fe958e40d63a5", "tree": "4a93f9b6181cabbec0f19e222449f3ec3b6faca0", "parents": [ "1eeb00c7499c520159a38af4be3835aee7ee7b38" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Nov 03 08:20:17 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Nov 03 11:16:36 2016 -0700" }, "message": "ART: Add \"explicit\" to constructors\n\nAdd explicit to two constructors with single argument.\n\nBug: 32619234\nTest: m\nChange-Id: I6ee6c8264d76017247e2236170c776c801719a0a\n" }, { "commit": "e9f67125f9a9c54b226c7daa95ebd0c5b31e8510", "tree": "fd87b64461854249721ad08a76f95ee3aef0b8e6", "parents": [ "19dfeb0e725bca7fd85b4a8a96ce9f3d30408aa3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 27 19:21:23 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 28 15:08:37 2016 -0700" }, "message": "ART: Add synchronous checkpoint\n\nAdd a thread function that will run the given checkpoint function\non the thread and wait for its completion.\n\nBug: 31684812\nTest: m test-art-host\nChange-Id: Icbb9f3a6b0bbf31e62be0508714ee172456bea29\n" }, { "commit": "0a85576a71adb9af631c9b9f5e48f21888313c14", "tree": "31c7db15c43096ca7118b528f17aa73eef92ce63", "parents": [ "54f07b502f7ac216e00beccee67781a3b72ae00f" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 26 13:43:14 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 28 10:15:42 2016 -0700" }, "message": "ART: Clean up Thread\n\nMake some functions private. Move some test-only functionality to\nthe test using it.\n\nTest: m test-art-host\nChange-Id: Ic84c8bcb150f991c6fc264c2d490363a3bd3e1f4\n" }, { "commit": "af1e2990cd1406a0fb7cba1d2e208208e950e413", "tree": "07e80e4dcc31931003a87be7884134ea42ceec07", "parents": [ "3f699ae23266b2c4adc98958dcbd80c71c2c3284" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Oct 12 17:44:50 2016 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Oct 19 11:38:54 2016 -0700" }, "message": "jni: Support @FastNative methods that return objects\n\nBug: 32088975\nChange-Id: I16f8b7ec6b251812af60ab25f2153d9b72f37044\n" }, { "commit": "ae7ff92c430aa12484ff8258ee4ed13421ac7934", "tree": "32774f5b3b0f96b921145a4af62dce182882fb7c", "parents": [ "6e5fa09510c7280168e040382d27dd8b55760d9a" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Oct 06 14:59:19 2016 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Oct 13 17:26:37 2016 +0000" }, "message": "jni: Add read barrier fast path to jni compiler\n\nStatic method dispatch via JNI requires a read barrier\nfor the ArtMethod::GetDeclaringClass() load before adding it to the\nJNI StackHandleScope.\n\nWe used to call ReadBarrierJni unconditionally but add a branch\nto skip calling it if the GC is not currently in the marking phase.\n\nTest: ART_USE_READ_BARRIER\u003dtrue make test-art-host test-art-target\nBug: 30437917\nChange-Id: I4f505ebde17c0a67209c7bb51b3f39e37a06373a\n" }, { "commit": "e8a3c576301fd531d5f73a65fc8b84a63619d580", "tree": "a00e488f153505ee890c28873308d373e6ab231b", "parents": [ "e8317d90c61dde07c12e404a2bc1fabf584905c1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 11 16:52:17 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 12 13:30:54 2016 -0700" }, "message": "Replace StackHandleScopeCollection with VariableSizedHandleScope\n\nVariableSizedHandleScope\u0027s internal handle scopes are not pushed\ndirectly on the thread. This means that it is safe to intermix with\nother types of handle scopes.\n\nAdded test.\n\nTest: clean-oat-host \u0026\u0026 test-art-host\n\nChange-Id: Id2fd1155788428f394d49615d337d9134824c8f0\n" }, { "commit": "c4f3925490a73da8dc74884a1deb965d4ecaf14e", "tree": "7fe566827f8ab903af1acb5697c86a6efe3187b3", "parents": [ "18f7de841fee3ca6f0c04e7caa57c3ce76b36231" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 18:32:08 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 06 13:55:58 2016 -0700" }, "message": "Move remaining jobject related functions to use ObjPtr\n\nAlso added ObjPtr::DownCast.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I59c253211dc435579ffdfd49f856861ab13d262c\n" }, { "commit": "81c6c8e6ded6d19c6196205a1b19fafa77993850", "tree": "c1e7fb8317e2075c437b65dce5323956a3d3a0d6", "parents": [ "765dee21f34e8c191c247506cf8bc73a3c3fed36", "02e7f1a46d8dbb277d045182cd1fa4b058d55162" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Oct 06 02:57:57 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 06 02:57:58 2016 +0000" }, "message": "Merge \"Fix a deadlock between thread flip and suspend request.\"" }, { "commit": "02e7f1a46d8dbb277d045182cd1fa4b058d55162", "tree": "bd14cec6d5c4a545c2b9081d72b6e1b49ec7ef1e", "parents": [ "d1224dce59eb0019507e41da5e10f12dda66bee4" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Oct 03 15:32:01 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Oct 05 17:52:29 2016 -0700" }, "message": "Fix a deadlock between thread flip and suspend request.\n\nSee 31683379#9 for the deadlock scenario.\n\nMake ModifySuspendCount(+1) retry if the thread flip function is set.\n\nBug: 31683379\nBug: 12687968\nTest: test-art, N9 libartd boot, Ritz EAAC with CC.\nTest: 129-GetThreadId with gcstress and CC.\nChange-Id: Id5cdfcd90a08a2ff497f9f0e2842fa4c613549bc\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": "a59d9b228b1eda3bf71a81b6201ec64e26086c23", "tree": "91adf9f0e282eec50ccb28aa2dbc432a5e28d13f", "parents": [ "05ce52b406dcf6b9a24287d14f0e611211e0db3d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 26 18:13:17 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 16:10:35 2016 -0700" }, "message": "Use ObjPtr for reflection.cc/h/inl\n\nChanged Pretty helpers to use this to reduce usage of Decode. The\neventual goal is not have almost any calls to ObjPtr::Decode.\n\nMoved ObjPtr out of mirror namespace for convenience. Added more\nPoisonObjectPointers calls in class linker, thread suspension.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I44d08db5143d95ed1b65e2f00f9749ef5cf379f7\n" }, { "commit": "3f7f03ce9a102a23961753753b5aa500226b0581", "tree": "9ff7596cab02c78fc78c95129d12f836cb262531", "parents": [ "0cfe19af3b7395658210ea6044a65c9811962a7a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 26 11:39:52 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 26 16:30:40 2016 -0700" }, "message": "Add basic support for object pointer poisoning\n\nObjPtr is a pointer that checks for heap corruption and is meant\nto replace mirror::Object* in places where a mirror::Object* is a\nlocal variable. Whenever there is a possible suspend point, the\ncurrent thread\u0027s object pointers are all invalidated. This is done\nby storing a cookie in the object pointer associated with what thread\ncreated it.\n\nAdded test case in object_test.\n\nExample failure:\nobject_test F 25379 25379 object_pointer.h:70] Check failed:\nIsValid() Invalid cookie, expected 0 but got 2\n\nBug: 31113334\n\nTest: test-art-host-gtest-object_test\n\nChange-Id: I9fa80ccaf2f0448621942935af702a243a3e1ee6\n" }, { "commit": "26ead4975e1752e8ae2f5ed6fda73876c4f9ff59", "tree": "2ce1b047f54bfb4287483de8eaf79cd12f6058b6", "parents": [ "79f6d706185714dccf80aca20d9f3261337473ae", "268764da8022cafa5661c5b514eaa343c5257e57" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 14 16:55:56 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Sep 14 16:55:56 2016 +0000" }, "message": "Merge \"Make ScopedAssertNoThreadSuspension no overhead for non-debug\"" }, { "commit": "05846475c8d48ce191dcd333c76d5ccc17aea9dd", "tree": "bfbcbba324eda45fae27b31ce0ce589c5f472474", "parents": [ "e305eaf2c08a203e3e046c3a8608b90b9dc5a080" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 14 12:49:57 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 14 12:55:09 2016 +0100" }, "message": "Remove ThrowNoSuchMethod entrypoint.\n\nRun ART test suite on host and Nexus 6.\nBug: 31464666\nChange-Id: I5aa737726031adae0b132f759cf802a93d581a7f\n" }, { "commit": "268764da8022cafa5661c5b514eaa343c5257e57", "tree": "7c4b46847a6dcfac7a1492edd79b434fb1a0d6ea", "parents": [ "f9decf915f6fa5b33fceba7c10c119e4cad481e5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 13 12:09:38 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 13 15:18:29 2016 -0700" }, "message": "Make ScopedAssertNoThreadSuspension no overhead for non-debug\n\nPreviously it required Thread::Current() which may not be free.\nThe plan is to add a lot more ScopedAssertNoThreadSuspension in\nthe codebase.\n\nAlso cleaned up callers.\n\nBug: 31458474\nChange-Id: I5a1621a5435476504d22266cc01a9bf26aab7568\n" }, { "commit": "bdf7f1c3ab65ccb70f62db5ab31dba060632d458", "tree": "25cc77adfeb05232d0ab00aa561a693f1d71745c", "parents": [ "d7eabc2cc1a88c1f7f927da61246ae65aab0626c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 30 16:38:47 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 30 17:02:53 2016 -0700" }, "message": "ART: SHARED_REQUIRES to REQUIRES_SHARED\n\nThis coincides with the actual attribute name and upstream usage.\nPreparation for deferring to libbase.\n\nTest: m\nTest: m test-art-host\nChange-Id: Ia8986b5dfd926ba772bf00b0a35eaf83596d8518\n" }, { "commit": "ee23582af60b36f982de2ad16f485a61f35ae817", "tree": "ae35b6885b345e210c66acdd992103e345202131", "parents": [ "bf042503dc2c2d328a6331baf33697a7d06acccf" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Aug 19 17:03:27 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Aug 22 11:33:58 2016 -0700" }, "message": "Revert \"Revert \"Improve the thread flip.\"\"\n\nThis reverts commit db3204f87c3f7c4de89762ce9e8502a9dc25c2d8.\n\nImprove the thread flip.\n\n- In addition to the threads that are suspended in FullSuspendCheck(),\n prioritize the resume of threads that are blocking for the thread\n flip at the JNI critical section entry and threads are about to\n transition to runnable (eg. blocking at the SOA entry from JNI).\n- Shorten the length of the thread flip critical\n section (ThreadFlipBegin/End).\n- Add some systrace scopes.\n- Add a read barrier for the locked objects during the thread dump in\n case the thread is in the middle of flipping.\n\nBug: 30980189\nBug: 29517059\nBug: 12687968\nTest: test-art-host, Ritz EAAC, N9 libartd boot\nChange-Id: I3a903c47c0fcc746664ec376cc31dee8af3c3ecb\n" }, { "commit": "db3204f87c3f7c4de89762ce9e8502a9dc25c2d8", "tree": "abb3aa17279c6a9edc9dd1c0691738a7f7c69a10", "parents": [ "e61e6fdd404b8f04bd33fc3ea49e38e92e31ad69" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Aug 19 17:56:11 2016 +0000" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Aug 19 17:56:11 2016 +0000" }, "message": "Revert \"Improve the thread flip.\"\n\nThis reverts commit e61e6fdd404b8f04bd33fc3ea49e38e92e31ad69.\n\nThreadStress failing.\n\nBug: 29517059\nBug: 12687968\nChange-Id: I0ee8ef04f77c5f9378e4bd21c5dedb4e435a2dc6\n" }, { "commit": "e61e6fdd404b8f04bd33fc3ea49e38e92e31ad69", "tree": "a7d97c2a67dc1f8337f002a6748f50c9d32e303e", "parents": [ "d16ae7fe70d74091778e5952b7920df14866287f" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Aug 02 16:03:06 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Aug 17 20:24:46 2016 -0700" }, "message": "Improve the thread flip.\n\n- In addition to the threads that are suspended in FullSuspendCheck(),\n prioritize the resume of threads that are blocking for the thread\n flip at the JNI critical section entry and threads are about to\n transition to runnable (eg. blocking at the SOA entry from JNI).\n- Shorten the length of the thread flip critical\n section (ThreadFlipBegin/End).\n- Add some systrace scopes.\n\nBug: 29517059\nBug: 12687968\nTest: test-art-host, Ritz EAAC, N9 libartd boot.\nChange-Id: Idecec9936ae432c23d5f83321ba13339852018df\n" }, { "commit": "542451cc546779f5c67840e105c51205a1b0a8fd", "tree": "11e09bb5abaee12dddffefbe7e425291076dfa7a", "parents": [ "85c4a4b8c9eabfe16e4e49f9b4aa78c1bf4be023" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 26 09:02:02 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 01 18:54:48 2016 -0700" }, "message": "ART: Convert pointer size to enum\n\nMove away from size_t to dedicated enum (class).\n\nBug: 30373134\nBug: 30419309\nTest: m test-art-host\nChange-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269\n" }, { "commit": "10b218d32c0006aab747a53a9867d982cde9c938", "tree": "03b668bd44f89da024531145043c04f670363528", "parents": [ "e92730cbc5e05b8057cf103af876927c9c361b73" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jul 25 17:48:52 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jul 26 10:21:34 2016 -0700" }, "message": "Add missing filter cards to zygote mod union table\n\nWithout filtering the cards, every object in the zygote is grayed\nduring the GC. This was deleted in a PS of previous CL.\n\nGrayAllDirtyImmuneObjects goes from 1.974ms to 376.464us for CC on\nN9 maps. This happens during the pause. This CL also fixes regression\nin zygote PSS.\n\nBug: 29516968\nBug: 12687968\n\nChange-Id: I42014e78b1de3ce9b2eefa3bd32f0d24e2ff71c6\n" }, { "commit": "02b75806a80f8b75c3d6ba2ff97c995117630f36", "tree": "ecdb1852c3e33f120110091cc2d07a9737fbd3b5", "parents": [ "5f485719b166ceb8e591329d40e76c5e50988022" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Jul 13 11:54:35 2016 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Jul 13 11:54:35 2016 +0100" }, "message": "Introduce more compact ReadBarrierMark slow-paths.\n\nReplace entry point ReadBarrierMark with 32\nReadBarrierMarkRegX entry points, using register\nnumber X as input and output (instead of the standard\nruntime calling convention) to save two moves in Baker\u0027s\nread barrier mark slow-path code.\n\nTest: ART host and target (ARM, ARM64) tests.\nBug: 29506760\nBug: 12687968\nChange-Id: I73cfb82831cf040b8b018e984163c865cc44ed87\n" }, { "commit": "e8e1127da3f154fae8d2eb16a94203544a182159", "tree": "930aa5bd3f4e81dcf1b27043844da55a3dcb5cc4", "parents": [ "739dc72773c5dee583a1d322f91b5abd61f9889d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jun 28 18:08:46 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 12:31:52 2016 +0100" }, "message": "Do checks on the fault address when we think it\u0027s an NPE.\n\nbug:29321958\nChange-Id: I28f4da56eb3e0b48721d3ac41114858bc80daadb\n" }, { "commit": "87f3fcbd0db352157fc59148e94647ef21b73bce", "tree": "5bdeabb246f5de86704333b3fcbccc6e9146d246", "parents": [ "b94b5706f0b8e2e1c7e1db22274f9f4bae0c4b5a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Apr 28 15:52:11 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jun 21 15:17:38 2016 +0100" }, "message": "Replace String.charAt() with HIR.\n\nReplace String.charAt() with HArrayLength, HBoundsCheck and\nHArrayGet. This allows GVN on the HArrayLength and BCE on\nthe HBoundsCheck as well as using the infrastructure for\nHArrayGet, i.e. better handling of constant indexes than\nthe old intrinsic and using the HArm64IntermediateAddress.\n\nBug: 28330359\nChange-Id: I32bf1da7eeafe82537a60416abf6ac412baa80dc\n" }, { "commit": "952e1e3710158982941fc70326e9fddc3021235d", "tree": "79ba4922cbe7a28d32078516d1667f90354bee6b", "parents": [ "ece5386eeb30aa6be663af2416b68a7ebf9bd42e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 13 14:04:02 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 13 18:10:02 2016 -0700" }, "message": "Support unlimited pending checkpoints\n\nPrevents the spinning that used to happen if RunCheckpoint was called\nwith 3 pending checkpoints. This spinning was done when holding\nthread_list_lock_ and thread_suspend_count_lock_ and could deadlock\nif any of the pending checkpoints required any of these locks.\n\nThe fix is to use an overflow list instead of having a fixed limit of\n3.\n\nChanged suspend stress test to have more threads and only compare last\nline since there may be libbacktrace spam like:\n\"+E/libbacktrace(69891): void SignalHandler(int, siginfo_t *, void *):\nTimed out waiting for unwind thread to indicate it completed.\"\n\nBug: 28988206\n\nChange-Id: I2ae611506147d5199d59a08eee0395f7fa35d448\n" }, { "commit": "ea8e02bef0f26a2212247210cc72b168a8157bad", "tree": "248d58fa61a647ef724c13e3ec4a7a49f8d26dc4", "parents": [ "347eff2699557340c493a026300a73b84a796cd3", "f711f2cf3c28dfe865e36f17419a16f06a0ebb5a" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu May 26 22:49:55 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu May 26 22:49:55 2016 +0000" }, "message": "Merge \"Partial fragment deoptimization\"" }, { "commit": "f711f2cf3c28dfe865e36f17419a16f06a0ebb5a", "tree": "3657d68b2a2fb67f4919cb97ca279cf9310224e9", "parents": [ "bbfa42aa4c4b5cbc2ed2d8bdea6ff76da615a8aa" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon May 23 12:29:39 2016 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu May 26 13:24:54 2016 -0700" }, "message": "Partial fragment deoptimization\n\nWe used to do either single frame deoptimization, or full fragment\ndeoptimization which deoptimizes all the frames in a fragment.\nThis change allows some methods to be not deoptimizeable, likely due\nto some kind of optimization. So we need another deoptimization mode\nthat unwinds partial fragment. Deoptimizations are now generalized into\neither full or partial fragment. A full fragment deoptimization will\ndeopt all frames in the fragment, and then returns from the invoke stub\nto enter interpreter. A partial fragment deoptimization will deopt a\nsingle frame, or all frames up to the method that\u0027s not deoptimizeable,\nand then jumps to the interpreter bridge.\n\nCurrently code not deoptimizeable is the code in boot image since the\ncode may not be compiled with debuggable flag.\n\nBug: 28769520\nChange-Id: I875c694791cc8ebd5121abcd92ce7b0db95aca38\n" }, { "commit": "98810e34804e224e5ab302cd353393f72a45202b", "tree": "696693111fa137965fe03efc29a20b687cfb3b66", "parents": [ "783d02e527107e91880f12bddf41afd313919132" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue May 24 14:55:40 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu May 26 11:45:13 2016 -0700" }, "message": "Remove SetStateUnsafe in Debugger::Disconnected.\n\nThe debugger thread has the exclusive mutator lock and doen\u0027t need to\nswitch to runnable while updating instrumentation.\n\nThis avoids a spurious failure of the DCHECK in the jdwp test.\n\nBug: 25262836\n\nChange-Id: Ibfd0bc3936a877fd264396fcc4cc1af8d19a1645\n" }, { "commit": "97cbc9206e9adc473a90650ebdb5d620f517ff04", "tree": "bef15a43a1c3f3dc2639f156aeea40d95763258e", "parents": [ "85e47976a483844177eb486d6e501fa070fbe6e2" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Apr 15 16:16:35 2016 +0100" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Apr 28 16:12:33 2016 +0100" }, "message": "Allow the framework to register sensistive threads to the runtime\n\nBug: 27865109\nBug: 28065407\n\n(cherry picked from commit 8fff24953c78bd58f3a42ac83b340b90e7e7228a)\n\nChange-Id: I82094e46a2fd12617e091d98831193f9ff56d26b\n" }, { "commit": "e42888f9df4163303244070c65d5229d3e201742", "tree": "c3c3e380ed16676186040f1da6042fc2cd58e041", "parents": [ "7f98c9a6babe3a21d84ce1f1e1273c99975a47f5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 14 10:49:19 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 27 12:45:30 2016 -0700" }, "message": "Write conflict tables in image\n\nAdd image sections for runtime methods and conflict tables. This\nmakes it that we do not need to fake up a length prefixed array\nfor runtime methods.\n\nReduces .art private dirty and PSS.\n\nSystem wide .art PSS goes from 32.5MB to 30.5MB after system boot.\n\nBusiness card .art private dirty goes from 588K to 504K.\n\nIncreases image size by ~100K.\n\nBug: 27906566\n\n(cherry picked from commit cda9386add68d94697449c6cb08b356747e55c21)\n\nChange-Id: I38cbe3150c3eeb385b8cad7cf93614e3980f4162\n" }, { "commit": "61b3cd45c5e3a5c24f7ccce94843a718375f9b7a", "tree": "fc71e27e0ddf6be1f672b69524b7ee978fd9d29c", "parents": [ "74b3c8f3850c2e4a7b17e9821ad78b59e6f8e047" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 18 11:43:29 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 18 15:29:53 2016 -0700" }, "message": "Prevent holding stale Thread pointers\n\nIt is only really safe to hold non-self Thread* if you hold the\nthread list lock. Changed a few places to use thread ids instead\nof Thread.\n\nBug: 28223501\n\n(cherry picked from commit 81c170fede9af9174aba71428334ac8f366a4b4f)\n\nChange-Id: I15e50b699303a5c3739e4d19c153dd306e2ee504\n" }, { "commit": "b2771b41a956b50266d4d83fbb067f99faf7b7dc", "tree": "797b73b8b97c496938a9f6bc498d75911b1ff3c9", "parents": [ "f9eed16e175f8c7cb37d878acc772d653b07101e" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Apr 07 17:09:25 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Apr 13 12:42:29 2016 +0100" }, "message": "Add option to tune sample collection based on thread sensitivity\n\nBug: 28065407\nBug: 27865109\n\nChange-Id: Icdb89f8f8874a41c07e73185523d18e8956620d3\n" }, { "commit": "2c2d2a05eaf81d07df27418f8dfd68de6fa28ac1", "tree": "f7d49c6caf3e3ed1fa1d59699c00b4ede6921b2f", "parents": [ "db1f7dac02f6dcecac3e032f10abbcdbf3cf4331" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Mar 17 21:27:19 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Mar 18 10:32:25 2016 -0700" }, "message": "ART: Speed up stack guard page install\n\nOnly the main thread doesn\u0027t have its stack mapped in under normal\nconditions. Reading each page is a lot of overhead and we should\ntry to avoid it.\n\nRewrite to first try a (non-fatal) protect. If the outcome is a\nsuccess, finish. Otherwise do the stack mapping, and try again.\n\nBug: 27718174\nChange-Id: I16b214567585ed2f09970f618ccdec7eed219fd3\n" }, { "commit": "7e1ce285c3c2cbd8d97c744699c2f2347fa7487b", "tree": "40d525eeda56a05576cb396a72479c366d2d463f", "parents": [ "4e4e64511e530db37b33f450016afe49db3c4b20" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Dec 11 15:46:19 2015 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Feb 25 14:00:48 2016 -0800" }, "message": "Assembly TLAB allocation fast path for arm.\n\nSpeedup (GSS GC with TLAB on N5):\n BinaryTrees: 1872 -\u003e 796 ms (-57%)\n MemAllocTest: 2522 -\u003e 2219 ms (-12%)\n\nBug: 9986565\nChange-Id: Icb9d1259461f3abe83a4a82c8aff911937eaf57d\n" }, { "commit": "20a0be0299183b54be9643820294e011d3358823", "tree": "b5f13c1362dc146675792d438ec7528a39c286a4", "parents": [ "eeb3b0bf5339e24b6f5e22c5a884910000c9bcfb" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Feb 19 15:44:06 2016 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Feb 22 11:41:18 2016 -0800" }, "message": "Fix CC collector thread flip and JNI critical section deadlocks.\n\nThere\u0027s a bug in the synchronization between the CC collector\u0027s thread\nflip and JNI critical sections where it incorrectly attempts to make\nthe synchronization scheme to be writer (GC) preference to avoid\nstarvation of GC in the presence of frequent JNI critical section\nenter/exit. This could cause a deadlock between them if a thread\nenters a nested JNI critical section after a thread flip occurs. This\nis reproduced in the added test.\n\nThe fix is to use a thread local disable counter in addition to the\nglobal counter to detect a nested enter by the same thread and avoid\nwaiting if nested.\n\nBug: 19235243\nBug: 12687968\nChange-Id: Idf7720a6906c9ea508219935af3727f76680d2d8\n" }, { "commit": "a73280df8cac1279b6dea0424722f42ef0048613", "tree": "22adc28cc88bc440b8e28a13a389c3a3c2b22341", "parents": [ "47a2a45a6673ddf3322115ff5058763f82a9368f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 15 13:05:16 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 15 16:24:56 2016 +0000" }, "message": "Add an option to disable native stack dumping on SIGQUIT.\n\nSome of our (stress) run-tests do ANR dumping, which end up\nstressing libunwind, that has known problems. To avoid getting\nflakes due to libunwind, disable native stack dumping on SIGQUIT\nfor our run-tests.\n\nbug:27185632\nbug:24664297\nChange-Id: I69085e48db903d6240448d71666ae2dcd091922e\n" }, { "commit": "e6141d7aba1078c0465294408133aaf00ae47f1b", "tree": "159b05407974c46ac39f0be10912f16bc910aeff", "parents": [ "f3c35954350ab3386cff538a123bc1b8e78e89f0", "7ed9c561048d79083b6d0576c71a986a3123bca6" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Feb 05 22:35:25 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 05 22:35:25 2016 +0000" }, "message": "Merge \"Use 8-byte increment bracket sizes for rosalloc thread local runs.\"" }, { "commit": "7ed9c561048d79083b6d0576c71a986a3123bca6", "tree": "8e352f05dc5e82126dd61e0e8e6ddb2a1003e16a", "parents": [ "c3cf1d2e35a45bd2e2c60fbc9c2a1a6a56dfc529" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Feb 02 15:22:09 2016 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Feb 05 12:12:44 2016 -0800" }, "message": "Use 8-byte increment bracket sizes for rosalloc thread local runs.\n\nVery small space savings (\u003c 1%) after device boot and up to 10%\nallocation speedup.\n\nSome minor cleanup.\n\nBug: 9986565\n\nChange-Id: I51d791c4674d6944fe9a7ee78537ac3490c1a02c\n" }, { "commit": "b331febbab8e916680faba722cc84b66b84218a3", "tree": "35f985b021e476914bfe91492da23fee218014a7", "parents": [ "586996afc905518ed926e4680aab67bedabec9b7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 05 16:51:53 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 05 17:11:54 2016 +0000" }, "message": "Revert \"Revert \"Implement on-stack replacement for arm/arm64/x86/x86_64.\"\"\n\nThis reverts commit bd89a5c556324062b7d841843b039392e84cfaf4.\n\nChange-Id: I08d190431520baa7fcec8fbdb444519f25ac8d44\n" }, { "commit": "dfe02f6aafee264478d510b9742ee266ea52e8a8", "tree": "d56729901410de2b5cea9395c4686b49c38b245d", "parents": [ "85fdcda926fbab0e5a7b3e8b3541f5d225b2bcdb" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 01 20:15:11 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 02 15:43:05 2016 -0800" }, "message": "Fix remaining read barrier issues in image relocation\n\nAdded a way to disallow read barriers, this makes it easy to find\nthe issues.\n\nBug: 26786304\nChange-Id: I7ebb50832686d03e096a979aae9741239371683f\n" }, { "commit": "1452bee8f06b9f76a333ddf4760e4beaa82f8099", "tree": "8ffb80498cd8fdff66a4ab272723cf4d4e69e57b", "parents": [ "7f9ea1451bb74055ee5d51ae04fc869faa06cbdd" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Fri Mar 06 14:43:04 2015 -0800" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Mon Jan 11 13:04:49 2016 -0800" }, "message": "Fast Art interpreter\n\nAdd a Dalvik-style fast interpreter to Art.\nThree primary deficiencies in the existing Art interpreter\nwill be addressed:\n\n1. Structural inefficiencies (primarily the bloated\n fetch/decode/execute overhead of the C++ interpreter\n implementation).\n2. Stack memory wastage. Each managed-language invoke\n adds a full copy of the interpreter\u0027s compiler-generated\n locals on the shared stack. We\u0027re at the mercy of\n the compiler now in how much memory is wasted here. An\n assembly based interpreter can manage memory usage more\n effectively.\n3. Shadow frame model, which not only spends twice the memory\n to store the Dalvik virtual registers, but causes vreg stores\n to happen twice.\n\nThis CL mostly deals with #1 (but does provide some stack memory\nsavings). Subsequent CLs will address the other issues.\n\nCurrent status:\n Passes all run-tests.\n Phone boots interpret-only.\n 2.5x faster than Clang-compiled Art goto interpreter on fetch/decode/execute\n microbenchmark, 5x faster than gcc-compiled goto interpreter.\n 1.6x faster than Clang goto on Caffeinemark overall\n 2.0x faster than Clang switch on Caffeinemark overall\n 68% of Dalvik interpreter performance on Caffeinemark (still much slower,\n primarily because of poor invoke performance and lack of execute-inline)\n Still nearly an order of magnitude slower than Dalvik on invokes\n (but slightly better than Art Clang goto interpreter.\n Importantly, saves ~200 bytes of stack memory per invoke (but still\n wastes ~400 relative to Dalvik).\n\nWhat\u0027s needed:\n Remove the (large quantity of) bring-up hackery in place.\n Integrate into the build mechanism. I\u0027m still using the old Dalvik manual\n build step to generate assembly code from the stub files.\n Remove the suspend check hack. For bring-up purposes, I\u0027m using an explicit\n suspend check (like the other Art interpreters). However, we should be\n doing a Dalvik style suspend check via the table base switch mechanism.\n This should be done during the alternative interpreter activation.\n General cleanup.\n Add CFI info.\n Update the new target bring-up README documentation.\n Add other targets.\n\nIn later CLs:\n Consolidate mterp handlers for expensive operations (such as new-instance) with\n the code used by the switch interpreter. No need to duplicate the code for\n heavyweight operations (but will need some refactoring to align).\n Tuning - some fast paths needs to be moved down to the assembly handlers,\n rather than being dealt with in the out-of-line code.\n JIT profiling. Currently, the fast interpreter is used only in the fast\n case - no instrumentation, no transactions and no access checks. We\n will want to implement fast + JIT-profiling as the alternate fast\n interpreter. All other cases can still fall back to the reference\n interpreter.\n Improve invoke performance. We\u0027re nearly an order of magnitude slower than\n Dalvik here. Some of that is unavoidable, but I suspect we can do\n better.\n Add support for our other targets.\n\nChange-Id: I43e25dc3d786fb87245705ac74a87274ad34fedc\n" }, { "commit": "7c1559a06041c9c299d5ab514d54b2102f204a84", "tree": "c50b54bf82f457f44dbf0741947d836749d4a96a", "parents": [ "7cd230c8f74a227ea04f1dd93c8d855aa81fc1fe" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Dec 15 10:55:36 2015 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Dec 15 11:45:56 2015 +0000" }, "message": "x86 Baker\u0027s read barrier fast path implementation.\n\nIntroduce an x86 fast path implementation in Optimizing for\nBaker\u0027s read barriers (for both heap reference loads and GC\nroot loads). The marking phase of the read barrier is\nperformed by a slow path, invoking a new runtime entry point\n(artReadBarrierMark).\n\nOther read barrier algorithms continue to use the original\nslow path based implementation, which has been renamed as\nGenerateReadBarrierSlow/GenerateReadBarrierForRootSlow.\n\nBug: 12687968\nChange-Id: Ie610c4befc19ff22378a8cba38b422dcacb54320\n" }, { "commit": "9f62af0e1cd770ec3e98f0d1a83a1193e2493cac", "tree": "4d47b48f4dcc5a40f41650a6e2417ea2a6252108", "parents": [ "3cf8583a2fe1a2793b4c67098df62f4362b47f0c", "6aa137071c53b5f4a04284be9d8cc51362ad3716" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Dec 07 23:20:18 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Dec 07 23:20:18 2015 +0000" }, "message": "Merge \"ART: Fix PACKED size of tls_ptr_size.\"" }, { "commit": "73be1e8f8609708f6624bb297c9628de44fd8b6f", "tree": "47a22745797a345dea8dd846aad00334da0d5bf1", "parents": [ "47229aa5848df7d45578dbdd9285f57dfa9399a8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Sep 17 15:22:56 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 04 12:08:02 2015 +0000" }, "message": "Inline monomorphic calls.\n\nChange-Id: If38171c2dc7d4a4378df5d050afc4fff4499c98f\n" }, { "commit": "04302dbb106d590ff72c0dfecda23d85b6565059", "tree": "2fe66e258de36bcebc913d48e04468b7c639578f", "parents": [ "0e06a0989f4fc53e0d281a5a2cb5b5a17feaea0c" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Nov 11 23:45:34 2015 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Nov 13 15:21:29 2015 -0800" }, "message": "Fix class unloading with the CC collector.\n\nAvoid unnecessarily decoding dex cache and class loader weak roots,\nwhich would trigger read barriers.\n\nRe-enable 141-class-unload with the CC collector.\n\nBug: 12687968\nBug: 24468364\nChange-Id: Ib4c19f25000873cab0e06047040442d135285745\n" }, { "commit": "6aa137071c53b5f4a04284be9d8cc51362ad3716", "tree": "784e9cd0a2d8e6635c1c0190b0856e0a534570d3", "parents": [ "28b48026f0686f93b31b9cad47671c18837acc56" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 28 10:57:25 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 28 10:57:58 2015 -0700" }, "message": "ART: Fix PACKED size of tls_ptr_size.\n\nThis should be PACKED to the current pointer size.\n\nChange-Id: I31e23cd7f1fe57655b71b901642617df96a48aa1\n" }, { "commit": "6cff48f05894a60aa4742472983a75504536ca15", "tree": "bd399d46ef08cd836010f3119c8c5db054561267", "parents": [ "823e693aa946ba75cd047429e1290011a2ed8729" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Sun Jan 26 21:36:13 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 23 11:16:11 2015 -0700" }, "message": "ART: Speed up thread stack tracing for ANRs\n\nShare a BacktraceMap for all threads being dumped, which speeds up\nANR dumping. Results from Nexus 9 (average of five):\n\nBefore: 0.587s\nAfter: 0.206s\n\nChange-Id: Ia70e0dbd39049318c02de561e7b95258d4849467\n" }, { "commit": "dc412b6f49a65774b7af654f65cbff619cb7d85a", "tree": "f8455c6deb4bb3c96d21e2135d55a1a658045e47", "parents": [ "114873103db3d4d6e0da42ca02bad1ea8826443b" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Oct 15 12:26:57 2015 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Oct 15 15:32:41 2015 -0700" }, "message": "Revert \"Revert \"Implement rosalloc fast path in assembly for 32 bit arm.\"\"\n\nWith a heap poisoning fix.\n\nThis reverts commit cf91c7d973f3b2f491abc61d47c141782c96d46e.\n\nBug: 9986565\nChange-Id: Ia72edbde65ef6119e1931a77cc4c595a0b80ce31\n" }, { "commit": "a2b2e7365d66553164247bafae214f157d480277", "tree": "11fc65a360fb2ff3b4388be0558a39b09eca330a", "parents": [ "d2e0dd179fd2e8c6c820194e187fc5e9164154ff", "cf91c7d973f3b2f491abc61d47c141782c96d46e" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Oct 15 07:30:00 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 15 07:30:00 2015 +0000" }, "message": "Merge \"Revert \"Implement rosalloc fast path in assembly for 32 bit arm.\"\"" }, { "commit": "cf91c7d973f3b2f491abc61d47c141782c96d46e", "tree": "b94c289f567014ba56bcd4dec76066085819e90b", "parents": [ "e6316940db61faead36f9642cce137d41fc8f606" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Oct 15 07:29:38 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Oct 15 07:29:38 2015 +0000" }, "message": "Revert \"Implement rosalloc fast path in assembly for 32 bit arm.\"\n\nTentative. Looks like heap poisoning breaks with this change.\n\nbug: 9986565\n\nThis reverts commit e6316940db61faead36f9642cce137d41fc8f606.\n\nChange-Id: I5c63758221464fe319315f40ae79c656048faed0\n" }, { "commit": "2248d278460f18db9bcdc5a1bdb2dcdfdde2d301", "tree": "769455c0ea777310518cf4d8a4af9ecb45c968d0", "parents": [ "bf8fd42a6390d5694fe387b9bc876fc510645810", "e6316940db61faead36f9642cce137d41fc8f606" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Oct 14 18:18:24 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Oct 14 18:18:24 2015 +0000" }, "message": "Merge \"Implement rosalloc fast path in assembly for 32 bit arm.\"" }, { "commit": "e6316940db61faead36f9642cce137d41fc8f606", "tree": "cf0a97827af4679be4112d056855b5a673ee3a32", "parents": [ "8c812b71552a8a1b2bf06e430d355b12b7084807" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Oct 07 15:41:48 2015 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Oct 13 13:06:20 2015 -0700" }, "message": "Implement rosalloc fast path in assembly for 32 bit arm.\n\nMeasurements (N5, ms)\nBinaryTrees: 1702 -\u003e 987 (-42%)\nMemAllocTest: 2480 -\u003e 2270 (-8%)\n\nBug: 9986565\n\nChange-Id: I460af3626ad724078463d27cf74a94b7ff7468c5\n" }, { "commit": "3cfa4d05afa76e19ca99ec964b535a15c73683f0", "tree": "28c4950dff3700f69e3d32fc496890a722cfc379", "parents": [ "f4156986d1ba5b6dd7dfe89754c8562557cd1bc5" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 06 17:04:01 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 07 09:50:25 2015 -0700" }, "message": "ART: Remove interpreter entrypoints\n\nThese are no longer used as entrypoints. Make them proper runtime\nfunctions local to the interpreter.\n\nBump oat version.\n\nChange-Id: Icdd92e20eafc5668b68eeebf55cf624560f462b3\n" }, { "commit": "8ac9c91cfa2408cf8d27bcf12785a60923aab775", "tree": "fec6ebbdf2031aa648fa3699b3511048d64e0656", "parents": [ "25217af2a7cae96b32ba566aaf697288f3374c99" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 01 15:58:41 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 01 18:06:22 2015 -0700" }, "message": "Make SetStateUnsafe run checkpoints before going to suspended\n\nThis prevents cases where we can get a checkpoint request in runnable\nand then unsafe set to suspended. This was bad behavior since the\ncheckpoint may not be run in a timely manner.\n\nBug: 24191051\nChange-Id: I73605c1a1154f67699f3902141f3232ab064793f\n" }, { "commit": "639bdd13993644a267f177f8f5936496bda65e2b", "tree": "fc68e344355ab1c0cfef4d47dd8f2cbc9fc7e022", "parents": [ "b175a45be78152606a1db1eac0b79ef8777644f5" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 03 11:22:45 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 28 15:37:28 2015 -0700" }, "message": "ART: Single-frame deopt\n\nAdd deoptimization of a single frame. Works by removing the managed\ncode frame and jumping into the quick-to-interpreter bridge, and\nthe bridge understanding a stored ShadowFrame.\n\nWe need a separate fixup pass. For x86, we leave the return address\non the stack so we don\u0027t need to push it there.\n\nBug: 21611912\nChange-Id: I06625685ced8b054244f8685ab50b238a705b9d2\n" }, { "commit": "99170c636dfae4908b102347cfe9f92bad1881cc", "tree": "1ecff5dd69ea20b09e622cb88e7835a55dfa4be8", "parents": [ "5836f8465ddd0a7f5cbeec9c30d739bd0619867f" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Jul 06 11:10:37 2015 -0700" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Sep 17 17:07:22 2015 +0200" }, "message": "Deoptimization support in optimizing compiler for setting local values\n\nDue to compiler optimizations, we may not always be able to update\nthe value of a local variable in a compiled frame (like a variable\nseen as constant by the compiler). To avoid that situation, we simply\ndeoptimize compiled frames updated by the debugger so they are\nexecuted by the interpreter with the updated value.\n\nWhen the debugger attempts to set a local variable (actually a DEX\nregister or a pair of registers) in a compiled frame, we allocate a\nShadowFrame associated to that frame (using its frame id) and set the\nnew value in that ShadowFrame. When we know we are about to continue\nthe execution of the compiled frame, we deoptimize the stack using\nthe preallocated ShadowFrame (instead of creating a new one). We\ninitialize it with the current value of all DEX registers except\nthe ones that have been set by the debugger. Therefore, the\nShadowFrame represent the runtime context modified by the debugger.\n\nBumps oat version to force recompilation.\n\nBug: 19944235\nChange-Id: I0ebe6241264f7a3be0f14ee4516c1f7436e04da6\n" }, { "commit": "0747466fca310eedea5fc49e37d54f240a0b3c0f", "tree": "6d27922fa35330dd8d863ea56c82c5f04b711822", "parents": [ "9ee5d6cdc14ac94b64ea1961bf221bad48746929" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:12:33 2015 +0000" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Aug 26 16:54:32 2015 +0200" }, "message": "Revert \"Revert \"Fix deoptimization with pending exception\"\"\n\nThis reverts commit 6e2d5747d00697a25251d25dd33b953e54709507.\n\nFixes the deoptimization path from compiled code (generated by the\nOptimizing compiler) by adding wrapper artDeoptimizeFromCompiledCode.\nThis wrapper, called through the matching assembler stub\nart_quick_deoptimize_from_compiled_code, pushes the deoptimization\ncontext just before deoptimizing the stack.\n\nBug: 23371176\nBug: 19944235\nChange-Id: Ia7082656998aebdd0157438f7e6504c120e10d3e\n" }, { "commit": "aa50d3a01a4137cd89c152be5487fed23683d0ff", "tree": "a676638c9d43eb519214445173c1a66ba38355ac", "parents": [ "f0170ded84fbcd3d9152cdbcf9ffbf74c71b03ba" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:25:41 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:56:54 2015 +0200" }, "message": "Add missing field init in constructors\n\nAdds missing field initializations detected by Eclipse IDE.\n\nChange-Id: I5f2f32bcccb12545fc9f0b42fcec74a23dc81376\n" } ], "next": "00370827646cc21cb370c3e7e93f9c0cff4c30c2" }