OSDN Git Service

android-x86/hardware-interfaces.git
6 years agobug fix: during fatal fails bypass freeNode() calls
Ram Mohan M [Wed, 11 Oct 2017 10:08:25 +0000 (15:38 +0530)]
bug fix: during fatal fails bypass freeNode() calls

If test application undergoes a fatal failure bypass all the omx
api calls till the application safely returns.

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: If5a33672a31665db913e53c6f8ba8bea6cfd3ebd

6 years agoMerge "omxvts: exit gracefully during fatal fails" into oc-mr1-dev
TreeHugger Robot [Sat, 7 Oct 2017 02:26:15 +0000 (02:26 +0000)]
Merge "omxvts: exit gracefully during fatal fails" into oc-mr1-dev

6 years agoMerge "Added VTS tests for MediaCas - II" into oc-mr1-dev
TreeHugger Robot [Fri, 6 Oct 2017 16:49:20 +0000 (16:49 +0000)]
Merge "Added VTS tests for MediaCas - II" into oc-mr1-dev

6 years agoAdd neuralnetworks HAL to the current.txt
Michael Butler [Thu, 5 Oct 2017 20:30:14 +0000 (13:30 -0700)]
Add neuralnetworks HAL to the current.txt

Bug: 63905942
Test: None
Change-Id: I7c247381e8fd97b91d11531d3db595408d8f1b74

6 years agoAdded VTS tests for MediaCas - II
Suresh Sivaraman [Thu, 21 Sep 2017 13:14:24 +0000 (18:44 +0530)]
Added VTS tests for MediaCas - II

Bug: 63914034
Bug: 65593293

Test:
Build & test vts:

> make vts -j64

> vts-tradefed run vts -m VtsHalCasV1_0Target

Change-Id: If5a0e7cb3d520843ba45147a085c43b94027e584

6 years agoCorrected performanceInfo documentation
Jean-Luc Brouillet [Thu, 5 Oct 2017 01:24:59 +0000 (18:24 -0700)]
Corrected performanceInfo documentation

We'll be using relative numbers, change the doc to reflect that.

Bug: 63905942
Test: Compiled
Change-Id: Iad67621cfc76490c2dc471865a3f82cebf175440

6 years agoomxvts: exit gracefully during fatal fails
Ram Mohan M [Wed, 27 Sep 2017 11:12:19 +0000 (16:42 +0530)]
omxvts: exit gracefully during fatal fails

Bug: 63796949

Change-Id: I735db42f7de6926adddd328db1a48de35faaa608

6 years agoMerge "Fix VTS failures on tests with optional inputs" into oc-mr1-dev
TreeHugger Robot [Thu, 5 Oct 2017 06:15:59 +0000 (06:15 +0000)]
Merge "Fix VTS failures on tests with optional inputs" into oc-mr1-dev

6 years agoFix VTS failures on tests with optional inputs
I-Jui (Ray) Sung [Thu, 5 Oct 2017 03:49:57 +0000 (20:49 -0700)]
Fix VTS failures on tests with optional inputs

RequestArgument for optional inputs should have its .hasNoValue set
and other fields left empty. The test harness was incorrectly ignoring
the hasNoValue field and initializing the offset field even for missing
arguments. That inconsistency was caught by the sample driver and caused
an invalid argument error. With this CL the test harness now populates
requests correctly for tests with intentionally missing inputs, and that
fixes lstm* and lsh_projection.

Test: VtsHalNeuralnetworksV1_0TargetTest on Angler
Bug: 63905942
Change-Id: I8729124b720f0d4163f4b9efd452f82b825b0885

6 years agoEarly test termination for unsupported models
Michael Butler [Thu, 5 Oct 2017 01:13:52 +0000 (18:13 -0700)]
Early test termination for unsupported models

Some service drivers won't be able to support all operations. In those
cases, try to prepare the model anyway. If it works, proceed with the
test; if it fails, terminate the test early.

For example, the hvx nn service does not support svdf, so it should
attempt to prepare the model, but the test should not fail if the hvx
driver reports an error.

Bug: 63905942
Test: mm, expected behavior when running on hvx service
Change-Id: I51aa1911145f0d17590906c21314d2bdc24cffea

6 years agoMerge "Only verify output of models a vendor nn service fully supports" into oc-mr1-dev
Michael Butler [Wed, 4 Oct 2017 23:49:51 +0000 (23:49 +0000)]
Merge "Only verify output of models a vendor nn service fully supports" into oc-mr1-dev

6 years agoOnly verify output of models a vendor nn service fully supports
Michael Butler [Wed, 4 Oct 2017 09:38:42 +0000 (02:38 -0700)]
Only verify output of models a vendor nn service fully supports

Some service drivers won't be able to support all operations. In those
cases, run the test as far as it can go, but ignore the output data.

For example, the hvx nn service does not support svdf, so it should
attempt to prepare and execute the model, but the test should not fail
if the hvx driver reports an error. However, hvx indicates it is able to
run MAX_POOL_2D, so the test should fail if that output is incorrect.

Bug: 63905942
Test: mm
Test: run vts binary on hvx service
Change-Id: I3ebc05836c8e070b66daa2e38b107f96637ca812

6 years agoMerge "Fix VtsHalDrmV1_0Target clearkey tests." into oc-mr1-dev
TreeHugger Robot [Wed, 4 Oct 2017 16:38:02 +0000 (16:38 +0000)]
Merge "Fix VtsHalDrmV1_0Target clearkey tests." into oc-mr1-dev

