)]}' { "log": [ { "commit": "375283dd6031a71e44fc17efad05b741302c78e5", "tree": "da9cbab41fb7a3610201cfe17d09d7a5e9d3a37e", "parents": [ "b77e219423638bd958a5673dba017ea72aaa2e39" ], "author": { "name": "jiabin", "email": "jiabin@google.com", "time": "Fri Aug 21 18:14:43 2020 -0700" }, "committer": { "name": "Jiabin Huang", "email": "jiabin@google.com", "time": "Tue Sep 15 18:02:58 2020 +0000" }, "message": "Add package name when creating AudioTrack.\n\nThe package name can be used when starting an external vibration, which\nis used by vibrator service to check if the application has the\npermission to start vibration.\n\nBug: 165910728\nBug: 162343845\nTest: atest AudioTrackTest MediaPlayerTest\nTest: start audio-coupled-haptic playback\nChange-Id: I6d12c4e141c037ee94ee79cbb9bf537421ec9507\nMerged-In: I6d12c4e141c037ee94ee79cbb9bf537421ec9507\n" }, { "commit": "9447b48f36b1bf26645e6256dd29224b6a398161", "tree": "62bc6e0895af051df03446dc2c5ecca8dd288d0f", "parents": [ "37e93d1bc4df8b2d9682c48236bb4dae87d6dd1b", "d1fa2375194096937bbdfd2def97969c4336191b" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Sep 03 00:53:20 2020 +0000" }, "committer": { "name": "Automerger Merge Worker", "email": "android-build-automerger-merge-worker@system.gserviceaccount.com", "time": "Thu Sep 03 00:53:20 2020 +0000" }, "message": "Merge \"audio flinger: handle race condition in AudioRecord creation\" into rvc-qpr-dev am: d1fa237519\n\nOriginal change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/12484593\n\nChange-Id: I5e330127a6ef2efe394b06524af5ae20f01a71bc\n" }, { "commit": "717bc28fc5959c0edb2bbd86692ba5164893f719", "tree": "9f55d492e1032f4224dc3420345868cf8d613456", "parents": [ "0b3f254028404b4772e02f9d1cdfcefba0e264b4" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Aug 21 17:10:39 2020 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Aug 28 22:51:36 2020 +0000" }, "message": "audio flinger: handle race condition in AudioRecord creation\n\nA rare race condition can cause a failure to create an AudioRecord if\ntwo creation requests are processed concurrently and thread preemption happens\nat a specific time within the sequence between audioflinger and\naudio policy manager.\n\nThis CL implements a retry mechanism on client side when this particular failure\nis detected.\n\nBug: 161656190\nTest: Regression tests for audio capture use cases.\nTest: CTS tests for AudiRecord\nChange-Id: I155ee4899baeec21e10a1141cb9e323cbc8aa23b\n(cherry picked from commit d52a28ca2c19db7d10bc6c4639a8bd78e1afd5f1)\n" }, { "commit": "19fa75e6821383146b7be540e3a7264131fb2fc7", "tree": "4aeae0bf5c738066fa4bae84e3a5a48bef824d21", "parents": [ "5c029e5511f5de207c691945eab0675714b7431a", "43fd672bd7cfd9eef13012c545ad4630d12e325c" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue Jun 23 00:42:30 2020 +0000" }, "committer": { "name": "Automerger Merge Worker", "email": "android-build-automerger-merge-worker@system.gserviceaccount.com", "time": "Tue Jun 23 00:42:30 2020 +0000" }, "message": "Merge \"AudioFlinger: Fix spurious underrun logging\" into rvc-dev am: 54cc1e3c43 am: 43fd672bd7\n\nOriginal change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/11956760\n\nChange-Id: Iabdf6f2c67f14ad564cdc3c0eba2fd5586fca6af\n" }, { "commit": "fc62917d0adf02637011960ecd2ac0c3d20317a7", "tree": "7e9350c96601dca8ed5391324009e0d3b1fbb819", "parents": [ "f3f98cc259b9ab73a1069e0a9bb63b3de8288dab" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Jun 22 10:06:23 2020 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Jun 22 10:41:48 2020 -0700" }, "message": "AudioFlinger: Fix spurious underrun logging\n\nRemoves false positive errors in metrics and debugging.\n\nTest: Play Music, check underruns\nBug: 159620406\nChange-Id: I008381bbf3aae13c0fe8289a0eeec47b7780e587\n" }, { "commit": "e97fe1bb2f76ff9a1bc04a292dc117712f8c21f7", "tree": "8563dcb399d7d4eec4fc0940348281f2c5ca3242", "parents": [ "80a1fdc4072da2cb596cc3a38361c3dd23664c1d", "91dcb74b24fa28658baba13a1658d2ecbeca065d" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Jun 12 19:20:16 2020 +0000" }, "committer": { "name": "Automerger Merge Worker", "email": "android-build-automerger-merge-worker@system.gserviceaccount.com", "time": "Fri Jun 12 19:20:16 2020 +0000" }, "message": "MediaMetrics: Prepare audio device usage protos for statsd am: a629bd13e9 am: 91dcb74b24\n\nOriginal change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/11835151\n\nChange-Id: I16c8e24115d1ae0dd2fe7f943af3a5d3fd1d82aa\n" }, { "commit": "a629bd13e9f224b8472365135757f0676636f8e2", "tree": "4bdce444fd726634f20803fd279ab254a478c262", "parents": [ "752adf7f1a340cbb6599fcff1842cf3659c75612" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Jun 05 16:03:53 2020 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Jun 12 10:43:37 2020 -0700" }, "message": "MediaMetrics: Prepare audio device usage protos for statsd\n\nMediaMetricsAudioRecordDeviceUsageReported\nMediaMetricsAudioThreadDeviceUsageReported\nMediaMetricsAudioTrackDeviceUsageReported\nMediaMetricsAudioDeviceConnectionReported\n\nAdd track traits to identify static AudioTracks.\nFix AudioTrack/AudioRecord flag communication.\nMove debug logging to verbose (delivered protos show in dumpsys).\nUnknown strings which fail validation become empty.\nTest statsd type validation.\n\nTest: atest mediametrics_tests\nTest: adb shell dumpsys media.metrics (after BT connection)\nTest: statsd_testdrive (294-297)\nBug: 149850236\nChange-Id: I53c5d18a346af54941007b9938a8ab10e34b7038\n" }, { "commit": "b26a9631061a91cb05df425f1d8641dbc54481e6", "tree": "9ff99290c57635e25ef9eb64bd7e908974f95b55", "parents": [ "2f39d8816b9e3b773ab075ef14a4cba4151a35fe", "8b3295e9db03306bcbacf17b184124e277601802" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu May 28 02:41:21 2020 +0000" }, "committer": { "name": "Automerger Merge Worker", "email": "android-build-automerger-merge-worker@system.gserviceaccount.com", "time": "Thu May 28 02:41:21 2020 +0000" }, "message": "Merge \"Tracks: Stabilize the latency/startup metrics for BT\" into rvc-dev am: 3fdcc34690 am: 8b3295e9db\n\nChange-Id: I3a675b0e836a08989706826a8441abe89a2dad2c\n" }, { "commit": "6292112eca818eab9fa5056bc4a28092812b291b", "tree": "1e37d150cd5325d37b7d1b115c18fc7e4e1c0d0c", "parents": [ "b4a3b65d297a8436ba48138ce7ce2ba6b7a652b8" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon May 18 10:47:31 2020 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed May 27 11:47:47 2020 -0700" }, "message": "Tracks: Stabilize the latency/startup metrics for BT\n\nUse kernel timestamps instead of server timestamps.\nPostpone latency / startup collection until later\nto ensure BT is stabilized.\n\nTest: adb shell dumpsys media.metrics\nBug: 149850236\nChange-Id: Ie4e0658cedec13000f7b713407630e273df391b7\n" }, { "commit": "0692c1c60e44b2e8e844cffe5118812f200aa6af", "tree": "7c8678fb84dcec99aa1408dc1cfc66b0c652e6bc", "parents": [ "1a383b651501f4eaf154238b2787e382064adb24", "2e34a502ab975ad9a22ae7e0ea631cd62ce310a0" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed May 13 22:49:58 2020 +0000" }, "committer": { "name": "Automerger Merge Worker", "email": "android-build-automerger-merge-worker@system.gserviceaccount.com", "time": "Wed May 13 22:49:58 2020 +0000" }, "message": "Merge \"MediaMetrics: Prepare statistics for protos\" into rvc-dev am: 09869a1c36 am: 2e34a502ab\n\nChange-Id: I076d342ccf603a92084c42390a11aa8f5c44c0c8\n" }, { "commit": "ea84038803580b4c35ce4cb60918c0c384127417", "tree": "d8c91402af2f630e4f1c36f7798d161767d01301", "parents": [ "38e65911a65a51edf2efd5ca63c4e00daac24852" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue May 05 21:50:17 2020 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed May 13 11:15:49 2020 -0700" }, "message": "MediaMetrics: Prepare statistics for protos\n\n1) Add AudioRecord to Analytics trigger.\n2) Log RecordThread statistics.\n3) Rename DURATIONNS to EXECUTETIMENS to avoid confusion.\n4) Use streamType from server not client.\n5) Split MMAP types to MMAP_PLAYBACK and MMAP_CAPTURE.\n6) Use string flags for AudioTrack and AudioRecord.\n7) Compute underrun frames for MixerThread.\n8) Log BT connection state.\n\nTest: adb shell dumpsys media.metrics\nTest: atest mediametrics_tests\nTest: adb shell dumpsys stats --metadata\nBug: 149850236\nChange-Id: Iacf3f117c00ec9f377e62862080252aef9dc084f\n" }, { "commit": "5abdc4797594aa459cd2dc0d0793826ccf242169", "tree": "1a7124140f7cac8dfdc4c119af5368af243b41e6", "parents": [ "16031a2e835aef650a1411b970023b484698846d", "a1d771f00c3661864b0e6f69c44d0dc39d94a0a1" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri May 01 20:27:08 2020 +0000" }, "committer": { "name": "Automerger Merge Worker", "email": "android-build-automerger-merge-worker@system.gserviceaccount.com", "time": "Fri May 01 20:27:08 2020 +0000" }, "message": "Merge \"TrackMetrics: Add device-based statistics for audio\" into rvc-dev am: 31300b43da am: a1d771f00c\n\nChange-Id: I86cdc347ae3cf748a71c210c2c7cd4942f059edc\n" }, { "commit": "c2b11cbcb88c5970fb4ec6ff8a562b1465dfa70a", "tree": "dedee4f6e9c95af20ef76e79bddbc79ed850781a", "parents": [ "199012b0c66d555207d81fedb7cce16305c865f8" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed Apr 22 09:04:01 2020 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Apr 30 17:18:26 2020 -0700" }, "message": "TrackMetrics: Add device-based statistics for audio\n\nCompute summary statistics based on the current device\nrather than the entire AudioTrack or AudioRecord duration.\n\nTest: adb shell dumpsys media.metrics\nBug: 149850236\nChange-Id: Ia3a5707c43c4530f5a6ac90f52901bd2e0bd0bab\n" }, { "commit": "f9a222bdf076fc733dd02b06a4dea13469624535", "tree": "77e98ccb24caf0a5fb3693b8c538b9c33bf14717", "parents": [ "4da69588f7216fa2e175267147f7ef644695478d", "91ac0157d9515374918950f6d56e3abc9dcbaf3f" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue Apr 28 15:46:38 2020 +0000" }, "committer": { "name": "Automerger Merge Worker", "email": "android-build-automerger-merge-worker@system.gserviceaccount.com", "time": "Tue Apr 28 15:46:38 2020 +0000" }, "message": "Merge \"MediaMetrics: Use AllowUid property for services to control client access\" into rvc-dev am: 5559ae7c9d am: 91ac0157d9\n\nChange-Id: I79a14803bd9748d0ce5c6d00f7ccc94b61b94e4f\n" }, { "commit": "d203eb6b3cd99af3c9616df2bd0d982796b0f918", "tree": "0f2cfa6c24a30d32913de484c6dd52514b37a9a3", "parents": [ "2578a95a57dd9796b55095dc22b8171dd5da73c3" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Apr 27 09:12:46 2020 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Apr 27 23:42:41 2020 -0700" }, "message": "MediaMetrics: Use AllowUid property for services to control client access\n\nRemove implicit first client uid access.\n\nTest: adb shell dumpsys media.metrics\nBug: 149850236\nChange-Id: I4f953db4688bf5f174c3e1611f492f31d2926e96\n" }, { "commit": "f029612a8139de19858ef06525f9fe63bc7cee77", "tree": "ed4bbab65c302fba4c208ab2b9ea523093fd5a57", "parents": [ "7faaf4bc0076863c8b99432afcc71153ef0491d5", "13367c57dd73bf01d7a007b51efc0dab68826d03" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Mar 27 15:33:08 2020 +0000" }, "committer": { "name": "Automerger Merge Worker", "email": "android-build-automerger-merge-worker@system.gserviceaccount.com", "time": "Fri Mar 27 15:33:08 2020 +0000" }, "message": "Merge \"PatchTrack: Ensure enough data to write once\" am: 4943e65ba7 am: 13367c57dd\n\nChange-Id: I83be105022a7905bbeb9b5011b11209d904019a4\n" }, { "commit": "f37f9980992e84e0abb8445af8c94b041eef4d5f", "tree": "487f348b634b361e576eeda8627f95590b138ef8", "parents": [ "b1d2e747ede4994547eae75b2b929896ac6a7e8e" ], "author": { "name": "naoki miyazu", "email": "naoki.x.miyazu@sonymobile.com", "time": "Thu Nov 28 11:18:18 2019 +0900" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue Mar 24 20:12:51 2020 +0000" }, "message": "PatchTrack: Ensure enough data to write once\n\nUnderrun can occur for AudioPlaybackCapture\nin the secondary PlaybackThread.\n\nWhen the primary PlaybackThread is blocked by any reason\nduring playback, e.g. A2DP suspend, interceptBuffer is also\nblocked and data will not be provided to the PatchRecord buffer.\nBut the secondary PlaybackThread is still running,\nexhausting the data in the PatchTrack.\n\nAs mFillingUpStatus of the PatchTrack is still FS_ACTIVE,\nthe secondary PlaybackThread continues even though framesReady()\nis not filled to threshold (mFrameCountToBeReady).\nSo, underrun happens repeatedly.\n\nTo avoid this issue, check if the PatchTrack has enough data in\nreleaseBuffer(); if not, change mFillingUpStatus to FS_FILLING,\nand wait for the buffer to fill.\n\nBug: 144009535\nChange-Id: Ifcbaea05228064284dfaa9b629c8860b1fbe977d\n" }, { "commit": "b68f5ebb53434e800c43715127e2d1032e3a9b54", "tree": "7e1c75e02cb63d4c9d1f0b7dc3322001e1463b33", "parents": [ "3b4c1f0320efc7a54c4ab811727c432cda73dd37" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue Dec 03 16:49:17 2019 -0800" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue Jan 28 13:03:22 2020 -0800" }, "message": "MediaMetrics: Add AudioFlinger logging\n\nTest: atest mediametrics_tests\nTest: mediametrics dumpsys\nBug: 138583596\nChange-Id: I2086c5a780744d409052fde4c5ebc1dc92d95ea8\n" }, { "commit": "e2e830fd3b444c27abf258718cfba5ad8d209ec4", "tree": "4e0642cbfe2b8de89c0abab3b9fd9238515fcabd", "parents": [ "eff0d0824c9c087c00153dece27e5dc3f8cd0c15" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue Dec 03 12:54:46 2019 -0800" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue Dec 03 13:23:11 2019 -0800" }, "message": "AudioFlinger: Fix incorrect underrun count on track pause\n\nAlso add track state name helper function for logging.\n\nTest: Play video clip then pause and resume several times\nBug: 145613591\nChange-Id: If4fc2565da9b3d4964fb9fdb6f5854ff6fbadfb8\n" }, { "commit": "4f3ac4d0555229a41273107ff0987018ef6b69e1", "tree": "b17a402fe68ef0b1c80d234d14ff6d6b7f45715e", "parents": [ "bde7aa5551cc67c2b47f1f67cb66c56b824c1a65", "d6f189b4c790f88d72c2227cebbbb8a59cbe8a4e" ], "author": { "name": "Xin Li", "email": "delphij@google.com", "time": "Fri Nov 22 10:56:08 2019 -0800" }, "committer": { "name": "Xin Li", "email": "delphij@google.com", "time": "Sun Nov 24 05:16:33 2019 +0000" }, "message": "DO NOT MERGE - Merge QQ1A.191205.011 into stage-aosp-master\n\nBug: 144955631\nMerged-In: I242aebe71ffe7d348eca213252f1691fd444703e\nChange-Id: I40ae7732b4435dc9aa9a3edb57275505e4112a4c\n" }, { "commit": "58a0dd832177e58a53fc428ab3e9ab86156b576d", "tree": "d4db6d477b38b251b9a1a14031791daa4cf2cebe", "parents": [ "9f03917ce011e4c860306d42c3f3cc70d2061190" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Thu Oct 24 12:42:17 2019 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Oct 25 09:35:47 2019 -0700" }, "message": "Audio policy: modify permission needed to capture from source FM_TUNER\n\nCapturing from source AUDIO_SOURCE_FM_TUNER is possible only via @SystemApi\non one end and does not capture from an actual on the other end.\nAccordingly, do not check android.permission.RECORD_AUDIO anymore but\ncheck privileged permission android.permission.CAPTURE_AUDIO_OUTPUT instead.\nAlso bypass App Ops OP_RECORD_AUDIO check which is applicable only to capture from\nmicrophones.\n\nAlso fix audio recording permission check in MediaRecordClient to use\nrecordingAllowed() from ServiceUtilities\n\nBug: 135717621\nTest: CTS tests for AudioRecord\nChange-Id: Ibb1d72f018d2e3ceee195338f2e262183eee2a23\n" }, { "commit": "935876110cb820240fcc7ad4a3ada0c1e8cb18e3", "tree": "21d6f0b632ed473e3e4f361388984aa10990b3aa", "parents": [ "58a25b54c7247c0e39ce156e4fbeebc0253ee6f6", "3686203afbfa2e3a59255a9280e3f9175d3fdd7c" ], "author": { "name": "TreeHugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Oct 16 22:54:39 2019 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Wed Oct 16 22:54:39 2019 +0000" }, "message": "Merge \"Propagate sample rate to static audio track\"" }, { "commit": "b0de569edd2d27b8dae9e3048f9c131b6ebc2d45", "tree": "90e0f48fde7d3f07367b62be0c2f98527fc327d6", "parents": [ "bcb57f62b0f75630a5c1199c78f5af311934b6ce" ], "author": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Tue Aug 20 15:42:04 2019 -0700" }, "committer": { "name": "android-build-team Robot", "email": "android-build-team-robot@google.com", "time": "Fri Oct 11 02:34:35 2019 +0000" }, "message": "AudioFlinger: enforce OP_RECORD_AUDIO during recording\n\n Fix issue where RecordTrack silencing didn\u0027t silence the\nfull buffer: the memset to 0 was using the RecordThread frame\nsize, not the RecordTrack frame size.\n OP_RECORD_AUDIO was only enforced at the start of a recording\nwhich would fail if not granted. This patch silences the recording\n(i.e. silence is recorded instead) when it is lost, and undoes that\nwhen granted again. This requires:\n- propagating the package name of the client to the RecordTrack class\n- registering an appOp callback in RecordTrack (through a new\n OpRecordAudioMonitor class) to (un)silence the recording\n- update the isSilenced() method to take into account the appOp.\n\nBug: 138968594\nBug: 138636979\nTest: run app that records audio, then \"adb shell appops __pack_name__ 27 2\"\n and verify recording is silent after that.\n\nChange-Id: Ib33f5b592185a67204997213bab1ac2594d90d37\n(cherry picked from commit ddf87effa4d80f413a80fde861f73877ccd80599)\nMerged-In: Ib33f5b592185a67204997213bab1ac2594d90d37\n(cherry picked from commit 7307293e8befafa12b244b74cd3c8cb84b27997c)\n" }, { "commit": "20a44f84f1a80ba1464ca35b62417e33ed88ac29", "tree": "406a9c22867b2928338aa3e7eeae7d1f313b04eb", "parents": [ "229c543d081cc29f1f07df8d0b1657cdb71e0c15" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Sep 18 11:24:52 2019 +0100" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Thu Oct 10 13:34:23 2019 +0100" }, "message": "Fix secondary output under\u0026over run\n\nThere were three issues with the implementation\nof Audio Playback Capture patches on secondary outputs.\n\n1) Patch tracks were always forced to be ready, even if there were not\nenough audio to be played. That led to playing partial buffers if the\nsource output had smaller (time wise) buffer than the secondary output.\n\nThis is fixed by avoiding setting CBLK_FORCEREADY on every buffer push\nfrom the primary output thread.\n\n2) After 1 is fixed, the patch tracks now behave like regular (non fast)\ntracks. This means that the track only starts when it is full.\nThat leads to overrun on startup as the primary and secondary outputs\nusually do not have the same write period.\n\nWhat makes the issue worst is that Remote submix has a fixed buffer in bytes,\nso its write period changes depending on sample rate contrary to the\nprimary HAL which pulls with fixed time periods (usually 20/40ms).\n\nThis patch solution is to introduce a ready threshold. The track will be\nconsidered ready to be active if there are more than frames in its buffer.\n\nTo avoid changing previous latency behaviour except for APC,\nlegacy tracks have this threshold set to their buffer size and\nnon APC patch tracks have it set to 1, similar to setting\nCBLK_FORCEREADY.\n\n3) The patch track buffer size calculation of the patch track did not take\ninto account primary and secondary output could be of different sampling\nrate.\nThis mean that the patch track buffer was usually too small as the\nsecondary output usually had smaller sampling rate (Live caption\nrequests 16kHz) than the track (music usually plays in 48kHz).\nThis made the two problems even worst.\n\nThis was easily fixed by scaling the buffer size with each side sample rate.\n\nBug: 136691300\nTest: play audio\n adb shell audiorecorder --target /data/file1.raw\n # Check that the recorded file has no underrun\nChange-Id: Ib2846b2827afd1b953d4a25acb18c7cadf57cd3e\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "40715983f3d12e4e287260244559667001db967a", "tree": "4000f689070b321cdf478d01c5f214d237d522c2", "parents": [ "40143b19a0dbb2cf3d24f100bf3844f8a4a60f8d", "01c7d9ee1b170a516e20b9c7aad3f9d1db982c55" ], "author": { "name": "TreeHugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Oct 10 12:27:10 2019 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Oct 10 12:27:10 2019 +0000" }, "message": "Merge \"Fix secondary output under\u0026over run\"" }, { "commit": "3686203afbfa2e3a59255a9280e3f9175d3fdd7c", "tree": "fae91b412fdf5f69ff822fff96dc98feb24c7d6d", "parents": [ "40143b19a0dbb2cf3d24f100bf3844f8a4a60f8d" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Thu Oct 10 10:52:19 2019 +0100" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Thu Oct 10 10:00:10 2019 +0000" }, "message": "Propagate sample rate to static audio track\n\nBug: 136691300\nTest: playback on static buffer track\nChange-Id: Ib11c39e381a1b78450ab8a688310453aa3f1daf7\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "01c7d9ee1b170a516e20b9c7aad3f9d1db982c55", "tree": "101e342ee5d35403385d64c6a7f8e4108ef1dfc3", "parents": [ "f73f877182014e662c4c3ab650724975c884839e" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Sep 18 11:24:52 2019 +0100" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Thu Oct 10 10:52:05 2019 +0100" }, "message": "Fix secondary output under\u0026over run\n\nThere were three issues with the implementation\nof Audio Playback Capture patches on secondary outputs.\n\n1) Patch tracks were always forced to be ready, even if there were not\nenough audio to be played. That led to playing partial buffers if the\nsource output had smaller (time wise) buffer than the secondary output.\n\nThis is fixed by avoiding setting CBLK_FORCEREADY on every buffer push\nfrom the primary output thread.\n\n2) After 1 is fixed, the patch tracks now behave like regular (non fast)\ntracks. This means that the track only starts when it is full.\nThat leads to overrun on startup as the primary and secondary outputs\nusually do not have the same write period.\n\nWhat makes the issue worst is that Remote submix has a fixed buffer in bytes,\nso its write period changes depending on sample rate contrary to the\nprimary HAL which pulls with fixed time periods (usually 20/40ms).\n\nThis patch solution is to introduce a ready threshold. The track will be\nconsidered ready to be active if there are more than frames in its buffer.\n\nTo avoid changing previous latency behaviour except for APC,\nlegacy tracks have this threshold set to their buffer size and\nnon APC patch tracks have it set to 1, similar to setting\nCBLK_FORCEREADY.\n\n3) The patch track buffer size calculation of the patch track did not take\ninto account primary and secondary output could be of different sampling\nrate.\nThis mean that the patch track buffer was usually too small as the\nsecondary output usually had smaller sampling rate (Live caption\nrequests 16kHz) than the track (music usually plays in 48kHz).\nThis made the two problems even worst.\n\nThis was easily fixed by scaling the buffer size with each side sample rate.\n\nBug: 136691300\nTest: play audio\n adb shell audiorecorder --target /data/file1.raw\n # Check that the recorded file has no underrun\nChange-Id: Ib2846b2827afd1b953d4a25acb18c7cadf57cd3e\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "91beec3e38677fc9d168c3e774755ace009e60db", "tree": "6016c5b7ff6fca19abf6bdd4b8e82cbd0bfa71ff", "parents": [ "48f1d4579f9fe1cd6ff672fe9daa27fec83d82c7" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Mon Oct 07 09:57:15 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Mon Oct 07 12:50:58 2019 -0700" }, "message": "PassthruPatchRecord: fix null timeOut case in obtainBuffer\n\nProperly handle \u0027timeOut \u003d\u003d nullptr\u0027 case in\nPassthruPatchRecord::obtainBuffer.\n\nBug: 117564323\nTest: make\nMerged-In: Id352e47682c0992b817f26b3b594ec5913e663be\nChange-Id: Id352e47682c0992b817f26b3b594ec5913e663be\n" }, { "commit": "4de4997e66a1b3d2b6021942058c4f49824ae8eb", "tree": "577e1d41eedd6241b38908d4a4ab9132b8e3cfbe", "parents": [ "7ae95c1a44390ccfeb5bdd1d5e7fa4edd6d91725" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Mon Oct 07 09:53:58 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Mon Oct 07 12:49:58 2019 -0700" }, "message": "PassthruPatchRecord: fix null timeOut case in obtainBuffer\n\nProperly handle \u0027timeOut \u003d\u003d nullptr\u0027 case in\nPassthruPatchRecord::obtainBuffer.\n\nBug: 117564323\nTest: make\nChange-Id: Id352e47682c0992b817f26b3b594ec5913e663be\n" }, { "commit": "e6eb348b793736f1f4e441b53167d0c09e614f6b", "tree": "d0abca0d2c30e2279a6e631a5c25b4430b7c73e9", "parents": [ "af28887c617eca259444f743ef83eda85d52d31e" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Sep 25 14:05:29 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Fri Oct 04 15:59:52 2019 -0700" }, "message": "Add PassthruPatchRecord for low latency software patches\n\nImplement a subclass of PatchRecord that uses PatchTrack\u0027s\nthread for reading from HAL. This eliminates the need for\nbuffering that adds latency.\n\nThe only modification needed for PatchTrack is to indicate\nunlimited amount of available frames to the playback thread.\nThis is to prevent PatchTrack from being deactivated by\nDirectOutputThread due to lack of frames available.\n\nRecordThread believes it reads audio data on its thread,\nand manages timestamps as usual. The data that it \"reads\"\nand passes to PassthruPatchRecord is discarded by the latter.\n\nBug: 117564323\nTest: with MSD module\nMerged-In: I376656e3c791e91e2196331ecdf2b425697c4e18\nChange-Id: I376656e3c791e91e2196331ecdf2b425697c4e18\n" }, { "commit": "d368d9166ea7df77479a2908e3a7dbf4c5f21c12", "tree": "051128854a67d74eee4d073b113b2b76ac6cf681", "parents": [ "5269849effaa40a96ccffc214efaf804f3378bd0" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Sep 25 14:59:54 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Fri Oct 04 14:06:22 2019 -0700" }, "message": "Move PlaybackThread::Track::writeFrames to PatchRecord\n\nThis code logically belongs to PatchRecord because it emulates\nobtaining recorded data from a client.\n\nTest: make\nMerged-In: Icba4e33d9d0ca57e6ad964aaf3209e7db766f284\nChange-Id: Icba4e33d9d0ca57e6ad964aaf3209e7db766f284\n" }, { "commit": "5269849effaa40a96ccffc214efaf804f3378bd0", "tree": "1801581cde4d446ca5e56b9fb8012a81be9659e7", "parents": [ "32f0d1689bd2eea6f43d7fb53faaedc23bd101d7" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Sep 04 11:38:47 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Fri Oct 04 14:06:22 2019 -0700" }, "message": "audioflinger: Add tracing of buffer frames to PatchTrack/Record\n\nSeeing requested / obtained buffer frame counts in trace\nhelps to identify discontinuities in software patch pipeline.\n\nThere is no need to trace requested frame count for PatchRecord\nas it always requests as much as possible (\"-1\" frameCount).\n\nBug: 117564323\nTest: with software patch active,\n external/chromium-trace/systrace.py audio\n\nMerged-In: I78400f6cc1d1940b3290fb23bb1aacfbffe042e5\nChange-Id: I78400f6cc1d1940b3290fb23bb1aacfbffe042e5\n" }, { "commit": "caf5994f652e33336e7bded702c30e29c1d85eb6", "tree": "9c7dd60e484c3d42ae402b504355d3872226be50", "parents": [ "2534b388b5ab10712b9b6c24a49e3dc827ce0e00" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Sep 25 14:05:29 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Thu Oct 03 12:20:34 2019 -0700" }, "message": "Add PassthruPatchRecord for low latency software patches\n\nImplement a subclass of PatchRecord that uses PatchTrack\u0027s\nthread for reading from HAL. This eliminates the need for\nbuffering that adds latency.\n\nThe only modification needed for PatchTrack is to indicate\nunlimited amount of available frames to the playback thread.\nThis is to prevent PatchTrack from being deactivated by\nDirectOutputThread due to lack of frames available.\n\nRecordThread believes it reads audio data on its thread,\nand manages timestamps as usual. The data that it \"reads\"\nand passes to PassthruPatchRecord is discarded by the latter.\n\nBug: 117564323\nTest: MS12 MSD, AOSP MSD\nChange-Id: I376656e3c791e91e2196331ecdf2b425697c4e18\n" }, { "commit": "8296c2522746d3500c024121577007dd5e9e685c", "tree": "7ee1d84a9f09fc729375bdbfcd2a47d612d85415", "parents": [ "e3cdab3c40cb71284a0e186d7d5a82257e7d34a1" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Sep 25 14:59:54 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Tue Oct 01 16:28:01 2019 -0700" }, "message": "Move PlaybackThread::Track::writeFrames to PatchRecord\n\nThis code logically belongs to PatchRecord because it emulates\nobtaining recorded data from a client.\n\nTest: make\nChange-Id: Icba4e33d9d0ca57e6ad964aaf3209e7db766f284\n" }, { "commit": "7dd39723b34c8378fc5eabb4d4ccacf32f55ffe2", "tree": "222de47bcb84931d714d250fe51d3249771edc18", "parents": [ "61d4a093f4b4041b145fbd7a3ab5ec04679f87d9" ], "author": { "name": "Ytai Ben-Tsvi", "email": "ytai@google.com", "time": "Thu Sep 05 15:14:30 2019 -0700" }, "committer": { "name": "Ytai Ben-Tsvi", "email": "ytai@google.com", "time": "Fri Sep 13 10:59:51 2019 -0700" }, "message": "Improve visibility of IMemory security risks\n\nThis change renames the IMemory raw pointer accessors to\nunsecure*() to make it apparent to coders and code reviewers\nthat the returned buffer may potentially be shared with\nuntrusted processes, who may, after the fact, attempt to\nread and/or modify the contents. This may lead to hard to\nfind security bugs and hopefully the rename makes it harder\nto forget.\n\nThe change also attempts to fix all the callsites to make\neverything build correctly, but in the processes, wherever the\ncallsite code was not obviously secure, I added a TODO requesting\nthe owners to either document why it\u0027s secure or to change the\ncode. Apologies in advance to the owners if there are some false\npositives here - I don\u0027t have enough context to reason about all\nthe different callsites.\n\nTest: Completely syntactic change. Made sure code still builds.\nChange-Id: I5fb99aa797c488406083178a6b05355d98710d3b\n" }, { "commit": "9ff75796dfece494696b18837692cfc8838e1a39", "tree": "c014fe64b039ca613ceeb2f2331adc0dea0d6e2a", "parents": [ "3ce82b056a93012e90de5c906b60e99cf40bad7e", "938be41e28c3f508a14624cc01dc348b60f38711" ], "author": { "name": "TreeHugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Sep 05 01:10:09 2019 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Sep 05 01:10:09 2019 +0000" }, "message": "Merge \"audioflinger: Add tracing of buffer frames to PatchTrack/Record\"" }, { "commit": "938be41e28c3f508a14624cc01dc348b60f38711", "tree": "4725d79abb3eef2c072442bd0ca3b5ded8fd1b45", "parents": [ "cc57ed2b6b79374934755018f6ca4b15eff7cf6b" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Sep 04 11:38:47 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Sep 04 11:38:47 2019 -0700" }, "message": "audioflinger: Add tracing of buffer frames to PatchTrack/Record\n\nSeeing requested / obtained buffer frame counts in trace\nhelps to identify discontinuities in software patch pipeline.\n\nThere is no need to trace requested frame count for PatchRecord\nas it always requests as much as possible (\"-1\" frameCount).\n\nBug: 117564323\nTest: with software patch active,\n external/chromium-trace/systrace.py audio\n\nChange-Id: I78400f6cc1d1940b3290fb23bb1aacfbffe042e5\n" }, { "commit": "fb1ae807222a43d977790c028297496dc0583f53", "tree": "e8fe68038039316b1cb49b7fd00e842ec602449e", "parents": [ "e67d86100f23f56d78b3a666bc2b87284d0486b1", "ddf87effa4d80f413a80fde861f73877ccd80599" ], "author": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Tue Aug 27 15:50:18 2019 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Tue Aug 27 15:50:18 2019 +0000" }, "message": "Merge \"AudioFlinger: enforce OP_RECORD_AUDIO during recording\"" }, { "commit": "ddf87effa4d80f413a80fde861f73877ccd80599", "tree": "0e89ff4a5be97c07c81bc11c36f5e8741fca9eed", "parents": [ "28355761d73f1495bba8994dc3c0dc631ae245a8" ], "author": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Tue Aug 20 15:42:04 2019 -0700" }, "committer": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Mon Aug 26 16:00:34 2019 -0700" }, "message": "AudioFlinger: enforce OP_RECORD_AUDIO during recording\n\n Fix issue where RecordTrack silencing didn\u0027t silence the\nfull buffer: the memset to 0 was using the RecordThread frame\nsize, not the RecordTrack frame size.\n OP_RECORD_AUDIO was only enforced at the start of a recording\nwhich would fail if not granted. This patch silences the recording\n(i.e. silence is recorded instead) when it is lost, and undoes that\nwhen granted again. This requires:\n- propagating the package name of the client to the RecordTrack class\n- registering an appOp callback in RecordTrack (through a new\n OpRecordAudioMonitor class) to (un)silence the recording\n- update the isSilenced() method to take into account the appOp.\n\nBug: 138968594\nTest: run app that records audio, then \"adb shell appops __pack_name__ 27 2\"\n and verify recording is silent after that.\n\nChange-Id: Ib33f5b592185a67204997213bab1ac2594d90d37\n" }, { "commit": "689e82c64b8e21b89f4ccc877816cc15253d6ad9", "tree": "2c796ca9f6ce5187d5f894d0597182cd13f53215", "parents": [ "34bbfc447ad5c3b615acf9e924a97b2a3d678ab7" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed Aug 21 17:53:17 2019 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Aug 22 12:14:07 2019 -0700" }, "message": "AudioFlinger: Enforce per uid track limitation\n\nTest: atest AudioTrackTest#testMaxAudioTracks\nBug: 139811855\nChange-Id: I09d6eec98dfdae953d797037374b70ed9332a6bd\n" }, { "commit": "9066ad3846deb54fe2bba86b25a934d5bf6033d7", "tree": "c23c4f7093b5f9775f4284471b913e14c1dc028b", "parents": [ "86bd6afc79a46d586be816172f3cef7e7c89dc09" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon May 20 14:40:10 2019 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue May 21 14:28:58 2019 -0700" }, "message": "audioflinger: fix App Ops for services\n\nDo not check app ops for audio playback for services in\ngeneral, not just for audioserver or root user.\n\nBug: 133178934\nTest: audio smoke tests\nChange-Id: I3a34e6418dc750cc12ed35d465ca8874b0ce0f73\n" }, { "commit": "2dab03030c95b40902e66b81a247738e375ebcf9", "tree": "cd140cffff1b7ff1c7e35f0d3d56479bfef054d7", "parents": [ "af0b1c1b5cd67cafa81dbcd1ccb7eb68821173f9" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed May 08 18:15:55 2019 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed May 08 18:19:49 2019 -0700" }, "message": "audioflinger: honor flag AUDIO_FLAG_BYPASS_INTERRUPTION_POLICY\n\ncommit 74e01fa7 did not bypass app ops policy when flag\nAUDIO_FLAG_BYPASS_INTERRUPTION_POLICY is set.\n\nBug: 131873101\nTest: repro steps in bug\nChange-Id: Idbce26cfdcddbb7a2ae8702ce3d135ef5a69f047\n" }, { "commit": "09f1ed2cb4479c93859557f5b4d40e1c3732f6f2", "tree": "27f2c1c23755b45acdfd0832bb9e493f6b890aaa", "parents": [ "2f7a6072bbafd4faba8cbaf170d3cb72e37e3454" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Apr 24 17:45:17 2019 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Apr 29 09:59:31 2019 -0700" }, "message": "audio: improve audio routing callbacks\n\nDo not force audio device changed callback when the client\n(AudioTrack or AudioRecord) registers to AudioFlinger but only when\nit starts playback or capture. Doing so prevents a spurious callback\nhapping at registration time where a stale device\n(previously selected by AudioFlinger thread) but irrelevant to this\nclient is indicated. This causes a disconnection of AAudio streams\ndespite no real device change.\n\nBug: 128630993\nTest: CTS: android.nativemedia.aaudio.AAudioTests, android.media.cts.RoutingTest\n CTS Verifier: Audio Input/Output Routing Notifications Test\n\nChange-Id: Ia7f1d11490989b0287c97479466c1c07a223aab3\n" }, { "commit": "f7e3a3a2eb753da4500b9000c113ae0f3b058792", "tree": "83b0c8c42a44d092b1b054fd059a98df23b81521", "parents": [ "3208826f676b1be78e1c04692d7f17249e45c3ea" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Mon Apr 22 16:43:26 2019 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Mon Apr 22 16:53:41 2019 -0700" }, "message": "audioflinger: Fix reference counting protocol in OpPlayAudioMonitor\n\nOpPlayAudioMonitor was constructing a weak pointer to itself\nin the constructor. This practice can lead to crashes due to\nrace conditions vs object destruction. This code is now moved\nto onFirstRef method which is called when at least one strong\nreference exists.\n\nThis change also reduces the number of created OpPlayAudioMonitor\nobjects by using a factory method.\n\nBug: 130038586\nTest: enable / disable DND mode\nChange-Id: I22e63a883ebaa25b9c96e79271bb9693b5ed75cd\n" }, { "commit": "6c79632bbece71f7af0c2453bb62a5d6f0cca664", "tree": "a70a17d1f723b7e0241bb762bf5478e3c9947348", "parents": [ "2cb1a8ccbd402d74a06ee34721f5dd787ecb396f" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Apr 09 14:13:17 2019 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Apr 17 10:29:30 2019 -0700" }, "message": "refactor mutexes for audio effects in audio flinger and audio policy\n\nRemove effect specific mutex (mEffectLock) in AudioPolicyService: Due to\nconcurrent capture (among other reasons), it is necessary that the audio\npolicy manager state preserved by mLock includes audio effects\nregistration and enabling.\n\nMoved all audio policy API calls from audio flinger out of locked regions\nfor audio flinger, thread and effects mutexes to avoid cross deadlocks\nbetween audioflinger and audio policy manager:\n- centralized audio policy API calls in EffectModule::updatePolicyState()\n- the enabled state now reflects the state requested by the controlling\nhandle, not the actual effect processing state: a suspended effect is\nnow considered enabled.\n\nA new audio policy manager API moveEffectsToIo() is added to atomically\nhandle moving effects to a new input or output without having to call\nunregister \u003e register \u003e enable sequence.\n\nAlso fix assert in setStreamVolume to match volume group refactoring\nin audio policy manager.\n\nBug: 128419018\nTest: CTS tests for audio effects.\nTest: manual tests with Duo calls, Play Music, Youtube, notifications\n with and without Bluetooth and wired headset.\n\nChange-Id: I8bd3af81026c55b6be283b3a9b41fe4998e060fd\n" }, { "commit": "74e01fa7d96b71952b32ec98f1ac352e6e74186b", "tree": "c48d9297fb55046c3dc0a14d0652b8072fa0db62", "parents": [ "c70268aa433e7d307819184ae5c2ec96a3e5b6af" ], "author": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Mon Feb 25 12:18:09 2019 -0800" }, "committer": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Tue Apr 02 15:04:30 2019 -0700" }, "message": "Audioflinger: tracks monitor OP_PLAY_AUDIO\n\nMute/unmute tracks according to changes in OP_PLAY_AUDIO for\n the current usage.\nIn audio policy: always assign AUDIO_STREAM_ENFORCED_AUDIBLE\n to sonification tracks with AUDIBILITY_ENFORCED flag.\nDo not mute tracks from root / audio server.\nDo not mute UI sounds on AUDIO_STREAM_ENFORCED_AUDIBLE\n stream type.\n\nBug: 112339570\nTest: enter DnD, play notifications, verify not heard\nChange-Id: Ia5f1118481cf0573101acf2092fbd0ce2cf8c038\n" }, { "commit": "1234008771662d2b250d6d4972adb04de963fb0d", "tree": "d050c75b3935ed9d20676312df0e7cd1d94cd13c", "parents": [ "60c73a445b632cc2e233b1a275efeee7414a41a4" ], "author": { "name": "Paul McLean", "email": "pmclean@google.com", "time": "Tue Mar 19 09:35:05 2019 -0600" }, "committer": { "name": "Paul McLean", "email": "pmclean@google.com", "time": "Thu Mar 21 21:51:49 2019 -0600" }, "message": "API changes in MicrophoneDirection\n\nBug: 128523813\nTest: Build, Testbed app\n\nChange-Id: I7b5ee8ba4eec1090866d8aef891a98d50cd09cc5\n" }, { "commit": "abfab20fdbdc41168f816272c31caa6c1264ba46", "tree": "940b8444b6e54cd0beee1b51427dab2715d7453a", "parents": [ "4e7c08a1343cc9a47c17950359fb0845b262dc40" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Mar 07 19:45:54 2019 -0800" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Mar 08 10:37:15 2019 -0800" }, "message": "PatchPanel: Keep peer alive during use.\n\nTest: BT call, switch from hearing aid to headset\nBug: 126789266\nChange-Id: I589e5ecaac25c1ce1d504a387662b96cf50f63ba\n" }, { "commit": "d83b08ac54d11ed557b3cff9fef60dfa84386550", "tree": "4903cccde055fb89cfa7d6558c6b22a64113a381", "parents": [ "e1e9cad4d0c7c9825b7f29ebc082dfcbdcace25c" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Feb 27 15:05:54 2019 -0800" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Thu Feb 28 00:26:13 2019 +0000" }, "message": "Audioflinger: Do not intercept if no audio\n\nThis was not efficient and leaded to an assert in obtainBuffer.\nI\u0027m not sure which condition can lead to a getNextBuffer of size 0,\nbut it has been observed and is not forbidden by getNextBuffer\ndocumentation.\n\nTest: atest android.media.cts.AudioPlaybackCaptureTest#testCaptureMediaUsage\nBug: 111453086\nChange-Id: I5accf7c1d488ff4686272588329bab71d64f67cd\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "6057fa270854d700cc85804fdd4517a8f8831f1f", "tree": "a4d5ba8560df80a6221403bf87f0a84820b360bb", "parents": [ "a134b006bc5714b6c8dcfbe5900a3ee029aa1044" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Fri Feb 08 14:08:07 2019 -0800" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Fri Feb 08 17:31:07 2019 -0800" }, "message": "Log if intercepting tracks takes too long\n\nTest: adb shell audiorecorder --target /data/file.raw\nBug: 111453086\nChange-Id: Ic56cd0d223e83938a2d5eeb98cbcb754629799d8\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "a134b006bc5714b6c8dcfbe5900a3ee029aa1044", "tree": "8961e932f8db72b1f24a7f7d83ca91ced0369975", "parents": [ "2ec0674146dc0a9d885a18067a7a5c84786b59ad" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Thu Feb 07 18:05:31 2019 -0800" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Fri Feb 08 17:31:07 2019 -0800" }, "message": "Audioflinger intercept track retry on buffer end\n\nTrack::interceptBuffer failed to write all the audio if the source BP\ncould not returned the requested buffer size.\nThis is actually normal when the source circular buffer wraps around.\n\nHandle it by retrying if the first buffer is too small.\n\nTest: adb shell audiorecorder --target /data/file.raw\nBug: 111453086\nChange-Id: I42a7962449a0f075909a29f5f8f5ba82ca1d0085\n" }, { "commit": "c43ea147033817dc46b76d8993a84f0e5ee4e4e0", "tree": "4900174eb6395b5980753c40fbdab9064501b37a", "parents": [ "153f92d57b6a1d2fdc7fa601bbc86db31646a8f9" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Thu Jan 31 18:17:37 2019 -0800" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Feb 06 16:13:13 2019 -0800" }, "message": "Propagate track status to interception patch track\n\nTest: adb shell audiorecorder --target /data/file.raw\nBug: 111453086\nChange-Id: Icbf627ce36b6d9390439f51237ef9515a2eba8cb\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "153f92d57b6a1d2fdc7fa601bbc86db31646a8f9", "tree": "0d380f13a58ff8fdf073dd974dcb55e4d18ef75c", "parents": [ "45986c74cd5253bcc81b1b4954e3ff3300f7e6d8" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Tue Dec 18 18:33:28 2018 -0800" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Feb 06 15:49:11 2019 -0800" }, "message": "Add secondary output to audio tracks\n\nThe secondary are returned from mixes (from DAP loopback\u0026render),\nfrom getOutputForAttr.\nAll getOutputForAttr* of the stack are update.\nInternal getOutputForAttr use descriptor, external one use handles.\n\nThe secondary output are saved in each track and the track is\ninvalidated if the list of secondary output changes.\n\nIn audio flinger, create a pair of recordTrack \u0026 patchTrack to pipe\nthe intercepted audio audio to the secondary output.\n\nTest: adb shell audiorecorder --target /data/file.raw\nBug: 111453086\nChange-Id: Id6523d9e383c15a0e39313d5f355df809b7e72fe\n" }, { "commit": "45986c74cd5253bcc81b1b4954e3ff3300f7e6d8", "tree": "7c549d046cb5b342affb16ac8fdfeb13d3f53f3b", "parents": [ "ba98acb34e60b2bf2dac5a55c4acbb80b5408568" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Tue Dec 18 18:22:59 2018 -0800" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Feb 06 09:38:33 2019 -0800" }, "message": "Audioflinger: add timeout to PatchTrack\n\nAdd support for timeout that was only supported by PatchRecord.\nThis is implemented by moving the timeout and common code in a base\nclass.\n\nAlso remove PatchRecord useless virtual inheritance of RecordTrack.\n\nTest: adb shell audiorecorder --target /data/file.raw\nBug: 111453086\nChange-Id: I833148f31a311ca41092be1d7e2d170f086322c5\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "51f0e98712a7378073ec97ab59ea406a1fbaf169", "tree": "fc2a6b55a8c9957e52499d7763f24741b917272f", "parents": [ "5a49f40a70302ac1ffc54e0d96db2bec17546fe1" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Fri Feb 01 19:19:11 2019 -0800" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Sat Feb 02 22:57:17 2019 +0000" }, "message": "AudioFlinger: Fix Tee track filename\n\nTest: adb shell audiorecorder --target /data/file.raw\nBug: 111453086\nChange-Id: I5cd1ecd9f85bb2278e6d1f9434aa453833afa173\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "57303cc4312686e02e754ba99a81f0cf71144757", "tree": "5118a0c03e035d13c203118e1336e33803b1de08", "parents": [ "77270b880570e07a5d80b23371f53638ec30fb5c" ], "author": { "name": "jiabin", "email": "jiabin@google.com", "time": "Tue Dec 18 15:45:57 2018 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Jan 30 17:07:10 2019 -0800" }, "message": "Interface between audio server and vibrator service\n\nThe haptic playback should be controlled by vibrator service. Via the\ninterface, audio server could notify vibrator service about starting or\nstopping haptic playback and get vibrator intensity from vibrator\nservice. Vibrator service could call mute/unmute to control the haptic\nplayback.\n\nTest: Manually\nChange-Id: Iad24813977e4dea0d67a91f8f8b390a016ce4ca2\n" }, { "commit": "03a6e6a08dbda1c94e0c9ce6fba4a4abd3df6d59", "tree": "e609e29b32764e6fd970f16c78d3bbee5be401bb", "parents": [ "c2684ac61bdbfd9913cf1361a6436272756bc6d6" ], "author": { "name": "Paul McLean", "email": "pmclean@google.com", "time": "Tue Dec 04 10:54:13 2018 -0700" }, "committer": { "name": "Paul McLean", "email": "pmclean@google.com", "time": "Thu Dec 20 17:37:55 2018 -0700" }, "message": "Adding Audio HAL V5: Direction API\n\nBug: 119137468\nTest: Testbed app.\n\nChange-Id: I1b3c81079000958da80d36616fe06930d876aac3\n" }, { "commit": "973db02ac18fa1de9ce6221f47b01af1bdc4bec2", "tree": "e2dda15a881c963a14520117128b5b9857834759", "parents": [ "f4a342abd2959904caa18b5a0671e8107581ff24" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Nov 20 14:54:31 2018 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Dec 04 12:06:37 2018 -0800" }, "message": "audio flinger: return port ID as track ID to client\n\nReturn the port ID allocated by audio policy manager instead of the\ninternal track ID allocated by audio flinger when an AudioTrack or\nAudioRecord is created.\nThis information is more useful for logs and allows to associate information coming\nfrom audiopolicy manager with a specific client instance.\n\nBug: 111438757\nTest: Manual playback and capture tests\nChange-Id: Ib467d8fcc34d9a8aa7bcaac0770a741982b847c5\n" }, { "commit": "63d2d774249b0f06b2f1b0eae79e844d92fb96ac", "tree": "92e223e69d84d865bcb44481dd1613ff4f705c27", "parents": [ "1268b28c5c65691532d3ffb503a199c677c801b9", "5d3d956d695cac9551ecffd4ec33f56a7ab40989" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Dec 03 21:12:21 2018 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Mon Dec 03 21:12:21 2018 +0000" }, "message": "Merge \"AudioFlinger: Remove latency reporting for AUDIO_INPUT_FLAG_HW_AV_SYNC\"" }, { "commit": "ac917aca235ff5170f00c2ee933f41f07a40232d", "tree": "2927cc6419b7fe615ccdc977224f6f3ed656f58e", "parents": [ "e4eb88fa5fe62a9b6b709fbf2ff26f37acf6db20" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Nov 28 14:03:52 2018 -0800" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Wed Nov 28 14:44:17 2018 -0800" }, "message": "Add selectPresentation API to IAudioTrack\n\nThe Java AudioTrack interface has a setPresentation API. This\ncalls the setParameters API in IAudioTrack. However, this does\nnot eventuate into a call into the android.hardware.audio@4.0\nIStreamOut selectPresentation API.\n\nAdd selectPresentation API to IAudioTrack and call down to the\nandroid.hardware.audio@4.0 IStreamOut selectPresentation API.\nTranslate into calls to setParameters API for legacy HAL\ninterfaces.\n\nBug: 63901775\nTest: compile\nChange-Id: Id634a107f3f93ab18dc80d2bd0af67bda35e859f\n" }, { "commit": "ce685406448709895391dcc83a75fa165a72174e", "tree": "a4a1490819ffce35bd113ca96b905ff83de168e0", "parents": [ "b5be68edecd62d8edbf86fd4858d1ceb81a74b6c" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Oct 05 17:23:27 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Oct 15 18:53:25 2018 -0700" }, "message": "AudioRecord: Clean up state handling\n\nHandle spurious wakeups in stop.\nHandle track invalidation.\nClarify state.\n\nNote: start and stop should not be called simultaneously from different\nbinder threads.\n\nTest: audio sanity\nBug: 117303225\nChange-Id: I587d9cda7b0eac5215eb6a88f93ee08e396e3ef9\n" }, { "commit": "5d3d956d695cac9551ecffd4ec33f56a7ab40989", "tree": "abde7e0f65bb718fe56e4b81ba4cfe82ab1fa826", "parents": [ "d10b735cfd4eea74846bb5f53d37c8c2a3114d58" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Oct 04 19:27:26 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Oct 04 19:40:51 2018 -0700" }, "message": "AudioFlinger: Remove latency reporting for AUDIO_INPUT_FLAG_HW_AV_SYNC\n\nTest: No regression\nBug: 117299316\nChange-Id: I8f7c70454baf93f36e87ea98ba795beea8d75678\n" }, { "commit": "17a4a0feba513a721b23edeb9a915e4ce87b58a8", "tree": "8def40ef260d85ecf5414654a2af5cc1a7e3a94c", "parents": [ "bd9b5935fdbd67d3eaf124e3e80470b8cae1ecd1", "d06ae78ee7169fc80c8a29390b2156e33c846c08" ], "author": { "name": "Chih-hung Hsieh", "email": "chh@google.com", "time": "Mon Sep 17 16:44:40 2018 -0700" }, "committer": { "name": "android-build-merger", "email": "android-build-merger@google.com", "time": "Mon Sep 17 16:44:40 2018 -0700" }, "message": "Merge \"Suppress implicit-fallthrough warnings in audioflinger etc.\" am: a494bb1527 am: 8662f3b9a2\nam: d06ae78ee7\n\nChange-Id: Iee095b6e1278d372b3a7c171f926bd730dbc388b\n" }, { "commit": "2b48703770efab81143d389d7389166eb1a6c8db", "tree": "f309a4bbfbb4f716dee1af20f82ca95aa04b907e", "parents": [ "f8d13dee65d3708ea527e9f565be85031d7021ba" ], "author": { "name": "Chih-Hung Hsieh", "email": "chh@google.com", "time": "Thu Sep 13 14:16:02 2018 -0700" }, "committer": { "name": "Chih-Hung Hsieh", "email": "chh@google.com", "time": "Mon Sep 17 12:50:12 2018 -0700" }, "message": "Suppress implicit-fallthrough warnings in audioflinger etc.\n\nAdd FALLTHROUGH_INTENDED for clang compiler.\n\nBug: 112564944\nTest: build with global -Wimplicit-fallthrough.\nChange-Id: I8875bc7ed073aae84d5df97eb39c8c745ca627ef\n" }, { "commit": "9d84af54244f7ff086afdcc6e477fe4d04874652", "tree": "dc3f3d5c5bc956b413fe3cc234ed45cc20cff1d9", "parents": [ "c06913801424b90dfbc94b401039352869f20e3e" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed Sep 12 18:03:44 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Sep 13 17:54:07 2018 -0700" }, "message": "AudioFlinger: Use track ids in log messages\n\nTest: Audio sanity test, check logcat\nBug: 115400221\nChange-Id: I6b3ed72c3a2584f03bb2183b099c5a42d2d49527\n" }, { "commit": "c06913801424b90dfbc94b401039352869f20e3e", "tree": "5b648a0897db335b91c9365bdd2ba01d0eb708e6", "parents": [ "867f39f2797ae01139fe956fe4f029548de4d662" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed Sep 12 18:01:57 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Sep 13 10:44:18 2018 -0700" }, "message": "AudioFlinger: Replace track name with id\n\nSimplifies Mixer handling, makes debug messages consistent.\n\nTest: Audio sanity test, check logcat\nBug: 115435564\nChange-Id: I216e2746170605dfb7994ac660c5534869e887fe\n" }, { "commit": "30282562329b3031b6bfa0869bd76ff5f543ebba", "tree": "82ba1641c178b26419735faf7f99be6bc1ffc9c1", "parents": [ "182d359515fbf4ba0fc14709b583c29edce30897" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed Aug 08 18:27:03 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Aug 13 16:27:59 2018 -0700" }, "message": "AudioFlinger: Add patch latency for direct record/playback\n\nThis relies on being able to obtain presentation position and\ncapture position accurately from the HAL, in the presence\nof potential underruns or overruns.\n\nTest: MSD hal dumpsys\nBug: 112428710\nChange-Id: I9aad574baaff60b5e0c5d8c39a2147d19ee613f5\n" }, { "commit": "7f39f56e33bc0feb5188cc0372e0be69239ba78f", "tree": "c80bffac38de0e373d6455b46ae5ff382a2c33eb", "parents": [ "a29488990bcb34a2723ec4ece2ea34f714070f5f" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed Aug 08 17:30:20 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Aug 09 23:08:30 2018 +0000" }, "message": "AudioFlinger: Do not show latency for non-PCM record in dumpsys\n\nTest: Check AudioFlinger dumpsys during MSD direct record\nBug: 112388863\nChange-Id: Id404c70af0c0e23a1e349522568b20f50573b1e9\n(cherry picked from commit 2ceaf3dee96d1696b315afdee9ffc0df662f88d8)\n" }, { "commit": "d7fe086d3b45f088083170006367dfc4246f70b0", "tree": "fa89edfbebbd2cffdccc0ab6f01caa9a06d02390", "parents": [ "b05e9b139f8cc5c20f7e94d0a9cf2f7430b52cf2" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Sat Jul 14 16:48:01 2018 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Jul 17 13:02:28 2018 -0700" }, "message": "audio policy: refactor audio playback APIs\n\nRefactor audio policy service APIs controlling audio playback (startOutput, stopOutput, releaseOutput)\nTo allow finer grain control per AudioTrack client.\n\nTest: AudioTrack CTS test. manual test of playback use cases.\nChange-Id: I49a681f3c2a8211e524824993049b24d8086376d\n" }, { "commit": "7c6ae98049e422454f6e9ea21d9d3fd6a7bfd217", "tree": "06f9d2dc1df52505e59968677bc1dc8d042a1e55", "parents": [ "495d8f29c82be95d499f8897311411ccc74ec52b" ], "author": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Thu Jun 14 12:33:38 2018 -0700" }, "committer": { "name": "Mikhail Naganov", "email": "mnaganov@google.com", "time": "Thu Jun 14 17:03:06 2018 -0700" }, "message": "audioflinger: Add support for RecordTrack with no conversion\n\nWhen piping encoded audio data via software patch, it needs\nto be created without the buffer converter. In this case the audio\ndata is copied directly from ResamplerBufferProvider.\n\nAdded input flag AUDIO_INPUT_FLAG_DIRECT to indicate to PatchPanel\nthat the PatchRecord track needs to be opened in this mode.\nThis flag can be later added to minor Audio HAL update.\n\nBug: 63901775\nTest: MSD scenario; verified that phone over USB headset still works\n on marlin / sailfish\n\nChange-Id: If2745778d356769b143fb3c29910275ddef119ac\n" }, { "commit": "495d8f29c82be95d499f8897311411ccc74ec52b", "tree": "23e0f0c1fc5af2b845f1434672602689a8b445bf", "parents": [ "7048c863f5cc4eb793bb96fd8e37da5107208e26", "5f2136e6323c1e241c918a59cd06e715480241cb" ], "author": { "name": "TreeHugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jun 13 15:36:21 2018 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Wed Jun 13 15:36:21 2018 +0000" }, "message": "Merge \"AudioFlinger: Log audio attributes in dumpsys\"" }, { "commit": "5f2136e6323c1e241c918a59cd06e715480241cb", "tree": "e79e161a8adce8ea6722def58dd64933078ab6ab", "parents": [ "70b576df7d88c2d5843fc31eb726ed7c33c1af89" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Fri May 11 22:03:00 2018 -0700" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Tue Jun 12 19:16:38 2018 -0700" }, "message": "AudioFlinger: Log audio attributes in dumpsys\n\nTest: adb shell dumpsys media.audio_flinger|grep FrmRdy -A1\nChange-Id: I47d402133ab30d0f2276577d2b51c30956f873c6\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "1c86ebe110381daa6b05d07020fb75f166280f34", "tree": "3fb0e5e985a35d01ad428098d4cadb1f66a03b0a", "parents": [ "dfc0730c3375c64b001b5672eedf94a2dbda59e1" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue May 29 20:29:08 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Jun 07 09:58:09 2018 -0700" }, "message": "AudioFlinger: Add latency information for duplicating threads\n\nTest: Bluetooth and ringtones\nBug: 80447941\nChange-Id: I0af9090f08636609e4c5a8c47d1d4c10f446746d\n" }, { "commit": "000adb5e9135a739b5e3a0607604cdf6be6fe11b", "tree": "9d751b3c072179e53bf4b9796f72dc9958dd55c6", "parents": [ "cef2daaf49e8802e3377ebc604012aa244b7231a" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Jun 01 15:43:26 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Jun 04 10:30:06 2018 -0700" }, "message": "AudioFlinger: Add latency information for RecordTrack\n\nTest: audioflinger dumpsys\nBug: 80493498\nChange-Id: I902783b7705f69f00a5f5bd713d93d8657ba3ccb\n" }, { "commit": "cef2daaf49e8802e3377ebc604012aa244b7231a", "tree": "5cd12c0b322858c80d616b40d65bd7ecadc59eac", "parents": [ "20bd30baf9fcd59e7000e7c626cc8bc1e85637bc" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Jun 01 15:31:49 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Jun 04 10:27:52 2018 -0700" }, "message": "AudioFlinger: refine latency computation if track is drained\n\nUse server timestamp if track has no actively presented frames\n(i.e. drained). Sometimes track frames may not have reached the HAL.\n\nTest: audioflinger dumpsys with BT audio\nBug: 80447764\nChange-Id: Iffc52f4cfcbadd419c6b6ccfa278e0712f3af4af\n" }, { "commit": "f6ab58dfbc394e88115eb1483aaf47db91279981", "tree": "3a1c63e4a0e66fcb960201ec9852cc8c7e8dc8c0", "parents": [ "59fef32845e5fec2f5c97c9637965b8c3ca24a6f" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri May 25 12:50:39 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri May 25 14:03:16 2018 -0700" }, "message": "AudioFlinger: Add latency measurements from timestamp\n\nReplaces main and aux buf from track dump.\n\nBug: 80272001\nTest: adb shell dumpsys media.audio_flinger\nChange-Id: I5d6565410e652ec7fc6701b171d299dea9f7bc3e\n" }, { "commit": "298e1b3db0f94396249933c823b6ff55cf8fc1cc", "tree": "8deeed6f3a7001d40440a130e9f377ec3fb59a89", "parents": [ "868c0a9e91df17dcad52b5a0dcacc51205dd8f46", "4ef19fa450e8b523741a22fabee8f62691abec37" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue May 22 20:47:54 2018 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Tue May 22 20:47:54 2018 +0000" }, "message": "Merge \"Update audio permission checking\"" }, { "commit": "8946a28aaaadf426ee22b098b4aec7be4d9edc00", "tree": "85a9520841616e8c1188d51fd44591543148e874", "parents": [ "acb7d9ddc1955682379ea514082511b9ea44482e" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Apr 19 20:04:56 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon May 21 11:37:46 2018 -0700" }, "message": "AudioFlinger: Fix Tee logging\n\n1) Fix bad WAV files for tee sources that aren\u0027t 16 bit PCM.\n2) Allow multiple RecordThread tees instead of one.\n3) Allow dump from non-fast MixerThreads (e.g. deep buffer).\n4) Parallelize tee dumping for improved concurrency;\n dump outside of AF lock.\n5) Async dumping to allow dump to be issued in time critical code.\n6) Improve file naming to distinguish record vs playback tracks,\n threads, and dump reason.\n7) Allow Tee insertion anywhere in code with global running\n Tee management.\n8) Increase resolution of filename time to msec avoid file overwrite.\n9) Dump track data upon removal from active tracks to improve\n timeliness of dumped data.\n10) Dump tee data on tee destruction.\n11) Refactor Tee code out of AudioFlinger.cpp; minimize footprint.\n\nAudioFlinger enabling code requires Configuration.h define TEE_SINK;\nthis is disabled, hence avoiding regression risk.\n\nTest: Enable tee log, repeatedly call dumpsys, examine files.\nBug: 78369241\nChange-Id: I7b22cfa7dbbc1601828de931522065a509ab4047\n" }, { "commit": "4ef19fa450e8b523741a22fabee8f62691abec37", "tree": "5d0eaaf73cebe029d1d8dd935d5c6fddb12ee70f", "parents": [ "ab7ef300b74904bce6c5c810d9e5679abfa582e6" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue May 15 19:35:29 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu May 17 18:11:55 2018 -0700" }, "message": "Update audio permission checking\n\nChange captureAudioOutputAllowed to check client pid.\n\nCheck calling uid with AID_AUDIOSERVER instead of calling pid with\nown pid. This is consistent, and works if AudioFlinger and\nAudioPolicyManager exist as different processes.\n\nRemove getpid_cached since getpid() is very fast. This removes\nany initialization issues.\n\nReplace getuid() with AID_AUDIOSERVER to remove ambiguity of\nmultiple native audio services for multiple users. Only\none exists regardless of users.\n\nDo not use multiuser UID checks for certain native services\nthat do not spawn for multiple users to prevent accidently exposure.\n\nMove permission checks to use ServiceUtilities for control and\nconsistency.\n\nRename isTrustedCallingUid to isAudioServerOrMediaServerUid\nso that permission check is explicitly known to caller.\n\nUpdate MediaLogService to use ServiceUtilities.\n\nTest: Basic sanity\nTest: AudioTrackTest, AudioRecordTest, SoundPool, SoundTrigger\nBug: 79485140\nChange-Id: Ib8ccb36929a9b4806c01626f32fa023a046d6020\n" }, { "commit": "ab7ef300b74904bce6c5c810d9e5679abfa582e6", "tree": "dadb05ba8da176c046139ca43c3fae48a14191c3", "parents": [ "1f2bfd3ac9d61982acb348d2c78f95c7d4b0bf05" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue May 15 19:35:29 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Wed May 16 18:49:09 2018 -0700" }, "message": "Move ServiceUtilities from audioflinger to libmediautils\n\nTest: Play Music, Play Movies, Camera, Photos\nBug: 79485140\nChange-Id: Ib8382547828e386024145c416321d24e4af182db\n" }, { "commit": "f62e1a2887bedb440bb98c4e7071896dd3f46163", "tree": "e888731841186f29889629dd796fc5bd213ca614", "parents": [ "d5fa1baa8b0a985d80cbfb53c3960504314ca8f6" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue May 08 18:32:11 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Tue May 08 18:38:44 2018 -0700" }, "message": "Fix PatchTrack retry for ClientProxy::obtainBuffer\n\nRestore buffer mFrameCount as error clears it.\n\nTest: phone call with usb device, simulate underrun.\nBug: 79423841\nChange-Id: Ia00d08b6753b5c65cf7f07bf1c889fd6d1eefd0a\n" }, { "commit": "9378eb9634d07010bebf8a2d4bda84320ec80e57", "tree": "9d9dd22652b7860882e3155a6d218042d55cd3f6", "parents": [ "6f4512914d90430bf135de221897887638c79f25" ], "author": { "name": "jiabin", "email": "jiabin@google.com", "time": "Wed May 02 15:26:35 2018 -0700" }, "committer": { "name": "jiabin", "email": "jiabin@google.com", "time": "Wed May 02 17:13:07 2018 -0700" }, "message": "Restrict setting silenced in RecordTrack.\n\nWhen initializing, always set mSilenced as false. Never set silenced for\npatch track.\n\nBug: 77193174\nTest: make phone call with usb headset multiple times.\nChange-Id: I93f4646cc6d88749c602bdd0003afc48f7dec36f\n" }, { "commit": "331679c8a5cd21a0e6a7a1d851e965e4721f7420", "tree": "9e3835bb805631ca42a90ded01796b73307a4642", "parents": [ "554e08c6b00b2a57348f4545d450915a9e8c870a" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Apr 16 17:03:16 2018 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Apr 16 17:08:06 2018 -0700" }, "message": "Revert \"Revert \"audioflinger: implement silenced capture for mmap\"\"\n\nThis reverts commit 611ebb95e76c230214780b7b583b8a3c5277ac7a.\n\nBug: 78118584\nbug: 72134552\nTest: \u0027adb shell write_sine_callback -m3 -pl -s100 -x\u0027 and Plug headphones in and out.\nTest: AAudio CTS tests\nChange-Id: I7558ad7b9c6c2d352ebce105382fe914c26fd35e\n" }, { "commit": "554e08c6b00b2a57348f4545d450915a9e8c870a", "tree": "b66615256ee7a7eeac8a7e54c54c2fd872a3fdd0", "parents": [ "c620796251df712274c16858c76628d802bbb4b1", "611ebb95e76c230214780b7b583b8a3c5277ac7a" ], "author": { "name": "TreeHugger Robot", "email": "treehugger-gerrit@google.com", "time": "Mon Apr 16 23:52:19 2018 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Mon Apr 16 23:52:19 2018 +0000" }, "message": "Merge \"Revert \"audioflinger: implement silenced capture for mmap\"\" into pi-dev" }, { "commit": "611ebb95e76c230214780b7b583b8a3c5277ac7a", "tree": "1c8ef30309a0954a51a6b4c872f30ca399f064d2", "parents": [ "a2f478ca0b2bda8db9b321f0cfa9e9d3cf6130f9" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Apr 16 19:43:59 2018 +0000" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Apr 16 19:43:59 2018 +0000" }, "message": "Revert \"audioflinger: implement silenced capture for mmap\"\n\nThis reverts commit a2f478ca0b2bda8db9b321f0cfa9e9d3cf6130f9.\n\nBug: 78118584\nChange-Id: Ie0da3dea445f2474c7a889c5f9919305c1c96d40\n" }, { "commit": "c16f23e399969e9d82f0b245afb0870fa448e78e", "tree": "39c75acf7d8a311cce349d3edf4f0dd89d0873da", "parents": [ "247cc0a3c81ec629a2ae51b40bdcb2afc0e94210", "a2f478ca0b2bda8db9b321f0cfa9e9d3cf6130f9" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Apr 13 00:39:54 2018 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Fri Apr 13 00:39:54 2018 +0000" }, "message": "Merge \"audioflinger: implement silenced capture for mmap\" into pi-dev" }, { "commit": "fe463237ab61459ddc30120962c51fb089d25ec4", "tree": "651656f0c3ca3b0a693f2a675dd5e775ad17284c", "parents": [ "b856955f27ca4ae146e940ab8a4ababab04d8e35", "7d665abb442a84d82760d88d228914a116e34532" ], "author": { "name": "TreeHugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Apr 12 09:53:21 2018 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Apr 12 09:53:21 2018 +0000" }, "message": "Merge \"AudioFlinger: input thread dump shows if RecordTrack is silenced\" into pi-dev" }, { "commit": "a2f478ca0b2bda8db9b321f0cfa9e9d3cf6130f9", "tree": "00e39e41acac99cc35c292c65c7bca60973fdd5a", "parents": [ "9fc25229e0723c93b453ab34957f22a643e2b602" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Tue Apr 10 19:09:40 2018 -0700" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Wed Apr 11 19:07:45 2018 -0700" }, "message": "audioflinger: implement silenced capture for mmap\n\nImplement silencing of audio capture for idle UIDs for\nMMAP streams.\n\nUse audio HAL setMicGain() API to silence the mmap buffer for\nexclusive mode and when only one client is active in shared mode.\n\nAdd new MmapStreamCallback method onStreamSilenced() for AAudio\nservice to selectively silence streams from silenced UIDs in\nshared mode.\n\nBug: 72134552\nTest: manual\nChange-Id: I7c92ba0002bc5ba003c1a3c887edddf9ae53b538\n" }, { "commit": "7d665abb442a84d82760d88d228914a116e34532", "tree": "4ea9c0a7b93749bf20b89d3eb10acc4d751cfa65", "parents": [ "7121c67c340048fb520cd71bf510769140cc5bcd" ], "author": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Wed Apr 11 17:26:51 2018 -0700" }, "committer": { "name": "Jean-Michel Trivi", "email": "jmtrivi@google.com", "time": "Wed Apr 11 17:26:51 2018 -0700" }, "message": "AudioFlinger: input thread dump shows if RecordTrack is silenced\n\nFor each record thread, show if tracks are \u0027s\u0027ilenced\n or \u0027n\u0027ormal\n\nBug: 70304055\nTest: during recording: adb shell dumpsys media.audio_flinger\n look at active tracks under \"Input thread\"\n\nChange-Id: I8b745bd5f8a9b6b228129dd8981acea7fb230acd\n" }, { "commit": "12381092fdb5810cb27f738435ea8a0e3d2d0f42", "tree": "534768433f4916952e94e212fdb923b6a3ed5b40", "parents": [ "80ee2722d64a73777fa76027b1568e61dec8d910" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Apr 11 09:19:59 2018 -0700" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Apr 11 16:26:58 2018 +0000" }, "message": "Revert \"Revert \"Audio V4: Send volume and duplicated track attr in update metadata\"\"\n\nThe original patch was breaking the fast track volume, so it was\nreverted.\nIn order to fix the original patch, revert the revert and in the\nfollowing patch apply the fix.\nThis makes review easier as only the difference to the original patch\nwill be present.\n\nBug: 38184704\nBug: 69623109\nBug: 77892019\nTest: Patch already tested for original patch and is tested for following\n patch that fixes this patch.\nThis reverts commit 80ee2722d64a73777fa76027b1568e61dec8d910.\nChange-Id: I06914911c2435df167fff05ce5ca40dd1ed90f39\n" }, { "commit": "80ee2722d64a73777fa76027b1568e61dec8d910", "tree": "e3e8744cefcf0785addf9ceb1519fceac3020293", "parents": [ "c86a7f7de52f0ff0059e8c6c4fa4619aa7fae5c5" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Apr 11 15:53:48 2018 +0000" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Wed Apr 11 16:09:19 2018 +0000" }, "message": "Revert \"Audio V4: Send volume and duplicated track attr in update metadata\"\n\nThis reverts commit c86a7f7de52f0ff0059e8c6c4fa4619aa7fae5c5.\n\nReason for revert: fast track volume broken\nTest: none, it is a revert\nBug: 77892019\nChange-Id: If00616afae5c069cf0b6bd7c2da2a6d4af72180e\n" }, { "commit": "c86a7f7de52f0ff0059e8c6c4fa4619aa7fae5c5", "tree": "534768433f4916952e94e212fdb923b6a3ed5b40", "parents": [ "b31f96ffee12f920d32d868c627cf34f1afeb988" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Tue Apr 03 09:00:09 2018 -0700" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Mon Apr 09 16:19:11 2018 -0700" }, "message": "Audio V4: Send volume and duplicated track attr in update metadata\n\nOnly the volume of MMAP thread was previously sent and\nthe attributes of duplicated tracks were skipped.\n\nBug: 38184704\nBug: 69623109\nTest: check that the HAL receives correct track metadata volume\nTest: check that the HAL receives correct duplicated track metadata\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n\nChange-Id: I1e3b14edd8c9a5a65d2b9f4cf8660997c81c701c\n" }, { "commit": "0967c6c06d030636e2c584ce3e24924357a3a854", "tree": "9614233f9011d7645c95a1b84b610aa609845fe3", "parents": [ "2383098d54b6503875bdf0c6b649fa66f22d79ac", "1d3556d6a068f141ac390a04f6a9d92a70f8d7ed" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Apr 05 19:36:31 2018 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Apr 05 19:36:31 2018 +0000" }, "message": "Merge \"AudioTrack: Prevent server from reading client data after stop\" into pi-dev" }, { "commit": "1d3556d6a068f141ac390a04f6a9d92a70f8d7ed", "tree": "e89b4aa887013efbc8227dcf4c56addf4ab284b9", "parents": [ "1f03085b596f185d91ccae11b7c0d9f0bc8cd206" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Thu Mar 29 16:30:14 2018 -0700" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Mon Apr 02 14:19:24 2018 -0700" }, "message": "AudioTrack: Prevent server from reading client data after stop\n\nPrior to this CL, if an AudioTrack client wrote audio data after\nAudioTrack stop(), but before the track was drained by the server,\nthe newly written client data would be consumed in the drain.\n\nWe now limit the server read to the client write position on stop.\nThis interlocking is essential for rapid asynchronous AudioTrack\ncommand processing.\n\nTest: AudioTrack CTS, SoundPool looping, bug test case\nBug: 75788313\nChange-Id: Ib70e3dc46afe047a8c6cf1fb906a618b3c66cc7f\n" }, { "commit": "1f564acca544826daaf7dca44e39cec6016b82fd", "tree": "4adc2f15866c6c7bb47f7eb351aa251a94b1aaa2", "parents": [ "a8975a7e857fd745fc88d226d86b53ae088eafb9" ], "author": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Thu Mar 29 13:53:10 2018 -0700" }, "committer": { "name": "Kevin Rocard", "email": "krocard@google.com", "time": "Fri Mar 30 13:15:05 2018 -0700" }, "message": "Audio V4: Propagate audio attributes to tracks\n\nThis patch has not functional changes.\n\nBug: 38184704\nBug: 69623109\nTest: play music and a audio\nChange-Id: Ide6ec7de4dbfab4af2653475901c3b3daee36e52\nSigned-off-by: Kevin Rocard \u003ckrocard@google.com\u003e\n" }, { "commit": "1bc088a918d7038603230637d640941953b314d4", "tree": "e231c2b22d944eaed29445fe4a35861fddb50e54", "parents": [ "e54461ae132a79d0ffa9d3ec4631df13eb5c7809" ], "author": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Feb 09 15:57:31 2018 -0800" }, "committer": { "name": "Andy Hung", "email": "hunga@google.com", "time": "Fri Feb 16 15:50:32 2018 -0800" }, "message": "Call AudioMixer only from MixerThread threadLoop.\n\nAs part of change:\nRemove track name offset by TRACK0.\nMove track name management to the Tracks class.\nSync mixer track name to FastMixer track index.\n\nFixes regression introduced by commit 8ed196a.\n\nTest: SoundPool, AudioTrack CTS, Usability\nBug: 72937362\nBug: 73004420\nChange-Id: I2f1a33f6f0da66bcd7aa91e2a4b663ff822df645\n" }, { "commit": "fee1976a2849c37a53d8a01ac10327d522a1ba93", "tree": "5b8a259a93f6f84a964badf5417c113a308e67c9", "parents": [ "98c15702ca0c814fee9110efa14d70fc9937514e" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Jan 29 18:44:13 2018 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Fri Feb 09 16:08:31 2018 -0800" }, "message": "Revert \"Revert \"audio policy: refactor audio record APIs\"\"\n\nThis reverts commit e9ebcdbb0580bd4a75f4530b957b1859e535c028.\n\nBug: 72628781\nTest: manual camcorder tests\nChange-Id: I9fb550a9f62d2d15cafbd7534e5e7eaa155a7213\n" }, { "commit": "5dd03a5825d06033c1e66353dbbfcfc7f60990c5", "tree": "9deed77dba8b49bf83bf5f039c4ebaee3aa5b34c", "parents": [ "5dca017406b057bf71f8934ef57aab6098ecec6c", "653cc0ab6d5b714d1f26adcd08f9bc3269b62fa0" ], "author": { "name": "TreeHugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Feb 01 01:35:26 2018 +0000" }, "committer": { "name": "Android (Google) Code Review", "email": "android-gerrit@google.com", "time": "Thu Feb 01 01:35:26 2018 +0000" }, "message": "Merge \"Support query active microphones information in AudioRecord.\"" }, { "commit": "e9ebcdbb0580bd4a75f4530b957b1859e535c028", "tree": "a603313572aeef80fb8a26d2ff38f7a6541e835c", "parents": [ "67651f97c9c3880a4440c86618337f8a64c3e1a9" ], "author": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Jan 29 14:27:18 2018 -0800" }, "committer": { "name": "Eric Laurent", "email": "elaurent@google.com", "time": "Mon Jan 29 14:37:24 2018 -0800" }, "message": "Revert \"audio policy: refactor audio record APIs\"\n\nThis reverts commit 0f4b3c5449f85c1cd78e1b9ac4850de962b8edbe.\n\nBug: 72628781\nTest: Capture several videos from camera app\nChange-Id: I6bcc87c618ac9bfe5c911915cbb37de616bc727a\n" }, { "commit": "653cc0ab6d5b714d1f26adcd08f9bc3269b62fa0", "tree": "6ce8eb9ed7297d8b2524c6e25376319175758f50", "parents": [ "46a76fa5ffb88dbde2c67335dbc5cf20b75b10b6" ], "author": { "name": "jiabin", "email": "jiabin@google.com", "time": "Wed Jan 17 17:54:10 2018 -0800" }, "committer": { "name": "jiabin", "email": "jiabin@google.com", "time": "Fri Jan 26 17:23:07 2018 -0800" }, "message": "Support query active microphones information in AudioRecord.\n\nThis is part of device enumeration. With the new add API, developer\ncould get the active microphones information for each channel.\n\nBug: 64038649\nTest: Run cts and check the print log.\nChange-Id: Ic63d86e533a30e40697da7522a5a81f7cfcea988\n" } ], "next": "0f4b3c5449f85c1cd78e1b9ac4850de962b8edbe" }