)]}' { "log": [ { "commit": "fdac7c00f9201bb3a9862069145f01d37e39755b", "tree": "f82c52ff982030d0b1c4d7ea75a987300393c7b3", "parents": [ "9be49c915852b592a881be3905658e4fc698749d" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jan 28 11:03:28 2014 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jan 28 20:06:35 2014 +0000" }, "message": "Replace size_t in shared memory by uint32_t\n\nEventually we may want to use uint64_t, but will need to confirm atomicity.\n\nBug: 12381724\nChange-Id: Ia2c591d262d22b47b6f7dab4b9d9faa14b86d865\n" }, { "commit": "74935e44734c1ec235c2b6677db3e0dbefa5ddb8", "tree": "dc7bc2c9f496097e95ff36440fcd79886517ba53", "parents": [ "bd72d22097f78f5bd668b223bc8c94e351311e31" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Dec 19 08:56:45 2013 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Dec 20 08:40:55 2013 -0800" }, "message": "Replace control block frameCount_ by explicit in/out parameter\n\nin IAudioFlinger::createTrack and IAudioFlinger::openRecord\n\nChange-Id: I09c644c80e92c8e744b1b99055988a2588b2a83d\n" }, { "commit": "a153b68f2260a8ed7fbb236fa659b13264ac5ac0", "tree": "fde0bd5a83da59bde7dc9b146a8a8af48d911adc", "parents": [ "6a7327f110b86613e6e8c7de105c49803acacc1d", "77177fa20773d02b4f9c4147ecb98107f019fa7d" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Oct 18 15:09:03 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Fri Oct 18 15:09:03 2013 -0700" }, "message": "am 77177fa2: am 95c03858: Merge \"Allow releaseBuffer after flush\" into klp-dev\n\n* commit \u002777177fa20773d02b4f9c4147ecb98107f019fa7d\u0027:\n Allow releaseBuffer after flush\n" }, { "commit": "2e422c472c91aa7912befd0fc038d1e11f354bc1", "tree": "7583e04f353f9cadf5081fafdcb8042450e3c85c", "parents": [ "dedc7b0fa23c21b525dc8abb572bb8161ad66a9b" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Oct 18 13:00:29 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Oct 18 13:53:41 2013 -0700" }, "message": "Allow releaseBuffer after flush\n\nAfter AudioTrack start checks for pending flush,\nallow releaseBuffer on any previously obtained buffer.\nFor example, this can happen if the resampler has obtained\na buffer but not released the whole buffer yet.\nNote that the resampler will be reading obsolete data.\n\nBug: 11285590\nChange-Id: I0614fbb62e43604aac3089cce4b7797c87a306b5\n" }, { "commit": "771f1b6d196b7924707fc44177ac3a88fc31fa9e", "tree": "ffd599ada0a44511112fe0586c8f56121ce9ce6e", "parents": [ "a529ee78d7cc5d94cbe79cea2d7e6469c88a0fba", "7f8c397378a7ee5abd395413be71388ad36d3ed2" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Oct 16 15:38:30 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Wed Oct 16 15:38:30 2013 -0700" }, "message": "am 7f8c3973: am 120a8847: Merge \"AudioTrack: fix head position after restore\" into klp-dev\n\n* commit \u00277f8c397378a7ee5abd395413be71388ad36d3ed2\u0027:\n AudioTrack: fix head position after restore\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": "16f3201d2fe99a39a8a9bb478528ae9003645dfe", "tree": "44cfe4ab9e4004df968b8334e3928c00e78f412a", "parents": [ "44173e2805eba1c5812dd39c02e3964ebdd2104e", "11454092e4a94d3c8b4576c981595339abdfac0d" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Sep 17 15:07:10 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Tue Sep 17 15:07:10 2013 -0700" }, "message": "am 11454092: am 18f86140: Merge \"Fix underruns when fast track denied due to SRC\" into klp-dev\n\n* commit \u002711454092e4a94d3c8b4576c981595339abdfac0d\u0027:\n Fix underruns when fast track denied due to SRC\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": "f557872b594d15599bdb65efd91eb5adeb1d0840", "tree": "8df16a0c49bac71163035d11562e06bcc12a2490", "parents": [ "30c296c153447f5c2bb5251928bf4d3c4e28e552", "908d3c09ca7f2ccb280aa5dc8d876099ff9a9d0f" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Aug 29 17:12:45 2013 -0700" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Thu Aug 29 17:12:45 2013 -0700" }, "message": "am 908d3c09: am 51ec03c2: Merge \"Implement Track::getTimestamp()\" into klp-dev\n\n* commit \u0027908d3c09ca7f2ccb280aa5dc8d876099ff9a9d0f\u0027:\n Implement Track::getTimestamp()\n" }, { "commit": "bd096fd9d8e5fc0e62f98807f4818a06f70d0812", "tree": "8ff27bce9f34e4bdd0f8e233494a24efef475a7d", "parents": [ "573d80a8f463f648a515fc0975bf83951b272993" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Aug 23 13:53:56 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Aug 29 09:59:31 2013 -0700" }, "message": "Implement Track::getTimestamp()\n\nusing a new timestamp latch in PlaybackThread, and\nAudioTrackServerProxy::framesReleased() which returns mServer.\n\nChange-Id: I1ebfba968c773faaab95648c272fd3ebd74718d6\n" }, { "commit": "e198c360d5e75a9b2097844c495c10902e7e8500", "tree": "be6884fcddb31fb8feda454e38b00053ccac7bf1", "parents": [ "2fc14730e4697a6f456b4631549c9981f6b0b115" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Aug 13 09:13:36 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Aug 13 09:55:52 2013 -0700" }, "message": "Whitespace\n\nChange-Id: Id3ccc183a03421330d0498faaa62a45915cdc3d6\n" }, { "commit": "82aaf94a5b18939e4d790bbc752031f3070704a3", "tree": "4e39e32d0ccde726548afcbd81e253d5e316b295", "parents": [ "b08ab81bab9e8cd2e3a7fff02725a387070fd078" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jul 17 16:05:07 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jul 31 09:15:00 2013 -0700" }, "message": "Report underruns for fast tracks also\n\nThis fixes a regression that was introduced earlier\nby commit 9f80dd223d83d9bb9077fb6baee056cee4eaf7e5\ncalled \"New control block for AudioTrack and AudioRecord\".\nThat commit broke underrun reporting for fast tracks.\n\nAlso remove Track::mUnderrunCount, which counted the number of underrun\nevents, and was only used by dumpsys media.audio_flinger.\n\nNow dumpsys media.audio_flinger reports the number of underrun frames,\n\nIsolated underrun-related control block accesses via the proxy, so that\nthe server is not directly poking around in the control block.\n\nThe new proxy APIs are AudioTrackServerProxy::getUnderrunFrames() and\nAudioTrackServerProxy::tallyUnderrunFrames(). getUnderrunFrames() returns\na rolling counter for streaming tracks, or zero for static buffer tracks\nwhich never underrun, but do a kind of \u0027pause\u0027 at end of buffer.\ntallyUnderrunFrames() increments the counter by a specified number of frames.\n\nChange-Id: Ib31fd73eb17cbb23888ce3af8ff29f471f5bd5a2\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": "f20e1d8df84c5fbeeace0052d100982ae39bb7a4", "tree": "997cd41de1adb829c194f56cf29eb02945666b94", "parents": [ "eced2daaa6c91a3731eef978ce65c6ec319c5e6a" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jul 12 09:45:18 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jul 30 09:24:03 2013 -0700" }, "message": "Rename control block server to mServer and add comments\n\nChange-Id: Ieabd91acee92d0e84e66fbd358df5282b856306e\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": "bfb1b832079bbb9426f72f3863199a54aefd02da", "tree": "5007d22456776dc77d124be84bcfe8bc4152bbb2", "parents": [ "fc5ea08326b510c6f82f71845d95a8758a6ab698" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Jan 07 09:53:42 2013 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Jul 25 14:08:09 2013 -0700" }, "message": "AudioFlinger: offload playback, non-blocking write\n\n- Added specialized playback thread class for offload playback,\nderived from directoutput thread.\nThis thread type handles specific state transitions for offloaded\ntracks and offloading commands (pause/resume/drain/flush..) to audio HAL.\nAs opposed to other threads, does not go to standby if the track is paused.\n\n- Added support for asynchronous write and drain operations at audio HAL.\nUse a thread to handle async callback events from HAL: this avoids locking\nplayback thread mutex when executing the callback and cause deadlocks when\ncalling audio HAL functions with the playback thread mutex locked.\n\n- Better accouting for track activity: call start/stop and release Output\nmethods in audio policy manager when tracks are actually added and removed\nfrom the active tracks list.\nAdded a command thread in audio policy service to handle stop/release commands\nasynchronously and avoid deadlocks with playback thread.\n\n- Track terminated status is not a state anymore. This condition is othogonal\nto state to permitted state transitions while terminated.\n\nChange-Id: Id157f4b3277620568d8eace7535d9186602564de\n" }, { "commit": "7db7df0e8d9d7cee8ba374468cdbfa0108e3337c", "tree": "d1f5c23d596c07854e75999dbf333194d500aaa5", "parents": [ "8f13428cb29a9b610a9e9a0f5fea437cd2d3f032" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jun 25 16:13:23 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jul 02 14:45:19 2013 -0700" }, "message": "AudioTrackShared cleanup\n\nMaintain unreleased frame count on client side also (was already there on server side).\nAssertion failure instead of BAD_VALUE status for incorrect usage of APIs.\nClean up error handling code.\n\nChange-Id: I23ca2f6f8a7c18645309ee5d64fbc844429bcba8\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": "0d09a9bec07b3bec78bd473ff0bfcf0a261f3f25", "tree": "410d8c22e5d15f9143d04212cd96939c217cacc5", "parents": [ "a2cf13c77260f98fc674f103dcbd2bb2488a80ce" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Jun 24 12:06:46 2013 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Jun 24 13:34:08 2013 -0700" }, "message": "Use mFutex as an event flag rather than semaphore\n\nAn event flag can be more fault-tolerant in case of loss of synchronization,\nas it cannot overflow. It also allows more bits to be used in the future.\nSee http://en.wikipedia.org/wiki/Event_flag\n\nChange-Id: I01ca25d951eb263124da54bb4738f0d94ec4a48b\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": "5c4cc0d99d3b1cb35c5d7c237272ee53142745fb", "tree": "96c55aba5ce8363f33a05641f0a4be8ed09128c9", "parents": [ "d573622dc001c23223cb26b1f55fb75be189e77d" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 26 10:40:24 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Mar 05 08:26:50 2013 -0800" }, "message": "Add template class SingleStateQueue\n\nChange-Id: If7e2bc9b2a216524ee9cbb68682e2634933b4973\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": "24ee6aaf1368e3a07061948273c629d05cb9b12a", "tree": "c915ddc3f5b594050258178b31075ff073b1b570", "parents": [ "45f7c446777171a728f9eb257f48145dde5d19e2", "ba6b1bc38e0c355277f69af286469adb5f02e876" ], "author": { "name": "Andreas Huber", "email": "andih@google.com", "time": "Fri Nov 30 08:51:51 2012 -0800" }, "committer": { "name": "Android Git Automerger", "email": "android-git-automerger@android.com", "time": "Fri Nov 30 08:51:51 2012 -0800" }, "message": "am ba6b1bc3: am e7550eb2: Merge \"Initialize and copy mRotationAngle in VideoFrame\"\n\n* commit \u0027ba6b1bc38e0c355277f69af286469adb5f02e876\u0027:\n Initialize and copy mRotationAngle in VideoFrame\n" }, { "commit": "37007147edde731cd3dc74febd7bb7e093a35020", "tree": "3e85f5a609e879554d48945b502c951f75f44724", "parents": [ "5768fa034ede834656697d3612c525595ff85ef9" ], "author": { "name": "Christer Fletcher", "email": "christer.fletcher@sonyericsson.com", "time": "Thu Sep 13 10:28:39 2012 +0200" }, "committer": { "name": "Johan Redestig", "email": "johan.redestig@sonymobile.com", "time": "Fri Nov 30 15:32:18 2012 +0100" }, "message": "Initialize and copy mRotationAngle in VideoFrame\n\nWhen thumbnails were generated they could be generated at random\nangles as the mRotationAngle variable was not initialized to any\nvalue. This variable would have to be explicitly overwritten to not\ncause random rotation. Changed the implementation to initialize the\nvalue to 0 (no rotation). mRotationAngle was also missing in the\ncopy constructor.\n\nChange-Id: I67a5340fdd807c6ab3a3da5eecb09b5b9d5f4666\n" }, { "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": "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": "b36a7a68af073b1e7fd5cad6aa2c52223fd30efd", "tree": "ffab4964484a5eac7e9ca8f50a40c03cf4cd4e82", "parents": [ "93a7efd45c28becb994f0818ea29c00ec45eea57" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Nov 12 15:46:10 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Nov 13 08:51:39 2012 -0800" }, "message": "Simplify AudioRecord::restoreTrack_l()\n\nFinish removing CBLK_RESTORING and CBLK_RESTORED from control block flags,\nand remove constant RESTORE_TIMEOUT_MS.\n\nAlso minor cleanup:\n - Cache mCblk in local variable cblk and make cblk allocatable in a register.\n - Use \"iMem\" for sp\u003cIMemory\u003e.\n - Add missing error log to AudioRecord; it was already in AudioTrack.\n\nThis is part of a series to clean up the control block.\n\nChange-Id: Ia5f5ab4763c392bc06a45851b167ddaee29e3455\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": "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": "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": "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": "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": "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": "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": "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": "e53b9ead781c36e96d6b6f012ddffc93a3d80f0d", "tree": "4bcde0bc9a797851ec1bee4f630c8c4f0735f623", "parents": [ "b87396f9ebabbb7b47683bceca96cbe635a1ca00" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Mon Mar 12 16:29:55 2012 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Mar 13 11:09:47 2012 -0700" }, "message": "Whitespace and indentation\n\nFix indentation to be multiple of 4.\nMake it easier to search:\n sp\u003c not sp \u003c to\n \"switch (...)\" instead of \"switch(...)\" (also \"if\" and \"while\")\nRemove redundant blank line at start or EOF.\nRemove whitespace at end of line.\nRemove extra blank lines where they don\u0027t add value.\n\nUse git diff -b or -w to verify.\n\nChange-Id: I966b7ba852faa5474be6907fb212f5e267c2874e\n" }, { "commit": "99e53b86eebb605b70dd7591b89bf61a9414ed0e", "tree": "8939550ada02fe21121550a83952ebf7638c0dac", "parents": [ "6dbc1359f778575d09d6da722b060a6d72c2e7c5" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Thu Jan 19 08:59:58 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Feb 14 09:17:59 2012 -0800" }, "message": "Update comments\n\nWe no longer put the filename at start of file.\n\nChange-Id: Ic435b159a23105681e3d4a6cb1ac097bc853302e\n" }, { "commit": "1a0ae5be3d1273cba12584b33830d859510fbf82", "tree": "8073be7b5e33ee230159f7c0aed22bb6de63d3f9", "parents": [ "787bae0578fbaab6219ebf23494866b224d01438" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Feb 03 10:24:48 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Feb 03 16:16:46 2012 -0800" }, "message": "Don\u0027t double destruct audio_track_cblk_t\n\nFortunately audio_track_cblk_t doesn\u0027t have a destructor, but for clarity\nremove the double destruction.\n\nAlso add warning not to add any virtuals to audio_track_cblk_t.\n\nChange-Id: I70ebe1a70460c7002145b2cdf10f9f137396e6f3\n" }, { "commit": "83d86538c4c479a9225c75ab27938e8f05abb9c8", "tree": "72fdff9fb9bb7e41fe48c41bfed65ab83b64f85f", "parents": [ "58f30210ea540b6ce5aa6a46330cd3499483cb97" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jan 17 14:39:34 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jan 20 16:19:59 2012 -0800" }, "message": "Make AudioTrack control block volume field private\n\nThis is part of the process of abstracting the control block\nto make it easier to maintain.\n\nChange-Id: Idb8f461e68dab3bcf268159cc0781651c6fb7094\n" }, { "commit": "b1cf75c4935001f61057989ee3cf27bbf09ecd9c", "tree": "e3daaea6160475e36dedfccaabf5d8b94406d05e", "parents": [ "f237a30e9344d43f832ba11db6f62c6ad2084444" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jan 17 12:20:54 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jan 17 16:25:17 2012 -0800" }, "message": "Track volume cleanup\n\nAlways read and write track volumes atomically. In most places this was\nalready being done, but there were a couple places where the left and\nright channels were read independently.\n\nChanged constant MAX_GAIN_INT to be a uint32_t instead of a float.\nIt is always used as a uint32_t in comparisons and assignments.\nUse MAX_GAIN_INT in more places.\n\nNow that volume is always accessed atomically, removed the union\nand alias for uint16_t volume[2], and kept only volumeLR.\n\nRemoved volatile as it\u0027s meaningless.\n\nIn AudioFlinger, clamp the track volumes read from shared memory\nbefore applying master and stream volume.\n\nChange-Id: If65e2b27e5bc3db5bf75540479843041b58433f0\n" }, { "commit": "613882293184e575a44bff681a3decaefe889e69", "tree": "052f84c8fe6e7b201f03117f617c3ad77b8b0dd0", "parents": [ "0107954f72153db747a3727dc1157e9236dfed90", "b9980659501d0428d65d8292f3c32da69d37fbd2" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jan 17 07:27:46 2012 -0800" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Tue Jan 17 07:27:46 2012 -0800" }, "message": "Merge \"Use size_t for frame size\"" }, { "commit": "05632a5fa4b88ca474294887fc92a9fcdf0e2352", "tree": "28e36d2cce839d66870ea066a2b334dd9b63c833", "parents": [ "09192653e836b21689f004bf8dee375356641181" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Tue Jan 03 14:22:33 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jan 13 15:25:14 2012 -0800" }, "message": "AudioTrack and AudioFlinger send level cleanup\n\nAdd an API to control block for getting/setting send level.\nThis allow us to make the mSendLevel field private.\n\nDocument the lack of barriers.\n\nUse 0.0f to initialize floating-point values (for doc only).\n\nChange-Id: I59f83b00adeb89eeee227e7648625d9a835be7a4\n" }, { "commit": "b9980659501d0428d65d8292f3c32da69d37fbd2", "tree": "8b703aca299f4f715a65d07cc37b753e9753a941", "parents": [ "f4e76c538f81068bc2e125f3f9ef8b8dd40f2d2a" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jan 11 09:48:27 2012 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Fri Jan 13 14:58:44 2012 -0800" }, "message": "Use size_t for frame size\n\nexcept in the control block, where we don\u0027t have room.\n\nIn AudioFlinger::ThreadBase::TrackBase::getBuffer,\nread the frame size from control block only once.\n\nChange-Id: Id6c4bccd4ed3e07d91df6bbea43bae45524f9f4e\n" }, { "commit": "362c4e697d8e9c034e964ac7b40227e054491547", "tree": "dde3ca3f11e00cc82f1b6a6bd7ebe7b1c24421c5", "parents": [ "eebeceec684a36222b4559e3157b0db04c0a67ed" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Dec 14 10:28:06 2011 -0800" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Dec 14 14:38:26 2011 -0800" }, "message": "Audio C++ comments\n\nChange-Id: I84906ebb9dfcfa5b96b287d18364b407f02a30c1\n" }, { "commit": "c6854100cea4fcd0f20cb2ac8235c02d1849b3a1", "tree": "f99b0d979380e2208f3d07c56397205c06b38373", "parents": [ "d217a8c4632b3e3065f8c2a26b9ce4dc4c97171f" ], "author": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jun 01 16:46:29 2011 -0700" }, "committer": { "name": "Glenn Kasten", "email": "gkasten@google.com", "time": "Wed Jun 01 16:46:29 2011 -0700" }, "message": "Remove unnecessary level of priority indirection\n\nChange-Id: I942d43973c20a7ace8b0d3f78b4da97e45e996c6\n" }, { "commit": "0d255b2d9061ba31f13ada3fc0f7e51916407176", "tree": "4df26e48710542386bddab7a0d1cbfa5d6835ac0", "parents": [ "65580f9adf6c4d98449ad0716488f9fe3869aa5a" ], "author": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Tue May 24 15:53:33 2011 -0700" }, "committer": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Wed Jun 01 10:55:29 2011 -0700" }, "message": "Use channel mask instead of channel count for track creation\n\nRecord and playback objects (resp AudioRecord and AudioTrack)\nare created using a channel mask, but this information is lost\nin the mixer because only the channel count is known to\nAudioFlinger. A channel count can always be derived from a\nchannel mask.\n\nThe change consists in:\n- disambiguiting variable names for channel masks and counts\n- passing the mask information from the client to AudioFlinger\n and the mixer.\n- when using the DIRECT ouput, only verifying the format of\n the track is compatible with the output\u0027s for PCM.\n\nChange-Id: I50d87bfb7d7afcabdf5f12d4ab75ef3a54132c0e\n" }, { "commit": "38ccae2c0324daa305f3fe77d25fdf5edec0b0e1", "tree": "c7aec92091f634a6994ed7e836a2a1ba4e22c035", "parents": [ "7fb4b427d5fca79c09d69b212ddca41539c215a2" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Mar 28 18:37:07 2011 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Apr 05 12:05:41 2011 -0700" }, "message": "New fix for issue 4111672: control block flags\n\nThe first fix (commit 913af0b4) is problematic because it makes threads\nin mediaserver process block on the cblk mutex. This is not permitted\nas it can cause audio to skip or worse have a malicious application\nprevent all audio playback by keeping the mutex locked.\n\nThe fix consists in using atomic operations when modifying the control\nblock flags.\n\nAlso fixed audio_track_cblk_t::framesReady() so that it doesn\u0027t block\nwhen called from AudioFlinger (only applies when a loop is active).\n\nChange-Id: Ibf0abb562ced3e9f64118afdd5036854bb959428\n" }, { "commit": "1703cdfee717b1b312bf8979816a9e2f16a82e5d", "tree": "eb9d3cfe8ffebe45679de91af6f9ffd676ca5067", "parents": [ "fbb1909036ba7de3d9fb8738daba60b357881153" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Mar 07 14:52:59 2011 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Mar 08 16:33:15 2011 -0800" }, "message": "Fix issue 3439872: video chat and bluetooth SCO\n\nThis change fixes the stability problems experienced when using\na bluetooth headset supporting both A2DP and SCO. Problems occur\nwhen starting the video chat at which time the A2DP output is being\nstopped to start SCO. At that time, active AudioTracks are invalidated\nby AudioFlinger so that a new AudioTrack binder interface can be\nrecreated by the client process on the new mixer thread with correct parameters.\nThe problem was that the process to restore the binder interface was not\nprotected against concurrent requests which caused 2 binder interfaces\nto be created sometimes. This could lead to permanent client deadlock\nif one of the client threads was waiting for a condition of the first\ncreated binder interface while the second one was created (as the AudioFlinger\nwould only signal conditions on the last one created).\nThis concurrent request situation is more likely to happen when a client\nuses the JAVA AudioTrack as the JNI implementation uses simultaneously the\nnative AudioTrack callback and write push mechanisms. By doing so, the code\nthat checks if the binder interface should be restored (in obtainBuffer()) is\nmuch more likely to be called concurrently from two different threads.\n\nThe fix consists in protecting the critical binder interface restore phase\nwith a flag in the AudioTrack control block. The first thread acting upon the binder\ninterface restore request will raise the flag and the second thread will just wait for\na condition to be signaled when the restore process is complete.\n\nAlso protected all accesses to the AudioTrack control block by a mutex to prevent\naccess while the track is being destroyed and restored. If a mutex cannot be held\n(e.g because we call a callback function), acquire a strong reference on the IAudioTrack\nto prevent its destruction while the cblk is being accessed.\n\nModified AudioTrack JNI to use GetByteArrayElements() instead of\nGetPrimitiveArrayCritical() when writing audio buffers. Entering a critical section would\ncause the JNI to abort if a mediaserver crash occurs during a write due to the AudioSystem\ncallback being called during the critical section when media server process restarts.\nAnyway with current JNI implementation, either versions do not copy data most of the times\nand the criticial version does not guaranty no data copy.\n\nThe same modifications have been made to AudioRecord.\n\nChange-Id: Idc5aa711a04c3eee180cdd03f44fe17f3c4dcb52\n" }, { "commit": "ce0febae177d1816ee7a6750ed9fba52472b9a71", "tree": "6e6283e217fc18d53b63652e1d3fe9da2487919f", "parents": [ "85d9b4225d024bb0d602b48bd6d5219cbebd7b8e" ], "author": { "name": "James Dong", "email": "jdong@google.com", "time": "Mon Nov 08 16:04:27 2010 -0800" }, "committer": { "name": "James Dong", "email": "jdong@google.com", "time": "Mon Nov 08 16:52:59 2010 -0800" }, "message": "Support extracting thumbnail from rotated video tracks\n\nChange-Id: Ife0a2536aaac5ff1efdf1035b9d2c892773ee16c\n" }, { "commit": "44d9848d6656777a18019223e0d35f2fcc67719a", "tree": "fe71003d3400dcbdfe078f8d8fad1b1fde960b2c", "parents": [ "922855214d0e8ae4159794d7f751f780b3243552" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Sep 30 16:12:31 2010 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Sep 30 17:21:23 2010 -0700" }, "message": "Issue 3032913: improve AudioTrack recovery time\n\nThis issue showed that when an AudioTrack underruns during a too long period\nof time and is therefore disabled by audioflinger mixer, it takes an additional\ndelay of up to 3 seconds to recover.\nThis fix adds a simple mechanism to recover immediately when the client application\nis ready to write data again in the AudioTrack buffer\n\nAlso throttle warnings on record overflows\n\nChange-Id: I8b2c71578dd134b9e60a15ee4d91b70f3799cb3d\n" }, { "commit": "be916aa1267e2e6b1c148f51d11bcbbc79cb864c", "tree": "7e2be6182cde7d023ae9afec28463d69d231d7bc", "parents": [ "c282e3eee921453fc9188705b4879d6289b71f9c" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Jun 01 23:49:17 2010 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Jun 03 03:21:53 2010 -0700" }, "message": "Issue 2667801: [Audio Effect Framework] AudioFlinger, AudioMixer AudioTrack modifications.\n\nFirst drop of audio framework modifications for audio effects support.\n\n- AudioTrack/AudioRecord:\nAdded support for auxiliary effects in AudioTrack\nAdded support for audio sessions\nFixed left right channel inversion in setVolume()\n\n- IAudioFlinger:\nAdded interface methods for effect enumeraiton and instantiation\nAdded support for audio sessions.\n\n- IAudioTrack:\nAdded method to attach auxiliary effect.\n\n- AudioFlinger\nCreated new classes to control effect engines in effect library and manage effect connections to tracks or\noutput mix:\n EffectModule: wrapper object controlling the effect engine implementation in the effect library. There\n\tis one EffectModule per instance of an effect in a given audio session\n EffectChain: group of effects associated to one audio session. There is one EffectChain per audio session.\n\tEffectChain for session 0 is for output mix effects, other chains are attached to audio tracks\n\twith same session ID. Each chain contains a variable number of EffectModules\n EffectHandle: implements the IEffect interface. There is one EffectHandle object for each application\n\tcontrolling (or using) an effect module. THe EffectModule maintians a list of EffectHandles.\n\nAdded support for effect modules and effect chains creation in PlaybackThread.\nmodified mixer thread loop to allow track volume control by effect modules and call effect processing.\n\n-AudioMixer\nEach track now specifies its output buffer used by mixer for accumulation\nModified mixer process functions to process tracks by groups of tracks with same buffer\nModified track process functions to support accumulation to auxiliary channel\n\nChange-Id: I26d5f7c9e070a89bdd383e1a659f8b7ca150379c\n" }, { "commit": "d1b449aad6c087a69f5ec66b7facb2845b73f1cb", "tree": "b16f7d28fdac76c90d66be56161116fe09822992", "parents": [ "ae77ffa16bda593fb3751e41d45327d867f3c8e1" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri May 14 03:26:45 2010 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon May 17 02:23:47 2010 -0700" }, "message": "Fix issue 2553359: Pandora does not work well with Passion deskdock / Cardock.\n\nThe problem is due to a too big difference between the buffer size used at the hardware interface and at the A2DP interface.\nWhen no resampling occurs we don\u0027t notice problems but the timing is very tight. As soon as resampling is activated, the AudioTrack underruns.\nThis is because the AudioTrack buffers are not resized when moving the AudioTrack from hardware to A2DP output.\nThe AudioTrack buffers are calculated based on a hardware output buffer size of 3072 bytes. Which is much less than the A2DP output buffer size (10240).\n\nThe solution consists in creating new tracks with new buffers in AudioFlinger when the A2DP output is opened\ninstead of just transfering active tracks from hardware output mixer thread to the new A2DP output mixer thread.\nTo avoid synchronization issues between mixer threads and client processes, this is done by invalidating tracks\nby setting a flag in their control block and having AudioTrack release the handle on this track (IAudioTrack)\nand create a new IAudioTrack when this flag is detected next time obtainBuffer() or start() is executed.\n\nAudioFlinger modifications:\n- invalidate the tracks when setStreamOutput() is called\n- make sure that notifications of output opening/closing and change of stream type to output mapping are sent synchronously to client process.\nThis is necessary so that AudioSystem has the new stream to output mapping when the AudioTrack detects the invalidate flag in the client process.\nPreviously their were sent when the corresponding thread loop was executed.\n\nAudioTrack modifications:\n- move frame count calculation and verification from set() to createTrack() so that is is updated every time a new IAudioTrack is created.\n- detect track invalidate flag in obtainBuffer() and start() and create a new IAudioTrack.\n\nAudioTrackShared modifications\n- group all flags (out, flowControlFlag, forceReady...) into a single bit filed to save space.\n\nChange-Id: I9ac26b6192230627d35084e1449640caaf7d56ee\n" }, { "commit": "45df6a2a25ff36606a5301155d8d97e9bf80d00b", "tree": "5a7046dc5d6e5727fa7e2ea01b42f9aa06f45201", "parents": [ "65ae665befd929efd1a7d1c9addac41c699f04b2" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri May 14 05:45:46 2010 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri May 14 05:45:46 2010 -0700" }, "message": "AudioFlinger: rename variables to clarify reference to track channel count or channel mask\n\nSome variables and structure members should be renamed to reflect the fact that they contain the\nnumber of channels in a track (channel count) or the actual channels used by a track (channel mask).\nEspecially member \"channels\" of track control block (struct audio_track_cblk_t) is actually the\nnumber of channels (channels count).\n\nChange-Id: I220c8dede9fc00c8a5693389e790073b6ed307b8\n" }, { "commit": "c2f1f07084818942352c6bbfb36af9b6b330eb4e", "tree": "88ac93be41edadd8cbfe6448e1421d5165883f59", "parents": [ "a64c8c79af1a15911c55306d83a797fa50969f77" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Jul 17 12:17:14 2009 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Jul 23 06:03:39 2009 -0700" }, "message": "Fix issue 1795088 Improve audio routing code\n\nInitial commit for review.\nIntegrated comments after patch set 1 review.\nFixed lockup in AudioFlinger::ThreadBase::exit()\nFixed lockup when playing tone with AudioPlocyService startTone()\n" }, { "commit": "573266210fb2b2e7d86fbd46d0dfe16763611d91", "tree": "f7565b32ee2d59d5098dd165658c0aefa65f1fc4", "parents": [ "76aa28371e93829927c66f1a48e39e18e01d704d" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Jul 07 07:10:45 2009 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Jul 07 07:10:45 2009 -0700" }, "message": "Fix issue 1743700: AudioTrack: setPlaybackRate can not set the playback rate to twice of the ouputSR\n\nStore sample rate on 32 bits instead of 16 bits in audio_track_cblk_t.\nRemoved sampleRate() methods from AudioTrack and AudioRecord: replaced by getSampleRate().\nAudioTrack::setSampleRate() no returns a status.\n" }, { "commit": "1179bc9b0e3d17c984e8f4ad38561c049dd102fa", "tree": "1fd069efb58601650c80f77da0e2dc7d93641068", "parents": [ "c048cae0367db6fbb4fe1127be5011910713d4ad" ], "author": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Wed Mar 18 17:39:46 2009 -0700" }, "committer": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Wed Mar 18 17:39:46 2009 -0700" }, "message": "auto import from //branches/cupcake_rel/...@140373\n" }, { "commit": "89fa4ad53f2f4d57adbc97ae1149fc00c9b6f3c5", "tree": "28d26f7b71e943e25c7da6e8043d79b7b8d9cf7b", "parents": [ "15f767b960b38059a74a42a33e16d8df2aec8bc1" ], "author": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Tue Mar 03 19:31:44 2009 -0800" }, "committer": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Tue Mar 03 19:31:44 2009 -0800" }, "message": "auto import from //depot/cupcake/@135843\n" }, { "commit": "15f767b960b38059a74a42a33e16d8df2aec8bc1", "tree": "4b825dc642cb6eb9a060e54bf8d69288fbee4904", "parents": [ "ad04d9201452001dbaac4349f084cc9316190b89" ], "author": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Tue Mar 03 18:28:45 2009 -0800" }, "committer": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Tue Mar 03 18:28:45 2009 -0800" }, "message": "auto import from //depot/cupcake/@135843\n" }, { "commit": "e5198b620a9a208ec59ea8457282404725f8ff6e", "tree": "2ff913c152607cd9d58f0e6ba620461090be7541", "parents": [ "54ffb8abf94365496a2dbf5d792b6977d6a8ef06" ], "author": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Tue Jan 20 14:03:58 2009 -0800" }, "committer": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Tue Jan 20 14:03:58 2009 -0800" }, "message": "auto import from //branches/cupcake/...@127101\n" }, { "commit": "7b5eb023f8d87cca6d830ae6c11c6aadbe02aca8", "tree": "d9838a098f868ee2fcffc20627a8c9a33cb23377", "parents": [ "2729ea9262ca60d93047e984739887cfc89e82eb" ], "author": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Wed Dec 17 18:05:43 2008 -0800" }, "committer": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Wed Dec 17 18:05:43 2008 -0800" }, "message": "Code drop from //branches/cupcake/...@124589\n" }, { "commit": "2729ea9262ca60d93047e984739887cfc89e82eb", "tree": "25962a3f1c5f70586af45c926c037e03c643857f", "parents": [], "author": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Tue Oct 21 07:00:00 2008 -0700" }, "committer": { "name": "The Android Open Source Project", "email": "initial-contribution@android.com", "time": "Tue Oct 21 07:00:00 2008 -0700" }, "message": "Initial Contribution\n" } ] }