6 years agoMerge "Move android.hardware.tests.libhwbinder@1.0-impl to system.img" into oc-mr1-dev
TreeHugger Robot [Wed, 4 Oct 2017 13:41:00 +0000 (13:41 +0000)]
Merge "Move android.hardware.tests.libhwbinder@1.0-impl to system.img" into oc-mr1-dev

6 years agoFix VtsHalDrmV1_0Target clearkey tests.
Edwin Wong [Fri, 29 Sep 2017 22:58:37 +0000 (15:58 -0700)]
Fix VtsHalDrmV1_0Target clearkey tests.

1. Fix the following issues:
  - queryKeyStatus is now implemented, do not return
    ERROR_DRM_CANNOT_HANDLE anymore
  - expectedKeyRequest and knownKeyResponse should use
    '-' and '_' instead of '+' and '/' for base64Url encode;
    one uint8_t change from 0x2b to 0x2d for each vector

2. Add tests to create clearkey drm and crypto plugin using
both common pssh box format system ID and clearkey scheme
UUID.

Test: Vts test on Marlin and Taimen
  adb shell /data/nativetest/VtsHalDrmV1_0TargetTest/
  VtsHalDrmV1_0TargetTest

Test: Cts ClearKeyTest on Marlin and Taimen
  ANDROIOP= ./android-cts/tools/cts-tradefed run cts
  -m CtsMediaTestCases
  --test android.media.cts.ClearKeySystemTest

Test: ClearKey plugin unit tests
  adb shell LD_LIBRARY_PATH="/vendor/lib/mediadrm"
  /data/nativetest/ClearKeyDrmUnitTest/ClearKeyDrmUnitTest

Test: netflix and Play Movies & TV for sanity check

bug: 66327050
Change-Id: Icfbe715cf246a7c5253d5f104d0fd446618b18bd

6 years agoMerge "Fix problem passing null optional operands." into oc-mr1-dev
TreeHugger Robot [Wed, 4 Oct 2017 01:40:37 +0000 (01:40 +0000)]
Merge "Fix problem passing null optional operands." into oc-mr1-dev

6 years agoMerge "NNAPI HAL: Change IEvent to explicit callbacks" into oc-mr1-dev
TreeHugger Robot [Wed, 4 Oct 2017 00:07:22 +0000 (00:07 +0000)]
Merge "NNAPI HAL: Change IEvent to explicit callbacks" into oc-mr1-dev

6 years agoMerge "wifi(implementation): Increase stop timeout" into oc-mr1-dev
TreeHugger Robot [Tue, 3 Oct 2017 23:42:36 +0000 (23:42 +0000)]
Merge "wifi(implementation): Increase stop timeout" into oc-mr1-dev

6 years agoFix problem passing null optional operands.
Jean-Luc Brouillet [Thu, 28 Sep 2017 02:22:04 +0000 (19:22 -0700)]
Fix problem passing null optional operands.

Allow optional operands to be specified as having no value in the model.
Make sure that both these and model arguements that have no value can
be passed to through the HAL. Change the validation to ensure that all
required ops are there.

Bug: 63905942
Test: Compiled and ran runtime/test and VTS tests.

Change-Id: Ia20954b9712042f5c0571822967521aedb66150b

6 years agoNNAPI HAL: Change IEvent to explicit callbacks
Michael Butler [Fri, 22 Sep 2017 20:26:12 +0000 (13:26 -0700)]
NNAPI HAL: Change IEvent to explicit callbacks

IEvent was a synchronization primitive which caused some confusion
in the interface. Originally the event object was paired with an
asynchronous task, and the asynchronous task would signal this event
when the corresponding output was ready to be used.

In the case of IDevice::prepareModel, the function call would return an
IPreparedModel object that was not guaranteed to be prepared until the
runtime had returned from waiting on the corresponding event object.
The event object has been changed to two explicit callbacks--
IPreparedModelCallback and IExecutionCallback. Now,
IDevice::prepareModel no longer returns an unfinished IPreparedModel;
instead, it will pass the IPreparedModel object to the runtime through
IPreparedModelCallback::notify. When the runtime retreives the
IPreparedModel object, the asynchronous task has already finished
preparing the model.

The two callbacks are used for different purposes. Each has its own
version of notify to pass the data back to the runtime:
* IPreparedModelCallback::notify(ErrorStatus, IPreparedModel)
* IExecutionCallback::notify(ErrorStatus)

Bug: 63905942
Test: mm, vts, ml/nn/runtime/tests
Change-Id: I0c88cd262ba762e0af15e9da31ebe813a5d150b2

6 years agoCamera: Update VTS for new camera@3.3 interfaces
Eino-Ville Talvala [Fri, 29 Sep 2017 23:37:59 +0000 (16:37 -0700)]
Camera: Update VTS for new camera@3.3 interfaces

- Cross check device version in name with ICameraDeviceSession version
- Switch all calls to configureStreams to configureStreams_3_3 when
  a v3.3 ICameraDeviceSession is available

Test: Test passes on v3.3 devices, doesn't regress on v3.2
Bug: 62358514
Change-Id: I7450e5d5119c5ec1fe9f1360bfc99fc8a47d0936

6 years agoCamera: Enable VTS tests for 3.3 camera devices
Emilian Peev [Thu, 28 Sep 2017 10:10:08 +0000 (11:10 +0100)]
Camera: Enable VTS tests for 3.3 camera devices

A new 3.3 Camera device API is available and devices supporting it
need to be tested as well.

Bug: 67028051
Test: run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO

For passthrough devices: run commandAndExit vts
--skip-all-system-status-check --skip-preconditions --abi armeabi-v7a
--module VtsHalCameraProviderV2_4Target -l INFO

