Commit Graph

331 Commits

Author SHA1 Message Date
Yiming Pan
7716a06508 Merge "Backfill owner information for VTS module VtsHalAudioV6_0TargetTest." am: 86e736283b am: fd3c91e19c am: ea0f857a55 am: 27d1fb72a9 am: 74b121c32f
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1814068

Change-Id: I66ea58bc1db7cba06a89b0d5d853b90cb5e885ec
2021-09-09 01:33:15 +00:00
Yiming Pan
74b121c32f Merge "Backfill owner information for VTS module VtsHalAudioV6_0TargetTest." am: 86e736283b am: fd3c91e19c am: ea0f857a55 am: 27d1fb72a9
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1814068

Change-Id: I97909ec55faf79c676feb8347eebd0aa8fe3f1c6
2021-09-09 01:18:32 +00:00
Yiming Pan
fd3c91e19c Merge "Backfill owner information for VTS module VtsHalAudioV6_0TargetTest." am: 86e736283b
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1814068

Change-Id: I5d66345749cff4df2faf38964c177a9829785588
2021-09-09 00:39:39 +00:00
Yiming Pan
86e736283b Merge "Backfill owner information for VTS module VtsHalAudioV6_0TargetTest." 2021-09-09 00:20:43 +00:00
Yiming Pan
56ba9d39fd Backfill owner information for VTS module VtsHalAudioV6_0TargetTest.
Each VTS module is required to have OWNERS file. The ownership is based on
go/vts-owners. For more information about ownership policy, please visit
go/xts-owners-policy.

Test: Tree Hugger
Bug: 143903671
Change-Id: I743c12b239732c85cecb421b4b4ee4ebcf36074a
2021-08-30 12:57:51 -07:00
Mikhail Naganov
e9f41f1fff Move common audio AIDL types to audio.media.audio.common
Since tests depend statically on libaudiofoundation,
they also need to depend on the generated interfaces code,
which contains vtables for interface classes.

Update 'sthal_cli_3' test app.

Bug: 188932434
Test: m HalAudioV6_0GeneratorTest
Test: m sthal_cli_3
Change-Id: Ifb8f81d02764f7710ce207603aa6fe2d20123e89
2021-08-23 17:43:20 -07:00
Andy Hung
9ef0cdad00 Merge "Remove libaudioclient from vendor" am: 0f23937288 am: 4df56a5449 am: 789c5284c9 am: 23569ce80e am: 4dfbfdcd79
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1801733

Change-Id: I97366381bcfeb42bad1e38bb002970951c71d9b0
2021-08-20 17:04:55 +00:00
Treehugger Robot
07a3c8d98b Merge "Provide defaults for android.hardware.audio@7.0-impl" am: 9124807755 am: 03e3cb179a am: ef4c911657 am: d9fa71397a am: 95060bc08f
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1803397

Change-Id: I3342c6fa2f1ac37968c243d0b25573b7a9e68fe2
2021-08-20 17:04:47 +00:00
Andy Hung
4dfbfdcd79 Merge "Remove libaudioclient from vendor" am: 0f23937288 am: 4df56a5449 am: 789c5284c9 am: 23569ce80e
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1801733

Change-Id: I038609d3dca5ed207bc1c69a8e4c40078d4d1685
2021-08-20 16:48:09 +00:00
Treehugger Robot
95060bc08f Merge "Provide defaults for android.hardware.audio@7.0-impl" am: 9124807755 am: 03e3cb179a am: ef4c911657 am: d9fa71397a
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1803397

Change-Id: Id7bef815e74ea1909f6128215ee7743c2219400a
2021-08-20 16:47:51 +00:00
Andy Hung
4df56a5449 Merge "Remove libaudioclient from vendor" am: 0f23937288
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1801733

Change-Id: Ib8b74771d2e7131a6bcc593312bee23acebe93fd
2021-08-20 16:05:28 +00:00
Treehugger Robot
03e3cb179a Merge "Provide defaults for android.hardware.audio@7.0-impl" am: 9124807755
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1803397

Change-Id: Id4138b78fa392c99148950e8615ec2cbebed0dea
2021-08-20 16:05:13 +00:00
Andy Hung
0f23937288 Merge "Remove libaudioclient from vendor" 2021-08-20 15:48:02 +00:00
Roman Kiryanov
4c5752d903 Provide defaults for android.hardware.audio@7.0-impl
see aosp/1366502.

