)]}' { "log": [ { "commit": "f048af49e219c4f5568607d12aa91cc7439c9f97", "tree": "b8c8cf72917cb4e9aacfedd626182518d555aae0", "parents": [ "d3038c30ef8436810cd1c2e5265eab00e242e765", "9eba0a74a1fa9aabb7f7078a32c591bc7f0a7168" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Wed Jul 23 23:27:57 2014 +0000" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Wed Jul 23 23:27:57 2014 +0000" }, "message": "am 9eba0a74: am 3879792f: Merge \"Remove 64-bit android_atomic uses from StateQueue.\"\n\n* commit \u00279eba0a74a1fa9aabb7f7078a32c591bc7f0a7168\u0027:\n Remove 64-bit android_atomic uses from StateQueue.\n" }, { "commit": "f39b560952d3706a7ff47ef0d82c1a836daeea42", "tree": "931d10307a6eff848328792bf93eb71bcb665cc8", "parents": [ "7878f087b0f3737a270b242763e97d1a4e059672" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Wed Jul 16 12:13:16 2014 -0700" }, "committer": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Wed Jul 23 16:14:22 2014 -0700" }, "message": "Remove 64-bit android_atomic uses from StateQueue.\n\nUse stdatomic.h instead. We\u0027re trying to remove android_atomic use\nwherever possible. The 64-bit uses seem easiest to remove first.\nThis cleans up the code, though not as much as C++ \u003catomic\u003e would,\nif it worked everywhere.\n\nChange-Id: I3c29bdbd5915cb9d47118834a3a742fe296cf87f\n" }, { "commit": "5b17c0b5b418c340d3e5201a72ee8c88c1755355", "tree": "6a9669a90265a46bba05b89ad35ed9edbc13a8c4", "parents": [ "963698408b1ce0f2ac7eab1e863dc7271de116a3" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue May 13 10:55:33 2014 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue May 13 13:26:33 2014 -0700" }, "message": "Explicitly include Configuration.h\n\nMust include Configuration.h in any source file with #ifdef or #if that\ndepends on Configuration.h. This avoid inconsistencies that could result\nin subtle bugs.\n\nChange-Id: I99fdf19d56e7c73de4e7d672b937336b932a2a00\n" }, { "commit": "fb1fdc9d6603aa228362e7349451f6455c9849c2", "tree": "1caac6c881db81bcfedb596b292ce62ecbaf6a2c", "parents": [ "551ababe67ce31778000d52958e53e1c56e1a294" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jul 10 17:03:19 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jul 10 17:08:15 2013 -0700" }, "message": "Add comments\n\nChange-Id: Ifbf3a46a4183c8abc0feee1c588953ab10303cc1\n" }, { "commit": "7f5d335f7b4caecd0dfb8f1085f352f1d2da5d2e", "tree": "dda17a3e4149815397c7bef198e64bc81dee3552", "parents": [ "32584a7d672864b20ab8b83a3cb23c1858e908b7" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Feb 15 23:55:04 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Fri Feb 15 23:55:04 2013 +0000" }, "message": "Revert \"Temporary additional logging to investigate bug\"\n\nThis reverts commit 32584a7d672864b20ab8b83a3cb23c1858e908b7\n\nChange-Id: I9dc680578b955b1af462eeb7a49d61a0d45eb81b\n" }, { "commit": "32584a7d672864b20ab8b83a3cb23c1858e908b7", "tree": "87a3d8c3b801d13ceee09abab5048aef46e65332", "parents": [ "ab89ac209fd1c3b0a2227168a48d7f3ae9bc43f3" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Feb 13 14:46:45 2013 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Feb 15 15:44:50 2013 -0800" }, "message": "Temporary additional logging to investigate bug\n\nThe bug appears related to continuing to use an invalid buffer provider\nin fast mixer after track destruction, so focus the added logs in that area.\n\nAlso includes a bug fix: was calling log in an unsafe place\nnear Threads.cpp AudioFlinger::PlaybackThread::createTrack_l line 1250.\n\nDetails:\n - include caller pid or client pid where appropriate\n - increase log buffer size\n - log mFastIndex when AudioMixer sees an invalid bufferProvider.\n - log both potentially modified and actually modified tracks in FastMixer.\n - fix benign bug where sq-\u003eend() was called more than once.\n - log StateQueue push() call and return.\n - increase StateQueue size from 4 to 8 entries\n - log mixer-\u003eenable(), bufferProvider, and currentTrackMask\n - log buffer provider addresses\n - increase fast mixer log buffer again\n - check logf format vs. argument list compatibility\n - add logging to AudioMixer\n - add checking of magic field in AudioMixer to detect overwrites\n - add bool AudioMixer::enabled()\n - increase log buffer sizes yet again\n - enable assertion checking without ALOGV\n - improve a few log messages\n - check for corruption in more places\n - log in all the process hooks\n - add new mixer APIs so we can check for corruption of mixer state\n - fix a build warning\n\nBug: 6490974\nChange-Id: Ib0c4a73dcf606ef9bd898313b3b40ef61ab42f51\n" }, { "commit": "e186b51e0a9834b287d7a509e960eaf1b688db75", "tree": "82b017a4c77a82fca62571eccaa3723ee93738d1", "parents": [ "639482c24c911b125398b31883ba6d55faebe28b" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Feb 14 23:57:02 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Feb 14 23:57:02 2013 +0000" }, "message": "Revert \"Temporary additional logging to investigate bug\"\n\nThis reverts commit 639482c24c911b125398b31883ba6d55faebe28b\n\nChange-Id: I11f2829072ab11e18b0663024f27bf31192f1d39\n" }, { "commit": "639482c24c911b125398b31883ba6d55faebe28b", "tree": "698e25baf47d4e6a10663d21ca86c3a0702fe606", "parents": [ "085205bd23c3604c1b792c3046fdae0a489c04be" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Feb 13 14:46:45 2013 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Feb 14 14:35:29 2013 -0800" }, "message": "Temporary additional logging to investigate bug\n\nThe bug appears related to continuing to use an invalid buffer provider\nin fast mixer after track destruction, so focus the added logs in that area.\n\nAlso includes a bug fix: was calling log in an unsafe place\nnear Threads.cpp AudioFlinger::PlaybackThread::createTrack_l line 1250.\n\nDetails:\n - include caller pid or client pid where appropriate\n - increase log buffer size\n - log mFastIndex when AudioMixer sees an invalid bufferProvider.\n - log both potentially modified and actually modified tracks in FastMixer.\n - fix benign bug where sq-\u003eend() was called more than once.\n - log StateQueue push() call and return.\n - increase StateQueue size from 4 to 8 entries\n - log mixer-\u003eenable(), bufferProvider, and currentTrackMask\n - log buffer provider addresses\n - increase fast mixer log buffer again\n - check logf format vs. argument list compatibility\n - add logging to AudioMixer\n - add checking of magic field in AudioMixer to detect overwrites\n - add bool AudioMixer::enabled()\n\nBug: 6490974\nChange-Id: I1f3f18aa62d9fbd35bc32285b669f5ba40efe28e\n" }, { "commit": "22eb4e239fbe9103568147d566d7482e480350b8", "tree": "132fa6ac8c0bc47fd6d3b1cc67d1724dc3b939a2", "parents": [ "2d81f665461409f0c5d0be674a35a0aea63a4fc8" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 07 14:03:00 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 14 11:58:17 2012 -0800" }, "message": "Update audio comments\n\nChange-Id: I85d7d2f6381b251db5695202fec75128883a8662\n" }, { "commit": "2188bc912a56d9bc577fcec7bf2208f49455e744", "tree": "387f53b6882c758186b9832755b51575fa915972", "parents": [ "692c3e54c63aaaf7e9ef4d89761f98975bd34f33" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Oct 26 16:10:30 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Oct 26 16:10:30 2012 -0700" }, "message": "StateQueue comments\n\nChange-Id: Id8c76d1e2ab5201e7ab3875f1dbcc0a126e611a4\n" }, { "commit": "399930859a75d806ce0ef124ac22025ae4ef0549", "tree": "2471b70dedc0bb33f7310de8ae0e5622344c6f82", "parents": [ "1295bb4dcff7b29c75cd23746816df12a871d72c" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu May 31 13:40:27 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Jun 04 08:27:25 2012 -0700" }, "message": "State queue dump\n\nBug: 6591648\nChange-Id: Iac75e5ea64e86640b3d890c46a636641b9733c6d\n" }, { "commit": "dc998c809e084b617990b281e2ed5271830cc2e0", "tree": "7d05323b201206ccb6cecf72d82faf82055afd70", "parents": [ "eee482b84bb456e87fa42e8ad02fd2d803839586" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Mar 23 18:53:59 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Apr 18 07:34:34 2012 -0700" }, "message": "Add template class StateQueue\n\nChange-Id: Iccc5eb42bc295a22b2e429a4551f083cd7b6831a\n" } ] }