Change-Id: I91adb3c1867cdd8f8f810a2bd5b0a8ba2b0c1fcf

6 years agoMerge "omxvts: fix binary for 64bit platforms" into oc-mr1-dev
TreeHugger Robot [Sun, 1 Oct 2017 23:09:21 +0000 (23:09 +0000)]
Merge "omxvts: fix binary for 64bit platforms" into oc-mr1-dev

6 years agoMerge "omxvts: misplaced lock can cause race condition" into oc-mr1-dev
TreeHugger Robot [Sun, 1 Oct 2017 23:09:00 +0000 (23:09 +0000)]
Merge "omxvts: misplaced lock can cause race condition" into oc-mr1-dev

6 years agowifi(implementation): Increase stop timeout
Roshan Pius [Fri, 29 Sep 2017 22:04:33 +0000 (15:04 -0700)]
wifi(implementation): Increase stop timeout

The current timeout of 50ms seems to be insufficient on some devices.
So, changing it to a higher value.

Bug: 66334487
Test:  Ran the VTS tests continously in a loop for 30 minutes
without failure. Previously, failure occurence was about once every
5-10 minutes.
`while adb shell /data/VtsHalWifiSupplicantV1_0TargetTest; do :;
done; adb bugreport`

Change-Id: I884de64d748b46b9710a702f3c72cc30dd51627e

6 years agoomxvts: fix binary for 64bit platforms
Ram Mohan M [Wed, 27 Sep 2017 15:23:22 +0000 (20:53 +0530)]
omxvts: fix binary for 64bit platforms

Bug: 63796949

Change-Id: Ieddae0bd9fc45a88c6e1579bdd588b77b4174a1f

6 years agoomxvts: misplaced lock can cause race condition
Ram Mohan M [Tue, 26 Sep 2017 10:06:16 +0000 (15:36 +0530)]
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

6 years agoMore tetheroffload VTS fixes for over-specified tests.
Erik Kline [Tue, 26 Sep 2017 09:56:35 +0000 (18:56 +0900)]
More tetheroffload VTS fixes for over-specified tests.

[1] Call addDownstream before removeDownstream in affirmative tests

In order to properly test removeDownstream() where the HAL might
reasonably reject downstreams not previously added we now call
addDownstream() first (for affirmative tests).

[2] Clarify when stopOffload() return values can be safely ignored.

Test: as follows
    - make vts -j30 BUILD_GOOGLE_VTS=true && \
      vts-tradefed run commandAndExit vts \
          --skip-all-system-status-check \
          --primary-abi-only \
          --skip-preconditions \
          --module VtsHalTetherOffloadControlV1_0Target \
          -l DEBUG
      still doesn't pass but it's better than before  :)
Bug: 65270149
Change-Id: I27a574bd2110e3a1626de343f6b57b9efb8cdf83

6 years agoTweaks to tetheroffload VTS: fixes and fd leaks
Erik Kline [Tue, 12 Sep 2017 07:21:55 +0000 (16:21 +0900)]
Tweaks to tetheroffload VTS: fixes and fd leaks

Specifically:
    - always use hidl_handle::setTo(..., true) to force fd ownership
      (and thereby cleanup in hidl_handle destructor)
    - initialize config HAL multiple times to try to trigger fd leaking bugs

Test: as follows
    - make vts -j30 BUILD_GOOGLE_VTS=true && \
      vts-tradefed run commandAndExit vts \
          --skip-all-system-status-check  \
          --primary-abi-only              \
          --skip-preconditions            \
          --module VtsHalTetherOffloadConfigV1_0Target -l INFO

      I/ResultReporter: Invocation finished in 48s. PASSED: 4, FAILED: 0, MODULES: 1 of 1

    - make vts -j30 BUILD_GOOGLE_VTS=true && \
      vts-tradefed run commandAndExit vts \
          --skip-all-system-status-check  \
          --primary-abi-only              \
          --skip-preconditions            \
          --module VtsHalTetherOffloadControlV1_0Target -l INFO
Bug: 29337859
Bug: 32163131
Bug: 64976634
Bug: 65270149
Bug: 65529504
Change-Id: Ib47b35a01092bb2052d241f7bba951bbf7691b66

6 years agoRemove operationTuple.
Jean-Luc Brouillet [Sat, 23 Sep 2017 22:15:58 +0000 (15:15 -0700)]
Remove operationTuple.

Removed operationTuple from Model in the HAL, as the data type was
redundant information.

Removed supportedOperationTuples from Capabilities, as real drivers need
more complex restrictions than this provided.  For the OC-MR1, we'll just
rely on getSupportedNodes.

Also removed the unused cachesCompilation.

Bug: 63905942
Test: Compiled and ran tests.
Change-Id: I15f33d14634f2e1c8d726b1bd01d5b9e123b47ea

6 years agoMerge "Removed WIP tag from Neuralnetworks HAL" into oc-mr1-dev
Michael Butler [Sat, 23 Sep 2017 03:38:47 +0000 (03:38 +0000)]
Merge "Removed WIP tag from Neuralnetworks HAL" into oc-mr1-dev

6 years agoMerge "Add negative tests to the Neuralnetworks HAL" into oc-mr1-dev
Michael Butler [Sat, 23 Sep 2017 01:07:43 +0000 (01:07 +0000)]
Merge "Add negative tests to the Neuralnetworks HAL" into oc-mr1-dev

