| 35d10a7c78a88c680910edfdf2869ba871cc3db0 |
|
19-Nov-2016 |
Glenn Kasten <gkasten@google.com> |
Fix a few clang-tidy warnings
Test: still builds
Change-Id: I6583f4f600a24a0784cdda186f77d5cacffc6cc5
/system/media/audio_utils/tests/fifo_tests.cpp
|
| c0924bc38334c7adbf8cbc9cfa9d00559c002652 |
|
02-Oct-2016 |
Glenn Kasten <gkasten@google.com> |
fifo: improve blocking and hysteresis
Work-in-progress to eventually allow the application to independently
specify whether or not each index (both front and rear) is also treated
as a futex, and if so whether it is single-process or multi-process.
This involves replacing mIsPrivate by an enum for futex types.
Still to do:
- provide an API to configure this
- better testing
Continued work on the setHysteresis() API.
Still to do:
- better testing and debugging, especially for the read side.
- do the right thing based on data currently in the buffer
Added available() API to return the number of frames that could
be obtain()ed. For reader, this is the fill level.
Miscellaneous:
- renamed sharedRear to writerRear
- improved Doxygen comments
- address code review comments by explaining why some paramters are
passed by reference and some as pointers
- updated tests
- fixed bug: iovec was not being clear on error
- changed default value of throttlesWriter to true
Test: tests/fifo_*
Change-Id: I0e19052abc90453ca48a0299d2c2de62468de6cb
/system/media/audio_utils/tests/fifo_tests.cpp
|
| 6d7ad76b2a2b04f3a61605ca1ca019a80b94d339 |
|
16-Jun-2016 |
Glenn Kasten <gkasten@google.com> |
Add multi-reader support
At most one reader throttles writer.
All other readers must keep up with writer, or are informed of lost frame count.
Refactor reader, writer, and buffer provider APIs as separate classes.
Change-Id: If2007a4f56d40ac892e899632d261fcfee0ea9e4
/system/media/audio_utils/tests/fifo_tests.cpp
|
| 9b4fe47208fbc80e5a56508fc8e7210fa4d888ad |
|
13-Jun-2016 |
Glenn Kasten <gkasten@google.com> |
Replace fifo C API and implementation by C++
Improvements to library:
- Replace Android atomics by C++ atomics
- struct -> class
- init -> constructor, deinit -> destructor
- Avoid signed arithmetic for security
- Fix usage in test
- Add README
- Improve error recovery by returning a status_t if indices are corrupt
- Fix bug in diff when assertions enabled
- Add local indices in preparation for multi-reader and protected pages
- Add more documentation for internal methods
Improvements to unit test:
- More error checks
- Fix double delete of fifoBuffer
- Support 8-bit wav files
- Enable assertion checks
Change-Id: I8b391b17084084d8e692765ed210a35aaa0b6747
/system/media/audio_utils/tests/fifo_tests.cpp
|
| bb2a8c3b9baecc508b30fdc20337a9326f35c5f9 |
|
21-Oct-2015 |
Glenn Kasten <gkasten@google.com> |
Add missing delete[]
Change-Id: I15c0ecb49d0e82db05e6857c338aa7c07b35598c
/system/media/audio_utils/tests/fifo_tests.cpp
|
| 328eb02a68c7f90af767173499cd77a6e2b457ae |
|
06-Jun-2015 |
Jim Miller <jaggies@google.com> |
Fix Mac build
ssize_t wasn't being picked because system headers weren't included
first.
Change-Id: I7748fedaa0c85ed9696e8ef77b0086ecf17ebe29
/system/media/audio_utils/tests/fifo_tests.cpp
|
| b598a7352fc85945f7fba1abdd646ddd621df58a |
|
14-Jan-2015 |
Glenn Kasten <gkasten@google.com> |
Add fifo_tests
Change-Id: I62a783bf551fd6c6c9e734b550f8bcc4e2a32df3
/system/media/audio_utils/tests/fifo_tests.cpp
|