Commit graph

358 commits

Author SHA1 Message Date
Ram Mohan M
7fe35b2d4a Merge "omxvts: fix binary for 64bit platforms" into oc-mr1-dev am: 7e0286404a
am: db285e9390

Change-Id: I2e79899adf6599684815bea6f6f6a9aaed41de4c
2017-10-01 23:14:48 +00:00
Ram Mohan M
969e586842 Merge "omxvts: misplaced lock can cause race condition" into oc-mr1-dev am: 99c319f781
am: d1817707b7

Change-Id: I3284ba15562b3aab4b705e5f1c776a5169d143bd
2017-10-01 23:14:12 +00:00
Ram Mohan M
db285e9390 Merge "omxvts: fix binary for 64bit platforms" into oc-mr1-dev
am: 7e0286404a

Change-Id: Id7c88291d1e8d846deea702f1ae30198361333dd
2017-10-01 23:12:44 +00:00
Ram Mohan M
d1817707b7 Merge "omxvts: misplaced lock can cause race condition" into oc-mr1-dev
am: 99c319f781

Change-Id: I151636a841c11310880601db6533d21ab0d8f7c0
2017-10-01 23:12:16 +00:00
TreeHugger Robot
8b77593b0d Merge "Revert "Loop all decode tests for all port modes supported"" 2017-09-29 22:29:27 +00:00
Pawin Vongmasa
ff1b48721e Revert "Loop all decode tests for all port modes supported"
This reverts commit 3864db6b0d.

Change-Id: Ia8913ce11811a5b0037f9683118c0d5c5bbc4e42
2017-09-29 11:57:00 +00:00
Pawin Vongmasa
cfc4b13b7b Revert "Port mode configuration test - II"
This reverts commit fa0d7a3b89.

Change-Id: I1f907482f89aaff5f01be38a83f5aca997e20a8a
2017-09-29 11:56:39 +00:00
Ram Mohan M
9502b237f7 omxvts: fix binary for 64bit platforms
Bug: 63796949

Change-Id: Ieddae0bd9fc45a88c6e1579bdd588b77b4174a1f
2017-09-29 04:41:41 -07:00
Ram Mohan M
028540b3a1 omxvts: misplaced lock can cause race condition
upon unsuccessful return during timed wait, relock the mutex before
accessing message list.

Bug: 63796949

Change-Id: I84a43e148a6415e629cb39a6b9dfe27259f8f2fd
2017-09-29 04:39:48 -07:00
Steven Moreland
91799c6cc7 Merge "Fix typo in update makefiles." am: 9960148420 am: f07e364ce3 am: 71a193a425
am: 9fdd241905

Change-Id: I1c852fa6600fecc44d5afd1a65a5f51c43ecf71c
2017-09-27 00:02:52 +00:00
Steven Moreland
9fdd241905 Merge "Fix typo in update makefiles." am: 9960148420 am: f07e364ce3
am: 71a193a425

Change-Id: I45ae43f07323254212532acbf0f891d11143b05f
2017-09-26 23:58:51 +00:00
Steven Moreland
71a193a425 Merge "Fix typo in update makefiles." am: 9960148420
am: f07e364ce3

Change-Id: If5bbf259458363b755989578c2f3828cbcb3d07c
2017-09-26 23:55:59 +00:00
Steven Moreland
f07e364ce3 Merge "Fix typo in update makefiles."
am: 9960148420

Change-Id: I949634e72f817f3a5411130e968acd8efd1d7725
2017-09-26 23:51:07 +00:00
Steven Moreland
a5299ee739 Fix typo in update makefiles.
Bug: 37518178
Test: pass
Change-Id: Ic401b3a473f15ca4c01e58b3072e19db7c31b653
2017-09-26 21:59:43 +00:00
Steven Moreland
7741c0e17b Update makefiles for hidl adapter. am: 26a0bb2762 am: 73949c1d7e am: b94d0c7290
am: 97228c0b7d

Change-Id: Ic4b459b7c1a4325453945b0b00db8f44e4e77c26
2017-09-26 13:36:09 +00:00
Steven Moreland
97228c0b7d Update makefiles for hidl adapter. am: 26a0bb2762 am: 73949c1d7e
am: b94d0c7290

Change-Id: If766cffbcc003cc0cc5eb98969b924aa918fff44
2017-09-26 12:53:13 +00:00
Steven Moreland
b94d0c7290 Update makefiles for hidl adapter. am: 26a0bb2762
am: 73949c1d7e

Change-Id: I9af2df255ccdac36c8c11bbca973c42ea2e758fe
2017-09-26 12:48:40 +00:00
Steven Moreland
73949c1d7e Update makefiles for hidl adapter.
am: 26a0bb2762

Change-Id: I3a811f5cd49bd4e81e8fcd5c8e88922115812539
2017-09-25 23:14:44 +00:00
Steven Moreland
26a0bb2762 Update makefiles for hidl adapter.
Bug: 37518178
Test: manual
Change-Id: I50e999907d3c64d2b039272b823971998da64d1b
2017-09-25 18:35:56 +00:00
Ram Mohan M
3864db6b0d Loop all decode tests for all port modes supported
Many tests run their test with port mode configured to either
byte buffer mode or dynamic anw buffer mode but not both.
This change ensures that all test cases are tested for all
supported port mode configurations.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I600b30e09b478fdf787f0bf0ac6914006e553203
2017-09-23 05:37:49 +00:00
Ram Mohan M
fa0d7a3b89 Port mode configuration test - II
Change port modes randomly before the start of test and see if it
effects the end behavior

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I28c923b8e6cdc0f7bf6dd394ef6d088ff489e2e0
2017-09-23 01:04:08 +00:00
Ram Mohan M
63fbc04833 Merge "Add flush test" into oc-mr1-dev am: f213eca917
am: c363ff0bb9

Change-Id: I2d0567f03144b269379e85e96286ad4e0d6b86e3
2017-09-21 03:55:18 +00:00
Ram Mohan M
e0a06fd651 Merge "Enable EOSTest_M test" into oc-mr1-dev am: f11a1ef8b5
am: 26748d514d

Change-Id: Iae66a8d960487cdb712e0d8d2f53736391f5763f
2017-09-21 03:54:58 +00:00
Ram Mohan M
25c4474179 Merge "Signal eof flag for each input buffer" into oc-mr1-dev am: 98d4fd9ed6
am: e27badbfc8

Change-Id: I8fab6195868dc8aa499584a4677d2d109d8121a2
2017-09-21 03:54:40 +00:00
Ram Mohan M
3abd1eb75e Merge "Add support for preset ANW buffer mode" into oc-mr1-dev am: 5f03369e25
am: c0424d71c0

Change-Id: Ief7ec70c50fa2b64a423df2cdf2522d4ad288bb4
2017-09-21 03:54:21 +00:00
Ram Mohan M
3ff297e585 Merge changes I4a4aad3a,I40b94919 into oc-mr1-dev am: 97d9efb4e0
am: 089b8813c2

Change-Id: Ia483bc7bc7988792b28826958fd759c9d27276bf
2017-09-21 03:54:01 +00:00
Ram Mohan M
e7d8061b04 Merge "Add vts audio dec test for flac decoder" into oc-mr1-dev am: 404a1c69c1
am: 30b429b58d

Change-Id: I622794406ce94b19dde51ce87aa95207d3d60b00
2017-09-21 03:53:17 +00:00
Ram Mohan M
6a0ead68a8 Merge "Add adaptive playback test" into oc-mr1-dev am: f4570eb0d0
am: 7e8396f309

Change-Id: I5689acd7e2aec820ad4052846b2f204cdae05731
2017-09-21 03:52:34 +00:00
TreeHugger Robot
f213eca917 Merge "Add flush test" into oc-mr1-dev 2017-09-21 01:01:13 +00:00
TreeHugger Robot
f11a1ef8b5 Merge "Enable EOSTest_M test" into oc-mr1-dev 2017-09-21 01:00:47 +00:00
TreeHugger Robot
98d4fd9ed6 Merge "Signal eof flag for each input buffer" into oc-mr1-dev 2017-09-21 01:00:22 +00:00
TreeHugger Robot
5f03369e25 Merge "Add support for preset ANW buffer mode" into oc-mr1-dev 2017-09-21 00:59:59 +00:00
TreeHugger Robot
97d9efb4e0 Merge changes I4a4aad3a,I40b94919 into oc-mr1-dev
* changes:
  Extend adaptive playback test for graphic buffer modes
  Move graphic buffer allocation to common
