)]}' { "log": [ { "commit": "896583ec8809497bb844c857dd042b0d2e7d1ba1", "tree": "1075075e9b9b9fdb8627d07eabf823776bdcf2e7", "parents": [ "cca7cb9ffa56d8ab8fd0c5997c8bfd965d7426c1" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jun 15 13:31:58 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 25 10:24:53 2018 -0700" }, "message": "ART: Use clang-tidy to warn on RAII issue\n\nRemove the macro hack to detect likely-incorrect usage of RAII wrappers.\nInstead make the clang-tidy pattern bugprone-unused-raii fatal.\n\nTest: mmma art\nChange-Id: I9d0eb1c5c3f469b2907111af9d38d947b36c4878\n" }, { "commit": "5c9715e0bafe58d28e24ad957db4f3adfe367bd4", "tree": "cdb1387ffcf2db9b75d9e66caff845d1ac09fec7", "parents": [ "faf742fe798038cd314ad06165017c9719cbce6e", "7a55ce0fed42310e6ff71e3fd648ea715c40aa83" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jun 20 15:18:14 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 20 15:18:14 2018 +0000" }, "message": "Merge \"ART: Ensure minimum stack size under ASAN\"" }, { "commit": "e383d23918db4eede30c3d78589d4639de3ec446", "tree": "a103f54ec49fac8d5cd15661f5c2d0cb5dab32e5", "parents": [ "dbf540399a2aaddad00b1233b80808c80dc0d443" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jun 19 12:29:51 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jun 19 17:09:45 2018 -0700" }, "message": "ART: Mask garbage-memory warnings\n\nAdd invariants or NOLINT to let clang-analyzer understand that\nthere\u0027s no garbage involved.\n\n(The analysis itself is too expensive to force it on for all\n local builds.)\n\nTest: mmma art\nChange-Id: I6148d743321248195abb3a8f646e63941f11f9a9\n" }, { "commit": "7a55ce0fed42310e6ff71e3fd648ea715c40aa83", "tree": "815de353045f47067630ba9e02911c5c03bdb3d8", "parents": [ "493bf736c698945ec88042500f24d93190ecfe74" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jun 19 10:07:18 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jun 19 10:31:53 2018 -0700" }, "message": "ART: Ensure minimum stack size under ASAN\n\nWhen running sanitized, ensure a minimum stack size of 2MB to\nensure that classes in the core library can be initialized in\ntests even under the access-checks interpreter.\n\nBug: 109813469\nTest: SANITIZE_HOST\u003daddress art/test/testrunner/testrunner.py -b --host --interp-ac -t 004-ThreadStress -t 054 -t 107 -t 203 -t 924 -t 1905 -t 1907 -t 1907 -t 1912 -t 1913 -t 1914 -t 1916 -t 1917 -t 1920 -t 1921 -t 1930 -t 1931 -t 1932 -t 1935 -t 1936\nChange-Id: I702d36552bbc753e286e619937370518aecf9692\n" }, { "commit": "e1402125e8363b49e176c6072893d1c110a05d2f", "tree": "44ec055be78cdcb8086a598cbf791f8c4627e157", "parents": [ "86decb6a3e3ebba8c3c67bfd25c12d9a85794f65" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jun 13 18:20:45 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jun 14 12:31:44 2018 +0100" }, "message": "Move some helper methods to DexRegisterLocation.\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: I0abab008159db023d531df69214cd3bb8c0639bd\n" }, { "commit": "74f2ccc9fad3cfd8e04d24aceba1c58ce08884dc", "tree": "ba0051646cb8e8192f690259475101b61101d5c3", "parents": [ "9fdd0125538fe63458aec68c4a40b025759bcf7c", "05e34f4cb8c9db165d1008721e874b9dd3db024b" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Jun 11 15:02:44 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jun 11 15:02:44 2018 +0000" }, "message": "Merge \"Revert^2 \"Remove support for Valgrind in ART.\"\"" }, { "commit": "05e34f4cb8c9db165d1008721e874b9dd3db024b", "tree": "c41d397ef5cfb9d7918fceadcda3195a0d0d91de", "parents": [ "428d80e3a49f732b39bb0c71cfde26451e3fdf4d" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu May 24 13:19:05 2018 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Jun 11 14:03:37 2018 +0100" }, "message": "Revert^2 \"Remove support for Valgrind in ART.\"\n\n- Disable test configuration art-gtest-valgrind64\n (art-gtest-valgrind32 was already disabled).\n- Remove Makefile logic regarding testing with Valgrind.\n- Remove occurrences of `TEST_DISABLED_FOR_MEMORY_TOOL_VALGRIND`.\n- Replace occurrences of `TEST_DISABLED_FOR_MEMORY_TOOL_ASAN` with\n `TEST_DISABLED_FOR_MEMORY_TOOL`.\n- Replace the potentially dynamically evaluated\n `RUNNING_ON_MEMORY_TOOL` expression with constant\n `kRunningOnMemoryTool`.\n- Simplify and fold the logic of\n `art::ArenaAllocatorMemoryToolCheckImpl` and\n `art::ArenaAllocatorMemoryToolCheck` into\n `art::ArenaAllocatorMemoryTool`.\n- Adjust comments regarding memory tools.\n- Remove Valgrind suppression files.\n- Remove `--callgrind` option from tools/art.\n\nThis reverts commit 8b362a87d52a6668ffd2283ef6ffc274315f41c8.\n\nChange-Id: I23c76845e6ccf766f19b22b58a0d5161f60842a9\nTest: art/test.py\nTest: art/test/testrunner/run_build_test_target.py art-asan\nBug: 77856586\nBug: 29282211\n" }, { "commit": "a4bae540b7c91136aaab02fd1638c8df6cb96970", "tree": "eb517cc7ba2a3b0d37b9be26e8b811d9f015d672", "parents": [ "bac28b0d5fb0765291adf8379bc502783cbb15b0", "fd89b0739d4000ea70f28bf53dea531027024f5a" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Jun 08 11:14:39 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jun 08 11:14:39 2018 +0000" }, "message": "Merge \"Remove unused GetDexRegisterMap arguments.\"" }, { "commit": "fd89b0739d4000ea70f28bf53dea531027024f5a", "tree": "1c9f3e1b046d60df405fb63f04cf83d3be4ff0f5", "parents": [ "6de8833fb64e59301eada4005ed04da995796170" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sun Jun 03 12:00:22 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jun 07 12:00:18 2018 +0100" }, "message": "Remove unused GetDexRegisterMap arguments.\n\nThey are no longer needed in the new encoding.\n\nI reuse the local variables in most places to DCHECK the size\nof the decoded register map. This has one catch though:\nWe sometimes omit all dex registers, so the DCHECK should be\ndone only after checking if the map is empty (if applicable).\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: I94b67029842374bc8eb7c9e5eac76fc93a651f24\n" }, { "commit": "bcf175247272d0e321c8d988c3c01c123b56e36e", "tree": "9f65ece5ce9474aa4fcf16fbfca6278109dc9a67", "parents": [ "09c5ca40635faee00f40f6ca0581dd475efd545e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 01 13:14:32 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 01 16:31:30 2018 +0100" }, "message": "ObjPtr\u003c\u003e-ify array allocations.\n\nAnd remove some unnecessary calls to ObjPtr\u003c\u003e::Ptr().\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: Ie313980f7f23b33b0ccea4fa8d5131d643c59080\n" }, { "commit": "a8bba7d0853b372aea3ed3ea154fb2b2a23c2c9d", "tree": "973a89870e0cc8912401503ebab1b709a9df5ba0", "parents": [ "65e463cf7ea91003ef030c8a819ae4596c9b820f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 30 15:18:48 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 31 09:48:59 2018 +0100" }, "message": "ObjPtr\u003c\u003e-ify ClassLinker::FindClass(), fix 1 stale reference use.\n\nThread::CreateAnnotatedStackTrace() was using a stale\nreference `aste_array_class`.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: I191907c0053456bb57de425aa6ccd9668df818a2\n" }, { "commit": "21d45b4f4ef3661ac33d622aaac5dd6d2ce7deb8", "tree": "121e046a949025bca4d766fdfdc5ea58e4ea4a10", "parents": [ "d02b23f7ee9664213216a82bfdcb0ee83824de04" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed May 30 06:35:05 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed May 30 17:59:24 2018 +0100" }, "message": "Simplify DexRegisterMap API.\n\nStore some of the needed decoding state explicitly to avoid passing it\naround all the time. The DexRegisterMap class is rewritten in next CL.\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: Ie268dff2a1c1da2e08f0e6799ae51c30e11f350b\n" }, { "commit": "08231f6cb3095a7dbde29299a7da5413a5f992e4", "tree": "d7cc4595bd633b3f5e7ade454d405a57468f3e77", "parents": [ "750cf88c8872983ae13259131cb2ed377e3eadff", "4b59d107f91601c4e0095d7a9db40970d4ed6956" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed May 30 16:21:15 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 30 16:21:15 2018 +0000" }, "message": "Merge \"Revert^2 \"Optimize register mask and stack mask in stack maps.\"\"" }, { "commit": "4b59d107f91601c4e0095d7a9db40970d4ed6956", "tree": "b120add4870b86cefc308ec3a0100851cfc1a4c0", "parents": [ "da4ff8bfcf03d6186b7d21c7bd8b90248f7b2f96" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue May 29 21:46:10 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed May 30 14:32:44 2018 +0100" }, "message": "Revert^2 \"Optimize register mask and stack mask in stack maps.\"\n\nThis reverts commit 8b20b5c1f5b454b2f8b8bff492c88724b5002600.\n\nReason for revert: Retry submit unmodified after fixing the test.\n\nUse BitTable to store the masks as well and move the\ndeduplication responsibility to the BitTable builders.\n\nDon\u0027t generate entries for masks which are all zeros.\nThis saves 0.2% of .oat file size on both Arm64 and Arm.\n\nEncode registers as (value+shift) due to tailing zeros.\nThis saves 1.0% of .oat file size on Arm64 and 0.2% on Arm.\n\nTest: test-art-target-gtest-exception_test\nTest: test-art-host-gtest-bit_table_test\nTest: test-art-host-gtest-stack_map_test\nChange-Id: Ib643776dbec3f051cc29cd13ff39e453fab5fae9\n" }, { "commit": "adbceb73b19cfad3ff9011390415a680382a21ec", "tree": "b7d4c12fb8cc8d4f9091d48d8e2c35a06b7a8e26", "parents": [ "aab8bc928a8eff425d5e5755fde75b1cb55998a9" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 29 14:34:14 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 29 17:48:45 2018 +0100" }, "message": "Remove GcRoot\u003c\u003e static from Throwable and related classes.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: I8115e6413a07419ec261af3ec5068833b72b5218\n" }, { "commit": "8b20b5c1f5b454b2f8b8bff492c88724b5002600", "tree": "51bea8ddfff23b1f6b0323eaeacf42e6f6199015", "parents": [ "ffaf87a429766ed80e6afee5bebea93db539620b" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 29 15:32:55 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 29 15:32:55 2018 +0000" }, "message": "Revert \"Optimize register mask and stack mask in stack maps.\"\n\nThis reverts commit ffaf87a429766ed80e6afee5bebea93db539620b.\n\nReason for revert: Breaks exception_test32 on target\nfor CMS and heap poisoning configs.\n\nChange-Id: I127c17f693e28211a799f73a50e73105edee7e4c\n" }, { "commit": "ffaf87a429766ed80e6afee5bebea93db539620b", "tree": "d79637f4b6a564facf4b837c3ff125bb3755594e", "parents": [ "5513c2b68a08109a5bfd811c7b2c8bbc66244e8e" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed May 23 14:44:39 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon May 28 08:45:37 2018 +0100" }, "message": "Optimize register mask and stack mask in stack maps.\n\nUse BitTable to store the masks as well and move the\ndeduplication responsibility to the BitTable builders.\n\nDon\u0027t generate entries for masks which are all zeros.\nThis saves 0.2% of .oat file size on both Arm64 and Arm.\n\nEncode registers as (value+shift) due to tailing zeros.\nThis saves 1.0% of .oat file size on Arm64 and 0.2% on Arm.\n\nTest: test-art-host-gtest\nChange-Id: I636b7edd49e10e8afc9f2aa385b5980f7ee0e1f1\n" }, { "commit": "b4eb1b19e1dd35d12a408358656c1421f507d231", "tree": "03ba65d206b5e6222dcca0f62a973bcbb1bf8a50", "parents": [ "ff7ff426e136aa8fd6e33d873f6259311982f6bb" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 24 11:09:38 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 25 11:32:31 2018 +0100" }, "message": "Refactor ClassRoot/GetClassRoot().\n\nMove it outside the ClassLinker, into its own header file,\nand add retrieval based on a mirror class template argument.\nKeep the SetClassRoot() as a private member of ClassLinker.\n\nMake the new GetClassRoot()s return ObjPtr\u003c\u003e.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: Icbc6b62b41f6ffd65b437297a21eadbb0454e2b7\n" }, { "commit": "aaebb1468cc95838aa58ed537d03de000c9cfe28", "tree": "80f325dab976dccd846e3bd5d1b86266be24e3dc", "parents": [ "79866c1f089672eca1fb7776c669c2cef3d6ad3e", "6ec2a1bf1cbecf17546df780dd0ad769042e1874" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 23 14:58:19 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 23 14:58:19 2018 +0000" }, "message": "Merge \"ObjPtr\u003c\u003e-ify UnstartedRuntime, fix 2 stale reference uses.\"" }, { "commit": "6ec2a1bf1cbecf17546df780dd0ad769042e1874", "tree": "09f7430f020e04dc892ff2c2152bb773cd45c0dd", "parents": [ "2d3065e6ca0bd707bc998b7d260bb8e8ec07cf87" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 22 15:33:48 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 23 13:48:31 2018 +0100" }, "message": "ObjPtr\u003c\u003e-ify UnstartedRuntime, fix 2 stale reference uses.\n\nTest: Rely on TreeHugger.\nBug: 31113334\nChange-Id: I35f76c3e3b94dfca18dbe67aba065a1270f4e5ee\n" }, { "commit": "197f1ef6f4f2411d011d7e8f696775c3fbcffd86", "tree": "eacc720024479e75b03d12ac6764f2cd2ba945b3", "parents": [ "04f0eaffa70e0384c07bb3680aa0724144358330", "8b362a87d52a6668ffd2283ef6ffc274315f41c8" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue May 22 23:00:40 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue May 22 23:00:40 2018 +0000" }, "message": "Merge \"Revert \"Remove support for Valgrind in ART.\"\"" }, { "commit": "8b362a87d52a6668ffd2283ef6ffc274315f41c8", "tree": "14b187cd0b4afde501b9cc7a9dc82c997362e888", "parents": [ "8268cb677bd92bfbcfec7e803775c29687494e53" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 22 20:54:14 2018 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 22 20:54:14 2018 +0000" }, "message": "Revert \"Remove support for Valgrind in ART.\"\n\nThis reverts commit 8268cb677bd92bfbcfec7e803775c29687494e53.\n\nReason for revert: ASAN failures\n\nChange-Id: I7e66d3f3fb461ae4f6dea6ec7d506b7dface3402\nTest: SANITIZE_HOST\u003daddress m test-art-host\nBug: 77856586\nBug: 29282211\n" }, { "commit": "f31343f1eab771010b3f293366149d2524f30902", "tree": "5187b2bceee4c7d5347750dc6ba0020172d5ef51", "parents": [ "e816389b61d17b9a82d8e61426ddfcce20e1238c", "8268cb677bd92bfbcfec7e803775c29687494e53" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue May 22 16:46:43 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue May 22 16:46:43 2018 +0000" }, "message": "Merge \"Remove support for Valgrind in ART.\"" }, { "commit": "052f8ca1776ed7deb4f036498edd69eb6a1b942f", "tree": "ae993f1a634b258e124df12663a1d24859917b05", "parents": [ "1c515f0b9d621869a2e67ce5e0f5532d0f2f323a" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Apr 26 15:42:54 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon May 21 20:28:41 2018 +0100" }, "message": "Rewrite stackmap encoding code.\n\nRemove most of the code related to handling of bit encodings.\nThe design is still same; the encodings are just more implicit.\nMost of the complexity is replaced with a single BitTable class,\nwhich is a generic purpose table of tightly bit-packed integers.\nIt has its own header which stores the bit-encoding of columns,\nand that removes the need to handle the encodings explicitly.\n\nOther classes, like StackMap, are accessors into the BitTable,\nwith named getter methods for the individual columns.\n\nThis CL saves ~1% of .oat file size (~4% of stackmap size).\n\nTest: test-art-host-gtest\nChange-Id: I7e92683753b0cc376300e3b23d892feac3670890\n" }, { "commit": "8268cb677bd92bfbcfec7e803775c29687494e53", "tree": "b63ca42dcd440117187e3c859b4b311b11b8137e", "parents": [ "004b8875b291af5001d384fb1eb76ae0660ff056" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri May 11 15:06:17 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon May 14 18:40:49 2018 +0100" }, "message": "Remove support for Valgrind in ART.\n\n- Disable test configuration art-gtest-valgrind64\n (art-gtest-valgrind32 was already disabled).\n- Remove Makefile logic regarding testing with Valgrind.\n- Remove occurrences of `TEST_DISABLED_FOR_MEMORY_TOOL_VALGRIND`.\n- Replace occurrences of `TEST_DISABLED_FOR_MEMORY_TOOL_ASAN` with\n `TEST_DISABLED_FOR_MEMORY_TOOL`.\n- Replace the potentially dynamically evaluated\n `RUNNING_ON_MEMORY_TOOL` expression with constant\n `kRunningOnMemoryTool`.\n- Simplify and fold the logic of\n `art::ArenaAllocatorMemoryToolCheckImpl` and\n `art::ArenaAllocatorMemoryToolCheck` into\n `art::ArenaAllocatorMemoryTool`.\n- Adjust comments regarding memory tools.\n- Remove Valgrind suppression files.\n- Remove `--callgrind` option from tools/art.\n\nTest: art/test.py\nBug: 77856586\nBug: 29282211\nChange-Id: Ifdcbfccc1830104c455760457df66ede4a4cd135\n" }, { "commit": "a3ad0cdd711857f04f477e2cdc5b56a2c74a3018", "tree": "7e0e4c7888c2497d86ce96c9574d7c9886721ccf", "parents": [ "abd9e1515bc6be88372c61071971c5432a51553d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 04 10:06:38 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 09 11:11:09 2018 +0100" }, "message": "ART: Move JNI files to runtime/jni/ .\n\nTest: Rely on TreeHugger.\nChange-Id: I9cae11191ef1567ae9453be498882a7767285140\n" }, { "commit": "7b0e844d8f0a59c6e1e9ac0c441409d2e435ac36", "tree": "e127fcf1840a8e249bfdb0725f6c73181fe84246", "parents": [ "6c3533991522d036cbb5a656c44f63bf633a2925" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Apr 11 18:27:47 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Apr 11 19:04:04 2018 +0100" }, "message": "Distinguish the various pre-allocated OutOfMemoryError use cases.\n\nThe pre-allocated exception \"OutOfMemoryError thrown while trying to\nthrow OutOfMemoryError; no stack trace available\", can be thrown in\nthree occasions:\n1. OOME thrown while throwing an exception (not necessarily an OOME);\n2. OOME thrown while throwing an OOME;\n3. OOME thrown while handling a stack overflow.\n\nTo improve error reports, replace the unique pre-allocated OOME\nexception with three different exceptions for these three use cases.\n\nNoe that we were already manually dumping a stack trace in case #2.\nThis change implements this for case #1 too.\n\nTest: art/test.py\nBug: 68251879\nBug: 77567088\nBug: 77844013\nChange-Id: I84cd15b729b4d0bd7bf6c831caeb0eb93c458114\n" }, { "commit": "75b47b47c4c54adccbb723cfbf48e32953887713", "tree": "876ce211773db3e3785254ae8d7e6b66545c76be", "parents": [ "0b1d3ca323bd504818e4cdab1baa732964ee150a", "6ee497188b0e8c7bcf5126cefad04090f956616a" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Mar 30 16:54:27 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Mar 30 16:54:27 2018 +0000" }, "message": "Merge \"Revert^4 \"Add an option to disable native stack dumping on SIGQUIT.\"\"" }, { "commit": "6ee497188b0e8c7bcf5126cefad04090f956616a", "tree": "db2d06b01d255811726ea748db8eb6f62cd7b71c", "parents": [ "642e9d8249be5aff68022cabdc8ba576a57ff8d6" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 30 14:39:05 2018 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 30 14:39:05 2018 +0000" }, "message": "Revert^4 \"Add an option to disable native stack dumping on SIGQUIT.\"\n\nBug: 74121887\n\nStill failing :(\n\nThis reverts commit 642e9d8249be5aff68022cabdc8ba576a57ff8d6.\n\nChange-Id: I603ca9fdd2d8f2f759527130b3288efe5b23b5c3\n" }, { "commit": "99ebc264162a8cfc6c0298ce7a1739d4b050f1e3", "tree": "694b7af6ef7e4f2d37c808c86c16912e1056b907", "parents": [ "8867ed2f900944ac11051fd0abe8f5ccba93fc0e", "c4b1c0c42a707675755a468ba36026d7166a3cc0" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Mar 29 19:05:27 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 29 19:05:27 2018 +0000" }, "message": "Merge \"Fix failure to initialize AnnotatedStackTraceElement.\"" }, { "commit": "c4b1c0c42a707675755a468ba36026d7166a3cc0", "tree": "8e271a164f8bf584bff623a926d4ef323ee2f1c6", "parents": [ "8068bc3bc68e9560cc4650c6fb261ec9b8648fbd" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 29 17:07:17 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 29 17:39:44 2018 +0100" }, "message": "Fix failure to initialize AnnotatedStackTraceElement.\n\nTest: 171-init-aste\nTest: testrunner.py --host --interpreter\nBug: 76208924\nChange-Id: I2a0892c5cc8ab5cbc54a94c25a02add1031e68f5\n" }, { "commit": "642e9d8249be5aff68022cabdc8ba576a57ff8d6", "tree": "1fa53cee639241bd3968997e55e38ed282235569", "parents": [ "b939805d9d4b0957274c562fb8053512d04138b2" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Tue Mar 27 20:30:55 2018 +0000" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Tue Mar 27 22:36:09 2018 +0000" }, "message": "Revert \"Revert \"Revert \"Add an option to disable native stack dumping on SIGQUIT.\"\"\"\n\nThis reverts commit 74d25c9040dfd1e0985987eb38817e526878a3db.\n\nReason for revert: The original failing condition appears to be gone.\n\nBug: 74121887\n\nTest: Ran 004-ThreadStress many times against the target.\nChange-Id: Ie5bd050112e654a99bdfea7d6dd673882ca35567\n" }, { "commit": "3dada431768b980c65d8256327f160803aae1b41", "tree": "a1ab5bafe0ac9324710f3ff8f8dc7b5473bf1ce4", "parents": [ "2410a88a94b57247a1096562564c2f23528b3e46", "88591fe82f499de10591f5b77efac71f8954eae2" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Sun Mar 25 20:23:11 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Sun Mar 25 20:23:11 2018 +0000" }, "message": "Merge \"ART: Simplify atomic.h\"" }, { "commit": "318afe6c3ac1e734adbd769bbf22b8c7e373e80b", "tree": "2dada0828636c2dc8e40e0f34b2b3f01e6caa0a5", "parents": [ "ba1e9e73c07bffc046f651414db43aeee9953d15" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Mar 22 16:50:10 2018 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Mar 23 14:19:35 2018 -0700" }, "message": "Remove unneeded ScopedGCCriticalSections from openjdkjvmti.\n\nWe used ScopedGCCriticalSections in many parts of the openjdkjvmti\noften unnecessarily.\n\nWe removed a totally unneeded GCCriticalSection that was acquired when\nmodifying the instrumentation listeners.\n\nWe also removed RequestGCSafeSynchronousCheckpoint and the change to\nuse GcRoots instead. We added RequestGCSafeSynchronousCheckpoint as a\nway to prevent the GC from running when we are doing some JVMTI\noperations on other threads. This could interact with running GCs in\nnon-trivial ways, potentially causing deadlocks in some situations.\nThis changes the code to instead use read-barriers and GcRoots to\nensure that we do not read data from the wrong gc space.\n\nIn order for this to work correctly we need to make sure that we are\nonly ever reading the GcRoots from the thread that eventually needs\nthe reference. This required some re-writing of the checkpoint\nclosures since they would often just call AddLocalReference on\nnon-local Thread objects.\n\nChanges to Thread::RequestSynchronousCheckpoint and art::Barrier were\nneeded in order to allow this all to work since we needed to ensure\nthat the requesting thread did not suspend as the checkpoint was being\nrun.\n\nThis is a partial revert of commit 7585b91bfc77b8.\n\nBug: 67838964\nBug: 76003243\n\nTest: use gapid\nTest: ./test.py --host -j50\nTest: ./art/tools/run-libjdwp-tests.sh --mode\u003dhost\n\nChange-Id: I26d871089829639eccb973cecc315194f7bcf681\n" }, { "commit": "88591fe82f499de10591f5b77efac71f8954eae2", "tree": "bfa126ad55ee091e3b615bd3bb60d5f8cfb6e37a", "parents": [ "e8a4e378c5a928d5de07bee6db99150a57dabcd8" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Tue Mar 06 13:35:43 2018 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Mar 23 15:01:15 2018 +0000" }, "message": "ART: Simplify atomic.h\n\nPrefer std::atomic operations over wrappers in atomic.h. Exceptions\nare cases that relate to the Java data memory operations and CAS\noperations.\n\nBug: 71621075\nTest: art/test.py --host -j32\nTest: art/test.py --target --64 -j4\nChange-Id: I9a157e9dede852c1b2aa67d22e3e604a68a9ef1c\n" }, { "commit": "86821aca0eff7fb1fff18c0d71acfcc06f5d73f3", "tree": "a285269f9035de4767c2b166784b33ca3a8e9d01", "parents": [ "ec9855e6262cd477c9f61ed245b9e2526e351698", "74d25c9040dfd1e0985987eb38817e526878a3db" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 13 08:57:20 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Mar 13 08:57:20 2018 +0000" }, "message": "Merge \"Revert \"Revert \"Add an option to disable native stack dumping on SIGQUIT.\"\"\"" }, { "commit": "74d25c9040dfd1e0985987eb38817e526878a3db", "tree": "e63323329298a7cc67799324a6edc2378cfb6822", "parents": [ "8b089742252e827d863218413e8855e1bae75af5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 12 19:26:44 2018 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 12 19:26:44 2018 +0000" }, "message": "Revert \"Revert \"Add an option to disable native stack dumping on SIGQUIT.\"\"\n\nBug: 27185632\nBug: 74121887\n\n\nGot flakes on device for 004-ThreadStress and 141-static-field-sigquit\n\nThis reverts commit 8b089742252e827d863218413e8855e1bae75af5.\n\nChange-Id: Ie76c84a1889e85b885d5f4123d60e8a760cf186d\n" }, { "commit": "7a02c66fd9ed174fc2e49ccc9f582dd661b7de9e", "tree": "8f193c8ecea0ff418370bf72babb97487e8c9c65", "parents": [ "334f170654fed18711d8dcf5238c1ac8bad7ecd1", "8b089742252e827d863218413e8855e1bae75af5" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Mar 06 19:56:58 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Mar 06 19:56:58 2018 +0000" }, "message": "Merge \"Revert \"Add an option to disable native stack dumping on SIGQUIT.\"\"" }, { "commit": "c431b9dc4b23cc950eb313695258df5d89f53b22", "tree": "422273559c3ae52caff0c6b1cf1a62a8312f0e26", "parents": [ "f46f46cf5bd32788d5252b7107628a66594a5e98" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Mar 02 12:01:51 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Mar 05 13:58:20 2018 -0800" }, "message": "Move most of runtime/base to libartbase/base\n\nEnforce the layering that code in runtime/base should not depend on\nruntime by separating it into libartbase. Some of the code in\nruntime/base depends on the Runtime class, so it cannot be moved yet.\nAlso, some of the tests depend on CommonRuntimeTest, which itself needs\nto be factored (in a subsequent CL).\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host\n\nChange-Id: I8b096c1e2542f829eb456b4b057c71421b77d7e2\n" }, { "commit": "8b089742252e827d863218413e8855e1bae75af5", "tree": "bbfe333fced26fbc42f87a4469c59822611acd9f", "parents": [ "f46f46cf5bd32788d5252b7107628a66594a5e98" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Mar 05 11:47:30 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Mar 05 13:43:15 2018 -0800" }, "message": "Revert \"Add an option to disable native stack dumping on SIGQUIT.\"\n\nThis reverts commit a73280df8cac1279b6dea0424722f42ef0048613.\n\nBug: 27185632\nBug: 74121887\nTest: m test-art-host\nChange-Id: I24af48619577a78371c93cbad24d307d4d7a217d\n" }, { "commit": "b2ec9f5c128673c43f776cbe12c8eeb0a6884ebb", "tree": "724022162a4f3de021cebab32ee5ac67ec860f32", "parents": [ "f13343eb8b2738e2812fd7bd9d5592f529e93c21" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 21 13:20:31 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 21 13:51:15 2018 -0800" }, "message": "Remove duplication, split tests\n\nThe code move to libdexfile/dex/descriptors_names.cc apparently did not\nremove the original code from runtime/utils.cc. Fix that duplication\nand all the header mentions needed. Also, split the test files to go\nalong with the new locations for the code to be tested.\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host-gtest\n flash \u0026 boot marlin\n\nChange-Id: Ie734672c4bca2c647d8016291f910b5608674545\n" }, { "commit": "ad0777d89df7eb21d7d2001f9743882d10de3f5c", "tree": "15bf8fbfd1a3534972195d4ada9d92f1d0c98903", "parents": [ "db8d9091bbab41060584ab80882b60df20337da7" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Feb 12 20:00:18 2018 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Feb 15 18:51:24 2018 +0000" }, "message": "Visit proxy methods reference arguments when visiting Quick frames roots.\n\nThe arguments of a proxy method, stored in the proxy method\u0027s stack\nframe, need to be visited as GC roots. This is especially important\nin the case of a moving GC, where these reference arguments may be\nmoved like any object. Previously, we would only visit the target\n(`this` argument) of proxy methods when visiting Quick frames roots.\n\nTest: art/test/testrunner/testrunner.py --gcstress -t 999-proxy-method-arguments\nTest: m test-art-host\nTest: m test-art-target\nBug: 73149739\nBug: 70216372\nBug: 67679263\nChange-Id: Ieacc966ab1038935600f2193c14e6ca01e88602e\n" }, { "commit": "2ae376f5af8953d3524cd8ed915ebdacf505625c", "tree": "f6c1fa068b877054ea425bc4494462f2a9cbe955", "parents": [ "dc46115ab283bac2453b4f9e454e66107e64e8ef" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Jan 30 11:35:11 2018 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Feb 09 17:50:28 2018 +0000" }, "message": "Stylistic and aesthetic changes.\n\nTest: art/test.py\nChange-Id: Ic41aa80430d16af748994c80f049c5b479fd9980\n" }, { "commit": "fa854e4834c383daafd240902ecb291ecfdad7b6", "tree": "ac3d740dc0c7c6e91b286f467c3b111b178822c5", "parents": [ "bae3065e57e2e82aa29d6207312c5ba38cff7426" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Feb 07 13:09:55 2018 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Feb 07 14:27:57 2018 +0000" }, "message": "Visit targets of proxy methods when visiting thread roots.\n\nThe target of a non-static proxy method (`this` object), stored in the\nproxy method\u0027s stack frame, needs to be visited as GC root. This is\nespecially important in the case of a moving GC, where the proxy\ninstance may be moved like any object.\n\nFix initially provided by Robert Vollmer.\n\nTest: m test-art-host\nTest: art/test/testrunner/testrunner.py --gcstress -t 1939-proxy-frames\nTest: art/test/testrunner/testrunner.py --gcstress -t 1914-get-local-instance\nBug: 70216372\nBug: 67679263\nChange-Id: Iea27a8eba51ccd9c9055efaf6b263892830170b5\n" }, { "commit": "0225f8e2939a9340cb7dcebfcfe7996a2bd9bce9", "tree": "fd32362dff4ecdc8ff8b0d800a1b46fb0d9d0104", "parents": [ "c7e546ff3963a1d51b1f100d308db735bd19f736" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 31 08:52:24 2018 +0000" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 31 01:24:53 2018 -0800" }, "message": "Revert \"Revert \"Make libdexfile build independent of runtime dir\"\"\n\nThis reverts commit 787784f9effb126b5d0d3dc97d544c4a477b5daf.\n\nReason for revert: Bot configuration issue.\n\nChange-Id: I6a10bb4a9571f89c7e4dd095f9157e830a44e2de\nBug: 22322814\nTest: make -j 50 checkbuild\n" }, { "commit": "cec262e99d2be886fa0b39de87cf937b69878ed3", "tree": "9cdd156b576249d968bbd0283da27575e64a0b79", "parents": [ "85f890b792d1547182d195db3f43240180ab880b", "56776014d181b7ee386a2ba5d85cf66d8bb4dd22" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Jan 30 19:19:41 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jan 30 19:19:41 2018 +0000" }, "message": "Merge \"ART: Call ThreadGroup.add for attached threads\"" }, { "commit": "787784f9effb126b5d0d3dc97d544c4a477b5daf", "tree": "1ff6d9fd284a4dd2a337165dff20e71137997c9d", "parents": [ "b40b7e73469339a6b667b4a2e2b8690112a74dc9" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:48:51 2018 +0000" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:48:51 2018 +0000" }, "message": "Revert \"Make libdexfile build independent of runtime dir\"\n\nThis reverts commit b40b7e73469339a6b667b4a2e2b8690112a74dc9.\n\nReason for revert: on device libdexfile.so missing\n\nChange-Id: I9bd61a98bef870400580e8c991cb061d3f57fa72\n" }, { "commit": "56776014d181b7ee386a2ba5d85cf66d8bb4dd22", "tree": "21b9f0411f565d9e49bce916e720b70a79145750", "parents": [ "47ec7ad0f819c171e41d323aff71f5ac425b1ccd" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jan 26 17:40:55 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 30 09:26:22 2018 -0800" }, "message": "ART: Call ThreadGroup.add for attached threads\n\nWhen attaching threads to the runtime, call ThreadGroup.add to\nlet the thread\u0027s group know that the thread is now started. This\nfixes incorrect internal accounting.\n\nTest: art/test/testrunner/testrunner.py -b --host -t 169\nTest: m test-art-host\nChange-Id: I60362b6b53acf06b97779ea9b3b0bc6264f1dc2d\n" }, { "commit": "b40b7e73469339a6b667b4a2e2b8690112a74dc9", "tree": "7e8a34d9c7894c4c946f674f19f0a0a512b95184", "parents": [ "9690ad794b324ba54e936608881ac0f62538b97a" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 25 17:11:07 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 01:17:58 2018 -0800" }, "message": "Make libdexfile build independent of runtime dir\n\nRemove libdexfile\u0027s dependency on utils.cc and move utf.cc into\n/dex. Remove libdexfile\u0027s constituent sources from libart and\nuse libdexfile wherever libart is. Also remove some ART-specific\ninterfaces. Libdexfile\u0027s tests remain to be converted, plus\nmoving the files to a new directory peer to runtime/.\n\nBug: 22322814\nTest: make -j 50 test-art-host\nChange-Id: Ifaf695216e4a0e43d3aa377984d933f7a2a243c2\n" }, { "commit": "fb6b0b1b04081f9ef7a240f702d8ce4e61a02e9f", "tree": "ae24c7d7bf4493ab9dc881c7b3b95ef8f706974d", "parents": [ "680e88ba30d1c599c5eaab4a207db3e39bf2d57f" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Dec 11 20:47:56 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 08 18:48:47 2018 -0800" }, "message": "ART: Add support for VMStack.getAnnotatedStackTrace\n\nAdd Thread.CreateAnnotatedStackTrace to return an array that\ncontains an AnnotatedStackTraceElement for each stack frame,\nwith the StackTraceElement describing the frame, an array\ncontaining all objects that are locked at the described location,\nand optionally for the top frame an object the thread is blocked\non, waiting for or sleeping on.\n\nAdd a test.\n\nBug: 70538431\nTest: m test-art-host\nTest: art/test/testrunner/testrunner.py -b --host -t 168\nChange-Id: I0d92e3d8182c4a592549a6445854816f71afd29e\n" }, { "commit": "71a7b36894114d02860d2e64b9396dbd5e0b9b38", "tree": "ceb72fb04618577004707042f4f7a9086da0dbeb", "parents": [ "bf84c1bbdbfd8d1e071ac8f3d6cc09e934212b5b", "4131d1096ac57a29ba1939c1cf2cb74144c16c92" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Jan 08 08:10:48 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jan 08 08:10:48 2018 +0000" }, "message": "Merge \"ART: Additional Atomic methods\"" }, { "commit": "9e734c7ab4599d7747a05db0dc73c7b668cb6683", "tree": "dce1d1993734a947fb2e6f626eb1b425cb72143b", "parents": [ "b496af808eaf3af5ebac50aef4fbec33323b5016" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 04 17:56:19 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Jan 05 11:07:19 2018 -0800" }, "message": "Create dex subdirectory\n\nMove all the DexFile related source to a common subdirectory dex/ of\nruntime.\n\nBug: 71361973\nTest: make -j 50 test-art-host\nChange-Id: I59e984ed660b93e0776556308be3d653722f5223\n" }, { "commit": "4131d1096ac57a29ba1939c1cf2cb74144c16c92", "tree": "d3ac7e0732eac24602718ee2d2af33641b4cf13e", "parents": [ "bd018439d78f89c84df061173521dbd91666c02e" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 03 14:04:42 2018 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jan 05 09:34:26 2018 +0000" }, "message": "ART: Additional Atomic methods\n\nA few additional methods on the Atomic class to support the\ninterpreter VarHandles implementation.\n\nMinor renaming of Atomic::FetchAnd{Or,And}.* to\nAtomic::FetchAndBitwise{Or,And}.*.\n\nMinor updates for consistency in the choice of argument names and doc\ncomments.\n\nBug: 65872996\nTest: art/test.py --host -j32\nChange-Id: Iae35286935fbb270478082fed2fe74abb7b68c57\n" }, { "commit": "4557b3858a66aa20e42bce937e1f0620aad880a2", "tree": "8f34d8f014b11f17c6351bb955fcc74c940b8d16", "parents": [ "90f20973356900e340998e8e2b34230e5c4c8fb0" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 03 11:47:54 2018 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 03 12:45:40 2018 +0000" }, "message": "ART: Rename Atomic::CompareExchange methods\n\nRenames Atomic::CompareExchange methods to Atomic::CompareAndSet\nequivalents. These methods return a boolean and do not get the witness\nvalue. This makes space for Atomic::CompareAndExchange methods in a\nlater commit that will return a boolean and get the witness value.\n\nThis is pre-work for VarHandle accessors which require both forms.\n\nBug: 65872996\nTest: art/test.py --host -j32\nChange-Id: I9c691250e5556cbfde7811381b06d2920247f1a1\n" }, { "commit": "1d32a16f93d8bb479523fd237277ecbbff5bf1f4", "tree": "399e9c129d6d1bc6ca761026654711ba774879a4", "parents": [ "9ad08a3ea53487066cd39fafa46b98bb70685b14", "55256cb60e11d4fac71affb4b9760a2931a3598d" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Dec 22 20:09:22 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Dec 22 20:09:22 2017 +0000" }, "message": "Merge \"Extensions to check JNI.\"" }, { "commit": "55256cb60e11d4fac71affb4b9760a2931a3598d", "tree": "fc1ed6885b013e0aa2bcfd9ef4dd94fec29bd382", "parents": [ "64bae9fb677aa0e2406d13ea9f8ebaa92e16f978" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Dec 21 17:07:11 2017 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Dec 22 10:11:39 2017 -0800" }, "message": "Extensions to check JNI.\n\nEnsure critical lock isn\u0027t held when returning from a down-call.\nLog a warning if the critical lock is held for a significant period of\ntime.\nRefactor JNIEnvExt to be a class rather than a struct.\n\nTest: mma test-art-host\n\nChange-Id: I4d149cb04d3a7308a22b92b196e51e2f1ae17ede\n" }, { "commit": "808c7a57bb913b13c22884f57cdacd59bf1fdb3f", "tree": "d7f0d7cabaac5a7646c25bae584a82a9aa279cc0", "parents": [ "64bae9fb677aa0e2406d13ea9f8ebaa92e16f978" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 15 11:19:33 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 22 09:41:17 2017 -0800" }, "message": "Make CodeItem fields private\n\nMake code item fields private and use accessors. Added a hand full of\nfriend classes to reduce the size of the change.\n\nChanged default to be nullable and removed CreateNullable.\nCreateNullable was a bad API since it defaulted to the unsafe, may\nadd a CreateNonNullable if it\u0027s important for performance.\n\nMotivation:\nHave a different layout for code items in cdex.\n\nBug: 63756964\nTest: test-art-host-gtest\nTest: test/testrunner/testrunner.py --host\nTest: art/tools/run-jdwp-tests.sh \u0027--mode\u003dhost\u0027 \u0027--variant\u003dX32\u0027 --debug\n\nChange-Id: I42bc7435e20358682075cb6de52713b595f95bf9\n" }, { "commit": "9ff1251fed35356bf74dc875ae9b0cea9e50a3b0", "tree": "ff371b868daeb8092f879b8c29bf93f819b09494", "parents": [ "192cf6e300316d3fbbead115b1ca50ed3dc2f4b3" ], "author": { "name": "Kazuhiro Inaba", "email": "kinaba@google.com", "time": "Mon Dec 04 16:26:38 2017 +0900" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 15 09:33:37 2017 +0000" }, "message": "ART: Null thread_name check for loggings in rare occasions.\n\nPassing null char* to ostream::operator\u003c\u003c is undefined and can cause\na crash (for instance when libc++ is used) instead of printing the log.\n\nBug: None (crbug.com/531282)\nTest: None\n\nChange-Id: Ieb1f0fc50723b06e72b66b1da7b6abe58d5b9a02\n" }, { "commit": "9310495f3bb601058a3b6dedc298ad9f5275510e", "tree": "14472899a051a890a429c687836e15407f76cb74", "parents": [ "ed416f76ae679d92f0bc648581de2684ff93e29a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 13 17:13:15 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 14 17:11:22 2017 -0800" }, "message": "ART: Factor out stack dumping with monitor objects\n\nFactor out generic stack walking with monitor object visitation into\nMonitorObjectsStackVisitor. Rewrite StackDumpVisitor in terms of this,\nremoving Monitor::DescribeWait on the way.\n\nBug: 70538431\nTest: m test-art-host\nTest: manual (SIGQUIT dumping)\nChange-Id: I6f359f9a5f665f308328ad466bc7a437f52aea24\n" }, { "commit": "b6f309e4e9b7c0531f8d09998b48d82b45714173", "tree": "da2e45e4ef317fbd9ff1e9ef2913c407332689fa", "parents": [ "ba3a790338725a37ecd4cb314c4a6147e29aef38" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Dec 11 20:45:41 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 14 11:17:41 2017 -0800" }, "message": "ART: Factor out StackTraceElement creation\n\nMove to a helper function to aid future reuse.\n\nBug: 70538431\nTest: m test-art-host\nChange-Id: Ia7c26d707abbae3a9d5cc5db99007e34a8ab6b62\n" }, { "commit": "7919db947bc41f6f5d194c50b88d0cba47319a1c", "tree": "ae1aa7fed4e9a6c6bbec3fc1c2b85cef173e737c", "parents": [ "50f90a44f8cd1928054245cab4cd3bd47a1a6347" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Nov 29 09:00:55 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Dec 01 08:55:08 2017 -0800" }, "message": "Speed up MterpShouldSwitchInterpreters check\n\nWe were often performing a pair of TLS reads in order to determine in\nthe current thread has any pending asynchronous exceptions (exceptions\nthrown by the JVMTI StopThread function). This is quite slow and was\nimpacting some benchmarks. Since it is expected that asynchronous\nexceptions are extremely rare we will first check to see if any\nasynchronous exceptions have been sent on the current process. Only if\nat least one asynchronous exception has been thrown will we do the\nexpensive TLS lookups to determine if one has been thrown on the\ncurrent thread.\n\nUsing a global instance value without synchronization or atomics is ok\nhere since the checkpoint that actually sets the async_exception_thrown\nflag provides synchronization by either occurring on the target thread\nor passing the checkpoint.\n\nAccording to go/lem this gives us a 7% increase on the caffeine\nstring benchmark.\n\nTest: go/lem runs\nTest: ./test.py --host -j50\nBug: 68010816\nChange-Id: I62684a5b3a7fc7cc600f5efd2a2393d9c4025917\n" }, { "commit": "e7441631a11e2e07ce863255a59ee4de29c6a56f", "tree": "a0488fac018ea391d01b751b7254719937bdee8f", "parents": [ "aa25db7d2a6f7f507c27ce49c99a33daf3059f8f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 29 13:00:56 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 29 14:04:26 2017 +0000" }, "message": "Revert \"Revert \"JIT JNI stubs.\"\"\n\nThe original CL,\n https://android-review.googlesource.com/513417 ,\nhad a bug for class unloading where a read barrier was\nexecuted at the wrong time from\n ConcurrentCopying::MarkingPhase() -\u003e\n ClassLinker::CleanupClassLoaders() -\u003e\n ClassLinker::DeleteClassLoader() -\u003e\n JitCodeCache::RemoveMethodsIn() -\u003e\n JitCodeCache::JniStubKey::UpdateShorty() -\u003e\n ArtMethod::GetShorty().\nThis has been fixed by removing sources of the read barrier\nfrom ArtMethod::GetShorty().\n\nTest: testrunner.py --host --prebuild --jit --no-relocate \\\n --no-image -t 998-redefine-use-after-free\nBug: 65574695\nBug: 69843562\n\nThis reverts commit 47d31853e16a95393d760e6be2ffeeb0193f94a1.\n\nChange-Id: I06e7a15b09d9ff11cde15a7d1529644bfeca15e0\n" }, { "commit": "47d31853e16a95393d760e6be2ffeeb0193f94a1", "tree": "344b0fddbbcc9a64bed2ba20dbe73fb227c2bdf4", "parents": [ "3417eaefe4e714c489a6fb0cb89b4810d81bdf4d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 28 18:36:12 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 28 18:36:12 2017 +0000" }, "message": "Revert \"JIT JNI stubs.\"\n\nSeems to break 998-redefine-use-after-free in\nsome --no-image configuration.\n\nBug: 65574695\nBug: 69843562\n\nThis reverts commit 3417eaefe4e714c489a6fb0cb89b4810d81bdf4d.\n\nChange-Id: I2dd157b931c17c791522ea2544c1982ed3519b86\n" }, { "commit": "3417eaefe4e714c489a6fb0cb89b4810d81bdf4d", "tree": "fe97f5191d25d26ef4250280f4c599b3a50f2059", "parents": [ "7bdc6e73fd97eb75f30b77f183e4fe6c2c599a09" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 21 18:14:28 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 28 13:00:09 2017 +0000" }, "message": "JIT JNI stubs.\n\nAllow the JIT compiler to compile JNI stubs and make sure\nthey can be collected once they are not in use anymore.\n\nTest: 667-jit-jni-stub\nTest: Pixel 2 XL boots.\nTest: m test-art-host-gtest\nTest: testrunner.py --host --jit\nTest: testrunner.py --target --jit\nBug: 65574695\nChange-Id: Idf81f50bcfa68c0c403ad2b49058be62b21b7b1f\n" }, { "commit": "33bff25bcd7a02d35c54f63740eadb1a4833fc92", "tree": "553db4f60878acf2a0fa7036a739d406df9a29b7", "parents": [ "321b3ca9a36d769283c64d4bdee0798db80af524" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 01 14:35:42 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 02 10:11:02 2017 +0000" }, "message": "ART: Make InstructionSet an enum class and add kLast.\n\nAdding InstructionSet::kLast shall make it easier to encode\nthe InstructionSet in fewer bits using BitField\u003c\u003e. However,\nintroducing `kLast` into the `art` namespace is not a good\nidea, so we change the InstructionSet to an enum class.\nThis also uncovered a case of InstructionSet::kNone being\nerroneously used instead of vixl32::Condition::None(), so\nit\u0027s good to remove `kNone` from the `art` namespace.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: I6fa6168dfba4ed6da86d021a69c80224f09997a6\n" }, { "commit": "98ac0cce960ed7df71def630871aec04ac878dfa", "tree": "0ca17af216cd1a4846e05bbfbac89244afc1468d", "parents": [ "ca4e1014634aefb9c2dbb431d6b312e4ad912ca2", "df00a1ed1a0b633a1e66f1f650f53c22ea260e5b" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Nov 01 20:42:05 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 01 20:42:05 2017 +0000" }, "message": "Merge \"Prevent abort in situations with recursive checkpoints\"" }, { "commit": "df00a1ed1a0b633a1e66f1f650f53c22ea260e5b", "tree": "82b965a44ce46f51def31e1e311a26ee991ee868", "parents": [ "a2cbb2b0723a3fa05cc44e13cbe90543b3236883" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Nov 01 09:29:53 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Nov 01 11:47:01 2017 -0700" }, "message": "Prevent abort in situations with recursive checkpoints\n\nIn situations where there were multiple checkpoints queued and the\nfirst one causes the thread to suspend itself again then the\nRunCheckpointFunction function will hit a LOG(FATAL) and abort. This\nis because the recursive checkpoint will clear out the checkpoint\nbacklog and the first RunCheckpointFunction invocation will\nunexpectedly find itself without any more checkpoints to run and\nabort.\n\nTo fix this, and simplify the code at the same time, we have changed\nthe RunCheckpointFunction method to (as its name suggests) only run a\nsingle checkpoint function. It will pop this function off the stack of\npending checkpoints and run it relying on the caller to ensure that\nall pending checkpoints are handled. This is fine since, due to the\nmultithreaded nature of checkpoints, the caller must call\nRunCheckpointFunction in a loop anyway to ensure it does not advance\nuntil all checkpoints have been handled.\n\nWe add test 203-multi-checkpoints that tests that the checkpoint\nsystem does not fall over if there are multiple checkpoints some of\nwhich can suspend.\n\nBug: 67838964\nTest: ./test.py --host -j50\nChange-Id: Ib6a3e083e6069d4839647d194bee6849d973633e\n" }, { "commit": "891a50e9dc0f02ee34bd7348f80b621249153af5", "tree": "6af57b3aa8961ed8e6cd39fdf5a0ac335905b4c6", "parents": [ "13e6f2935d67742e9eec14b412d33bf71b20697b" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Oct 27 17:01:07 2017 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Nov 01 11:39:19 2017 -0700" }, "message": "Separate file utilities from utils.cc\n\nReduce the dependencies from utils.cc to arch/ subdirectories, etc.\n\nBug: 22322814\nTest: make test-art-host\nChange-Id: I4decd15ff4ec460735bfb58cebb1c6a1bf32b68f\n" }, { "commit": "373a9b5c718a45ac484afcf4fe6ce84f4bb562b3", "tree": "52a3e78fa8bc94a76674caaae837c728332d05dc", "parents": [ "d0332e01ac26c2fd0a900de000a55ed5768ff936" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 18 09:01:57 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 25 10:13:59 2017 -0700" }, "message": "ART: Depend on libnativehelper headers only\n\nDepend on header-only versions where possible. Move projects\nexcluding libart to libnativehelper_header_only.\n\nBug: 65522645\nTest: mmma art\nChange-Id: I53dd3b2a97e94ee685f72de007ed3858f7f5c6b6\n" }, { "commit": "b1e31a8337b9cce2f3160a972123886b355dce27", "tree": "30413acc04bd03c385b3b6023e332ecc535ff4ee", "parents": [ "8cdfac752808784d29185710095ad104922cd861" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Oct 04 16:57:36 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Oct 18 17:01:37 2017 -0700" }, "message": "Change RequestSynchronousCheckpoint to release thread_list_lock_\n\nThe RequestSynchronousCheckpoint function in some cases needs to\nrelease the thread_list_lock_ as it waits for a checkpoint to be\nexecuted. This means that the thread being checkpointed might be\ndeleted. Previously it was not obvious this was the case since the\nthread_list_lock_ seemed to be held throughout the execution of the\nmethod.\n\nIn order to prevent bugs we make RequestSynchronousCheckpoint\nexplicitly release the thread_list_lock_ when executed, meaning code\nwill be aware that threads might die during its execution.\n\nBug: 67362832\nTest: ./test.py --host -j50\nChange-Id: I1cbdf7660096dc1908b0eeabc1062447307bc888\n" }, { "commit": "f1671357148b0ed205c93d051efd03f49e774885", "tree": "3a54ee282a807d5f2ee6a878945aa9e896131288", "parents": [ "844a4edc7f72e33a3b328c3d53ef710909d2273d", "54d39dc42630cd83f2d1bec5704805febb894819" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Oct 03 18:29:28 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Oct 03 18:29:28 2017 +0000" }, "message": "Merge changes from topic \"thread-stop\"\n\n* changes:\n Implement JVMTI can_signal_thread capability.\n Implement asynchronous exception support\n" }, { "commit": "f67f115423c9ef5aa62a33c12670cd8f89457c9c", "tree": "9381a1f2a0d5d8c7a52bcbce3c09c819177972a3", "parents": [ "2cda354d7fcd4e0a32462d4aaa448869a93b20a3", "df13240f4b9325b34d09e20cdac4e9a0b12ead61" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Sep 29 21:38:44 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Sep 29 21:38:44 2017 +0000" }, "message": "Merge \"Send ThreadEnd after clearing ThreadGroup.\"" }, { "commit": "df13240f4b9325b34d09e20cdac4e9a0b12ead61", "tree": "ed2e646165904d2e8c15210ddc84bfd98b8c8d9f", "parents": [ "05727c50c8c329d03692ffe51dce453c54379d70" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Sep 29 12:54:33 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Sep 29 13:09:58 2017 -0700" }, "message": "Send ThreadEnd after clearing ThreadGroup.\n\nThe removal of a thread from it\u0027s thread group does cause managed code\nto execute so it could cause additional events to trigger in agents,\npotentially causing deadlocks. This changes where the event is\ntriggered to put it after the thread has been removed from its group.\n\nTest: ./test.py --host -j50\nBug: 67104807\n\nChange-Id: I53f53b2823835a9629842fb5c22079b3e42042d5\n" }, { "commit": "a78f5b6da738d38b0833b223b42d4486273f05bd", "tree": "0d43fc24fbe1e8fb10e380124e1954cd7961d0b6", "parents": [ "eb5bedd1ac81597aa9448f329d0269780f329aa3" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Sep 29 13:50:44 2017 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Sep 29 13:50:44 2017 +0100" }, "message": "Refactor conditional uses of ReadBarrier::AssertToSpaceInvariant.\n\nTest: m test-art-host\nBug: 12687968\nChange-Id: Idf7de149bc0d0dcfc9efe01918aa026754f9070d\n" }, { "commit": "848574ca50bb7e2d109608359d1086b3ca6bb4b3", "tree": "6a3d55ae8b66279c1e417c7c5116394613037ae8", "parents": [ "9c0a6ce0a41eaf78f3fc3224fc34195601b6e0b3" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Sep 25 16:59:39 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Sep 28 12:56:13 2017 -0700" }, "message": "Implement asynchronous exception support\n\nThis CL implements async exception support in the switch interpreter.\nIt also adds support for the MTerp to detect and switch back to the\nswitch interpreter in cases where an async exception is detected.\n\nTests follow in next CL.\n\nTest: ./test.py --host -j50\n\nBug: 62821960\nBug: 34415266\n\nChange-Id: Idb53711a40c20f962de8aa6b74662676b8bd25c6\n" }, { "commit": "b77b698f4550458725b72d3a451adca765f369cc", "tree": "edfc6a5b96325a6b4cdaa1a4e4428f1b677c7745", "parents": [ "e5b35ed787fbfb388d162361310bae5b0e7682a7" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Jun 08 18:03:48 2017 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Sep 28 18:05:39 2017 +0100" }, "message": "Check the to-space invariant on a delivered exception.\n\nTest: art/test/testrunner/testrunner.py\nBug: 62339341\nBug: 12687968\nChange-Id: Ida5e91031cd94429a72fcc9d4d1333d21dd07421\n" }, { "commit": "4d159807a4854caa6396b708a38bbd6fa49d736f", "tree": "25c2ea8ce12881cf26c933c89a0a3096d260de46", "parents": [ "08601a494e87dfba9b06f9fdea37c9342e4896d1", "0882af2e3ca253184b6ab56a8966a2f37407144e" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Tue Sep 19 16:37:30 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Sep 19 16:37:30 2017 +0000" }, "message": "Merge \"Shrink ART Mutex exclusive_owner_ field to Atomic\u003cpid_t\u003e\"" }, { "commit": "e2abbc604ce003c776c00ecf1293796bb4c4ac5a", "tree": "f7d124d1861cad2162c30dfe932bb4e1beaf41ef", "parents": [ "7090dfe84f78b1928fcbdfd664d0dd9ea52633ff" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 15 11:59:26 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 18 11:36:24 2017 -0700" }, "message": "ART: Move kDexNoIndex to dex_file_types.h\n\nDefine the constant with the types to allow lowering the dependency\non DexFile.\n\nTest: m\nChange-Id: I3c61421db45be96d2057e01b1a7825883d8bd178\n" }, { "commit": "0882af2e3ca253184b6ab56a8966a2f37407144e", "tree": "dc61cf0d66ec576c6b2e3b78b5fb304afd6994c9", "parents": [ "6b411fc9fd94da3c552d8b96cee14f6414e7b734" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Thu Aug 31 15:21:57 2017 -0700" }, "committer": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Wed Sep 13 13:15:12 2017 -0700" }, "message": "Shrink ART Mutex exclusive_owner_ field to Atomic\u003cpid_t\u003e\n\nThe old volatile uint64_t version had a data race, and was thus\ntechnically incorrect. Since it\u0027s unclear whether volatile uint64_t\nupdates are actually atomic on 32-bit platforms, even the informal\ncorrectness argument here already effectively assumed that the upper\n32 bits were zero. Don\u0027t store them. Explicitly complain if a pid_t\nmight be too big to support lock-free atomic operations.\n\nRemove many explicit references to exclusive_owner to avoid\nlittering the code with LoadRelaxed calls.\n\nThe return convention for GetExclusiveOwnerTid() was unclear\nfor the shared ownership case. It was previously treated\ninconsistently as 0 (pthread locks), (uint64_t)(-1U) and\n(uint64_t)(-1). Make it as consistent as easily possible, and\ndocument remaining weirdness.\n\nBug: 65171052\n\nTest: AOSP builds. Host tests pass.\n\nChange-Id: Ia99aca268952597a90b3c798b714cddbdc2c365e\n" }, { "commit": "2ee17909eadd7155f4a7751c38398b36fc267f04", "tree": "2d13de7e9aae9d24ceb4ff1e03f81e46a5aee54b", "parents": [ "1accd636dc90edef2878f0b8e2716b078379842e" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Aug 30 11:37:08 2017 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Tue Sep 05 13:22:15 2017 -0700" }, "message": "Revert^4 \"Allow deoptimization when returning from a runtime method.\"\n\nThis reverts commit 07c7028e518b98d3267a77dfe0d149db1adbe858.\n\nNeed some special treatment of string init\u0027s shorty.\n\nTest: run-test/gtest on both host and target\nTest: 597-deopt-busy-loop, 597-deopt-invoke-stub\nBug: 33616143\nChange-Id: Id4c64910acfdd088835b6db6fc503e6ade0218e7\n" }, { "commit": "8ef4b6121ea496d38c0b8aeebad3f1227f819b17", "tree": "37847d1e1637ecf43cc11017c03f6cd626769311", "parents": [ "8ee58600c368b7b00e54aff09c575d4e2904316e", "07c7028e518b98d3267a77dfe0d149db1adbe858" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Aug 30 09:49:52 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Aug 30 09:49:52 2017 +0000" }, "message": "Merge \"Revert \"Revert \"Revert \"Allow deoptimization when returning from a runtime method.\"\"\"\"" }, { "commit": "07c7028e518b98d3267a77dfe0d149db1adbe858", "tree": "71fbc9572541cdf08c21c88e3a1d5ff92f1d9885", "parents": [ "047abb20d02546d3dd6e8630befc31e5568fa90e" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Aug 30 08:09:42 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Aug 30 08:09:42 2017 +0000" }, "message": "Revert \"Revert \"Revert \"Allow deoptimization when returning from a runtime method.\"\"\"\n\nBug: 33616143\n\ndeopt string test still failing on occasion.\n\nThis reverts commit 047abb20d02546d3dd6e8630befc31e5568fa90e.\n\nChange-Id: I89fc28696290da52317d0e3dd07ecf0d1bdac823\n" }, { "commit": "0b5cb35e40901303a4d4283722025d20f2e68e17", "tree": "1fc1d1de751e8587218c79291e09bdbfc4ab19df", "parents": [ "5575513f1529ea43371a6d4d9a495b57e7d68409", "047abb20d02546d3dd6e8630befc31e5568fa90e" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Aug 28 16:30:48 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Aug 28 16:30:48 2017 +0000" }, "message": "Merge \"Revert \"Revert \"Allow deoptimization when returning from a runtime method.\"\"\"" }, { "commit": "047abb20d02546d3dd6e8630befc31e5568fa90e", "tree": "dfe08cef1f53a7559f792221e3e06dccbe704380", "parents": [ "95d9bdd36b7ae6682f8602292fd547a0faa73829" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Aug 23 15:26:57 2017 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Aug 24 14:36:21 2017 -0700" }, "message": "Revert \"Revert \"Allow deoptimization when returning from a runtime method.\"\"\n\nThis reverts commit 2b87ae0073256e909e15f464300912552e58ee48.\n\nFor an invocation runtime method such as quick-to-interpreter bridge,\nadd a special stack walk to get the shorty for the invoked method.\n\nTest: run-test/gtest on both host and target, and 597-deopt-runtime-method.\nBug: 33616143\n\nChange-Id: I53ae93880f62c95dcf48005239b925d7f7b11eb6\n" }, { "commit": "6e1607e247d979a1671a1fd5a98de3f1031fe719", "tree": "1174cdcce601419dcd5d1563777f721477d5cb6a", "parents": [ "2ee17e69110a9ef98ea8c94219c7da23b918ef3f" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Aug 23 10:06:18 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Aug 23 13:15:17 2017 -0700" }, "message": "Rename instrumentation ExceptionCaught to ExceptionThrown\n\nThe instrumentation ExceptionCaught event was badly named since it is\nactually sent at the time the exception is thrown and is sent even if\nthe exception is never actually caught. Rename it and associated\nmethods/fields to ExceptionThrown to more accurately reflect the\nevent\u0027s semantics.\n\nTest: ./test.py --host -j50\nChange-Id: I1e9c401ab619259b25928dbf19f62d3fa9573d47\n" }, { "commit": "2b87ae0073256e909e15f464300912552e58ee48", "tree": "0b9db32799d6efa93d3db6dca5170d185627e138", "parents": [ "fc63babf6be7bdd96be1492391594ad628f1fd3c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Aug 23 10:16:26 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Aug 23 10:16:47 2017 +0000" }, "message": "Revert \"Allow deoptimization when returning from a runtime method.\"\n\nReason for revert:\n Some tests failing even after a minor fix.\n\nThis reverts commit edeba10d523c3e283ab939a16c7203af32c7707e.\n\nChange-Id: I6407c9b489c016d19a12c28d1da0efa55ad554a7\n" }, { "commit": "edeba10d523c3e283ab939a16c7203af32c7707e", "tree": "88f13395178e7a17979dcb0666c65c024732be6b", "parents": [ "c9d88538d73680563f7a6e73885acfff2c55ef58" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Apr 12 13:43:15 2017 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Aug 21 16:44:57 2017 -0700" }, "message": "Allow deoptimization when returning from a runtime method.\n\nThis CL patches the return pc of a runtime method to allow the top\nJava frame to be deoptimized. This should fix the issue that debugger\ncannot break in a busy loop. It also means we can now do full async\ndeoptimization, if we want to enable it by letting environment keep\nregisters live.\n\nart_quick_instrumentation_exit and art_quick_deoptimize now need to save\nall registers since some compiler slow paths assume runtime methods save\neverything.\n\nSome special handling needs to be done to decide whether dex_pc should\nbe advanced when deoptimized back to interpreter.\n\nTest: run-test/gtest on both host and target, and 597-deopt-runtime-method.\nBug: 33616143\nChange-Id: I2e2c199998825afd5057f7deadfc8fa203ce1936\n" }, { "commit": "331f4c4e287791611733120c1a1c2afd55ecdd65", "tree": "1fcf7810c6c8e2df8b6191bb14a69084f3c7cf11", "parents": [ "13c8343a3394414c90f2fcd1e8efff70e7d2387e", "ba118827465d12177f3996e50133960087b1c916" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jul 20 14:09:32 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jul 20 14:09:32 2017 +0000" }, "message": "Merge \"ART: Change method lookup to be more consistent to JLS and the RI.\"" }, { "commit": "ba118827465d12177f3996e50133960087b1c916", "tree": "f39728cdafc7810004d51c0bef2728b98993daa9", "parents": [ "64a102dde8c5daad83b991710decb418ce43aac5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 12 15:41:56 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 20 13:35:06 2017 +0100" }, "message": "ART: Change method lookup to be more consistent to JLS and the RI.\n\nThe method lookup for different invoke types was previously\nwidely different and didn\u0027t work well with the dex cache\nmethod array where we have only a single slot for each\nMethodId. The new behavior is to perform the same lookup for\nall cases, distinguishing only between interface and\nnon-interface referencing class, and to further align the\nbehavior with the JLS and the RI. Where the JLS conflicts\nwith the RI, we follow the JLS semantics.\n\nThe new lookup for class methods first searches the methods\ndeclared in the superclass chain (ignoring \"copied\" methods)\nand only then looks in the \"copied\" methods. If the search\nin the superclass chain finds a method that has not been\ninherited (i.e. either a private method or a package-access\nmethod where one of the classes in the chain does not belong\nto the same package, see JLS 8.4.8), we still search the\n\"copied\" methods as there may actually be a method inherited\nfrom an interface. This follows the JLS semantics where\ninherited methods are included in the search (JLS 15.12.2.1)\nbut conflicts with the RI where the private or\npackage-access method takes precedence over methods\ninherited from interfaces.\n\nNote that this search can find an accessible method that is\nnot inherited by the qualifying type, either for a package\naccess method when the referrer is in the same package but\nthe qualifying type is in another package, or for a private\nmethod where the referrer is in the same class but the\nqualifying type is actually a subclass. For the moment we\nallow such calls and we shall consider whether to throw\nan IncompatibleClassChangeError in this situation in future\nto comply with JLS 15.12.4.3.\n\nThe new lookup for interface methods searches the interface\nclass, then all the superinterfaces and then the\njava.lang.Object class, see implicitly declared methods in\ninterfaces, JLS 9.2. The search for the maximally-specific\nnon-abstract superinterface method is not yet implemented,\nbut the difference should be difficult to observe as the\nusual subsequent call to FindVirtualMethodForInterface()\nshould yield the same result for any matching method.\n\nThe new test 162-method-idx-clash exposes several cases\nwhere we previously completely messed up due to the effects\nof the DexCache, or where we were out of line with the RI.\nIt also tests a case where the JLS and the RI disagree and\nwe follow the JLS.\n\nTest: art/test/run-test --host --jvm 162-method-resolution\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --host --interp-ac\nTest: Nexus 6P boots.\nTest: testrunner.py --target\nBug: 62855082\nBug: 30627598\nChange-Id: If450c8cff2751369011d649c25d28a482a2c61a3\n" }, { "commit": "e431e2758d62cf56f7f347f5a8c9d79e41b6dcd7", "tree": "b8733533e2478069aabbdc3a34d0f40aff33e1b1", "parents": [ "b98d384eb424ad7e1dc9816392ded33dce0e0372" ], "author": { "name": "Steven Moreland", "email": "smoreland@google.com", "time": "Tue Jul 18 16:53:49 2017 -0700" }, "committer": { "name": "Steven Moreland", "email": "smoreland@google.com", "time": "Tue Jul 18 18:25:55 2017 -0700" }, "message": "art: use proper nativehelper headers\n\nlibnativeheader exports headers under nativeheader. These were\navailable before incorrectly as global headers in order to give\naccess to jni.h.\n\nTest: modules using art find headers\nBug: 63762847\nChange-Id: I5c820d677e94e07b2859e78610bc997fe51b41dc\n" }, { "commit": "5e7eb2faccf8f4a28e9fcda26053a5b388f2190a", "tree": "12b2936d445607d1bd7294d3f4a6ba19c27d8951", "parents": [ "6e875347976b1f332731be775dfd066fda730085", "88fd720b6799184c8ad61e766a6d37af33ed30ef" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Jul 14 15:13:41 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jul 14 15:13:41 2017 +0000" }, "message": "Merge \"Add Jvmti Suspend/ResumeThread functions\"" }, { "commit": "88fd720b6799184c8ad61e766a6d37af33ed30ef", "tree": "80e49456eafd44b0ad8790be456ae06949fcc506", "parents": [ "1cba8d219331e4d6994359e8f9104e5db2c8f8a9" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jun 30 08:31:59 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jul 13 09:11:03 2017 -0700" }, "message": "Add Jvmti Suspend/ResumeThread functions\n\nEnable the can_suspend jvmti capability and implement all required\nfunctionality associated with it.\n\nTest: ./test.py --host -j40\nBug: 34415266\nBug: 62821960\nBug: 63579748\n\nChange-Id: I83b92de7f81622e1658114b034918e8295805b6e\n" }, { "commit": "08dd84adab79f98b2f2e7dfbccf8ef6b07cd3f2e", "tree": "f171c11372ad3826d91853a0e125be14e12d029d", "parents": [ "8ea20746896a5930d99b74d54213ff5a742ae937", "97c46466aea25ab63a99b3d1afc558f0d9f55abb" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Jul 12 10:38:10 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jul 12 10:38:10 2017 +0000" }, "message": "Merge \"Introduce a Marking Register in ARM64 code generation.\"" }, { "commit": "8c5e2dad9f585562f6a5b3ed2242a5d4a11fce3c", "tree": "b5e05e9b6fc2a2682d26c27d873c719fda03fd63", "parents": [ "733e441e3bcaa6efb8248fe3429b18d3199ee30e" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jul 11 10:32:20 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jul 11 17:55:33 2017 +0000" }, "message": "Move ThreadDeath event before clearing Thread.nativePeer\n\nWe were clearing the Thread.nativePeer field prior to calling\nThreadDeath event. Since this event can call into user-supplied code\nwe want to call it before we do any real cleanup.\n\nTest: ./test.py --host -j40\nBug: 62117833\n\nChange-Id: I6b93d82d23946c07dfbeb837d841339108e4b084\n" }, { "commit": "97c46466aea25ab63a99b3d1afc558f0d9f55abb", "tree": "afd225f51d28a77329bc2590a025400e088f260c", "parents": [ "00cca3a275562d110a8b35094b9b12fac37f67ab" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu May 11 14:04:03 2017 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Jul 11 17:43:27 2017 +0100" }, "message": "Introduce a Marking Register in ARM64 code generation.\n\nWhen generating code for ARM64, maintain the status of\nThread::Current()-\u003eGetIsGcMarking() in register X20,\ndubbed MR (Marking Register), and check the value of that\nregister (instead of loading and checking a read barrier\nmarking entrypoint) in read barriers.\n\nTest: m test-art-target\nTest: m test-art-target with tree built with ART_USE_READ_BARRIER\u003dfalse\nTest: ARM64 device boot test\nBug: 37707231\nChange-Id: Ibe9bc5c99a2176b0a0476e9e9ad7fcc9f745017b\n" }, { "commit": "6f1c7517a9e411e4000aa7b13743ed49e0c38bc0", "tree": "958355c402c445b531d8c6a219c69e6406d1b094", "parents": [ "71bde0dff0159976fdad4b6a45e903a96bd51aca" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 05 19:18:31 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 05 19:22:08 2017 -0700" }, "message": "ART: Make Touch\u0027s stack array smaller under ASAN\n\nUnder ASAN, especially on x86, there\u0027s a lot of overhead. To keep\nbelow a single page, add a multiplier to the deduction.\n\nBug: 31098551\nBug: 62952017\nTest: m\nTest: SANITIZE_HOST\u003daddress SANITIZE_TARGET\u003daddress m ; manual assembly inspection\nChange-Id: I8c80181e0fc5c6f4a1d3c6544f1a0e1089bb0631\n" }, { "commit": "dfcf10b92330164f8af6c82c8232e85cfff1ae3c", "tree": "1cf419939a816a1ab2d1b5de68083ece7ec7c5c1", "parents": [ "dd3240ce699129007935ba0bae4872e28652b90c", "46f9340f2a055a8fdfebbfbb739c697c20d83e7c" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Jun 30 16:18:22 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jun 30 16:18:22 2017 +0000" }, "message": "Merge \"Add SuspendReason enum and change Suspension functions.\"" }, { "commit": "46f9340f2a055a8fdfebbfbb739c697c20d83e7c", "tree": "acb172f74723e4ea9ac7ded73b94c661217ec90d", "parents": [ "fe9a4f061841a3c597aac6817a47c799c54fcad7" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jun 29 11:59:50 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jun 30 14:24:56 2017 +0000" }, "message": "Add SuspendReason enum and change Suspension functions.\n\nMore self-documenting and more type safe.\n\nBug: 62821960\nTest: ./test.py\nChange-Id: Ic7a1ae6a25e687d65f5aa10c1aad54a7b80dd086\n" }, { "commit": "6c63f863f9dbf734732ff837c1e772b3482e59ff", "tree": "cf2844dac5603567763f439fe5db650dccc993ad", "parents": [ "fe9a4f061841a3c597aac6817a47c799c54fcad7" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 28 17:20:15 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 29 12:13:55 2017 -0700" }, "message": "ART: Change main-thread thread paging scheme\n\nModify the code that ensures we can install a stack guard page into\nthe main thread.\n\nA recent kernel change means that our previous approach of using a\nfree pointer does not work. It is important to actually extend the\nstack properly. For portability, use a function with a large stack\nframe (suggested by and adapted from hboehm).\n\nBug: 62952017\nTest: m\nTest: m test-art-host\nTest: Device boots (x86_64 emulator)\nTest: Device boots (bullhead)\nChange-Id: Ic2a0c3d6d05a1ea9f655329d147b46949e1b9db3\n" }, { "commit": "117f9b3a821e5a81673a9859488a5c05cdf901ac", "tree": "c5b7a108bac7d1a096d49694bfe56dfbef7952f8", "parents": [ "721276bf926bdd5e3097830e20f6b6015409e28f" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jun 23 13:33:50 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jun 23 13:41:16 2017 -0700" }, "message": "ART: Fix issues in RequestSynchronousCheckpoint\n\nDo not run the checkpoint for another thread while being suspended.\nOnly suspend while waiting for that thread to become suspended.\n\nAfter decreasing the suspend count, broadcast on the resume condition,\nto tell the thread that it may progress.\n\nBug: 62353392\nTest: m test-art-host\nChange-Id: I63082b1fac5ec1db6ac31199548b9ae3cb30ea47\n" } ], "next": "28c4a233681040de4b2785ab5beef0a6d150e46a" }