History log of /external/adhd/cras/src/tests/rstream_unittest.cc
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
1ae0e348e987bcd17cd27e96b8e72936e9297890 04-Jan-2017 Chinyue Chen <chinyue@chromium.org> CRAS: ucm - Use "Speech" instead of "Voice" for UCM verb.

Use "Speech" as UCM verb and CRAS_STREAM_TYPE_SPEECH_RECOGNITION
as stream type so that it's easier to tell them from "Voice Call"
and CRAS_STREAM_TYPE_VOICE_COMMUNICATION.

BUG=None
TEST=make check

Change-Id: I9fc9dcceef920ba8a831bfcc0a4c595d43692888
Reviewed-on: https://chromium-review.googlesource.com/424496
Commit-Ready: Chinyue Chen <chinyue@chromium.org>
Tested-by: Chinyue Chen <chinyue@chromium.org>
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
492ef0c27628a4b7725644998d5bf721aadb1da3 17-Oct-2016 Chinyue Chen <chinyue@chromium.org> CRAS: Support more stream types.

When a client opens a stream, CRAS can know from the stream type
specified to configure audio hardware by using corresponding UCM config
and apply DSP processing if needed.

This is the very first step to support audio use cases in CRAS.

BUG=chromium:643039
TEST=FEATURES="test" emerge-${BOARD} -v media-sound/adhd

Change-Id: I998eb2fba0c3a92b0012940db36539c7865fd498
Reviewed-on: https://chromium-review.googlesource.com/398942
Commit-Ready: Chinyue Chen <chinyue@chromium.org>
Tested-by: Chinyue Chen <chinyue@chromium.org>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
b4ec16539ec6001d4b56808b7795c29e4db7c249 15-Jun-2016 John Muir <muirj@google.com> CRAS: Fix some unittest files that don't build under Brillo.

Some files need to be modified to not include <sys/shm.h>.
One doesn't compile under a release build.

BUG=None
TEST=CRAS build and test passes in Chrome.
Brillo release build passes.
Brillo debug build passes.

Change-Id: I6c7e60f2a3380ce6e35ff9a4406b573a9130a15d
Reviewed-on: https://chromium-review.googlesource.com/353037
Commit-Ready: John Muir <muirj@google.com>
Tested-by: John Muir <muirj@google.com>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
c32f7fdd0c91929a43bb854b02969d9840d639c1 13-Nov-2015 Dylan Reid <dgreid@chromium.org> CRAS: rstream - use posix shm for stream audio.

Change over to posix shm so an fd can be passed to a client in another
namespace and the client can play or record audio.

Change-Id: I72991117f0178b55a56980480f90a8e8257390fc
Reviewed-on: https://chromium-review.googlesource.com/312772
Commit-Ready: Dylan Reid <dgreid@chromium.org>
Tested-by: Dylan Reid <dgreid@chromium.org>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
f37fe057d81f40bd7c808ad67e9079169da262b0 13-Nov-2015 Dylan Reid <dgreid@chromium.org> CRAS: rstream_ut - Remove unused stub

rstream no longer calls this so remove the stub

Change-Id: I2a05840a9440887e603c9bb3bf26dd31b6038f84
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/312549
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
2eac24b5c760e73a9fda04374d43a1000cb0a74a 26-Mar-2015 Dylan Reid <dgreid@chromium.org> CRAS: move rstream create/destroy to stream_list

Create rstreams when adding them to the list. Destroy rstreams when
they are removed from the list. This will allow for delaying the
destruction while they drain. Keeping creation and destruction in one
file makes managing the memory easier too.

Change-Id: I7772337a4af715e83fb53e2d4e7f76034df74801
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/262713
Reviewed-by: Chinyue Chen <chinyue@chromium.org>
Commit-Queue: Chinyue Chen <chinyue@chromium.org>
Tested-by: Chinyue Chen <chinyue@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
37972eb088d1f05664f0d6b5645e94cbbfdafeac 25-Mar-2015 Dylan Reid <dgreid@chromium.org> CRAS: Move some book keeping to rstream

rclient was managing setting the audio fd and notifying the system_state
when an rstream was added. Move that logic to rstream. This will allow
for easier passing of rstream to stream list to facilitate draining the
stream before removing it.