2017-09-21 00:59:35 +00:00
TreeHugger Robot
404a1c69c1 Merge "Add vts audio dec test for flac decoder" into oc-mr1-dev 2017-09-21 00:59:09 +00:00
TreeHugger Robot
f4570eb0d0 Merge "Add adaptive playback test" into oc-mr1-dev 2017-09-20 10:14:12 +00:00
Pawin Vongmasa
7e3570f08e Merge "Fix typos in IOmxStore.hal" into oc-mr1-dev am: 941b614064
am: 87165bb092

Change-Id: I4724699054648351683d3b139f45dab7799a30aa
2017-09-19 01:30:07 +00:00
TreeHugger Robot
5bb23b736b Merge "Revert "Enable SetDefaultPortParams and EOSTest_M tests"" 2017-09-18 21:54:59 +00:00
Pawin Vongmasa
ede52502cb Revert "Enable SetDefaultPortParams and EOSTest_M tests"
This reverts commit 42958b71b0.

Change-Id: Icf95434d0d2a647f3e3014dcec3e0c9b4279ebb0
2017-09-17 14:34:21 +00:00
Ram Mohan M
b7de1a6ae3 Add vts audio dec test for flac decoder
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: Ie504de96007c1c0ec6dc9c8b7da78f6c1da45631
2017-09-17 05:24:08 -07:00
Ram Mohan M
17fa59b431 Add support for preset ANW buffer mode
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I17452d9f83cf9749cd654eaf55735563f0c5f82f
2017-09-17 05:20:19 -07:00
Ram Mohan M
500052d8f0 Extend adaptive playback test for graphic buffer modes
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I4a4aad3a0dd4f528c8e855164cbb2998b33be1cd
2017-09-17 05:19:16 -07:00
Ram Mohan M
957367805b Move graphic buffer allocation to common
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I40b949199856beab73933a083045ed2e4e018c75
2017-09-17 05:17:34 -07:00
Ram Mohan M
0a9ac2ac78 Enable EOSTest_M test
eostest_m was disabled during the time of o-release. This is
enabled now. Also improved logging

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I4797fce4f5911d9f37f5c91f179d9ab9263177f0
2017-09-17 05:10:39 -07:00
Ram Mohan M
05f81e4d07 Add adaptive playback test
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I96f19946edb9a333776a60fa2c159dad67801b27
2017-09-17 05:05:21 -07:00
Ram Mohan M
3ca3988e4e Add flush test
1. omx api allows flushing all ports at once by passing the
argument OMX_ALL. This fixture tests the flush call with the
arg OMX_ALL and expects behavior to be same as when the ports
are individually flushed.
2. Flush a port that is not listed and the api call shall
return with error code OMX_ErrorBadPortIndex
3. Test the behavior of flush API call when the component is
not in Execute State (expects OMX_ErrorIncorrectStateOperation)

Note: 2, 3 tests are disabled as of now

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I6f0cccea0c923116f7e41ed0ef7d8d584631256b
2017-09-17 04:23:25 -07:00
Ram Mohan M
0cf6b3df5b Signal eof flag for each input buffer
As the client sends single frame worth of data for processing all
the time, at the time of dispatch, signal eof flag for each buffer.

Also code that was repititive and doesnt serve any need is removed.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I530cbe69c27f5e4b1fae56ab808ef63107586275
2017-09-17 04:09:18 -07:00
Pawin Vongmasa
ed95ab31e1 Fix typos in IOmxStore.hal
Test: make cts -j123 && cts-tradefed run cts-dev -m \
CtsMediaTestCases --compatibility:module-arg \
CtsMediaTestCases:include-annotation:\
android.platform.test.annotations.RequiresDevice

Bug: 65719609
Bug: 65769027
Change-Id: I44d62cd5a8aa21d9fa816d5f0bf663e943a21bb5
2017-09-16 23:10:51 -07:00
Ram Mohan M
d0733af6de Merge "bug fix: add missing color formats to the check for valid formats" into oc-mr1-dev am: 6a56323738
am: cbc8c1003f

Change-Id: Ic8d7960eeacd35b36dcb2011cba854dd6d0c2ff6
2017-09-14 06:00:54 +00:00
TreeHugger Robot
6a56323738 Merge "bug fix: add missing color formats to the check for valid formats" into oc-mr1-dev 2017-09-13 22:12:19 +00:00
TreeHugger Robot
a5a5586dd1 Merge "Add portMode configuration test" into oc-mr1-dev 2017-09-12 02:21:17 +00:00
Ram Mohan M
bb3fab4688 Merge "bug fix: handle multiple port settings change events signalled at once" into oc-dev am: f8677dcfdb
am: 37b490df35

Change-Id: Ib89e1de10cee360e0a190d7c162d45f96c48c384
2017-09-07 23:45:42 +00:00
Ram Mohan M
a24f85f9b9 Merge "bug fix: handle multiple port settings change events signalled at once" into oc-dev am: f8677dcfdb
am: 37b490df35

Change-Id: I028c248ebef366eb439b369adad87047086aa07b
2017-09-07 23:45:42 +00:00
Ram Mohan M
37b490df35 Merge "bug fix: handle multiple port settings change events signalled at once" into oc-dev
am: f8677dcfdb

Change-Id: If65c243e72e890660fc4481b375b23ba109ae577
2017-09-07 23:40:43 +00:00
TreeHugger Robot
f8677dcfdb Merge "bug fix: handle multiple port settings change events signalled at once" into oc-dev 2017-09-07 23:30:17 +00:00
Ram Mohan M
d1b6aed4c3 bug fix: handle multiple port settings change events signalled at once
Components can send various kinds of port settings changed events all
at once. During a full port reconfiguration this is causing problems
due to limited flexibility in the message handling design. For now,
before committing to a full port reconfiguration defer any events
waiting in the queue to be addressed to a later point.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 64468705

Change-Id: I537cdd3e7d92cdc6d54168f091d7897afe541599
2017-09-06 03:49:18 +00:00
Ram Mohan M
3a0e8687d7 Merge "bug fix: restore support for broken flag" into oc-dev am: 40aad4e988 am: 062192a1be am: 381b0f4c72
am: eb5f31b4ff

Change-Id: I1b195a6baba885a7118c077cd3dbf8b0176df8fa
2017-09-03 13:19:05 +00:00
Ram Mohan M
eb5f31b4ff Merge "bug fix: restore support for broken flag" into oc-dev am: 40aad4e988 am: 062192a1be
am: 381b0f4c72

Change-Id: I395488e5a7184399a9b2dfa9f2fb5d4ce56b9f47
2017-09-03 13:11:05 +00:00
Ram Mohan M
381b0f4c72 Merge "bug fix: restore support for broken flag" into oc-dev am: 40aad4e988
am: 062192a1be

Change-Id: I82cbbf582216644c073d14ba74360590c1a387b1
2017-09-03 13:02:35 +00:00
Ram Mohan M
fee5be0064 Merge "bug fix: restore support for broken flag" into oc-dev am: 40aad4e988
am: 062192a1be

Change-Id: I8349103fed48e5eafeffa9ea4dfc78cbe82e1d1b
2017-09-03 13:02:34 +00:00
Ram Mohan M
062192a1be Merge "bug fix: restore support for broken flag" into oc-dev
am: 40aad4e988

Change-Id: Ied23d04f5c38f81ad732424d3577edf4d3cc085d
2017-09-03 12:55:06 +00:00
TreeHugger Robot
40aad4e988 Merge "bug fix: restore support for broken flag" into oc-dev 2017-09-03 12:42:39 +00:00
Ram Mohan M
dc6270ecc1 bug fix: restore support for broken flag
timestampDevTest flag can be used to disable timestamp deviation
tests. The flag is not working as intended. This is corrected.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 64743270
Bug: 65166112

Merged-In: I611e96498a4677b6ccbff33c291d56eed7b48f6f
Change-Id: I611e96498a4677b6ccbff33c291d56eed7b48f6f
2017-09-02 01:39:07 -07:00
Ram Mohan M
4c4e678613 Merge "Modify SetDefaultPortParams test" into oc-mr1-dev am: 5fe60768c2
am: 1cce2cd0f8

Change-Id: I2c2a62edf425a05178d95108c9773d58310d9fb6
2017-09-02 08:37:29 +00:00
TreeHugger Robot
5fe60768c2 Merge "Modify SetDefaultPortParams test" into oc-mr1-dev 2017-09-02 08:22:50 +00:00
Ram Mohan M
8043ecc06b Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev am: febad058bd am: a87079c3d4 am: 402b5dcf28
am: cc7c018200

