)]}' { "log": [ { "commit": "b6dcdbf436ffacb52a41ac8ef9b07f99fd64779f", "tree": "56c8eebf0dda1560737408bf8c8e20019d17193b", "parents": [ "620208dc0bbd7a0792702df3ab08800fdad60cec" ], "author": { "name": "Chris Elliott", "email": "chriselliott@google.com", "time": "Thu Oct 16 02:17:27 2014 +0000" }, "committer": { "name": "Chris Elliott", "email": "chriselliott@google.com", "time": "Thu Oct 16 02:18:57 2014 +0000" }, "message": "Revert \"audio: prevent larger than required sleeps - DO NOT MERGE\"\n\nRevert change if we require a build beyond P\n\nThis reverts commit 620208dc0bbd7a0792702df3ab08800fdad60cec.\n\nChange-Id: I07f65e35dbd9f814725f50f7bb346f7b285f41aa" }, { "commit": "620208dc0bbd7a0792702df3ab08800fdad60cec", "tree": "c079d953bc10a492f8145a3984a929bb9b936254", "parents": [ "2feb18ddfdbd20941ebcc3314780a582da5608e4" ], "author": { "name": "Chris Elliott", "email": "chriselliott@google.com", "time": "Wed Oct 15 20:46:15 2014 +0000" }, "committer": { "name": "Chris Elliott", "email": "chriselliott@google.com", "time": "Wed Oct 15 20:49:02 2014 +0000" }, "message": "audio: prevent larger than required sleeps - DO NOT MERGE\n\nThis will be cherry picked into P build after O completes for other devices.\n\nThis reverts the revert from commit 2feb18ddfdbd20941ebcc3314780a582da5608e4.\n\nb/17962037\n\nFrom: Haynes Mathew George \u003chgeorge@codeaurora.org\u003e\nDate: Wed, 26 Mar 2014 16:18:42 -0700\nSubject: [PATCH] AudioTrack: prevent larger than required sleeps\n\nAudioTrackThread can end up waiting for larger than necessary\ntime for free space to be available in the cblk. Fix this by\nwaiting on the cblk futex instead of the (internal) condition variable.\n\nChange-Id: Iba5b266f9b4082b3833f4abd52ebc6601c8e3034" }, { "commit": "2feb18ddfdbd20941ebcc3314780a582da5608e4", "tree": "56c8eebf0dda1560737408bf8c8e20019d17193b", "parents": [ "8c0417c173d1fd618ae54e2d721b0ec360ea70cc" ], "author": { "name": "Chris Elliott", "email": "chriselliott@google.com", "time": "Wed Oct 15 00:49:17 2014 +0000" }, "committer": { "name": "Chris Elliott", "email": "chriselliott@google.com", "time": "Wed Oct 15 00:49:17 2014 +0000" }, "message": "Revert \"audio: prevent larger than required sleeps - DO NOT MERGE\"\n\nThis reverts commit 8c0417c173d1fd618ae54e2d721b0ec360ea70cc.\n\nReverting as we only wanted to include this CL the KNX01M build\n\nChange-Id: Ib53292de658078285a38710915168e1746486fea\n" }, { "commit": "8c0417c173d1fd618ae54e2d721b0ec360ea70cc", "tree": "c079d953bc10a492f8145a3984a929bb9b936254", "parents": [ "e6b5d4c2d58a02184dd97815b5f76980025379bc" ], "author": { "name": "Haynes Mathew George", "email": "hgeorge@codeaurora.org", "time": "Thu Oct 09 20:06:00 2014 +0800" }, "committer": { "name": "Chris Elliott", "email": "chriselliott@google.com", "time": "Tue Oct 14 17:20:37 2014 +0000" }, "message": "audio: prevent larger than required sleeps - DO NOT MERGE\n\nb/17962037\n\nFrom: Haynes Mathew George \u003chgeorge@codeaurora.org\u003e\nDate: Wed, 26 Mar 2014 16:18:42 -0700\nSubject: [PATCH] AudioTrack: prevent larger than required sleeps\n\nAudioTrackThread can end up waiting for larger than necessary\ntime for free space to be available in the cblk. Fix this by\nwaiting on the cblk futex instead of the (internal) condition variable.\n\nChange-Id: I75bd14cd46f3b75afe0ea8bd9b48d7a2f95f654a" }, { "commit": "6db20dcb54dad932a3ea2e3a5dca41fce5802eb4", "tree": "295ef5012f98e4b9251f28774cfb3f83710ea197", "parents": [ "2f55c2c03b17795e94b325d402ac5b409e3ba0e8", "2048c2292c0466b184fb8f67c91f4d0ab9f5c3f3" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Sat Mar 08 00:29:56 2014 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Sat Mar 08 00:29:56 2014 +0000" }, "message": "Merge \"AudioTrack: When paused, return cached playback position\" into klp-dev" }, { "commit": "dcec903589ac651b9f558454392d286c13f743fb", "tree": "e436514f1a63d4b26dad1c142912411c23e60578", "parents": [ "5bc83fc352e5682ecafe61d77062c26ac8db4229" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Sep 20 09:28:56 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Mar 07 11:09:51 2014 -0800" }, "message": "Simplify AudioTrack stream end and fix race\n\nBug: 10994052\nChange-Id: Ib2e38e7a600bcffef8cbc68c1722e40fbbc7ea67\n" }, { "commit": "2048c2292c0466b184fb8f67c91f4d0ab9f5c3f3", "tree": "126fcbfff298dd8e0f8ddb834c93c8850b8654ac", "parents": [ "a11111f9135c98cf4247bbd1a61d7df3053e549e" ], "author": { "name": "Haynes Mathew George", "email": "hgeorge@codeaurora.org", "time": "Wed Jan 08 13:59:53 2014 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Mar 06 18:00:39 2014 -0800" }, "message": "AudioTrack: When paused, return cached playback position\n\nAn offload output can be re-used between two audio tracks having\nthe same configuration. A timestamp query for a paused track\nwhile the other is running would return an incorrect time.\nTo fix this, cache the playback position on a pause() and return\nthis time when requested until the track is resumed.\n\nBug: 12826612.\nChange-Id: Ia42b8b8fd2ba8993dfcc9abca72da48d71d78d74\n" }, { "commit": "d812fc012298470a1b8120e6d60a24b0b1d48047", "tree": "bf3c68ea3a908a0792e766714cd9d54f98ebfb46", "parents": [ "88876fb4eb8365343f2a9ca0e8f53148ca38b845" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Dec 03 09:06:43 2013 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Mar 05 14:10:41 2014 -0800" }, "message": "Increase kFastTrackMultiplier from 1 to 2\n\nBug: 11967381\nChange-Id: Iedec06280aa745d9df5d661f4916940cede9c191\n" }, { "commit": "88876fb4eb8365343f2a9ca0e8f53148ca38b845", "tree": "7891ad7f6e93d856f4fc8b5184bf2d8c95a84ead", "parents": [ "145b727b4cce8b1b8d161612f30d12bbdfc8d134" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Jan 27 18:13:39 2014 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Mar 05 14:07:43 2014 -0800" }, "message": "AudioTrack: fix obtainBuffer timeout calculation\n\nAudioTrack::obtainBuffer() passes a pointer to a timeout\nvariable that has gone out of scope when calling\nClientProxy::obtainBuffer().\n\nSame fix for AudioRecord.\n\nBug: 11968591.\nChange-Id: I5b3d0d86ed2bd8c0412eb2432b1ab519d70efce9\n" }, { "commit": "d0115d8c4db2f337030dea706bc0a434c4a03ed6", "tree": "239450c60bc4cb8519edeb8985dbc601fe51c652", "parents": [ "7e92abeafb184e8a34213d7149592e95a72601b0" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Jul 26 17:16:50 2013 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Mar 05 12:17:54 2014 -0800" }, "message": "update offloaded audio track sampling rate\n\nAudioPlayer must read the sampling rate from offloaded audio sinks\nwhenever a new time position is computed as the decoder can update\nthe sampling rate on the fly.\n\nBug: 12823955.\n\nChange-Id: I997e5248cfd4017aeceb4e11689324ded2a5bc88\n" }, { "commit": "a4f24ff9b239300b4bb2098c1413c0a60edc2d7e", "tree": "1b328f28835885a21e67780129d6764f5b4384a9", "parents": [ "5ea5b020b311a68f513d4a40aca5551abc6a07d8" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Nov 27 14:29:13 2013 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Nov 27 14:29:13 2013 -0800" }, "message": "AudioTrack: fix position callback after restore\n\nWhen restoring an AudioTrack, the next position callback point\nshould not be modified and set ahead of current buffer head.\nOtherwise, as frames are dropped, the new position is never reached\nand an application relying on position callbacks to reload the buffer\nwould be stalled.\n\nBug: 11868603.\nChange-Id: I93b2a311642a0c89944b78bcc0482d4ceed98ae4\n" }, { "commit": "9cae217050aa1347d4ac5053c305754879e3f97f", "tree": "dff82f277cba8f34af39eaeeb00adec01bbda26c", "parents": [ "be567be3f92026f5964c1c9ecce00e30bb7df412" ], "author": { "name": "Marco Nelissen", "email": "marcone@google.com", "time": "Mon Jan 14 14:12:05 2013 -0800" }, "committer": { "name": "Marco Nelissen", "email": "marcone@google.com", "time": "Fri Oct 25 14:06:03 2013 -0700" }, "message": "Assign blame for playback wakelocks.\n\nSet a work source for the playback wakelock, so that playback is\ncounted against the requesting app instead of the media server.\n\nCherrypicked from master.\n\nb/9464621\n\nChange-Id: I7329f88a288a95a582a78005a1c3d16a5a611e31\n" }, { "commit": "f8f15b05fe051009945c9042a1a9260280e0feb2", "tree": "28fd5dede6b2aae34a9b3b987049f3331ace5037", "parents": [ "787c0d04ac4f9c938a1311f8e30d9d7ac2b8a6da", "598de6c701e989385eeffa7c5dfd61f0459a2631" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Oct 17 00:59:33 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Oct 17 00:59:33 2013 +0000" }, "message": "Merge \"Fix race condition in AudioTrack::pause followed by start\" into klp-dev" }, { "commit": "598de6c701e989385eeffa7c5dfd61f0459a2631", "tree": "090b30e255f0aed57a73ca6c9252e19da162fd97", "parents": [ "014e7fa2e90827d911c37bb0ce4d2e10e14d0bb3" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Oct 16 17:02:13 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Oct 16 17:20:11 2013 -0700" }, "message": "Fix race condition in AudioTrack::pause followed by start\n\nBug: 11148722\nChange-Id: Iec88f00c8510363d4418e4b8d5b34feb06ecf04d\n" }, { "commit": "cc21e4f1e41dfa17e7e2bef995fcd22c45f6bcd0", "tree": "a446f08a1adaf31ef5dddc491c620209176b67aa", "parents": [ "014e7fa2e90827d911c37bb0ce4d2e10e14d0bb3" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Oct 16 15:12:32 2013 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Oct 16 15:12:32 2013 -0700" }, "message": "AudioTrack: fix head position after restore\n\nThe head position transfered to the new track\nby restoreTrack_l() must take into account the frames that\nare dropped from the old track to avoid a non recoverable\noffset in the playback head position returned to applications.\n\nBug: 11230062.\nChange-Id: I51143a08b95e8f264ed709ae2054360315f2b8b1\n" }, { "commit": "59fe010bcc072597852454a2ec53d7b0a2002a3b", "tree": "b6b42e04a89ead77c9ecd9635a328686a8bb9863", "parents": [ "5d6d86a4d102704f49b9235eaf282c428d7100b6" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Sep 27 18:48:26 2013 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Sep 30 12:05:38 2013 -0700" }, "message": "fix volume and effect enable delay on offloaded tracks\n\nVolume: add a method to wake up the mediaserver playback\nthread when a volume command is received on an offloaded track.\n\nEffects: call effect chain process on offloaded playback threads\nasynchronously from writes to allow effect state updates while\nwaiting for async write callback.\n\nBug: 10796540.\n\nChange-Id: Id2747ae88783575d1d7ffd6fc86fbd054ab2c739\n" }, { "commit": "9d2c78c4798ffd8c276c1bf0eaa0b34bc255a2da", "tree": "f81f585ba1aae36086bb9790b83de5673fdce14a", "parents": [ "4b701cc041d635e5ec56e382043a4c5d01aedd80" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Sep 23 12:29:42 2013 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Sep 23 12:32:34 2013 -0700" }, "message": "AudioTrack: fix music resume\n\nFix regression introduced by commit 5a6cd22 in AudioTrack resume:\nthe callback thread was not signaled if paused internaly.\n\nBug: 10895013.\nChange-Id: Ic356b115132d6fccbcee2d9bb855e92671dc20c5\n" }, { "commit": "5a6cd224d07c05b496b6aca050ce5ecf96f125af", "tree": "c5d2fe55f0080c9660d283d040616d9a9fb6decc", "parents": [ "a3b97ad288e6808c54c6275eaac1bd53108d8f11" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Sep 20 09:20:45 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Sep 20 12:22:29 2013 -0700" }, "message": "Fix slow AudioTrack and AudioRecord destruction\n\nThere were two causes for the slowness:\n\nWhen thread was paused, it used nanosleep and sleep. These usually\nrun to completion (except for POSIX signal, which we avoid because it\nis low-level). Instead, replace the nanosleep and sleep by condition\ntimed wait, as that can be made to return early by a condition signal.\nAnother advantage of condition timed wait is that a condition wait was\nalready being used at top of thread loop, so it is a simpler change.\n\nThe AudioRecord destructor was missing a proxy interrupt that was correct\nin AudioTrack. This proxy interrupt is needed in case another thread\nis blocked in proxy obtainBuffer.\n\nDoes not address the 1 second polling for NS_WHENEVER.\n\nBug: 10822765\nChange-Id: Id665994551e87e4d7da9c7b015f424fd7a0b5560\n" }, { "commit": "ce8828a016b082f730152af2204b8ea3610dc1ec", "tree": "aad0c32a17516519b22bcea0d4f2fa7c81c61d28", "parents": [ "d176ec1e371a686c58385692a28da677e398c302" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Sep 16 18:07:38 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Sep 17 11:24:02 2013 -0700" }, "message": "Fix underruns when fast track denied due to SRC\n\nOpenSL ES requests a fast track. If sample rate conversion is needed,\nthe request is denied by server, and a larger client buffer is used\nto handle the higher latency of a normal track. However the client\nnotification period was calculated based on buffer being divided into\n2 sub-buffers. That resulted in the notification period being too long.\nThe server pulls chunks that are smaller than half the total buffer.\nSo now the client uses 3 sub-buffers when there is SRC.\n\nAlso removed the \u0027defer wake\u0027 optimization because it was incorrect.\nThis optimization attempted to reduce the number of wakeups of client,\nwhen server releaseBuffer knows that another releaseBuffer will be\nfollowing. But there is no way for the first releaseBuffer to predict\nhow soon the second releaseBuffer will occur. In some cases it was\na long time, and the client underran. So now the client is woken up\nimmediately if the total number of available frames to client is \u003e\u003d\nthe minimum number the client wants to see (the notification period).\n\nAlso fix bug where minimum frame count was not being used in the\ncalculation of notification period.\n\nBug: 10342804\nChange-Id: I3c246f4e7bc3684a344f2cf08268dc082e338e2a\n" }, { "commit": "fe346c707f59d763ded93bc3d27b51f0c0408258", "tree": "3e6f12ac7b37bc8ef10845c4c7ccdcfef4b51cd5", "parents": [ "4d0815d694e5a2edb3ce48427de50f55d0f84c0b" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Aug 30 13:28:22 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Sep 03 15:19:03 2013 -0700" }, "message": "Fix miscellanous AudioTrack::getTimestamp() bugs\n\nCheck that get_presentation_position is non-NULL before calling.\n\nAudioTrack::getTimestamp not implemented for fast tracks.\n\nFix typo in Track::getTimestamp().\n\nFix bugs in AudioTrack::getTimestamp after stop:\n - getTimestamp while stopped is not allowed.\n - stop, start, getTimestamp now returns the correct value.\n\nChange-Id: Ie8d9dc1f28d8927634e04175a68b147ffc2ea8eb\n" }, { "commit": "56b592240073b1ab116c6e23a9c4c34ef5ad80d8", "tree": "99d167994fa92857f39b459da1b52e0695d13a68", "parents": [ "865f6f248bf10189272020ebfdbf4827433caf0f", "53cec22821072719ee02c856e9ac2dda2496c570" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Aug 30 00:04:37 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Fri Aug 30 00:04:37 2013 +0000" }, "message": "Merge \"Add IAudioTrack::getTimestamp()\" into klp-dev" }, { "commit": "865f6f248bf10189272020ebfdbf4827433caf0f", "tree": "02c8e8e58c3b3e549c718bda3bd344f332682ce9", "parents": [ "e3b8b6e3e3a78ec6f9978f56c4573e193866ef53", "ce70374bf105f8a6160bf5dd70affea012b2a464" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Aug 29 23:47:34 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Aug 29 23:47:34 2013 +0000" }, "message": "Merge \"New AudioTrack C++ API for audio timestamps\" into klp-dev" }, { "commit": "42a6f422c09ca6a960673e0e805ddf71a9b51bef", "tree": "7c38dd1f569bd9546f5cef3c5f58ff6ca35cbeff", "parents": [ "bdd467edb8550ab006c89a59b556759386e99898" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Aug 29 14:35:05 2013 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Aug 29 14:35:05 2013 -0700" }, "message": "AudioTrack: fix write retries for compressed audio\n\nWhen the amount of frames that can be written to the buffer\nis less than requested, AudioTrack::processAudioBuffer() estimates\nthe time needed to free the missing amount of frames and asks the\ncallback thread to sleep.\nThis behavior is not possible for compressed audio and should not\nbe enabled for offloaded tracks.\n\nChange-Id: I5b657283cfba06254c9ac0ea9b447467cce7eb61\n" }, { "commit": "53cec22821072719ee02c856e9ac2dda2496c570", "tree": "1ac72eaf3ba6059680f8c3d7e39ae58a9e03adc8", "parents": [ "ce70374bf105f8a6160bf5dd70affea012b2a464" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Aug 29 09:01:02 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Aug 29 09:19:07 2013 -0700" }, "message": "Add IAudioTrack::getTimestamp()\n\nwith dummy implementation in AudioFlinger::TrackHandle, and\nimplement AudioTrack::getTimestamp() using IAudioTrack.\n\nAlso document invariant that mAudioTrack and control block are always\nnon-0 after successful initialization.\n\nChange-Id: I9861d1454cff7decf795d5d5898ac7999a9f3b7e\n" }, { "commit": "ce70374bf105f8a6160bf5dd70affea012b2a464", "tree": "60d2673b83bff3622e7e299974104e572656b5a6", "parents": [ "e07f53720bcf543bcfe90c228a49c2a85a3fdb4c" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jul 19 16:33:58 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Aug 28 17:24:58 2013 -0700" }, "message": "New AudioTrack C++ API for audio timestamps\n\nThis new API is intended to replace latency(), especially for A/V sync.\nThe application will receive periodic timestamp notifications. The period\nis unspecified, but will likely be more frequent after a pause or stop,\nset position, underrun, display on/off change, route change, or when audio\nframework notices drift. It will be up to the higher level application\n(e.g. Stagefright) to reconstruct a clock that updates more frequently.\n\nThe current latency() method doesn\u0027t indicate when latency changes\ndue to screen on/off state, route changes, etc.\n\nIncludes squahsed change-Id: I2082f8752040be0c234b1a6f1be2e269abf2ce7c\n Dummy implementation of AudioTrack:getTimestamp()\n Rename AudioTrack::Timestamp to AudioTimestamp.\n Renaming and pulling up to a higher level allows more modules to use it.\n\nChange-Id: Ibf7f6a207c3f8d8697f25ede2cd5200697fadb86\n(cherry picked from commit dd69eb893867634fd169c03204a6ad7c74b351e7)\n" }, { "commit": "ec9a032c2e104ab1e3b5bf73e69dab1408ced0ad", "tree": "de4b71e9f43dc9a3a46aec43bc08f910ad5b8127", "parents": [ "3bd3690fb0a423f9d0ba635e8db1c148ac9e93fc" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Aug 28 10:23:01 2013 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Aug 28 10:30:08 2013 -0700" }, "message": "AudioTrack: fix unwanted underrun when restarting\n\nWhen restarting an AudioTrack from stopped state, it is necessary\nto force refresh of mRemainingFrames by processAudioBuffer() as\nthe last write before stop() could be partial. No doing so will lead\ninto unnecessary sleep before filling the non contiguous part\nof the buffer returned by obtainBuffer() when processAudioBuffer()\nis executed for the first time after start().\n\nChange-Id: Id703f8dc092a6f07c905eee194054b4a978f979d\n" }, { "commit": "d054c32443a493513ab63529b0c8b1aca290278c", "tree": "fc68ba259ded1dd4960bee2938a9bcc7c599c9a3", "parents": [ "96f60d8f04432a1ed503b3e24d5736d28c63c9a2" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jul 12 12:59:20 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jul 30 09:50:48 2013 -0700" }, "message": "Move control block mName to createTrack() output\n\nThis is part of a series of CLs to clean up the shared memory\ncontrol block, by removing any fields that don\u0027t have to be there.\n\nChange-Id: I6e51003a1293b6800258c31b22cff2eba42162e7\n" }, { "commit": "96f60d8f04432a1ed503b3e24d5736d28c63c9a2", "tree": "083f6224327e0965a925076b3e1aec5636598203", "parents": [ "97252bb0e40a5a9b41f020163393cf6c5da10b7e" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jul 12 10:21:18 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jul 30 09:40:17 2013 -0700" }, "message": "Rename control block flags to mFlags\n\nChange-Id: I7b6d31e24531954ab1ecdf3ed56c19433700bd89\n" }, { "commit": "b1a270d1e926fb9a01b4265a7675ed0c2c8f4868", "tree": "091d06b02bc126ff1bb2dd1fabc9be829582ae69", "parents": [ "bfb1b832079bbb9426f72f3863199a54aefd02da" ], "author": { "name": "Richard Fitzgerald", "email": "rf@opensource.wolfsonmicro.com", "time": "Tue May 14 12:12:21 2013 +0100" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Jul 25 14:08:09 2013 -0700" }, "message": "libmedia: offloaded playback support\n\n- start() returns a status so that upper layers can\nrecreate a non offloaded track in case of error.\n\n- Added states to handle offloaded tracks specific:\n - waiting for stream end (drain) notification by\naudio flinger\n - allow pause while waiting for stream end notification\n\n- getPosition() queries the render position directly from\naudio HAL.\n\n- disable APIs not applicable to offloaded tracks\n\n- Modified track restoring behavior for invalidated\noffloaded tracks: just send the callback and wait for\nupper layers to create a new track.\n\n- Added wait for stream end management in audio track client\nproxy. Similar to obtainBuffer and should be factored in.\n\nChange-Id: I0fc48117946364cb255afd653195498891f622bd\nSigned-off-by: Eric Laurent \u003celaurent@google.com\u003e\n" }, { "commit": "cff1b2359b8a302f19539c0bc5d25c98a52733fd", "tree": "e0d7b7b64f4b3a9cf3eb7d2158de59efb4bd0b3a", "parents": [ "bb849b5d1a0364b69aa15ed00709edf4b7e348fc", "fb1fdc9d6603aa228362e7349451f6455c9849c2" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jul 17 14:44:50 2013 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Wed Jul 17 14:44:51 2013 +0000" }, "message": "Merge \"Add comments\"" }, { "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": "b0dfd4613225a3b2a17bdf8d85e89a4b04d65ef3", "tree": "f732881ddedfd52ce6f3b14820e4bf1e7bd9bba9", "parents": [ "551ababe67ce31778000d52958e53e1c56e1a294" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jul 10 16:52:47 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jul 10 16:52:47 2013 -0700" }, "message": "Fix type error in AudioTrack::processAudioBuffer\n\nIt returned a bool instead of nsecs_t\n\nChange-Id: If0c096dac411afc0a4142ec1e59c1fdd36d4867c\n" }, { "commit": "1ab85ec401801ef9a9184650d0f5a1639b45eeb9", "tree": "c3a39f141461642c3ac3c7f1370ca679d08d80e6", "parents": [ "8f13428cb29a9b610a9e9a0f5fea437cd2d3f032" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri May 31 09:18:43 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jul 02 13:43:24 2013 -0700" }, "message": "Include what is needed\n\nRemove old includes.\nHeader files only include other header files that they directly need themselves.\n\nChange-Id: Ic471386808d9f42ea19ccbd59cb50a5f83a89dd0\n" }, { "commit": "ad3af3305f024bcbbd55c894a4995e449498e1ba", "tree": "2e242d4c49cce9faefc28665c6ee63a2a5da170c", "parents": [ "7919fa2c33b1fa7f5e49b2188d671bfe519c231e" ], "author": { "name": "Richard Fitzgerald", "email": "rf@opensource.wolfsonmicro.com", "time": "Mon Mar 25 16:54:37 2013 +0000" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Jun 27 17:16:24 2013 -0700" }, "message": "Public API changes for audio offload support.\n\nNOTE: this does _not_ include all private member variables added\nto classes as part of offload support. Only public/protected functions\nand stubs functions/variables needed to make the changes buildable.\n\n- isOffloadSupported() added to audio policy service\n A stub implementation is required to build, this always returns false\n- setParameters() added to IAudioTrack\n A stub implementation is required to build, this always returns\n INVALID_OPERATION\n\n- CBlk flag for stream end\n\n- Change AudioSystem::getRenderPosition() to take an audio_output_t\n so caller can specify which output to query\n\n- Add AudioSystem::isOffloadSupported()\n This is fully implemented down to the AudioFlinger function\n AudioPolicyServer::isOffloadSupported() which is just a stub\n that always returns false.\n\n- Add EVENT_STREAM_END to AudioTrack interface.\n STREAM_END is used to signal when the hardware has actually finished\n playing all the data it was sent.\n\n- Add event type enumeration to media player interface AudioSink callbacks\n so that the same callback can be used to handle multiple types of\n event. For offloaded tracks we also have to handle STREAM_END and\n TEAR_DOWN events\n\n- Pass audio_offload_info_t to various functions used for opening outputs,\n tracks and audio players. This passes additional information about the\n compressed stream down to the HAL when using offload.\n For publicly-available APIs this is an optional parameter (for some of\n the internal and low-level APIs around the HAL interface it is mandatory)\n\n- Add getParameters() and setParameters() API to AudioTrack\n Currently dummy implementations.\n\n- Change AudioPlayer contructor so that it takes a set of bitflags defining what\n options are required. This replaces the original bool which only specified\n whether to use deep buffering.\n\n- Changes to StageFright class definition related to handling tearing-down of\n an offloaded track when we need to switch back to software decode\n\n- Define new StageFright utility functions used for offloaded tracks\n Currently dummy implementations.\n\n- AudioFlinger changes to use extended audio_config_t.\n Fills in audio_offload_info_t member if this info is passed in when\n opening an output.\n\n- libvideoeditor changes required to add the new event type parameter\n to AudioSink callback functions\n\n- libmediaplayerservice changes required to add the new event type parameter\n to AudioSink callback functions\n\nChange-Id: I3ab41138aa1083d81fe83b886a9b1021ec7320f1\nSigned-off-by: Richard Fitzgerald \u003crf@opensource.wolfsonmicro.com\u003e\nSigned-off-by: Eric Laurent \u003celaurent@google.com\u003e\n" }, { "commit": "9f80dd223d83d9bb9077fb6baee056cee4eaf7e5", "tree": "79f19f80ebb7978e9bab16e1d9b835849c735592", "parents": [ "9fef8d453b15a91a2b748faac2bfaff713bcf1e1" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Dec 18 15:57:32 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jun 12 14:33:10 2013 -0700" }, "message": "New control block for AudioTrack and AudioRecord\n\nMain differences between old and new control block:\n - removes the mutex, which was a potential source of priority inversion\n - circular indices into shared buffer, which is now always a power-of-2 size\n\nChange-Id: I4e9b7fa99858b488ac98a441fa70e31dbba1b865\n" }, { "commit": "bb6f0a0bb413817d707cfb4c4a30847fda520787", "tree": "4fa6837f6302ae59367583d52565e70f3479c140", "parents": [ "587f8425a9522b2d0fe1ae0d211a259ec83786d1" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Jun 03 15:00:29 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jun 04 16:42:58 2013 -0700" }, "message": "Fix underruns when sample rate !\u003d native sample rate\n\nThis forces a minimum of 3 application buffers when the sample rates\ndon\u0027t match, using the normal mixer and low latency HAL.\n\nThere is still an issue that the latency() varies depending on whether\nscreen was off or on at the time of creating the AudioTrack.\n\nWith screen on:\nI/AudioTrack( 2028): afFrameCount\u003d960, minBufCount\u003d2, afSampleRate\u003d48000, afLatency\u003d50\nI/AudioTrack( 2028): minFrameCount: 2646, afFrameCount\u003d960, minBufCount\u003d3, sampleRate\u003d44100, afSampleRate\u003d48000, afLatency\u003d50\n\nWith screen off:\nI/AudioTrack( 2817): afFrameCount\u003d960, minBufCount\u003d4, afSampleRate\u003d48000, afLatency\u003d84\nI/AudioTrack( 2817): minFrameCount: 3528, afFrameCount\u003d960, minBufCount\u003d4, sampleRate\u003d44100, afSampleRate\u003d48000, afLatency\u003d84\n\nChange-Id: Ib45515edff2afcd672dda34881b658c800ffc25a\n" }, { "commit": "7c027248e1a4ccd5b22bc4deafb03e2d87ac8f38", "tree": "ed6b92a79faa533f7c1156836f00b9dcaec8834d", "parents": [ "8bae43ae86b9942892980c3c01bff5c812b1db7f" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Dec 26 14:43:16 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu May 30 13:40:56 2013 -0700" }, "message": "Consistent whitespace\n\nChange-Id: I118cce68d3b777f9ec9b6bfb70367496422a40f2\n" }, { "commit": "61aa989a5768b9b0571ca14edaf15886509a0ec8", "tree": "e272f0ac691569a8c5a54ecb5910f277afe369ca", "parents": [ "b0864b6a20dc2e72bd6d08ad3c06f0fa5cef92e0", "1641f328d4de3b2b61481607dc482ccaefe808a9" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri May 03 14:54:07 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Fri May 03 14:54:07 2013 -0700" }, "message": "am 1641f328: am 4170f515: Merge \"Remove timing jitter during startup of audio\" into jb-mr2-dev\n\n* commit \u00271641f328d4de3b2b61481607dc482ccaefe808a9\u0027:\n Remove timing jitter during startup of audio\n" }, { "commit": "a07f17ca46db04c9d5d9e7d6b2878db59ca2b9ea", "tree": "5f35039b1b7ed7518280e9a436536600a5aac5c9", "parents": [ "5e43772296d7c5b8faf4b0ce4efe68778729bc74" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Apr 23 12:39:37 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Apr 23 14:18:18 2013 -0700" }, "message": "Remove timing jitter during startup of audio\n\nThis fixes a regression introduced recently, that increased timing jitter\nduring the startup of the FastMixer and AudioTrack callback threads.\n\nThe regression was to make requestPriority() asynchronous as a way to\navoid an apparent priority inversion in system_server. This means that\nthe target thread could run briefly with the initial priority, before\nthe new priority takes effect.\n\nThis change removes the startup jitter for FastMixer, by making the\nrequestPriority() synchronous again for that case. It doesn\u0027t matter\nthat this restores the priority inversion involving normal mixer thread,\nbecause it happens during startup of both threads.\n\nThe change also removes the startup jitter for the AudioTrack callback\nthread, by having the target thread check whether the requestPriority()\nhas completed yet. If not, the target thread blocks with a timeout\nuntil the priority boost finishes.\n\nFinally, we now log an error message if the expected priority boost\ndoesn\u0027t happen.\n\nBug: 8698989\nChange-Id: Id590e9a274b70ec1ba85b44a585ee37a22e41cbc\n" }, { "commit": "9c6745f128648f6e0144b74ee593911a9fa10d51", "tree": "04921173242fd90eec6f5d6589d0e7c2edc53c4d", "parents": [ "f78f2629f5f6f5fc37a2089a85107fed97f12192" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 30 13:35:29 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Jan 07 14:24:21 2013 -0800" }, "message": "Propose new interpretation for setPosition and setLoop\n\nAdd new API getBufferPosition to return position relative\nto start of fixed buffer.\n\nChange-Id: I7aca8e392d45b988545f07b36b5032691057b03e\n" }, { "commit": "d5681bc9a38fe4cd1d591e6ae62b9c68fb851041", "tree": "88d51865d4e4f98d05408975762b85894050e6b7", "parents": [ "49d24d3fcd01aef0ec1fa4abd9c1b85473f64c0f", "e3aa659e9cee7df5c12a80d285cc29ab3b2cbb39" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Dec 21 15:04:30 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Fri Dec 21 15:04:30 2012 -0800" }, "message": "Merge \"Start isolating control block accesses in a proxy\"" }, { "commit": "8d6cc842e8d525405c68e57fdf3bc5da0b4d7e87", "tree": "d3296fc4f886ae058cb353cec79576b7e4428897", "parents": [ "53bc0a2fc54330534330118edd5c393f39eff8ab" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Feb 03 11:06:53 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Dec 18 16:16:30 2012 -0800" }, "message": "Remove unnecessary parameter\n\nJust get the parameter on server side\n\nChange-Id: I433a63104dbb257e0d862be2ab61847cb36d1c15\n" }, { "commit": "e3aa659e9cee7df5c12a80d285cc29ab3b2cbb39", "tree": "90fdb6e9da1a03f4f24691aae27ae94ff913f1c7", "parents": [ "84b6440fa1d1274267deabe0bd8dc626b5548e4c" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Dec 04 12:22:46 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Dec 12 12:21:59 2012 -0800" }, "message": "Start isolating control block accesses in a proxy\n\nThe proxy object will eventually be the only code that understands the\ndetails of the control block. This should make it easier to change the\ncontrol block in the future.\n\nInitial set of control block fields that are isolated:\n - sample rate\n - send level\n - volume\n\nPrepare for streaming/static separation by adding a union to the control\nblock for the new fields.\n\nFix bug in handling of max sample rate on a track. It was only checking\nat re-configuration, not at each mix.\n\nSimplify OutputTrack::obtainBuffer.\n\nChange-Id: I2249f9d04f73a911a922ad1d7f6197292c74cd92\n" }, { "commit": "a8190fc518b6769257896605f3aee091aeb60b50", "tree": "6b235739b3e020180516761cb19f2163cacdb094", "parents": [ "083d1c1492d496960d5b28f4664ff02101736677" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Dec 03 17:06:56 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Dec 03 17:11:19 2012 -0800" }, "message": "Split off the current control block to separate file\n\nPrepare for a new implementation of step() etc.\n\nChange-Id: I268421976ba577aa1fb5d7015de5441c05861190\n" }, { "commit": "083d1c1492d496960d5b28f4664ff02101736677", "tree": "5640adaecd5293c1088bf6bf3475b4887f4f9cb2", "parents": [ "4bae3649d504d590a546717a8e49f96a30d9a745" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 30 15:00:36 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Dec 03 11:38:57 2012 -0800" }, "message": "Emphasize distinction between streaming and static\n\nUpdate comments and improve error checks to match\n\nChange-Id: I7370d6e59a7ef26dfb284a8b058d5ab2e0a42ccf\n" }, { "commit": "4bae3649d504d590a546717a8e49f96a30d9a745", "tree": "0e9cc2e5390671a4021d85193b1ad31544040f3c", "parents": [ "e4756fe3a387615acb63c6a05788c8db9b5786cb" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 30 13:41:12 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Dec 03 10:42:52 2012 -0800" }, "message": "flush() comments and checks\n\nflush() is only useful for streaming mode.\nIt is a no-op if track is active or uses a static buffer.\n\nChange-Id: I918ac181ffae3d16a0d67d8a7208f4aec61b5bd6\n" }, { "commit": "e4756fe3a387615acb63c6a05788c8db9b5786cb", "tree": "7e0b0278f71c7667784612a83af8f43ff08bf708", "parents": [ "391f1bb93d5720bfe1a025307178b3208a3ce7d7" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 29 13:38:14 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Dec 03 09:57:37 2012 -0800" }, "message": "AudioTrack::mute() is unused so remove it\n\nIf ever needed again, it could be implemented on client side by forcing\na track volume of 0.\n\nChange-Id: I88a9b4f675b6dca2948549414f9ec2c192d29269\n" }, { "commit": "01437b7cdaecf53acb46b50ff8b5d86b9d36eb20", "tree": "da1b17ed3bd02368e33b0c83c872db25f78bd36c", "parents": [ "24ee6aaf1368e3a07061948273c629d05cb9b12a" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 29 07:32:49 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 30 15:23:44 2012 -0800" }, "message": "AudioTrack inline short const methods\n\nChange-Id: I142917edb454d510bbe545e94e6eaea30b650fae\n" }, { "commit": "a42ff007a17d63df22c60dd5e5fd811ee45ca1b3", "tree": "58ba8c1bf24f27680d48b6040132a02bfaedc0ff", "parents": [ "aa25c0321bd4b9eb46a4d1cb4ac5fa47af30eeb4" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 14 12:47:55 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 19 11:25:58 2012 -0800" }, "message": "Clean up channel count and channel mask\n\nChannel count is uint32_t.\nRemove redundant mask parameter to AudioTrack::createTrack_l()\n and AudioRecord::openRecord_l().\n\nChange-Id: I5dc2b18eb609b2c0dc3091994cbaa4628062c17f\n" }, { "commit": "aa25c0321bd4b9eb46a4d1cb4ac5fa47af30eeb4", "tree": "efd13606a807054398ad8e3ab12981fb85a97a82", "parents": [ "77035d10a740914313500811b31a90ab948bd267", "d65d73c4ae74d084751b417615a78cbe7a51372a" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 16 15:47:51 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Fri Nov 16 15:47:52 2012 -0800" }, "message": "Merge \"\"if\" statements use curly braces per media style\"" }, { "commit": "67ba2b6b8018ce7556cdd880dbf28d06a92ebb78", "tree": "085267f87c37752d3b6b4d79956f03ca74005df5", "parents": [ "e65d2797bc33897e1f61b702cf182e801b33990a", "b603744e96b07b1d5bf745bde593fb2c025cefcf" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 16 15:23:42 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Fri Nov 16 15:23:42 2012 -0800" }, "message": "Merge \"Don\u0027t use control block frame count after create\"" }, { "commit": "b603744e96b07b1d5bf745bde593fb2c025cefcf", "tree": "0c8ae15dbd430ccfcb872c701de48907ec48afc8", "parents": [ "f91a3abb7d136e75f0f5d999193b8c8297d97628" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 14 13:42:25 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 16 15:01:47 2012 -0800" }, "message": "Don\u0027t use control block frame count after create\n\nThis is part of a series to clean up the control block.\n\nChange-Id: I7f4cb05aef63053f8e2ab05b286d302260ef4758\n" }, { "commit": "d65d73c4ae74d084751b417615a78cbe7a51372a", "tree": "5f9f492220c0983554fb102ddc558462dc1ca26c", "parents": [ "f91a3abb7d136e75f0f5d999193b8c8297d97628" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jun 22 17:21:07 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 16 14:32:45 2012 -0800" }, "message": "\"if\" statements use curly braces per media style\n\nChange-Id: I130e7849fd1da7a0b7fe56c3c53919d26e3843b8\n" }, { "commit": "827e5f1237757aee78b677efcf0f7c44fd0dd3d8", "tree": "dfebb3c40be3ca42b017a2e9546930b5a718a86d", "parents": [ "f91a3abb7d136e75f0f5d999193b8c8297d97628" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 02 10:00:06 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 16 14:26:21 2012 -0800" }, "message": "Don\u0027t explicitly log tid\n\nIf needed, it can be obtained with adb logcat -v threadtime\n\nChange-Id: I91b3911d20f7bcfc3361db4052db21ff9181f1cf\n" }, { "commit": "e33054eb968cbf8ccaee1b0ff0301403902deed6", "tree": "b91f7abd2927e25a4cb7c50a8ef197c643ca9db6", "parents": [ "7d9c126be8dfe3016683eeb2b7a2d88ba5b24c0b" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 14 12:54:39 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 15 16:43:40 2012 -0800" }, "message": "Use size_t for frame counts\n\nAlso fix typo: bufferCount should be frameCount.\n\nChange-Id: Ibed539504db75ef99dc21c8ff1bf2987122063a5\n" }, { "commit": "60a839204713e0f8258d082af83262b1eb33a6c3", "tree": "5b8ef6ace475c96637bb6498b1c0a602611f9a58", "parents": [ "3b16c766d1ae2cfd8487e8ffb2b23936fc0a8e17" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Jun 21 12:56:37 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 14 16:45:53 2012 -0800" }, "message": "Clean up frame size in AudioTrack and AudioFlinger\n\nTrackBase::mFrameSize, mChannelMask, and mChannelCount are now const.\nUse TrackBase::mFrameSize instead of re-calculating frame size.\nAudioFlinger only sees 16-bit PCM format, conversion from 8-bit is\n now entirely on the client side. Previously a small part of the\n responsibility was on server side also.\nsize_t is unsigned, so use %u in logs.\nFix theoretical bug where TrackBase constructor was over-allocating space\n for non-linear AudioTrack or 8-bit PCM AudioRecord (probably benign).\n\nChange-Id: I7cbbba0bf4dba29ea751d8af341ab8e5cbbdc206\n" }, { "commit": "3b16c766d1ae2cfd8487e8ffb2b23936fc0a8e17", "tree": "56e1f373606202e5d5277f9645158d91b1a0a80c", "parents": [ "b4a17e834b718eff1ba2eac4232de6e73a4bf9f5" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 14 08:44:39 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 14 16:19:23 2012 -0800" }, "message": "Use uint32_t for sample rate\n\nChange-Id: Ie240b48fb54b08359f69ecd4e5f8bda3d15cbe80\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": "83a0382dc17364567667a4e6135db43f5bd92efc", "tree": "c3738c183fa49d9ccd023e27b37024860ccfff22", "parents": [ "bc0f6b92bba33ca9c2e76f2a520d290f055da6b2" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 12 07:58:20 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 12 15:13:34 2012 -0800" }, "message": "Move frame size out of the control block\n\nThis is part of a series to clean up the control block.\n\nChange-Id: Ifab1c42ac0f8be704e571b292713cd2250d12a3f\n" }, { "commit": "bc0f6b92bba33ca9c2e76f2a520d290f055da6b2", "tree": "680eed54d506a181ba8a6844e93d32a9a69747e4", "parents": [ "e35a582d02586cb43ad9e344138b4edad62868b3" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 12 14:32:06 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 12 14:32:06 2012 -0800" }, "message": "Fix regression for AudioTrack::write() 8-bit PCM\n\nBug: 7526532\nChange-Id: I8ddd1f0e9d035b54401788dcc422591281dcd97a\n" }, { "commit": "b929e417853694e37aba1ef4399f188987b709d9", "tree": "670f2da01fd3d33df6da2998c78273e0701300ad", "parents": [ "a47f3165f53c8e8fb8907a94de7417e2c3047eeb" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 08 12:13:58 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 08 12:18:59 2012 -0800" }, "message": "Move buffers pointer out of the control block\n\nThis is part of a series to clean up the control block.\n\nChange-Id: Ie474557db7cb360f2d9a0f11600a68f5a3d46f07\n" }, { "commit": "a47f3165f53c8e8fb8907a94de7417e2c3047eeb", "tree": "d6f691af946f8754b8534347c9b0b3ec535e776e", "parents": [ "659004c2949620d8adb29e1d950a2dd1c75ba9a9" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 07 10:13:08 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 08 11:22:38 2012 -0800" }, "message": "Simplify AudioTrack::restoreTrack_l()\n\nRemove CBLK_RESTORING and CBLK_RESTORED from control block flags,\nfor AudioTrack only. They are still used by AudioRecord.\n\nThis is part of a series to clean up the control block.\n\nChange-Id: Iae4798f5b527c492bdaf789987ff3a1dadd0cb37\n" }, { "commit": "659004c2949620d8adb29e1d950a2dd1c75ba9a9", "tree": "0619d1c2fa1515119ddd92648724e89a1073e76a", "parents": [ "755e8e1dcf36f282b727ff22d87d3c2fac70d30a", "b1c0993b215c5c3eebd1c6bafc22bba23d57a70b" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 08 10:35:06 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Nov 08 10:35:07 2012 -0800" }, "message": "Merge \"Add all-channel AudioTrack::setVolume() API\"" }, { "commit": "755e8e1dcf36f282b727ff22d87d3c2fac70d30a", "tree": "e0c8468a87427f51deba8d018931afb4394b3f1d", "parents": [ "864585df53eb97c31e77b3ad7c0d89e4f9b42588", "d5ed6e88a9bea1879e41d7defaf1edea7c09f554" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 08 10:04:06 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Nov 08 10:04:07 2012 -0800" }, "message": "Merge \"Fix call to restoreTrack_l() without lock held\"" }, { "commit": "d5ed6e88a9bea1879e41d7defaf1edea7c09f554", "tree": "99a34f51474700f4873e3f817b6b9f2c47001fc3", "parents": [ "8fa3ad0e4c77f7868348ad2a4cacc34c34e7c39e" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 02 13:05:14 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 07 16:11:56 2012 -0800" }, "message": "Fix call to restoreTrack_l() without lock held\n\nAlso document lock order\n\nChange-Id: I2c1f273a0a51fa79ee3dd766de8d23083e270051\n" }, { "commit": "864585df53eb97c31e77b3ad7c0d89e4f9b42588", "tree": "529492712326aa5aa997aa2d2f66a65baf4356b1", "parents": [ "8fa3ad0e4c77f7868348ad2a4cacc34c34e7c39e" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Nov 06 16:15:41 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 07 08:32:40 2012 -0800" }, "message": "Remove CBLK_DIRECTION from control block flags\n\nThis is part of a series to clean up the control block.\n\nChange-Id: I0265fece3247356b585d4d48fbda6f37aea8a851\n" }, { "commit": "8fa3ad0e4c77f7868348ad2a4cacc34c34e7c39e", "tree": "b2ca5d7c9f8ff86f0e69359251a873478115c503", "parents": [ "9d91e2a52b74380dae2662c8c00477c013b83c2c", "e0b07179a48ee50fda931d2aa1b3c751d167e4d7" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Nov 07 08:29:30 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Wed Nov 07 08:29:31 2012 -0800" }, "message": "Merge \"Remove CBLK_FAST from control block flags\"" }, { "commit": "e0b07179a48ee50fda931d2aa1b3c751d167e4d7", "tree": "d495cc851dba53652011108297d100e64a4c4acd", "parents": [ "48c6b2628f8a6037309c7957b07129d850187aa3" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Nov 06 15:03:34 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Nov 06 17:25:22 2012 -0800" }, "message": "Remove CBLK_FAST from control block flags\n\nThis is part of a series to clean up the control block.\n\nChange-Id: Ic881a3560d9547cb63fcc0cefec87aa3da480e0d\n" }, { "commit": "05d499958e4030938ed77a924ebdd9899f36752e", "tree": "c1eb564773575a2905de4b99f2a352e315611a5e", "parents": [ "48c6b2628f8a6037309c7957b07129d850187aa3" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Nov 06 14:25:20 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Nov 06 14:37:04 2012 -0800" }, "message": "Remove unused fields in AudioTrack::Buffer\n\nChange-Id: Iab75f6e2348d8b6d1f3cec95aeb3fcd5135dfb50\n" }, { "commit": "d2c38fc4d5dc742d7441444316849510dd2b7363", "tree": "5f7da68ffa30a22cc567a71af0d0fbf2c3ffb4c3", "parents": [ "cce9dfb4b17ef5a42c5805aeeae28d3390e4655f" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 01 14:58:02 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Nov 06 10:33:38 2012 -0800" }, "message": "Cache mCblk in local variable cblk\n\nUse \"iMem\" for sp\u003cIMemory\u003e\n\nChange-Id: I2f1fbbc517fbd77cfc92f6c3b1f253c26bae93b0\n" }, { "commit": "8e825a57202ce6cb2ddac67683ee5885334ce75d", "tree": "967d141640b7c6cbb737cd6c1ec6a6335fec9310", "parents": [ "7eb54958353458953ad052418c1c98a1b5338b7d", "9c5fdd83f9b9f49be35107971feb33528d60b945" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 05 16:32:24 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Mon Nov 05 16:32:24 2012 -0800" }, "message": "Merge \"Simplify control block flag names\"" }, { "commit": "b1c0993b215c5c3eebd1c6bafc22bba23d57a70b", "tree": "7b2d2eb6d08c5460fc896e083983ef6e65b5db45", "parents": [ "ad4e408b8ea397caadbfee85e1e39515e7e08104" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Feb 27 16:21:04 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 05 15:12:51 2012 -0800" }, "message": "Add all-channel AudioTrack::setVolume() API\n\nAdd combined channel APIs setVolume to AudioTrack,\nand remove obsolete getVolume.\n\nChange-Id: I0c87bfdbff4f4292259fa33e65f67badbafd270b\n" }, { "commit": "847d05dc8fa144dcf8f4f435d6a6ac1727f00937", "tree": "c079d882c6070ba620cad05b2b5254b4ba2ecf50", "parents": [ "f27bdebd0c611f9618577f65346f44574146dc00" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Feb 27 16:05:09 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 05 13:49:59 2012 -0800" }, "message": "Remove deprecated AudioTrack APIs\n\nChange-Id: I88be6525f3e33df529c0c3cb701d12a484809477\n" }, { "commit": "9c5fdd83f9b9f49be35107971feb33528d60b945", "tree": "cd8f20f3c4355d55057cc70369be9c2d3f890838", "parents": [ "ad4e408b8ea397caadbfee85e1e39515e7e08104" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 05 13:38:15 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 05 13:43:51 2012 -0800" }, "message": "Simplify control block flag names\n\nUse only one symbol per flag\n\nChange-Id: Ia3582e2134abd60c896d11337face65383e79c7c\n" }, { "commit": "9a08ebc8de71e260efb86cd1a04559b075b38ebc", "tree": "3974062bbee1bc0287d7f0d1ac00a1db1108e584", "parents": [ "821cea93f38065592456d6644600f5ee1123fe72" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 02 09:59:51 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Nov 02 13:24:50 2012 -0700" }, "message": "Fix spurious wakeup waiting for new IAudioTrack\n\nIf there was a spurious wakeup while waiting for another thread to create\na new IAudioTrack, we assumed that the track has been created when it\nmight not have been.\n\nChange-Id: I5f3999b4f7a06a00aabd65a746cc7222fff396ab\n" }, { "commit": "85ab62c4b433df3f1a9826bed1c9bec07a86c750", "tree": "0c5443a20824924cb7403f4cabfee84062489793", "parents": [ "5fe6138bd839297a1eed16885102b3bdfc98c040" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 01 11:11:38 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Nov 01 12:19:25 2012 -0700" }, "message": "Line length 100\n\nChange-Id: Ib28fd7b9ce951a6933f006e7f8812ba617625530\n" }, { "commit": "896adcd3ae6a1c7010e526327eff54e16179987b", "tree": "8c999b21698b1b7b45ac1d290ef3c64c808f380d", "parents": [ "4362f5300162ed55f56cf167eccf7f2e5b89d435" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Sep 13 11:18:23 2012 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Sep 13 17:03:00 2012 -0700" }, "message": "audioflinger: send priority request from a thread\n\nWhen creating a fast AudioTrack, a request is sent to SchedulingPolicyService\nto elevate the requesting thread priority. This generates a binder\ncall into system_server process and to a JAVA service via JNI.\nIf the thread from which the track was created is in the system_server\nprocess and does not have the \"can call java\" attribute, a crash occurs because\nthe binder optimization reuses the same thread to process the returning binder\ncall and no JNI env is present.\n\nThe fix consists in sending the priority change request from the AudioFlinger\nmixer thread, not from the binder thread.\n\nThis also reverts the workaround in commit 73431968\n\nBug 7126707.\n\nChange-Id: I3347adf71ffbb56ed8436506d4357eab693078a3\n" }, { "commit": "73431968e3c5cf420b5d63c54cf72d10ff7ee7bc", "tree": "11239e23b4a170e479addf4a102200b46e6c8ebb", "parents": [ "359cf1cac2a7851a59dfe4960a1a5992ac6fdb84" ], "author": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Wed Sep 12 19:42:34 2012 -0700" }, "committer": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Wed Sep 12 19:42:34 2012 -0700" }, "message": "Workaround for track recreation bug\n\nIgnore thread can call Java parameter to work around bug where\n AudioTrack instance gets recreated when key clicks are enabled\n and WFD gets turned on, with the wrong parameters.\n\nChange-Id: Ia42c8704b46fe3ffea560b05b60939fa2e4b29e1\n" }, { "commit": "c3ae93f21280859086ae371428ffd32f39e76d50", "tree": "542d16fac64eb1f6e40c32517d60c2da5078e900", "parents": [ "c9936c72c128a4a9288424fb082d7e7fe4b9b91f" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Jul 30 10:59:30 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Aug 30 15:31:00 2012 -0700" }, "message": "Update audio comments\n\nChange-Id: Ie7504d0ddb252f7e4d4f99ed0b44cfc7b1049816\n" }, { "commit": "a997e7a7c5aa2fc7e95316218316f8b7b98786ba", "tree": "4f98e352833c4de61728588cacc08d40d1d01778", "parents": [ "8da4cd7d706d9f551bf2d08cd8b61907a10cd38d" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Aug 07 09:44:19 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Aug 07 15:13:42 2012 -0700" }, "message": "Revert \"Swap the order of creating IAudioTrack and thread\"\n\nThis reverts commit 5d464eb0b8cffb994a754ff108795e858a882414.\nIt caused the wrong thread ID -1 to be passed to IAudioFlinger::createTrack().\n\nChange-Id: Ic221d2bb4af572d3d2d752af19238c52f6728e3a\n" }, { "commit": "5d464eb0b8cffb994a754ff108795e858a882414", "tree": "e5d78a91a03a1895cc03cfeb89c26aac49d15a9e", "parents": [ "0ec23ce0d1ff79566c402bc30df3074f6e25a22b" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jun 22 17:19:53 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jul 27 14:58:43 2012 -0700" }, "message": "Swap the order of creating IAudioTrack and thread\n\nSimplifies the error recovery in case IAudioTrack fails.\n\nChange-Id: I6aee41a2ac747a5689fb4836b04174e6107bf32f\n" }, { "commit": "28b76b334f92a15a2be3cc9e2f7d229a3275d1ac", "tree": "c605d3e995f4b6be55dd4b10e1c998b059990d0d", "parents": [ "fd4e20c226eca185fc789de761beae64855bfbbb" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jul 03 17:24:41 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jul 17 11:31:59 2012 -0700" }, "message": "Use audio_channel_mask_t for channel mask\n\nChange-Id: I1c1896da48983aa9f1462a4b471f910498816f60\n" }, { "commit": "04cd0186305e2b59d23c9147787046c6662029cc", "tree": "5c2da34c5489ab0741729ba6b66cec980c384720", "parents": [ "5f0033d27b75fead593ed7ca2b86f5b6f5adf6cc" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Jun 25 11:49:27 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jul 13 09:05:50 2012 -0700" }, "message": "getMinFrameCount error handling\n\nConvention is for \"get\" APIs that directly return status_t and indirectly\nreturn a value via a pointer, to return BAD_VALUE if the pointer is NULL.\nAlso indirectly return 0 for other errors.\n\nChange-Id: I1599f20ecb26e9723f9fb384ffbf911ff3a2ce1c\n" }, { "commit": "d4070955e28ae62aa4be1657f9d32acde104bb86", "tree": "4e289a66bb82da52f8e0e1c1691b35f1309d9f35", "parents": [ "39c54f68804c1ce5c85ec588f3c2c63447a807b4" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Jul 12 11:03:31 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Jul 12 11:09:09 2012 -0700" }, "message": "Remove dead code in libmedia\n\nChange-Id: I7d8201590cda29c9fa99662a4fdba222091febfe\n" }, { "commit": "d3a9ff4b725de612bf0354c035ba8f8564dbc6e8", "tree": "6352c2917870180111ebb84d3b1839ac2392bdae", "parents": [ "a43787a8cc513715543fb5c80c7c464e2cd51b6e" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Jun 21 12:11:38 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Jun 21 12:11:38 2012 -0700" }, "message": "Move declarations of local variables to first use\n\nChange-Id: I48b193a742b32b6746aa938b84dc405124a6a5c3\n" }, { "commit": "192cbbad773979a6fe3b5a0c223356de4fc3309c", "tree": "23bdbff8fd2a2b88a63a54287c6112908b050fb7", "parents": [ "82233b4b5bee06f11618a558ae465abbfdedfd6e" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Jun 13 08:38:36 2012 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Jun 14 10:56:59 2012 -0700" }, "message": "Fix audio track pause.\n\nAudioTrack::pause() should signal the control block condition\nto release threads waiting for available buffers in obtainBuffer().\nOtherwise the behavior relies on the timout on the condition\nor the fact that audioflinger will mix a new audio buffer while executing\nthe pause.\n\nBug 6653769.\n\nChange-Id: I5f8f73c471fe306070f30b814f32fd4b4dc1d575\n" }, { "commit": "0c9d26d187017f7fb028ab52a0fbc6395142faa4", "tree": "35fd2f6babef082b85058fd27a69bda048687295", "parents": [ "b4ea1ab821d652cb080910d4081f5f1318597a90" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu May 31 14:35:01 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jun 13 15:12:42 2012 -0700" }, "message": "Log track name on obtain/releaseBuffer warnings\n\nThis should help diagnose problems by allowing us to correlate\nthe logs with the dumpsys media.audio_flinger output.\n\nChange-Id: I8c7c592b4f87d13b0f29c66ce7a2f301a0f063c9\n" }, { "commit": "df839841d2db4cb8e2acb10205b3942622b3e7a2", "tree": "183ce49d98a66d866aeed85e98da6709206074ac", "parents": [ "83410a85993ad6f5f0c122036ff0bda42bf1d4f7" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu May 31 14:27:14 2012 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu May 31 14:32:38 2012 -0700" }, "message": "Do not keep audio wake lock when apps underrun.\n\nDo not automatically restart an AudioTrack after an underrun\nif the callback is executed but no data is written by the app.\n\nBug 6541286.\n\nChange-Id: I11e7ab8dc968d7ff087058fec68f44490d3a7731\n" }, { "commit": "2986460984580833161bdaabc7f17da1005a8961", "tree": "bd8aa210c0faee61dc45e23479e29cfc85f313c2", "parents": [ "dfa29ab13647f22b30b2de34d4830c9e815bf120" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue May 08 18:57:51 2012 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon May 14 17:20:12 2012 -0700" }, "message": "Fix issues with synchronous record start.\n\n- Added a timeout in case the trigger event is never fired.\n- Extend AudioRecord obtainBuffer() timeout in case of\nsynchronous start to avoid spurious warning.\n- Make sure that the event is triggered if the track is\ndestroyed.\n- Reject event if the triggering track is in an incompatible state.\n\nAlso fix a problem when restoring a static AudioTrack after\na mediaserver crash.\n\nBug 6449468.\n\nChange-Id: Ib36e11111fb88f73caa31dcb0622792737d57a4b\n" }, { "commit": "093000f7d11839b920e8dfaa42ed1d09f48e24b8", "tree": "ffecb5126696e2c2c83980ce8ad561b60278064a", "parents": [ "6a3969cc3bcbfc84dbcc27aa726eb96bc03a830b" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu May 03 09:35:36 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu May 03 10:05:00 2012 -0700" }, "message": "Don\u0027t allow AudioTrack frameCount to decrease\n\nThis is a workaround for bug that client can cache return value of\nframeCount(), and is not notified when this value changes due to automatic\nre-recreation of the underlying IAudioTrack.\n\nA better long-term fix would be to notify clients when these kinds of\nparameters change, and to fix assumptions in client code that they are\nconstant (e.g. in SoundPool and maybe obtainBuffer).\n\nAlso, once a fast track request is denied, don\u0027t request it again.\n\nBug: 6431187\nChange-Id: I55b4ff30bbd9ed3a402e39452a38de52cdea53a9\n" }, { "commit": "f4022f90db5acb680870db8c1150b673cdd211d9", "tree": "0ffaec361f808d5710d42298fa7d01873771cf6c", "parents": [ "2c3297ab6c4daaaa7b27eed8418c64cf168fe2a1" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed May 02 10:34:59 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed May 02 10:37:55 2012 -0700" }, "message": "Fix race in AudioTrack destruction\n\nBug: 6427369\nChange-Id: Id3b4487406235b881f6f0b4b95c5a02a9b797e75\n" }, { "commit": "31dfd1db7a4d2228d9642008af6f3dd744368eb6", "tree": "5182b898a5f21161ea879c4fe173f91d54366fd7", "parents": [ "2409e4337b1e5968cbf75a839778f849266b64d9" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue May 01 11:07:08 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue May 01 11:07:48 2012 -0700" }, "message": "Disable fast track log spam\n\nexcept for \"denied by client\" and \"denied by server\"\n\nChange-Id: I133ab747933729cc1f386813ee06ece055bdb294\n" }, { "commit": "e0fa467e1150c65a7b1b1ed904c579b40f97c9df", "tree": "44ae1e182fd15695fccf5d1f396f99edc7942ee5", "parents": [ "e370bb62b89d2f2980f519392010ea08a24c558e" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Apr 24 14:35:14 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Apr 25 18:41:32 2012 -0700" }, "message": "Move frame count calculations for fast tracks\n\nFor fast tracks: move the default and minimum frame count calculations\nfrom client to server. If accepted, the default and minimum frame count\nis the fast mixer (HAL) frame count. If denied, the default and minimum\nframe count is the same as it currently is for normal tracks.\n\nFor normal tracks: there is no change yet, preserve legacy behavior for\nnow but add a FIXME to change this later.\n\nBug fix: the test for buffer alignment matches channelCount was wrong.\n\nBug fix: check for 8-bit data in shared memory, which isn\u0027t supported.\n\nOptimizations:\n - in set(), only call AudioSystem::getOutputSamplingRate() when needed\n - in createTrack_l(), only call AudioSystem::getSamplingRate() and\n AudioSystem::getFrameCount() when needed\n\nChange-Id: I79d2fe507db1a8f7bb094c71da8a129951dbb82f\n" }, { "commit": "a636433cbd09c0708b85f337ef45c0cdef3bcb4d", "tree": "8a629f33d648d1ddd8b79305f02763c539fef7cd", "parents": [ "9adb12830ac2da72c91b842d24725ced6d236b16" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Apr 19 09:35:04 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Apr 23 16:54:41 2012 -0700" }, "message": "Use C APIs instead of C++ APIs for policy\n\nThe C++ APIs are going away.\n\nNote: we use tid \u003d\u003d 0 which is not supported yet by the C APIs,\ndo not submit this until that is added.\n\nChange-Id: I0e90789e6c81c69f2544e899c52421ea5d1342be\n" }, { "commit": "ca8b28013c0558a4a3323a1a0f58520277200086", "tree": "56e351a74a524a059ed1759a6612cf88f6fd5b82", "parents": [ "4a6c9c59a0297c3c6fcfadddaed7a9a976046daf" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Apr 23 13:58:16 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Apr 23 14:15:45 2012 -0700" }, "message": "Fix regression in AudioTrack::pause()\n\nBug: 6379646\nChange-Id: I12b53bc4118499ddc73a53a981f3f56328140868\n" }, { "commit": "3acbd053c842e76e1a40fc8a0bf62de87eebf00f", "tree": "1d6dcc01039a1af770131c6b427efa27a2e5755f", "parents": [ "f853499c6a088ef8506cacbb3ccdffa87e0cec3e" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Feb 28 10:39:56 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Sun Apr 22 14:50:01 2012 -0700" }, "message": "Configure policy of mediaserver threads\n\nChange-Id: Ifd825590ba36996064a458f64453a94b84722cb0\n" }, { "commit": "1948eb3ea6eee336e8cdab9b0c693f93f5f19993", "tree": "448eed49a4ba18fb1ab4f2fb1b2c0d5cef6d242f", "parents": [ "21e8c50bd13ebe44f3088e26c9c6df0e163c469c" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Apr 13 16:50:19 2012 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Apr 20 14:17:02 2012 -0700" }, "message": "Add support for deep audio buffers\n\nAllow AudioSink to use deep audio buffering when the\nsource is audio only and its duration is more than\na certain threshold.\nThis helps improve battery life but implies higher\naudio latency.\n\nChange-Id: Ie79915b61c370292f05aabda9779356570e03cbb\n" }, { "commit": "0ca3cf94c0dfc173ad7886ae162c4b67067539f6", "tree": "bc156e7822e663626b78ec2ad09ef3970fbb48c5", "parents": [ "ee685f6d19a6cc3b0bf02e69eb5dea11a2aeb601" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Apr 18 09:24:29 2012 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Apr 18 15:04:46 2012 -0700" }, "message": "rename audio policy output flags\n\nChange-Id: I27c46bd1d1b2b5f96b87af7d05b951fef18a1312\n" }, { "commit": "1a9ed11a472493cac7f6dfcbfac2064526a493ed", "tree": "5f4a3c6856f294b3c0371afd87df96277215107b", "parents": [ "290ca3a767a974ce9d041e5bcb0c69153f7851d5" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Mar 20 18:36:01 2012 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Apr 03 16:53:25 2012 -0700" }, "message": "audio policy: add configuration file\n\nremoved outputs to stream mapping cache in audio system: the output for a\ngiven stream type must always be queried from audio policy manager as the cache\nis not always updated fast enough by audioflinger callback.\n\nremoved AudioFlinger::PlaybackThread::setStreamValid() not used anymore if\nstream to output mapping is not cached.\n\nChange-Id: Ieca720c0b292181f81247259c8a44359bc74c66b\n" }, { "commit": "4a4a0959bca78e03e3c3f486ba17829c28314d8c", "tree": "5fbfe5302ca28741483ae64d0ccf2630830f167e", "parents": [ "a1472d9883e35edd280201c8be3191695007dfd4" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Mar 19 11:38:14 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Apr 02 07:34:39 2012 -0700" }, "message": "AudioTrack client fast policy\n\nChange-Id: I42ce691df3f586ac061b62237f35a263a0116f1f\n" } ], "next": "a1472d9883e35edd280201c8be3191695007dfd4" }