Change-Id: Id3570b715532d57d712902a0708d1db4435804f0
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/262513
Reviewed-by: Chinyue Chen <chinyue@chromium.org>
Commit-Queue: Chinyue Chen <chinyue@chromium.org>
Tested-by: Chinyue Chen <chinyue@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
a7978895f3f660c5691887f83fda26169013247d 14-Mar-2015 Dylan Reid <dgreid@chromium.org> CRAS: Use stream_list instead of per client list

For now this doesn't change anything functionally. In future commits,
stream_list will become the canonical list of streams that are attached
to the system.

Change-Id: I18060b4cbd617da7362153eba1ff8828782130b4
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/262506
/external/adhd/cras/src/tests/rstream_unittest.cc
5aa5e261e5d66a74360fe8519f796a3622acbc9a 03-Dec-2014 Dylan Reid <dgreid@chromium.org> CRAS: rstream - rename get_buffer_size to get_buffer_frames

Make the name say what it means. Do this before adding a user for
this function.

Change-Id: Iaba6fb61373f388d42a01815d1822e9d295b6b87
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/233100
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
Reviewed-by: Chinyue Chen <chinyue@chromium.org>
Commit-Queue: Hsinyu Chao <hychao@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
71531caea43173e3d81e66415d2df6266fea82d4 03-Dec-2014 Dylan Reid <dgreid@chromium.org> CRAS: rstream - save flags.

The flags will be used now, save them when creating an rstream.

Change-Id: I2d0b2dc59c1800340c49bd990f9f4d6e7a238de0
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/233058
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
Reviewed-by: Chinyue Chen <chinyue@chromium.org>
Commit-Queue: Hsinyu Chao <hychao@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
fa7cd22a551aa724f671387d95092ebebc6f3a41 07-Oct-2014 Dylan Reid <dgreid@chromium.org> CRAS: buffer_share - rename dev to id.

buffer share will be used by other users, make the names more generic.

BUG=chroimum:420173
TEST=compiles

Change-Id: Ic8538139e8788db5f98977da001f3528514c2c1b
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/221961
Commit-Queue: Hsinyu Chao <hychao@chromium.org>
Tested-by: Hsinyu Chao <hychao@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
6c602764e9210491b6074b2a764d74a12d5340f0 26-Sep-2014 Dylan Reid <dgreid@chromium.org> CRAS: Rename input/dev_mix to buffer_share.

This can be used to manage the read level of multiple devices from and
rstream as well, make the names more generic.

Change-Id: Iec7aec55ba0832dceb20069a93c3d09daf1b6134
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/220110
/external/adhd/cras/src/tests/rstream_unittest.cc
0b123a52b29a09c563535a576dcc6a6eb9497b2d 24-Sep-2014 Dylan Reid <dgreid@chromium.org> CRAS: dev_stream - remove redundant mix_offset member.

This information is now kept in the input_mix state within rstream.

Change-Id: Ic12fc4e3f1e7dca61f555b76cb75b0386e095484
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/219975
/external/adhd/cras/src/tests/rstream_unittest.cc
ed0e3ba0dc54a1ddd192ae571e15c0acebacbc81 23-Sep-2014 Dylan Reid <dgreid@chromium.org> CRAS: Make input streams work with new timing.

This gets input streams working with the new timing scheme. It also
lays the foundation for multiple input devices to feed one stream.

Change-Id: I39334dae100527366cf344b7b810b893bdbd866d
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/219967
/external/adhd/cras/src/tests/rstream_unittest.cc
c378b8b7289d9c18b221069f1644fcae765eb47c 09-Aug-2014 Dylan Reid <dgreid@chromium.org> CRAS: rstream - Remove unused flags variable.

After three years we haven't found a use for flags, remove it.

BUG=none
TEST=make check

Change-Id: I4a3e7af0c23a27d5fedac8f3a21bd5d7aa6580da
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/216542
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
b52bb1c97e6420ce2e1c27a68c9d23576c6412f0 09-Aug-2014 Dylan Reid <dgreid@chromium.org> CRAS: Remove min_cb_level.

min_db_level hasn't been used in a very long time, for input it is
totally ignored for output it was always set to cb_threshold anyways.

BUg=none
TEST=make check, and play audio on Blaze

Change-Id: I230c763d1db8e4816fe45e146c6afdc39438f132
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/216509
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
1b6da1337657b5f525762db7800eb51784f5b898 16-Jun-2014 Dylan Reid <dgreid@chromium.org> CRAS: rstream - Keep an audio area for input and output.