Change-Id: I021a33561104fdd840454c0eda55a5b3f5a11a92
2017-09-02 04:09:45 +00:00
Ram Mohan M
cc7c018200 Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev am: febad058bd am: a87079c3d4
am: 402b5dcf28

Change-Id: I34d237e2ad51fc80986865f178cb2a0d389c2977
2017-09-02 03:57:14 +00:00
Ram Mohan M
402b5dcf28 Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev am: febad058bd
am: a87079c3d4

Change-Id: Ieada35dd4756867bf7e0bc55c150d689b1bd7dd7
2017-09-02 03:50:43 +00:00
Ram Mohan M
f88227bd94 Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev am: febad058bd
am: a87079c3d4

Change-Id: Ied9396646a58ca9b3f123c7db25df418b040a478
2017-09-02 03:50:43 +00:00
Ram Mohan M
a87079c3d4 Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev
am: febad058bd

Change-Id: I09128311ab40316b17d99daeae375ff7535d0e63
2017-09-02 03:44:45 +00:00
TreeHugger Robot
febad058bd Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev 2017-09-02 03:40:15 +00:00
TreeHugger Robot
aa93654feb Merge "Enable SetDefaultPortParams and EOSTest_M tests" 2017-09-02 03:11:49 +00:00
Ram Mohan M
845d9fc89c Add portMode configuration test
Configuration of port mode of a component is allowed only when the
component is in loaded state. When the component is in idle/execute
state this operation is denied. This commit adds a test fixture that
verifies this.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 63796949

Merged-In: Ib958c671873fe1d9bb4227ce3e7214f948469f19
Change-Id: Ib958c671873fe1d9bb4227ce3e7214f948469f19
2017-09-01 13:45:03 +00:00
Ram Mohan M
7fe0b2bfac Add portMode configuration test
Configuration of port mode of a component is allowed only when the
component is in loaded state. When the component is in idle/execute
state this operation is denied. This commit adds a test fixture that
verifies this.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 63796949

Merged-In: Ib958c671873fe1d9bb4227ce3e7214f948469f19
Change-Id: Ib958c671873fe1d9bb4227ce3e7214f948469f19
2017-09-01 06:41:10 -07:00
Ram Mohan M
9fe004c216 Modify SetDefaultPortParams test
1. For output port in video encoder and input port in video decoder,
it is allowed to configure the size of the buffer. Make adjustments
in the test application to validate this scenario.

2. useBuffer api call is allowed only after sending loaded to idle
state transition command. Ensure this requirement is met.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 63796949

Change-Id: I1aba7703e1344b87f556d3388a4cde8199898499
2017-09-01 06:37:21 -07:00
Ram Mohan M
42958b71b0 Enable SetDefaultPortParams and EOSTest_M tests
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 63796949

Change-Id: Ib341c977256b33f22aedaa3e1ae05db96ef89d26
2017-09-01 13:32:12 +00:00
Ram Mohan M
a230bad8d4 bug fix: handle multiple port settings change events signalled at once
Components can send various kinds of port settings changed events all
at once. During a full port reconfiguration this is causing problems
due to limited flexibility in the message handling design. For now,
before committing to a full port reconfiguration defer any events
waiting in the queue to be addressed to a later point.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 64468705

Change-Id: I537cdd3e7d92cdc6d54168f091d7897afe541599
2017-09-01 06:23:39 -07:00
Ram Mohan M
eb9e25a3e2 bug fix: disable timestamp deviation test for audio decoder
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 64128634
Bug: 65166112

Change-Id: I8a08a8da7dd203cd16cbfbfd3f15e27966db711f
2017-09-01 06:22:17 -07:00
Ram Mohan M
a7ea9ae197 bug fix: add missing color formats to the check for valid formats
Component can support 420PackedPlanar and 420PackedSemiPlanar as
their choice of color formats. The current test application can
fail in this case. This is corrected.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 63796949

Change-Id: I38bbbb40b34dcdd4526e87708667dc116af900f1
2017-09-01 05:56:54 -07:00
Ram Mohan M
8421c086dd Merge "bug fix: output colorformat configuration fixed" into oc-dev am: 1e15657ca1
am: 0e42ac3399

Change-Id: I4270a6f4fc3b0ab34851d073d45896ef0f558fcd
2017-08-25 05:20:42 +00:00
Ram Mohan M
0e42ac3399 Merge "bug fix: output colorformat configuration fixed" into oc-dev
am: 1e15657ca1

Change-Id: I1bf3c443058ab57baba7c51e3f9f0ae662e61882
2017-08-25 05:16:14 +00:00
Ram Mohan M
d4ed557260 Merge "Modify StateTransitions test" into oc-mr1-dev am: 51b64c30c7
am: 8a61c0f593

Change-Id: I5b96699f2368c18cfbef004c006019fe4cdcecf0
2017-08-24 12:43:23 +00:00
Ram Mohan M
07674691a0 Merge "bug fix: close file pointer" into oc-dev am: 8735fb45a5
am: 51f04884ba

Change-Id: Ia0ec90fe324c7ec87eb42de42754f15bdc3fc0be
2017-08-19 01:48:11 +00:00
Ram Mohan M
ad012dc507 Merge "bug fix: configure input port buffer size" into oc-dev am: bc769a7e5a
am: 2eedb46745

Change-Id: Ic30a77e7f4bfaf9e249e10e4916c1452defacb9b
2017-08-19 01:47:54 +00:00
Ram Mohan M
51f04884ba Merge "bug fix: close file pointer" into oc-dev
am: 8735fb45a5

Change-Id: I7ca0f4992804da30eaac7ca24b2dbe9004a5ac55
2017-08-19 01:45:20 +00:00
Ram Mohan M
2eedb46745 Merge "bug fix: configure input port buffer size" into oc-dev
am: bc769a7e5a

Change-Id: I3a9b89f4c64d1129ea59a0144937832d93ed81fd
2017-08-19 01:44:50 +00:00
Ram Mohan M
1441864f4c bug fix: output colorformat configuration fixed
The eColorFormat field in OMX_PARAM_PORTDEFINITIONTYPE structure
has no effect on the component. This has to be configured from
OMX_VIDEO_PARAM_PORTFORMATTYPE. This is corrected. This color format
information is passed to graphic buffer during its allocation

Bug: 63796949

Merged-In: I51b1e59631647d1291ecfac373781ac5e5a0ffd5
Change-Id: I51b1e59631647d1291ecfac373781ac5e5a0ffd5
2017-08-17 21:29:32 -07:00
Ram Mohan M
727b2ba98c bug fix: close file pointer
Bug: 63796949

Merged-In: Ifd8a27d851cbc659ea8108e0879747cd9169680a
Change-Id: Ifd8a27d851cbc659ea8108e0879747cd9169680a
2017-08-17 05:31:52 -07:00
Ram Mohan M
9ce313a7b3 bug fix: configure input port buffer size
In video decoders, size of a single input frame (elementary-stream)
is dependent on width, height, color format, profile settings, ...
Most of this information is part of sps, pps and requires parsing.
But as the max size of the stream is known ahead, use it to configure
the input buffer size requirements

Bug: 63875287
Bug: 63796949

Merged-In: Ib760c4f55b094260a0abd120f852dcf1899df4e0
Change-Id: Ib760c4f55b094260a0abd120f852dcf1899df4e0
2017-08-17 05:31:33 -07:00
Ram Mohan M
c3aa23fa3e Merge "bug fix: close file pointer" into oc-mr1-dev am: 22898c0ed7
am: 6de8410c3e

Change-Id: Ied2620d4f39f17036e17ea547d4e5b2338b67d65
2017-08-16 05:08:25 +00:00
Ram Mohan M
31a46fa4f5 Merge "bug fix: configure input port buffer size" into oc-mr1-dev am: 97d68ba8f2
am: a225141858

Change-Id: I801371cbce35e701996710d4110c6b3d28705e18
2017-08-16 05:07:54 +00:00
Ram Mohan M
0bd706936f Modify StateTransitions test
During loaded <-> idle state transition, check for events
is done after allocating all the buffers on a port. This is now
done after allocation of every single buffer. This is done to
improve scope of testing

Bug: 63796949

Change-Id: Icf3405ac6716f9533d31189f6c4a4c4ac908da5f
2017-08-15 04:57:15 -07:00
Ram Mohan M
285f8b116e bug fix: close file pointer
Bug: 63796949

