)]}' { "log": [ { "commit": "fb3de3d72832177e4a8d1f322ed11cbe58e45c9f", "tree": "de8cded0436b54b2e5b1142b07fa75f0ed9791b0", "parents": [ "2827ff64c7a385cfb9d6e01e6385708461eb65fc" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jan 29 16:11:49 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Feb 02 14:14:08 2018 +0000" }, "message": "Create list of open dex files for libbacktrace.\n\nThis fixes unwinds after recent changes (oob apks; cdex data sharing).\n\nBug: 72520014\nTest: m test-art-host-gtest\nChange-Id: Ie2a02657b2afbe899acd2e61f0a57d207e688b99\n" }, { "commit": "a6e0e7df143deb1d022feac6a987e510fb4b2213", "tree": "f8f113ab203ba2b0bef7b74ffe0717c994033c3a", "parents": [ "8006cf1cd24d32809c937619d8d8568c8a1645ea" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 26 13:16:50 2018 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 26 14:13:19 2018 +0000" }, "message": "Special case JIT update for native methods.\n\nThe update might apply to ArtMethods that are going to be\nunloaded, so ensure we don\u0027t do read barriers there.\n\nTest: while ./art/test/run-test --host --no-dex2oat --jit --host --no-prebuild \\\n --compact-dex-level none --dex2oat-jobs 4 --no-relocate --runtime-option -Xcheck:jni \\\n --build-with-javac-dx 674-hiddenapi ; do true; done\n\nChange-Id: I95ec6107c65da25f4b98f7fb77647b3ab382a93f\n" }, { "commit": "c9e020852645b24ce9e3db7d1480aa9e72057146", "tree": "7dbdb71d2513d0885d9698f266f5ef728e6458fd", "parents": [ "39d7589745f559b84b58d008f67c3aaf364b0370" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jan 24 16:44:02 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jan 24 16:52:06 2018 +0000" }, "message": "Log JIT mini-debug-info memory usage.\n\nChange-Id: I0ffa6fb466b1635e724b0e782702303b92355408\n" }, { "commit": "c684f337eae6bd3d77fa44cb568d2bab113e4cee", "tree": "21c3e40cc150da71d91aef03f33010813f3a8368", "parents": [ "26037dd60df16d84fbd5c0ee3215b78aa2bcb408" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Jan 19 17:38:06 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Jan 19 17:42:54 2018 +0000" }, "message": "Refactor jit debugger interface and its ELF creation.\n\nMake it possible to store more then one method per entry,\nand ref-count the number of live methods per entry.\n\nTest: m test-art-host-gtest\nChange-Id: I45d69185e85e47fbee88a8d1f549ede9875a3c0a\n" }, { "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": "170331f0e44a0e07fcfe0b5932517e0500f5cd1f", "tree": "4bb837aa3b6e19c898df23894beb61957885e985", "parents": [ "39b378ca3b4b6dc6da1651b84ee4289cd9bff0f8" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 07 18:41:03 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 07 18:41:03 2017 -0800" }, "message": "ART: Remove base/logging from heap-inl.h\n\nHide the single uncommon VLOG and remove the include. Fix up\ntransitive includes.\n\nTest: m\nChange-Id: I917df597cb62c57040c1fb0e0079df4d95e5a658\n" }, { "commit": "2196c651ecc77e49992c6c329dfce45f78ff46cb", "tree": "4eb151632fc7b851101b4264286ce5e900fa06b5", "parents": [ "dc93cac66f1db225474cec5bf0350fd7a148085e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 30 16:16:07 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 30 16:20:39 2017 +0000" }, "message": "Revert^4 \"JIT JNI stubs.\"\n\nThe original CL,\n https://android-review.googlesource.com/513417 ,\nhas a bug fixed in the Revert^2,\n https://android-review.googlesource.com/550579 ,\nand this Revert^4 adds two more fixes:\n - fix obsolete native method getting interpreter\n entrypoint in 980-redefine-object,\n - fix random JIT GC flakiness in 667-jit-jni-stub.\n\nTest: testrunner.py --host --prebuild --no-relocate \\\n --no-image --jit -t 980-redefine-object\nBug: 65574695\nBug: 69843562\n\nThis reverts commit 056d7756152bb3ced81dd57781be5028428ce2bd.\n\nChange-Id: Ic778686168b90e29816fd526e23141dcbe5ea880\n" }, { "commit": "056d7756152bb3ced81dd57781be5028428ce2bd", "tree": "154ae8968a450a2e02d88207fa09e380fd380553", "parents": [ "e7441631a11e2e07ce863255a59ee4de29c6a56f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Nov 30 09:12:13 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Nov 30 09:12:13 2017 +0000" }, "message": "Revert \"Revert \"Revert \"JIT JNI stubs.\"\"\"\n\nStill seeing occasional failures on 667-jit-jni-stub\n\nBug: 65574695\nBug: 69843562\n\nThis reverts commit e7441631a11e2e07ce863255a59ee4de29c6a56f.\n\nChange-Id: I3db751679ef7bdf31c933208aaffe4fac749a14b\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": "b0b68cfc0d2444e2333adcc5a6bc6f670b89fe83", "tree": "2cd71b6d221470f3c26fd089e68c6349009bdd80", "parents": [ "f4580f43bcb74061ba8b7847ec65f2fe160f1751" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 14 18:11:50 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 14 18:29:20 2017 +0000" }, "message": "ART: Clean up #includes in jit_code_cache.h .\n\nAnd fix code that relied on those indirect #includes.\n\nAlso remove forward declaration of art::LinkerPatch because\nwe actually use art::linker::LinkerPatch.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 65574695\nChange-Id: I102affed9a3eacbd21b79c370fbc72b5fc762c96\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": "79c87da9d4698ec58ece65af0065eebd55a1cfe0", "tree": "399797f37712da9d20829d5b5059b6c84943cb02", "parents": [ "656e97f949df4081fba908113406b32915174502" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 10 11:54:29 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 11 14:47:06 2017 -0700" }, "message": "Add DexFileLoader class\n\nAdded DexFileLoader class, moved functionality from DexFile there:\n- Multidex loading logic\n- DexFile opening logic for Zip and etc\n- Some other helpers\n\nBug: 63756964\nTest: test-art-host\n\nChange-Id: Ic3dfa458947d4b69912dea5cdd836e7e8f55061c\n" }, { "commit": "8d8de0c92c45a25ac71375c689af29e168bc2ec1", "tree": "f60102a6d4493207ed3f8a4df86a004c4f5a046b", "parents": [ "1602bf67be5d9e377c1afde175b65634f4b8b0b5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 04 09:35:30 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 04 09:43:31 2017 -0700" }, "message": "Use CheckedCall for mprotects\n\nAims to prevent silent failures causing SIGSEGV in random places.\n\nBug: 66910552\nTest: test-art-host\n\nChange-Id: I91742b0b0059b5125ac663c3c3422ac5562df0fa\n" }, { "commit": "cfcc9cfb44bab79f7381bcc4bfd9bf2d4435f734", "tree": "6637daf0cde4f14afd02793ad268f22f9257bd4c", "parents": [ "f67f115423c9ef5aa62a33c12670cd8f89457c9c" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Sep 29 15:07:27 2017 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Sun Oct 01 12:08:08 2017 +0100" }, "message": "ART: Increase the number of potential instrinsics\n\nThe new limit is 256 intrinsics.\n\nAdds additional sanity checks.\n\nAvoids setting the kAccPreviouslyWarm bit for intrinics (defaults to\ntrue).\n\nBug: 65872996\nTest: art/test.py --host -j32\nChange-Id: I33ea67c9b6b8500b3ceb8a085358f075f6fcbb82\n" }, { "commit": "d1ee20fed62cd8db8fd496c4e5760dc554136020", "tree": "3252b0983fc23118846b68e0fc9b65b86af030eb", "parents": [ "2ade881db8642f10007c1c46b5e7f073d463c2d3" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Aug 17 09:21:16 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Aug 17 10:38:57 2017 +0100" }, "message": "Revert^4 \"ART: Use proxy ArtMethod\u0027s data_ to store the interface method.\"\n\nFix a DCHECK() failure while visiting references for non-CC\nmoving GC by avoiding the DCHECK(); reference equality may\ngive false negatives at that stage as we may compare the new\nreference against an old one for the same object.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --host --jit\nTest: testrunner.py --host --gcstress\nTest: ART_DEFAULT_GC_TYPE\u003dSS ART_USE_READ_BARRIER\u003dfalse \\\n testrunner.py --host --gcstress -t 048-reflect-v8\n\nThis reverts commit 76ccd09c3d98317dfbd179c6f5c231dcfc5d6996.\n\nChange-Id: I13cc339d5b31fceedf39ea9a77f27369ba72279f\n" }, { "commit": "76ccd09c3d98317dfbd179c6f5c231dcfc5d6996", "tree": "07f1546ce7a090af8c88110598920769bfb5e669", "parents": [ "c42fab35f18539957debb8703195eeb76c9718f7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Aug 15 19:11:54 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Aug 15 19:11:54 2017 +0000" }, "message": "Revert^3 \"ART: Use proxy ArtMethod\u0027s data_ to store the interface method.\"\"\n\nStill failling:\n\n+dalvikvm64 F 08-15 18:54:16 218912 218912 art_method-inl.h:395] Check failed: !GetDeclaringClass()-\u003eIsResolved() || interface_method-\u003eGetDeclaringClass()-\u003eIsAssignableFrom(GetDeclaringClass()) \n\n\nThis reverts commit c42fab35f18539957debb8703195eeb76c9718f7.\n\nChange-Id: I91b51f8c1e7f6e0ae876b751d8e9cd3b8acb62f6\n" }, { "commit": "c42fab35f18539957debb8703195eeb76c9718f7", "tree": "65bcf5256213282a315a34621b3fbc73786c1585", "parents": [ "f0c66eb04db3c9fc513f50dfaad1385cf703820e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Aug 15 14:11:49 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Aug 15 15:18:07 2017 +0100" }, "message": "Revert^2 \"ART: Use proxy ArtMethod\u0027s data_ to store the interface method.\"\n\nFix a DCHECK() to ignore unresolved proxy classes.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --host --jit\nTest: testrunner.py --host --gcstress\n\nThis reverts commit 7ef52f75dd6e981e1e8c7567f593c197e8a94dcc.\n\nChange-Id: Idccdb04958a8bdb1c58b529dfb92885993ac62ea\n" }, { "commit": "7ef52f75dd6e981e1e8c7567f593c197e8a94dcc", "tree": "9685b783f628e290c6307d6ead024aac749a11ab", "parents": [ "0888cf1821d6622fd623db31000be19b9365f81c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Aug 14 18:52:32 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Aug 14 18:52:32 2017 +0000" }, "message": "Revert \"ART: Use proxy ArtMethod\u0027s data_ to store the interface method.\"\n\nBroke a few tests with:\n+dalvikvm64 F 08-14 18:36:32 136697 136697 art_method-inl.h:392] Check failed: interface_method-\u003eGetDeclaringClass()-\u003eIsAssignableFrom(GetDeclaringClass()) \n\n\nThis reverts commit 0888cf1821d6622fd623db31000be19b9365f81c.\n\nChange-Id: Idc93b6686392f9dfeca3bc11bd22104948fd9fd3\n" }, { "commit": "0888cf1821d6622fd623db31000be19b9365f81c", "tree": "fc1e251066fa30ffd475b8a51c15f17bc976b13f", "parents": [ "f9fd236b047a4851f24a3829acfd7e3340676305" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jul 28 12:55:17 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Aug 14 14:06:49 2017 +0100" }, "message": "ART: Use proxy ArtMethod\u0027s data_ to store the interface method.\n\nThis immensely simplifies the interface method retrieval\nand removes one dependency on dex_cache_resolved_methods_.\nWe may later consider removing that member if we deem the\nmemory savings worth the performance impact.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --host --jit\nChange-Id: Id76349c69e4c4dea4e3b297bd504db8f98f1b7cc\n" }, { "commit": "dbd05fe1a6ed2c3e23c9f6b372dd439ad59e777b", "tree": "81968f4f0a16cb37b57933be5cea6542795488a1", "parents": [ "1d2d8b1b6b359f5f476ef78662baa4b8782be530" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Aug 10 11:41:35 2017 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Aug 10 11:57:53 2017 +0100" }, "message": "Revert recent JIT code cache changes\n\nFlakiness observed on the bots.\n\nRevert \"Jit Code Cache instruction pipeline flushing\"\nThis reverts commit 56fe32eecd4f25237e66811fd766355a07908d22.\n\nRevert \"ARM64: More JIT Code Cache maintenace\"\nThis reverts commit 17272ab679c9b5f5dac8754ac070b78b15271c27.\n\nRevert \"ARM64: JIT Code Cache maintenance\"\nThis reverts commit 3ecac070ad55d433bbcbe11e21f4b44ab178effe.\n\nRevert \"Change flush order in JIT code cache\"\nThis reverts commit 43ce5f82dae4dc5eebcf40e54b81ccd96eb5fba3.\n\nRevert \"Separate rw from rx views of jit code cache\"\nThis reverts commit d1dbb74e5946fe6c6098a541012932e1e9dd3115.\n\nTest: art/test.py --target --64\nBug: 64527643\nBug: 62356545\nChange-Id: Ifa10ac77a60ee96e8cb68881bade4d6b4f828714\n" }, { "commit": "646d6388758db566eb1302baf274e538e5bdb8b9", "tree": "72d192b2120450589c40d1462b5ff373efd2cb69", "parents": [ "18e91619f38349e29367567a70f2f974a8cdaf67" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Aug 09 10:50:00 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Aug 09 13:23:41 2017 +0100" }, "message": "Move deopt and jit logs from LOG(INFO) to VLOG(jit).\n\nThey served their purposes. Now stop spamming logcat.\n\nTest: test.py\nChange-Id: I53c961a6ad95ea891f2f09b7e11e7535da4fdaff\n" }, { "commit": "8bb1786f09ebab4e0b93172b45425d5b4940cbaa", "tree": "c9a1d6e5554721cada972251768c63bd023ffca6", "parents": [ "c8a8a56d20f40ce9d024b5198184f7627e520aa7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Aug 03 17:42:37 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Aug 03 17:42:37 2017 +0000" }, "message": "Revert \"Flush icache using rw to r transition\"\n\nFailures on armv7 bots.\n\nBug: 62356545\n\nThis reverts commit c8a8a56d20f40ce9d024b5198184f7627e520aa7.\n\nChange-Id: I62adc6ee6536f4c567825bd92424d4b2b72ec9be\n" }, { "commit": "c8a8a56d20f40ce9d024b5198184f7627e520aa7", "tree": "4c63469c3411617622f2b598b642400f4e8b8697", "parents": [ "d335e7b4d034bc8b69efde641779c6e1cd7bba42" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Aug 02 10:40:20 2017 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Aug 02 10:40:20 2017 -0700" }, "message": "Flush icache using rw to r transition\n\nRather than using mprotects that require executable permissions, attempt\nto use just write permission toggle.\n\nBug: 62356545\nTest: make -j 40 test-art-host\nChange-Id: I262855d110e27d5de2f6df517752757cd53d09cb\n" }, { "commit": "06c42a571358b5e5adb69104b183af8f32f4c07d", "tree": "7b218fdb67fef74cdcbc2e3665757485ef89918b", "parents": [ "7f14c2ec37c70010d99cab6806d85018df56c555" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 26 14:17:14 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 31 07:38:43 2017 -0700" }, "message": "ART: Move openjdkjvmti to art/\n\nMove libopenjdkjvmti out of the runtime directory. Let\u0027s not\npollute the runtime library.\n\nTest: m test-art-host\nChange-Id: Idb6b9cebcd61777bd3200437a2ae584a63a4a341\n" }, { "commit": "c1ac9ee48ddf483ec770b424d63de4982262ea33", "tree": "f0fe2958793a2eccef85c40ac94d056b046ca7de", "parents": [ "aed968d130d62d23cfe7f6051ca72c6a84fe1510" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 24 22:35:49 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 25 11:15:09 2017 -0700" }, "message": "ART: Move CHA to ClassLinker\n\nMove the ClassHierarchyAnalysis singleton to ClassLinker. Hierarchy\nanalysis is a link-time effort and belongs to class-loading instead\nof the global Runtime object.\n\nAlso clean up to use a unique_ptr, and add null checks to allow\nbypassing the analysis.\n\nBug: 34193647\nTest: m test-art-host\nChange-Id: I14ef536e37eb19c0dede372ffd68a826c482cf71\n" }, { "commit": "ff80dfc82a3e5177d45099c090e33f1a060f9152", "tree": "9ee750c2766161d310a625450ada830bc91230b2", "parents": [ "9710fbaea9022149887da4459b47011a803c76fb", "56fe32eecd4f25237e66811fd766355a07908d22" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Jul 25 08:23:46 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jul 25 08:23:46 2017 +0000" }, "message": "Merge \"Jit Code Cache instruction pipeline flushing\"" }, { "commit": "56fe32eecd4f25237e66811fd766355a07908d22", "tree": "7c2c75d54edf0865598c106cb013f8c3794bd767", "parents": [ "84b65e977302e1cf16d188636c22c164c7793554" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jul 21 11:42:10 2017 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Jul 24 17:29:40 2017 +0100" }, "message": "Jit Code Cache instruction pipeline flushing\n\nRestores instruction pipeline flushing on all cores following crashes\non ARMv7 with dual JIT code page mappings. We were inadvertantly\ntoggling permission on a non-executable page rather than executable.\n\nRemoves the data cache flush for roots data and replaces it with a\nsequentially consistent barrier.\n\nFix MemMap::RemapAtEnd() when all pages are given out. To meet\ninvariants checked in the destructor, the base pointer needs to be\nassigned as nullptr when this happens.\n\nBug: 63833411\nBug: 62332932\nTest: art/test.py --target\nChange-Id: I705cf5a3c80e78c4e912ea3d2c3c4aa89dee26bb\n" }, { "commit": "17272ab679c9b5f5dac8754ac070b78b15271c27", "tree": "bb7e52a01a6a28e4906c4f1238079b4478087a7e", "parents": [ "84b65e977302e1cf16d188636c22c164c7793554" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jul 21 14:32:52 2017 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Jul 24 13:11:35 2017 +0100" }, "message": "ARM64: More JIT Code Cache maintenace\n\nMove implementation closer to ARM\u0027s sample code.\n\nTest: art/test.py --target\nChange-Id: Ic205c700528f45afc281fb20e9336c9737d5f409\n" }, { "commit": "3ecac070ad55d433bbcbe11e21f4b44ab178effe", "tree": "565803b40b7f8be3483c502e1bc6809974c4afee", "parents": [ "24f4f7956edeffb1ae41809e09f14b6d5e7875c8" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Jul 20 15:28:44 2017 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Jul 20 16:22:47 2017 +0100" }, "message": "ARM64: JIT Code Cache maintenance\n\nCache maintenance operations on ARM64 can cause segmentation faults if\naddresses are not backed by physical RAM. Some Linux kernel verions\nhave a bug that exposes these faults to user-land which crashes\nART. Some of the test devices used by the ART buildbots have this\nissue.\n\nThis CL explicitly faults in memory relevant addresses before\nperforming cache maintenance operations.\n\nTest: art/test.py --target --64 --r\nChange-Id: Id7390c14f75c13b3fbbbbb742afeb9ffef9a3801\n" }, { "commit": "dcc6617d48692ba7bf19426d767e9d094050fdb4", "tree": "d14aee12364040ff79acec4f92693863a47cb226", "parents": [ "57fe2725629681c8a68f6ba5b58d87304ca3f0b1", "520dadff2190a5bcdb48838dbcccddc8fbbf0c97" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jul 19 16:41:41 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jul 19 16:41:41 2017 +0000" }, "message": "Merge \"Unbreak Odroid measurements.\"" }, { "commit": "520dadff2190a5bcdb48838dbcccddc8fbbf0c97", "tree": "6e4eda47f0e258bf807b5ce0a62cb063b9f0407c", "parents": [ "c5b1b067fb91c10c75dd0e6dbfd91bebe74347d5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 19 15:33:11 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 19 15:34:09 2017 +0100" }, "message": "Unbreak Odroid measurements.\n\nOdroid run Linux/target, which doesn\u0027t support ashmem.\n\nTest: test.py\nChange-Id: Icd92d2a87ea9987c9580037ef195f6e63641283d\n" }, { "commit": "43ce5f82dae4dc5eebcf40e54b81ccd96eb5fba3", "tree": "32775c9d2c16b91a2899dab318d54d9ad6e3c5b9", "parents": [ "c5b1b067fb91c10c75dd0e6dbfd91bebe74347d5" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jul 19 10:34:27 2017 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jul 19 13:26:02 2017 +0100" }, "message": "Change flush order in JIT code cache\n\nFlush the data cache before flushing the instruction when dual mapped.\n\nBug: 62356545\nTest: art/test.py -j4 --jit --64 -r --verbose -t test-art-target-run-test-debug-prebuild-jit-no-relocate-ntrace-gcstress-checkjni-picimage-npictest-ndebuggable-no-jvmti-916-obsolete-jit64\nTest: Using an Android N image and testing on device from master-art\nChange-Id: Ia73da7449cc3a40be0128825233086924eca90a5\n" }, { "commit": "d1dbb74e5946fe6c6098a541012932e1e9dd3115", "tree": "7c3ad5ffd888ea97ee24e255bccb7c1a7acf4466", "parents": [ "2f0ac4fb4486e7d9e5c1545d45a2b9b818a80dc3" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Jul 17 11:20:38 2017 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jul 18 11:38:13 2017 -0700" }, "message": "Separate rw from rx views of jit code cache\n\nCreate two separate mappings of the ashmem region used for the JIT code\ncache. These two mappings have unrelated addresses.\n\nBug: 62356545\nTest: make -j test-art-target\nChange-Id: If176552e6d72ef71e17c96994901f3f3baac44f6\n" }, { "commit": "a5dcdf346158d47bfe80d8adb29b1b020cede286", "tree": "50e56b3445dc8e4724d6707ffecd23f9dae96e31", "parents": [ "942c597b004aa328cf291a78392b405ef5d5f040", "bbe3a5efcdfb2fb15a4e5f724cc323ae0ada5111" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 15 20:21:23 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jun 15 20:21:24 2017 +0000" }, "message": "Merge \"Refactor some profiling info functions\"" }, { "commit": "bbe3a5efcdfb2fb15a4e5f724cc323ae0ada5111", "tree": "65fd149e1159674681d866e47e485d85b8464334", "parents": [ "d58932f98a673c9d687863a7efafb2da168063a2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 13 16:36:17 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 15 11:45:06 2017 -0700" }, "message": "Refactor some profiling info functions\n\nDeleted profile_compilation_info-inl.h\n\nAdded a new helper class MethodHotness to reduce the number of\nrequired getters for reading method hotness, startup, and post\nstartup booleans.\n\nTest: test-art-host\nBug: 62040831\n\nChange-Id: I799a38e2bea6177ba5816c54524f4ccacedba772\n" }, { "commit": "e4deaf3ccdfaf33f0b4526e9963612bfa2dc79e8", "tree": "6af8ed52f680e3f65ffed43d52353c34803b668a", "parents": [ "651ba599961ff0cdc36e83d3d58b1744d37ee9f7" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jun 09 15:27:15 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 14 14:44:53 2017 -0700" }, "message": "ART: JIT code cache allocation cleanup\n\nRequest the data cache with kProtData instead of kProtAll. It isn\u0027t\nnecessary nor intended to be executable. The tail remap for the code\ncache will make all required parts executable, if possible.\n\nAlso use a unique_ptr to plug a leak on failure.\n\nBug: 36138508\nTest: m\nTest: m test-art-host\nTest: Test SANITIZE_LITE\u003dtrue build with wrap.system_server \u003d asanwrapper\nChange-Id: Id9e2e51bc8d7a29db99c406cd1e9ef6bbb8d444c\n" }, { "commit": "d38b67c9892b603f57a6b51a1b9d7f4e610158aa", "tree": "4bb95ce9f7733f8b9e54b6e80eaeb3751b4e78a8", "parents": [ "af14a86fb40f83b3d76185dd8bd09c1e7c409d2f", "eced692a34d2cf63d584c703704592984cc50394" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jun 14 05:50:39 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 14 05:50:42 2017 +0000" }, "message": "Merge \"ART: Add JIT cache race test\"" }, { "commit": "fbb22de21c30b1233aa32b56b0c92b2bd0ec2b47", "tree": "c4dca0c88a5526fceb057e7255f614c1c8a9a2c0", "parents": [ "f052aa7268f656f97099185ac6b51eeacd78a97d", "88f50b13aca3b29b4be144b2b8c14bc95506d240" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Jun 10 13:24:45 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Sat Jun 10 13:24:48 2017 +0000" }, "message": "Merge \"Don\u0027t reset to a level above warmup threshold.\"" }, { "commit": "b2d18fa4e33ca119654ced872c70fe198b0b2db5", "tree": "9f0e3ac72b9b7b08794e1ecf458620ba19437465", "parents": [ "58794c5c23f46a7476a58e5a10dbeebb6321aa90" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jun 06 20:46:10 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jun 09 11:36:45 2017 -0700" }, "message": "ART: Clean up asm_support.h\n\nThe includes are only ever really needed for tests. Factor out\ngeneration of the test function into asm_support_check.h\n\nFix up missing includes, mainly intern_table.h.\n\nTest: m\nTest: m test-art-host\nChange-Id: I435199e6211e368be0a06c80d8fa95b9593aca31\n" }, { "commit": "88f50b13aca3b29b4be144b2b8c14bc95506d240", "tree": "19ee500bccff6b2d4e1b5b94a9e7071cec54240e", "parents": [ "fed9cb178be878f4e5ff862ac78ae2cf5ad305d3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jun 09 16:08:47 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jun 09 16:10:36 2017 +0100" }, "message": "Don\u0027t reset to a level above warmup threshold.\n\nOtherwise, we could be in a situation where a profiling info\nnever gets allocated.\n\nTest: 655-jit-clinit\nChange-Id: I7eebab66e841784a8ce5cd32765df9fc7376cea8\n" }, { "commit": "eced692a34d2cf63d584c703704592984cc50394", "tree": "97240d0eae41b0b3596d92c17419f2dc3ae7b674", "parents": [ "cff1b21b3e19c5d3a2d726fdb60dacd7de2a6f0d" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Jun 01 10:54:28 2017 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Jun 08 13:08:34 2017 +0100" }, "message": "ART: Add JIT cache race test\n\nAdds a test for the JIT code cache that attempts to trap races between\nthreads generating code and executing it.\n\nBug: 38417984\nTest: run-test --jit 707\nChange-Id: I408b2680b1d266ebe624d6e39113f0261d538e8a\n" }, { "commit": "5d08fcc21a14190c4dba3028a3e35d281afdd016", "tree": "c42de65ced190778342d9b022e400f4a7f9d346b", "parents": [ "57cf00bde719ddc84a6015b107b90a20169e3099" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 05 17:56:46 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 05 20:28:16 2017 -0700" }, "message": "ART: More header cleanups\n\nPull out more dependencies through forward declarations.\n\nTest: m test-art-host\nChange-Id: I7d86726928937f788b956ec9eac91532d66d57ae\n" }, { "commit": "513061a792b22c417c938d31c19581390709561c", "tree": "80b4fdce03711170626aa5640d07b07de4a326a1", "parents": [ "38c4ae5f4c5a033b7a7441032f39ea58f5772d4c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 01 09:17:34 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 01 12:59:13 2017 -0700" }, "message": "ART: Clean up thread.h and thread_list.h\n\nRemove dependency on stack.h and gc_root.h. Remove unused object\ncallbacks include. Factor out ManagedStack into its own set of files.\nFix up users of transitive includes.\n\nTest: m test-art-host\nChange-Id: I01286c43d8c7710948c161b1348faabb05922e59\n" }, { "commit": "f044c229e12f1d49b7024ab5d7353b2d83335501", "tree": "010bc946819a5190b19fbf55f50bef75ef789991", "parents": [ "854461a4cd5e4a38debe3616e12b52fe7f160782" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 31 15:27:54 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 31 20:46:50 2017 -0700" }, "message": "Add access flag for previously warm methods\n\nWe want to know if the method was warm instead of just having a non\nzero counter. This is required if we want to not compile all of the\nstartup methods, but still compile warm methods.\n\nTest: test-art-host ART_TEST_JIT\u003dtrue\n\nBug: 62200509\n\nChange-Id: I6e04866f39f970b04b47342b7af5ed474e1f4172\n" }, { "commit": "00286db781e56d676d6bfb1ab22dc22fbaa96a41", "tree": "61f4cf0bb8c90f9e8551119ebb1a672db2040566", "parents": [ "413d0bec8015525ec15b57ced821b4aa049cf067" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue May 30 16:53:07 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue May 30 16:53:07 2017 -0700" }, "message": "Reset JIT method counters to 0 instead of 1 when clearing.\n\nWhen the counters need to be cleared, the count should reset to 1\ninstead of 0. The counters may be reset during startup, and startup\nmethods can be lost by resetting the counters to 0.\n\nBug: 62200509\nTest: mm test-art-host\nChange-Id: I8936631d3bab369c6a14acf76b20e7245de14789\n" }, { "commit": "2d7ed749be6772130cc9fa5658732951701fc450", "tree": "b66f6931c569e59f7a41535ed99a5aa37b378174", "parents": [ "1f51ec0e1a80c4146793d2f853c2c7408073fe7c", "352b17af5263b27cad7a1dbf2351f9cb71d6af58" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri May 26 10:41:42 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri May 26 10:41:44 2017 +0000" }, "message": "Merge \"Flush data cache after updating the JIT table.\"" }, { "commit": "dbddc22f5dc2d1ff4d4783fbd66c27812f4980d1", "tree": "2a0a8efa1c2630e57ab48ab2de171f2847ff282f", "parents": [ "a559fa1b0d6c276dde2cdc707de1acd4950f7190" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 24 12:04:13 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 25 16:59:44 2017 -0700" }, "message": "Refactor profiles to use TypeReference instead of ClassReference\n\nRefactor type reference into runtime and use it for profiles.\nClassReference was just duplicated code since it wasn\u0027t even using\nthe class def indexes.\n\nTest: test-art-host\n\nBug: 62040831\nChange-Id: Ia92f21c0e85c00321f52c97bb7a90158d882849b\n" }, { "commit": "352b17af5263b27cad7a1dbf2351f9cb71d6af58", "tree": "2c3cd2c36727cd4ba994457a29028f9ee9086272", "parents": [ "ec89dfa25ae9e718730d775022aef505f7363d3a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu May 25 12:54:31 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu May 25 13:00:45 2017 +0100" }, "message": "Flush data cache after updating the JIT table.\n\nOtherwise, other cores could see an incomplete table when executing\nthe new code.\n\nbug: 37949368\nTest: test.py --jit\nChange-Id: I29fb62f4a13c6db509a1f5719b05ee9297a403ba\n" }, { "commit": "a39fd980000d10ee0b3a49555cc8bcbbfacde943", "tree": "cad6936a2b618ec31fc7d5b724ff8f433b373e23", "parents": [ "0b183409d8dd2ed5a720eb28ad77c46bccd09247" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu May 18 10:15:52 2017 -0700" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue May 23 13:29:45 2017 -0700" }, "message": "Record inline caches only for the strictly hot methods\n\nWarm methods might have incomplete inline caches which could cause\nunnecessary deoptimizations.\n\nTest: m test-art-host\nBug: 38426301\nBug: 38412648\n\nChange-Id: I3023a2d462cb1bd291f7ea63bcf363a75d8f20a9\n" }, { "commit": "5504aef8c13ad1cc8825c108836a32ba8e54ce6c", "tree": "0b11e66eb1ed9597b77908fb02673adec877e2ed", "parents": [ "4632f5be49f4a4a9be981284fdbbc8a181a8d627", "5629d2deda9bbdf53a229922dac23b88e286d6bd" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed May 17 16:20:05 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 17 16:20:07 2017 +0000" }, "message": "Merge changes Ic48ccda9,Ibc0a29c9\n\n* changes:\n ART: Move JIT code cache sentinel to sentinel fault page\n ART: Reserve sentinel fault page\n" }, { "commit": "39e673811d73ac09a822d9ee0082bba5d8e8cbad", "tree": "2c298cbabbd476b103da0269e5cbb68de18faf61", "parents": [ "ed00ddb614d68559931d831cb74d3ba8f01599c9" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon May 15 19:26:38 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon May 15 19:26:38 2017 -0700" }, "message": "ART: Fix iterator invalidation\n\nErasing an item of a map invalidates the iterator.\n\nBug: 38324641\nTest: m test-art-host\nChange-Id: I9f7499eb4a4b49e4fcd6c99d170d9b681adfd9b0\n" }, { "commit": "5629d2deda9bbdf53a229922dac23b88e286d6bd", "tree": "6edd6558115881b1df763faf90165f1535e293d7", "parents": [ "2ac67d526eb348019433138b04eb1473d45cb0f4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon May 15 16:28:13 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon May 15 18:21:49 2017 -0700" }, "message": "ART: Move JIT code cache sentinel to sentinel fault page\n\nUse an address in the fault page to get more expressive fault\naddress.\n\nTest: m test-art-host\nChange-Id: Ic48ccda976eeb965774c9356242a0151ccbc2a10\n" }, { "commit": "6ca115b5fa185a2cc162a453f425efd87d8f38bd", "tree": "6c6ccb5ec5094a54a8c558707f4b2a305c90b87c", "parents": [ "ddc38fe3e5618e3922ecc445193dacb2f39ef736" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 10 15:09:35 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu May 11 15:46:10 2017 +0100" }, "message": "Use a sentinel when clearing JIT table entries.\n\nThis can help diagnosing crashes we see.\n\nbug: 38128052\nbug: 37949368\nTest: test.py --jit\nChange-Id: I7bf70785e8bd93fb0575ac64b1fb5bcb7d462b10\n" }, { "commit": "4e92c3ce7ef354620a785553bbada554fca83a67", "tree": "42029deff4d3ba7f89b5fdbf79ff410da575f431", "parents": [ "549844e9ccf432d1396b19af890eedb602b8ba04" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 08 09:34:26 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 10 09:27:33 2017 +0100" }, "message": "Add runtime reasons for deopt.\n\nCurrently to help investigate. Also:\n1) Log when deoptimization happens (which method and what reason)\n2) Trace when deoptimization happens (to make it visible in systrace)\n\nbug:37655083\nTest: test-art-host test-art-target\nChange-Id: I0c2d87b40db09e8e475cf97a7c784a034c585e97\n" }, { "commit": "f6d4668c42933e2f85ddbc94e276c49db4e2b1dd", "tree": "3343addb8cde45eb4070d4876f26bef1002248dc", "parents": [ "0bbb750f6ff9a52bc4649995fc78ebea87e31dd1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 28 17:41:45 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 24 14:02:54 2017 +0000" }, "message": "Improvements in the Inliner.\n\n- Change from a depth limit to a total number of HInstructions\n inlined limit. Remove the dex2oat depth limit argument.\n- Add more stats to diagnose reasons for not inlining.\n- Clean up logging to easily parse output.\n\nIndividual Ritz benchmarks improve from 3 to 10%.\n\nNo change in other heuristics. There was already an instruction budget.\nNote that the instruction budget is rarely hit in the \"apps\" I\u0027ve tried\nwith.\n\nCompile-times improve from 5 to 15%.\nCode size go from 4% increase (Gms) to 1% decrease (Docs).\n\nbug:35724239\ntest: test-art-host test-art-target\n\nChange-Id: I5a35c4bd826cf21fead77859709553c5b57608d6\n" }, { "commit": "cbcedbf9382bc773713cd3552ed96f417bf1daeb", "tree": "58eed766a69ddddbc41d21624244e14547fa1dff", "parents": [ "ef62210a1466c133056c4bfdae9f8f5417065eff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 12 22:24:50 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 16 10:20:34 2017 -0700" }, "message": "Add method info to oat files\n\nThe method info data is stored separately from the code info to\nreduce oat size by improving deduplication of stack maps.\n\nTo reduce code size, this moves the invoke info and inline info\nmethod indices to this table.\n\nOat size for a large app (arm64): 77746816 -\u003e 74023552 (-4.8%)\nAverage oat size reduction for golem (arm64): 2%\n\nRepurposed unused SrcMapElem deduping to be for MethodInfo.\nTODO: Delete SrcMapElem in a follow up CL.\n\nBug: 36124906\n\nTest: clean-oat-host \u0026\u0026 test-art-host-run-test\n\nChange-Id: I2241362e728389030b959f42161ce817cf6e2009\n" }, { "commit": "13439f0c4769a0768cf1bbaa7b3f2f9ee5a317c5", "tree": "a5d52373571fad4249544a7b27c8009233d69f9d", "parents": [ "74f94dd8edb4a6070f6f7bfef6d28a8b4a21fc30" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Feb 21 01:17:21 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Mar 09 17:14:02 2017 -0800" }, "message": "Use offline inline caches during AOT compilation\n\nAlso:\n- extend the testing script to understand profile when --profile is\npassed\n- filter inline cache types which are not loaded by the caller class\nloader\n\nTest: m test-art-host-run-test-638-checker-inline-caches\nBug: 32434870\nChange-Id: Ifcc27b3cebc79b84617412aaae64a73324151b55\n" }, { "commit": "589e71eb71477603c88b14118b2d3a33508c3c79", "tree": "95d88c7c8ba3a9b2009361d61164cef6a6339308", "parents": [ "d9f647ac9a5cb0c6f112332d584756251c2cfb9f" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Mar 03 16:05:05 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon Mar 06 12:57:30 2017 -0800" }, "message": "Encode inline caches with missing types in the profile\n\nNot all runtime types can be encoded in the profile. For example if the\nreceiver type is in a dex file which is not tracked for profiling its\ntype cannot be encoded.\n\nPreviously we would just skip over these types but that can lead to\nencode a polymorphic inline cache when in fact it should be megamorphic.\n\nWith this CL, inline caches for which types are missing are marked in\nthe profile with a special bit, kIsMissingTypesEncoding.\n\nAlso, extend profman to understand text lines which specify an inline\ncache with missing types.\n\nTest: test-art-host\n\nBug: 35927981\nBug: 32434870\nChange-Id: I34528a39c227f3133771fd4454701c1ddc234f40\n" }, { "commit": "da3355a9af91e9825d9554658bf2020cbbe08e18", "tree": "2aa451b4ae91f0d0c792437b187945c4271dd70b", "parents": [ "7ac79b99c09fecb33af43e435008868f48406726", "3b1a7f40ad69ab5332d13ed0abf0a3a1fa7f2a9e" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 24 09:02:57 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 24 09:02:58 2017 +0000" }, "message": "Merge \"Update entrypoint directly when doing JIT code cache GC.\"" }, { "commit": "3b1a7f40ad69ab5332d13ed0abf0a3a1fa7f2a9e", "tree": "48f2ad70be640c4f2088d0548716b5070f493997", "parents": [ "d3cb7fae3acdef02dbb67729353ba4ade8944630" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 22 10:21:00 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 23 12:55:33 2017 +0000" }, "message": "Update entrypoint directly when doing JIT code cache GC.\n\nOtherwise, the Instrumentation::UpdateMethodsCode might access\nthe declaring class while the GC is concurrently processing\nreferences.\n\nbug:35598395\ntest: run-libcore-tests.sh --mode\u003dhost --variant\u003dX64\nChange-Id: I0f059b1823e81672d43cf1f7eed64482f8927232\n" }, { "commit": "e21806f053c1a001738b121e7f7807f6cb79b97a", "tree": "360e84befa7ac35783a712368d66c33362bba3c8", "parents": [ "0def68d5f201b0a6421cabedcf951c7927f4fd50" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Feb 22 11:49:43 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Feb 22 11:49:43 2017 -0800" }, "message": "Add more comments in GetProfiledMethods\n\nExplain why we are looking in the method dex file for array types.\n\nTest: m test-art-host\nChange-Id: Ib94475f79844968591f72f46a2c714f9971778ba\n" }, { "commit": "4ca70a3102c10e2e77e97fc3b9180f00304ed29b", "tree": "3caed68e8a27b910779eec62614ff48151d9490f", "parents": [ "2fee212df6f688295475f3b2ddee4f2db8db27a2" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Feb 21 16:22:24 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Feb 21 17:02:19 2017 -0800" }, "message": "Fix handling of array types when extracting the offline inline caches\n\nTest: m test-art-host\n regression test is being worked out\nBug: 35621933\n\nChange-Id: If762db76cc0493db8c8babab62035e915b660da5\n" }, { "commit": "940eb0c00ef531dd9a0a68dbd61e377832e81eb4", "tree": "c65e43a702bb781b7a2f4ebedfb6b3d131fd8312", "parents": [ "9fb10fb39bcb3d9a4dc7e16f8c1d38dcc112639c" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon Jan 30 19:30:44 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Feb 16 13:52:56 2017 -0800" }, "message": "Add inline caches to offline profiles\n\nAdd support for inline caches in profiles:\n- extract inline caches from the jit cache when the profile saver\nqueries the hot methods\n- bump profile version to support the new data\n- add new tests\n- inline caches are only supported for same-apk calls (including\nmultidex)\n\nTest: m art-test-host-gtest-profile_compilation_info_test\nBug: 32434870\nChange-Id: I38b4ca0a54568d2224765ff76023baef1b8fd1a2\n" }, { "commit": "6def7024f36c39ea79b522c8c68cb253355dda8f", "tree": "9cca19acfe065dbc5e0d763d65d0c56a2b0387f4", "parents": [ "aff5505d85612621bd112d9af1ba4cc31a4619d1", "eee0bd448ec057d3f224895ddb868786758eeb5b" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Feb 14 19:11:26 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Feb 14 19:11:28 2017 +0000" }, "message": "Merge \"Revert^6 \"Make class redefinition work with native methods on stack.\"\"" }, { "commit": "eee0bd448ec057d3f224895ddb868786758eeb5b", "tree": "32283e1b1ccb97bced2999a364cf21361c8d07ac", "parents": [ "02b2349b1cd2a78e86b1a7542f8330e6c3aaeb35" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 14 15:31:45 2017 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 14 09:28:19 2017 -0800" }, "message": "Revert^6 \"Make class redefinition work with native methods on stack.\"\n\nWe were incorrectly trying to obtain the profiling information of a\nnative method.\n\nThis reverts commit 02b2349b1cd2a78e86b1a7542f8330e6c3aaeb35.\n\nReason for revert: Fixed test failure with jit configurations\n\nTest: ART_TEST_JIT\u003dtrue mma -j40 test-art-host\n\nChange-Id: Ic7112104aec64e597d2df80db5dc2a505d2cc2dd\n" }, { "commit": "b93ce1853e1a28bbd6ca992c5b646aa2d2699892", "tree": "4d33276a5d969385124d1745f7afbe23d760f29d", "parents": [ "806ac631e53f12061cb0ae7640aa9cd0dd79243d" ], "author": { "name": "Kevin Brodsky", "email": "kevin.brodsky@linaro.org", "time": "Thu Dec 15 14:23:09 2016 +0000" }, "committer": { "name": "Kevin Brodsky", "email": "kevin.brodsky@linaro.org", "time": "Fri Feb 10 15:48:16 2017 +0000" }, "message": "Clarify comment on JIT cache flush issue\n\nThe original fix (#304947) is needed, but the comment was suggesting a\nhardware cause and a link with A53 errata. In all likelihood neither\nof these is true, and the issue is actually caused by a kernel issue\naffecting many Qualcomm devices (like those using the msm tree).\n\nChange-Id: Ie750e6fd9eb7130982d184c22917b5c3ea4beed0\n" }, { "commit": "1ebe4fe1a6cb0835197ee3a848d554b79ca9d367", "tree": "69e01b73d1b5eccc37cf19a1daea89052de06a94", "parents": [ "44790fe8b33c7a80da2e77787d5f8d4d4549114b" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Jan 30 14:57:11 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Feb 01 15:39:49 2017 -0800" }, "message": "Fix creating self-recursive obsolete methods.\n\nWe were using recursive loading of the current art method which was\nmaking us miss obsolete methods in some cases.\n\nWe could also end up checking the wrong method when walking the stack.\n\nWe also add tests for recursive obsolete methods in general.\n\nBug: 34815470\n\nTest: mma -j40 test-art-host\nTest: ART_TEST_JIT\u003dtrue mma -j40 test-art-host\nChange-Id: I522fd4cac4e3f9d35d03b128bad6d6971cfe6c4a\n" }, { "commit": "433b79a9130df5f2375b9d3d7518c1f3fb84a467", "tree": "dddc053886464b99be29b9ca69039d74d49605ca", "parents": [ "81356645157af44152c7b7db383596b5cf3479b5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:54:45 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:54:45 2017 +0000" }, "message": "Revert \"Revert \"Make --debuggable rely on JIT code.\"\"\n\nFixed by https://android-review.googlesource.com/#/c/330165/\n\nThis reverts commit 81356645157af44152c7b7db383596b5cf3479b5.\n\nChange-Id: Ifb74e1cc90ab6dea621f7f54a00b540d6ccd0cf6\n" }, { "commit": "81356645157af44152c7b7db383596b5cf3479b5", "tree": "59aca3792abfa47c7424e4dac13248651e4662b2", "parents": [ "a0619e25aacf8b8074132f4951f75fdbcfd42925" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:18:02 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:19:28 2017 +0000" }, "message": "Revert \"Make --debuggable rely on JIT code.\"\n\nBreaks 137-cfi with ART_TEST_RUN_TEST_DEBUGGABLE\u003dtrue\n\nThis reverts commit a0619e25aacf8b8074132f4951f75fdbcfd42925.\n\nbug: 28769520\nChange-Id: Ifd7b8fc7c9b72c0a523fd57c4b1b80edd3547caa\n" }, { "commit": "a0619e25aacf8b8074132f4951f75fdbcfd42925", "tree": "dddc053886464b99be29b9ca69039d74d49605ca", "parents": [ "ca21dc47adeed92a15a9d3fd090bdd0e6654679c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 20 13:57:43 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 12:41:52 2017 +0000" }, "message": "Make --debuggable rely on JIT code.\n\nRemoves -Xfully-deoptable in the process, which was added as a\ntemporary workaround until this CL.\nPartial revert of https://android-review.googlesource.com/#/c/302232/\n\nMakes things consistent with existing infrastructure:\n\n- Parse the --debuggable from the compiler options, just like\n --compiler-filter.\n\n- Add DEBUG_JAVA_DEBUGGABLE, passed by the zygote, for debuggable apps.\n\n- Java debuggable now solely relies on JIT for simplicity.\n\n- Debugging under userdebug for non-java-debuggable apps is still\n best effort.\n\nTest: test-art-host, jdwp\n\nbug: 28769520\nChange-Id: Id0593aacd85b9780da97f20914a50943957c858f\n" }, { "commit": "dba61481035b7944173181ec9ee02aea41dd0e29", "tree": "8629775e657cc8968e2f8fa2af6256c7a58c7dd4", "parents": [ "cb2ce333ee2d1053919fe0acc390369bea3ad77e" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 21 08:20:29 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 21 11:43:14 2016 -0800" }, "message": "Revert \"Revert \"Revert \"Revert \"Basic obsolete methods support\"\"\"\"\n\nA GetDeclaringClass()-\u003eGetDexCache() got inserted during the\nmerge/review process meaning that we would try to access incorrect\ndex-cache in obsolete methods in some situations.\n\nAlso when using tracing we would loop forever (or at least until an\nOOM error) in test 916 due to tracing forcing InterpretOnly mode\nmeaning methods would never be jitted.\n\nBug: 32369913\nBug: 33630159\n\nTest: ART_TEST_TRACE\u003dtrue \\\n ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\n\nThis reverts commit f6abcda293b115a9d7d8a26376ea2dcf2d1dc510.\n\nChange-Id: I0773bfcba52e3cd51a83be815c6a50c189558f48\n" }, { "commit": "f6abcda293b115a9d7d8a26376ea2dcf2d1dc510", "tree": "a30ce596848d14c95a73449dbf2bfccf26a510b9", "parents": [ "ce77fc0e7f60a15354bb20c356537cbf8b53b722" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 21 09:26:18 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 21 09:26:18 2016 +0000" }, "message": "Revert \"Revert \"Revert \"Basic obsolete methods support\"\"\"\n\nFails in tracing mode\n\nBug: 32369913\nBug: 33630159\n\nThis reverts commit ce77fc0e7f60a15354bb20c356537cbf8b53b722.\n\nChange-Id: I1bdcf6ad467f2e31f9c5d0c3c987b90a4f5efc69\n" }, { "commit": "ce77fc0e7f60a15354bb20c356537cbf8b53b722", "tree": "b4301967c423ab190a4274409d99a3d6e23f0e8a", "parents": [ "d54f43ca39dfa92f08c2d760123f185f0f65fb86" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 01:05:52 2016 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Dec 19 14:26:18 2016 -0800" }, "message": "Revert \"Revert \"Basic obsolete methods support\"\"\n\nThis reverts commit b81a9840b44480bfeacd74b8d9f51e06f295411d.\n\nThere were 2 issues with the original CL\u0027s test 916-obsolete-jit that\ncaused it to sporadically fail.\n\nFirst, when checking if we had jitted the function under test in\n916-obsolete-jit we failed to check to see if the function \u0027doCall\u0027,\nwhich is used to work-around bugs in our deoptimization \u0026 compilation\nsystems, had also been jitted.\n\nIn the case where the \u0027sayHi\u0027 function had been jitted but the\n\u0027doCall\u0027 function had not we would (correctly) fail to redefine the\n\u0027Transform\u0027 class since we would not be able to deoptimize the \u0027sayHi\u0027\nfunction since it is under a quick_to_interpreter_bridge (runtime)\nframe.\n\nSecondly, the function Main.isInterpretedFunction was incorrect and\nwould always return false, regardless of the actual state of the\nfunction. This would cause the test to fail as the\nquick_to_interpreter_bridge frame prevented deoptimization of the\nobsoleted function. Usually the warm-up period was enough to make sure\nthe methods were jitted anyway but this was not guaranteed.\n\nBoth of these problems become more likely to occur on systems with\nmore cpu contention such as the buildbots.\n\nTest: stress --cpu 60 \u0026; while ./test/run-test --host --jit 916; do ; done\n\nReason for revert: Fixed bug in test that was causing failures.\n\nOriginal Tests:\n\nTest: ./test/run-test --host 914\nTest: ./test/run-test --host 915\nTest: ./test/run-test --host 916\nTest: mma -j40 test-art-host\nTest: ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64 --no-jit\n\nBug: 32369913\nBug: 33630159\n\nChange-Id: If1a92e47b90965a7dc21c5826185debe62bd1554\n" }, { "commit": "ed015acdd0fbb1b837b3f74e8c5f624956189691", "tree": "b8ec86c504bf665cd59074fae49136dfe7c6c9f5", "parents": [ "9538f9e2a5d03f1b1bc07ebfbd93b61dcf8ad604" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 15 17:58:48 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 15 18:01:52 2016 +0000" }, "message": "Flush JIT data cache when committing code.\n\nOtherwise, executing compiled code may see old data from\nthe data cache.\n\nTest: ART_TEST_JIT\u003dtrue test-art-target on arm/arm64\nTest: run-libcore-tests.sh (especially jsr166) on arm/arm64\nChange-Id: Id037c68897aa0e9ccacd4c5121f3743fb722c6f3\n" }, { "commit": "b7cc593a34896e230aab6d9718239b4f85c2dc63", "tree": "fdad7132bcdf125e8c2a499bf794bc6bcbba76d3", "parents": [ "bdca0a27eb034152afba52f1d499d773b81908df", "b81a9840b44480bfeacd74b8d9f51e06f295411d" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 01:04:03 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Dec 15 01:04:03 2016 +0000" }, "message": "Merge \"Revert \"Basic obsolete methods support\"\"" }, { "commit": "b81a9840b44480bfeacd74b8d9f51e06f295411d", "tree": "d79a3567b272b6eb4fb94d66a537cf92bef455db", "parents": [ "d8936da27b792d1ca02e59c92456a1a53c7b9905" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 00:59:05 2016 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 00:59:05 2016 +0000" }, "message": "Revert \"Basic obsolete methods support\"\n\nThis reverts commit d8936da27b792d1ca02e59c92456a1a53c7b9905.\n\nReason for revert: Some sort of race in JIT\n\nChange-Id: Ibb4e520bb0721d6d7aa2c841a52eb5baff07740e\n" }, { "commit": "6d736d8edb0692ac70044c18045e2fcb3d14c1e5", "tree": "7b7c9973a04d9231af44fc967ebe49f1c835a861", "parents": [ "7a8e16c827efc866d01b5cbca835b18fe29a83ab", "d8936da27b792d1ca02e59c92456a1a53c7b9905" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 14 22:21:27 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Dec 14 22:21:28 2016 +0000" }, "message": "Merge \"Basic obsolete methods support\"" }, { "commit": "d8936da27b792d1ca02e59c92456a1a53c7b9905", "tree": "6a593dd426e7f4a15c651dd3b872f22fb64f09ce", "parents": [ "7c73bf3518eea7544406eadfed6946411c54a267" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Nov 28 16:24:32 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 14 13:01:27 2016 -0800" }, "message": "Basic obsolete methods support\n\nAdd support for executing obsolete methods following redefinitions.\nThis support includes methods that have been jitted. This does not add\nany additional validity checks to our redefinition functions.\n\nNote using work-arounds to ensure 916 works pending some fixes to the\ncompiler, deoptimizer, or both.\n\nTest: ./test/run-test --host 914\nTest: ./test/run-test --host 915\nTest: ./test/run-test --host 916\nTest: mma -j40 test-art-host\nTest: ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64 --no-jit\n\nBug: 32369913\nBug: 33630159\n\nChange-Id: I78ef95f484146f1fb93c37fc50f56575bdab2432\n" }, { "commit": "22384aeab988df7fa5ccdc48a668589c5f602c39", "tree": "daca06adfc92c93017618c3729af54ed40214ba4", "parents": [ "0ee6447c63e354131dec78743ccabcbc964129e3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 22:33:36 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 23:06:38 2016 +0000" }, "message": "Revert \"Revert \"Add kJitTableAddress for HLoadClass.\"\"\n\nThis reverts commit d2d5262c8370309e1f2a009f00aafc24f1cf00a0.\n\nChange-Id: I6149d5c7d5df0b0fc5cb646a802a2eea8d01ac08\n" }, { "commit": "d2d5262c8370309e1f2a009f00aafc24f1cf00a0", "tree": "15b542ac079f30043cd3654cf5d3c40ae3ea34d0", "parents": [ "5b12f7973636bfea29da3956a9baa7a6bbe2b666" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 16:28:54 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 16:28:54 2016 +0000" }, "message": "Revert \"Add kJitTableAddress for HLoadClass.\"\n\nOne test failure after merge.\n\nThis reverts commit 5b12f7973636bfea29da3956a9baa7a6bbe2b666.\n\nChange-Id: I120c49e53274471fc1c82a10d52e99c83f5f85cc\n" }, { "commit": "5b12f7973636bfea29da3956a9baa7a6bbe2b666", "tree": "a2cd41c1d3c09abc594a76af11b7bebc302a2870", "parents": [ "0dd27eb2b51d030866c25dbf8e7bb737eb35a888" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 09 11:26:35 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 11:50:24 2016 +0000" }, "message": "Add kJitTableAddress for HLoadClass.\n\nThis new kind loads classes from the root table associated with\nJIT compiled code.\n\nAlso remove kDexCacheAddress, which is replaced by kJitTableAddress.\n\ntest: ART_TEST_JIT\u003dtrue test-art-host-jit test-art-target-jit\nChange-Id: Ia23029688d1a60c178bf2ffa7463927c5d5de4d0\n" }, { "commit": "f4b94427072a42ee6814aa6e1abc04ced5235355", "tree": "c50d0748fdcc78d53c7cab5bdd9555a922c6cc73", "parents": [ "3bdc662e713289655c744c7ba3d3e5316b276c45" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 05 00:10:09 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 05 00:10:09 2016 +0000" }, "message": "Set the root table size when allocating.\n\nIn order to deallocate it at any point.\n\ntest: ART_TEST_JIT\u003dtrue m test-art-host-run-test-jit\nChange-Id: I36574fd1a34c9d67800a164fa801db7a17c827bf\n" }, { "commit": "063fc772b5b8aed7d769cd7cccb6ddc7619326ee", "tree": "bc165781989087a998721991504e589a7d5b0926", "parents": [ "48d08a4233ee4450b0d5073d41445f9dd1f17191" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Tue Aug 02 11:02:54 2016 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Dec 01 11:15:47 2016 -0800" }, "message": "Class Hierarchy Analysis (CHA)\n\nThe class linker now tracks whether a method has a single implementation\nand if so, the JIT compiler will try to devirtualize a virtual call for\nthe method into a direct call. If the single-implementation assumption\nis violated due to additional class linking, compiled code that makes the\nassumption is invalidated. Deoptimization is triggered for compiled code\nlive on stack. Instead of patching return pc\u0027s on stack, a CHA guard is\nadded which checks a hidden should_deoptimize flag for deoptimization.\nThis approach limits the number of deoptimization points.\n\nThis CL does not devirtualize abstract/interface method invocation.\n\nSlides on CHA:\nhttps://docs.google.com/a/google.com/presentation/d/1Ax6cabP1vM44aLOaJU3B26n5fTE9w5YU-1CRevIDsBc/edit?usp\u003dsharing\n\nChange-Id: I18bf716a601b6413b46312e925a6ad9e4008efa4\nTest: ART_TEST_JIT\u003dtrue m test-art-host/target-run-test test-art-host-gtest\n" }, { "commit": "b84defb1b09803d5a47035e8010ac43984da1d4c", "tree": "081b63ee8a4ce0f06a208e269a4076331d0d6fce", "parents": [ "07e6a7947f3279f66f14f7af045065b5297294c2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 30 16:02:16 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 30 16:02:16 2016 +0000" }, "message": "Do not call IsMarked on null.\n\nThe concurrent collector checks it.\n\nTest: ART_TEST_JIT\u003dtrue test-art-host-run-test-jit\nChange-Id: I02c2462ce1d8f3092f0b2f52f8b6dede8ff4a339\n" }, { "commit": "c9e2fc27c57d4c007d26081a8e60b244ecba8665", "tree": "cd59bdb099bf2342f4400c37cfbf6ea827a56632", "parents": [ "9ebca690fdfb36fe1a689fd32c3d05d999497768", "e51ca8bfa8a193b64901ad214842f213adca92eb" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 30 13:57:30 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 30 13:57:30 2016 +0000" }, "message": "Merge \"Move inline caches GC handling in JitCodeCache.\"" }, { "commit": "e51ca8bfa8a193b64901ad214842f213adca92eb", "tree": "b07b2534edc592d68b5461a3bdb8a31241892d54", "parents": [ "eb9c58e88f8e0f4a0a413efce04b49b183ad23e3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 22 14:49:31 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 29 21:05:51 2016 +0000" }, "message": "Move inline caches GC handling in JitCodeCache.\n\nMake the classes they hold weak references and visit\nthem during SweepJitRoots.\n\nThis fixes the newly introduced deadlock:\n\nThread1:\n1) Lock JitCodeCache lock to create Profiling info for\n ArtMethod m.\n2) m is a copied method, we need to track the actual holder,\n needing to decode a weak reference.\n3) Weak references are not accessible due to GC.\n\nGC Thread:\n- Disallow weak reference access.\n- Wait for checkpoint.\n\nThread2:\n- Try to lock JitCodeCache lock\n- Deadlock, as Thread1 owns the JitCodeCache lock.\n\nTest: test-art-host\nbug: 31289185\nbug: 33198826\n\nChange-Id: I7ee17631015450ace8d2a0264415a81c5a902bb8\n" }, { "commit": "b18a669259aa6ba08d9ca01b5b32c1aa0417138b", "tree": "c8f5260d9b68614f85b2947d81122ffcaf99fdec", "parents": [ "35b6546b0a4fe423ed6f27d8c5b709d5191e99ef" ], "author": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Nov 17 10:51:58 2016 +0300" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 28 11:38:03 2016 +0000" }, "message": "Cache flush/invalidate needs RWX permission\n\nWhen generating JIT code, perform cache maintenance operations before\nremoving page write permissions. Errata on some cores require data\nflush operations to be followed by data invalidate operations\nrequiring write permission.\n\nTest: ART_TEST_JIT\u003dtrue test-art-target on an arm64 device.\nbug: 27265969\n\nChange-Id: I70cbb0dc2ea37e42f2011d23333e386ae3f639c7\n" }, { "commit": "7a704bea1b1ff1e7deb903513997e0f35040287b", "tree": "e94262da7f17cc3ffd4e7a8df337cbd96f62da0b", "parents": [ "c394d0558de9afff6ab8b87ae9706497e01f37ff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 22 13:24:40 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 22 13:30:47 2016 -0800" }, "message": "Always fill in root table length\n\nOtherwise, there may be DCHECK failure in JitCodeCache::ClearData.\nFixes gcstress JIT tests.\n\nTest: test/run-test --host --relocate --prebuild --64 --jit --gcstress --dev 021-string2\n\nChange-Id: I757ccd1e5a015c5e686254417a2a3f68b650da1c\n" }, { "commit": "f46501c6ad11025843682267c10f221323a206b1", "tree": "a5b77de81f5ab31e66082b02b225e7fe56877f98", "parents": [ "71d15102b52af67e8fe1193192aa2b4cd1956ae0" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 22 13:45:36 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 22 14:21:31 2016 +0000" }, "message": "Clear the right data pointer.\n\nInternally, stack_map_data is an offset of the roots_data allocation.\nPass both to the ClearData method, which will use the right pointer\nto deallocate.\n\nTest: test-art-host in debug mode\nChange-Id: Ibfe38e7f1853870076048427f2efe7121c11c136\n" }, { "commit": "132d8363bf8cb043d910836672192ec8c36649b6", "tree": "42ee8b08f0323e9762675333b50a4c218ecdefc7", "parents": [ "e89667815b9d0a1eacb91678fed2a7518bb07cc2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 16 09:19:42 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Nov 18 14:31:30 2016 +0000" }, "message": "Revert \"Revert \"Revert \"Revert \"JIT root tables.\"\"\"\"\n\nTest: 626-set-resolved-string, test-art-host, test-art-target\nTest: run-libcore-tests.sh\nTest: phone boots and runs\n\nThis reverts commit 3395fbc20bcd20948bec8958db91b304c17cacd8.\n\nChange-Id: I104b73d093e3eb6a271d564cfdb9ab09c1c8cf24\n" }, { "commit": "3395fbc20bcd20948bec8958db91b304c17cacd8", "tree": "e0d00b412961e2aaa0dff91317a9f4168189e5fb", "parents": [ "75afcdd3503a8a8518e5b23d21b6e73306ce39ce" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 14 12:40:52 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 14 12:40:52 2016 +0000" }, "message": "Revert \"Revert \"Revert \"JIT root tables.\"\"\"\n\nlibcore failures:\ndalvikvm32 F 11-14 03:04:06 14870 14870 jit_code_cache.cc:310] Check failed: new_string !\u003d nullptr\n\nThis reverts commit 75afcdd3503a8a8518e5b23d21b6e73306ce39ce.\n\nChange-Id: I5a6b6b48aa79a763d1ff1ba4d85d63811254787d\n" }, { "commit": "75afcdd3503a8a8518e5b23d21b6e73306ce39ce", "tree": "b49afc4772fa72319475f96ed66d744bb03e803c", "parents": [ "ee48b0f11df812d98de5c989e1f6d95a21515564" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Nov 10 10:38:11 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Nov 11 13:18:24 2016 +0000" }, "message": "Revert \"Revert \"JIT root tables.\"\"\n\nAlso contains Revert \"Support kJitTableAddress in x86/arm/arm64.\"\n\nThis reverts commit 4acd03638fcdb4e5d1666f8eec7eb3bf6d6be035.\nThis reverts commit 997d1217830c0a18b70faeabd53c04700a87d7d9.\n\nTest: ART_USE_READ_BARRIER\u003dtrue/false test-art-host test-art-target\n\nChange-Id: I77cb1e9bf8f1b4c58b72d3cf5ca31ced2aaa1ea3\n" }, { "commit": "4acd03638fcdb4e5d1666f8eec7eb3bf6d6be035", "tree": "1220262d0ed22d91777c68a8a568348cddf820cb", "parents": [ "ac3ebc3150760425ed00abd56da48f9a6e0666bc" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 09 17:30:31 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 09 17:30:31 2016 +0000" }, "message": "Revert \"JIT root tables.\"\n\nMay be the offender for jit-gcstress failure of 902.\n\nThis reverts commit ac3ebc3150760425ed00abd56da48f9a6e0666bc.\n\nChange-Id: I9ea6c9236fd1729fed7d1868dd8a111172932308\n" }, { "commit": "ac3ebc3150760425ed00abd56da48f9a6e0666bc", "tree": "3c5c923bb00a898e035c0e7c4b8a6a684a578419", "parents": [ "76ad2405b40547e0ab4fc401fb8303189ff783eb" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 05 13:13:50 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 08 17:21:30 2016 +0000" }, "message": "JIT root tables.\n\nImplement root tables for the JIT. Each JIT compiled method\ngets a table allocated before the stack maps.\n\nThe table gets visited through Runtime::SweepSystemWeaks.\n\nImplement String roots for x86_64 as an example.\n\nTest: test-art-host test-art-target\n\nChange-Id: Id3d5bc67479e08b52dd4b253e970201203a0f0d2\n" }, { "commit": "709b070044354d9f47641f273edacaeeb0240ab7", "tree": "3a8ac051d7c35076303984d0d892cdd396b60427", "parents": [ "1a4de6a2453a3ad0310aca1a44e7e2d3b6f53bc1" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 13 09:12:37 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Oct 18 14:10:04 2016 -0700" }, "message": "Remove mirror:: and ArtMethod deps in utils.{h,cc}\n\nThe latest chapter in the ongoing saga of attempting to dump a DEX\nfile without having to start a whole runtime instance. This episode\nfinds us removing references to ArtMethod/ArtField/mirror.\n\nOne aspect of this change that I would like to call out specfically\nis that the utils versions of the \"Pretty*\" functions all were written\nto accept nullptr as an argument. I have split these functions up as\nfollows:\n1) an instance method, such as PrettyClass that obviously requires\nthis !\u003d nullptr.\n2) a static method, that behaves the same way as the util method, but\ncalls the instance method if p !\u003d nullptr.\nThis requires using a full class qualifier for the static methods,\nwhich isn\u0027t exactly beautiful. I have tried to remove as many cases\nas possible where it was clear p !\u003d nullptr.\n\nBug: 22322814\nTest: test-art-host\nChange-Id: I21adee3614aa697aa580cd1b86b72d9206e1cb24\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": "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": "ba65cc4a71273904294245cb37ce70e5bce797e3", "tree": "d53a7a816ec4c8e5bdbf80729ac945787b27bf2c", "parents": [ "1a827a05afbffd5bee241f245f9aa3c40b4dbae4", "542451cc546779f5c67840e105c51205a1b0a8fd" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Aug 02 03:47:35 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Aug 02 03:47:35 2016 +0000" }, "message": "Merge \"ART: Convert pointer size to enum\"" } ], "next": "542451cc546779f5c67840e105c51205a1b0a8fd" }