Bug: 196868480
Bug: 161485545
Test: presubmit
Signed-off-by: Roman Kiryanov <rkir@google.com>
Change-Id: Ie582038f2212fbf881497ed3db62ef04bac286f3
2021-08-19 17:59:03 -07:00
Andy Hung
de0d3df464 Remove libaudioclient from vendor
Test: compiles
Bug: 189371870
Change-Id: Ic96bbb0c370883866fac731ef5e6941c1994c7c6
2021-08-18 20:30:02 -07:00
Xin Li
e287df40b9 Merge sc-dev-plus-aosp-without-vendor@7634622
Merged-In: Ifa71accba7ecf2ab15980227a4413831cfceb187
Change-Id: I7fb44c6c578a51c8ad36649d20a46233fd9609ca
2021-08-14 06:30:57 +00:00
Treehugger Robot
68de233181 Merge "Backfill owner information for VTS module VtsHalAudioV2_0TargetTest" am: 635c0b8416 am: 989612a1b6 am: ae777d6d52 am: af2040a2b4 am: 0fcf430b60
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1793835

Change-Id: I85625b79f5ff113519ecf54ee42cff0cccf97044
2021-08-13 22:06:41 +00:00
Treehugger Robot
0fcf430b60 Merge "Backfill owner information for VTS module VtsHalAudioV2_0TargetTest" am: 635c0b8416 am: 989612a1b6 am: ae777d6d52 am: af2040a2b4
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1793835

Change-Id: I5c651d67a525623989f84792fdf7e7b670d4d9d0
2021-08-13 21:36:00 +00:00
Treehugger Robot
ae777d6d52 Merge "Backfill owner information for VTS module VtsHalAudioV2_0TargetTest" am: 635c0b8416 am: 989612a1b6
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1793835

Change-Id: Ia49a8b36da4ca0cf4ef57cf8acdf23a512096dfb
2021-08-13 18:06:55 +00:00
Treehugger Robot
635c0b8416 Merge "Backfill owner information for VTS module VtsHalAudioV2_0TargetTest" 2021-08-13 17:37:57 +00:00
Yiming Pan
494d01a431 Backfill owner information for VTS module VtsHalAudioV2_0TargetTest
Owner information already exists. Add bug component information.

Test: Tree Hugger
Bug: 143903671
Change-Id: I7142c9dea2af4ae037a0244d7d306e401c1d5cfb
2021-08-13 09:30:58 -07:00
Kuowei Li
e3fe4153f5 audio: Allow SetAudioProperties to not be supported am: de829572f9 am: 4f21be2965 am: 2950416a76 am: 9ee709c323
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1762588

Change-Id: I6c6ae7038c424eaf91e4f1ed7eac4bbab9e982a1
2021-07-23 21:47:40 +00:00
Kuowei Li
9ee709c323 audio: Allow SetAudioProperties to not be supported am: de829572f9 am: 4f21be2965 am: 2950416a76
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1762588

Change-Id: Ice1083c4c6e5f5f924880f3464d961b5687fc2d2
2021-07-23 21:44:07 +00:00
Kuowei Li
2950416a76 audio: Allow SetAudioProperties to not be supported am: de829572f9 am: 4f21be2965
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1762588

Change-Id: I8711fc885403a22d7a09fc51aeb3d6e69ef759ea
2021-07-23 21:28:20 +00:00
Andy Hung
b7e250e7a9 Merge "Audio: Add memory leak checking for HAL" into sc-v2-dev am: 6a95f0508c am: 2588d4c379
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15239828

Change-Id: I84e8bcdf95d05febfddb3421360adfba8fc62883
2021-07-23 18:47:52 +00:00
Kuowei Li
de829572f9 audio: Allow SetAudioProperties to not be supported
The parameter of sample rate, channel mask and format is not
mandatory to support by Audio HAL. At meantime, there is no
corresponding handle in framework.
Hence, SetAudioProperties should allow reporting not supported.

Bug: 194368657

Test: run vts -m VtsHalAudioV7_0Target
Change-Id: Id0505bfb5d4812dd4c5d31a6e9d72c4c9c0cffa2
2021-07-22 17:44:48 +00:00
Andy Hung
8f836b9499 Audio: Add memory leak checking for HAL
$ adb shell setprop libc.debug.malloc.program android.hardware.audio.service
$ adb shell setprop libc.debug.malloc.options backtrace=8
$ adb shell setenforce 0
$ adb shell pkill audioserver
$ adb shell dumpsys media.audio_flinger

Test: Check the audio flinger dumpsys as above.
Bug: 186054996
Bug: 187462632
Change-Id: I2e8db14b816cc4cd7e1420c538505bf71fa58c97
2021-07-21 17:31:26 -07:00
TreeHugger Robot
1db6338893 Merge "Audio VTS: Make the active microphone query test more robust in V7" into sc-dev am: 997554db63 am: e06c6fdf7d
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15329863