Change-Id: Ifd8a27d851cbc659ea8108e0879747cd9169680a
2017-08-15 04:56:40 -07:00
Ram Mohan M
96743b4257 bug fix: configure input port buffer size
In video decoders, size of a single input frame (elementary-stream)
is dependent on width, height, color format, profile settings, ...
Most of this information is part of sps, pps and requires parsing.
But as the max size of the stream is known ahead, use it to configure
the input buffer size requirements

Bug: 63875287
Bug: 63796949

Change-Id: Ib760c4f55b094260a0abd120f852dcf1899df4e0
2017-08-15 04:55:40 -07:00
Steven Moreland
dfd8287506 Updating all makefiles.
Bug: 64487114
Test: none
Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a
Merged-In: I2fa89b6661c39859ec4fb62c4bb0a05a35e645f0
Merged-In: Ifdc3c17cb2b85c18b37dac2d03bb5c8935c23180
Change-Id: I170fa1c4fe39f8109b1670db58ef99bb11afc0be
2017-08-14 20:25:09 +00:00
Steven Moreland
ff308ea6ba Updating all makefiles.
Bug: 64487114
Test: none
Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a
Merged-In: I2fa89b6661c39859ec4fb62c4bb0a05a35e645f0
Change-Id: Ifdc3c17cb2b85c18b37dac2d03bb5c8935c23180
2017-08-11 22:58:47 +00:00
Tri Vo
054857a90d Merge "Media omx tests static link to dependencies." into oc-mr1-dev am: 24ea76c31c
am: 3b45577f47

Change-Id: I5836cafcd7019663e3cd03e067f22a428954371d
2017-08-10 02:09:03 +00:00
Ram Mohan M
c2e4716c73 Merge changes I51b1e596,I3aece23a into oc-mr1-dev am: 438bbdac30
am: 25a9a159b8

Change-Id: I33585da6fc8506403bd4b18b16f09f31d53f8412
2017-08-10 02:08:50 +00:00
Tri Vo
24ea76c31c Merge "Media omx tests static link to dependencies." into oc-mr1-dev 2017-08-09 23:41:15 +00:00
TreeHugger Robot
438bbdac30 Merge changes I51b1e596,I3aece23a into oc-mr1-dev
* changes:
  bug fix: output colorformat configuration fixed
  bug fix: android list management in dequeueMsg()
2017-08-09 21:23:20 +00:00
Tri Vo
5396b25cba Media omx tests static link to dependencies.
These tests now statically link to libs not guaranteed to be on the
device, which include HAL definition libs.

Also, remove global include paths.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check 
--skip-preconditions --module VtsHalMediaOmxV1_0Host

Change-Id: Iea1dc549704f61b1416357d72dc1bf26864978d4
2017-08-08 01:57:44 +00:00
Ram Mohan M
2921996e7a bug fix: handle OMX_EventBufferFlag events
am: c8df8940e1

Change-Id: Ifa9493d869e42a76100011d5af20c5062bc44891
2017-08-08 00:47:23 +00:00
Ram Mohan M
c8df8940e1 bug fix: handle OMX_EventBufferFlag events
OMX_EventBufferFlag event is sent when the component has processed a buffer
with its EOS flag set. This event is not sent by soft omx components.
Vendor components can send this. From IOMX point of view, this event is
not sent for processing

bug:64102197
Merged-In: I3a978a885b1e4446f82f2356ae677f70ea6f8150
Change-Id: I3a978a885b1e4446f82f2356ae677f70ea6f8150
2017-08-07 14:21:12 -07:00
Justin Yun
723c2dbd43 Update make file for vndk enabled. am: 608d773ef8
am: d66371512c

Change-Id: I35b778c39c2349346278deb6dea56b8cd3701f7d
2017-08-05 02:17:38 +00:00
Justin Yun
608d773ef8 Update make file for vndk enabled.
Update the Android.bp generated with hidl-gen.

Test: build with and without BOARD_VNDK_VERSION=current
Bug: 63866913
Change-Id: I1a9db1df49e0f13c5790da2b118ae9ec63ba34a7
2017-08-04 14:12:23 +09:00
Tri Vo
0621d2f517 Update Android.bp HIDL makefiles am: f5d136c231
am: 7694738e26

Change-Id: I7592025d540bbe46d2e2726ae39ae34f352a1a30
2017-08-03 15:37:24 +00:00
Tri Vo
f5d136c231 Update Android.bp HIDL makefiles
Allow HAL definition libs to be static.

Bug: 32920003
Bug: 64040096
Test: update-all-google-makefiles.sh
Change-Id: I1483d572bea6799717d1614fb7d52fe225e31104
2017-08-03 00:30:01 +00:00
Ram Mohan M
bf556d1b91 bug fix: output colorformat configuration fixed
The eColorFormat field in OMX_PARAM_PORTDEFINITIONTYPE structure
has no effect on the component. This has to be configured from
OMX_VIDEO_PARAM_PORTFORMATTYPE. This is corrected. This color format
information is passed to graphic buffer during its allocation

Bug: 63796949
Change-Id: I51b1e59631647d1291ecfac373781ac5e5a0ffd5
2017-08-01 20:10:41 -07:00
Ram Mohan M
5b25fa66ca bug fix: android list management in dequeueMsg()
1. use erase() to update iterator as it is safer
2. correct abstime (delayUs) for waitRelative() call

Bug: 63796949
Change-Id: I3aece23a6bf4bd33aebfeecd070214911b58c75b
2017-08-01 17:46:48 -07:00
TreeHugger Robot
542bba12f1 Merge "Add media owners file." 2017-07-30 10:01:29 +00:00
Zhuoyao Zhang
5941366441 Add media owners file.
Test: none
Change-Id: I246a96654e8cc7fd9b94c018542b1cbef020a6e2
2017-07-28 11:04:22 -07:00
Ram Mohan M
e955b18fa7 bug fix: handle OMX_EventBufferFlag events
OMX_EventBufferFlag event is sent when the component has processed a buffer
with its EOS flag set. This event is not sent by soft omx components.
Vendor components can send this. From IOMX point of view, this event is
not sent for processing

bug:64102197
Merged-In: I3a978a885b1e4446f82f2356ae677f70ea6f8150
Change-Id: I3a978a885b1e4446f82f2356ae677f70ea6f8150
2017-07-28 13:53:42 +00:00
Ram Mohan M
28cd8404d6 bug fix: handle OMX_EventBufferFlag events
OMX_EventBufferFlag event is sent when the component has processed a buffer
with its EOS flag set. This event is not sent by soft omx components.
Vendor components can send this. From IOMX point of view, this event is
not sent for processing

bug:64102197
Merged-In: I3a978a885b1e4446f82f2356ae677f70ea6f8150
Change-Id: I3a978a885b1e4446f82f2356ae677f70ea6f8150
2017-07-28 06:52:51 -07:00
Ram Mohan M
df47955adf improve timeout handling
1. Maintaining large delays between successive input/output queue calls
unnecessarily increases execution duration of the application. This is
brought down to 2ms.
2. Flush and EOS sends events via callback. So offer them some leeway
by increasing their timeout
3. No response breakout is changed to 5sec.
4. bug fix: decrement twice with in a loop removed

Change-Id: I37dbee0dbdc79044beced89077f6ef46c7075e9f
2017-07-18 05:34:56 -07:00
Pawin Vongmasa
eadfa778d9 Merge "Print test information to the target device's logcat" into oc-dr1-dev am: 4bd2ed2323
am: 3f1ce37ed4

Change-Id: If3b8e780cf3a7c505a17c38ed11e8a98ce94a031
2017-07-14 04:45:48 +00:00
Pawin Vongmasa
dd7e4c414b Print test information to the target device's logcat
Test: make vts -j99 && \
vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 62885431
Change-Id: I71bfa9c5ff4825259c7452ad7ac40fef87103b5f
2017-07-06 17:29:43 -07:00
Ram Mohan M
335d3d9859 bug fix: handle portReconfiguration in testEOS()
When CSD and sync frame are sent together, component parses the
CSD and allocates buffers internal to the codec and decodes the frame
sent and port reconfiguration event is sent to the client. Now if this
is the only frame in the sequence, the client reaches testEOS(). But
port reconfiguration is not handled here. This is corrected.

bug: 62930419

Change-Id: Ic0c7625e3c2b415330f37734663f0cf7689e8734
2017-06-27 03:54:42 -07:00
Ram Mohan M
99d23a09f1 move duplicate code to common
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 62894603

