)]}' { "log": [ { "commit": "c311d0c85e17773042daaa7a4abc58b3e3c0a0c1", "tree": "12103a18b43dd863ab7f3fd51ce9600ef4754d95", "parents": [ "0a01b38244053e72f33339e733b3ae5e15a5966b", "3f52eafe5577b8489f90dc8ed5981b3455206147" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Apr 15 06:43:14 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Apr 15 06:43:14 2014 +0000" }, "message": "Merge \"Prepare field watchpoint support\"" }, { "commit": "3f52eafe5577b8489f90dc8ed5981b3455206147", "tree": "7155df948d345c8a5a2d801c23b396af76527ba0", "parents": [ "9b417e4f0f87da6bfe8dc5f02c987acfcb6dca31" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Apr 04 17:50:18 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Apr 15 08:39:18 2014 +0200" }, "message": "Prepare field watchpoint support\n\nAdds field read/write events in the instrumentation. The debugger now registers\nas a listener for these events so JDWP field access and field modification\nevents can be reported.\n\nThis CL will be followed by another one to report these events from the\ninterpreter. Therefore no JDWP field access and field modification events\ncan be sent for now.\n\nBug: 8267708\nChange-Id: If2a93eb590805567d69015c83cce9cd2ab712cbd\n" }, { "commit": "bb43b433c845e253a11346f0f03706d58a44f23c", "tree": "406fda2973407461cbadd7d0bf45afc109bfedad", "parents": [ "9b417e4f0f87da6bfe8dc5f02c987acfcb6dca31" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Apr 14 11:59:08 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Apr 14 12:01:15 2014 +0200" }, "message": "Clear unused roots from debugger\n\nSingleStepControl and DebugInvokeReq hold roots which are only used during a\nlimited amount of time. Clear these roots once we finish using them.\n\nAlso cleans some code around single-step.\n\nChange-Id: Ica2f65e04a4d60272199490cd40ad1803d3b9f02\n" }, { "commit": "1d66e88ea74f7d85f75db7f145fe955d5f0f6872", "tree": "715cee13931192549a592a7dac91235c0a1d77c0", "parents": [ "b70b09f6cc1f1fbbf604da7b6b1ea5f74e43571c" ], "author": { "name": "nikolay serdjuk", "email": "nikolay.y.serdjuk@intel.com", "time": "Mon Apr 07 13:54:24 2014 +0700" }, "committer": { "name": "nikolay serdjuk", "email": "nikolay.y.serdjuk@intel.com", "time": "Mon Apr 07 13:54:24 2014 +0700" }, "message": "JDWP request is parsed incorrectly due to junk in the descriptor\n\nThe ClassHelper object is created and deleted on the same line.\nTherefore it is wrong to refer to any class members of the object in\nthe next line. Any references to the object and the object itself\nmust have the same live ranges.\n\nChange-Id: I792d7baa3828b0389254f2941705e006cab092cd\nSigned-off-by: nikolay serdjuk \u003cnikolay.y.serdjuk@intel.com\u003e\n" }, { "commit": "dd7624d2b9e599d57762d12031b10b89defc9807", "tree": "c972296737f992a84b1552561f823991d28403f0", "parents": [ "8464a64a50190c06e95015a932eda9511fa6473d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 17:43:00 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Apr 01 08:24:16 2014 -0700" }, "message": "Allow mixing of thread offsets between 32 and 64bit architectures.\n\nBegin a more full implementation x86-64 REX prefixes.\nDoesn\u0027t implement 64bit thread offset support for the JNI compiler.\n\nChange-Id: If9af2f08a1833c21ddb4b4077f9b03add1a05147\n" }, { "commit": "6c90df9d49b29bddc1e3d27ff2698ae1df89d517", "tree": "1a16923b1d323dc1a6ff2d115a0a4a9029c48a4a", "parents": [ "e18b027344e521e1b374d967948c7dd55cc02c8a", "4d8fd49509fdcf203107fb33c62d8f451b6eb1d0" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Apr 01 09:07:49 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Apr 01 09:07:50 2014 +0000" }, "message": "Merge \"Return correct JDWP type tag for array class\"" }, { "commit": "4d8fd49509fdcf203107fb33c62d8f451b6eb1d0", "tree": "cfb6cd9417159ba197b2f7dae0c9cc03a8eb7c94", "parents": [ "a708e32a9f764a48175e705ec4bcd2201c84f492" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Mar 28 16:29:41 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Mar 31 11:14:40 2014 +0200" }, "message": "Return correct JDWP type tag for array class\n\nUses GetTypeTag function to get JDWP type tag for a class. This fixes the\nreturned type tag in ClassObjectReference.ReflectedType command.\n\nBug: 13689172\nChange-Id: Ibb9006eb604d84cfb8e279eaeed1982a136d6510\n" }, { "commit": "1cd53dbb2163f18b689d2a65cf9c6bdcdb01b407", "tree": "b6a214bcc7bd33cfa60e0e7ee51329b3327a537a", "parents": [ "0fd52d5d0cf01e5a109851098a43a79f5615dc0f" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Mar 28 15:26:48 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Mar 28 15:26:48 2014 -0700" }, "message": "Fix a DCHECK failure when causing GC from DDMS.\n\nBug: 13647069\nChange-Id: Iae2746b2b7b4493fcf5f0d40d2bf36a9b2d2efc8\n" }, { "commit": "a76a6d4cb3dedec909bd927f20bc0ab23198a337", "tree": "34ceb525611dde3413cdcab4a3ab8ced3b131fc9", "parents": [ "78bd9b2198f0ccc48036c1517b2d9a9023157dfb" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 20 16:40:17 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 27 16:22:09 2014 +0100" }, "message": "Support inlining with breakpoint\n\nWhen installing/uninstalling a breakpoint in a method, we fully or selectively\ndeoptimize/undeoptimize depending on whether the method can be inlined. When\nthe method can be inlined, it requires full deoptimization. Otherwise, it only\nrequires selective deoptimization.\n\nWe add sanity check to control we are in a consistent state each time we add or\nremove a breakpoint. We also add some comments to better describe the process\nof deoptimization for breakpoint.\n\nBug: 12187616\nChange-Id: Id15adc6e5e2fe783c83c925cbcd19ae02431b7e0\n" }, { "commit": "78bd9b2198f0ccc48036c1517b2d9a9023157dfb", "tree": "dd08a821ee89dfea00f730ffa41ccd8c45295622", "parents": [ "03fd6e179d3bcd7f1a67a0b5de71b756f1637ec6", "4d25df3f76f864b7629ac8c0046d46997f293d8d" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 27 07:46:58 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 27 07:46:58 2014 +0000" }, "message": "Merge \"Refactor deoptimization support in debugger\"" }, { "commit": "4d25df3f76f864b7629ac8c0046d46997f293d8d", "tree": "31840831d8c81d06ffd575fdb0adc8403cd8dbb1", "parents": [ "909f133bc938928a2403baccc983701cb9ebb17f" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Mar 21 17:44:46 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Mar 26 11:46:43 2014 +0100" }, "message": "Refactor deoptimization support in debugger\n\nThis CL prepares breakpoint support for inlined methods where we\u0027ll have to\ndeoptimize everything.\n\nWe move deoptimization-related information to Dbg class only (deoptimization\nrequest queue, full deoptimization event count and deoptimization lock). We\nreplace MethodInstrumentionRequest by DeoptimizationRequest. This is used to\nknow which kind of deoptimization is required for a particular event.\n\nIt also simplifies lock ordering a bit during event setup: we no longer need to\nhold the deoptimization lock while holding the breakpoint lock. Moreover, the\ndeoptimization lock should be held only after the event list lock.\n\nBug: 12187616\nChange-Id: Iff13f004adaeb25e5d609238bacce0b9720510e6\n" }, { "commit": "b98063a4d54c5a741152a1eebc262d8e223b3b25", "tree": "4a5b3bf342db4a28ead0772746017566a6eab723", "parents": [ "909f133bc938928a2403baccc983701cb9ebb17f" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Mar 26 10:57:20 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Mar 26 11:05:11 2014 +0100" }, "message": "Fix alloc tracker locking issue\n\nFixes a lock level violation where we attempt to take the thread list lock\nwhile already holding the alloc tracker lock. We now avoid holding the alloc\ntracker lock when updating allocation entrypoints.\n\nBug: 13646642\nChange-Id: Iab505dae67d0c754031fe30d50a7cbd5e4ba5785\n" }, { "commit": "3b05e9ba874449dbff65b01b8781001f7d93eea6", "tree": "f341766bff83ebea510344c4857af493725d3f58", "parents": [ "027f7fa539514d2a50b448de1de39ac307087483" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 25 09:29:43 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 25 13:56:58 2014 -0700" }, "message": "Add missing debugger root visiting.\n\nBug: 13634574\nChange-Id: I2a76f6c43f1d0ad1922f06deb40a71ff651129fd\n" }, { "commit": "77bef430373a56f7dd9ba99ab8471bf5f571253a", "tree": "02c59b09bbec56227ffc18811f1d5b8814dfea4f", "parents": [ "c5db77cce4138ac1f473872a06d230d876374337", "83a47d8a950867da833dfb5c3d8ddfb3d87a8cfe" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 20 10:22:36 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 20 10:22:37 2014 +0000" }, "message": "Merge \"Fix JDWP ObjectReference.InvokeMethod for virtual method call.\"" }, { "commit": "83a47d8a950867da833dfb5c3d8ddfb3d87a8cfe", "tree": "4d5699876abc9abbed3b89725fd6e97061de1e9e", "parents": [ "0d9c02e661813abdf18b4e7544e204d2da719d20" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 20 09:57:40 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 20 09:57:40 2014 +0100" }, "message": "Fix JDWP ObjectReference.InvokeMethod for virtual method call.\n\nFixes virtual method call by invoking the concrete method in sirt reference\nafter devirtualization, not the original method.\n\nBug: 13526099\nChange-Id: I2e3548eca2f5434e8cece64c22aaf80d1cd8badf\n" }, { "commit": "5426324ea7a4c334cde1fdc4b8aae39f8a8e362d", "tree": "a3547599de816989a9f4a8937203d1b618b0d97c", "parents": [ "0d9c02e661813abdf18b4e7544e204d2da719d20" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Mar 19 18:16:50 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Mar 19 18:17:32 2014 +0100" }, "message": "Fix thread suspension in ObjectReference.MonitorInfo JDWP command.\n\nUses ThreadList::SuspendAll and ThreadList::ResumeAll to suspend/resume all\nthe threads so we collect monitor info while all threads are suspended.\n\nBug: 13528439\nChange-Id: I81ad0698a759cae428aa6a0149dafda3db4cd198\n" }, { "commit": "cb19ebf7609f74b223bd86c94f721498795f9bba", "tree": "0db31e4f83046b3d8159d4ba28d9298ff94e06c3", "parents": [ "f17ce4c5e3b6882aa8849d1ed82df4238c436da2" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Mar 11 15:26:35 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Mar 17 09:15:31 2014 +0100" }, "message": "Fix debugger crash in native method frames.\n\nThe main crash happens when we try to read (StackFrame::GetValues) or write\n(StackFrame::SetValues) values in native frames. We use the method\u0027s vmap to\nknow where Dalvik registers live but native methods don\u0027t have vmap. The fix\nis to reply with the OPAQUE_FRAME error which indicates local values are not\naccessible in the frame.\n\nWe prevent from dereferencing null code item which causes some crashes too.\nThis happens when we compute the line table (Method::LineTable) and variable\ntable (Method::VariableTable) of methods without code: native, proxy and\nabstract methods. We do not expect to encounter abstract methods though. We\ntake care of these kinds of method when mangling/demangling local value slots.\n\nWe also fix the location\u0027s pc of native and proxy frames where it must be -1\n(as 8-byte value). We\u0027ll use this property to detect such frames in the JDWP\ntests.\n\nBug: 13366758\nChange-Id: I78e3263fbf2681b5573571c846390d52b9193849\n" }, { "commit": "53b8b09fc80329539585dcf43657bc5f4ecefdff", "tree": "cac0f82fbb89bd907104e3fed6c36203e11a3de0", "parents": [ "0dea9872082bc3e576ed6cefed86b0d6c0c45ffd" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 13 23:45:53 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 11:28:10 2014 -0700" }, "message": "Refactor reflective method invocation.\n\nMove invocation code out of JNI internal into reflection, including ArgArray\ncode. Make reflective invocation use the ArgArray to build arguments rather\nthan allocating a jvalue[] and unboxing arguments into that.\nMove reflection part of jni_internal_test into reflection_test.\nMake greater use of fast JNI.\n\nChange-Id: Ib381372df5f9a83679e30e7275de24fa0e6b1057\n" }, { "commit": "3314dcf81d116c3142520970286ffaede67ab522", "tree": "424291dddb6c7c3458c33771b6f979035a8e60c4", "parents": [ "cd4f41a667b5eee6b12ec1cbec1912024b3b122e", "b7054baf28d4d652fbd98a94b089344a31898d53" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 13 16:16:13 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 13 16:16:14 2014 +0000" }, "message": "Merge \"Fix debugger crashes in presence of proxy objects.\"" }, { "commit": "52d131d8e71ae9ea915ea84a3f95d49547e8e661", "tree": "4a723664a9634747d273e12c87a396db160f50f6", "parents": [ "37d4c1db4d705f5a28001f65afdd68d0527948d8" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 13 16:17:40 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 13 17:03:50 2014 +0100" }, "message": "Fix thread suspension assertion in debugger.\n\nUpdates Dbg::GetThreadGroup to ensure we do call EndAssertNoThreadSuspension\nafter StartAssertNoThreadSuspension and not returning in the middle. Note this\nonly happens in debug mode where this assertion is enabled.\n\nAlso add missing thread safety annotations.\n\nBug: 13425576\nChange-Id: Idb9f32d289038b77771369c1283774676ff433c7\n" }, { "commit": "b7054baf28d4d652fbd98a94b089344a31898d53", "tree": "54d2548fab1faa859d6e610d03881e71074b1cb7", "parents": [ "37d4c1db4d705f5a28001f65afdd68d0527948d8" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 13 11:52:31 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Mar 13 15:11:19 2014 +0100" }, "message": "Fix debugger crashes in presence of proxy objects.\n\nFix ClassHelper::NumDirectInterfaces to use IfTable::Count instead of the\niftable array\u0027s length (which is twice the interface count). This happens when\nprocessing ReferenceType::Interfaces command and was causing a crash.\n\nReturn ABSENT_INFORMATION error code when we\u0027re asked for the source file of a\nproxy class. Otherwise we call ClassHelper::GetSourceFile which expect the\nclass has a corresponding ClassDef item in the DEX file which is not true for\nproxy classes.\n\nAdd new proxy_test to check ClassHelper works correctly with proxy classes.\n\nBug: 13426918\nChange-Id: I5c1212b1a697dd7dc1ab18e99552ee113c533a5a\n" }, { "commit": "719d1a33f6569864f529e5a3fff59e7bca97aad0", "tree": "fcd84efd7b9806b93ec1a44e2317e6f882e7fe0e", "parents": [ "5365eea9940269b662cfbe103caa348816ff1558" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 06 12:13:39 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 06 19:16:01 2014 -0800" }, "message": "Enable annotalysis on clang ART builds.\n\nFix clang build errors aswell as restructure locking/mutex code for correct\nthread safety analysis support.\nReorder make dependencies so that host builds build first as they should\nprovide better compilation errors than target.\nRemove host\u0027s use of -fno-omit-frame-pointer as it has no value with correct\nuse of CFI, which we should have.\n\nChange-Id: I72cea8da9a3757b1a0b3acb4081feccb7c6cef90\n" }, { "commit": "6fac447555dc94a935b78198479cce645c837b89", "tree": "bcf1449999084b1e1dec3dac287f6f3670d7eda0", "parents": [ "7f0ff7e7fff82566bca5f9353eaa2c4f81f0671a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 25 17:01:10 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 26 16:38:22 2014 -0800" }, "message": "Make allocations report usable size.\n\nWork-in-progress to allow arrays to fill usable size. Bug: 13028925.\nUse C++11\u0027s override keyword on GCC \u003e\u003d 2.7 to ensure that we override GC and\nallocator methods.\nMove initial mirror::Class set up into a Functor so that all allocated objects\nhave non-zero sizes. Use this property to assert that all objects are never\nlarger than their usable size.\nOther bits of GC related clean-up, missing initialization, missing use of\nconst, hot methods in .cc files, \"unimplemented\" functions that fail at\nruntime in header files, reducing header file includes, move valgrind\u0027s space\ninto its own files, reduce number of array allocation routines.\n\nChange-Id: Id5760041a2d7f94dcaf17ec760f6095ec75dadaa\n" }, { "commit": "aaea7343831b64f4351c9abc7493ec062adfaf53", "tree": "9ace06bf09aecdce840d590d9673880461287bdf", "parents": [ "bdbc11899d04b284c0ec4b055e1a009c7034aaa5" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Feb 25 15:10:04 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Feb 25 18:06:04 2014 +0100" }, "message": "Fix debugger disconnection crash.\n\nFixes a check failure where we uninstall instrumentation while we did not\ninstall it before. This happens when the JDWP thread stops with no debugger\nconnected.\n\nBug: 13168905\nChange-Id: I83766ced79721747e5a7aaf27ce01a853ad55f03\n" }, { "commit": "9837939678bb5dcba178e5fb00ed59b5d14c8d9b", "tree": "00f0e6b54d7c4cac78a02752e268724157e50b6e", "parents": [ "3fcf18e25241253f23efbeebe77b2a4c4a7c54d3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 24 16:53:16 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 24 18:47:23 2014 -0800" }, "message": "Avoid std::string allocations for finding an array class.\n\nIntroduce ClassLinker::FindArrayClass which performs an array class lookup\ngiven the element/component class. This has a 16 element cache of recently\nlooked up arrays.\nPass the current thread to ClassLinker Find .. Class routines to avoid calls\nto Thread::Current().\nAvoid some uses of FindClass in the debugger where WellKnownClasses is a\nfaster and more compacting GC friendly alternative.\n\nChange-Id: I60e231820b349543a7edb3ceb9cf1ce92db3c843\n" }, { "commit": "11d40c250ec86c784cf76b8c31e1cf9285d555dc", "tree": "a0c03a28963cf6683b3832cf4548b4bf8961b912", "parents": [ "3188d117d6f1ba5f3a30d0ff231d816ebb59a7f7" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Feb 19 18:00:17 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Feb 20 15:23:01 2014 +0100" }, "message": "Fix deoptimization deadlock\n\nFixes a deadlock occuring during undeoptimization on debugging session end.\n\nBefore disconnecting debugger, we must unregister all requested events. We take\nthe event list lock to browse all these events. Some of them (as METHOD_EXIT)\nmay have activated deoptimization so we need to undeoptimize. During this\nprocess, we restore all original entrypoints of every method in the stack and\nnotify method exit events to the instrumentation listener (see method\nInstrumentationLister::MethodExited). In our case, the instrumentation listener\nis the debugger. It takes the event list lock (to browse the event list and see\nif this event must be posted) but hangs waiting for it. Since it\u0027s already\nholding the event list lock (which is not recursive), it ends up in a deadlock\nsituation.\n\nThis CL fixes the way we prevent from posting method enter/exit events during\nthe process of deoptimization/undeoptimization. We now explicitly set a flag\nindicating if deoptimization is enabled (by the debugger).\n\nAlso removes unused field in InstallStackVisitor class and remove debugger as\nlistener before disabling deoptimization to ensure it does not receive any\nevent when disconnecting.\n\nChange-Id: I49a2ae43e86cf29094f4b462bfa754d7740d3e97\n" }, { "commit": "3c95c591e054ecd2eac40a4d25668f239782aabd", "tree": "6f3389c0826ee3beafe8b20d0faf0d40d928b2d6", "parents": [ "e7da4cf71712a3d7a64166ceadfb176d07018d32", "de34eea060edb4eb34d6ecd840e6e1c4c6489642" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Feb 20 10:10:04 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 20 10:10:05 2014 +0000" }, "message": "Merge \"Do not set ACC_SUPER flag for interfaces\"" }, { "commit": "de34eea060edb4eb34d6ecd840e6e1c4c6489642", "tree": "a15298c69afc2814186973ee46b6faea8d271a7a", "parents": [ "209a3a03595984a7a94103c3c6ba8c5afaf358a1" ], "author": { "name": "Yevgeny Rouban", "email": "yevgeny.y.rouban@intel.com", "time": "Sat Feb 15 01:06:03 2014 +0700" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Feb 20 10:42:40 2014 +0100" }, "message": "Do not set ACC_SUPER flag for interfaces\n\nThe debugger\u0027s GetModifier method forces the ACC_SUPER flag set for all\nclasses and interfaces because dex files don\u0027t contain this flag.\nBut according to the JVM Spec (Class File Structure) the ACC_SUPER flag\nmust not be set for interfaces:\n If the ACC_INTERFACE flag of this class file is set,\n its ACC_ABSTRACT flag must also be set (JLS 9.1.1.1). Such a class\n file must not have its ACC_FINAL, ACC_SUPER or ACC_ENUM flags set.\n\nThe patch sets ACC_SUPER only if ACC_INTERFACE is not set.\n\nSigned-off-by: Yevgeny Rouban \u003cyevgeny.y.rouban@intel.com\u003e\nChange-Id: I6cc4b215a6584d177845a4f8cce1efeb1650f646\n" }, { "commit": "d2fe10a3a34af171bf1631219cd2d6ff6b7778b5", "tree": "b6b7eb8eba23a5c2723518da99c03bf47b97f58a", "parents": [ "5a3f55ad9519e87c0d3bbddaf3d8a186a887a79b" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Jan 15 10:20:56 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Feb 17 11:32:15 2014 +0100" }, "message": "Remove blacklist\n\nRemoves the class initialization blacklist and use transaction to detect and\nrevert class initialization attempting to invoke native method. This only\nconcerns class initialization happening at compilation time when generating an\nimage (like boot.art for the system).\n\nIn transactional mode, we log every object\u0027s field assignment and array update.\nTherefore we\u0027re able to abort a transaction to restore values of fields and\narray as they were before the transaction starts. We also log changes to the\nintern string table so we can restore its state prior to transaction start.\n\nSince transactional mode only happens at compilation time, we don\u0027t need to log\nall these changes at runtime. In order to reduce the overhead of testing if\ntransactional mode is on/off, we templatize interfaces of mirror::Object and\nmirror::Array, respectively responsible for setting a field and setting an\narray element.\n\nFor various reasons, we skip some specific fields from transaction:\n- Object\u0027s class and array\u0027s length must remain unchanged so garbage collector\ncan compute object\u0027s size.\n- Immutable fields only set during class loading: list of fields, method,\ndex caches, vtables, ... as all classes have been loaded and verified before a\ntransaction occurs.\n- Object\u0027s monitor for performance reason.\n\nBefore generating the image, we browse the heap to collect objects that need to\nbe written into it. Since the heap may still holds references to unreachable\nobjects due to aborted transactions, we trigger one collection at the end of\nthe class preinitialization phase.\n\nSince the transaction is held by the runtime and all compilation threads share\nthe same runtime, we need to ensure only one compilation thread has exclusive\naccess to the runtime. To workaround this issue, we force class initialization\nphase to run with only one thread. Note this is only done when generating image\nso application compilation is not impacted. This issue will be addressed in a\nseparate CL.\n\nBug: 9676614\nChange-Id: I221910a9183a5ba6c2b99a277f5a5a68bc69b5f9\n" }, { "commit": "fd14755c09ada4769b5b6658e484441ca3f628b5", "tree": "c70c10b6e319f44dadb56c78acaf7ce89b502d82", "parents": [ "3c976a8f82007285316dd9d1bbccb659b0668a06", "0177e53ea521ad58b70c305700dab32f1ac773b7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 12 01:16:29 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Feb 12 01:16:30 2014 +0000" }, "message": "Merge \"Work in the direction of hard float quick ABIs.\"" }, { "commit": "0177e53ea521ad58b70c305700dab32f1ac773b7", "tree": "db15627a19a04634cf84cecd15b813319d80d225", "parents": [ "abaf927f29f6feceb3df3e6ced7d01970ba0dbe9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 11 16:30:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 11 17:02:21 2014 -0800" }, "message": "Work in the direction of hard float quick ABIs.\n\nPass a shorty to ArtMethod::Invoke so that register setup can use it.\nDocument x86-64 ABI.\nAdd extra debug output for one JNI native method registration fails, namely a\ndump of the Class and its dex file\u0027s location.\nAdd hack to get testing of OatMethod\u0027s without GC maps working in 64bit.\n\nChange-Id: Ic06b68e18eac33637df2caf5e7e775ff95ae70f3\n" }, { "commit": "83c8ee000d525017ead8753fce6bc1020249b96a", "tree": "d5167ed15dee2629905ac3640b6ea0578d4ae312", "parents": [ "7cba217ab0661d74deccbb97160cdf60b74d4ea3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 28 14:50:23 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 11 10:40:10 2014 -0800" }, "message": "Add root types and thread id to root visiting.\n\nEnables us to pass the root type and thread id to hprof.\n\nBug: 12680863\nChange-Id: I6a0f1f9e3aa8f9b4033d695818ae7ca3460d67cb\n" }, { "commit": "412c7fced915fc8d4d5e4166e977d55c809168a6", "tree": "b8efee7f46a440f3e89765b1e9b2aa6454839c87", "parents": [ "109e2b34799a377a0407781fc32ec1ec607d6c41" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Feb 07 12:18:39 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sat Feb 08 15:23:36 2014 -0800" }, "message": "Make debugger / jdwp compaction safe.\n\nFixed GetInstances, GetReferringObjects, CountInstances to use\nVisitObjects instead of the live bitmap.\n\nWe now treat the object registry as system weaks and update the\nobjects when/if they move. Also added the recent_allocation_records_\nas roots.\n\nBug: 12936165\n\nChange-Id: I615c289efbf2977ceab5c4ffa73d216d799e6e33\n" }, { "commit": "a52214230ad091359956ed7566670963aedf2045", "tree": "d5af74460066ce7ef7e6f7bf212552ba086867f2", "parents": [ "d1b9241a91f5eef9995e54cf80f9789045e5a233", "0376e6b62bfb43b515b791f0a7afed2d0d8030bc" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Feb 07 09:00:45 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 07 09:00:46 2014 +0000" }, "message": "Merge \"Fix JDWP thread crash.\"" }, { "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": "0376e6b62bfb43b515b791f0a7afed2d0d8030bc", "tree": "776dc23a8313dac196ac485975aa6d8f228d7756", "parents": [ "78c0bb9bc0668be07baa0bde1e82a475a0ac1f86" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Feb 06 18:12:59 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Feb 06 18:13:54 2014 +0100" }, "message": "Fix JDWP thread crash.\n\nFix a crash happenning when JDWP thread attempts to process incoming packets\nfrom a closed socket. It happens when the runtime shuts down while JDWP thread\nis still running and waiting for incoming JDWP packets.\n\nWhen we close the JDWP connection from runtime destructor, we wake up the JDWP\nthread. To avoid this, we call Dbg::Dispose before closing the connection.\n\nBug: 12578041\nChange-Id: I8d03478b6c3f5f49e08899d476f292b7a401d5f3\n" }, { "commit": "65637ebbb3c5bdf3f287a76df29a0340c10414c8", "tree": "b9f365900059818cb0dedd4acef38c8cd78143d7", "parents": [ "49c670f268218f3b71dbdd99f26606487713f864" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Jan 10 17:40:02 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jan 14 10:30:37 2014 +0100" }, "message": "Fix JDWP IsCollected command handling.\n\nUpdate the handling of the JDWP IsCollected command for \u0027null\u0027 object id and\ninvalid object id.\n\nWe follow the RI where \u0027null\u0027 object id triggers an INVALID_OBEJCT error but\nan invalid object id does not trigger error and return \u0027true\u0027 result.\n\nAlso fix handling of \u0027null\u0027 object id to avoid confusion with valid object id.\nWe used to rely on ObjectRegistry::Get returning NULL to detect null object id\n(in which case we return an INVALID_OBJECT error). The issue is this method\nmay also return NULL for a valid object id if the corresponding object has\nbeen collected. In this case, we must not return any JDWP error and the result\nof the command should be \u0027true\u0027 meaning the object identified by the given id\nhas been collected.\n\nBug: 12221035\nChange-Id: I013f3759bac7cf5b1e9ee3e8c0d1bb2a11f1b654\n" }, { "commit": "138dbfc3336e379d74d157086f69a0fbe830089b", "tree": "f5fa88466e6ab339d2a6faf9c9105bafcaee3ea7", "parents": [ "62e45483db4aa1be096d3ce91903e01ef52fb901" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Dec 04 18:15:25 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Jan 13 15:08:24 2014 +0100" }, "message": "Selective deoptimization.\n\nUpdate the instrumentation to allow selective deoptimization.\n\nSeparate instrumentation listener registration from stubs configuration. A\nlistener is now responsible for configuring the appropriate stubs.\n- The method tracing listener installs instrumentation entry/exit stubs or\nthe interpreter depending on the accuracy of events we want (controlled by\nkDeoptimizeForAccurateMethodEntryExitListeners).\n- The debugger registers itself as an instrumentation listener but does not\nmodify methods entrypoints. It only does this on demand when deoptimizing one\nmethod or all the methods.\n\nThe selective deoptimization is used for breakpoint only. When a breakpoint is\nrequested, the debugger deoptimizes this method by setting its entrypoint to\nthe interpreter stub. As several breakpoints can be set on the same method, we\ndeoptimize only once. When the last breakpoint on a method is removed, we\nreoptimize it by restoring the original entrypoints.\n\nThe full deoptimization is used for method entry, method exit and single-step\nevents. When one of these events is requested, we force eveything to run with\nthe interpreter (except native and proxy methods). When the last of these\nevents is removed, we restore all methods entrypoints except those which are\ncurrently deoptimized.\n\nDeoptimizing a method requires all mutator threads be suspended in order to\nwalk each thread\u0027s stack and ensure no code is actually executing while we\nmodify methods entrypoints. Suspending all the threads requires to not hold\nany lock.\nIn the debugger, we deoptimize/undeoptimize when the JDWP event list changes\n(add or remove a breakpoint for instance). During the update, we need to hold\nthe JDWP event list lock. This means we cannot suspend all the threads at this\ntime.\nIn order to deal with these constraints, we support a queue of deoptimization\nrequests. When an event needs selective/full deoptimization/undeoptimization,\nwe save its request in the queue. Once we release the JDWP event list lock, we\nsuspend all the threads, process this queue and finally resume all the threads.\nThis is done in Dbg::ManageDeoptimization. Note: threads already suspended\nbefore doing this remain suspended so we don\u0027t \"break\" debugger suspensions.\n\nWhen we deoptimize one method or every method, we need to browse each thread\u0027s\nstack to install instrumentation exit PC as return PC and save information in\nthe instrumentation stack frame. Now we can deoptimize multiple times during\nthe execution of an application, we need to preserve exisiting instrumentation\nframes (which is the result of a previous deoptimization). This require to push\nnew instrumentation frames before existing ones so we don\u0027t corrupt the\ninstrumentation stack frame while walking the stack.\n\nBug: 11538162\nChange-Id: I477142df17edf2dab8ac5d879daacc5c08a67c39\n" }, { "commit": "f288bd1b268bbbd94c163c9c23bdd20e42bbc36b", "tree": "9e70a2778d34bfd5ca096d27c254c2639346b033", "parents": [ "ff8d298c9a937822492e85b264bc0c4a3b3642a9", "e96060aa2483529d087031f7cdcc0405f1ef9218" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Dec 12 15:52:24 2013 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Dec 12 15:52:26 2013 +0000" }, "message": "Merge \"Manage JDWP errors related to garbage collection.\"" }, { "commit": "e96060aa2483529d087031f7cdcc0405f1ef9218", "tree": "72812d15a71818a8b197b17e48199560bc83a8ae", "parents": [ "315ab6c077c4db2031f1ffa40b78722d8269dc9b" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Dec 11 12:06:28 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Dec 12 10:03:39 2013 +0100" }, "message": "Manage JDWP errors related to garbage collection.\n\nReturns INVALID_OBJECT error in case of invalid object ID or null object ID in\ncommands ObjectReference.DisableCollection, ObjectReference.EnableCollection\nand ObjectReference.IsCollected.\n\nNote: JDWP specs do not state ObjectReference.EnableCollection must return any\nerror code. We choose to be more strict so these three commands all behave the\nsame.\n\nChange-Id: I06fb75b75f7d33cf4d23e121d9541bfd70b778bb\n" }, { "commit": "6054239c1d10e35dfa953c9c3d414db366e86650", "tree": "3f06f1636d4142ef9c518374d82999a527fb7781", "parents": [ "19031d51ab9edfc3b1996891d654a6b86ca937c0", "0630ab5239a7d7be24dedbc3f66c822332446fc3" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Dec 12 08:33:37 2013 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Dec 12 08:33:37 2013 +0000" }, "message": "Merge \"Fix null argument handling during invoke from JDWP.\"" }, { "commit": "0630ab5239a7d7be24dedbc3f66c822332446fc3", "tree": "3043efa35ae177db590340138b52772bf71642dd", "parents": [ "315ab6c077c4db2031f1ffa40b78722d8269dc9b" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Nov 28 18:53:35 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Dec 12 09:29:04 2013 +0100" }, "message": "Fix null argument handling during invoke from JDWP.\n\nFix a crash when passing null argument to a JDWP invoke.\n\nChange-Id: I167f93f3a411f0de8458db3ba5bed3169d557ee9\n" }, { "commit": "8755359a35a4aa915fe3753633015263c7e97b74", "tree": "57c0b62f7e9b6cacfbd18d1e33f437b69621fa48", "parents": [ "0b5a3fe08213dff52b73b2388b50a87d09b7da4f", "c528dba35b5faece51ca658fc008b688f8b690ad" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Dec 11 20:25:08 2013 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Dec 11 20:25:09 2013 +0000" }, "message": "Merge \"Enable moving classes.\"" }, { "commit": "c528dba35b5faece51ca658fc008b688f8b690ad", "tree": "d95c113a639d07f7e3b88e8a6caa9b5f6e044801", "parents": [ "76f552307ee980221626d1dff0ada56c989d24ca" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 26 12:00:11 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Dec 10 16:00:58 2013 -0800" }, "message": "Enable moving classes.\n\nSlight reduction in Zygote size, memory savings are in the noise.\nBefore: Zygote size: 8739224\nAfter: Zygote size: 8733568\n\nFixed a bug where we didn\u0027t set the concurrent start bytes after\nswitching the allocator from bump pointer to ROSAlloc in the\nzygote. This caused excessive memory usage.\n\nAdded the method verifiers as roots to fix an issue caused by\nRegTypes holding a Class*.\n\nAdded logic to clear card table in the SemiSpace collector, this\nreduces DalvikOther from ~2400k -\u003e ~1760k when using the SemiSpace\ncollector.\n\nAdded a missing lock to the timing loggers which caused a rare\none time crash in std::set.\n\nBug: 11771255\nBug: 8499494\nBug: 10802951\n\nChange-Id: I99d2b528cd51c1c5ed7012e3220b3aefded680ae\n" }, { "commit": "123756a041baf8421ed933312605daa5ef082f6f", "tree": "0b9524660399b9e42adcc178027c9ddd8a61bd38", "parents": [ "315ab6c077c4db2031f1ffa40b78722d8269dc9b" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Nov 27 15:49:42 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Dec 05 15:02:35 2013 +0100" }, "message": "Fix JDWP class-only modifier.\n\nFix Dbg::MatchType which is used for class-only and exception-only modifiers.\n\nAlso fix crash happening when notifying an exception. A debugger may walk the\nstack of the current thread so we need to keep the instrumentation stack frame\nsynced with the native stack.\n\nBug: 11856587\nChange-Id: Ibf95f8a83ce9ee640abf945e498b42cc88ea92a0\n" }, { "commit": "87118ed6f3f99e7df33214c277cf200a7b9a7499", "tree": "ffcd467906f1e23dfcce6ec58327bf878b6a75ae", "parents": [ "5da3778746375b73e7e77c5f1371f29684674776" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Nov 26 17:57:18 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Nov 26 17:57:18 2013 +0100" }, "message": "Fix JDWP single-step removal.\n\nChange-Id: I4b977ea7c60b32723009e571269ca5fc1c6e264e\n" }, { "commit": "dc10addb338cabc936462b889f5be63030299eb4", "tree": "75d583b2e3bab97c77ca37c8e6e28c860f536095", "parents": [ "d9cffeaa478bd30ad89a9dfc9680a27ce5efaadf", "d38667a055d507492fd05f78519a7e1f0b85ea03" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Nov 25 16:23:26 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Mon Nov 25 16:23:27 2013 +0000" }, "message": "Merge \"Minor field name cleanup in debugger.\" into dalvik-dev" }, { "commit": "d38667a055d507492fd05f78519a7e1f0b85ea03", "tree": "d71f7695f0ec3f4fa5beae55a81789592b295502", "parents": [ "2054553237998043f0cd3d6b9fef799a4e6d95c0" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Nov 25 15:43:54 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Nov 25 17:02:54 2013 +0100" }, "message": "Minor field name cleanup in debugger.\n\nUse the same naming convention for all fields of DebugInvokeReq structure.\n\nChange-Id: Ieaf65eef592f96efa47975eef15334279ed4fc8a\n" }, { "commit": "34a0e6a6d315999ddf6b64bb10b6ece25b90b25a", "tree": "03a1c591da3ff125afa391869bcdd9df95821b67", "parents": [ "2054553237998043f0cd3d6b9fef799a4e6d95c0", "cbb2d20bea2861f244da2e2318d8c088300a3710" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 20 19:34:11 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Wed Nov 20 19:34:11 2013 +0000" }, "message": "Merge \"Refactor allocation entrypoints.\" into dalvik-dev" }, { "commit": "cbb2d20bea2861f244da2e2318d8c088300a3710", "tree": "9735d496716cf165ea0ee2d7e2f62d723ffc7734", "parents": [ "d31fb9718a6180304cd951619dc36be8e090a641" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Nov 14 17:45:16 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 20 11:14:11 2013 -0800" }, "message": "Refactor allocation entrypoints.\n\nAdds support for switching entrypoints during runtime. Enables\naddition of new allocators with out requiring significant copy\npaste. Slight speedup on ritzperf probably due to more inlining.\n\nTODO: Ensuring that the entire allocation path is inlined so\nthat the switch statement in the allocation code is optimized\nout.\n\nRosalloc measurements:\n4583\n4453\n4439\n4434\n4751\n\nAfter change:\n4184\n4287\n4131\n4335\n4097\n\nChange-Id: I1352a3cbcdf6dae93921582726324d91312df5c9\n" }, { "commit": "2054553237998043f0cd3d6b9fef799a4e6d95c0", "tree": "417ccbc5f75eaf36cfd0b62c8eb169b29975dddc", "parents": [ "212d2ab3f5b92095e90008f27293531fc311e96d", "61b7f1b05d1fe12d4009316263bf990903e4edff" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Nov 20 08:47:55 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Wed Nov 20 08:47:56 2013 +0000" }, "message": "Merge \"Move single-step control into thread.\" into dalvik-dev" }, { "commit": "212d2ab3f5b92095e90008f27293531fc311e96d", "tree": "67d62c792372509e65070cbfc284790d2126dc8b", "parents": [ "d31fb9718a6180304cd951619dc36be8e090a641", "579b02416e05e32e535126e1ed61613a2cdb030e" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Nov 19 21:55:36 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Tue Nov 19 21:55:36 2013 +0000" }, "message": "Merge \"Add support for JDWP METHOD_EXIT_WITH_RETURN_VALUE events.\" into dalvik-dev" }, { "commit": "61b7f1b05d1fe12d4009316263bf990903e4edff", "tree": "7929d410750a8099ef254d3858d1da87d595bac8", "parents": [ "03aca12cc4b15a4d2fe1c6acbcb1d62b2baa9eb4" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Nov 15 15:59:30 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Nov 19 21:13:27 2013 +0100" }, "message": "Move single-step control into thread.\n\nThis CL moves single-step control into the Thread structure. This is stored in\nThread::single_step_control_ member. This allows to support single-stepping of\nmultiple threads at the same time.\n\nSince each thread holds its single-step information, we no longer need to use\nthe breakpoint lock to support single-stepping. It helps reduce lock contention\non this lock while debugging.\n\nAll JDWP tests passed on the host and on the target with this CL.\n\nBug: 11667502\nChange-Id: I886d5c8c625ca5a072803e296c32eec5f7e9e82d\n" }, { "commit": "579b02416e05e32e535126e1ed61613a2cdb030e", "tree": "4431587a30c1d5a3e7c0435decbd7ac21104bf7a", "parents": [ "e4a50ee34695a9d90cf03fbb1e8afd1e434f6ee1" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Nov 18 13:16:49 2013 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Nov 19 11:16:44 2013 -0800" }, "message": "Add support for JDWP METHOD_EXIT_WITH_RETURN_VALUE events.\n\nBug: 11569539\nChange-Id: Ibc7a80df83470ffd726d73695a05f4938248f292\n" }, { "commit": "e24fa3e678b1432f8e7325b9b5718d5a31cb74e1", "tree": "1711c7ef571a4ac030d4995de0c53fbeb7c6ea88", "parents": [ "d08a4984b4153f3be3d65d339eaac985da7e17ab", "51db44a194bafc3810a41164a8b39614f10e79df" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Nov 19 18:38:58 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Tue Nov 19 18:38:59 2013 +0000" }, "message": "Merge \"Propagate \u0027this_object\u0027 for method unwind event.\" into dalvik-dev" }, { "commit": "51db44a194bafc3810a41164a8b39614f10e79df", "tree": "c5a02ad6cafe66b10f98957cb17db469fb008ede", "parents": [ "03aca12cc4b15a4d2fe1c6acbcb1d62b2baa9eb4" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Nov 19 10:00:29 2013 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Nov 19 10:06:03 2013 +0100" }, "message": "Propagate \u0027this_object\u0027 for method unwind event.\n\nPropagates the \u0027this_object\u0027 to InstrumentationListener::MethodUnwind callback.\n\nChange-Id: I12561f1a611b8399b94e669f9b8a6eaaf1a58631\n" }, { "commit": "cf58d4adf461eb9b8e84baa8019054c88cd8acc6", "tree": "c49fa473b17e299bc649688076e7d69938741e4e", "parents": [ "be56c9e63335ce99f1042e8660aeac4690b965a9" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Sep 26 14:21:22 2013 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Sat Nov 16 21:35:03 2013 -0800" }, "message": "A custom \u0027runs-of-slots\u0027 memory allocator.\n\nBug: 9986565\nChange-Id: I0eb73b9458752113f519483616536d219d5f798b\n" }, { "commit": "b7cefc7f5cac99a62fd4e662c1bdeec750434e28", "tree": "4d5a37f95ef3af6fa1febfa62bf0af207724077c", "parents": [ "ffcd9d25199a944625bd3c9a766349c23dcbdb66" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Nov 14 14:51:09 2013 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Nov 14 15:42:55 2013 -0800" }, "message": "Put arguments first in debugger variable table and fix name bug.\n\nBug: 11569468\n\nChange-Id: I63d45427ded0937c3ab2456fe5cec22da5558e53\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": "fa82427c68b09f4aedbee319dc71579afbfc66f5", "tree": "b8ae1c7be4a00dce290887ee857be5e466a0902b", "parents": [ "6b98c91275d4361d1b74effad36995cc2d687a55" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 05 16:12:57 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Nov 06 12:00:14 2013 -0800" }, "message": "Fix oatdump and valgrind.\n\nBug: 11531382\nMove allocation instrumentation out of runtime into instrumentation. Don\u0027t\nattempt to suspend threads in unstarted runtimes.\nMake indentation support sputc returning eof, on which it will sync and try\nagain. A further failure likely means the disk is full.\nMove the dump-oat output directory to be art as now there\u0027s too much output to\nfit all the dump-oat data in our standard /tmp.\n\nChange-Id: I8ea848ace318552c180e2efa46570288ff1ca62c\n" }, { "commit": "4598669063e65a66b7a35f63ba035139d4c795d2", "tree": "b48c2ac07195d931351ae95987c105e2ad1fec24", "parents": [ "05c487d13791ffa238bef540dc11af1113ff1966", "e2946917852af9d2ec643c7c987d2bb66cf2f086" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Oct 30 21:26:12 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Wed Oct 30 21:26:12 2013 -0700" }, "message": "am e2946917: am ebaa337a: am c3b800aa: Merge \"Fix JDWP line table output.\" into klp-dev\n\n* commit \u0027e2946917852af9d2ec643c7c987d2bb66cf2f086\u0027:\n Fix JDWP line table output.\n" }, { "commit": "dfb325e0ddd746cd8f7c2e3723b3a573eb7cc111", "tree": "0b63738b04404f42015da7164dfe5ec506452b02", "parents": [ "eda6df7374bf61c933d4a36635e2a9a30c9892a0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 30 01:00:44 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 30 01:00:44 2013 -0700" }, "message": "Don\u0027t use UTF16 length as length for MUTF8.\n\nBug 11367555.\n\nChange-Id: Ia0b07072a1a49d435c3b71ed9a668b316b7ff5d8\n" }, { "commit": "f2910eef247b45ce1d489e323b36b5de6b6157aa", "tree": "cd7e7bef7b016180ce90df635b907201c346bf5c", "parents": [ "bcf09e40f1b5edb7c7e90a96efadba5482b13551" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Sat Oct 19 16:39:24 2013 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Sat Oct 19 16:39:24 2013 +0200" }, "message": "Fix JDWP line table output.\n\nBug: 11238911\nChange-Id: I952fba04393b23b1d42dc6a2d22cd583ce2512ce\n" }, { "commit": "d9c4fc94fa618617f94e1de9af5f034549100753", "tree": "1305efbbc3d4bc306c0947bb6d4b01553667f98e", "parents": [ "7ef126ce0593929bcf8fb73d8b1119ce3b95b3f2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 01 19:45:43 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 02 09:31:55 2013 -0700" }, "message": "Inflate contended lock word by suspending owner.\n\nBug 6961405.\nDon\u0027t inflate monitors for Notify and NotifyAll.\nTidy lock word, handle recursive lock case alongside unlocked case and move\nassembly out of line (except for ARM quick). Also handle null in out-of-line\nassembly as the test is quick and the enter/exit code is already a safepoint.\nTo gain ownership of a monitor on behalf of another thread, monitor contenders\nmust not hold the monitor_lock_, so they wait on a condition variable.\nReduce size of per mutex contention log.\nBe consistent in calling thin lock thread ids just thread ids.\nFix potential thread death races caused by the use of FindThreadByThreadId,\nmake it invariant that returned threads are either self or suspended now.\n\nCode size reduction on ARM boot.oat 0.2%.\nOld nexus 7 speedup 0.25%, new nexus 7 speedup 1.4%, nexus 10 speedup 2.24%,\nnexus 4 speedup 2.09% on DeltaBlue.\n\nChange-Id: Id52558b914f160d9c8578fdd7fc8199a9598576a\n" }, { "commit": "d91d6d6a80748f277fd938a412211e5af28913b1", "tree": "5c45f4ae53aab3f530284a97dc328b4e38ce7c94", "parents": [ "9c11b56296b8cffc94b0b854ce6cff7e014b8d57" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 25 20:26:14 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 26 21:08:25 2013 -0700" }, "message": "Introduce Signature type to avoid string comparisons.\n\nMethod resolution currently creates strings to then compare with strings formed\nfrom methods in other dex files. The temporary strings are purely created for\nthe sake of comparisons. This change creates a new Signature type that\nrepresents a method signature but not as a string. This type supports\ncomparisons and so can be used when searching for methods in resolution.\n\nWith this change malloc is no longer the hottest method during dex2oat (now its\nmemset) and allocations during verification have been reduced. The verifier is\ncommonly what is populating the dex cache for methods and fields not declared\nin the dex file itself.\n\nChange-Id: I5ef0542823fbcae868aaa4a2457e8da7df0e9dae\n" }, { "commit": "3b4c18933c24b8a33f38573c2ebcdb9aa16efeb5", "tree": "5298ccd9c1f1f6b329c0cb6cefac6a8df43dd633", "parents": [ "f7e090ebcded6d6693894c018d89c4add79253ff" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Sep 12 21:33:12 2013 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Sep 25 20:28:49 2013 -0700" }, "message": "Split the allocation path into \u0027instrumented\u0027 and \u0027uninstrumented\u0027\nones.\n\nThe instrumented path is equivalent to the existing allocation path\nthat checks for three instrumentation mechanisms (the debugger\nallocation tracking, the runtime allocation stats collection, and\nvalgrind) for every allocation. The uinstrumented path does not\nperform these checks. We use the uninstrumented path by default and\nenable the instrumented path only when any of the three mechanisms is\nenabled. The uninstrumented version of Heap::AllocObject() is inlined.\n\nThis change improves the Ritz MemAllocTest by ~4% on Nexus 4 and ~3%\non Host/x86.\n\nBug: 9986565\nChange-Id: I3e68dfff6789d77bbdcea98457b694e1b5fcef5f\n" }, { "commit": "fc0e94bed3f88ed7e50854fd8dfaf5dcb345250f", "tree": "5cfbe05084351576e9659cb8f7b66dcb6163a37b", "parents": [ "576fe9d4181c749aa510e32d2521ed4192bdfda0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Sep 23 23:51:32 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Sep 24 17:07:24 2013 -0700" }, "message": "StringPiece clean up.\n\nProfile guided clean up.\nTry to avoid creating StringPieces with the contents of a dex file where\nthe length is known.\nTry to avoid RegTypeCache::FromDescriptor when there\u0027s a class available.\nMake ConstantType::ConstantValue inlinable.\nSaving of about 50ms from a 2 threaded ThinkFree compile on host.\n\nChange-Id: I47a12c3c76f46e2c9805be1c3a3e3870fe1f5d85\n" }, { "commit": "3e47a748eb646b8d2fc8e8c4f11b270d9ae2c607", "tree": "fe03f35aa83f71059294944796ebf6c077ab1eae", "parents": [ "3b99337ec2a92ca5fc0b3bd1d44be40e5eb2a30f" ], "author": { "name": "Ruben Brunk", "email": "rubenbrunk@google.com", "time": "Mon Sep 09 17:56:07 2013 -0700" }, "committer": { "name": "Ruben Brunk", "email": "rubenbrunk@google.com", "time": "Tue Sep 10 17:26:49 2013 -0700" }, "message": "Remove dependency on JNIHelp header side effects.\n\nBug: 10680559\nChange-Id: I2f8ab34803a4554dac35c941ed58c0c845754655\n" }, { "commit": "920af3e556c730a5fbdab90a6d0ec1a2dbe8940b", "tree": "29d7aa78d4a4fec87a403c8e2c93214d19b7cc99", "parents": [ "10cc81056261625902a8a073bafb9f499eec4ff7" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Aug 28 15:46:38 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Aug 29 10:03:52 2013 -0700" }, "message": "Fix thread status in THST chunk to use JDWP thread state.\n\nBug: 10492287\n\nChange-Id: I830a2ce71d02c5b09c7040583be98d818eef9385\n" }, { "commit": "02e25119b15a6f619f17db99f5d05124a5807ff3", "tree": "7be4cbbf28033e5ee0621565b410fe5d8170a8fb", "parents": [ "7d70a7932f0ba09eb01a93caab060aef1403d4e6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 14 16:14:24 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 16 13:15:37 2013 -0700" }, "message": "Fix up TODO: c++0x, update cpplint.\n\nNeeded to update cpplint to handle const auto.\n\nFixed a few cpplint errors that were being missed before.\n\nReplaced most of the TODO c++0x with ranged based loops. Loops which\ndo not have a descriptive container name have a concrete type instead\nof auto.\n\nChange-Id: Id7cc0f27030f56057c544e94277300b3f298c9c5\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": "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": "f52935278fca8c7aa220543eef4544e3d1105d91", "tree": "16f1882fc07c3a8a7dcd271919bb3a7823ebc520", "parents": [ "05eecea70b64095db2bc5f9f8aef7475964a9d7b" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 19 00:24:00 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 19 00:24:00 2013 -0700" }, "message": "Move JDWP to std::vector\u003ciovec\u003e to remove runtime/arrays warning\n\nChange-Id: I8494a25a65de4ebac2b1f4f41c0f5eedf117b7ac\n" }, { "commit": "2d88862f0752a7a0e65145b088f49dabd49d4284", "tree": "3fac6bbd9a0ff3836f361e46579151c8a93420b6", "parents": [ "751d4ed43e8118f145edf18db4d987daf7dabbae" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 17:02:00 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 17:02:52 2013 -0700" }, "message": "Fixing cpplint readability/casting issues\n\nChange-Id: I6821da0e23737995a9b884a04e9b63fac640cd05\n" }, { "commit": "df62950e7a32031b82360c407d46a37b94188fbb", "tree": "038bf95a2ce296ae3e0c30a131ac22c0986f0f52", "parents": [ "0cd7ec2dcd8d7ba30bf3ca420b40dac52849876c" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 22:39:56 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 00:13:18 2013 -0700" }, "message": "Fix cpplint whitespace/parens issues\n\nChange-Id: Ifc678d59a8bed24ffddde5a0e543620b17b0aba9\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": "93ba893c20532990a430741e0a97212900094e8c", "tree": "019990a1957562c1f573c77297df5923cce98f47", "parents": [ "b1eba213afaf7fa6445de863ddc9680ab99762ea" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 21:31:49 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 21:46:12 2013 -0700" }, "message": "Fix cpplint runtime/explicit issues\n\nChange-Id: I352ba0b427f1ff9b22887693952b180eae0839ba\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": "3d92d523089bdd7881d2319414a29bf77172b432", "tree": "e11d0af1421cfd41966b892722cfa8d5fa787524", "parents": [ "1581766afea0b31de0ed0f9f677fc3b92a29daae" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 09:03:08 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 09:03:08 2013 -0700" }, "message": "Add HAVE_ANDROID_OS check around cutils/properties.h include\n\nChange-Id: Ibe3ffd4a829d3ec0e41f861b09da0990e603f2c2\n" }, { "commit": "b1a5879d2c87a5af9f8dbd63a112b138b87135e4", "tree": "663098cb9c72b39ae76f3b8d8dec6b5166772431", "parents": [ "d3d41ba95c7760891a1ca4fc02e3f88baaa25dda" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Jul 11 18:10:58 2013 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Jul 11 18:10:58 2013 -0700" }, "message": "Port 1778d17feda232e5f794ba1ebdf400a600a17fd8 to art.\n\nIncrease the default alloc tracker maximum, and add a new system\nproperty \"dalvik.vm.allocTrackerMax\" that allows developers to choose\nan arbitrary limit.\n\nChange-Id: I8b3fcfb00ba87d1cf515d5835c101b5375b1d8a2\n" }, { "commit": "46e811b07533a4c21f8c9abe22d22d6a3d8b18b0", "tree": "45415fdee46c30737742663d0389d6ce21fa77ac", "parents": [ "534f2b6f430c7984fda9333f8e72b37962b144b4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 10 17:09:14 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 11 09:19:00 2013 -0700" }, "message": "Fix DDMS allocation tracker.\n\nThe allocation tracker previously did a NewByteArray while holding the\ngAllocTrackerLock. This was problematic due to the fact that this new\nallocation would also try to acquire said lock. Fixed the issue by\nmoving the newByteArray call outside of the lock.\n\nBug: 9767557\n\nChange-Id: I7e94e631d9c96aa642637de52f100b371f813294\n" }, { "commit": "1d54e73444e017d3a65234e0f193846f3e27472b", "tree": "1de93661e95a0ce6fa78fdfc23d0cfd3dd2a06f7", "parents": [ "4c22e7eabef3f815841dfc6e0d5bbead96150752" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 02 21:10:01 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 21 11:10:42 2013 -0700" }, "message": "GC clean up.\n\nGreater use of directories and namespaces.\nFix bugs that cause verify options to fail.\nAddress numerous other issues:\n\nGC barrier wait occurring holding locks:\nGC barrier waits occur when we wait for threads to run the check point function\non themselves. This is happening with the heap bitmap and mutator lock held\nmeaning that a thread that tries to take either lock exclusively will block\nwaiting on a thread that is waiting. If this thread is the thread we\u0027re waiting\nto run the check point then the VM will deadlock.\nThis deadlock occurred unnoticed as the call to check for wait safety was\nremoved in: https://googleplex-android-review.googlesource.com/#/c/249423/1.\n\nNewTimingLogger:\nExisting timing log states when a split ends but not when it begins. This isn\u0027t\ngood for systrace, in the context of GC it means that races between mutators\nand the GC are hard to discover what phase the GC is in, we know what phase it\njust finished and derive but that\u0027s not ideal.\n\nSupport for only 1 discontinuous space:\nCode special cases continuous and large object space, rather than assuming we\ncan have a collection of both.\n\nSorted atomic stacks:\nUsed to improve verification performance. Simplify their use and add extra\nchecks.\n\nSimplify mod-union table abstractions.\n\nReduce use of std::strings and their associated overhead in hot code.\n\nMake time units of fields explicit.\n\nReduce confusion that IsAllocSpace is really IsDlMallocSpace.\n\nMake GetTotalMemory (exposed via System) equal to the footprint (as in Dalvik)\nrather than the max memory footprint.\n\nChange-Id: Ie87067140fa4499b15edab691fe6565d79599812\n" }, { "commit": "33e9566255c426e7a2c8fca5b8a1b6a94a5d352c", "tree": "80965d3d329c09ec5a574863a356562e42f639ff", "parents": [ "34d64cba998cb86867109db351113e6f7c9293f7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 20 20:29:14 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 20 21:16:03 2013 -0700" }, "message": "Various GCC 3.6 and clang build fixes.\n\nRemove #error in common_test.h that fires with clang build and replace with\nruntime error.\nFix bit rot caused by not compiling with Wthread-safety.\nFix clang build issues in compiler relating to missing header file definitions\nin object files.\nOther minor build and tidying issues.\n\nChange-Id: Ife829ab0664581936155be524de46e6181c750b0\n" }, { "commit": "cfa9cfa3a94b33d300e1b08b48b3cb72de343dbd", "tree": "0367ee9bfed2d80ca750606ee491e597f67d8043", "parents": [ "ee9889502a34a08741a6f8ecc02917202de9d773" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Apr 16 16:52:01 2013 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Apr 16 16:52:01 2013 -0700" }, "message": "Fix JDWP log spam.\n\nIf you have a breakpoint that logs an expression, this spam makes\nthe output unreadable.\n\nBug: 8191824\nChange-Id: I0bbb9ac3a6165cbbd770adf75139f87079f87410\n" }, { "commit": "0920163b0ee4ce3fbf57db5506659de14b77be75", "tree": "b098097d5e8477a200d158f92c1153f12c6f6d55", "parents": [ "449db33fafa29578df60e8a323f78d5eb6247e76" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Apr 15 15:50:07 2013 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Apr 15 15:50:07 2013 -0700" }, "message": "Fix ObjectReference.InvokeMethod.\n\nThis probably broke when I rewrote the object registry, but because\nthe test was so crap, we may have gotten away with passing bad pointers.\n(Though for me, CheckJNI was catching this.)\n\nWhile I\u0027m here, fix the argument checking, which was previously\nvery weak; we\u0027d accept any reference type rather than instances of\nthe parameter\u0027s specific type.\n\nChange-Id: I08c001cabde02a0509fe28df17523a2d2519d1ca\n" }, { "commit": "449db33fafa29578df60e8a323f78d5eb6247e76", "tree": "a868b59dc78593b140d65239b087b77f59bdebed", "parents": [ "ed1790e83352e54420018d1495eb010f7cd48a64" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Apr 12 18:30:52 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Apr 12 18:35:01 2013 -0700" }, "message": "For JDWP, suspend thread before configuring it for single stepping.\n\nChange-Id: I05a7c28c9e977885195797a78a492aa0f72801b7\n" }, { "commit": "14dd5a82ad85a28612bf86e755ff929d767cea4c", "tree": "59643b02b8931887e6b00589e41af137ee13a149", "parents": [ "4b4d711852982716ac478288f210bb2a2f504589" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Apr 11 10:23:36 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Apr 11 11:22:36 2013 -0700" }, "message": "Fix exception caught listeners and frame type for deopt.\n\nChange-Id: I5792ebe9a7772d068b6ec556b4fc27ddd44d7ca1\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": "6474d190d5604898354ebf767f1944b6e3e9b445", "tree": "ac4be675c62fd8129fe85f740dcd2e3bfee424e2", "parents": [ "52003fe9e2020e9d68cee1300fdb4a3f55be85a5" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Mar 26 14:08:09 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Mar 26 16:03:53 2013 -0700" }, "message": "Remove use of compiled invoke stubs from portable.\n\nNow the invoke stubs can be safely removed. Tested and working on ARM,\nbasic testing done on x86/MIPS, but portable is currently broken for them\neven without the change.\n\nChange-Id: Ib73f2b7aa9d81f5f0e1e817d16b9bec464c5a5aa\n" }, { "commit": "4f6ad8ab428038129b2d0d6c40b7fd625cca15e1", "tree": "d1025ed9203d0cdf7aa959d9cecd8dc1f0c0bfeb", "parents": [ "aabe8adacc7bf3a8ab6f6f3dd37fc839b6fd762f" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 18 15:27:28 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 18 15:29:59 2013 -0700" }, "message": "Various performance improvements.\n\nPerformance had regressed due to verify object and method invocation changes.\nAvoid trampolines for static calls in same class.\nVarious inlining changes.\nMake verify object something that\u0027s only compiled-in in debug builds.\n\nChange-Id: Ia261a52232c3b10667c668f8adfadc0da3048bc5\n" }, { "commit": "5d9173014c1ca09f7249a6b07629aa37778b5f8f", "tree": "80543dbe9af6795212373bb12532b638ff3b22f3", "parents": [ "aed0716b9592bb3095cdfc4b111011f9ed74877d" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Feb 27 17:57:33 2013 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 15 14:18:28 2013 -0700" }, "message": "Changes to remove need for compiled invoke stubs for quick.\n\nARM, x86, and MIPS implementation complete, though MIPS is untested.\n\nThe ArgArray is changed to be a uint32_t array instead of a JValue array.\nAlso, a separate result for float/double was needed for x86/MIPS. The invoke\nstubs are currently still there, but only used for portable.\n\nChange-Id: I0647f8d5d420cea61370e662e85bdc0c13b5e378\n" }, { "commit": "4b9702c6912c6f8745a77f5b5af56e7fe196e7c2", "tree": "02fee3631b7f3b60eb15f791b5dd004b27d6a571", "parents": [ "64f574f474aa77c72778640ab21f8cfa72546812" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Feb 20 18:13:24 2013 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Feb 21 07:41:59 2013 -0800" }, "message": "Add a JDWP::Request type to replace the old uint8_t* and int.\n\nThis also lets us check that all the data in a (successful) request\nis actually read, though doing so caught two bugs in the tests, and\nmay well catch bugs in the actual debuggers.\n\nChange-Id: Ibed402e6f1c0c7a1d19d61f0be9bddd0c2f5a388\n" }, { "commit": "64f574f474aa77c72778640ab21f8cfa72546812", "tree": "c90d3ef8f56144e61a4a043138a5118c3c9e7b9a", "parents": [ "492899e445c21f09db458c76d773d571cc6cc0ac" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Feb 20 14:57:12 2013 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Feb 20 15:46:06 2013 -0800" }, "message": "Implement EnableCollection, DisableCollection, IsCollected, and fix Exit.\n\nRewrites the object registry to not just be one big leak. We use jweaks\nby default, which means (a) the GC doesn\u0027t need to know anything about us\nand (b) we don\u0027t cause unnecessary heap retention. We promote to regular\nJNI global references if the debugger asks us to.\n\nThe problem with VirtualMachine.Exit turned out to be that you\u0027re supposed\nto send a success reply to the command before exiting. This is a bit awkward\nwith our current division of responsibilities, but it lets us pass another\ntest.\n\nAlso log a summary of our replies when -verbose:jdwp is in effect, not\njust the requests.\n\nChange-Id: Idb33e99fe7d8bee7a79152d81fee42e2af00852b\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": "d636b0623307a379e255a9aaa682c12a2acc3a92", "tree": "ac5a98eab337b23b6f14b750683b6540f8883166", "parents": [ "d75afb99f0c8b8a27ad84e2e31010383ecf72120" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jan 18 17:51:18 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jan 18 17:51:18 2013 -0800" }, "message": "Avoid crash in HeapChunkContext::Flush post Reset.\n\nPort to ART of 22d8dba8ae5be7aecb07322a410712206896fa15.\n\nChange-Id: I74445f2aac22a037533f13aa894ae4f6cea1abc9\n" }, { "commit": "a96836a4115ad08762567c10bd4d198c5b644985", "tree": "ae523d318a1936108b9f137af3681de930628aa1", "parents": [ "fa011c82efce92a8e0e739ae9732b4c6f3903294" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Jan 17 12:27:49 2013 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Jan 17 12:27:49 2013 -0800" }, "message": "Improve -verbose:jdwp.\n\nMove toward automated logging instead of ad hoc logging in every command.\n\nChange-Id: I55427022374390745209677bae4e0b3146a9d126\n" }, { "commit": "9777ba230c83a0edcbda2cf7b208339e77bf171b", "tree": "253559eb1e962c3b21cf55cd59addc10823d65c4", "parents": [ "ff7ae7adf223b3485f4153442e4841189601af1e" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Jan 17 09:04:19 2013 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Jan 17 09:04:19 2013 -0800" }, "message": "Implement Method.Bytecodes.\n\nChange-Id: Ib24c1244818c1d5aa10606a4998c0e52fa2b70bd\n" }, { "commit": "0cbaff584244ee767027aff35cd3c625aaee2994", "tree": "6209981f3814b9d43f8152f261455eb3f1fab15d", "parents": [ "f9a52cd81ad39aacd1a5c41478d5ee0cb6dff1bf" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Jan 16 15:28:01 2013 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Jan 16 16:42:41 2013 -0800" }, "message": "Implement ObjectReference.ReferringObjects.\n\nBug: 4159882\nChange-Id: Iac145715bec80b5900256d3f6bdb18e283eb9a12\n" } ], "next": "3b78c949ab839d21454bc6f18c7640d2ae8c22f3" }