Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 1 | page.title=Avoiding Priority Inversion |
| 2 | @jd:body |
| 3 | |
Clay Murphy | bc92aea | 2014-10-16 10:13:18 -0700 | [diff] [blame] | 4 | <!-- |
| 5 | Copyright 2013 The Android Open Source Project |
| 6 | |
| 7 | Licensed under the Apache License, Version 2.0 (the "License"); |
| 8 | you may not use this file except in compliance with the License. |
| 9 | You may obtain a copy of the License at |
| 10 | |
| 11 | http://www.apache.org/licenses/LICENSE-2.0 |
| 12 | |
| 13 | Unless required by applicable law or agreed to in writing, software |
| 14 | distributed under the License is distributed on an "AS IS" BASIS, |
| 15 | WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 16 | See the License for the specific language governing permissions and |
| 17 | limitations under the License. |
| 18 | --> |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 19 | <div id="qv-wrapper"> |
| 20 | <div id="qv"> |
| 21 | <h2>In this document</h2> |
| 22 | <ol id="auto-toc"> |
| 23 | </ol> |
| 24 | </div> |
| 25 | </div> |
| 26 | |
| 27 | <p> |
| 28 | This article explains how the Android's audio system attempts to avoid |
Glenn Kasten | 978bec8 | 2014-12-23 15:15:20 -0800 | [diff] [blame^] | 29 | priority inversion, |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 30 | and highlights techniques that you can use too. |
| 31 | </p> |
| 32 | |
| 33 | <p> |
| 34 | These techniques may be useful to developers of high-performance |
| 35 | audio apps, OEMs, and SoC providers who are implementing an audio |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 36 | HAL. Please note implementing these techniques is not |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 37 | guaranteed to prevent glitches or other failures, particularly if |
| 38 | used outside of the audio context. |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 39 | Your results may vary, and you should conduct your own |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 40 | evaluation and testing. |
| 41 | </p> |
| 42 | |
| 43 | <h2 id="background">Background</h2> |
| 44 | |
| 45 | <p> |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 46 | The Android AudioFlinger audio server and AudioTrack/AudioRecord |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 47 | client implementation are being re-architected to reduce latency. |
Glenn Kasten | 978bec8 | 2014-12-23 15:15:20 -0800 | [diff] [blame^] | 48 | This work started in Android 4.1, and continued with further improvements |
| 49 | in 4.2, 4.3, 4.4, and 5.0. |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 50 | </p> |
| 51 | |
| 52 | <p> |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 53 | To achieve this lower latency, many changes were needed throughout the system. One |
| 54 | important change is to assign CPU resources to time-critical |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 55 | threads with a more predictable scheduling policy. Reliable scheduling |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 56 | allows the audio buffer sizes and counts to be reduced while still |
Glenn Kasten | 978bec8 | 2014-12-23 15:15:20 -0800 | [diff] [blame^] | 57 | avoiding underruns and overruns. |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 58 | </p> |
| 59 | |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 60 | <h2 id="priorityInversion">Priority inversion</h2> |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 61 | |
| 62 | <p> |
| 63 | <a href="http://en.wikipedia.org/wiki/Priority_inversion">Priority inversion</a> |
| 64 | is a classic failure mode of real-time systems, |
| 65 | where a higher-priority task is blocked for an unbounded time waiting |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 66 | for a lower-priority task to release a resource such as (shared |
| 67 | state protected by) a |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 68 | <a href="http://en.wikipedia.org/wiki/Mutual_exclusion">mutex</a>. |
| 69 | </p> |
| 70 | |
| 71 | <p> |
| 72 | In an audio system, priority inversion typically manifests as a |
| 73 | <a href="http://en.wikipedia.org/wiki/Glitch">glitch</a> |
| 74 | (click, pop, dropout), |
| 75 | <a href="http://en.wikipedia.org/wiki/Max_Headroom_(character)">repeated audio</a> |
| 76 | when circular buffers |
| 77 | are used, or delay in responding to a command. |
| 78 | </p> |
| 79 | |
| 80 | <p> |
| 81 | In the Android audio implementation, priority inversion is most |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 82 | likely to occur in these places. And so you should focus your attention here: |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 83 | </p> |
| 84 | |
| 85 | <ul> |
| 86 | |
| 87 | <li> |
| 88 | between normal mixer thread and fast mixer thread in AudioFlinger |
| 89 | </li> |
| 90 | |
| 91 | <li> |
| 92 | between application callback thread for a fast AudioTrack and |
| 93 | fast mixer thread (they both have elevated priority, but slightly |
| 94 | different priorities) |
| 95 | </li> |
| 96 | |
| 97 | <li> |
Glenn Kasten | 978bec8 | 2014-12-23 15:15:20 -0800 | [diff] [blame^] | 98 | between application callback thread for a fast AudioRecord and |
| 99 | fast capture thread (similar to previous) |
| 100 | </li> |
| 101 | |
| 102 | <li> |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 103 | within the audio Hardware Abstraction Layer (HAL) implementation, e.g. for telephony or echo cancellation |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 104 | </li> |
| 105 | |
| 106 | <li> |
| 107 | within the audio driver in kernel |
| 108 | </li> |
| 109 | |
| 110 | <li> |
Glenn Kasten | 978bec8 | 2014-12-23 15:15:20 -0800 | [diff] [blame^] | 111 | between AudioTrack or AudioRecord callback thread and other app threads (this is out of our control) |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 112 | </li> |
| 113 | |
| 114 | </ul> |
| 115 | |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 116 | <h2 id="commonSolutions">Common solutions</h2> |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 117 | |
| 118 | <p> |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 119 | The typical solutions include: |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 120 | </p> |
| 121 | |
| 122 | <ul> |
| 123 | |
| 124 | <li> |
| 125 | disabling interrupts |
| 126 | </li> |
| 127 | |
| 128 | <li> |
| 129 | priority inheritance mutexes |
| 130 | </li> |
| 131 | |
| 132 | </ul> |
| 133 | |
| 134 | <p> |
| 135 | Disabling interrupts is not feasible in Linux user space, and does |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 136 | not work for Symmetric Multi-Processors (SMP). |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 137 | </p> |
| 138 | |
| 139 | |
| 140 | <p> |
| 141 | Priority inheritance |
| 142 | <a href="http://en.wikipedia.org/wiki/Futex">futexes</a> |
| 143 | (fast user-space mutexes) are available |
| 144 | in Linux kernel, but are not currently exposed by the Android C |
| 145 | runtime library |
| 146 | <a href="http://en.wikipedia.org/wiki/Bionic_(software)">Bionic</a>. |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 147 | They are not used in the audio system because they are relatively heavyweight, |
| 148 | and because they rely on a trusted client. |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 149 | </p> |
| 150 | |
| 151 | <h2 id="androidTechniques">Techniques used by Android</h2> |
| 152 | |
| 153 | <p> |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 154 | Experiments started with "try lock" and lock with timeout. These are |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 155 | non-blocking and bounded blocking variants of the mutex lock |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 156 | operation. Try lock and lock with timeout worked fairly well but were |
| 157 | susceptible to a couple of obscure failure modes: the |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 158 | server was not guaranteed to be able to access the shared state if |
| 159 | the client happened to be busy, and the cumulative timeout could |
| 160 | be too long if there was a long sequence of unrelated locks that |
| 161 | all timed out. |
| 162 | </p> |
| 163 | |
| 164 | |
| 165 | <p> |
| 166 | We also use |
| 167 | <a href="http://en.wikipedia.org/wiki/Linearizability">atomic operations</a> |
| 168 | such as: |
| 169 | </p> |
| 170 | |
| 171 | <ul> |
| 172 | <li>increment</li> |
| 173 | <li>bitwise "or"</li> |
| 174 | <li>bitwise "and"</li> |
| 175 | </ul> |
| 176 | |
| 177 | <p> |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 178 | All of these return the previous value and include the necessary |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 179 | SMP barriers. The disadvantage is they can require unbounded retries. |
| 180 | In practice, we've found that the retries are not a problem. |
| 181 | </p> |
| 182 | |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 183 | <p class="note"><strong>Note:</strong> Atomic operations and their interactions with memory barriers |
Clay Murphy | 5d83ab4 | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 184 | are notoriously badly misunderstood and used incorrectly. We include these methods |
| 185 | here for completeness but recommend you also read the article |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 186 | <a href="https://developer.android.com/training/articles/smp.html"> |
| 187 | SMP Primer for Android</a> |
| 188 | for further information. |
| 189 | </p> |
| 190 | |
| 191 | <p> |
| 192 | We still have and use most of the above tools, and have recently |
| 193 | added these techniques: |
| 194 | </p> |
| 195 | |
| 196 | <ul> |
| 197 | |
| 198 | <li> |
| 199 | Use non-blocking single-reader single-writer |
| 200 | <a href="http://en.wikipedia.org/wiki/Circular_buffer">FIFO queues</a> |
| 201 | for data. |
| 202 | </li> |
| 203 | |
| 204 | <li> |
| 205 | Try to |
| 206 | <i>copy</i> |
| 207 | state rather than |
| 208 | <i>share</i> |
| 209 | state between high- and |
| 210 | low-priority modules. |
| 211 | </li> |
| 212 | |
| 213 | <li> |
| 214 | When state does need to be shared, limit the state to the |
| 215 | maximum-size |
| 216 | <a href="http://en.wikipedia.org/wiki/Word_(computer_architecture)">word</a> |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 217 | that can be accessed atomically in one-bus operation |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 218 | without retries. |
| 219 | </li> |
| 220 | |
| 221 | <li> |
| 222 | For complex multi-word state, use a state queue. A state queue |
| 223 | is basically just a non-blocking single-reader single-writer FIFO |
| 224 | queue used for state rather than data, except the writer collapses |
| 225 | adjacent pushes into a single push. |
| 226 | </li> |
| 227 | |
| 228 | <li> |
| 229 | Pay attention to |
| 230 | <a href="http://en.wikipedia.org/wiki/Memory_barrier">memory barriers</a> |
| 231 | for SMP correctness. |
| 232 | </li> |
| 233 | |
| 234 | <li> |
| 235 | <a href="http://en.wikipedia.org/wiki/Trust,_but_verify">Trust, but verify</a>. |
| 236 | When sharing |
| 237 | <i>state</i> |
| 238 | between processes, don't |
| 239 | assume that the state is well-formed. For example, check that indices |
| 240 | are within bounds. This verification isn't needed between threads |
| 241 | in the same process, between mutual trusting processes (which |
| 242 | typically have the same UID). It's also unnecessary for shared |
| 243 | <i>data</i> |
| 244 | such as PCM audio where a corruption is inconsequential. |
| 245 | </li> |
| 246 | |
| 247 | </ul> |
| 248 | |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 249 | <h2 id="nonBlockingAlgorithms">Non-blocking algorithms</h2> |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 250 | |
| 251 | <p> |
| 252 | <a href="http://en.wikipedia.org/wiki/Non-blocking_algorithm">Non-blocking algorithms</a> |
| 253 | have been a subject of much recent study. |
| 254 | But with the exception of single-reader single-writer FIFO queues, |
| 255 | we've found them to be complex and error-prone. |
| 256 | </p> |
| 257 | |
| 258 | <p> |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 259 | Starting in Android 4.2, you can find our non-blocking, |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 260 | single-reader/writer classes in these locations: |
| 261 | </p> |
| 262 | |
| 263 | <ul> |
| 264 | |
| 265 | <li> |
| 266 | frameworks/av/include/media/nbaio/ |
| 267 | </li> |
| 268 | |
| 269 | <li> |
| 270 | frameworks/av/media/libnbaio/ |
| 271 | </li> |
| 272 | |
| 273 | <li> |
| 274 | frameworks/av/services/audioflinger/StateQueue* |
| 275 | </li> |
| 276 | |
| 277 | </ul> |
| 278 | |
| 279 | <p> |
| 280 | These were designed specifically for AudioFlinger and are not |
| 281 | general-purpose. Non-blocking algorithms are notorious for being |
Clay Murphy | c28f237 | 2013-09-25 16:13:40 -0700 | [diff] [blame] | 282 | difficult to debug. You can look at this code as a model. But be |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 283 | aware there may be bugs, and the classes are not guaranteed to be |
| 284 | suitable for other purposes. |
| 285 | </p> |
| 286 | |
| 287 | <p> |
Clay Murphy | 5d83ab4 | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 288 | For developers, some of the sample OpenSL ES application code should be updated to |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 289 | use non-blocking algorithms or reference a non-Android open source library. |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 290 | </p> |
| 291 | |
| 292 | <h2 id="tools">Tools</h2> |
| 293 | |
| 294 | <p> |
| 295 | To the best of our knowledge, there are no automatic tools for |
| 296 | finding priority inversion, especially before it happens. Some |
| 297 | research static code analysis tools are capable of finding priority |
| 298 | inversions if able to access the entire codebase. Of course, if |
| 299 | arbitrary user code is involved (as it is here for the application) |
| 300 | or is a large codebase (as for the Linux kernel and device drivers), |
| 301 | static analysis may be impractical. The most important thing is to |
| 302 | read the code very carefully and get a good grasp on the entire |
| 303 | system and the interactions. Tools such as |
| 304 | <a href="http://developer.android.com/tools/help/systrace.html">systrace</a> |
| 305 | and |
| 306 | <code>ps -t -p</code> |
| 307 | are useful for seeing priority inversion after it occurs, but do |
| 308 | not tell you in advance. |
| 309 | </p> |
| 310 | |
Clay Murphy | 3a7af3a | 2014-09-09 17:29:09 -0700 | [diff] [blame] | 311 | <h2 id="aFinalWord">A final word</h2> |
Glenn Kasten | 98afa53 | 2013-04-15 14:02:36 -0700 | [diff] [blame] | 312 | |
| 313 | <p> |
| 314 | After all of this discussion, don't be afraid of mutexes. Mutexes |
| 315 | are your friend for ordinary use, when used and implemented correctly |
| 316 | in ordinary non-time-critical use cases. But between high- and |
| 317 | low-priority tasks and in time-sensitive systems mutexes are more |
| 318 | likely to cause trouble. |
| 319 | </p> |
| 320 | |