Change-Id: I4f9bec22c6437935f49603f11d27642ef2135382
2017-06-27 03:54:14 -07:00
Ram Mohan M
1bb4cc080b bug fix: pass comp name information to port reconfiguration at all calls
This is a follow up for 553f633762ef748d04717c3cc4b4dbfecb203620 commit

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 62893665

Change-Id: Ic979964bff9c2240e9d9469238882efa86467399
2017-06-22 02:17:27 -07:00
Ram Mohan M
b7ae92bacb bug fix: configure output port frame dimensions
frame dimensions are configured for input port but not output port.
As a result in qcom encoder components, the encoded image is getting
downsized to default dimensions configured for output port. This is now
corrected.

Test: make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module VtsHalMediaOmxV1_0Test
-l INFO

Bug: 62669756
Change-Id: Ib38006edbc82427d322632534ca78d78c5f30eff
2017-06-16 16:26:41 -07:00
Ram Mohan M
c36d38fb55 Increase Timeout for dequeue message
am: 4e39ff9235

Change-Id: If484fa82b3215380f17057e5cadbb6fb42685f81
2017-06-10 05:19:54 +00:00
Ram Mohan M
9fe2d4c99d Increase Timeout for dequeue message
am: 4e39ff9235

Change-Id: Ia6fc966e66668b944c4a484ea413f0bc0982082f
2017-06-10 05:19:54 +00:00
Ram Mohan M
4e39ff9235 Increase Timeout for dequeue message
In vp9 encoder component, the callback responses are being recevied
after timeout. This is seen only in vp9. Even when no buffer processing
is done, Flush test, state transitions test fail due to slow responses.
This behavior is not seen in other components. This requires deeper
understanding of what is main culprit. But for now increase the timeout

Test: VtsHalMediaOmxV1_0Host
Bug: 37501531
Change-Id: I08abfefcf0cb38d312a4a2dc10908e24f88c54fd
2017-06-09 20:08:13 -07:00
Ram Mohan M
db3214fb49 Merge "bug fix: fix encoder input port configuration issues" into oc-dev
am: 7d1592a732

Change-Id: I82164b85e4ad3d2a3f363c14526fb3645ec47397
2017-06-10 01:28:45 +00:00
Ram Mohan M
0eb102a18b Merge "bug fix: fix encoder input port configuration issues" into oc-dev
am: 7d1592a732

Change-Id: I697e916574fc20e1bb9cad79d1e948854078f6bf
2017-06-10 01:28:44 +00:00
Ram Mohan M
e808bef25e Merge "bug fix: configure input port format before setInputSurface" into oc-dev
am: bd5b6f8535

Change-Id: Ic894f1553d826ab1887535e2e70dfce889e715e6
2017-06-10 01:28:41 +00:00
Ram Mohan M
4dc4fa7340 Merge "bug fix: configure input port format before setInputSurface" into oc-dev
am: bd5b6f8535

Change-Id: If5ee86d7d1203019b565743057c77baf1ffb96a8
2017-06-10 01:28:40 +00:00
Ram Mohan M
3b79a2102f choose input color format between 420p, 420sp
am: 1d273b3bcf

Change-Id: I6ffeed6b9bba24936e446efb2c508338180c04b9
2017-06-10 01:28:36 +00:00
Ram Mohan M
0113be3ac3 choose input color format between 420p, 420sp
am: 1d273b3bcf

Change-Id: I95ff1be5ec0e0f2b99a5fc21329b45b87a0f0077
2017-06-10 01:28:36 +00:00
TreeHugger Robot
7d1592a732 Merge "bug fix: fix encoder input port configuration issues" into oc-dev 2017-06-10 01:23:30 +00:00
TreeHugger Robot
bd5b6f8535 Merge "bug fix: configure input port format before setInputSurface" into oc-dev 2017-06-10 01:23:30 +00:00
Ram Mohan M
1d273b3bcf choose input color format between 420p, 420sp
Test: VtsHalMediaOmxV1_0Host
Bug: 37501531
Change-Id: I5fc2f5c69ca545f2ca0b444ce89a8c970f2ff98e
2017-06-09 07:48:25 -07:00
Ram Mohan M
7e4b5758b7 Merge "AudioDec Test: Ignore output port configuration" into oc-dev
am: 11e9b588c4

Change-Id: Ic9b93340791209553374de705101a2e02aed4d09
2017-06-09 03:17:20 +00:00
Ram Mohan M
8cbb07a5f6 Merge "AudioDec Test: Ignore output port configuration" into oc-dev
am: 11e9b588c4

Change-Id: Ib50f2f61385a5382d7f9f24608c93682594dfa1e
2017-06-09 03:17:20 +00:00
Ram Mohan M
9c7ba7deaf Merge "VideoEnc Test: extend lockYCbCr() to hal yuv formats" into oc-dev
am: 452fe6aef2

Change-Id: I67002961c875a6caaa8f3ffde78ca68ded46471f
2017-06-09 03:17:16 +00:00
Ram Mohan M
72ba614357 Merge "VideoEnc Test: extend lockYCbCr() to hal yuv formats" into oc-dev
am: 452fe6aef2

Change-Id: I9c89448a47eb1c0280a0282b9534911886279179
2017-06-09 03:17:16 +00:00
Ram Mohan M
7b106b9c5a Merge "Vts AudioDec Test: Disable EOSTest_M" into oc-dev
am: a12a573955

Change-Id: I90035e92beb470f7910f1f43cc203c4e84d43e29
2017-06-09 03:17:13 +00:00
Ram Mohan M
2e6a7a08f7 Merge "Vts AudioDec Test: Disable EOSTest_M" into oc-dev
am: a12a573955

Change-Id: I437d8e59167209e86d1f8679e7ff2486fbf81c1a
2017-06-09 03:17:13 +00:00
TreeHugger Robot
11e9b588c4 Merge "AudioDec Test: Ignore output port configuration" into oc-dev 2017-06-09 03:12:08 +00:00
TreeHugger Robot
452fe6aef2 Merge "VideoEnc Test: extend lockYCbCr() to hal yuv formats" into oc-dev 2017-06-09 03:12:08 +00:00
TreeHugger Robot
a12a573955 Merge "Vts AudioDec Test: Disable EOSTest_M" into oc-dev 2017-06-09 03:12:08 +00:00
Ram Mohan M
951c9212fb bug fix: configure input port format before setInputSurface
Test: VtsHalMediaOmxV1_0Host
Bug: 37501531
Change-Id: I7b48bf240a243b0b25c1c5e0f36c858b04271194
2017-06-08 16:13:22 -07:00
Ram Mohan M
79312c8aa2 bug fix: fix encoder input port configuration issues
1. Number of P frames being INT_MAX - 1 is misleading few components.
avoid this value
2. enumerate level: The level information is obtained from
enumerateProfileandLevel. Now the component supports up to this level
or 1 step below this level is unclear. This is causing few problems in
the configuration of input port.
3. Disable few error checks for index types that are not universally
supported

Test: VtsHalMediaOmxV1_0Host
Bug: 37501531
Change-Id: I0c3220ec8d17d3527fa5fdae1a2f9c5add11a5de
2017-06-08 16:11:09 -07:00
Zhuoyao Zhang
d56b3083dd Merge "Vts Omx Tests: Disable test that are not IOMX compatible" into oc-dev
am: 6fe8953866

Change-Id: If733f379b1433fc1c86754a9cba772cdf2034f12
2017-06-08 16:29:00 +00:00
Zhuoyao Zhang
c95851eb06 Merge "Vts Omx Tests: Disable test that are not IOMX compatible" into oc-dev
am: 6fe8953866

Change-Id: Ia9804c27bcec133b8570c7f180bce591a7a41bd3
2017-06-08 16:28:44 +00:00
Zhuoyao Zhang
ca7c33143d Merge "Omx vts tests" into oc-dev
am: 2e2e255125

Change-Id: I47b2613f0cfaf1f237bbdc432262b46fc3de3b9f
2017-06-08 16:28:23 +00:00
Zhuoyao Zhang
cb3dc9f239 Merge "Omx vts tests" into oc-dev
am: 2e2e255125

Change-Id: Ifcf78ffd3c042e3eeab90016dc822b747a9cadc1
2017-06-08 16:28:23 +00:00
Ram Mohan M
59197e8cf9 AudioDec Test: Ignore output port configuration
audio dec components vorbis, opus and raw undergo auto output port
configuration and donot allow client to configure. By pass output port
configuration for these components

