| 159e03c9c1b6993f7649dcfe468fad2753994f02 |
|
14-Apr-2017 |
Mikhail Naganov <mnaganov@google.com> |
libaudiohal: Remove dependencies on libeffects and libhardware
These dependencies are only needed when building with
USE_LEGACY_LOCAL_AUDIO_HAL=true.
Test: adb shell lsof | grep libeffects
Bug: 33241851
Change-Id: I9c5eaf52d02c2101c55ea420780d0eb435d6493a
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| 1ba4041b6248aff42c01d27832995ebdf431bbd1 |
|
20-Mar-2017 |
Mikhail Naganov <mnaganov@google.com> |
audiohal: Exit instead of crashing when HAL factory can't be obtained
Replace LOG_ALWAYS_FATAL with ALOGE + exit, so the process still
restarts, but without a crash. This avoids confusion in bug triaging
and also helps to eliminate noise in bug reports statistics.
Bug: 36424777
Change-Id: I623cbc6a5b0d3351f849810dd67708d421efd61a
Test: reproduced error condition, checked that audio server restarts
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| 425a50282ebdcc68a4fc01a268be87426889152e |
|
14-Mar-2017 |
Mikhail Naganov <mnaganov@google.com> |
audiohal: Restart audioserver if HAL services can't be obtained
HAL services are crucial for the audioflinger to work.
If for some reason device or effect factories can't be obtained,
log a error message and crash af. If the problem with obtaining
services is transient, restarting af helps to restore audio.
Bug: 36153684
Change-Id: Iec17a6c6e19650cf1d1de1ec30caf05753b9bce1
Test: reproed the bug, checked that af restarts and finds devices
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| a843e7bac59fd8a97ab3b4d5171c5d21bbd325fd |
|
13-Mar-2017 |
Steven Moreland <smoreland@google.com> |
audio: remove unused using statements.
Bug: 36099713
Test: pass
Change-Id: Ib7a48469788a60778186368667c7043c33129e91
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| 3516b25e0ef2cdc8fe9f13234d10b48612db8bbb |
|
22-Feb-2017 |
Steven Moreland <smoreland@google.com> |
Remove waitForHwService for ashmem.
Now that b/34772726 is fixed, we can remove this temporary code.
Bug: 34772726
Test: audio works on marlin
Change-Id: I1a268e7030d27b4e944659bd7d82ba321be13a6e
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| dfa54faf298f6b617bf1d7684418fc8a2e6e4858 |
|
24-Jan-2017 |
Chris Phoenix <cphoenix@google.com> |
audio_effects HAL uses "default" service name
The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.
Test: builds; VTS HalAudioEffectHidlTargetBasicTest passes
Bug: 33844934
Change-Id: I5283bf7c2d0645bd38a16e92cab05103c1bcdf26
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| 13a926879249276771b0097538459c7550e2f5ce |
|
17-Feb-2017 |
Steven Moreland <smoreland@google.com> |
Use hidl.allocator for IAllocator.
android.hidl.memory is being split into android.hidl.memory and
android.hidl.allocator since the former is a passthrough hal and the
latter is a binderized service.
Bug: 35327976
Test: hidl_test, lshal, youtube/camera works
Change-Id: I7a2e72dcd3721d2689eb7ef95f5b140ecc1c446f
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| c2656e8c22a938bf3fe389c7473acc24649c7345 |
|
28-Jan-2017 |
Mikhail Naganov <mnaganov@google.com> |
audiohal: Wait for IAllocator to become available
IAllocator is a framework service, and currently dependency on it
isn't handled correctly.
Bug: 34771147
Change-Id: Id3d50dd3bf9456aa363ae20a30ad319cf9c72e10
Test: manual
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| e83be8af690ef1ac820a63414d522e77ca9d4db6 |
|
06-Jan-2017 |
Steven Moreland <smoreland@google.com> |
Remove usage of getStatus.
- remove from libaudiohal
- remove from services/radio
- remove from soundtrigger
- remove from omx
Bug: 31348667
Test: compiles
Change-Id: I75bb7b7388527dacfdc5bf128712cb8690bc286b
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| 9f57e3c9da8923649adff62e63c644537f565e90 |
|
05-Dec-2016 |
Mikhail Naganov <mnaganov@google.com> |
Crash audioserver if EPIPE (DEAD_OBJECT) is received from the HAL
audioflinger doesn't have a proper way to re-establish all
the opened streams if HAL crashes. The only valid way of dealing
with them is to restart the audioserver.
Although broadcastradio and sountrigger have cleaner protocols
for dealing with HAL failures and can handle HAL crash and restart,
since the audio side will anyway get broken, it makes sense
to also kill the audioserver when they detect HAL crash.
Change-Id: I5f8e8c1d0981042cdd435a3ddbb6526e9f226e6b
Test: kill audio-hal process, watch dmesg and logcat
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| 34e5d30e54650b210c1b9730b1e4b412ec1cdb0c |
|
10-Dec-2016 |
Samuel Saccone <samccone@google.com> |
Revert "Crash audioserver if EPIPE (DEAD_OBJECT) is received from the HAL"
This reverts commit 9f55e5f998644d58cb8f3625e06d9b7a4e78555c.
Change-Id: Ic76c0cb5032af422699da2b43106b4b1de5ac935
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| 9f55e5f998644d58cb8f3625e06d9b7a4e78555c |
|
05-Dec-2016 |
Mikhail Naganov <mnaganov@google.com> |
Crash audioserver if EPIPE (DEAD_OBJECT) is received from the HAL
audioflinger doesn't have a proper way to re-establish all
the opened streams if HAL crashes. The only valid way of dealing
with them is to restart the audioserver.
Although broadcastradio and sountrigger have cleaner protocols
for dealing with HAL failures and can handle HAL crash and restart,
since the audio side will anyway get broken, it makes sense
to also kill the audioserver when they detect HAL crash.
Test: kill audio-hal process, watch dmesg and logcat
Change-Id: Ib2150b69c791ac49de9404de08fc6c309a3fcbd5
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|
| f558e0218d2677a813c9c600886f673894eec927 |
|
15-Nov-2016 |
Mikhail Naganov <mnaganov@google.com> |
libaudiohal: Implement simple HIDL client
This client requires both HIDL client and server to be of the same
ABI version (e.g. 32-bit).
EffectHalHidl::process and EffectHalHidl::processReverse are stubs.
Some extra warnings are logged due to unimplemented functions
in the legacy HAL.
Bug: 30222631
Change-Id: Id074131dec6d6c4f1031695abcf3ea066e92a5af
Test: Play Music (play / EQ), YouTube, Loopback RTLT, Ok Google on N5X
/frameworks/av/media/libaudiohal/EffectsFactoryHalHidl.cpp
|