6 years agoMerge "Downgrade generated test harness for NDK libc++" into oc-mr1-dev
Michael Butler [Sat, 23 Sep 2017 00:57:48 +0000 (00:57 +0000)]
Merge "Downgrade generated test harness for NDK libc++" into oc-mr1-dev

6 years agoRemoved WIP tag from Neuralnetworks HAL
Michael Butler [Tue, 19 Sep 2017 23:26:47 +0000 (16:26 -0700)]
Removed WIP tag from Neuralnetworks HAL

Additionally cleaned up some comments and formatting.

Bug: 63905942
Test: not needed
Change-Id: I0306f6d37f3c9ce9b365b5f838c27e81fa001ccd

6 years agoMerge "Added VTS tests for MediaCas" into oc-mr1-dev
TreeHugger Robot [Thu, 21 Sep 2017 23:26:52 +0000 (23:26 +0000)]
Merge "Added VTS tests for MediaCas" into oc-mr1-dev

6 years agoDowngrade generated test harness for NDK libc++
I-Jui (Ray) Sung [Wed, 20 Sep 2017 20:45:50 +0000 (13:45 -0700)]
Downgrade generated test harness for NDK libc++

NDK libc++ does not yet support full C++14. This CL replaced use of
std::get with type as well as auto lambda arguments with equivalent
C++11 constructs in the VTS test harness for NNAPI.

Test: VtsHalNeuralnetworksV1_0TargetTest
Bug: 63905942
Change-Id: If75e7c088e9221a70bcc47bc647e0dd7b045bfa1

6 years agoAdded VTS tests for MediaCas
Suresh Sivaraman [Fri, 15 Sep 2017 06:21:15 +0000 (11:51 +0530)]
Added VTS tests for MediaCas

Bug: 63914034
Bug: 65593293
Change-Id: I1064ac14513508d2f821a1a27dafcc7389b1a691

6 years agoMerge "Add flush test" into oc-mr1-dev
TreeHugger Robot [Thu, 21 Sep 2017 01:01:13 +0000 (01:01 +0000)]
Merge "Add flush test" into oc-mr1-dev

6 years agoMerge "Enable EOSTest_M test" into oc-mr1-dev
TreeHugger Robot [Thu, 21 Sep 2017 01:00:47 +0000 (01:00 +0000)]
Merge "Enable EOSTest_M test" into oc-mr1-dev

6 years agoMerge "Signal eof flag for each input buffer" into oc-mr1-dev
TreeHugger Robot [Thu, 21 Sep 2017 01:00:22 +0000 (01:00 +0000)]
Merge "Signal eof flag for each input buffer" into oc-mr1-dev

6 years agoMerge "Add support for preset ANW buffer mode" into oc-mr1-dev
TreeHugger Robot [Thu, 21 Sep 2017 00:59:59 +0000 (00:59 +0000)]
Merge "Add support for preset ANW buffer mode" into oc-mr1-dev

6 years agoMerge changes I4a4aad3a,I40b94919 into oc-mr1-dev
TreeHugger Robot [Thu, 21 Sep 2017 00:59:35 +0000 (00:59 +0000)]
Merge changes I4a4aad3a,I40b94919 into oc-mr1-dev

* changes:
  Extend adaptive playback test for graphic buffer modes
  Move graphic buffer allocation to common

6 years agoMerge "Add vts audio dec test for flac decoder" into oc-mr1-dev
TreeHugger Robot [Thu, 21 Sep 2017 00:59:09 +0000 (00:59 +0000)]
Merge "Add vts audio dec test for flac decoder" into oc-mr1-dev

6 years agoAdd negative tests to the Neuralnetworks HAL
Michael Butler [Wed, 20 Sep 2017 02:59:45 +0000 (19:59 -0700)]
Add negative tests to the Neuralnetworks HAL

This CL adds negative tests to:
* IDevice::getSupportedOperations
* IDevice::prepareModel
* IPreparedModel::execute

It does not make sense to have negative tests for the remaining HAL
entries:
* IDevice::getStatus (all returned values are valid)
* IDevice::getCapabilities (no input or state)
* IEvent::notify (synchronization callback primitive)

Bug: 63905942
Test: mm, running on sample driver has expected output
Change-Id: Ia3cf5ce05dd07ff3f688395b8d0e581851909028

6 years agoMerge "Add adaptive playback test" into oc-mr1-dev
TreeHugger Robot [Wed, 20 Sep 2017 10:14:12 +0000 (10:14 +0000)]
Merge "Add adaptive playback test" into oc-mr1-dev

6 years agoMerge "Camera: Add default implementation of camera.device@3.3" into oc-mr1-dev
Eino-Ville Talvala [Tue, 19 Sep 2017 15:53:43 +0000 (15:53 +0000)]
Merge "Camera: Add default implementation of camera.device@3.3" into oc-mr1-dev

6 years agoMerge "Fix typos in IOmxStore.hal" into oc-mr1-dev
TreeHugger Robot [Tue, 19 Sep 2017 01:14:00 +0000 (01:14 +0000)]
Merge "Fix typos in IOmxStore.hal" into oc-mr1-dev

6 years agoMerge "Add capability for sensor hal to use wakelock" into oc-mr1-dev
Peng Xu [Mon, 18 Sep 2017 19:07:09 +0000 (19:07 +0000)]
Merge "Add capability for sensor hal to use wakelock" into oc-mr1-dev

6 years agoAdd vts audio dec test for flac decoder
Ram Mohan M [Fri, 15 Sep 2017 14:18:17 +0000 (19:48 +0530)]
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

6 years agoAdd support for preset ANW buffer mode
Ram Mohan M [Thu, 14 Sep 2017 08:12:12 +0000 (13:42 +0530)]
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