Test: VtsHalMediaOmxV1_0Host
Bug: 37501531
Change-Id: Ib1539b36589db5bdf1b6d82fcd0c50ba420a0299
2017-06-08 05:25:29 -07:00
Ram Mohan M
2b7a309e25 VideoEnc Test: extend lockYCbCr() to hal yuv formats
Test: VtsHalMediaOmxV1_0Host
Bug: 37501531
Change-Id: I4661f41a92aac44e8a8d2721dfebbd25045fbe1c
2017-06-08 05:23:50 -07:00
Ram Mohan M
5052e189e9 Vts AudioDec Test: Disable EOSTest_M
Sending first buffer as EOS, is not supported across all components.
We will revisit this fixture at a later stage

Test: VtsHalMediaOmxV1_0Host
Bug: 37501531
Change-Id: Ic9f59f7b0a5291710bc50710ef054b0c0c49ca98
2017-06-08 05:02:28 -07:00
Ram Mohan M
2aedf7596d Vts Omx Tests: Disable test that are not IOMX compatible
Android IOMX differs slightly from OMX. Disable tests that are not
needed (for now). We will revisit them at a later stage

Bug:32023356
Bug:37501531

Test: make vts.

Change-Id: Iad71d72a4a66d621f415731d2c0e1a3794ed3524
2017-06-07 17:53:26 -07:00
Ram Mohan M
f0e839e0e7 Omx vts tests
Bug:32023356
Test: make vts.

cleanup: move duplicate code to common

moved code shared across application to common and
removed duplicate code

Change-Id: I2dacb9d72bdf545addd2cc377dd4342c92f50f11

restrict secure component testing to VTS*ComponentTest

Change-Id: I1d2878408df2e4cbe31b3b47a3d6500a90692a7a

Master Test: test component node alloc/dealloc

Change-Id: I89ef853cf5a9f6b0a0af9e25ef484968aec614f6

Component Test: fix dispatchMsg test fixture

Change-Id: Ie1068a5d73a3a573e552cf023c9c7dc78a43ea59

AudioDec Test : missing csd in opus resource files

opus decoder test resource files have a missing csd. This is causing
timestamp list empty test to fail. This is fixed

Change-Id: I6ae3b241fbf3e384943a0e6e80838a45c1fc6687

Change-Id: Ie7775456542f12d5f5d87dded4b0a57fd0723037
2017-06-07 17:50:50 -07:00
Yi Kong
15ec48374b Fix build, put back 'this' capture
'this' capture is not unused, need to put back.

This was due to a bad manual merge.

Test: build
Change-Id: I35f7ddbe7bf8a3b57ea8a83f334afee2156109f6
2017-06-07 14:20:31 -07:00
Chih-Hung Hsieh
d3bc681f26 Remove unused lambda capture
New clang compiler gives warning to unused lambda captures.
Apply clang-format.

Bug: 37423073
Test: Build

Change-Id: Icdf019df1a6c9dcadeb35551377d7afa09eebb3e
(cherry picked from commit 7eb57b60ab)
2017-06-07 12:57:20 -07:00
Zhuoyao Zhang
176c564b54 Merge "Omx vts tests." into oc-dev
am: 89ed70727d

Change-Id: I83ae4865778f42a768a556bebfc6524cbfbc0918
2017-06-06 18:36:48 +00:00
Ram Mohan M
92fb9ab603 Omx vts tests.
Bug:32023356
Test: make vts.

bug fix : fix issues in port reconfiguration

soft and vendor omx components communicate differently. The existing
design doesnt cover this. This is now fixed

Change-Id: I898e60c93961f468e749ab5f13db996c5fd53967

add support for additional color formats

tested encoder for bgra & 420sp color format inputs.
added code to get/set encoder latency.
some more necessary code cleanup.

Change-Id: I05b79d0a7706e20afaf91c88caf3e523c98b4540

IOmxBufferSource callback tests

Change-Id: I2826597c6ab110a0bf52290e3c52ff3fb3c6ea23
2017-05-31 12:53:37 -07:00
Zhuoyao Zhang
77a91e8f08 Merge "Omx vts test." into oc-dev
am: 9a9c8da39b

Change-Id: I4ac6e9ffe3ccd4b473d7355c1c18df14f31cfc3e
2017-05-26 00:26:02 +00:00
Ram Mohan M
ccc194cffd Omx vts test.
Bug:32023356
Test: make vts.

Merged-In: I83b454b999203fb94bc4cc50c91bd99f788131b3

secure buffer allocation for secure components

As of now, we see a crash in encoder test application on secure components
This is due to nullptr access. setting property media.mediadrmservice.enable
to 1 causes a different crash sigabrt(). In decoder securebufferallocation
call passes but we havent done anything with it. This commit needs more work.

Change-Id: I19127e39ad7daf66ac5277406e3857ec45c99e0a

add debug code

This can come in handy to view the decoded/encoded content or
for computation of psnr

Change-Id: I80e60349c76c02e5098df667223a0227f59b8324

video encoder test with anw input buffer

In this commit, anw buffers are provided as input for encoding
as opposed to byte buffers. In process IOmxNode, IGraphicBufferSource api
calls are tested

Change-Id: I7ec4af0746fe59221de42e56b344852d8fadc4f2

add support for additional color formats

Change-Id: Ia88ef9c95882958a68fee5cc68e146c3502a1b48

add eos and timestamp deviation test to video encoder

Change-Id: I9ebeaa53a986ec3f8a2ef55306877aec808d2add

add metamode support for video decoders

Change-Id: I7d6d6991cdc0ed36241d0e99c7a23675664acb10

add a timeout while processing input buffers

do not wait on input buffers processing to complete for an
indefinite amount of time. wait for a predefined duration and
leave

Change-Id: Icceaf0737a52e62f47bc052367e7ddbdc1868a46

move duplicate code to a library

move routines common across audio, video, component folders to a static
library

Change-Id: I55bf21e47571490e989b52b82c48c6e9a4b23745

add end of stream test for audio encoder

add eos test for audio encoder and some more code cleanup

Change-Id: I9d670ed53f6bba5802f919ec915e67bb0fa83518

Ensure all the buffers allocated are cycled

use all buffers provided by the component (even if it is greater than the
number of buffers needed by the bitstream)

Change-Id: I83b454b999203fb94bc4cc50c91bd99f788131b3

move duplicate code to a library - II

Change-Id: I375603feb9da303adc6a2d75ef59e43a11c3bd6d

Change-Id: I83b454b999203fb94bc4cc50c91bd99f788131b3
(cherry picked from commit 1da607c513)
2017-05-25 17:07:10 -07:00
TreeHugger Robot
daa038e999 Merge "Move media_profiles_V1_0.dtd to hardware/interfaces" into oc-dev 2017-05-22 23:53:36 +00:00
Steven Moreland
f203167814 Update makefiles. (2/2)
am: 76cfb84140

Change-Id: I24d21438a5b2f447d2160929c6ff6e17b2b66820
2017-05-22 17:15:20 +00:00
Steven Moreland
76cfb84140 Update makefiles. (2/2)
Adds default configuration for all hals.

Bug: 38415912
Test: pass
Change-Id: Idd1f3a2b7b16ad956d31784a513e93a066cdd02e
2017-05-19 15:54:03 -07:00
Pawin Vongmasa
644fd569bd Move media_profiles_V1_0.dtd to hardware/interfaces
Test: media_profiles_V1_0.dtd exists in system/etc
Bug: 36430270
Merged-In: I43d6eb68cc866a280cbe24fcf0f0af75a3c3ae67
Change-Id: I43d6eb68cc866a280cbe24fcf0f0af75a3c3ae67
2017-05-19 01:11:59 -07:00
Pawin Vongmasa
d30c749adb Move media_profiles_V1_0.dtd to hardware/interfaces
Test: media_profiles_V1_0.dtd exists in system/etc
Bug: 36430270
Merged-In: I43d6eb68cc866a280cbe24fcf0f0af75a3c3ae67
Change-Id: I43d6eb68cc866a280cbe24fcf0f0af75a3c3ae67
2017-05-18 20:42:26 -07:00
Zhuoyao Zhang
640d3f330a Merge "Omx vts tests." into oc-dev
am: 1d619f743d

Change-Id: I80b3c97d186f56210b761e1cd642c209e5071358
2017-05-16 00:08:54 +00:00
Zhuoyao Zhang
1d619f743d Merge "Omx vts tests." into oc-dev 2017-05-15 23:10:37 +00:00
Ram Mohan M
a558d9d5fd Omx vts tests.
Bug: 32023356
Test: make vts.

