)]}' { "log": [ { "commit": "38a4223fcc0493553d9ad324a1dc145869eb663a", "tree": "a55f0ae7b46175db7cd66844a614952ce4bd5ec5", "parents": [ "c25dfeab1a66c942ae658fc7cd367f7c1ac502a1", "31e88225b2ef68e7f32f11186acf922c74ddabab" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 17 20:29:37 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Oct 17 20:29:37 2016 +0000" }, "message": "Merge \"Move most mirror:: args to ObjPtr\"" }, { "commit": "31e88225b2ef68e7f32f11186acf922c74ddabab", "tree": "ca50f60634854056bb652e7c375e5ccca2def5cd", "parents": [ "602142692643d8cd2c48d9aaeece26470f702e3b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 14 18:43:19 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 17 11:36:39 2016 -0700" }, "message": "Move most mirror:: args to ObjPtr\n\nFixed possible moving GC bugs in ClinitImageUpdate class.\n\nBug: 31113334\n\nTest: test-art-host\nChange-Id: I0bf6578553d58b944aaa17665f1350bdf5ed15ec\n" }, { "commit": "f5b0f2cef8554efaa90adc6a1ae04c2b533970ed", "tree": "431874a41548ba5d18598a5b1b520de042e012a9", "parents": [ "4c91f05a9ad59759760fb8c1543684dfdb9ab7dc" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Oct 14 01:04:26 2016 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Oct 14 16:52:42 2016 -0700" }, "message": "Ensure OpenDexFilesFromImage closes file to prevent file descriptor leak\n\n(cherry picked from commit 645e477c2ce7a59ac98bc86463826cf082a13ad8)\n\nTest: m -j32 test-art-host\nBug: 32095919\nChange-Id: I44541edeb403a508e1b9794ee09fb1f55e3f42bf\n" }, { "commit": "1cc62e4ea24828fdb3f3da0b8603f0b107d09a04", "tree": "718e322f3d05ac095770d4dc2a68b824bf3974a7", "parents": [ "82d4838d6bb3480cd25327cedc5179fb2d86881c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 18:01:28 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 10:46:16 2016 -0700" }, "message": "Rename ObjPtr::Decode to ObjPtr::Ptr\n\nDone to prevent ambiguity with ScopedObjectAccess::Decode.\n\nBug: 31113334\n\nTest: test-art-host\nChange-Id: I07a2497cc9cf66386311798933547471987fc316\n" }, { "commit": "3398c7874e002beaa6c2b2fadf183e7d1ddad23a", "tree": "7db754b5b0c3a6c68bc1f91d7cb2477f534d2ad3", "parents": [ "462e21cc8d28a4d48342cf0f634bd1ecc63647d7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 30 10:27:43 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 14:06:27 2016 -0700" }, "message": "Move ArtField to ObjPtr\n\nAdded EXPECT_OBJ_PTR_EQ and variants to gtests.\n\nFixed moving GC bugs in:\nClassLinker::CreatePathClassLoader\nClassLinkerTest: StaticFields\n\nObjPtr Decode call sites: 186 -\u003e 181.\n\nSome tests fail due to ResolvedFieldAccessTest, will fix in follow\nup CL.\n\nBug: 31113334\n\nTest: test-art-host CC baker\n\nChange-Id: I8b266ad00f3c20c8cbe7cfdf280d175083df0b88\n" }, { "commit": "0795f23920ee9aabf28e45c63cd592dcccf00216", "tree": "ff3f880c5e84f3316532b47d0e9a7729ade848ac", "parents": [ "d1224dce59eb0019507e41da5e10f12dda66bee4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 18:43:30 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 29 17:31:09 2016 -0700" }, "message": "Clean up ScopedThreadStateChange to use ObjPtr\n\nAlso fixed inclusion of -inl.h files in .h files by adding\nscoped_object_access-inl.h and scoped_fast_natvie_object_access-inl.h\n\nChanged AddLocalReference / Decode to use ObjPtr.\n\nChanged libartbenchmark to be debug to avoid linkage errors.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I4d2e160483a29d21e1e0e440585ed328b9811483\n" }, { "commit": "3fec9ac0d5af1358d216eb2fdc2000ec0205f3f0", "tree": "f38d8d8aae51f53e7ee6b474f47597b784fc2316", "parents": [ "0cfe19af3b7395658210ea6044a65c9811962a7a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 13 10:47:28 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 26 10:59:22 2016 -0700" }, "message": "ART: Use libbase logging\n\nMove most of our logging infrastructure over to system/core/base.\nRetain VLOG.\n\nUsing unified Android infrastructure has two main advantages. First,\nit reduces the complexity/maintenance burden in ART. Second, it\nallows to detach logging for the cases where we do not want or need\na runtime, e.g., dexdump, the disassembler, etc. As a part of the\nlatter, libbase is also supported for all hosts (including Windows).\n\nFrom a developer viewpoint, there are minor behavior changes for the\nLOG statements (see above), but otherwise usage is the same. Explicit\nseverity enum items are in the android::base namespace now.\n\nBug: 31338270\nTest: m test-art-host\nChange-Id: I5abcb2f45f5b03d49951874c48544f72a283a91b\n" }, { "commit": "0e8443995af602bcc2e70dcef1d20ee48acec934", "tree": "2d81b8db23bea139e40e33e4e9b8c69027fb1e56", "parents": [ "c4cb3d532db8a500c1a15e8a0052895960259209" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Thu Sep 08 13:43:31 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Wed Sep 21 11:42:21 2016 +0100" }, "message": "Collapse ART\u0027s AbstractMethod class into Executable\n\nAfter the OpenJDK 8 pull there is now a common base class\nfor Method and Constructor that can be used instead.\n\nMethods that are abstract on Executable but on Android\nwere implemented in AbstractMethod are now consistently\nrenamed to \"{method}Internal\", declared final and\ngiven package-protected access.\n\nFor consistency with all other private, natively\nimplemented methods, the method getDeclaredAnnotations()\nhas been renamed getDeclaredAnnotationsNative().\n\nBug: 28666126\nBug: 31052885\nTest: make test-art-host and cts run of CtsLibcoreTestCases\nChange-Id: Ibfe6af6a47fe0cfffd0859ffbe58980aeb84e053\n" }, { "commit": "c7376e0f8002d7838c3d69569028fbc9b91a38f3", "tree": "c325af4f653255d5957cd16463afbe880d5cdb0d", "parents": [ "47b451546d1a5c0fbb5e3816373395762a736515" ], "author": { "name": "Colin Cross", "email": "ccross@android.com", "time": "Thu Sep 08 12:52:18 2016 -0700" }, "committer": { "name": "Colin Cross", "email": "ccross@android.com", "time": "Mon Sep 12 16:29:02 2016 -0700" }, "message": "Convert more of art to Android.bp\n\nConvert dex2oat, imgdiag, patchoat, and profman to Android.bp.\n\nTest: m -j test-art-host\nTest: mmma -j art\nChange-Id: If2acd47189e5a944732c2c00c2bf31265f887161\n" }, { "commit": "7b49e6cade09bc65b3b5f22d45fc9d0a7184e4f2", "tree": "36bb77d376708ba0f054c9820f4b7ed51437fbb2", "parents": [ "3bac5443975cd6312e7c6282b94014db25f369d7" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Sep 01 11:06:18 2016 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Sep 12 14:39:07 2016 +0100" }, "message": "Introduce VDEX file, use it for DEX files\n\nThis patch introduces a new output file called VDEX. In the future,\nVDEX files will store pre-validated DEX files which do not need to be\nre-extracted and re-verified when recompiling, e.g. due to new\nprofiling information or after a system update.\n\nWith this CL, the OatWriter writes DEX files into the VDEX and the\nrest of its output into OAT. The OatFile class and related classes\nare updated to load the VDEX at runtime and mmap the DEX file section\nfrom it. Patchoat creates symlinks to the source VDEX files in the\ntarget directory or copies the files if passed in as file descriptors.\n\nThe feature can be disabled by setting the environment variable\nART_ENABLE_VDEX to false.\n\nTest: m test-art-host\nBug: 30937355\nChange-Id: I54dcaececf6814c258c80524ec15e2e2ef69c8dd\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": "bf44e0e5281de91f2e38a9378b94ef8c50ad9b23", "tree": "bb6e65a3434806dc58f286ee75ad3b78ba9d6c36", "parents": [ "d99565069c64fefc069005286de04599dc2619b8" ], "author": { "name": "Christina Wadsworth", "email": "cwadsworth@google.com", "time": "Thu Aug 18 10:37:42 2016 -0700" }, "committer": { "name": "Christina Wadsworth", "email": "cwadsworth@google.com", "time": "Thu Aug 18 16:18:36 2016 -0700" }, "message": "ART: Implement a fixed size string dex cache\n\nPreviously, the string dex cache was dex_file-\u003eNumStringIds() size, and\n@ruhler found that only ~1% of that cache was ever getting filled. Since\nmany of these string dex caches were previously 100,000+ indices in\nlength, we\u0027re wasting a few hundred KB per app by storing null pointers.\nThe intent of this project was to reduce the space the string dex cache\nis using, while not regressing on time that much. This is the first of a\nfew CLs, which implements the new fixed size array and disables the\ncompiled code so it always goes slow path. In four other CLs, I\nimplemented a \"medium path\" that regresses from the previous \"fast path\"\nonly a bit in assembly in the entrypoints. @vmarko will introduce new\ncompiled code in the future so that we ultimately won\u0027t be regressing on\ntime at all. Overall, space savings have been confirmed as on the order\nof 100 KB per application.\n\nA 4-5% slow down in art-opt on Golem, and no noticeable slow down in the\ninterpreter. The opt slow down should be diminished once the new\ncompiled code is introduced.\n\nTest: m test-art-host\n\nBug: 20323084\n\nChange-Id: Ic654a1fb9c1ae127dde59290bf36a23edb55ca8e\n" }, { "commit": "542451cc546779f5c67840e105c51205a1b0a8fd", "tree": "11e09bb5abaee12dddffefbe7e425291076dfa7a", "parents": [ "85c4a4b8c9eabfe16e4e49f9b4aa78c1bf4be023" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 26 09:02:02 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 01 18:54:48 2016 -0700" }, "message": "ART: Convert pointer size to enum\n\nMove away from size_t to dedicated enum (class).\n\nBug: 30373134\nBug: 30419309\nTest: m test-art-host\nChange-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269\n" }, { "commit": "75f0885d7c3d72a0351912b6d47682e1c860efdf", "tree": "d8a586b205ad3a502d8a007565a0140591d340ce", "parents": [ "27d99ed24328509cd3df7f895ce2696a1884bb21" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 19 08:06:07 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 21 11:26:47 2016 -0700" }, "message": "ART: Rename ArtMethod JNI field\n\nThe field is multi-purpose, rename it to data and clean up\naccessors in preparation of more checks.\n\nTest: m test-art-host\nChange-Id: Ib4b84b7b1a51ca201544bc488ce8770aa858c7fd\n" }, { "commit": "a62cb9bb6cb2278cb41ab0664191623e178c6a4f", "tree": "62263dac644e2d80a34a04f2649e4741a1bed6f4", "parents": [ "bb8d501c9bb882a8927c6ceda07bf9577e06c3e1" ], "author": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jun 30 09:18:25 2016 +0000" }, "committer": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jul 07 10:01:04 2016 +0000" }, "message": "Revert \"Revert \"Optimize IMT\"\"\n\nThis reverts commit 88f288e3564d79d87c0cd8bb831ec5a791ba4861.\n\nChange-Id: I49605d53692cbec1e2622e23ff2893fc51ed4115\n" }, { "commit": "88f288e3564d79d87c0cd8bb831ec5a791ba4861", "tree": "dd051a7b2985d1af3fea91ad6000de4bdc701a19", "parents": [ "e36389f20c83083c0aaba2dcac0888951e55cae1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:17:52 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:41:25 2016 +0000" }, "message": "Revert \"Optimize IMT\"\n\nBug: 29188168 (for initial CL)\nBug: 29778499 (reason for revert)\n\nThis reverts commit badee9820fcf5dca5f8c46c3215ae1779ee7736e.\n\nChange-Id: I32b8463122c3521e233c34ca95c96a5078e88848\n" }, { "commit": "badee9820fcf5dca5f8c46c3215ae1779ee7736e", "tree": "982948a67a88a1f9a734c935f919f8d307969f48", "parents": [ "614968198625a6693666bdc1e5609e2f663f5638" ], "author": { "name": "Nelli Kim", "email": "nelli.kim@samsung.com", "time": "Fri May 13 13:08:53 2016 +0300" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jun 14 10:05:47 2016 -0700" }, "message": "Optimize IMT\n\n* Remove IMT for classes which do not implement interfaces\n* Remove IMT for array classes\n* Share same IMT\n\nSaved memory (measured on hammerhead):\nboot.art:\nTotal number of classes: 3854\nNumber of affected classes: 1637\nSaved memory: 409kB\n\nChrome (excluding classes in boot.art):\nTotal number of classes: 2409\nNumber of affected classes: 1259\nSaved memory: 314kB\n\nGoogle Maps (excluding classes in boot.art):\nTotal number of classes: 6988\nNumber of affected classes: 2574\nSaved memory: 643kB\n\nPerformance regression on benchmarks/InvokeInterface.java benchmark\n(measured timeCall10Interface)\n1st launch: 9.6%\n2nd launch: 6.8%\n\nChange-Id: If07e45390014a6ee8f3c1c4ca095b43046f0871f\n" }, { "commit": "e42888f9df4163303244070c65d5229d3e201742", "tree": "c3c3e380ed16676186040f1da6042fc2cd58e041", "parents": [ "7f98c9a6babe3a21d84ce1f1e1273c99975a47f5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 14 10:49:19 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 27 12:45:30 2016 -0700" }, "message": "Write conflict tables in image\n\nAdd image sections for runtime methods and conflict tables. This\nmakes it that we do not need to fake up a length prefixed array\nfor runtime methods.\n\nReduces .art private dirty and PSS.\n\nSystem wide .art PSS goes from 32.5MB to 30.5MB after system boot.\n\nBusiness card .art private dirty goes from 588K to 504K.\n\nIncreases image size by ~100K.\n\nBug: 27906566\n\n(cherry picked from commit cda9386add68d94697449c6cb08b356747e55c21)\n\nChange-Id: I38cbe3150c3eeb385b8cad7cf93614e3980f4162\n" }, { "commit": "a13abbac97d38d726f9e7d5d786b571a5d37c227", "tree": "aec05c6218144261c55dbd64f9c97f555bc5202c", "parents": [ "121c8a6494f2224830ed0ca33df5af0d8c077647" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 21 10:23:16 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 21 12:01:48 2016 -0700" }, "message": "Fix heap check.\n\nPreviously, we only checked the image we were patching. This did not\nwork if the referent was in another image. Fixed the DCHECK to check\nall the boot images.\n\nBug: 28286961\n\n(cherry picked from commit 488849c584337a1aae8413a1d1e0fad8043bf672)\n\nChange-Id: If2e8e48b500935d0969e8673816d28218888bd13\n" }, { "commit": "d12f9c13599a6b5893cffd02c9b8d35b4db93bf3", "tree": "342ff38f30c85e0ac997fe3eb8401c3e3e0ee8db", "parents": [ "20eef176101924d5047895214bad4e73b8ae35ec" ], "author": { "name": "Serdjuk, Nikolay Y", "email": "nikolay.y.serdjuk@intel.com", "time": "Tue Mar 22 10:06:33 2016 +0600" }, "committer": { "name": "Serdjuk, Nikolay Y", "email": "nikolay.y.serdjuk@intel.com", "time": "Thu Mar 31 10:17:18 2016 +0600" }, "message": "ART: Save or erase the file even if I/O failed\n\nIt should close the file in any case.\n\nChange-Id: Idb7db15bfecb0ac47839be3544cc2dce91096d7b\nSigned-off-by: Serdjuk, Nikolay Y \u003cnikolay.y.serdjuk@intel.com\u003e\n" }, { "commit": "ec1514a91a34586aca3fd75d1565ad5992c54929", "tree": "a6b4df93baa8775eac5c7170391a5b94cca93bc4", "parents": [ "895375bb30716a488bf7b8234e900d1e15de1d21" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Mar 17 21:32:45 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 18 15:27:47 2016 -0700" }, "message": "Add patchoat check that input oat file has patch section.\n\nDo this check before opening the output file in case this could be\ncausing the file to become truncated.\n\nBug: 27723864\n\n(cherry-picked from commit 99503b9805def2ed7d4fe4bf285c6fdd8ddce6ed)\n\nChange-Id: Icf71016adae5404e4f8588f2f5e23da6405ef2e7\n" }, { "commit": "7a1c0148a9bfbd15132c949f7758719027e9deb4", "tree": "ebd69eee5596e32bf18381801224a23bac809b4a", "parents": [ "275f97e225f534582b446eb7b9eb00c6757255d9" ], "author": { "name": "Dimitry Ivanov", "email": "dimitry@google.com", "time": "Thu Mar 17 15:59:38 2016 -0700" }, "committer": { "name": "Dimitry Ivanov", "email": "dimitry@google.com", "time": "Thu Mar 17 15:59:38 2016 -0700" }, "message": "Remove TEMP_FAILURE_RETRY for unlink\n\nunlink(2) does not generate EINTR error\n\nChange-Id: I546b00a61b8ffcbd43b01f5ae6ff31f335be00ab\n" }, { "commit": "6eb6a39bdcf669655190d40d20f8d2e663b262f4", "tree": "1b56bd9527b56c8d24598245de0597bbb4f73d50", "parents": [ "0665090c880ec22655ff8b82084746c4407ea792" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Feb 10 20:18:37 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Feb 12 16:26:03 2016 -0800" }, "message": "ART: Simplify patchoat\n\nRemoved unused configurations. Cut out cruft that isn\u0027t used anymore\nsince multi-image. Make it smaller to not require new selinux\npermissions for OTAs.\n\nSplit up the main function, to make the flow clearer for image vs\napp/oat.\n\nBug: 25612095\nBug: 26317072\n\n(cherry picked from commit 02bc0b70beedfe258c2147c2b4a205fbf3d6eebc)\n\nChange-Id: Ieb55bd1a13cbb44fd0e2979a8a61a509df9c1e7e\n" }, { "commit": "4a26f17b055cadc949c3e9fdfa637fe5656339d9", "tree": "d3cee26249b5ac72c57a7125ae2a73765bf77d96", "parents": [ "caef579e906abc5c3174ba65e5205ea7fd5c44ad" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 26 14:26:18 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 26 17:49:12 2016 -0800" }, "message": "Remove GetImageRoots read barrier for image relocation\n\nFixes CC assertion that failed if an app image was loaded when the GC\nwas running.\n\nBug: 26786304\nChange-Id: I47bea7b4b947332b57be2e2db604c2c25d92b7e2\n" }, { "commit": "fbc31087932a65e036a153afab3049dc5298656a", "tree": "c728b9039a71db3be9cae888ad5e1afc31f34a13", "parents": [ "c7f4e3a5aeaa23342b4e03b0d751f60ac5c5815c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Jan 24 11:59:56 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jan 25 14:22:15 2016 -0800" }, "message": "Revert \"Revert \"Load app images\"\"\n\nThis reverts commit 1bc977cf2f8199311a97f2ba9431a184540e3e9c.\n\nBug: 22858531\n\nChange-Id: Ide00bf3a73a02cba3bb364177204ad1b13f70295\n" }, { "commit": "1bc977cf2f8199311a97f2ba9431a184540e3e9c", "tree": "580a02752d8e447f6dce7cce01386c7e2a9a87f4", "parents": [ "f7fd970244f143b1abb956e29794c446e4d57f46" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Jan 23 14:15:49 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Jan 23 14:15:49 2016 +0000" }, "message": "Revert \"Load app images\"\n\nFails when a method is duplicated (see test 097-duplicate-method)\n\nBug: 22858531\n\nThis reverts commit f7fd970244f143b1abb956e29794c446e4d57f46.\n\nChange-Id: Ib30ae5be00cc568e799290be6b3c8f29cbbe4c20\n" }, { "commit": "f7fd970244f143b1abb956e29794c446e4d57f46", "tree": "aac1f57ac70747957f609bb46305dfeca87645a1", "parents": [ "95005291d8ebdd1d2ac58ffc5181fef4fbbf2383" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 09 11:16:49 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 22 15:01:55 2016 -0800" }, "message": "Load app images\n\nSupport in-place patching of the app image based on boot image\nlocation and app oat location. Only loads for art run test so far\nsince we do not automatically generate app images for app installs.\n\nN5 maps launch time (~200 runs):\nBefore: 930ms\nAfter: 878.18ms\nAfter + image class table: 864.57ms\n\nTODO:\nOatdump support.\nStore class loaders as class roots in image.\n\nBug: 22858531\n\nChange-Id: I9cbc645645e62ea2ed1ad8e139e91af7d88514c1\n" }, { "commit": "ea0831f60d26e3297e6463634a9fbb6384f00661", "tree": "6b970dced071048d59d1857e4703521d76566f99", "parents": [ "664d7cfba0e734ba8917c260e424ea559b7e8f9f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Dec 29 13:17:37 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 06 10:17:01 2016 -0800" }, "message": "Re-enable adding intern table to image\n\nChanged intern table to have a stack of tables similarily to\nClassTable. Adding an image intern table adds to the front of the\nintern table stack. Also some cleanup.\n\nBug: 26317072\n\nChange-Id: I7bbf9485b5dbbbf3707fed21e29de3beccfb8705\n" }, { "commit": "0d2af30df4000101b7c4b4e6b6350460a141edc8", "tree": "a2599d57c060914fb10c9ba06f96ccd8a17a42e0", "parents": [ "376a6f3dbae7b71a6fc2c339ec416d3407277308" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Jan 04 17:38:06 2016 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Jan 04 19:08:51 2016 -0800" }, "message": "Fix multi-image TODOs in patchoat.\n\n- Reinstated checks that were still valid (one check was removed).\n- Removed unused path that patches only images.\n\nBug: 26317072\n\n(cherry-picked from commit e271fe1e2797205c57c052212c32139234f781ea)\n\nChange-Id: I0e140cb110abbfa469c097c805657ecfdc8552d6\n" }, { "commit": "dcdc85bbd569f0ee66c331b4219c19304a616214", "tree": "b5ab789248e279318f6c1e3f6c511703d7294476", "parents": [ "48944c760b196188b968b7af81439466cf987a75" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Dec 04 14:06:18 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 23 21:14:23 2015 -0800" }, "message": "Dex2oat support for multiple oat file and image file outputs.\n\nMultiple changes to dex2oat and the runtime to support a --multi-image\noption. This generates a separate oat file and image file output for\neach dex file input.\n\nChange-Id: Ie1d6f0b8afa8aed5790065b8c2eb177990c60129\n" }, { "commit": "ceb07b3285eaab350a8cd12f7d74be3e40a255dd", "tree": "4d5f7c1ab47ea6b8399d2112b69a818a0519b2de", "parents": [ "836ee764c86892aff1dca6b0f8b27e32c7374cc7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Dec 10 09:33:21 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Dec 10 10:28:19 2015 -0800" }, "message": "Revert \"Revert \"Add support for LZ4 compressed image files\"\"\n\nNeeded to call compiler_driver_-\u003eSetSupportBootImageFixup(false).\n\nBug: 22858531\n\nThis reverts commit 83d4d72aa0e4170209ab50c67ba22e46b71352c1.\n\nChange-Id: Iaed6a810a0c088f1f2c57cf2f12087f3978a3de1\n" }, { "commit": "83d4d72aa0e4170209ab50c67ba22e46b71352c1", "tree": "5444d2ba3e49c4bc8f7997ed0886264f53902ca6", "parents": [ "c6f41b5b3ca3d7ac3c12ad3995ffef4e831973a0" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 10 08:26:32 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 10 08:26:32 2015 +0000" }, "message": "Revert \"Add support for LZ4 compressed image files\"\n\nTentative. Looks like it breaks image_test for hammerhead:\nart/compiler/linker/arm/relative_patcher_thumb2.cc:36] Check failed: target_offset \u0026 1u \u003d\u003d 1u (target_offset \u0026 1u\u003d0, 1u\u003d1) \n\nBug: 22858531\n\nThis reverts commit c6f41b5b3ca3d7ac3c12ad3995ffef4e831973a0.\n\nChange-Id: I9bc5738a8b5c8f8bc8b73309f9420fd691bc79a9\n" }, { "commit": "c6f41b5b3ca3d7ac3c12ad3995ffef4e831973a0", "tree": "c32c87552120021c091540e010722eb317d04441", "parents": [ "d7d23eec777a8b84909f3364700fc1e8a075e5a2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 04 15:38:50 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Dec 09 11:40:19 2015 -0800" }, "message": "Add support for LZ4 compressed image files\n\nAdded dex2oat option --image-format\u003d(store|lz4). Using lz4 means\nthat the main image section (all data other than header and bitmap)\nare stored in a compressed state.\n\nN5 results:\nBoot image size: 8067128 -\u003e 2827605\nDecompression time 18.93ms\nDecompression rate: 426MB/s\n\nPatchoat is not currently supported since it maps the source image\ndirectly. In order to support compressed images we would need to\nrecompress the output image and then write it back out to a file.\nAlso there are not many cases where we would want to patch a\ncompressed image since they are going to be dirty memory when\nuncompressed anyways. Might as well just patch as we are loading.\n\nBug: 22858531\n\nChange-Id: I8c54ccf73408273011161a61bb891736735074d9\n" }, { "commit": "41dc8cef5d1f86e3ade30b8aaf20d29d8d0c712b", "tree": "000d84e9b7d321b86349da577f744977ecf191d6", "parents": [ "bf0e73604f836e3b8f7d6481e83110c3426350b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 04 15:07:48 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 04 15:19:10 2015 -0800" }, "message": "Combine class sets when writing a class table\n\nReduces the boot.art size since we allocate a new class set with the\ndefault load factor. The load factor in the combined class table is\nhigher to pruning classes, this reduces the size of the boot image.\n\nAlso fixes the hypothetical case where the class table being written\nhas multiple class sets.\n\nBoot.art size\nBefore: 8122368\nAfter: 8044544\n\nSlight cleanup in patchoat.cc.\n\nBug: 22858531\nChange-Id: I31b3ea2004ece6f6c853ee906074bb2792dee958\n" }, { "commit": "208a5cb383dd9dcd3461f89b74af5df67dc8d794", "tree": "9ed531a445d24a7685e577b1c5822e50dc1c1b1d", "parents": [ "5337bbfbd1cd502bb2842698a6b2a543ab6a6c32" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Dec 02 15:44:07 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Dec 03 16:29:03 2015 -0800" }, "message": "Store class tables in the image\n\nReduces how long it takes to load an application image.\n\nN5 boot.art size\nBefore: 8007680\nAfter: 8122368\n\nAlso reduces boot time by how long AddImageClassesToClassTable\nused to take (~20ms).\n\nChanged class hashes to be uint32_t to fix cross compilation. We need\nserialized hash tables to be valid with different pointer sizes.\n\nBug: 22858531\n\nChange-Id: I463fc83f499ff75f509e80c253a55b9116ee5b89\n" }, { "commit": "4b00d3415beb7a816a3b5948f43f3b2e4b856ea8", "tree": "a54f8c2704fbe9b9ffebdc0f1984a777beb80ac0", "parents": [ "991842a5273d20695dd5f35eb9a9e28c386c5b4b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 13 10:42:08 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 16 15:05:36 2015 -0800" }, "message": "Refactor some patching logic\n\nReduce duplication since app images in-place relocation will require\nthis code also.\n\nBug: 22858531\nChange-Id: Ibb901b67267e27ef3bc2a0baff77189d4dcd018a\n" }, { "commit": "073b16c8429d302d5413e8ffc488b03b8f770780", "tree": "a417fd8042df8c0a9621f9097643ecaede4deeab", "parents": [ "5165b6ba1fda85d769c5092a65f07bcfffb0b3e7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 10 14:13:23 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 11 19:22:33 2015 -0800" }, "message": "Image space cleanup for app images\n\nRemoved Heap::GetImageSpace, added Heap::GetBootImageSpace.\n\nGeneralized some logic in the class linker for image spaces.\n\nBug: 22858531\n\nChange-Id: Ib3e12bb061a247e232d3dc93b0d6b35eb3a34d25\n" }, { "commit": "42bddcec51e71d206f6d3b30a881ee6c1d50a63c", "tree": "2035b4dd7dcbefbe83c04d41987e520eeb8798f0", "parents": [ "78232f2482e71fef255796633da68bb769baa93a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 09 15:16:56 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 09 17:18:58 2015 -0800" }, "message": "Add low_4gb support to MapFile and MapFileAtAddress\n\nMotivation is to use this for loading app images in low 4GB at a\nnon fixed address.\n\nAdded test.\n\nBug: 22858531\nChange-Id: I0f79a4a7bfbfbdfc112e41b25c8682b1fb932ab7\n" }, { "commit": "05792b98980741111b4d0a24d68cff2a8e070a3a", "tree": "bad79a387bcbdaefc87c07b388099960ca9caff3", "parents": [ "c26b4512a01d46756683a4f5e186a0b7f397f251" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Aug 03 11:56:49 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 03 17:30:57 2015 +0100" }, "message": "ART: Move DexCache arrays to native.\n\nThis CL has a companion CL in libcore/\n https://android-review.googlesource.com/162985\n\nChange-Id: Icbc9e20ad1b565e603195b12714762bb446515fa\n" }, { "commit": "059ef3ddb2088f926ac452889e0953fdcd646a5e", "tree": "36067457d7e4aba11be908ddd235734d29b7c4f2", "parents": [ "f71ad9ede9ae322a897e8fe407208dc35c5dee65" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Aug 18 13:54:21 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Aug 18 16:11:21 2015 -0700" }, "message": "Always visit object class from VisitReferences\n\nWe don\u0027t want to unload classes which have instances.\n\nSlight increase in CMS GC time from ~6.5s to ~7.3s on\nEvaluateAndApplyChanges.\n\nBug: 22720414\nChange-Id: I467ff9c9d55163d2a90b999aef3bdd7b3f648bac\n" }, { "commit": "5e289b2bca7a0bc67fcf00a1017d70db8b363113", "tree": "c27deaacb51e42ff6407d521b9834e637bfa465a", "parents": [ "4b7b5fcda0f97ed874de0ccd18940030e6c71f04", "0eb76d2603a52d3e8759fcf480586886d70433d1" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Aug 17 17:20:46 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Aug 17 17:20:46 2015 +0000" }, "message": "Merge \"Make patchoat match offset when given a patched image\"" }, { "commit": "cf36d493124d8048efa0bd6f67d817ce3cd6b725", "tree": "fac1336f54ea477ce1afe2e99a04b68db38c7ac8", "parents": [ "1cad8c7c63b600a3da83bf05fb645e08ac8fafc0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Aug 12 19:27:26 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Aug 17 10:04:42 2015 +0100" }, "message": "ART: Compress LengthPrefixedArray on 32-bit targets.\n\nPreviously, the LengthPrefixedArray\u003cArtMethod\u003e on 32-bit\ntargets contained a 64-bit length field followed by the\nArtMethod elements with size only a multiple of 4, not 8.\nConsequently, an odd-length array broke the alignment for\nthe following array which would have the 64-bit length\nplaced at an unaligned address.\n\nTo fix that, we make the length field 32-bit and explicitly\npass the alignment information to the LengthPrefixedArray.\nThis also makes the 32-bit boot image a bit smaller.\nOn Nexus 5, AOSP, ToT, the field section is 11528B smaller\nand the method section is 21036B smaller. 64-bit targets\nshould see the same savings for the field section but no\ndifference for the methods section.\n\nChange-Id: I3e03e7b94129025c8a1c117c27645a34dec516d2\n" }, { "commit": "0eb76d2603a52d3e8759fcf480586886d70433d1", "tree": "8d38a89d6de7be77c437c9df12bf646b793ba639", "parents": [ "4500fcbe682d666a24c2e8f6e0cb90cfb35d3fa3" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Aug 11 18:03:47 2015 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Aug 14 19:34:57 2015 +0000" }, "message": "Make patchoat match offset when given a patched image\n\nPreviously if we gave patchoat a patched image and a already patched\n(but improperly relocated) oat file it would not correctly patch the\noat file to the same offset as the image.\n\nBug: 22599792\nBug: 23119724\nChange-Id: I8773022bd75c2e0b7eb529893b147cbd8792bcad\n" }, { "commit": "54d220eb9cc51215d75b9e0fe921b94bebbb3fd6", "tree": "fe08749dea3ca6cad97bce48416beb37203efdf5", "parents": [ "275bc5e5f318ba6ea1bad6c89c2924d1248b7523" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 30 16:20:06 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 05 18:15:22 2015 -0700" }, "message": "Move ArtFields and ArtMethods to be a length prefixed array\n\nFixes race conditions between changing method and fields arrays\nbeing seen in the wrong order by the GC.\n\nBug: 22832610\nChange-Id: Ia21d6698f73ba207a6392c3d6b9be2658933073f\n" }, { "commit": "da7c650022a974be10e2f00fa07d5109e3d8826f", "tree": "eb713496fee4a6885a3c240309e5efc78c6774cf", "parents": [ "2a7ea34cda9bf776339dfc7f4a22ab01f9525f2d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 23 16:01:26 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jul 24 11:50:43 2015 -0700" }, "message": "Visit class native roots from VisitReferences\n\nVisit class roots when we call Class::VisitReferences instead of in\nthe class linker. This makes it easier to implement class unloading\nsince unmarked classes won\u0027t have their roots visited by the class\nlinker.\n\nBug: 22181835\nChange-Id: I63f31e5ebef7b2a0b764b3ba3cb038b3f561b379\n" }, { "commit": "90443477f9a0061581c420775ce3b7eeae7468bc", "tree": "8c74b81dfae162e0fd0ccf8d5ac50827ba815174", "parents": [ "6078aec213dfaf111c29969706e8e5967cfc9bea" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 16 20:32:27 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 22 15:13:56 2015 -0700" }, "message": "Move to newer clang annotations\n\nAlso enable -Wthread-safety-negative.\n\nChanges:\nSwitch to capabilities and negative capabilities.\n\nFuture work:\nUse capabilities to implement uninterruptible annotations to work\nwith AssertNoThreadSuspension.\n\nBug: 20072211\n\nChange-Id: I42fcbe0300d98a831c89d1eff3ecd5a7e99ebf33\n" }, { "commit": "7bf2b4f1d08050f80782217febac55c8cfc5e4ef", "tree": "61b26b116454c5a114ac4b2f55c71153be7a9d43", "parents": [ "569e81e500725f52116b7d0342ec80a6d1e0089b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 08 10:11:59 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 08 14:22:19 2015 +0100" }, "message": "Revert \"Revert \"Remove interpreter entrypoint in ArtMethod.\"\"\n\nThe start of the interned strings in the image was not aligned\nproperly, now that ArtMethods just need to be word aligned.\n\nThis reverts commit 7070ccd8b6439477eafeea7ed3736645d78e003f.\n\nbug:22242193\n\nChange-Id: I580c23310c33c239fe0e5d15c72f23a936f58ed1\n" }, { "commit": "569e81e500725f52116b7d0342ec80a6d1e0089b", "tree": "c09e96441fa483fc8ba7f6ef2ead5e2aa137d951", "parents": [ "d5bd4fdd46dce3c33abf17376fbceadcc1aff547", "7070ccd8b6439477eafeea7ed3736645d78e003f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 08 09:42:05 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jul 08 09:42:05 2015 +0000" }, "message": "Merge \"Revert \"Remove interpreter entrypoint in ArtMethod.\"\"" }, { "commit": "7070ccd8b6439477eafeea7ed3736645d78e003f", "tree": "e32dca6b4342ce7b42952e0d9150a85fba361562", "parents": [ "fa2c054b28d4b540c1b3651401a7a091282a015f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 08 09:41:54 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 08 09:41:54 2015 +0000" }, "message": "Revert \"Remove interpreter entrypoint in ArtMethod.\"\n\nBuild failures on bots. Investigating.\n\nThis reverts commit fa2c054b28d4b540c1b3651401a7a091282a015f.\n\nChange-Id: Id65b2009aa66cb291fb8c39758a58e0b0d22616c\n" }, { "commit": "d5bd4fdd46dce3c33abf17376fbceadcc1aff547", "tree": "515156dffec1dde04c32c55ed39dc6dd8a1750b2", "parents": [ "a100332ee0f4b842c28ec2448f97e40a882cd837", "fa2c054b28d4b540c1b3651401a7a091282a015f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 08 08:56:49 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jul 08 08:56:50 2015 +0000" }, "message": "Merge \"Remove interpreter entrypoint in ArtMethod.\"" }, { "commit": "fa2c054b28d4b540c1b3651401a7a091282a015f", "tree": "d39c2eca12dce2e0366a092b05715b3eab1319b4", "parents": [ "c87c8939ea1bcfbddb954478d527cf1138f4f343" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 01 14:32:54 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 08 09:31:25 2015 +0100" }, "message": "Remove interpreter entrypoint in ArtMethod.\n\nSaves 4/8 bytes for each ArtMethod.\n\nChange-Id: I110ecdddf8516b0759a31fa157609643e6d60b15\n" }, { "commit": "01aaf6ef3e3e35cc8e41cf3fe899a7bf337042f4", "tree": "d19b35a078536b6190d861810aa443ccd7e2fcdb", "parents": [ "2b252dc15995a708aead33aab6d307205d860a88" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Jun 19 22:05:39 2015 +0100" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Jul 01 12:32:13 2015 +0100" }, "message": "Allow for sig chain to be disabled.\n\nTools like dex2oat or patchoat don\u0027t need the sig chain or the fault\nmanager. This also enables building a statically link version of\ndex2oat.\n\nChange-Id: I9897728cac48acade854bb027bfde860628ebf84\n" }, { "commit": "d39645e22b8db1767cf64dc1200a9e4b2f939ed2", "tree": "fd6488a428600f222c5a411e91ed9f1f3a280b05", "parents": [ "15306dcd03208f062737239753ab1eccfe91403f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 09 17:50:29 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 11 14:56:08 2015 -0700" }, "message": "Move image intern table into image\n\nPreviously we recreated this intern table during runtime startup.\nThis added 50-100ms of boot time.\n\nFixed bug where we didn\u0027t copy over hashcodes into the image.\n\nDeleted some stale code.\n\n(cherry picked from commit fac3a390a247fe33d4873773d742aad4cc100118)\n\nBug: 20727525\nBug: 19569780\nChange-Id: I08959e9aa2a73cedb52f393033e2ffea3a26e76b\n" }, { "commit": "e401d146407d61eeb99f8d6176b2ac13c4df1e33", "tree": "17927f9bfe7d2041b5942c89832d55f9dedb24c5", "parents": [ "2006b7b9b8e32722bd0d640c62549d8a0ac624b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 13:56:20 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 29 18:45:49 2015 -0700" }, "message": "Move mirror::ArtMethod to native\n\nOptimizing + quick tests are passing, devices boot.\n\nTODO: Test and fix bugs in mips64.\n\nSaves 16 bytes per most ArtMethod, 7.5MB reduction in system PSS.\nSome of the savings are from removal of virtual methods and direct\nmethods object arrays.\n\nBug: 19264997\nChange-Id: I622469a0cfa0e7082a2119f3d6a9491eb61e3f3d\n" }, { "commit": "33d75bc75c72db7e5b1ea4ad0be287b1dfd72e69", "tree": "f2bdf02aba34e7d59c760df1b54bb3c789ee6648", "parents": [ "65efe9192390187674108b9c895ba225628bb07b", "3b45ef277e4a5f7e0314d5df7ef82e480156ba75" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 27 22:41:30 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 27 22:41:31 2015 +0000" }, "message": "Merge \"ART: Fix VerifyObject runtime verification\"" }, { "commit": "3b45ef277e4a5f7e0314d5df7ef82e480156ba75", "tree": "1df28568b9e7e4e50102b7f834fab6e943c444a5", "parents": [ "c06841dac1443e99e92596f15caa82c99bfee010" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 26 21:34:09 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 27 08:24:03 2015 -0700" }, "message": "ART: Fix VerifyObject runtime verification\n\nUpdate some bit-rotted code to work again. Most tests now work, for\nsome the verification overhead results in a timeout.\n\nChange-Id: Ieab4f2de474a05e915e24abc93da3c2eeed996eb\n" }, { "commit": "80afd02024d20e60b197d3adfbb43cc303cf29e0", "tree": "ef054c7b4f2a739f7cf063e0bc4c501c2c7e41b5", "parents": [ "559b178e34c5d92e7932f92e5d8a981ac334606f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 19 18:08:00 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 26 15:59:02 2015 +0100" }, "message": "ART: Clean up arm64 kNumberOfXRegisters usage.\n\nAvoid undefined behavior for arm64 stemming from 1u \u003c\u003c 32 in\nloops with upper bound kNumberOfXRegisters.\n\nCreate iterators for enumerating bits in an integer either\nfrom high to low or from low to high and use them for\n\u003carch\u003eContext::FillCalleeSaves() on all architectures.\n\nRefactor runtime/utils.{h,cc} by moving all bit-fiddling\nfunctions to runtime/base/bit_utils.{h,cc} (together with\nthe new bit iterators) and all time-related functions to\nruntime/base/time_utils.{h,cc}. Improve test coverage and\nfix some corner cases for the bit-fiddling functions.\n\nBug: 13925192\nChange-Id: I704884dab15b41ecf7a1c47d397ab1c3fc7ee0f7\n" }, { "commit": "3fc9903407c6e89ffbbc92ded9e272d9de58e9b6", "tree": "ae8898200f09c4015ecc7d920711dd1053834e94", "parents": [ "c3912c8a2db109a15603554fd456f56cd0a69ad0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 13 19:06:30 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 14 13:34:08 2015 +0100" }, "message": "ART: Do not relocate app program headers in patchoat.\n\nChange the check whether to relocate program headers in\npatchoat to simply look whether there is a PT_LOAD section\nwith p_vaddr \u003d\u003d 0. If there is, don\u0027t relocate the headers,\nit should be an app. Otherwise, it\u0027s a boot image and needs\nto be relocated.\n\nAdd overflow checking to ElfFileImpl\u003c\u003e::GetLoadedSize().\n\nBug: 21047854\nChange-Id: Ie6737f338687296b6dbf4bb3b36358501dfb1587\n" }, { "commit": "2f6cdb01f74772c1c521a125776ef57ea3c73d43", "tree": "163f90841f02b9529997f4e3eea65e512e4795a3", "parents": [ "58565098b2298041ccc97371a3cc486df88d51b3" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sat Apr 11 00:17:53 2015 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sat Apr 11 19:14:10 2015 +0100" }, "message": "Relocate DWARF using .oat_patches.\n\nThe current solution is to hard-code knowledge of DWARF in the linker.\nThis works for simple use of DWARF, but breaks as soon as I try to do\nanything more complex. Making the linker fully support DWARF would be\nnon-trivial task and would be essentially rewrite. Using .oat_patches\nis much easier solution.\n\nRelocating .debug_* sections required extending .oat_patches to support\nmore sections than just .text. I have encoded each section as\nnull-terminated section name followed by ULEB128 deltas.\n\nThe ULEB128 encoding shrinks .oat_patches for .text by factor of\nabout 6 with 64-bit compiler, and factor of 3 with 32-bit compiler.\n\nOn the other hand, it grows by the extra .oat_patches for DWARF which\nwere not present before (if debug symbols are included).\n\nOverall, it is still a clear improvement even with the DWARF patches.\n\nChange-Id: I78ffeda0f8a3da03341995a3b5ef15c954e16e9f\n" }, { "commit": "c785344b87221f5e4e6473e5b762e4e61fe65dcf", "tree": "cd32ad2c2604596a18926f04d4c313dab255ecfd", "parents": [ "a29d93b380c9aeb8270e281aefbdd0c77a430d43" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 27 14:35:38 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 10 12:57:27 2015 -0700" }, "message": "Move ArtField to native\n\nAdd linear alloc. Moved ArtField to be native object. Changed image\nwriter to put ArtFields after the mirror section.\n\nSavings:\n2MB on low ram devices\n4MB on normal devices\n\nTotal PSS measurements before (normal N5, 95s after shell start):\nImage size: 7729152 bytes\n23112 kB: .NonMoving\n23212 kB: .NonMoving\n22868 kB: .NonMoving\n23072 kB: .NonMoving\n22836 kB: .NonMoving\n19618 kB: .Zygote\n19850 kB: .Zygote\n19623 kB: .Zygote\n19924 kB: .Zygote\n19612 kB: .Zygote\nAvg: 42745.4 kB\n\nAfter:\nImage size: 7462912 bytes\n17440 kB: .NonMoving\n16776 kB: .NonMoving\n16804 kB: .NonMoving\n17812 kB: .NonMoving\n16820 kB: .NonMoving\n18788 kB: .Zygote\n18856 kB: .Zygote\n19064 kB: .Zygote\n18841 kB: .Zygote\n18629 kB: .Zygote\n3499 kB: .LinearAlloc\n3408 kB: .LinearAlloc\n3424 kB: .LinearAlloc\n3600 kB: .LinearAlloc\n3436 kB: .LinearAlloc\nAvg: 39439.4 kB\n\nNo reflection performance changes.\n\nBug: 19264997\nBug: 17643507\n\nChange-Id: I10c73a37913332080aeb978c7c94713bdfe4fe1c\n" }, { "commit": "3296f4cb397f75044e7bd5f9b345f4b29d0275f3", "tree": "f7738e2878b6a7390bfd6b77646a806ecd915a88", "parents": [ "f5988cf0c47ff935bb19c89ed0398be5fbe44bb6" ], "author": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Fri Mar 20 17:06:26 2015 -0700" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Mon Mar 23 08:57:57 2015 -0700" }, "message": "Host dex2oat and patchoat require libcutils.\n\nWhen atrace_* functions are defined for both target and host, host\ndex2oat and patchoat require libcutils.\n\nChange-Id: I89397e83986686a2b6a6f245c25017eb379081b1\n" }, { "commit": "a473f510620958e94b586efb5398cb87f263be2f", "tree": "3fa5a538399a11ec10cc2a654b0a0ce1483ee177", "parents": [ "81d38f74219774d87389f83f23595ffc8fe814b1" ], "author": { "name": "Julien Delayen", "email": "julien.delayen@intel.com", "time": "Thu Mar 05 16:37:52 2015 +0100" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Mar 10 22:15:48 2015 -0700" }, "message": "patchoat: DisableAutoClose when patching in place\n\nInstalld can give the same input and output when\npatching directly in the dalvik-cache.\nFor this particular case, we need to DisableAutoClose\nto avoid closing twice the same fd.\n\nChange-Id: Ic1c0f1006f776e6aeb3772c83c52309c78a4bdfd\nSigned-off-by: Julien Delayen \u003cjulien.delayen@intel.com\u003e\nSigned-off-by: Guilhem IMBERTON \u003cguilhem.imberton@intel.com\u003e\n" }, { "commit": "794ad76e8d5b5b9132819d5b08a0570e27615644", "tree": "74b420e4337ecf9e5df689cc376fa965d765a908", "parents": [ "d98ff78976696fdde1e7868d4687719a0439544b" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Feb 23 08:12:24 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Feb 23 10:23:06 2015 -0800" }, "message": "ART: Introduce NO_RETURN, Mark DoLongJump noreturn\n\nAdd NO_RETURN macro that adds C++11 noreturn attribute. Mark\nDoLongJump methods as noreturn.\n\nChange-Id: Ifde4318e370493237050d4c1349285a0382df23f\n" }, { "commit": "6f6114140fbc09c5c7bec441922412635a7f7ff1", "tree": "4da7b68fddff767b668ffc4faeadb632ae310165", "parents": [ "3cfbb4aff30f32e49b3b57c7483ee0b5cb580e7b" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 21 22:25:24 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 21 22:25:24 2015 -0800" }, "message": "ART: Refactor common ELF-\u003eInstructionSet code\n\nMove code into instruction_set.h/cc.\n\nChange-Id: I34d5c82791042c68629df84e0f4b9321231d51b9\n" }, { "commit": "57b34294758e9c00993913ebe43c7ee4698a5cc6", "tree": "981821619027686f83fbe00445299b0522f1df05", "parents": [ "4945bfef00ac446d9c5458e55500229d463ab4c3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 14 15:45:59 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 15 11:32:48 2015 -0800" }, "message": "ART: Allow to compile interpret-only mips64 files\n\nInclude enough infrastructure to allow cross-compiling for mips64,\ninterpret-only. This includes the instruction-set-features, frame\nsize info and utils assembler.\n\nAlso add a disassembler for oatdump, and support in patchoat.\n\nNote: the runtime cannot run mips64, yet.\n\nChange-Id: Id106581fa76b478984741c62a8a03be0f370d992\n" }, { "commit": "c5a3ea7522b59c18daa4325d69703a6f7f743378", "tree": "22aced955bae73c2f032e7cd0fc970218a908419", "parents": [ "d277efe59cc922f6070056dc130e29281cc4955a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 13 16:41:53 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 13 20:19:56 2015 -0800" }, "message": "ART: Introduce Mips32 R6\n\nAdd an instruction-set feature for Mips R6. Currently restricted\nto the interpreter.\n\nChange-Id: Ic6d888e135bc87340229a0543dd94d0c1e863edd\n" }, { "commit": "956af0f0cb05422e38c1d22cbef309d16b8a1a12", "tree": "b558c804d206dad8da648b815750f1b3c97610ae", "parents": [ "407d77f344cfbdbbfb50531c5f0766bc0892e2fe" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Dec 11 14:34:28 2014 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Dec 12 09:33:34 2014 -0800" }, "message": "Remove portable.\n\nChange-Id: I3bf3250fa866fd2265f1b115d52fa5dedc48a7fc\n" }, { "commit": "957ca1cd025104fccb0b08928f955f9bdb4ab91c", "tree": "913ceffe992bb25898ac79977b47dc2af8b3354b", "parents": [ "4d3ed1a6f34bd31ed30faaca0433cf2a4b19bb7b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 21 16:51:29 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 24 10:44:33 2014 -0800" }, "message": "Delete ArtMethod gc_map_ field\n\nMoved the gc_map field from OatMethod to OatQuickMethodHeader.\nDeleted the ArtMethod gc_map_ field.\n\nBug: 17643507\n\nChange-Id: Ifa0470c3e4c2f8a319744464d94c6838b76b3d48\n\n(cherry picked from commit 807140048f82a2b87ee5bcf337f23b6a3d1d5269)\n" }, { "commit": "4303ba97313458491e038d78efa041d41cf7bb43", "tree": "5a5873651db918416c9ff63f4bb06b6eb7f4c71a", "parents": [ "a971100be7870544360fa8a46311ef7f5adb6902" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Nov 06 01:00:46 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 18 17:26:06 2014 -0800" }, "message": "ART: Track Flush \u0026 Close in FdFile\n\nImplement a check that aborts when a file hasn\u0027t been explicitly\nflushed and closed when it is destructed.\n\nAdd WARN_UNUSED to FdFile methods.\n\nUpdate dex2oat, patchoat, scoped_flock and some gtests to pass with\nthis.\n\n(cherry picked from commit 9433ec60b325b708b9fa87e699ab4a6565741494)\n\nChange-Id: I9ab03b1653e69f44cc98946dc89d764c3e045dd4\n" }, { "commit": "2d7210188805292e463be4bcf7a133b654d7e0ea", "tree": "7705a3bf841ae44b2396728fa22ed0b5dcb44dbf", "parents": [ "e0491682d101c69bf88c3c24a965312129cbfa38" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 10 11:08:06 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 18 12:27:37 2014 -0800" }, "message": "Change 64 bit ArtMethod fields to be pointer sized\n\nChanged the 64 bit entrypoint and gc map fields in ArtMethod to be\npointer sized. This saves a large amount of memory on 32 bit systems.\nReduces ArtMethod size by 16 bytes on 32 bit.\n\nTotal number of ArtMethod on low memory mako: 169957\nImage size: 49203 methods -\u003e 787248 image size reduction.\nZygote space size: 1070 methods -\u003e 17120 size reduction.\nApp methods: ~120k -\u003e 2 MB savings.\n\nSavings per app on low memory mako: 125K+ per app\n(less active apps -\u003e more image methods per app).\n\nSavings depend on how often the shared methods are on dirty pages vs\nshared.\n\nTODO in another CL, delete gc map field from ArtMethod since we\nshould be able to get it from the Oat method header.\n\nBug: 17643507\n\nChange-Id: Ie9508f05907a9f693882d4d32a564460bf273ee8\n\n(cherry picked from commit e832e64a7e82d7f72aedbd7d798fb929d458ee8f)\n" }, { "commit": "d582fa4ea62083a7598dded5b82dc2198b3daac7", "tree": "c76704c266ef4687eab425612ddf3fd24f93fe8d", "parents": [ "f20076ff813b8012096ff31af236d59db3c0f4e1" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Nov 05 23:46:43 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Nov 13 16:17:46 2014 -0800" }, "message": "Instruction set features for ARM64, MIPS and X86.\n\nAlso, refactor how feature strings are handled so they are additive or\nsubtractive.\nMake MIPS have features for FPU 32-bit and MIPS v2. Use in the quick compiler\nrather than #ifdefs that wouldn\u0027t have worked in cross-compilation.\nAdd SIMD features for x86/x86-64 proposed in:\n https://android-review.googlesource.com/#/c/112370/\n\nBug: 18056890\n\nChange-Id: Ic88ff84a714926bd277beb74a430c5c7d5ed7666\n" }, { "commit": "6a3c1fcb4ba42ad4d5d142c17a3712a6ddd3866f", "tree": "9df58b57af13240a93a6da4eefcf03f70cce9ad9", "parents": [ "c6e0955737e15f7c0c3575d4e13789b3411f4993" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 31 00:33:20 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Nov 03 20:01:04 2014 -0800" }, "message": "Remove -Wno-unused-parameter and -Wno-sign-promo from base cflags.\n\nFix associated errors about unused paramenters and implict sign conversions.\nFor sign conversion this was largely in the area of enums, so add ostream\noperators for the effected enums and fix tools/generate-operator-out.py.\nTidy arena allocation code and arena allocated data types, rather than fixing\nnew and delete operators.\nRemove dead code.\n\nChange-Id: I5b433e722d2f75baacfacae4d32aef4a828bfe1b\n" }, { "commit": "46774767fcf7780d1455e755729198648d08742e", "tree": "09a5d87ff0acbc7eb1fa94ec901ba10009178f03", "parents": [ "11bd683f6dbebe2f3d02fa383fc9dbc69a83ace8" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Oct 22 11:37:02 2014 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Mon Oct 27 20:19:37 2014 -0700" }, "message": "ART: Add support for patching and loading OAT files compiled with PIC\n\n* Images (.art) compiled with pic now have a new field added.\n* isDexOptNeeded will now skip patch-ing for apps compiled PIC\n* First-boot patching now only copies boot.art, boot.oat is linked\n\nAs a result, all system preopted dex files (with --compile-pic) no\nlonger take up any space in /data/dalvik-cache/\u003cisa\u003e.\n\nBug: 18035729\nChange-Id: Ie1acad81a0fd8b2f24e1f3f07a06e6fdb548be62\n" }, { "commit": "c7dd295a4e0cc1d15c0c96088e55a85389bade74", "tree": "0c08a2236bc9ba5d9a4dc75d4dd0ed2d76f8f1c6", "parents": [ "94e5af8602150efa95bde35cc9be9891ddf30135" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 21 23:31:19 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 12:06:23 2014 -0700" }, "message": "Tidy up logging.\n\nMove gVerboseMethods to CompilerOptions. Now \"--verbose-methods\u003d\" option to\ndex2oat rather than runtime argument \"-verbose-methods:\".\nMove ToStr and Dumpable out of logging.h, move LogMessageData into logging.cc\nexcept for a forward declaration.\nRemove ConstDumpable as Dump methods are all const (and make this so if not\ncurrently true).\nMake LogSeverity an enum and improve compile time assertions and type checking.\nRemove log_severity.h that\u0027s only used in logging.h.\nWith system headers gone from logging.h, go add to .cc files missing system\nheader includes.\nAlso, make operator new in ValueObject private for compile time instantiation\nchecking.\n\nChange-Id: I3228f614500ccc9b14b49c72b9821c8b0db3d641\n" }, { "commit": "d4c4d953035d4418126d36517e402f411d6a87f3", "tree": "735aacf812bbac7c1ae7c0788c1ca6f58cfa82ee", "parents": [ "6f3dbbadf4ce66982eb3d400e0a74cb73eb034f3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 16 20:31:53 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 16 21:30:37 2014 -0700" }, "message": "Some code clean-up.\n\nChange-Id: I4b745fd5298cd61c793e3b57514b48347bd66c0e\n" }, { "commit": "6e88ef6b604a7a945a466784580c42e6554c1289", "tree": "1e296564787b51514cf2eca5b732647c1a82912e", "parents": [ "58e51f38e2304a08aa9ec380383e0b3614f96a96" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 14 15:01:24 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 14 15:43:21 2014 -0700" }, "message": "Change MemMap::maps_ to not be global variable\n\nRuntime.exit() was causing globals to get destructed at the same time\nthat another thread was using it for allocating a new mem map.\n\nBug: 17962201\nChange-Id: I400cb7b8141d858f3c08a6fe59a02838c04c6962\n" }, { "commit": "7223d44a4893522e90d00bca38b119f710e55122", "tree": "c0bbe3f7b3cf04750b4b5359a03c2d63fc90978b", "parents": [ "3f5b8a1e7dfdf010591e17396d1d126740c5706c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 20:05:39 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 20:05:39 2014 -0700" }, "message": "Switch to C++11 style [[noreturn]].\n\nAlso remove old comment about calls to Abort getting merged. We have few\nLOG(FATAL)s so merging is less of an issue. Also, recompiling with O0 and\nsimilar will avoid the optimization and we expect that for GDB debugging.\n\nChange-Id: I840dc6f4b8511294be0e117c634ec15b745e7be5\n" }, { "commit": "647b1a86f518d8db0331b3d52a96392b7a62504b", "tree": "7370f795ef3c7fbdd2695d23bc6f8171f40f43f1", "parents": [ "acfbbd4df2fc1c79a7102587bebf398f95b5e5de" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 11:02:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 12:26:02 2014 -0700" }, "message": "Fix 2 new sets of clang compiler warnings.\n\nFix issues that are flagged by -Wfloat-equal and -Wmissing-noreturn.\nIn the case of -Wfloat-equal the current cases in regular code are deliberate,\nso the change is to silence the warning. For gtest code the appropriate fix is\nto switch from EXPECT_EQ to EXPECT_(FLOAT|DOUBLE)_EQ.\nThe -Wmissing-noreturn warning isn\u0027t enabled due to a missing noreturn in\ngtest. This issue has been reported to gtest.\n\nChange-Id: Id84c70c21c542716c9ee0c41492e8ff8788c4ef8\n" }, { "commit": "13735955f39b3b304c37d2b2840663c131262c18", "tree": "0a731ac42b8230f9929172fa3e3d8051874e2b18", "parents": [ "25b18bbdaa36ff936eb44f228f0518d4223e9d52" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 08 12:43:28 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 16:05:58 2014 -0700" }, "message": "stdint types all the way!\n\nChange-Id: I4e4ef3a2002fc59ebd9097087f150eaf3f2a7e08\n" }, { "commit": "8c52a3f534c49580688239549fca937b36fc0aed", "tree": "1574a4717541ed23930d5be3e3825f069c279057", "parents": [ "fc95176845e1db99115202d7f64c03d521ca0aab" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Sep 30 16:18:01 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Oct 03 17:23:09 2014 -0700" }, "message": "Add some missing TEMP_FAILURE_RETRYs around unlink(2)\n\nChange-Id: Ibf446d37092933dce165d7a95c0ff2e52b94ce46\n" }, { "commit": "62d1ca3182a6cbb921799825f43ad36821233fd7", "tree": "54d9663f5ce10f41e95fe774d4e4841c1a78bbc3", "parents": [ "63462448ca4e377074a10a4720aa22f71154dbe9" ], "author": { "name": "Tong Shen", "email": "endlessroad@google.com", "time": "Wed Sep 03 17:24:56 2014 -0700" }, "committer": { "name": "Tong Shen", "email": "endlessroad@google.com", "time": "Fri Oct 03 11:25:11 2014 -0700" }, "message": "ART: Prepare for ELF64.\n\nOnly expose necessary interface in ElfFile, and move all details into template class ElfFileImpl.\n\nChange-Id: I9df2bbc55f32ba0ba91f4f3d5d0009e84a2ddf74\n" }, { "commit": "c8ccf68b805c92674545f63e0341ba47e8d9701c", "tree": "fb360323538cb242ebf7c5c0aca27d3a0bce0abb", "parents": [ "fcabfbe577c0fd40910b565beb681bd4b66f6c5d" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 29 20:07:43 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 29 20:13:48 2014 -0700" }, "message": "ART: Fix some -Wpedantic errors\n\nRemove extra semicolons.\n\nDollar signs in C++ identifiers are an extension.\n\nNamed variadic macros are an extension.\n\nBinary literals are a C++14 feature.\n\nEnum re-declarations are not allowed.\n\nOverflow.\n\nChange-Id: I7d16b2217b2ef2959ca69de84eaecc754517714a\n" }, { "commit": "3c13a794845e0cf7887e33b2ec20de7e6ba85f8f", "tree": "ab9a9a426a371de0c32e48a85fdda44a321a06a0", "parents": [ "5cdd0734d2f79eedc530f5f1e876cd2110e29c86" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 18 20:56:04 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 18 23:43:07 2014 -0700" }, "message": "ART: Only allow the zygote to create the global boot image\n\nDo not allow arbitrary processes, even when root, to write the\nboot image in /data/dalvik-cache.\n\nBug: 17478752, 17510489, 17439961\nChange-Id: Iba2b74be6d0752f4221f4ff5ee295b45a34cb2e1\n(cherry picked from commit 33c36d4f22ab6a5e61eb47b654deaf647c34e49c)\n" }, { "commit": "daab38ca60c5b91787e29c87a161a2bb8c1b6f11", "tree": "0b155557e516ec1ff571c359af904e00d31d0c43", "parents": [ "d2e5a6934ca1a07c9e1e87d3d0b21b63e1011934" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 12 18:38:24 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 15 22:34:11 2014 -0700" }, "message": "ART: Make elf loading not abort\n\nChanges elf_file code to use less CHECKs and instead return error\nvalues (usually nullptr). This avoids aborts.\n\nIn oat_file, when loading an oat file fails, try to unlink at. If\nthis succeeds, on the next run we may compile again.\n\nBug: 17491333\n\n(cherry picked from commit afa6b8e93a0dc0de33c9d404945c7c5621e20b1a)\n\nChange-Id: I50fdd2edacd86f25d4dacf2180ce2a6105eaf4af\n" }, { "commit": "20c89303a9d89ba857bf969ad14a31f12c3be878", "tree": "b7072ce69f3c6896e70e84cfb7ae1058e7e7a55f", "parents": [ "192da5675b2e219f26a107bd7b60c755cd46ba74" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 19 17:28:06 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 20 15:25:48 2014 -0700" }, "message": "ART: Relax GetInstructionSetFromString\n\nDo not abort on an unknown instruction set string. Instead return\nkNone and let the caller handle this.\n\nAlso simplify the patchoat tool to use this.\n\nBug: 17136416\n\n(cherry picked from commit aabbb2066a715b3fd8e752291f74c6d77b970450)\n\nChange-Id: I24131914bcf91c04ae93179bf809a2907f1f2b7a\n" }, { "commit": "e3ea83811d47152c00abea24a9b420651a33b496", "tree": "dd3b8018176ada85d51b2f8ca46e515fbf55b50f", "parents": [ "9dcf75c80187504ec88e7ef91d64a6a68279eb9d" ], "author": { "name": "Yevgeny Rouban", "email": "yevgeny.y.rouban@intel.com", "time": "Fri Aug 08 16:29:38 2014 +0700" }, "committer": { "name": "Tong Shen", "email": "endlessroad@google.com", "time": "Fri Aug 15 15:04:12 2014 -0700" }, "message": "ART source line debug info in OAT files\n\nOAT files have source line information enough for ART runtime needs like\njump to/from interpreter and thread suspension. But this information\nis not enough for finer grained source level debugging and low-level\nprofiling (VTune or perf).\n\nThis patch adds to OAT files two additional sections:\n.debug_line - DWARF formatted Elf32 section with detailed source line\n information (mapping from native PC to Java source lines).\n\nIn addition to the debugging symbols added using the dex2oat option\n--include-debug-symbols, the source line information is added to\nthe section .debug_line.\n\nThe source line info can be read by many Elf reading tools like objdump,\nreadelf, dwarfdump, gdb, perf, VTune, ...\n\ngdb can use this debug line information in x86. In 64-bit mode\nthe information can be used if the oat file is mapped in the lower\naddress space (address has higher 32 bits zeroed). Relocation works.\n\nTesting:\n1. art/test/run-test --host --gdb [--64] 001-HelloWorld\n2. in gdb: break Main.java:19\n3. in gdb: break Runtime.java:111\n4. in gdb: run - stops at void java.lang.Runtime.\u003cinit\u003e()\n5. in gdb: backtrace - shows call stack down to main()\n6. in gdb: continue - stops at void Main.main() (only in 32-bit mode)\n7. in gdb: backtrace - shows call stack down to main()\n8. objdump -W \u003coat-file\u003e - addresses are from VMA range of .text\n section reported by objdump -h \u003cfile\u003e\n9. dwarfdump -ka \u003coat-file\u003e - no errors expected\n\nSize of aosp-x86-eng boot.oat increased by 11% from 80.5Mb to 89.2Mb\nwith two sections added .debug_line (7.2Mb) and .rel.debug (1.5Mb).\n\nChange-Id: Ib8828832686e49782a63d5529008ff4814ed9cda\nSigned-off-by: Yevgeny Rouban \u003cyevgeny.y.rouban@intel.com\u003e\n" }, { "commit": "76ab347dc9b89970da1451568965ee208c728c43", "tree": "9581645023d8ebceb1d9385145648f37a83cebe2", "parents": [ "99c251bbd225dd97d0deece29559a430b12a0b66" ], "author": { "name": "Junmo Park", "email": "junmoz.park@samsung.com", "time": "Mon Aug 11 21:28:16 2014 +0900" }, "committer": { "name": "Junmo Park", "email": "junmoz.park@samsung.com", "time": "Wed Aug 13 08:21:05 2014 +0900" }, "message": "Fix art build script\n\nUse ART_BUILD_HOST_NDEBUG instead of ART_BUILD_NDEBUG.\n\nChange-Id: I0ff590552f47d3354287a155b51936a7aef82f1f\nSigned-off-by: Junmo Park \u003cjunmoz.park@samsung.com\u003e\n" }, { "commit": "4b0d2d99b596eebd6da45679896d3af54bc5a935", "tree": "9c5f70398324b53edb47d1819648618f58ca24e4", "parents": [ "484e2c2d3531e5bb36f0e1e12f26c708939c6579" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Aug 06 13:37:23 2014 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Aug 06 15:12:32 2014 -0700" }, "message": "Make patchoat use the sht_entsize when patching.\n\nThis fixes an occasional problem with multiarch use of patchoat.\n\nChange-Id: I80799de36774720bd985704f9b709a8378bb5af5\n" }, { "commit": "cf4bf386ef3f527825c70e01130b9276da4f786a", "tree": "0f99e56bc53e150e070447410a66e8e5acca4a0a", "parents": [ "9fbe946f49724ff7ae30da5d366a002d1e6addfa" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jul 24 11:29:14 2014 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jul 25 09:47:56 2014 -0700" }, "message": "Tweaks to patchoat and other related things.\n\nRemoved some flags from patchoat that were poorly specified and fixed\nsome other issues with the relocation system.\n\nBug: 15358152\n\nChange-Id: Ia6d47b1a008f02373307d833ba45f00ea408d76f\n" }, { "commit": "1c83cbc4a817acbd7f9abb5b29a2d418a958e6a1", "tree": "aae7ea1d5559718d6e264175aa1cb2ae1fc1e9ed", "parents": [ "7dfc30b591aee167f0a38ab61a90894cc76f2066" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 22 18:52:29 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 23 10:03:00 2014 -0700" }, "message": "ART: Make run tests out of oat tests\n\nTransforms all former oat tests into run tests.\n\nChange-Id: I190dd39456454c36e5538a2c044d993965a67533\n" }, { "commit": "a59dd80f9f48cb750d329d4d4af2d99d72b484d1", "tree": "36958b15842205addaf6d2a13e40823eab47c8bc", "parents": [ "84568fdf08f8f476292996ad653b4453d2894d23" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Jul 02 16:28:08 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 22 08:24:14 2014 -0700" }, "message": "Runtime can now be set to require relocation\n\nAdd a pair of runtime flags -Xrelocate and -Xnorelocate that can force\nthe runtime to require that all files that are run are relocated, to\nprevent attacks based on the known art base address.\n\nAdd support for running patchoat on oat files compiled without an image.\n\nChange run-test to have new --prebuild and --relocate flags.\n\nBug: 15358152\n\nChange-Id: I91166c62dd1ab80e5cbcb7883a2cd0d56afca32d\n" }, { "commit": "e63db27db913f1a88e2095a1ee8239b2bb9124e8", "tree": "893dee6783bca6717259321a6e4ba029c9c123e2", "parents": [ "07b8441303ea82fca3cb85d71ecf8752d73cedd7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 15:36:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 17:07:49 2014 -0700" }, "message": "Break apart header files.\n\nCreate libart-gtest for common runtime and compiler gtest routines.\nRename CompilerCallbacksImpl that is quick compiler specific.\nRename trace clock source constants to not use the overloaded profiler term.\n\nChange-Id: I4aac4bdc7e7850c68335f81e59a390133b54e933\n" }, { "commit": "eefbe39799126afdf7d315a79359b9da23d1cef5", "tree": "7818d2124be53d5b01c11118ba6fe26f86adc0d5", "parents": [ "c4c601b6dbe5e1a7321d663b36a531a87f21e931" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jul 08 09:53:18 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Jul 08 10:06:08 2014 -0700" }, "message": "Fix some style nitpicks\n\nChange-Id: Icfdd327f4ddf129f0a8607162c09ba271c1d49d9\n" }, { "commit": "53cb16b98acf3cf6f3a1e2204ad4958ecf1b5a3c", "tree": "c0129ef3de7148dc6a114449b4f751a560283eb0", "parents": [ "ae2efea4582df773f80be274bdc754f732b07df3" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jun 12 11:26:29 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Jul 07 15:19:58 2014 -0700" }, "message": "Add patchoat tool to Art.\n\nAdd a new executable called patchoat to art. This tool takes already\ncompiled images and oat files and changes their base address, acting as\na cheap form of relocation.\n\nAdd a --include-patch-information flag to dex2oat and code to add\nrequired patch information to oat files created with the quick compiler.\n\nBug: 15358152\n\nChange-Id: Ie0c580db45bb14ec180deb84930def6c3628d97d\n" } ] }