6 years agoExtend adaptive playback test for graphic buffer modes
Ram Mohan M [Thu, 14 Sep 2017 06:57:44 +0000 (12:27 +0530)]
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

6 years agoMove graphic buffer allocation to common
Ram Mohan M [Thu, 14 Sep 2017 06:34:48 +0000 (12:04 +0530)]
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

6 years agoEnable EOSTest_M test
Ram Mohan M [Wed, 13 Sep 2017 09:58:46 +0000 (15:28 +0530)]
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

6 years agoAdd adaptive playback test
Ram Mohan M [Wed, 28 Jun 2017 12:50:03 +0000 (18:20 +0530)]
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

6 years agoAdd flush test
Ram Mohan M [Wed, 23 Aug 2017 12:38:27 +0000 (18:08 +0530)]
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

6 years agoSignal eof flag for each input buffer
Ram Mohan M [Thu, 24 Aug 2017 11:33:56 +0000 (17:03 +0530)]
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

6 years agoFix typos in IOmxStore.hal
Pawin Vongmasa [Sat, 16 Sep 2017 13:44:58 +0000 (06:44 -0700)]
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

6 years agoCamera: Add default implementation of camera.device@3.3
Eino-Ville Talvala [Tue, 22 Aug 2017 23:15:09 +0000 (16:15 -0700)]
Camera: Add default implementation of camera.device@3.3

Inherit as much as possible from camera.device@3.2

- Refactor CameraDeviceSession@3.2 implementation to separate out the
  HIDL session interface from the main implementation object. This
  avoids multiple inheritance issues
- Create CameraDeviceSession@3.3 with support for the new
  overrideDataspace field
- Add virtual factory method for CameraDevice to create the right version
  of Session.
- Create CameraDevice@3.3, which overrides createSession to return a
  CameraDeviceSession@3.3.
- Add system property to override selection of which minor HIDL
  version is used for legal HAL version 3.x; set the default to the
  newest available minor version.

Test: Camera CTS passes on device using @3.3.
Bug: 62358514
Change-Id: I497e4bc0de798b56ecdb2ea6467b79afccaf89f7

6 years agoMerge "HAL interface libs for testing are built w/o VNDK." into oc-mr1-dev
TreeHugger Robot [Fri, 15 Sep 2017 04:11:30 +0000 (04:11 +0000)]
Merge "HAL interface libs for testing are built w/o VNDK." into oc-mr1-dev

6 years agoMerge "[AWARE] Developer documentation on HAL usage" into oc-mr1-dev
TreeHugger Robot [Fri, 15 Sep 2017 01:08:27 +0000 (01:08 +0000)]
Merge "[AWARE] Developer documentation on HAL usage" into oc-mr1-dev

6 years agoHAL interface libs for testing are built w/o VNDK.
Jiyong Park [Tue, 29 Aug 2017 05:20:59 +0000 (14:20 +0900)]
HAL interface libs for testing are built w/o VNDK.

The HAL interfaces for testing were copied to /system/lib[64]/vndk
directory since they are built as cc_library and thus falsely recognized
as members of VNDK, which isn't true.

Build them as non-VNDK so that they are filtered out. To do so,
.hidl_for_test files are added for the test HALs and update-makefiles.sh
were ran.

Bug: 64776708
Test: no android.hardware.tests.*.so file in /system/lib[64]/vndk on
2017 pixel devices

Change-Id: I6167da8572081adfb53f432b248ac3bbee39dc42

6 years agoMerge "Revert "HAL interface libs for testing are now built as test lib"" into oc...
TreeHugger Robot [Thu, 14 Sep 2017 22:49:53 +0000 (22:49 +0000)]
Merge "Revert "HAL interface libs for testing are now built as test lib"" into oc-mr1-dev

6 years ago[AWARE] Developer documentation on HAL usage
Etan Cohen [Wed, 13 Sep 2017 23:40:49 +0000 (16:40 -0700)]
[AWARE] Developer documentation on HAL usage

Add documentation specifying which HAL APIs are used by the
framework.

Bug: 65646388
Test: N/A
Change-Id: Id1407a2d206b8eab47f5753a7cc6713b9581aeff

6 years agoMerge "Camera: synchronize access to mMemoryMap" into oc-mr1-dev
TreeHugger Robot [Thu, 14 Sep 2017 21:50:21 +0000 (21:50 +0000)]
Merge "Camera: synchronize access to mMemoryMap" into oc-mr1-dev

6 years agoAdd capability for sensor hal to use wakelock
Peng Xu [Tue, 12 Sep 2017 21:52:22 +0000 (14:52 -0700)]
Add capability for sensor hal to use wakelock

Bug: 63995095
Test: no more wake lock acquire failures in QCOM and nanohub sensor
      hal
Change-Id: I2a5c2e29ecb66869a408f53998b6074ed07b493a

6 years agoRevert "HAL interface libs for testing are now built as test lib"
Yifan Hong [Wed, 13 Sep 2017 00:53:29 +0000 (00:53 +0000)]
Revert "HAL interface libs for testing are now built as test lib"

This reverts commit 9a4b17e3e1ecb0dc0cd515d7445b7f200fc4b11b.

Reason of revert: this breaks hidl_test / lshal_test

Bug: 65603915

Change-Id: I73427657615ab9c7e3c0c7d336f03510d9a53fe6

6 years agoAudio VTS: do not test duplicate policy configuration file am: ff7dcd7e4d
Kevin Rocard [Thu, 14 Sep 2017 16:52:54 +0000 (16:52 +0000)]
Audio VTS: do not test duplicate policy configuration file am: ff7dcd7e4d
am: ecf7848f67

