)]}' { "log": [ { "commit": "ccc61972ca31bbb3bf82cdc30656c13bebfbe6a9", "tree": "2d04de6996ec6273a36ac485f0297a3a2771152e", "parents": [ "6387821209a03c5d873cf9dc6fd11434918bbdf4" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Oct 01 14:34:20 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Oct 02 09:58:48 2015 +0100" }, "message": "Do more inlining when JITting.\n\nWe now check the verification status of the class.\n\nThis triggered a bug when an inlined method was not compiled\n(typically in a JIT configuration), and the path for deopting in\nStackVisitor was using the wrong ArtMethod in order to know the\ncompiler that compiled the current frame.\n\nChange-Id: I81d3ca0cf5cd3864b83b63dd954c58e1f2adaad4\n" }, { "commit": "b30259251b22430fad12f1adeab671e4bf8f88f5", "tree": "80bff979beae84702b739962656a16d245a40ef4", "parents": [ "eae0927cb81437389dc96437e9f04903783282b4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 01 14:45:00 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 24 15:42:14 2015 -0700" }, "message": "ART: Use unique_ptr for alloca-ed ShadowFrames\n\nTo enable automated calls of the destructor (for extensions of\nShadowFrame), return a unique_ptr with a custom deleter. Use a\nmacro so that the alloca happens in the caller.\n\nChange-Id: Id28a6ea131f108e4a94ff1699fc22e7b44aec018\n" }, { "commit": "99170c636dfae4908b102347cfe9f92bad1881cc", "tree": "1ecff5dd69ea20b09e622cb88e7835a55dfa4be8", "parents": [ "5836f8465ddd0a7f5cbeec9c30d739bd0619867f" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Jul 06 11:10:37 2015 -0700" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Sep 17 17:07:22 2015 +0200" }, "message": "Deoptimization support in optimizing compiler for setting local values\n\nDue to compiler optimizations, we may not always be able to update\nthe value of a local variable in a compiled frame (like a variable\nseen as constant by the compiler). To avoid that situation, we simply\ndeoptimize compiled frames updated by the debugger so they are\nexecuted by the interpreter with the updated value.\n\nWhen the debugger attempts to set a local variable (actually a DEX\nregister or a pair of registers) in a compiled frame, we allocate a\nShadowFrame associated to that frame (using its frame id) and set the\nnew value in that ShadowFrame. When we know we are about to continue\nthe execution of the compiled frame, we deoptimize the stack using\nthe preallocated ShadowFrame (instead of creating a new one). We\ninitialize it with the current value of all DEX registers except\nthe ones that have been set by the debugger. Therefore, the\nShadowFrame represent the runtime context modified by the debugger.\n\nBumps oat version to force recompilation.\n\nBug: 19944235\nChange-Id: I0ebe6241264f7a3be0f14ee4516c1f7436e04da6\n" }, { "commit": "26f728661a08062a373a3203b72dc2555c2aed2d", "tree": "4f6e8168ac7ca422c7de3c7cdda6db21958358fd", "parents": [ "b6f7dd330f115fc977d1d5a10122c41c9dd7c210" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 15 09:52:07 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 15 11:15:12 2015 +0200" }, "message": "Cleanup thread access in StackVisitor\n\nAdds method StackVisitor::GetThread to give access to the visited\nThread* so we no longer need to copy that pointer in subclasses.\n\nAlso adds a few missing const and DISALLOW_COPY_AND_ASSIGN.\n\nChange-Id: I57649ee7742ef4ef1e01447ac2fbb66f977b22eb\n" }, { "commit": "2a5c4681ba19411c1cb22e9a7ab446dab910af1c", "tree": "883ea0c07aad9efdb7c86960056cbefd7992b2bc", "parents": [ "228b3973b2b24783c727a55fda2b4b80375f7207" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Aug 14 08:22:54 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 17 15:42:25 2015 -0700" }, "message": "ART: Some header cleaning around bit-utils\n\nTry to remove dependencies where they are not necessary.\n\nChange-Id: I5ff35cb17aea369bed3725b1610b50d7eb05b81e\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": "c449e8b79aaaf156ce055524c41474cc1200ed5a", "tree": "8e36a6ce3a1172891760e8dbc12cd73916c7264c", "parents": [ "26b97f87dcd86934f8b75e018c80bb4e8d501b03" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Jun 10 15:56:42 2015 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Jun 11 11:13:30 2015 -0700" }, "message": "runtime: Minor cleanup and extra comments around interpreter\n\nChange-Id: I24c0b261de8cf737babd9d01bf679482d48c8bc9\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": "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": "57f61612a6101a2d246727640676acf8d8a6b51c", "tree": "03f63405d5bf4fe38ed580721d05ca5b270c3b66", "parents": [ "17f1bc531ea2f8c1a6fac3def13dee1b901949dd" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri May 15 13:20:41 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 18 14:58:41 2015 +0100" }, "message": "Stack walking of inlined frames.\n\nBe \"true\" to the stack visitor design, which seems to be:\n1) Carry minimal state, so this CL just adds an inlining depth to it.\n2) Compute needed information for each method, which is what GetDexPc,\n GetMethod, and GetVRegFromOptimized code do.\n\nChange-Id: I3d14126766f12216a7b448ce856259f1e7f37395\n" }, { "commit": "8e5bd18fc665d7ec5461ea068e98740a65da754c", "tree": "83441cdfdab06709b573aad2ab731cc65c10b9f1", "parents": [ "cdeb0b5fede4c06488f43a212591e661d946bc78" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 06 11:34:34 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 11 15:23:25 2015 +0100" }, "message": "Add a flag to StackVisitor for inlining.\n\nThe flag tells whether the stack walk needs to include inlined\nJava frames.\n\nThis does not do anything just yet, as we\u0027re not inlining anyways.\n\nChange-Id: I716e25094fe56fa335ca1f9a398c1bcdba478e73\n" }, { "commit": "50030ef998be09789da4a9a56738362852068f12", "tree": "02df1bd0954f9b970434d1363466b0cfe182bc12", "parents": [ "6727a48193db2a0cf01af971cccffe1a6518c247" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 08 14:19:26 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 08 15:09:17 2015 -0700" }, "message": "Check IsReferenceVReg during deopt\n\nRequired since the quick GC maps may not agree with the verifier\nones. Without this check we may copy stale object references into\nthe shadow frame.\n\n(cherry picked from commit f00baf56ef647684888a407dbb6adadd704a2039)\n\nBug: 20736048\nChange-Id: I7783c8a8ee45cf601b08b4c38f1dec7f7d11380c\n" }, { "commit": "241a9588c6d7d0fcb4c92da40c7141863930083a", "tree": "4a6868893dc742f6322775d198a2305f645f2f3f", "parents": [ "a93b104b47da67eaa50f8a6b7280cc6c438ef37b" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Mon Apr 27 15:19:41 2015 -0700" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Mon May 04 12:10:07 2015 -0700" }, "message": "Fix mismatched new[]/delete.\n\nAnother two cases where a new[] is used but only a delete occurs.\n\nBug: 18202869\nChange-Id: If68264807150f3a9783e44ef8823cc366bff8df2\n" }, { "commit": "7cc56a1fe81475d5126785c45606dd491dec2d8c", "tree": "13ad863b2a693f68da23b36c24703ca990da5475", "parents": [ "9e85d55c085504b88a9ac68c3f9605629dc06853" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Apr 24 14:58:19 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Apr 24 14:58:19 2015 +0100" }, "message": "Debugger: Remove support for setting values in optimizing.\n\nbug:19944235\nChange-Id: I3bcd56c7844847a7f0367f8ce6a72bddcd09d441\n" }, { "commit": "2cebb24bfc3247d3e9be138a3350106737455918", "tree": "d04d27d21b3c7733d784e303f01f873bb99e7770", "parents": [ "1f02f1a7b3073b8fef07770a67fbf94afad317f0" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 21 16:50:40 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 12:44:27 2015 -0700" }, "message": "Replace NULL with nullptr\n\nAlso fixed some lines that were too long, and a few other minor\ndetails.\n\nChange-Id: I6efba5fb6e03eb5d0a300fddb2a75bf8e2f175cb\n" }, { "commit": "bb87e0f1a52de656bc77cb01cb887e51a0e5198b", "tree": "113f014c6e20fab3e936a3ac05f9f738639541f6", "parents": [ "e57fc0f0260fcb1d08cbb720ec95c04c0f394b91" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 03 11:21:55 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 06 10:44:37 2015 -0700" }, "message": "Refactor and improve GC root handling\n\nChanged GcRoot to use compressed references. Changed root visiting to\nuse virtual functions instead of function pointers. Changed root visting\ninterface to be an array of roots instead of a single root at a time.\nAdded buffered root marking helper to avoid dispatch overhead.\n\nRoot marking seems a bit faster on EvaluateAndApplyChanges due to batch\nmarking. Pause times unaffected.\n\nMips64 is untested but might work, maybe.\n\nBefore:\nMarkConcurrentRoots: Sum: 67.678ms 99% C.I. 2us-664.999us Avg: 161.138us Max: 671us\n\nAfter:\nMarkConcurrentRoots: Sum: 54.806ms 99% C.I. 2us-499.986us Avg: 136.333us Max: 602us\n\nBug: 19264997\n\nChange-Id: I0a71ebb5928f205b9b3f7945b25db6489d5657ca\n" }, { "commit": "15b9d5274399736ac09705f0507df24fac4f00c1", "tree": "04564a9265f5dccefdd32ea7bdd25adc0267f80b", "parents": [ "8bc616d09f93523f4bc982cc60c377b00161522a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Mar 12 15:05:13 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 13 10:26:47 2015 +0000" }, "message": "API change in StackVisitor::GetVReg*.\n\n- Remove GetVReg() and SetVReg() that were expecting to always succeed.\n- Change Quick-only methods to take a FromQuickCode suffix.\n- Change deopt to use dead values when GetVReg does not succeed:\n the optimizing compiler will not have a location for uninitialized\n Dex registers and potentially dead registers.\n\nChange-Id: Ida05773a97aff8aa69e0caf42ea961f80f854b77\n" }, { "commit": "0aa50ce2fb75bfc2e815a0c33adf9b049561923b", "tree": "9a3f9603ab30d5cbc7fc21aee0ceb48bbb0dd25a", "parents": [ "e8e42f3548fd894f860912bb1b71ce6fa2d7daf3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 11:03:29 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 14:51:11 2015 +0000" }, "message": "Remove ThrowLocation.\n\nNote that this is a cleanup change, and has no functionality change.\nThe ThrowLocation had no use anymore.\n\nChange-Id: I3d2126af1dc673cec3a0453ff3d56a172663a5f6\n" }, { "commit": "7cde48c56df5b57aed524cce44c902bc720f2d6c", "tree": "b9b9e33b29f46bfe0c2da89c7e3e87c8ee419ccc", "parents": [ "4b39eeea67b0fecf21588d7b00e92eb844014c24" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jan 20 16:06:43 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 03 11:22:00 2015 +0000" }, "message": "Stack support for Optimizing compiler\n\nAllows to read/write DEX registers from physical register or stack\nlocation when the method is compiled with the Optimizing compiler.\n\nRequired fixing arm and arm64 JNI compiler by saving floating\npoint registers.\n\nBug: 18547544\nChange-Id: I401579f251d1c0a130f6cf4a93a960cdcd7518f5\n" }, { "commit": "96ba8dc82e7bd859106af837623fe8b2e9e772c3", "tree": "d7df58e8a9fb853621700d90ec4864129f6b3a0a", "parents": [ "081d27817608c2ea035f2473c4ea1062a79bccef" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Jan 22 18:57:14 2015 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Jan 26 16:45:38 2015 +0100" }, "message": "Refactor register access from StackVisitor\n\nMoves register access checking up to StackVisitor by adding methods\nIsAccessibleGPR and IsAccessibleFPR in Context class. It allows to\nsimplify GetGPR/FPR and SetGPR/FPR methods in the Context class (and\nits subclasses).\n\nAlso simplifies code in StackVisitor by adding IsAccessibleRegister,\nGetRegister and SetRegister methods which then call either GPR or FPR\nspecific methods in Context depending on the nature of the accessed\nregister.\n\nBug: 18547544\nBug: 19106446\nChange-Id: I6e707608d935a71571d0e975a6e766053de3763a\n" }, { "commit": "2cd334ae2d4287216523882f0d298cf3901b7ab1", "tree": "b72d3d07e5a04151caca96cae345075b6e4452b0", "parents": [ "604e2828896fbb8663897d1e75112da7305ead4c" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Jan 09 14:03:35 2015 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Jan 23 14:07:32 2015 -0800" }, "message": "More of the concurrent copying collector.\n\nBug: 12687968\nChange-Id: I62f70274d47df6d6cab714df95c518b750ce3105\n" }, { "commit": "e34fa1df67fbe0173b4ea9abddcc3ae3d0537037", "tree": "a5148f079b5671a95f60910c41981ebf91db3a02", "parents": [ "9f06b1946ae3ef1d2fd75bbf7f0a288bc611fe58" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 14 14:55:47 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 15 12:23:28 2015 -0800" }, "message": "Print more info in MarkSweep::VerifyRoot\n\nRefactored old root callback to use a new class called RootInfo.\nRootInfo contains all the relevant info related to the root\nassociated with the callback. The MarkSweep::VerifyRoot function\nnow uses this info to print the StackVisitor\u0027s described location\nif the GC root is of the type kRootJavaFrame.\n\nSome other cleanup.\n\nExample output:\nE/art (12167): Tried to mark 0x123 not contained by any spaces\nE/art (12167): Attempting see if it\u0027s a bad root\nE/art (12167): Found invalid root: 0x123 with type RootJavaFrame\nE/art (12167): Location\u003dVisiting method\n\u0027void java.lang.Runtime.gc()\u0027 at dex PC 0xffffffff (native PC 0x0)\nvreg\u003d0\n\n(cherry picked from commit 12f7423a2bb4bfab76700d84eb6d4338d211983a)\n\nBug: 18588862\nChange-Id: Ic5a2781f704e931265ffb3621c2eab4b2e25f60f\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": "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": "1d8cdbc5202378a5f1a4b3a1fba610675ed4dcd5", "tree": "6a1b0f49aee5a97b513bd0becc734d284aa7fb65", "parents": [ "1c1786f193323d3bd706463894001117f3471595" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Sep 22 22:51:09 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Oct 20 16:01:28 2014 -0700" }, "message": "Refactor quick entrypoints\n\nRemove FinishCalleeSaveFrameSetup.\nAssembly routines write down anchor into TLS as well as placing runtime\nmethod in callee save frame.\nSimplify artSet64InstanceFromCode by not computing the referrer from the\nstack in the C++ code.\nMove assembly offset tests next to constant declaration and tidy arch_test.\n\nChange-Id: Iededeebc05e54a1e2bb7bb3572b8ba012cffa1c8\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": "2cbaccb67e22c0b313a9785bfc65bcb4b25d0676", "tree": "daeb766e19880b651fd9c4a719c9a07dd7d4bd0e", "parents": [ "bace0378d720a1d2938ec7f6be17e2814671d20a" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Sep 14 20:34:17 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Sep 16 12:50:08 2014 -0700" }, "message": "Avoid printing absolute addresses in oatdump\n\n- Added printing of OatClass offsets.\n- Added printing of OatMethod offsets.\n- Added bounds checks for code size size, code size, mapping table, gc map, vmap table.\n- Added sanity check of 100k for code size.\n- Added partial disassembly of questionable code.\n- Added --no-disassemble to disable disassembly.\n- Added --no-dump:vmap to disable vmap dumping.\n- Reordered OatMethod info to be in file order.\n\nBug: 15567083\n\n(cherry picked from commit 34fa79ece5b3a1940d412cd94dbdcc4225aae72f)\n\nChange-Id: I2c368f3b81af53b735149a866f3e491c9ac33fb8\n" }, { "commit": "8d0d03e24325463f0060abfd05dba5598044e9b1", "tree": "06e8ed7e47a4cfe108d4ed750de6a60e588b2f7a", "parents": [ "709368e616791209b02d39adb6da5e55782cb45f" ], "author": { "name": "Razvan A Lupusoru", "email": "razvan.a.lupusoru@intel.com", "time": "Fri Jun 06 17:04:52 2014 -0700" }, "committer": { "name": "Razvan A Lupusoru", "email": "razvan.a.lupusoru@intel.com", "time": "Tue Aug 26 18:36:46 2014 -0700" }, "message": "ART: Change temporaries to positive names\n\nChanges compiler temporaries to have positive names. The numbering now\nputs them above the code VRs (locals + ins, in that order). The patch also\nintroduces APIs to query the number of temporaries, locals and ins.\n\nThe compiler temp infrastructure suffered from several issues\nwhich are also addressed by this patch:\n-There is no longer a queue of compiler temps. This would be polluted\nwith Method* when post opts were called multiple times.\n-Sanity checks have been added to allow requesting of temps from BE\nand to prevent temps after frame is committed.\n-None of the structures holding temps can overflow because they are\nallocated to allow holding maximum temps. Thus temps can be requested\nby BE with no problem.\n-Since the queue of compiler temps is no longer maintained, it is no\nlonger possible to refer to a temp that has invalid ssa (because it\nwas requested before ssa was run).\n-The BE can now request temps after all ME allocations and it is guaranteed\nto actually receive them.\n-ME temps are now treated like normal VRs in all cases with no special\nhandling. Only the BE temps are handled specially because there are no\nreferences to them from MIRs.\n-Deprecated and removed several fields in CompilationUnit that saved\nregister information and updated callsites to call the new interface from\nMIRGraph.\n\nChange-Id: Ia8b1fec9384a1a83017800a59e5b0498dfb2698c\nSigned-off-by: Razvan A Lupusoru \u003crazvan.a.lupusoru@intel.com\u003e\nSigned-off-by: Udayan Banerji \u003cudayan.banerji@intel.com\u003e\n" }, { "commit": "649278cec7119cdd1bea3d0b710dbb2aa7c650b6", "tree": "6b5e08abee97e8af1ee5635488e6476d3fb3dc15", "parents": [ "99c251bbd225dd97d0deece29559a430b12a0b66" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Aug 13 11:12:22 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Aug 13 20:46:10 2014 -0700" }, "message": "More efficient stack walk in exception throwing.\n\nIn the exception handling code, we currently walk down the stack\ntwice, once to get the stack height which we use to compute frame IDs\n(the bottom frame is zero), and once more to find the catch block to\njump to.\n\nFor a deep stack, this could result in very slow exception\nhandling. That is, if have a lot of finally or catch blocks that we\nend up jumping to in a deep stack, we need to do a lot of\ncatch/rethrow chains. Since we\u0027d need to walk down to the bottom each\ntime to compute frames IDs in each catch/rethrow, we\u0027d need to walk\ndown O(N^2) frames at the worst case.\n\nInstead of frames IDs ((the bottom frame is zero), we will use the\nframe depth (the top frame is zero) and no longer need to walk down\nthe stack just to get the stack height. We walk down O(N) frames.\n\nThis was what was happening with\ncode.google.gson.functional.CircularReferenceTest. With this change,\nthe test run time went from ~120s down to ~3s on N5 and it no longer\ncrashes due to the thread suspension timeout.\n\nBug: 16800209\nChange-Id: Ie815df1e3e8fb9d82e40685d4cc2b8838fd8aa07\n" }, { "commit": "c901dd7bdc80b953d04100ef2f54b8d1ca5f466b", "tree": "de52f83ac4dab4ca5b48cbc5a4701abdc762f2d1", "parents": [ "81457a3cd8fca14396b5785a4e4c8070c259b07a" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Jul 16 11:56:07 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Jul 17 20:43:37 2014 +0200" }, "message": "Fix access to long/double stack values from debugger\n\nLong and double values live in a pair of DEX registers. When we compile DEX\ncode with the Quick compiler, a DEX register either lives in the stack or is\npromoted to a physical register. In the case of a pair of DEX registers, the\nQuick compiler assumes both registers live in the same \"area\": both live in\nthe stack or both are promoted to physical registers.\n\nFrom the debugger, we used to access these values by reading/writing each DEX\nregister separately. However, this does not work when only one DEX register of\na pair is promoted and the other lives in the stack. In this case, the compiled\ncode reads from/writes to the stack only.\n\nTo fix that, the debugger must follow the same rule than the Quick compiler: if\nboth DEX registers are promoted, read/write them from/to physical registers,\notherwise read/write them from/to the stack. We add StackVisitor:GetVRegPair and\nStackVisitor:SetVRegPair for this purpose.\n\nWe also follow the same rule when deoptimizing. However we need to do that only\nwhen we know two consecutive DEX registers are part of a pair (long or double).\nWe know that thanks to the verifier.\n\nBug: 15527793\nChange-Id: I04812285ff26ef0129f39792a1cf776f3591ca2d\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": "0bcb2902ec21393d71c94e63aa6733cb5311a0cc", "tree": "10beb60b5a8d212afdf0e7e58c5dfcbee691be2e", "parents": [ "838b38fa3b2fb4a64f8a316459d372020f6e8feb" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 17 15:52:45 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 17 16:00:54 2014 +0200" }, "message": "Revert \"Revert \"Fix access to FP registers when visiting stack\"\"\n\nThis reverts commit 8ebd94ab2e0d9867a7d384f00fa4cab24235216f.\n\nFixes StackVisitor::GetVReg to read register value in a uintptr_t local and\ncast it into uint32_t pointer argument.\n\nBug: 15433097\nChange-Id: I4e13ed5446e823e9ec50fbc378b16be5b17b2294\n" }, { "commit": "8ebd94ab2e0d9867a7d384f00fa4cab24235216f", "tree": "5fc48d8179f6ec6942ebada59bc88c4626608410", "parents": [ "aa9b3aee1e06f922e4518713f9b3dff00a0b2597" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 17 09:49:21 2014 +0000" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 17 09:49:21 2014 +0000" }, "message": "Revert \"Fix access to FP registers when visiting stack\"\n\nThis reverts commit aa9b3aee1e06f922e4518713f9b3dff00a0b2597.\n\nChange-Id: Ied27deb89cca5ec9094d391374e03f83fcb76c33\n" }, { "commit": "aa9b3aee1e06f922e4518713f9b3dff00a0b2597", "tree": "992b4565732a728dd901ed8a2f29c9246de328b7", "parents": [ "bc72903b909f5147b8cb207f3e5d02a8ef85e4e7" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Jun 13 14:49:27 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 17 11:13:03 2014 +0200" }, "message": "Fix access to FP registers when visiting stack\n\nAdds GetFPR and SetFPR to Context class so we can read from and write to\nfloating-point registers during stack visit. They return a boolean flag\nindicating whether the read/write is successful. This allows the debugger to\nreturn the JDWP error ABSENT_INFORMATION when we can\u0027t read/write a register.\nWe also update GetGPR and SetGPR for consistency. We keep a default GetGPR\nimplementation asserting the read was successful using a CHECK so we don\u0027t\nsilently fail.\n\nAdds missing JDWP object tags for StackFrame.SetValues to avoid crash when\nsetting corresponding objects (thread, thread group, class object or class\nloader). Also returns JDWP error INVALID_OBJECT (when the given object id is\ninvalid) instead of crashing with an unimplemented message.\n\nBug: 15433097\nChange-Id: I70843c9280e694aec1eae5cf6f2dc155cb9ea10e\n" }, { "commit": "828186492aac39397ee4c139e54c2767bf3d7b66", "tree": "73ccb5e4e32f4a697e34f60781aab42450d88bf0", "parents": [ "903e390ffd46955628d1161ccfa8de9177d42d22" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Wed Jun 04 15:35:41 2014 -0700" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Wed Jun 04 16:00:40 2014 -0700" }, "message": "Fix GetOutVROffset to use correct Method* size.\n\n[Tip \u0027o the hat to Mark Mendell for noticing this one]\n\nUse StackReference to find Method* size in GetOutVROffset.\n\nChange-Id: Ie1b25eb839c59305cadc5fdc7cbb5f41f1a85686\n" }, { "commit": "511c8a653d5896e81428393a1c3d427da64e36f3", "tree": "2d286df3b09a689a313bccd8925eb29fc2d1c4eb", "parents": [ "57795db7d44bcd6d106481fa192691400b2358c8" ], "author": { "name": "Zheng Xu", "email": "zheng.xu@arm.com", "time": "Tue Jun 03 16:22:23 2014 +0800" }, "committer": { "name": "Zheng Xu", "email": "zheng.xu@arm.com", "time": "Wed Jun 04 16:15:10 2014 +0800" }, "message": "AArch64: Fix cmp-long and method with long arguments.\n\n1. Fix cmp-long.\n2. Use single register to pass long argument.\n3. Flush StackReference\u003cArtMethod\u003e on arm64 the same as in common code.\n3. Fix the mismatch in calculate reg offset.\n\nChange-Id: Ie2723260fb143512e4da6ee88d4f3aded80d3d5e\n" }, { "commit": "5cf98196d488437acd1e989c08a554ef697fded1", "tree": "dd44bc0120562169b701e80dbec413a179862beb", "parents": [ "b7f02280f7f56ae94fe7f01e161be0b725b6e4a9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 29 21:31:50 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri May 30 12:46:10 2014 -0700" }, "message": "Don\u0027t report down-calls as unhandled exceptions.\n\nBug: 15310540\nAlso, narrow scope of catch/deoptimize stack visitors that are specific to\nquick exception delivery.\n\nChange-Id: Ib13a006ce1347acb93a36b0186550d4c3ec2034b\n" }, { "commit": "cf4035a4c41ccfcc3e89a0cee25f5218a11b0705", "tree": "323d9e98e6129c67e464a3e6857ee02593a2f2c2", "parents": [ "29b53d3d715b1ec19349e8cbf7c5e4ff529bd5fe" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 28 22:43:01 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu May 29 20:50:49 2014 -0700" }, "message": "ART: Use StackReference in Quick Stack Frame\n\nThe method reference at the bottom of a quick frame is a stack\nreference and not a native pointer. This is important for 64b\narchitectures, where the notions do not coincide.\n\nChange key methods to have StackReference\u003cmirror::ArtMethod\u003e*\nparameter instead of mirror::ArtMethod**. Make changes to\ninvoke stubs for 64b archs, change the frame setup for JNI code\n(both generic JNI and compilers), tie up loose ends.\n\nTested on x86 and x86-64 with host tests. On x86-64, tests succeed\nwith jni compiler activated. x86-64 QCG was not tested.\n\nTested on ARM32 with device tests.\n\nFix ARM64 not saving x19 (used for wSUSPEND) on upcalls.\n\nTested on ARM64 in interpreter-only + generic-jni mode.\n\nFix ARM64 JNI Compiler to work with the CL.\n\nTested on ARM64 in interpreter-only + jni compiler.\n\nChange-Id: I77931a0cbadd04d163b3eb8d6f6a6f8740578f13\n" }, { "commit": "92d1a666534aa98b173bb33dc5dba86b2d48aedb", "tree": "864887bc5e1220152998874d3be861b31a49b0fa", "parents": [ "3b2bcbf9daf39f4cece7fde1186f3fa494000ed9" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu May 15 21:43:59 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri May 16 13:37:45 2014 -0700" }, "message": "Visit methods in stack frames during root visits.\n\nThis is necessary for a Baker-style read barrier to-space invariant to\nhold. That is, an object either needs to be marked/forwarded as root\nor it must be accessed only through a read barrier by mutators. Since\nstack frames have direct pointers to methods that a mutator can access\nwithout a read barrier, stack frame methods have to be visited as\nroot, which makes sense as stack frames are thread roots. This is the\ncase even if methods do not move as they have to be marked \u0027gray\u0027 for\nthe objects pointed to by them, which can move, to be recursively\nmarked/forwarded.\n\nThis also puts us in the right direction toward moving methods (and\nfields) in the future.\n\nBug: 12687968\nChange-Id: Id32b913c021a140073deea9149a8782e8f308303\n" }, { "commit": "eb8167a4f4d27fce0530f6724ab8032610cd146b", "tree": "bcfeaf13ad78f2dd68466bbd0e20c71944f7e854", "parents": [ "6fb66a2bc4e1c0b7931101153e58714991237af7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 07 15:43:14 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue May 13 14:45:54 2014 -0700" }, "message": "Add Handle/HandleScope and delete SirtRef.\n\nDelete SirtRef and replaced it with Handle. Handles are value types\nwhich wrap around StackReference*.\n\nRenamed StackIndirectReferenceTable to HandleScope.\n\nAdded a scoped handle wrapper which wraps around an Object** and\nrestores it in its destructor.\n\nRenamed Handle::get -\u003e Get.\n\nBug: 8473721\n\nChange-Id: Idbfebd4f35af629f0f43931b7c5184b334822c7a\n" }, { "commit": "5cd33753b96d92c03e3cb10cb802e68fb6ef2f21", "tree": "b2217cb48e1f1db1aae7a92f8d5dbac5f2d95603", "parents": [ "8ea5baa10b04e9dbd0f0cf14b0358fe8b956bb2e" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Tue Apr 15 15:57:58 2014 -0700" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Mon May 05 12:44:15 2014 -0700" }, "message": "Handle implicit stack overflow without affecting stack walks\n\nThis changes the way in which implicit stack overflows are handled\nto satisfy concerns about changes to the stack walk code.\n\nInstead of creating a gap in the stack and checking for it in\nthe stack walker, use the ManagedStack infrastructure to concoct\nan invisible gap that will never be seen by a stack walk.\n\nAlso, this uses madvise to tell the kernel that the main stack\u0027s\nprotected region will probably never be accessed, and instead\nof using memset to map the pages in, use memcpy to read from\nthem. This will save 32K on the main stack.\n\nAlso adds a \u0027signals\u0027 verbosity level as per a review request.\n\nBug: 14066862\nChange-Id: I5257305feeaea241d11e6aa6f021d2a81da20b81\n" }, { "commit": "8194963098247be6bca9cc4a54dbfa65c73e8ccc", "tree": "547cc708e06e6541676b17066023ae6f07b2049b", "parents": [ "56a341a82ece9aa4f2a071629f3e1fd1adf988ae" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 02 11:53:22 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 02 12:21:02 2014 +0100" }, "message": "Replace CountOneBits and __builtin_popcount with POPCOUNT.\n\nClean up utils.h, make some functions constexpr.\n\nChange-Id: I2399100280cbce81c3c4f5765f0680c1ddcb5883\n" }, { "commit": "42fcd9838a87abaf7a2ef86853a5287f86dbe391", "tree": "d8ffcfcb9c1997c06c6ef93b551abaa900ebcf8e", "parents": [ "608168b380b741e2c7e1a2b0b568c0738986166b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Apr 22 11:03:52 2014 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri May 02 10:44:52 2014 +0100" }, "message": "Revert \"Revert \"64bit changes to the stack walker for the Quick ABI.\"\"\n\nThis reverts commit 8d5ead52a92675c258113d3dfa71bf8fceba5d9f.\n\nChange-Id: I6b4774a9bd180de33551975e106322ba1192e6ab\n" }, { "commit": "8d5ead52a92675c258113d3dfa71bf8fceba5d9f", "tree": "43f4c2518035b346d7b7ecd46c4b79d1ef45d882", "parents": [ "5cb328362a633302ca0fcdbaa0da7d94069df051" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Apr 22 08:30:19 2014 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Apr 22 08:30:19 2014 +0000" }, "message": "Revert \"64bit changes to the stack walker for the Quick ABI.\"\n\nThis reverts commit 5cb328362a633302ca0fcdbaa0da7d94069df051.\n\nChange-Id: Icc0ec1a9f15c2e4e9103e732d7ba75a4feb853b1\n" }, { "commit": "5cb328362a633302ca0fcdbaa0da7d94069df051", "tree": "bf31cf8e9aaab4e7d326cef1d2b8936bca658473", "parents": [ "957e2a2bb9ca9a59b4fb41170b1b1a53e99bed67" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Apr 17 14:05:19 2014 +0100" }, "committer": { "name": "Dmitry Petrochenko", "email": "dmitry.petrochenko@intel.com", "time": "Fri Apr 18 15:23:34 2014 +0700" }, "message": "64bit changes to the stack walker for the Quick ABI.\n\n- Spill registers have different sizes.\n- The ArtMethod at the bottom of the stack is always of kWordSize.\n\nChange-Id: I92f67ff928477970c393c7146980255d08e8e6af\n" }, { "commit": "f943914730db8ad2ff03d49a2cacd31885d08fd7", "tree": "885a781e5f8bd852e2c1615108ae7b17576a6567", "parents": [ "cfd5acf281b0c509f86b13d73c6a8dfa3ea9922c" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Thu Mar 27 15:10:22 2014 -0700" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Mon Mar 31 18:04:08 2014 -0700" }, "message": "Implement implicit stack overflow checks\n\nThis also fixes some failing run tests due to missing\nnull pointer markers.\n\nThe implementation of the implicit stack overflow checks introduces\nthe ability to have a gap in the stack that is skipped during\nstack walk backs. This gap is protected against read/write and\nis used to trigger a SIGSEGV at function entry if the stack\nwill overflow.\n\nChange-Id: I0c3e214c8b87dc250cf886472c6d327b5d58653e\n" }, { "commit": "b373e091eac39b1a79c11f2dcbd610af01e9e8a9", "tree": "034d820c4829e0dcf6161473cc39f7250123bfaa", "parents": [ "9545a446e99b22248099fe66f5f9431530c20851" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Thu Feb 20 16:06:36 2014 -0800" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Thu Mar 13 12:21:15 2014 -0700" }, "message": "Implicit null/suspend checks (oat version bump)\n\nThis adds the ability to use SEGV signals\nto throw NullPointerException exceptions from Java code rather\nthan having the compiler generate explicit comparisons and\nbranches. It does this by using sigaction to trap SIGSEGV and when triggered\nmakes sure it\u0027s in compiled code and if so, sets the return\naddress to the entry point to throw the exception.\n\nIt also uses this signal mechanism to determine whether to check\nfor thread suspension. Instead of the compiler generating calls\nto a function to check for threads being suspended, the compiler\nwill now load indirect via an address in the TLS area. To trigger\na suspend, the contents of this address are changed from something\nvalid to 0. A SIGSEGV will occur and the handler will check\nfor a valid instruction pattern before invoking the thread\nsuspension check code.\n\nIf a user program taps SIGSEGV it will prevent our signal handler\nworking. This will cause a failure in the runtime.\n\nThere are two signal handlers at present. You can control them\nindividually using the flags -implicit-checks: on the runtime\ncommand line. This takes a string parameter, a comma\nseparated set of strings. Each can be one of:\n\nnone switch off\nnull null pointer checks\nsuspend suspend checks\nall all checks\n\nSo to switch only suspend checks on, pass:\n-implicit-checks:suspend\n\nThere is also -explicit-checks to provide the reverse once\nwe change the default.\n\nFor dalvikvm, pass --runtime-arg -implicit-checks:foo,bar\n\nThe default is -implicit-checks:none\n\nThere is also a property \u0027dalvik.vm.implicit_checks\u0027 whose value is the same\nstring as the command option. The default is \u0027none\u0027. For example to switch on\nnull checks using the option:\n\nsetprop dalvik.vm.implicit_checks null\n\nIt only works for ARM right now.\n\nBumps OAT version number due to change to Thread offsets.\n\nBug: 13121132\nChange-Id: If743849138162f3c7c44a523247e413785677370\n" }, { "commit": "bf6b92a158053c98b15f4393abb3b86344ec9a20", "tree": "93a28d1e01b045a8df23fe5279601496b62af3dd", "parents": [ "359597ee8c0667a7d2c0b74dcbb0f90bb6059bae" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 05 16:11:04 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Mar 05 22:32:53 2014 -0800" }, "message": "Generic JNI implementation for x86_64\n\nStarting implementation for generic JNI on x86_64. Frames are of\nlarge static size (\u003e4K) right now, should be compacted later. Passes\nthe whole of jni_compiler_test.\n\nChange-Id: I88ac3e13a534afe7568d62a1ef97cb766e8260e4\n" }, { "commit": "4e30541a92381fb280cd0be9a1763b713ee4d64c", "tree": "84093651bbf0ad95b66b846c4f4cf4101994037b", "parents": [ "e266ba9935bd12d685d83f73cd8d759e46c3014d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 19 10:54:44 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Feb 21 15:24:04 2014 -0800" }, "message": "Fix and optimize verify object.\n\nVerifyObject no longer resides in heap. You can now enable\nVerifyObject for non-debug builds. VerifyStack is still slow, so it\nis now guarded by its own flag.\n\nFixed the image writer to not use verification at places where\nverification fails due to invalid reads.\n\nFixed RosAlloc to use SizeOf which doesn\u0027t call verify object.\n\nAdded a flag paremeter to some of the mirror getters / setters to\nbe able to selectively disable VerifyObject on certain calls.\n\nOptimized the GC to not verify each object multiple times during\nobject scanning if verify object is enabled.\n\nAdded 3 verification options: verify reads, verify this, and verify\nwrites so that you can select how much verification you want for\nmirror getters and setters.\n\nRemoved some useless DCHECKs which would slow debug builds without\nproviding any benefits.\n\nTODO: RosAlloc verification doesn\u0027t currently work with verify\nobjects.\n\nBug: 12934910\nBug: 12879358\n\nChange-Id: Ic61033104dfc334543f89b0fc0ad8cd4f4015d69\n" }, { "commit": "815873ecc312b1d231acce71e1a16f42cdaf09f2", "tree": "18ba2fa951775e60b240271bfe975e6e2cfc654c", "parents": [ "2befd09cf4fe89a18a655f3e1dd310831bfa769f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Feb 13 18:02:13 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 18 10:45:12 2014 -0800" }, "message": "Change root visitor to use Object**.\n\nSimplifies code and improves the performance of root visiting since\nwe usually don\u0027t need to check to see if the object moved.\n\nChange-Id: Iba998f5a15ae1fa1b53ca5226dd2168a411196cf\n" }, { "commit": "3bc01748ef1c3e43361bdf520947a9d656658bf8", "tree": "9ac4f9cfd8079f2e89c85986bb60205a6f3e7579", "parents": [ "68bb649b128cd8760732524bd7ba58b49780d9d3" ], "author": { "name": "Razvan A Lupusoru", "email": "razvan.a.lupusoru@intel.com", "time": "Thu Feb 06 13:18:43 2014 -0800" }, "committer": { "name": "Razvan A Lupusoru", "email": "razvan.a.lupusoru@intel.com", "time": "Thu Feb 13 14:00:16 2014 -0800" }, "message": "GenSpecialCase support for x86\n\nMoved GenSpecialCase from being ARM specific to common code to allow\nit to be used by x86 quick as well.\n\nChange-Id: I728733e8f4c4da99af6091ef77e5c76ae0fee850\nSigned-off-by: Razvan A Lupusoru \u003crazvan.a.lupusoru@intel.com\u003e\n" }, { "commit": "ef7d42fca18c16fbaf103822ad16f23246e2905d", "tree": "c67eea52a349c2ea7f2c3bdda8e73933c05531a8", "parents": [ "822115a225185d2896607eb08d70ce5c7099adef" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Jan 06 12:55:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Feb 06 23:20:27 2014 -0800" }, "message": "Object model changes to support 64bit.\n\nModify mirror objects so that references between them use an ObjectReference\nvalue type rather than an Object* so that functionality to compress larger\nreferences can be captured in the ObjectRefererence implementation.\nObjectReferences are 32bit and all other aspects of object layout remain as\nthey are currently.\n\nExpand fields in objects holding pointers so they can hold 64bit pointers. Its\nexpected the size of these will come down by improving where we hold compiler\nmeta-data.\nStub out x86_64 architecture specific runtime implementation.\nModify OutputStream so that reads and writes are of unsigned quantities.\nMake the use of portable or quick code more explicit.\nTemplatize AtomicInteger to support more than just int32_t as a type.\nAdd missing, and fix issues relating to, missing annotalysis information on the\nmutator lock.\nRefactor and share implementations for array copy between System and uses\nelsewhere in the runtime.\nFix numerous 64bit build issues.\n\nChange-Id: I1a5694c251a42c9eff71084dfdd4b51fff716822\n" }, { "commit": "da7a69b3fa7bb22d087567364b7eb5a75824efd8", "tree": "17aea3b34d6059b52fab73fc206470eca5e9d305", "parents": [ "353e494a7108f382daf1782596fc0a93d92f38a4" ], "author": { "name": "Razvan A Lupusoru", "email": "razvan.a.lupusoru@intel.com", "time": "Wed Jan 08 15:09:50 2014 -0800" }, "committer": { "name": "Razvan A Lupusoru", "email": "razvan.a.lupusoru@intel.com", "time": "Fri Jan 31 13:58:28 2014 -0800" }, "message": "Enable compiler temporaries\n\nCompiler temporaries are a facility for having virtual register sized space\nfor dealing with intermediate values during MIR transformations. They receive\nexplicit space in managed frames so they can have a home location in case they\nneed to be spilled. The facility also supports \"special\" temporaries which\nhave specific semantic purpose and their location in frame must be tracked.\n\nThe compiler temporaries are treated in the same way as virtual registers\nso that the MIR level transformations do not need to have special logic. However,\ngenerated code needs to know stack layout so that it can distinguish between\nhome locations.\n\nMIRGraph has received an interface for dealing with compiler temporaries. This\ninterface allows allocation of wide and non-wide virtual register temporaries.\n\nThe information about how temporaries are kept on stack has been moved to\nstack.h. This is was necessary because stack layout is dependent on where the\ntemporaries are placed.\n\nChange-Id: Iba5cf095b32feb00d3f648db112a00209c8e5f55\nSigned-off-by: Razvan A Lupusoru \u003crazvan.a.lupusoru@intel.com\u003e\n" }, { "commit": "9119c5f9e4f447f4209d51cff66d1aace510ce5e", "tree": "d6f5c783b8a43a849bcc0c4cfeaffd3f1b062bcd", "parents": [ "d5c0587bc26a820646950b634275ba10ab06c995" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Dec 16 11:31:45 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Dec 17 15:13:03 2013 +0100" }, "message": "Cleanup invoke\u0027s receiver handling in interpreter.\n\nTo comply with a moving collector, we used to load invoke\u0027s receiver (for non\nstatic methods only) from the caller shadow frame after resolving the method\nitself (in case the garbage collector is triggered inside) but before passing\ninvoke arguments, including loading receiver from the caller\u0027s shadow frame\ninto the callee\u0027s shadow frame. Therefore, we used to load the receiver 3 times\nin the fast path but only twice in the slow path.\n\nThe slow path is rarely used (only in method requiring extra runtime checks) so\nwe now move this extra reload to the slow path. Therefore an invoke using the\nfast path loads the receiver twice while the slow path loads it 3 times.\n\nI don\u0027t expect much improvement here. The main reason is to keep extra code in\nthe slow path.\n\nChange-Id: I10e96b10de4b8c2992e276bd564bc3e2f191779c\n" }, { "commit": "590fee9e8972f872301c2d16a575d579ee564bee", "tree": "b02db45c72f1911ec896b93379ada0276aea3199", "parents": [ "5b70680b8df6d8fa95bb8e1070d0107f3d388940" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 13 13:46:47 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 11 15:34:27 2013 -0800" }, "message": "Compacting collector.\n\nThe compacting collector is currently similar to semispace. It works by\ncopying objects back and forth between two bump pointer spaces. There\nare types of objects which are \"non-movable\" due to current runtime\nlimitations. These are Classes, Methods, and Fields.\n\nBump pointer spaces are a new type of continuous alloc space which have\nno lock in the allocation code path. When you allocate from these it uses\natomic operations to increase an index. Traversing the objects in the bump\npointer space relies on Object::SizeOf matching the allocated size exactly.\n\nRuntime changes:\nJNI::GetArrayElements returns copies objects if you attempt to get the\nbacking data of a movable array. For GetArrayElementsCritical, we return\ndirect backing storage for any types of arrays, but temporarily disable\nthe GC until the critical region is completed.\n\nAdded a new runtime call called VisitObjects, this is used in place of\nthe old pattern which was flushing the allocation stack and walking\nthe bitmaps.\n\nChanged image writer to be compaction safe and use object monitor word\nfor forwarding addresses.\n\nAdded a bunch of added SIRTs to ClassLinker, MethodLinker, etc..\n\nTODO: Enable switching allocators, compacting on background, etc..\n\nBug: 8981901\n\nChange-Id: I3c886fd322a6eef2b99388d19a765042ec26ab99\n" }, { "commit": "74e256b8e442417d4ba2054c771c1e4f41062768", "tree": "8060d58f101c7d0ea7493728c658aad7bfa535d3", "parents": [ "9f69b62e6b009b29e6420c49e7444e91466a6a33" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Oct 04 10:40:37 2013 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Oct 09 19:48:54 2013 +0200" }, "message": "Add missing references.\n\nThis mainly avoids implicit copies when extracting an object from a container.\n\nChange-Id: If4d0e6153e8c2b48345fde5bb546b4c65649fcf3\n" }, { "commit": "c61124bdeaae94f977ffc36ac69535e792c226f2", "tree": "41ba8619a6dea88a8c607836db21531b05a3787a", "parents": [ "7541c745e00b49983f277d6b6b18bc4c174c7e39" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 10 11:44:19 2013 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Sep 30 08:57:33 2013 +0200" }, "message": "Cleanup invoke in interpreter.\n\nSome cleanup in invocation stuff:\n- Get the number of invoke arguments from instruction (vA) rather than get it\nfrom its code item. This benefits to native invoke since we no longer need to\nparse the method\u0027s shorty. Also pass the low 16 bits of instructions to avoid\nfetching it twice when reading vA.\n- Remove \"is_static\" tests by taking advantage of invoke type template argument\nrather than testing method\u0027s access flags.\n- Ensure Instruction::GetArgs is inlined.\n- Check exception when initializing method\u0027s class when transitioning from\ninterpreter to compiled code (artInterpreterToCompiledCodeBridge).\n- Move UnstartedRuntimeInvoke function to interpreter_common.cc and make it\nstatic as it\u0027s only used by DoInvoke and DoInvokeVirtualQuick functions.\n- Avoid duplicating code in ShadowFrame::Create.\n\nPerformance remains the same according to benchmarks. Hopefully, this should be\naddressed in next CLs, especially by improving new shadow frame initialization.\n\nBug: 10668955\nChange-Id: I514b8f098d0ef3e35921ceb770383aac1a9c7902\n" }, { "commit": "7c748a2f0c04a088471eff7cd311c024cbbdc04e", "tree": "6fa0e7cabd5889de547e3a4977ca9eb6d52a1f19", "parents": [ "5712d5d04640925970db9c98938ffaf806b3962c", "e47637cc7c96e654d2c340e6006c232c8078673b" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Sep 19 16:08:57 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Thu Sep 19 16:08:57 2013 -0700" }, "message": "am e47637cc: 4-byte align 64-bit values in Get/Set Double/Long for GCC 4.8.\n\n* commit \u0027e47637cc7c96e654d2c340e6006c232c8078673b\u0027:\n 4-byte align 64-bit values in Get/Set Double/Long for GCC 4.8.\n" }, { "commit": "e47637cc7c96e654d2c340e6006c232c8078673b", "tree": "4e49c4d7bccbb7903b6de186b8676a77d7c157bc", "parents": [ "f466553d7f2f941765adaa657c340ff31b55d891" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Sep 19 15:13:16 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Sep 19 15:37:02 2013 -0700" }, "message": "4-byte align 64-bit values in Get/Set Double/Long for GCC 4.8.\n\nBug: 10837416\nChange-Id: Ibb562407d81c2085666ae8824e7570f22e56eaa7\n" }, { "commit": "423d2a3dcbb260b020efb5da59f784c9f02accbf", "tree": "79ed739e6072f8308c1cd880f9420a1c63539c95", "parents": [ "b048dd2b662c19644361f4396a1e8d6213445ee8" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 12 17:33:56 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 13 10:30:24 2013 -0700" }, "message": "Add support for changing roots through the root visitor callback.\n\nNeeded for copying collectors.\n\nChange-Id: Icc4a342a57e0cfb79587edb02ef8c85e08808877\n" }, { "commit": "ea46f950e7a51585db293cd7f047de190a482414", "tree": "9dddc8073547a2dcb58a19e1728932a89cb149c3", "parents": [ "5e3572709b5a5d59957f835db4f73760ecef08da" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Jul 30 01:26:50 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 13 18:09:46 2013 -0700" }, "message": "Refactor java.lang.reflect implementation\n\nCherry-picked from commit ed41d5c44299ec5d44b8514f6e17f802f48094d1.\n\nMove to ArtMethod/Field instead of AbstractMethod/Field and have\njava.lang.reflect APIs delegate to ArtMethod/ArtField.\n\nBug: 10014286.\n\nChange-Id: Iafc1d8c5b62562c9af8fb9fd8c5e1d61270536e7\n" }, { "commit": "1809a72a66d245ae598582d658b93a24ac3bf01e", "tree": "f864ff869185b2873637ca70584c66fd6a1e899d", "parents": [ "a9a50926963b5093fb851ed966d201f3e95f72d3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 09 22:05:32 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 09 22:23:50 2013 -0700" }, "message": "Uleb128 compression of vmap and mapping table.\n\nBug 9437697.\n\nChange-Id: I30bcb97d12cd8b46d3b2cdcbdd358f08fbb9947a\n" }, { "commit": "166db04e259ca51838c311891598664deeed85ad", "tree": "5ffa63c1b3bf58e92be2890c54f0ba6fdec2059e", "parents": [ "d41119e863a9f9c5809b7732d30870ffd237b60c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jul 26 12:05:57 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Jul 29 09:52:11 2013 -0700" }, "message": "Move assembler out of runtime into compiler/utils.\n\nOther directory layout bits of clean up. There is still work to separate quick\nand portable in some files (e.g. argument visitor, proxy..).\n\nChange-Id: If8fecffda8ba5c4c47a035f0c622c538c6b58351\n" }, { "commit": "7934ac288acfb2552bb0b06ec1f61e5820d924a4", "tree": "43f3acd8af7fd34d4ae7b64f6e06bb8429d74bb8", "parents": [ "fb331d7ca004f39608fcfdae49d38df90c702ea9" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 26 10:54:15 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 26 11:55:10 2013 -0700" }, "message": "Fix cpplint whitespace/comments issues\n\nChange-Id: Iae286862c85fb8fd8901eae1204cd6d271d69496\n" }, { "commit": "0cd7ec2dcd8d7ba30bf3ca420b40dac52849876c", "tree": "525a0f99f9381156367c988133b5d51d5dfef6f7", "parents": [ "f69863b3039fc621ff4250e262d2a024d5e79ec8" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 23:40:20 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 00:12:43 2013 -0700" }, "message": "Fix cpplint whitespace/blank_line issues\n\nChange-Id: Ice937e95e23dd622c17054551d4ae4cebd0ef8a2\n" }, { "commit": "fc0e3219edc9a5bf81b166e82fd5db2796eb6a0d", "tree": "5fc7f5b941724a62f8e3411df09fae431ff5e3cf", "parents": [ "56d947fbc9bc2992e2f93112fafb73e50d2aaa7a" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 14:40:12 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 16:49:05 2013 -0700" }, "message": "Fix multiple inclusion guards to match new pathnames\n\nChange-Id: Id7735be1d75bc315733b1773fba45c1deb8ace43\n" }, { "commit": "7940e44f4517de5e2634a7e07d58d0fb26160513", "tree": "ac90242d96229a6942f6e24ab137bc1f8f2e0025", "parents": [ "5cd9e3b122f276f610980cbaf0d2ad6ed4cd9088" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 13:46:57 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 17:49:01 2013 -0700" }, "message": "Create separate Android.mk for main build targets\n\nThe runtime, compiler, dex2oat, and oatdump now are in seperate trees\nto prevent dependency creep. They can now be individually built\nwithout rebuilding the rest of the art projects. dalvikvm and jdwpspy\nwere already this way. Builds in the art directory should behave as\nbefore, building everything including tests.\n\nChange-Id: Ic6b1151e5ed0f823c3dd301afd2b13eb2d8feb81\n" }, { "commit": "e701f48eb06fa59871412a11286429111270b211", "tree": "8026f7b25596642e142b67768047484a5307c149", "parents": [ "5243e912875026f99428088db7e80cb11651d64e" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri May 24 11:50:49 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Jun 03 14:26:02 2013 -0700" }, "message": "Clean up shadow frame.\n\nCreated version of GetThisObject that takes num_ins to get the object\nwithout having to get the code item again. Also using memset to clear\nthe registers on initialization.\n\nChange-Id: I02b255f0c47d2d3c55783f4c96dc93983015f019\n" }, { "commit": "6613519990bdb5616f2858c146f162ef760f03d2", "tree": "9ea428594c2062b9e97a7735f0cdadc1e8e85d5a", "parents": [ "807a25640d4f4de8143b160b3bb8f552ffbf6f4a" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue May 14 11:02:41 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue May 14 11:37:40 2013 -0700" }, "message": "Allocate shadow frames in interpreter on stack instead of on heap.\n\nInstead of using new, the shadow frames are allocated using alloca.\n\nChange-Id: Idc92fbccf1fe7589ace7b97811b21a5c67c97fd2\n" }, { "commit": "807a25640d4f4de8143b160b3bb8f552ffbf6f4a", "tree": "3972b74f95282a292f9fa8bfca82bd8bbe736cfc", "parents": [ "cf479b20dc78d08af75fac7192a92f7c4b259d54" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Apr 15 09:33:39 2013 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue May 14 17:48:52 2013 +0200" }, "message": "Improve interpreter performance.\n\nBug: 8196227\n\nThis CL removes the use of DecodedInstruction to read instruction opcode and\noperands. It now directly access to each operand according to instruction\nformat (like VRegA_11x for accessing register vA with format 11x).\n\nIt also caches some information used for instrumentation like \u0027this\u0027 object\nand current method and add missing check about pending exception.\n\nChange-Id: I8c03c0aea9d75068b89e3cb2c8c12383d7928281\n" }, { "commit": "167436311a08a65dea28dda079a137893821c9c7", "tree": "4754f47306a0ffa0ad97635015d9596b427dd425", "parents": [ "c44f68fb6a6d55c0df28dd05b93d8906ec20bd08" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 08 10:59:04 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri May 10 17:16:00 2013 -0700" }, "message": "Improve interpreter to interpreter invokes.\n\nThe interpreter constructs a shadow frame instead of arg array to make\ninterpreter to interpreter transitions faster. This adds a pointer to\nan entry for the interpreter to each method.\n\nChange-Id: If48911d3aa3470847b8548a9e92090b829f4f254\n" }, { "commit": "8a01a3a8caee37d4c4cf1a8c673f897c74aaf785", "tree": "2abfaf7d42e5aaabdc1faaeeda01f5a904fd64ed", "parents": [ "bf47e5f28b1aa39748dce8ac5abbabca1baee093" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 06 13:25:44 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 06 16:04:04 2013 -0700" }, "message": "Make ShadowFrame fields const when not in portable.\n\nShadowFrames in quick don\u0027t need the HasReferenceArray test as they always have\na reference array. Make the method_ and number_of_vregs_ fields const to aid\nG++\u0027s optimization of the interpreter.\n\nModify the OFFSETOF_MEMBER macro to handle const fields.\n\nChange-Id: I696480789190f7c5190449b9c278d37853903a5e\n" }, { "commit": "62d6c772205b8859f0ebf7ad105402ec4c3e2e01", "tree": "e2f2ba6d71ed5a39c9f6909e3f7c08e998053315", "parents": [ "c9b17c7ee96cd04fac9048aab624ed554fe260bf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 08:32:07 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Apr 08 14:24:13 2013 -0700" }, "message": "Interpreter entries and instrumentation as a listener.\n\nMake the instrumentation responsible for whether we want method entry/exit\nstubs, and allow it to use interpreter entry stubs when instruction by\ninstruction instrumentation is required. Improve deoptimization so more JDWP\ntest cases are passing.\n\nRefactor exception debug posting, in particular improve reporting in the\ninterpreter. Improve class linker exception throwing so that broken dex files\nare more likely to be reported. Fixes the performance issue Bug: 8410519.\n\nFix some error reporting lock level errors for the large object space. Make\nfast object verification faster.\n\nAdd some debug mode robustness to finding dex PCs in GC maps.\n\nAdd printf attributes to JniAbortF and fix errors.\n\nExpand run-test 044 to test return behaviors and fix issues with not throwing\nappropriate exceptions for proxies.\n\nEnsure causes are reported with a class linker NoClassDefFoundError and JNI\nNoSuchFieldError.\n\nRemove unused debugMe and updateDebuggerFromCode.\n\nThere\u0027s a minor sizing tweak to the arg array builder, and an extra reference\narray check in the interpreter.\n\nSome clean-up of trace code.\n\nFix reg type cache destructor if it is called after the reg type cache is\nshutdown (as is the case in oatdump).\n\nChange-Id: I6519c7b35df77f978d011999354c864f4918e8ce\n" }, { "commit": "a7b0c42fee3e01e657634be2bba9318eeaec591a", "tree": "749397cd4c2aec7fe37370d0e51b942fa51e98a6", "parents": [ "4c6aad2bbe24e5a1a95fc4d08a4b6bb4d1609669" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Apr 05 16:19:39 2013 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Apr 05 16:21:15 2013 +0200" }, "message": "Fix crash in stack walking.\n\nBug: 8537133\n\nThe G+ app throws an exception which leads to a crash when walking the stack.\nThe cause is the compiled code tries to set the top of quick stack while the\ncurrent managed stack is shadow frame based.\n\nThis CL adds the missing \"pop\" of the shadow frame and also some checks enabled\nin debug only.\n\nChange-Id: Iac04993b287abadee7a3c240e2e9cf1a07e50ce5\n" }, { "commit": "11ffc2d02b7d979f26a8a04013fce3eb4f9a2816", "tree": "a54cdfe70f2bd45ea36bfd6cf9e7ad8783a29234", "parents": [ "e9de2dd6bd51f9be3457b5e2ffdd97f457ddebcb" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Feb 01 11:52:17 2013 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Feb 01 14:45:01 2013 -0800" }, "message": "Changes to LLVM to support deoptimization.\n\nAdded a magic exception value (-1) and a handler to transition\nto the interpreter. This is currently untested.\n\nChange-Id: I2f53135e7505c54355ecf7c579897f68bbdcbda3\n" }, { "commit": "2dd0e2cea360bc9206eb88ecc40d259e796c239d", "tree": "9d619dc9508cbe73e4793bf6f08cbc761abfb48a", "parents": [ "ac21b797b3a425975d656d6b84a7b24401d35f42" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jan 24 12:42:14 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 30 16:48:31 2013 -0800" }, "message": "Directory restructuring of object.h\n\nBreak object.h into constituent files.\nReduce number of #includes in other GC header files.\nIntroduce -inl.h files to avoid mirror files #include-ing each other.\nCheck invariants of verifier RegTypes for all constructors.\n\nChange-Id: Iecf1171c02910ac152d52947330ef456df4043bc\n" }, { "commit": "7a22fa657b972e8323692368975bc5a7be1cc0f5", "tree": "4fe55837daae4e252eeaccf1be931f2e7ed434cb", "parents": [ "87b185f3bfd766001dcd5977fc0ee0489e5a8a28" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 23 12:16:16 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 23 12:21:05 2013 -0800" }, "message": "Refactor StackVisitor to take a Thread*.\n\nThis allows assertion checking on the thread, principally that we never try to\nwalk the stack of an unsuspended thread.\nFix bug in the OwnedMonitorVisitor where GetVReg could be called on a\nStackVisitor with no context.\n\nChange-Id: I06539b624b253b6fb7385e7be11a4bced1d417b2\n" }, { "commit": "306057fd278d75bf3794bd5243a3b6652c487d18", "tree": "62652a97caace11e50408d15688ced3514039219", "parents": [ "2d76b041be770102fc912effc398e629a18180d2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Nov 26 12:45:53 2012 -0800" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Wed Jan 02 11:15:42 2013 -0800" }, "message": "Support for deoptimization needed for debugging.\n\nThe deoptimization code is untested, and some sanity checks in the\ninstrumentation are disabled because they need debugging.\n\nChange-Id: I1b60a65a60bddc9b107ad4659da097b55ce901c3\n" }, { "commit": "761600567d73b23324ae0251e871c15d6849ffd8", "tree": "4757cb01233394fa2e9c461a68fc37a35c12dabb", "parents": [ "89c41e5f2fa10e7b97698b9714bd4883a73132f0" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Dec 12 16:31:20 2012 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Dec 12 16:31:20 2012 -0800" }, "message": "Switch over to the google3 unix_file File*.\n\nI also moved macros.h to base/macros.h to ease google3 porting, at\nthe expense of a larger than necessary change. (I learned my lesson,\nthough, and didn\u0027t make the equivalent base/logging.h change.)\n\nI\u0027m not sure whether we want to keep the unix_file MappedFile given\nour existing MemMap, but it\u0027s easier to bring it over and then remove\nit (and possibly revert the removal) than to bring it over later.\n\nChange-Id: Id50a66faa5ab17b9bc936cc9043dbc26f791f0ca\n" }, { "commit": "6702243ea2332b566d8e8b871cc9db0906d835ad", "tree": "dc530339fa07de6028c0771edb082f58ea3686f9", "parents": [ "a169e1d8d968800380245cda450505d969ebff76" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Nov 29 18:04:50 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 11 11:35:41 2012 -0800" }, "message": "Support for SetVReg and add SetGPR.\n\nThese changes are useful for debugging and are required for being able\nto change all GC roots.\n\nMIPS is untested.\n\nChange-Id: I2ba055de64264098bffe869a4fb192d0975f1c8f\n" }, { "commit": "ce4cc0d1818e872c1c7f3c3519a82259afd5c288", "tree": "308f2c30e5a842a5e459d96e730ca4432a020f4f", "parents": [ "823b8acdf2c8274ff93b685e50a93ab9afc6af8b" ], "author": { "name": "TDYa127", "email": "tdy@google.com", "time": "Sun Nov 18 16:59:53 2012 -0800" }, "committer": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Tue Dec 04 11:07:34 2012 -0800" }, "message": "Using gcmap instead of shadow frame.\n\nFix misuse of TBAAJRuntime \u0026 TBAARuntimeInfo. Now, the TBAAJRuntime is\nonly for runtime support function.\n\nUpdate DexPC before lock object and suspend.\n\nChange-Id: I40fa37f4863fe6e127328a8413285ee3c62e8505\n" }, { "commit": "df1ce91ba97bc79a0637e5504b39318fb1c9f577", "tree": "06fee7c60611e94264b3b6b2b318fec899c6e450", "parents": [ "af1b89281fd1f86efeb791b61f5a3f2031c70312" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 27 17:07:11 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 27 17:19:09 2012 -0800" }, "message": "Add alignment attribute to packed.\n\nMake packed objects 4 byte aligned or else GCC becomes paranoid and\nassumes only byte alignment on whole objects.\nChange CHECK in GetCalleeSaveMethod to DCHECK to encourage inlining.\n\nChange-Id: I79f1a13752492413c919b36f4e207e637647834f\n" }, { "commit": "40e3bacfd57bca2ca39c1caec64680bd0ed4a16d", "tree": "808eb0b5138cc1690149dc26f6937e0b14924a6c", "parents": [ "11d4ab2dd052b74923d4a3c6afc3ef4b15f064dc" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 20 00:09:14 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 20 00:09:14 2012 -0800" }, "message": "Make stack visitor location for bad roots more verbose.\n\nChange-Id: Ib2f3f48ccf1f5b2f3a4d1a61df553ad1a5c29d38\n" }, { "commit": "2bcb4a496b7aa00d996df3a070524f7568fb35a1", "tree": "8422ab8d65b7422008094b2eaadec0dad87b2df3", "parents": [ "efc6369224b036a1fb77849f7ae65b3492c832c0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Nov 08 10:39:18 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Nov 16 14:01:34 2012 -0800" }, "message": "Add \"kind\" argument to Get/SetVReg.\n\nIn order to determine where a register is promoted its necessary to know\nthe kind of use of the register.\nExtend notion of precise-ness to numeric verifier register types.\nDump verifier output in oatdump.\nDump vregs with their location or constant value.\nIntroduce indenting ostream utility.\n\nChange-Id: Ia3d29497877976bc24465484743bca08236e1768\n" }, { "commit": "725a957985171d712d5c048cc3d00ff14968784b", "tree": "3bd4b161268ecd477aa471665316fb27a3dda145", "parents": [ "35883cc623fdf475a4ead1dafcba9e9becc1ed11" ], "author": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Tue Nov 13 18:20:12 2012 -0800" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Wed Nov 14 11:42:33 2012 -0800" }, "message": "Pulled out instrumentation into its own class separate from trace.\n\nThis is in preparation for Ian\u0027s debugger changes. The instrumentation\nstill needs support for debugging, which I\u0027ll add later.\n\nChange-Id: I29ce0af1ff2eaec8f6d2f54963263381363ebf72\n" }, { "commit": "2fa6b2e2fc3d2a2fc27808ce518dc76b80ce369a", "tree": "daba5f5e64409452e31ae42e3b1b6b6491ccb007", "parents": [ "6ed960d260d0ec3cbbef455c646ccd47f4e78119" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 17 00:10:17 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 26 16:12:59 2012 -0700" }, "message": "Interpreter.\n\nThe opcodes filled-new-array and packed-switch aren\u0027t implemented but\nare trivial given that they are variants of implemented opcodes.\nRefactor Field::Get routines to take the declaring class in the case of\nstatic field accesses. This avoids a check on every use of a field.\nRefactor arg array builder to be shared by JNI invokes and invocations\ninto the interpreter.\nFix benign bug in const decoding in the verifier.\n\nChange-Id: I8dee6c1f4b7f033e6c003422c56e9471cfaccda8\n" }, { "commit": "5438ad85f97c5f7fe030a490fd98841a34d3c961", "tree": "8f3ae8276e56e099121ff907af03579f94bd2fe0", "parents": [ "c604d731730b43231f63040c8db1d58304da0cf3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Oct 15 17:22:44 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Oct 15 17:24:05 2012 -0700" }, "message": "Add vregs to ShadowFrame.\n\nChange-Id: I870ab3c19c0e727f0e67b570eb55d45c3497d2ad\n" }, { "commit": "6f1c94968ada57da433debf8e2d1b38a80ceb510", "tree": "3628c7d9f5273d2c83dd7c6f9e816816eed8b70a", "parents": [ "0051be6d118c511308ed7764ef41e0d594292c49" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 15 12:08:41 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 15 12:37:33 2012 -0700" }, "message": "Add root verification when we try to mark an invalid object.\n\nNow when we try to mark an object not contained by any spaces, We call verify\nroots. This prints the root\u0027s vreg and method when it finds an invalid root.\n\nFixed a error in the total paused time statistic.\n\nChange-Id: Id10e4097cce56bc54ee488de32183c18ba3f3780\n" }, { "commit": "b23a7729cf7855fa05345d03a4d84111d5ec7172", "tree": "5313e076b19387db3cbcac95225d3f098f19451d", "parents": [ "137e88f798857321f4007631fdf052d2830ec2c4" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 09 16:54:26 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 09 16:54:26 2012 -0700" }, "message": "Dump maps inline in disassembled code.\n\nIn pursuit of Bug: 7250540, dump mapping and GC map tables inline such\nas:\n\n0x607333a8: f8dfe11c ldr.w lr, [pc, #284] ; 0x6076416d\n0x607333ac: 1c05 mov r5, r0\n0x607333ae: f8df0144 ldr.w r0, [pc, #324] ; 0x6003ba08\n0x607333b2: 9a0b ldr r2, [sp, #44]\n0x607333b4: f04f0b2f orr r11, pc, ThumbExpand(47)\n0x607333b8: 1c29 mov r1, r5\n0x607333ba: 465b mov r3, r11\n0x607333bc: 2900 cmp r1, #0\n0x607333be: f0008070 beq.w +224 (0x607334a2)\n0x607333c2: 47f0 blx lr\nsuspend point dex PC: 44\nGC map objects: v2 (r7), v3 (r5), v6 ([sp + #84]), v7 (r6)\n...\n\nAs GC map and mapping tables are inline, don\u0027t dump them.\nAlso dump dex instructions before code.\n\nChange-Id: I9f0c04182a4cda6844027eae22e8151f2827dc99\n" }, { "commit": "81d425b0b232962441616f8b14f73620bffef5e5", "tree": "cd1e46656269acf20e78817d675e56d9b9133e6c", "parents": [ "bfaf917edbb1de8d158c3615e0da8ac3143d10c8" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 27 16:03:43 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 27 17:19:25 2012 -0700" }, "message": "Pass self to lock methods.\n\nThis avoids frequent recomputation of\nThread::Current/pthread_getspecific.\n\nAlso add a futex based reader/writer mutex that is disabled.\n\nChange-Id: I118fdb99ef1d1c4bfda6446ba3a0d8b6ab31eaee\n" }, { "commit": "66f19258f9728d4ffe026074d8fd429d639802fa", "tree": "fd94009774c6cbbb1528ea096e606133bd35f104", "parents": [ "a5e1e3d153990845d80cb8d013157210f11a473c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 18 08:57:04 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 21 17:32:56 2012 -0700" }, "message": "Change dex cache to be java object instead of array, add pointer to dex file in dex cache.\n\nGeneric clean up to facilitate having GDB macros for Pretty* helper functions.\n\nImproved cleanliness of DexCache since having it as an object array was not the best solution.\n\nFixed a bug in InOrderWalk caused by ResolveType sometimes allocating classes.\n\nRename C++ Method to AbstractMethod and add two new classes Constructor, Method which both inherit from AbstractMethod.\n\nRename done to have the C++ code be closer to the java code.\n\nChange-Id: I4995b4c5e47a3822192b08afa24a639d3b1f4da9\n" }, { "commit": "0c7abda482f53db3d153c073d1c7a145f84e0626", "tree": "837b89c75f0047cbcd16c9121739069a36b4f22a", "parents": [ "ba0b9c55adce3f533845ab1b25c552589e5b4118" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 19 13:33:42 2012 -0700" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Sep 20 15:26:35 2012 -0700" }, "message": "NativePcOffsetToReferenceMap\n\nRather than translate a native PC to a Dex PC and then to the reference\nbitmap, just go straight from the native PC to the reference bitmap.\nEncode the native PC offsets using a hash rather than linearly\nsearching.\n\nChange-Id: Iee1073d93c941c0a31f639e5f23cea9e9f747bee\n" }, { "commit": "8320f3867c02bae9bef6cdab267820cb7b412781", "tree": "7e2211068ccbd17da7e9c83d24b10fdd9db28c39", "parents": [ "c7b83a0d8ac73bdfff751619ae2a34948e3534b7" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Tue Sep 11 16:29:42 2012 -0700" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Thu Sep 13 10:49:20 2012 -0700" }, "message": "Prune useless entries from dex to pc map\n\nStep one of the change. Limit entries in the table to native\ncode safepoint locations (which generally are the return PC\naddresses of any call that might trigger a stack walk and the\nstart addresses of all catch blocks).\n\nPreviously, the mapping_table described ranges. No longer. Any\nnative PC located within compiled Dex code that is found in a\nstack walk should have an exact match in the table.\n\nIn future CLs we\u0027ll add data compression (probably uLeb128) and\nmay add inflation on first use to a faster access map (instead of\nthe current linear search).\n\nNote that this CL introduces somewhat of a regression in the\ncapabilities of oat-dump. Because the mapping table no longer\nassociates each native intruction with its Dex counter-part, the\nnative code disassembly no longer includes interspersed Dex\ndisassembly.\n\nNote also that as of this CL, the compiler is adopting the 100-char\nline length limit used in the rest of Art. The 80-char limit\nshould still be used in any code that we expect to upstream to\nllvm.\n\nChange-Id: I1beca4d57c41e8161bf746bc62abbce08d5bcb4d\n" }, { "commit": "b726dcb581bf72da46527378ccb6889020f0e6e9", "tree": "c0383ed788652926e93147e50d659ed226519548", "parents": [ "fd678beb171a4686a4f2d53ca4188a4ade8fa54e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 08:57:23 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 09:06:19 2012 -0700" }, "message": "Rename GlobalSynchronization to Locks\n\nAlso address some review comments in common_throws relating to\nToStr\u003cInvokeType\u003e and exception detail messages.\n\nChange-Id: Ibf2c0f147689fa236d349bd7f01eed3c2522552b\n" }, { "commit": "00f7d0eaa6bd93d33bf0c1429bf4ba0b3f28abac", "tree": "6a2172ece15c4699e6c2a67ce76f019db0a9a21d", "parents": [ "634ea28f78c4a138e6a1de54eae8696095422415" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 19 15:28:27 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 14 10:45:52 2012 -0700" }, "message": "Global lock levels.\n\nIntroduce the notion of the mutators/GC being a shared-exclusive (aka\nreader-writer) lock. Introduce globally ordered locks, analysable by\nannotalysis, statically at compile time. Add locking attributes to\nmethods.\n\nMore subtly, remove the heap_lock_ and split between various locks that\nare held for smaller periods (where work doesn\u0027t get blocked). Remove\nbuggy Dalvik style thread transitions. Make GC use CMS in all cases when\nconcurrent is enabled. Fix bug where suspend counts rather than debug\nsuspend counts were sent to JDWP. Move the PathClassLoader to\nWellKnownClasses. In debugger refactor calls to send request and\npossibly suspend. Break apart different VmWait thread states. Move\nidentity hash code to a shared method.\n\nChange-Id: Icdbfc3ce3fcccd14341860ac7305d8e97b51f5c6\n" }, { "commit": "0ec569a3e653e0358fcb59c89f2aad708843db53", "tree": "d2526757d84632c3dfd0848873438ce3de9a299c", "parents": [ "02a1e359707c4b4d091ec7b4b5cda3848a6bd8a5" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sun Jul 01 16:43:46 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sun Jul 01 16:43:46 2012 -0700" }, "message": "Disable dex register descriptions.\n\nDisable dex register descriptions. Make GetVReg and SetVReg frame\nroutines fail in unimplementeds.\n\nChange-Id: I0f05a188d3c81c424018b52e2dcea40891cd307e\n" }, { "commit": "08fc03ae5dded4adc9b45b7014a4b9dfedbe95a6", "tree": "3e767d037fd3e47e792b88d982ad38bce26d39e9", "parents": [ "f94404ead1f2e4a1b5a2ff8bbdb65c5e4d4a77ea" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Jun 26 17:34:00 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Jun 27 12:18:25 2012 -0700" }, "message": "Include held locks in SIGQUIT thread dumps.\n\nHandy if you have an ANR that\u0027s locking related. Quick tour:\n\n at org.apache.harmony.dalvik.NativeTestTarget.emptyJniStaticSynchronizedMethod0(Native method)\n - locked \u003c0x60135aa8\u003e (a java.lang.Class\u003corg.apache.harmony.dalvik.NativeTestTarget\u003e)\n at java.lang.reflect.Method.invoke(Native method)\n at C.whileTrue(Main.java:63)\n at C.synchronizedOnClassString(Main.java:56)\n - locked \u003c0x60002a70\u003e (a java.lang.Class\u003cjava.lang.String\u003e)\n at C.nestedSynchronizationWithTryCatch(Main.java:44)\n - locked \u003c0x61336b90\u003e (a java.lang.String)\n - locked \u003c0x61336bd0\u003e (a java.lang.String)\n at C.nestedSynchronization(Main.java:35)\n - locked \u003c0x61336b18\u003e (a java.lang.String)\n - locked \u003c0x61336b50\u003e (a java.lang.String)\n at C.synchronizedOnClassC(Main.java:30)\n - locked \u003c0x613366f8\u003e (a java.lang.Class\u003cC\u003e)\n at C.noLocks(Main.java:27)\n at C.\u003cclinit\u003e(Main.java:24)\n - locked \u003c0x613366f8\u003e (a java.lang.Class\u003cC\u003e)\n at Main.main(Main.java:19)\n\nA non-static synchronized native method works too:\n\n at org.apache.harmony.dalvik.NativeTestTarget.emptyJniSynchronizedMethod0(Native method)\n - locked \u003c0x613371a8\u003e (a org.apache.harmony.dalvik.NativeTestTarget)\n ...\n\nNote that most stack traces don\u0027t look any different; the above is a\npathological example that exercises different kinds of locking. Testing\nwith system_server shows most threads don\u0027t hold any locks.\n\nFuture work (marked by TODO) is that explicit JNI MonitorEnter calls in\nnative code aren\u0027t shown.\n\nChange-Id: I2747f5cddb4ef64b1935736f084a68fe8e4005e9\n" }, { "commit": "ca190666fb11820153f74274c495ba1f913d8a69", "tree": "cc813b9fd3b0ad38cec0e6952bcbbbbce78494f9", "parents": [ "1fb3bd3559eb6fc34f7ba429adf9bb15c05939b0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jun 26 15:45:57 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jun 26 19:50:52 2012 -0700" }, "message": "Fix initializer lists to be Google C++ style.\n\nChange-Id: Iada52ea5cf95492684033993b3299042bdaf6cee\n" }, { "commit": "365c10235438607541fa2259a5fec48061b90bd8", "tree": "26516a4985a8bbef4c8da36b99bf1e54af3ad24d", "parents": [ "147440427984816b6ed5235a9b1fb2cbf99496d0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 22 15:05:28 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jun 26 15:21:01 2012 -0700" }, "message": "Increase use of ScopedJniThreadState.\n\nMove the routines for changing Object* to jobject and vice-versa\n(AddLocalReference and Decode) to ScopedJniThreadState to enforce use of\nObject*s in the Runnable thread state. In the Runnable thread state\nsuspension is necessary before GC can take place.\n\nReduce use of const ClassLoader* as the code bottoms out in FindClass\nand with a field assignment where the const is cast away (ie if we\u0027re\nnot going to enforce the const-ness we shouldn\u0027t pretend it is).\n\nRefactor the Thread::Attach API so that we\u0027re not handling raw Objects on\nunattached threads.\n\nRemove some unreachable code.\n\nChange-Id: I0fa969f49ee6a8f10752af74a6b0e04d46b4cd97\n" } ], "next": "57b47f9b59b8850c5cf7ddba587cc0cb094ca878" }