cover basic encoder & decoder test to all components

Till the previous commit, few components went untested.
This commit covers all softomx components.

Test: tested on softomx components

Change-Id: I4c996a3743a6190d9c6d4c835aa89daefdc14658

enumerate profile and level for encoder components

Change-Id: I4b9d3b3106d33a0241a7e36bd7189c1b3ef40ef0

bug fix: omx android compile as 32bit on 64bit platforms

thanks lajos

Change-Id: If11b20eeb6a75319f9e0cd6a62524dadab2ba8af

Add more VTS tests for IOmxStore and IOmxNode

Change-Id: I3401025b4f0b821f469222707c75b49fe4f0b0c5

avoid false flagging of errors

If testing of the given component is not supported application do not flag
an error, instead exit gracefully.

Change-Id: Ic91a92614d18b204cc50b7d8f6dc1a0ca81ff782

Add support for g711, gsm and raw audio decoder components

Change-Id: Ied70abdabb397a7a63e13293a8b35365a6b23e4f

split eos test

the eos test in decoders handles three different cases. These are now
separated in to different test fixtures.

Change-Id: I30c102109ce60a345f878d03d481337462fc817f

Change-Id: I3401025b4f0b821f469222707c75b49fe4f0b0c5
2017-05-14 14:55:57 -07:00
Steven Moreland
a32ebce372 Update makefiles for c++-sources and c++-headers. (2/2)
am: bc71124120

Change-Id: Icf36068a4d863a8520854c7a83e51f960c188e12
2017-05-11 00:48:55 +00:00
Steven Moreland
bc71124120 Update makefiles for c++-sources and c++-headers. (2/2)
Test: pass
Bug: 38174080
Change-Id: Icad451f9f9ad5f46412356a4171a8a1222f8e545
2017-05-09 19:58:43 -07:00
Zhuoyao Zhang
d4e4937388 Merge "Omx VTS tests" into oc-dev
am: 9ec2c9e099

Change-Id: I59f8401168779e7b06302e6ab5ea939c7c116877
2017-05-03 16:31:23 +00:00
Ram Mohan M
23c658f118 Omx VTS tests
Bug: 32023356
Test: make vts.

preload elementary stream information during decoding

instead of using extractor to get the frame boundaries, frame
flags and timestamp information, they are read from a metadata
file in the resources directory on demand. Instead of doing it
on demand, we load the entire file information and use it. This
modification comes in handy for future test fixtures

Change-Id: I3a434018111eb66008f17c24b4c2c16d0db22ed1

bug fix: wait on input consumption

after queuing all the frames that need to be processed by
the component, the application continues its job of deinitializing
the component before ensuring all the inputs were decoded.
This is corrected.

Change-Id: I09a11fd5badabe2889ba243383145be07be51953

add end of stream test for audio decoder

component is provided an input with nFlags field set OMX_BUFFERFLAG_EOS.
the application expects this flag to be returned via one of its output
buffers. this flag is passed to the component at different timings to test
the robustness.

Change-Id: I00e3fdd5ef50d93f2620d8b0e9fc3c358de428d7

add timestamp deviation test for decoder components

the list of timestamps received from the component is
compared with the list provided to the component for any
differences

Change-Id: Ifcc16b7a00f58e1c1dfdba9f538d307ddcbfcc71

add option to configure resource files path

default location of resource files is /sdcard/media/. client can provide
choice of this location via the option -P.

example: VtsHalMediaOmxV1_0TargetVideoDecTest -C <component> -R <role> -P /sdcard/res/

Change-Id: Ib8d3af03c19a171e3bbb9839f7e331cbe576b24b

add flush test for audio & video dec components

while decoding a group of frames, flush call is made and it is
checked if all the i/o buffers are sent back to client. we then
seek to the next keyframe and test for decoding. monotonous
increase of timestamps is ensured.

Change-Id: Id48c76c157ebdfdefd3003e57519da3bd9715973
2017-05-02 23:44:26 -07:00
Steven Moreland
5c63b11f76 Remove redundant dependencies
android.hidl.base@1.0 and android.hidl.manager@1.0 are built into libhidltransport.

Test: links
Bug: 33276472
Merged-In: I08aaad80f7e2fc262aa3a8b66fe932e8133a928d
Change-Id: I08aaad80f7e2fc262aa3a8b66fe932e8133a928d
2017-05-02 15:29:30 -07:00
Ram Mohan M
f9c4003fcd Basic audio/video encoder/decoder test.
Bug: 32023356
TODO: cleanup the test code b/37719510

ComponentTest :-
1. allocatePortBuffers cleanup
2. More than one test fixture performs State transitions
(Loaded <-> Idle <-> Execute). So, this is moved to a routine and
invoked when needed.
3. Dispatch input/output buffers code section is moved to a routine
4. Seperated Test fixtures StateTransitions_Flush
5. Added Doc string
6. Test component port population

AudioDecTest :-
1. Backport Cleanup changes from ComponentTest to AudioDecTest
2. Added elementary streams for the following codecs mp3, amr-nb &
Updated paths to elementary streams in audio decoder test

Test: Tested ComponentTest and AudioDecTest for mp3

Change-Id: I90decde4dfab55c3d456dfd88b474dd5abe8eebc

List Available components (roles) in android media framework

Change-Id: I210665843e997e60b167f2d680c87b1cc4f90c58

add README.md

Change-Id: I3a2a057d10b18f4f374928a996b8b4ea4b084770

add basic video decoder tests

for a given component test for successful decoding of an input clip

Test: tested for softavc decoder

Change-Id: I2f4c034c34a456e1ed0c2d32ef381ca48aeda8ff

add basic audio encoder tests

for a given component, test for successful encoding of an input clip

Test: tested on softaac, softamrnb, softamrwb & softflac components

Change-Id: I05f5646ffed37143a21ea72f0fd782c2bb5c4d82

add basic video encoder tests

For a given component test successful encoding of nframes of a
raw video clip

Test: tested on softavc encoder.
Change-Id: Ia1d754061e8362f6c08a2be3d6384c95589a36de
2017-04-26 21:49:04 +00:00
Pawin Vongmasa
0048c96885 Make setTimeLapseConfig take frequencies instead of periods.
Test: Manual use of Camera, Movies, Photos and YouTube apps.

Test: With CtsMediaTestCases.apk installed,
adb shell am instrument -e size small -w
'android.media.cts/android.support.test.runner.AndroidJUnitRunner'

Bug: 37302530
Change-Id: I63ba807955c2e2a7ec7f6ecc2d8a832f252c0c1b
2017-04-20 18:28:55 -07:00
Hangyu Kuang
171bbf5ecb media: Add getStopTimeOffsetUs in GraphicBufferSource.
Test: Compiles
Bug: 35925528
Change-Id: Ie469e42c986211540880f04a5e47efc090817ba8
2017-04-20 18:17:51 -07:00
Andreas Huber
6fa374ad30 Changed the naming scheme for Treble java libraries (both static and dynamic)
android.hardware.foo@1.0-java => android.hardware.foo-V1.0-java

Bug: 37207894
Test: make
Change-Id: I177f4cfd42f635aaf096107af8309d1a82ed41c0
Merged-In: I8b91881dfaf507c0d852d56ebbb53ff0987e5a34
2017-04-13 15:23:46 -07:00
Zhuoyao Zhang
9eeda9561f Merge "media.omx VTS test baseline" into oc-dev 2017-04-13 19:24:29 +00:00
Ram Mohan M
527cf97159 media.omx VTS test baseline
Change-Id: I0cb1997e9cc0427fdfe8091de84e8cb75c7bab14

Test GetPortIndices

Tests support for omx index type: OMX_IndexParamAudioInit,
OMX_IndexParamImageInit, OMX_IndexParamVideoInit, OMX_IndexParamOtherInit.

Change-Id: I50c7c70c8e000684ef02a6e372be32f9c178e74d

Test Port Enabling/Disabling

Enumerate the list of ports available on the component.
Test if they can be disabled and enabled. This test is carried
when the component is in Loaded state. Also, omx-il specification
allows enabling and disabling all ports at once by passing the
argument OMX_ALL. The behavior of the component on this argument
is tested. Disable and enable a port index that is not supported
by the component is also tested.

Change-Id: I1ab3cc3652a63d05bd0c3ceae92e01ebd4aae12d

Test State Transitions