Change-Id: I73da94e78a87a26c8d685b4f0dcfcdd8a6ea3de5
2021-07-20 18:16:00 +00:00
Mikhail Naganov
65a23ad2d4 Audio VTS: Make the active microphone query test more robust in V7
Prior to V7 the test which exercises
IStreamIn.getActiveMicrophones was using a hardcoded
configuration for the input stream. This configuration no longer
works for some of new devices. To fix that, the part of the test
which calls getActiveMicrophones has been moved into a separate
test--a descendant of InputStreamTest which is parametrized
using the actual configuration of the DUT.

Tests for HAL versions prior to V7 are not affected
because they don't use a full parser for the DUT config.

Bug: 193849687
Test: atest VtsHalAudioV7_0TargetTest
Change-Id: I00fe8fedb6bfc6e034387b35c88f954cb2638dfa
Merged-In: I00fe8fedb6bfc6e034387b35c88f954cb2638dfa
2021-07-20 17:40:13 +00:00
TreeHugger Robot
997554db63 Merge "Audio VTS: Make the active microphone query test more robust in V7" into sc-dev 2021-07-20 16:52:02 +00:00
David Li
e88db67931 Merge "audio: make sure to set back to AudioMode::NORMAL after the test case" into sc-dev am: 183f6190a8 am: 906100f26d
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15310729

Change-Id: I9348ed121ca3a9341517eab922e2691b76566261
2021-07-20 08:45:06 +00:00
David Li
183f6190a8 Merge "audio: make sure to set back to AudioMode::NORMAL after the test case" into sc-dev 2021-07-20 08:20:47 +00:00
Mikhail Naganov
e4ce86bfb7 Audio VTS: Make the active microphone query test more robust in V7
Prior to V7 the test which exercises
IStreamIn.getActiveMicrophones was using a hardcoded
configuration for the input stream. This configuration no longer
works for some of new devices. To fix that, the part of the test
which calls getActiveMicrophones has been moved into a separate
test--a descendant of InputStreamTest which is parametrized
using the actual configuration of the DUT.

Tests for HAL versions prior to V7 are not affected
because they don't use a full parser for the DUT config.

Bug: 193849687
Test: atest VtsHalAudioV7_0TargetTest
Change-Id: I00fe8fedb6bfc6e034387b35c88f954cb2638dfa
2021-07-19 23:04:15 +00:00
David Li
238fbcc61a audio: make sure to set back to AudioMode::NORMAL after the test case
If the major version is greater than or equal to 6, the test tries
to set the mode to AudioMode::CALL_SCREEN. However, it doesn't set
back to AudioMode::NORMAL. Replace all ASSERT with EXPECT to ensure the
test can reach to the explicit call to reset to AudioMode::NORMAL.

Bug: 194022995
Test: atest VtsHalAudioV7_0TargetTest
Change-Id: Ib9b6e310965a85b016853b72c60716fa054641c6
2021-07-19 16:26:42 +08:00
Mikhail Naganov
91f3d41269 Update audio VTS Android.bp to add new dependency
AIDL conversion library libaudioclient_aidl_conversion
uses media type names from libstagefright_foundation.

Bug: 188932434
Test: m
Change-Id: Ida3623dddb63f4d156044d70211819f0f4f8908a
2021-07-16 14:59:15 -07:00
Bill Yi
ee86375d3a Merge RQ3A.210705.001 to aosp-master - DO NOT MERGE
Merged-In: I7b187c5e79bf4531bdffc7411a033a7c20a54294
Merged-In: I89cda19ca8521445e1608efbbd07c889cca59601
Merged-In: I89cda19ca8521445e1608efbbd07c889cca59601
Change-Id: I183701b2c03dc63d19c98779388c0b99bae9a4ce
2021-07-12 11:08:26 -07:00
Mikhail Naganov
dd0e20eba3 Revert "audio: exclude the echo reference device in capture position test"
This reverts commit b22f307ccf.

Reason for revert: As explained in b/192307382#comment12, the HAL must provide capture positions for the echo reference input regardless of whether there is any actual output.

This should not affect O6/R4 as according to b/192307382#comment10, the VTS tests pass w/o this patch after the HAL has been fixed.