Add an audio_area member for input and output streams. Later changes
will use this to match captured channels with stream channels.

BUG=none
TEST=none

Change-Id: I02cf29077ce433770cf1e6e1128aa14bea15670c
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/204671
Tested-by: Hsinyu Chao <hychao@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
3390915103edddf0576018402d81ad09c0bacdb1 14-Mar-2013 Chih-Chung Chang <chihchung@chromium.org> CRAS: Fix issues found by running valgrind on unit tests.

The issues are memory leak and uninitialized memory access.

After this change, all unit tests other than audio_thread_unittest and
rclient_unittest (they have issues only in unit tests and harder to fix)
run cleanly under valgrind.

BUG=none
TEST=make check and run each unit test under valgrind

Change-Id: I72829f1e88b5927eee58f438cd580563a73d95bb
Reviewed-on: https://gerrit.chromium.org/gerrit/45415
Reviewed-by: Dylan Reid <dgreid@chromium.org>
Tested-by: Dylan Reid <dgreid@chromium.org>
Commit-Queue: Chih-Chung Chang <chihchung@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
9edf52fea7a0fc8fa45b42b9efca38ed9e3ba96a 28-Nov-2012 Dylan Reid <dgreid@chromium.org> CRAS: rstream - Store input and output shm pointer.

Allow a stream to have both input and output shm areas. Only one of
the two is used until a following commit enables unified input/output
streams.

BUG=chromium-os:35272
TEST=updated unittests and play audio tiwh test client.

Change-Id: I00f4b3090767f4f0c0d286a6b649b2719ea28ce1
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/40958
Reviewed-by: Chih-Chung Chang <chihchung@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
cdf0a914c9542183736eba27e756f8468354f0ff 27-Nov-2012 Dylan Reid <dgreid@chromium.org> CRAS: rstream UT - Add input creation test.

BUG=none
TEST=this is a test.

Change-Id: I307df7c1ee0b0d9b7884ca16d8c3b0391ffeeb5e
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/40956
Reviewed-by: Chih-Chung Chang <chihchung@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
dd7bec3156b6a8da1f7bf954c78726f82e4a5cb8 06-Aug-2012 Dylan Reid <dgreid@chromium.org> CRAS: shm - Use volume scaler setter/getter.

A few places were accessing the member directly instead of using the
setter/getter.

BUG=chromium-os:33123
TEST=updated unittests.

Change-Id: Idb35ec87510ae69748d2867205413796723812f1
Reviewed-on: https://gerrit.chromium.org/gerrit/29291
Commit-Ready: Dylan Reid <dgreid@chromium.org>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
Tested-by: Dylan Reid <dgreid@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
991d36c2af3a07bde6fdbe9282d506ba5c915dcc 06-Aug-2012 Dylan Reid <dgreid@chromium.org> CRAS: shm - Remove size member.

This member was only being used in one place and can be easily
calculated. Add a function instead to return the total size of shm.

BUG=chromium-os:3312
TEST=updated unittests.

Change-Id: I367e240cfc995579a772ea8170cd8cbc52ef6262
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/29288
/external/adhd/cras/src/tests/rstream_unittest.cc
ce895e7bb75a63c593bec7a5a36bfed3241ab829 26-Jan-2012 Dylan Reid <dgreid@chromium.org> CRAS: Name enums, and favor size_t instead of uint32_t.

Give commonly used enums names, and use the names where the enum
values were previous passed as basic types.
Favor using size_t to uint32_t where we just want an unsigned
type.

BUG=none
TEST=make check and run cras on Mario.

Change-Id: I3b81b5963025e4a522030bd736e5fd009fc3e8f8
Signed-off-by: Dylan Reid <dgreid@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc
9f34e01d4c1d36df1214612512bf7c5833ddff3b 19-Jan-2012 Dylan Reid <dgreid@chromium.org> CRAS: Add cras_rstream.

cras_rstream is used for managing client streams attached to the
server.

BUG=chrome-os-partner:6382
TEST=supplied rstream_unittest

Change-Id: I942902cd079b75e469a77eab78d8314d0c21414e
Signed-off-by: Dylan Reid <dgreid@chromium.org>
/external/adhd/cras/src/tests/rstream_unittest.cc