Change the states of the component from loaded->idle->execute->idle->loaded
While the component is switching states, make sure the below dependencies are met.
1. When the component is switching from Loaded to Idle,
ensure this transition does not take place until the supplier has provided
the ports their necessary buffers
2. When the component is switching from Idle to execute,
change the buffer owner from Client to Component.
3. When the component is switching from execute to Idle,
ensure the client got all the buffers it supplied to the component back.
4. When the component is switching from Idle to loaded,
ensure this transition takes place only after freeing all the buffers

Change-Id: I7c3f6f65adf3afda4e54ca7cafb5bb7799a5b2d6

Test for Flush

When the component is in execute state and as buffers are being
exchanged between client and component, call a flush. Test that all
the unprocessed buffers are returned to the client

Change-Id: I00f4c356df0e832aecc457a53163532e82ead9a3

Fix object labels for master and component

This changes maintains consistency across Vts test folders

Change-Id: I0f3e40db04876a3dab170936037c42f994d67c18

Audio_enc/Audio_dec folders are removed.

The current folders needs work. We will revisit this again

Change-Id: Ic8287288d0727f1384e096b846e0066b2ec9f95f

Fix Object labels for common

Change-Id: I8f19a339983aa81d581ed3d55780180f8d976382

Test Port Definition structures

Tests support for omx index type OMX_IndexParamPortDefinition

Change-Id: I45d82199c6cf20f24aa42d9fea69bd1c88a51757

Bug fix : Pass i/o buffer structure information while dequeueing messages

Also fixed sending false port id during freeBuffer call.

Change-Id: Id5c4f67cccea6b577880e42ee0d294cc55d534ad

Test port reconfiguration and buffer flow in audio decoder

In the current commit we have added tests for audio decoder
components. The selected component decodes an elementary stream
and in this process port reconfiguration, buffer flow and flush
are tested.

Test: mma
Change-Id: I42c8397f704f5fa69878efe91932c477ac66d1b2
2017-04-13 17:06:33 +00:00
Steven Moreland
17f0c0dde7 Merge changes from topic 'hidl-vendor-available' into oc-dev
* changes:
  Update makefiles (2/2).
  Update makefiles.
2017-04-12 22:13:58 +00:00
Steven Moreland
3b36a83a2e Update makefiles (2/2).
Test: pass
Change-Id: Ic7e6289c7b69785491708ce0acbb657f4c704229
2017-04-11 09:58:03 -07:00
Lajos Molnar
c8949ecf81 Add IOmxStore and remove IOmxNode::setQuirks.
IOmxStore is used for passing results from parsing media_codecs.xml and
related XML files to the framework. Once XML parsing is moved to the HAL
side, IOmxNode::setQuirks will not be needed anymore.

Test: Media post-submit tests on Pixel phone.
Test: Manual use of Camera, Photos, Play Movies and YouTube apps.
Bug: 36952714
Change-Id: I6b24c486c6f8afcbef6ee88a13b2b5bb08d3e656
2017-04-07 23:49:16 -07:00
Steven Moreland
561b602a99 Update makefiles for hwtypes.h
Test: pass
Change-Id: I8e17eb8fec5d2f04c526476a866c8dd54ccb556c
2017-03-28 15:18:20 +00:00
Pawin Vongmasa
4a3cd13ddc Move IGraphicBufferProducer to graphics/bufferqueue.
Test: With CtsMediaTestCases.apk installed,
adb shell am instrument -e size small -w
'android.media.cts/android.support.test.runner.AndroidJUnitRunner'

Bug: 35442034
Change-Id: Ia1ef0eb069fcd97f7afcf42bf3bb16c8c6a752c4
2017-03-15 11:42:40 -07:00
Robert Shih
a27875035c Revert "Revert "Add return status to IGraphicBufferSource methods""
This reverts commit b729637648.

Bug: 36110748
Change-Id: Ia12c00d57b9e8d4b66265018c3970416c265091a
2017-03-11 01:22:12 +00:00
Rachad Alao
b729637648 Revert "Add return status to IGraphicBufferSource methods"
Bug: 36110748

This reverts commit ed32ff5150.

Change-Id: Ibb5e72fc11f705a571491103fc5c68005cf54a30
2017-03-10 21:48:02 +00:00
Robert Shih
ed32ff5150 Add return status to IGraphicBufferSource methods
Bug: 35449087
Test: Manual
Change-Id: I0a44630f2b5b07cfd62f41764d0ca43a924d221a
2017-03-03 22:37:42 +00:00
Hangyu Kuang
2f16dfed5d Merge "media: Update IGraphicBufferSource interface for the timestamp support." 2017-02-17 00:26:05 +00:00
Brian C. Anderson
2413ca1492 Merge "Update HIDL with CompositorTiming." 2017-02-15 22:32:45 +00:00
Brian Anderson
68214ec86d Update HIDL with CompositorTiming.
Test: Tested manually with Chrome.
Change-Id: I0e911612f4db1ae57098a5bc53afdce61dfe5837
2017-02-13 15:26:13 -08:00
Tri Vo
85120c9490 Update makefiles to use filgroup syntax.
Test: mmma hardware/interfaces
Change-Id: I9f4457c78b7820c242d46359f0debe05d1b3e482
2017-02-10 12:45:19 -08:00
TreeHugger Robot
930086babd Merge "Migrate driver/profiler to test/vts-testcase/hal." 2017-02-08 18:33:06 +00:00
TreeHugger Robot
eb1d38a5ef Merge "Update QueueBufferOutput." 2017-02-08 05:33:20 +00:00
Tri Vo
ce608bec74 Migrate driver/profiler to test/vts-testcase/hal.
Bug: 34893894
Test: mmma hardware/interfaces

Change-Id: Ic72716577a5f93700c5056986aff55a76b170b5d
2017-02-07 19:13:28 -08:00
Pawin Vongmasa
761622e5d4 Update QueueBufferOutput.
QueueBufferOutput was modified in ag/1806678.

Test: Compiles

Bug: 31399200
Bug: 34481539
Change-Id: I287a4ba84348323e2e39dcbac03c3bf17357d6bc
2017-02-06 18:22:40 -08:00
Zhuoyao Zhang
e9b43bbe2d Update Andriod.bp for hals.
* Add driver/profiler build rule for all hals.

Test: mma
Change-Id: I98325f7af14fec7dd1bb64b1668de8c7c20ace92
2017-02-03 10:03:01 -08:00
Hangyu Kuang
9ad0d00def media: Update IGraphicBufferSource interface for the timestamp support.
Test: Manual recording
Bug: 32765698

	modified:   1.0/IGraphicBufferSource.hal

Change-Id: I87935c67d161a1812629af3e0a9185bc025ce472
2017-02-01 18:51:10 -08:00
Pawin Vongmasa
120c4da82a Use HIDL's memory for shared memory and add wrapper interfaces for
IGraphicBufferProducer.

Test: Compiles
Bug: 31399200
Change-Id: I17a3b1c8f20e9c11474761d120edd2e75353cc6d
2017-01-25 17:14:20 -08:00
Yifan Hong
a18049ad13 Bp/nFoo -> Bp/nHwFoo.
Test: mma
Bug: 33554989
Change-Id: I4b3f852d20fdfc49f2da671dd3c12d618ffb4140
2017-01-11 13:14:59 -08:00
Pawin Vongmasa
38585c2532 Remove the return Status from IGraphicBufferSource's functions.
Now, only the transport-level status is needed, so
::android::hardware::Status is sufficient.

Also simplify IOmxNode::sendCommand().

Test: Compiles

Bug: 31399200
Change-Id: Ia4cfc16681d19eb96f89d8db715fcc14516cb41a
2016-12-19 13:04:51 +07:00
Pawin Vongmasa
cba23c6631 Change GraphicBuffer to AnwBuffer and update its fields.
The server side only needs information of type ANativeWindowBuffer, not
GraphicBuffer.

Test: Compiles

Bug: 31399200
Change-Id: Ic967c58d8c922e0c422e5b63fa5ce2188da89010
2016-12-19 13:04:44 +07:00
Pawin Vongmasa
d13c84bce7 Add an enum value to Status.
Test: Compiles.

Bug: 31399200
Change-Id: I5452f74e91d9ad83629143b9af7e56e70036cbb4
2016-12-19 13:04:37 +07:00
Yifan Hong
501839495f Update makefiles for hidlizing IBase.
A new dependency android.hidl.base@1.0 is added for
all projects.

Test: mma

Change-Id: Ia70d0eb0d74de06475a339698386d383d491a43a
2016-12-03 01:01:14 -08:00