Bug: 192307382
Change-Id: I224bd9de1dcb2e2c8dc138dbfd85f848378aea4f
Test: VtsHalAudioV7_0TargetTest --gtest_filter=*PcmOnlyConfigInputStreamTest*
2021-07-07 17:07:15 +00:00
Carter Hsu
b22f307ccf audio: exclude the echo reference device in capture position test
Bug: 192307382
Test:
VtsHalAudioV7_0TargetTest --gtest_filter=*PcmOnlyConfigInputStreamTest*
Signed-off-by: Carter Hsu <carterhsu@google.com>
Change-Id: I4e38e093bc3be7ee54c0c7cce4d1181e260a23bb
2021-06-30 06:55:24 +00:00
Steven Moreland
2f0fe27a2b audio HAL - fix UAFs
Bug: 185259758
Test: N/A
Change-Id: I5ec70b098a00746108e10ab39e966607d78c84ae
Merged-In: I5ec70b098a00746108e10ab39e966607d78c84ae
(cherry picked from commit a8ac7cf706)
(cherry picked from commit 7283cbe8cb)
2021-05-20 00:00:57 +00:00
Andy Hung
bec76e8c9e Merge "StreamOut: Fix STREAM_EVENT_CBK_TYPE_CODEC_FORMAT_CHANGED" am: 66ffc5b99a am: 1e1514be04 am: 6660615822
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1692490

Change-Id: I5bd4d6fdea2c18c9822ad061cdc24a488b148fb6
2021-05-04 21:54:03 +00:00
Andy Hung
6660615822 Merge "StreamOut: Fix STREAM_EVENT_CBK_TYPE_CODEC_FORMAT_CHANGED" am: 66ffc5b99a am: 1e1514be04
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1692490

Change-Id: I97c94630cd48b9a0354c9b76e593ea0205a633b4
2021-05-04 21:37:16 +00:00
Andy Hung
e6e3a17126 StreamOut: Fix STREAM_EVENT_CBK_TYPE_CODEC_FORMAT_CHANGED
Use audio_metadata_byte_string_len method in asyncEventCallback.

Test: atest metadata_tests
Bug: 186894288
Change-Id: I04914bc5b2c778002b53b8f8da5bf29fff5c80f8
2021-04-30 21:07:33 -07:00
Steven Moreland
f07b44c1f9 audio HAL - fix UAFs am: 7283cbe8cb am: e227bfd15c am: facfee6382 am: da947d5127
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/14212414

Change-Id: I7c4ba7e1ed4e7c71e10332ce986bbc7b6aba8f40
2021-04-26 23:10:51 +00:00
Steven Moreland
facfee6382 audio HAL - fix UAFs am: 7283cbe8cb am: e227bfd15c
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/14212414

Change-Id: I5b80e2810ef7c4010496687934d93685057d4c82
2021-04-26 22:31:02 +00:00
Steven Moreland
7283cbe8cb audio HAL - fix UAFs
Bug: 185259758
Test: N/A
Change-Id: I5ec70b098a00746108e10ab39e966607d78c84ae
Merged-In: I5ec70b098a00746108e10ab39e966607d78c84ae
(cherry picked from commit a8ac7cf706)
2021-04-16 18:36:09 +00:00
TreeHugger Robot
1cabad5e1e Merge "audio HAL - fix UAFs" into sc-dev 2021-04-14 00:24:26 +00:00
Steven Moreland
a8ac7cf706 audio HAL - fix UAFs
Bug: 185259758
Test: N/A
Change-Id: I5ec70b098a00746108e10ab39e966607d78c84ae
2021-04-13 20:15:40 +00:00
jiabin
5f82f49d4f Count channel mapping number from the back.
The channel mapping array may have leading UNUSED value. In that case,
count the channel mapping number from the back to avoid lost channel
mapping information.

Bug: 183558363
Test: repo steps in the bug
Test: atest android.hardware.audio@7.0-util_tests
Change-Id: Ic4238e520c16eb1daad7c9e92555e92637f8e7c9
2021-04-12 18:47:52 +00:00
Mikhail Naganov
cb766d2517 audio: Use get_audio_port_v7 in the default wrapper
get_audio_port_v7 interface method was added in aosp/1518624
but wasn't used in the default wrapper due to the refactoring
ongoing at that time. Fixing this omission.

Bug: 160352965
Test: atest VtsHalAudioV7_0TargetTest
Change-Id: I1e3594f30fe8fa319d0b5b4244531c9142870503
Merged-In: I1e3594f30fe8fa319d0b5b4244531c9142870503
2021-03-31 18:24:11 +00:00
TreeHugger Robot
457837f6b2 Merge "audio: Use get_audio_port_v7 in the default wrapper" into sc-dev 2021-03-30 20:00:55 +00:00