Change-Id: I79f97fb002e036c17b7ab42f1af00ed2b17e60fd

6 years agoAudio VTS: do not test duplicate policy configuration file
Kevin Rocard [Thu, 14 Sep 2017 16:48:56 +0000 (16:48 +0000)]
Audio VTS: do not test duplicate policy configuration file
am: ff7dcd7e4d

Change-Id: Ice54f75c307f9f25e210b1382554dc24162b9ace

6 years agoAudio VTS: do not test duplicate policy configuration file
Kevin Rocard [Thu, 14 Sep 2017 01:11:14 +0000 (18:11 -0700)]
Audio VTS: do not test duplicate policy configuration file

This patch only modifies the VTS tests.

O generic policy configuration fallback is incorrect
(see I2a39606cd820b5cd27a640248bf692204ec2cd71 for the fix).
Nevertheless, has no impact on products because this fallback
is only used in aosp builds and GSI.

Unfortunatly, this break the VTS test when run agains GSI.
Because it is too late to change GSI for Oc and DR,
relax the test to support current O GSI.

For P, ihis patch will not be part of VTS
and the fallback config will be fixed.

Bug: 65535343
Merged-In: I2a39606cd820b5cd27a640248bf692204ec2cd71
Test: Following script:

setup() {
    BUILD_ID=$1
    TARGET_PRODUCT=$2
    TARGET_BUILD_TYPE=${3:-userdebug}
    TARGET_ARCH=${4:-arm64}
    set +x

    echo "The following three variables can be overridden" \
         "by setting the corresponding environment variable"
    set -x
    TEST_SUITES_TARGET=${VTS_TEST_SUITES_TARGET:-test_suites_${TARGET_ARCH}}
    BASE_TARGET=${VTS_BASE_TARGET:-${TARGET_PRODUCT}-${TARGET_BUILD_TYPE}}
    AOSP_TARGET=${VTS_AOSP_TARGET:-aosp_${TARGET_ARCH}_ab-${TARGET_BUILD_TYPE}}
    set +x

    download() {
        local target=$1
        local file=$2
        test -f $file ||
            /google/data/ro/projects/android/fetch_artifact --bid $BUILD_ID --target $target $file ||
            { local r=$?; rm $file; return $r; }
    }

    echo "Downloading"
    download $TEST_SUITES_TARGET 'android-vts.zip'
    download $BASE_TARGET "${TARGET_PRODUCT}-img-${BUILD_ID}.zip"
    download $AOSP_TARGET "aosp_${TARGET_ARCH}_ab-img-${BUILD_ID}.zip"

    echo "Unzipping"
    rm -r system.img android-vts
    unzip aosp_${TARGET_ARCH}_ab-img-$BUILD_ID.zip system.img
    unzip android-vts.zip

    echo "Building vbmeta without verity"
    avbtool make_vbmeta_image --flag 2 --output vbmeta.img

    set -x
    adb reboot bootloader
    fastboot update ${TARGET_PRODUCT}-img-$BUILD_ID.zip --skip-reboot
    fastboot flash vbmeta vbmeta.img || echo "Warning: Device does not support vbmeta"
    fastboot erase system
    fastboot flash system system.img
    fastboot erase metadata
    fastboot -w
    fastboot reboot

    echo "Board setup"
    echo "You may now start vts-tradefed with:"
    echo '$ ANDROID_BUILD_TOP= PATH="$PWD:$PATH" vts-tradefed'
}
VTS_TEST_SUITES_TARGET=test_suites_arm64_fastbuild3d_linux setup 4327646 sailfish
 vts-tradefed run commandAndExit vts --skip-all-system-status-check --primary-abi-only --skip-preconditions --module VtsHalAudioV2_0Target -t CheckConfig.audioPolicyConfigurationValidation

Change-Id: Ibe1feed41325a3e4fb25e959bd254156348aa584
Signed-off-by: Kevin Rocard <krocard@google.com>
6 years agoSupport ignored output in test harness
I-Jui (Ray) Sung [Thu, 14 Sep 2017 01:47:12 +0000 (18:47 -0700)]
Support ignored output in test harness

Filtered out outputs that are not needed for correctness verification
during result validation.

Bug: 63905942
Test: VtsHalNeuralnetworksV1_0TargetTest on Angler
Change-Id: Ic967a821c2a60194c9d9f386601a865f5c409a4f

6 years agoMerge "Add documentation to the Neuralnetworks HIDL HAL." into oc-mr1-dev
Miao Wang [Thu, 14 Sep 2017 02:26:37 +0000 (02:26 +0000)]
Merge "Add documentation to the Neuralnetworks HIDL HAL." into oc-mr1-dev

6 years agoAdd documentation to the Neuralnetworks HIDL HAL.
Michael Butler [Mon, 11 Sep 2017 23:03:36 +0000 (16:03 -0700)]
Add documentation to the Neuralnetworks HIDL HAL.

Bug: 63905942
Test: not needed
Change-Id: I1582d20e66ef939581d82f21088f0db0bc0f5eb7

6 years agoMerge "Relax tolerance for FP comparison in test harness" into oc-mr1-dev
Ray Sung [Thu, 14 Sep 2017 00:53:27 +0000 (00:53 +0000)]
Merge "Relax tolerance for FP comparison in test harness" into oc-mr1-dev

6 years agoRelax tolerance for FP comparison in test harness
I-Jui (Ray) Sung [Wed, 13 Sep 2017 23:47:50 +0000 (16:47 -0700)]
Relax tolerance for FP comparison in test harness

Tests like RNN needs slightly more relaxed tolerance than 4ULP.
Use 1e-5 from corresponding TFLite tests.

Bug: 63905942
Test: VtsHalNeuralnetworksV1_0TargetTest on Angler
Change-Id: I721711e13575f56e956d87e3ad537467f85e93f4

6 years agoMake the enums signed int, remove unused info.
Miao Wang [Wed, 13 Sep 2017 22:51:54 +0000 (15:51 -0700)]
Make the enums signed int, remove unused info.

Bug: 63905942
Test: NeuralNetworksTest pass
Change-Id: I40ca1120d05ea8900f4f4e49409d3096f46ec0e1

6 years agoMerge "Move OEM codes out of NeuralNetworks.h to new file NeuralNetworksOEM.h." into...
David Gross [Wed, 13 Sep 2017 22:41:28 +0000 (22:41 +0000)]
Merge "Move OEM codes out of NeuralNetworks.h to new file NeuralNetworksOEM.h." into oc-mr1-dev

6 years agoMerge "bug fix: add missing color formats to the check for valid formats" into oc...
TreeHugger Robot [Wed, 13 Sep 2017 22:12:19 +0000 (22:12 +0000)]
Merge "bug fix: add missing color formats to the check for valid formats" into oc-mr1-dev

6 years agoMerge "Allow clock_gettime syscall for logging" into oc-mr1-dev
TreeHugger Robot [Wed, 13 Sep 2017 20:32:21 +0000 (20:32 +0000)]
Merge "Allow clock_gettime syscall for logging" into oc-mr1-dev

6 years agoMove OEM codes out of NeuralNetworks.h to new file NeuralNetworksOEM.h.
David Gross [Wed, 13 Sep 2017 19:31:33 +0000 (12:31 -0700)]
Move OEM codes out of NeuralNetworks.h to new file NeuralNetworksOEM.h.

Also remove FAKE_QUANT operation.

Bug: 63905942
Test: nn/runtime/tests, vts

Change-Id: Icfb5e7dbb9c2cca6e719ec2ab6344dbe5d95c86b

6 years agoMerge "Adjust AM/FM region ranges." into oc-mr1-dev
Tomasz Wasilczyk [Wed, 13 Sep 2017 15:36:09 +0000 (15:36 +0000)]
Merge "Adjust AM/FM region ranges." into oc-mr1-dev

6 years agoAllow clock_gettime syscall for logging
Jeff Vander Stoep [Wed, 13 Sep 2017 14:54:45 +0000 (07:54 -0700)]
Allow clock_gettime syscall for logging

Addresses:
__kernel_clock_gettime" access failed.

The call flow is as follow:
"configstore -> hidltransport.tryShortenProcessName -> ALOGI ->
__android_log_print -> logdOpen -> netdClientConnect -> clock_gettime".

Bug: 65611466
Test: build and boot sailfish
Change-Id: I9e12c3a0969a7fe67c017d70f39a85792cf10bae

6 years agoMove android.hardware.tests.libhwbinder@1.0-impl to system.img
Zhuoyao Zhang [Wed, 16 Aug 2017 17:21:09 +0000 (10:21 -0700)]
Move android.hardware.tests.libhwbinder@1.0-impl to system.img

Bug: 64723866
Test: make vts
      vts-tradefed run vts-performance

Merged-In: Ib5e993d98f7fa12dcb2b418a995887b59394c849
Change-Id: Ib5e993d98f7fa12dcb2b418a995887b59394c849

6 years agoTest harness for generated tests
I-Jui (Ray) Sung [Wed, 6 Sep 2017 09:15:54 +0000 (02:15 -0700)]
Test harness for generated tests

Created initial test harness for test models and examples from
NNAPI test generator in VtsHalNeuralnetworksV1_0TargetTest. As
an example, also added a test generated from test spec at
frameworks/ml/nn/tools/test_generator/tests/P_vts_full/.

Generated model setup code and examples are from:
frameworks/ml/nn/runtime/test/generated/examples and
frameworks/ml/nn/runtime/test/generated/vts_models respectively.

Bug: 63905942
Bug: 63525563
Test: VtsHalNeuralnetworksV1_0TargetTest with sample driver enabled
      by cherry-pick

Change-Id: Ief029eed9718c8724ef0b64fc6a7f6b9a7bc7b7b

6 years agoMerge "Neuralnetworks HAL cleanup -- hardware/interfaces" into oc-mr1-dev
Michael Butler [Wed, 13 Sep 2017 07:29:43 +0000 (07:29 +0000)]
Merge "Neuralnetworks HAL cleanup -- hardware/interfaces" into oc-mr1-dev

6 years agoNeuralnetworks HAL cleanup -- hardware/interfaces
Michael Butler [Tue, 12 Sep 2017 03:27:50 +0000 (20:27 -0700)]
Neuralnetworks HAL cleanup -- hardware/interfaces

Does three primary cleanups:
1) Removes unused OperandTypes, creates OEM types
2) Creates explicit ErrorStatus; functions return status
3) IDevice::initialize renamed to getCapabilities,
   IDevice::getSupportedSubgraph renamed to getSupportedOperations

Additionally fixes the corresponding VTS tests.

Bug: 63905942
Test: mm, vts
Change-Id: Ib4c61b9b13963ac2367f21dc3c20e5946eb955a9

6 years agoCamera: synchronize access to mMemoryMap
Yin-Chia Yeh [Tue, 12 Sep 2017 23:35:29 +0000 (16:35 -0700)]
Camera: synchronize access to mMemoryMap

Test: stress test on partner device
Bug: 64831570
Change-Id: Ib9ece2f876ccb309c4f1bb8f90242a3e91081f47

6 years agoMerge "Fix docstrings for RIL_CellIdentity" into oc-mr1-dev
Nathan Harold [Tue, 12 Sep 2017 22:51:10 +0000 (22:51 +0000)]
Merge "Fix docstrings for RIL_CellIdentity" into oc-mr1-dev

6 years agoAdjust AM/FM region ranges.
Tomasz Wasilczyk [Tue, 12 Sep 2017 18:36:06 +0000 (11:36 -0700)]
Adjust AM/FM region ranges.

Bug: 64541927
Test: instrumentation
Change-Id: I34ea5b0b426f0c0c73d1cbc2d64cb67e2d4b7bd6

6 years agoMerge "Make prepareModel asynchronous." into oc-mr1-dev
Michael Butler [Tue, 12 Sep 2017 18:41:00 +0000 (18:41 +0000)]
Merge "Make prepareModel asynchronous." into oc-mr1-dev

6 years agoFix docstrings for RIL_CellIdentity
Nathan Harold [Wed, 30 Aug 2017 03:41:36 +0000 (20:41 -0700)]
Fix docstrings for RIL_CellIdentity

Update the documentation to match the current
CTS requirements for Cell Identity reporting
-For LTE, WCDMA, and GSM require that the channel
 number be reported.
-For LTE and WCDMA, require that the correct
 physical channel identifier be reported. Note that
 due to b/32774471 we still cannot require BSIC
 to be reported.

Bug: 65174915
Test: none, comment-only change
Change-Id: I93fcce3b39401e8965f724722361c2cadc4a88cc

6 years agoMerge "Remove default implementation for PowerHal 1.1" into oc-mr1-dev
TreeHugger Robot [Tue, 12 Sep 2017 16:51:33 +0000 (16:51 +0000)]
Merge "Remove default implementation for PowerHal 1.1" into oc-mr1-dev

6 years agoMerge "Don't send more than 2K to addRngEntropy" into oc-dev am: 7eeb6b5079
Shawn Willden [Tue, 12 Sep 2017 16:34:52 +0000 (16:34 +0000)]
Merge "Don't send more than 2K to addRngEntropy" into oc-dev am: 7eeb6b5079
am: 35e1be70b4

Change-Id: I626faecd254e3e543308143cfe69d1393e1f9267

6 years agoMerge "Don't send more than 2K to addRngEntropy" into oc-dev
Shawn Willden [Tue, 12 Sep 2017 16:31:32 +0000 (16:31 +0000)]
Merge "Don't send more than 2K to addRngEntropy" into oc-dev
am: 7eeb6b5079

Change-Id: Id0ee0e3f22ee4f2d2b730054043d28c01ac19c6f

6 years agoMerge "Don't send more than 2K to addRngEntropy" into oc-dev
TreeHugger Robot [Tue, 12 Sep 2017 16:28:44 +0000 (16:28 +0000)]
Merge "Don't send more than 2K to addRngEntropy" into oc-dev

6 years agoRemove default implementation for PowerHal 1.1
Ahmed ElArabawy [Fri, 8 Sep 2017 16:11:48 +0000 (09:11 -0700)]
Remove default implementation for PowerHal 1.1

Default implementation in powerHAL 1.1 is not used by any device
Devices using powerHal 1.1 have their own implementation in their
device/<vendor>/<devicename> while older devices don't use 1.1 at all.
This CL removes the default implementation for the PowerHAL 1.1

Bug: 65453628
Test: Build successfully (code not used anyway)
Merged-In: I695348683c6e2e504a523897082543274dc239aa

Change-Id: I695348683c6e2e504a523897082543274dc239aa
Signed-off-by: Ahmed ElArabawy <arabawy@google.com>
6 years agoMerge "Reduce max keymaster message size to 2K" into oc-dev am: 42d61ce03a
Shawn Willden [Tue, 12 Sep 2017 06:53:16 +0000 (06:53 +0000)]
Merge "Reduce max keymaster message size to 2K" into oc-dev am: 42d61ce03a
am: ac06f7576e

Change-Id: I219c56653ec9051d35a4c7c8f012c9c06e555881

6 years agoMerge "Reduce max keymaster message size to 2K" into oc-dev
Shawn Willden [Tue, 12 Sep 2017 06:50:47 +0000 (06:50 +0000)]
Merge "Reduce max keymaster message size to 2K" into oc-dev
am: 42d61ce03a

Change-Id: Ia2915bb4305268e5048689ef3e4cf6cf9793d483

6 years agoMerge "Reduce max keymaster message size to 2K" into oc-dev
TreeHugger Robot [Tue, 12 Sep 2017 06:46:22 +0000 (06:46 +0000)]
Merge "Reduce max keymaster message size to 2K" into oc-dev

6 years agoMake prepareModel asynchronous.
Michael Butler [Mon, 11 Sep 2017 20:53:19 +0000 (13:53 -0700)]
Make prepareModel asynchronous.

Updated the corresponding VTS tests.

Bug: 63905942
Test: mm, vts
Change-Id: Ie520af845f491dbdd93ea66fd643d77cfb4703f7

6 years agoMerge "Fix bug in the way an Event handles a bound thread." into oc-mr1-dev
David Gross [Tue, 12 Sep 2017 04:02:53 +0000 (04:02 +0000)]
Merge "Fix bug in the way an Event handles a bound thread." into oc-mr1-dev