OSDN Git Service

android-x86/hardware-interfaces.git
6 years agoFix VTS test which turns off radio.
Sanket Padawe [Wed, 18 Oct 2017 23:56:12 +0000 (23:56 +0000)]
Fix VTS test which turns off radio.
am: 536818d17a

Change-Id: I74ebfa3894e27bdf0e02671cbeccac83b1917424

6 years agoFix VTS test which turns off radio.
Sanket Padawe [Mon, 4 Sep 2017 05:32:43 +0000 (22:32 -0700)]
Fix VTS test which turns off radio.

Since the purpose of most of the test cases in current VTS is to check
if proper errors are returned and there is no crash seen in vendor code,
updating setRadioPower test case to turn on the radio instead of
turning off. We want to avoid test cases which turn off
radio or leads to modem shut down as those test cases affect other tests.

Test: VTS
Change-Id: I4fb9f18884f7ef21162015a0032c4431444f7025
Merged-In: I4fb9f18884f7ef21162015a0032c4431444f7025
Bug: 65230472
(cherry picked from commit 9a721b808743069fc26d6cf226595e2823e347a6)

6 years agoEnlarge buffer size for direct report testing
Peng Xu [Thu, 12 Oct 2017 17:04:03 +0000 (17:04 +0000)]
Enlarge buffer size for direct report testing
am: 3c01fef56c

Change-Id: I9fce07b88a1ddb12a91554b823d9290c2571eaef

6 years agoEnlarge buffer size for direct report testing
Peng Xu [Wed, 11 Oct 2017 01:19:18 +0000 (18:19 -0700)]
Enlarge buffer size for direct report testing

This change is part of "Sensor direct report test - gralloc support".

Bug: 67439052
Test: vts test no longer fail on sailfish device
Change-Id: Id67a2a181b2d8847e2b1b9dbe46f5186e97e8563
Merged-In: Ibe1d076c24dc2cfe61dfd19aa5055c9075aa9e14

6 years agoAudio VTS: Allow 4GB buffer prepareToWrite/Read to succeed on 64 bits
Mikhail Naganov [Wed, 11 Oct 2017 01:33:25 +0000 (01:33 +0000)]
Audio VTS: Allow 4GB buffer prepareToWrite/Read to succeed on 64 bits
am: ce607879ba

Change-Id: I6869d49ab5ceb12e1b0d3e50e0952094c34b3c07

6 years agoAudio VTS: Allow 4GB buffer prepareToWrite/Read to succeed on 64 bits
Mikhail Naganov [Tue, 10 Oct 2017 17:31:07 +0000 (10:31 -0700)]
Audio VTS: Allow 4GB buffer prepareToWrite/Read to succeed on 64 bits

Because on oc-dev the runtime image can not be modified and that this is
not a bug but a test and implementation misalignment,
allow prepareForWrite/Read(UINT32_MAX, UINT32_MAX) to succeed on oc-dev.

MR1 will have a better fix where the HAL will always reject > 1GiB
buffers independently of the architecture.

Test: vts-tradefed run commandAndExit vts --module VtsHalAudioV2_0Target
Bug: 67030516
Change-Id: I10a5ee3868c67c45ed5297d9c5c89129d762c7cd
Merged-In: I4cc3efda9bb66e6dae8b4e6785f52d9e51440aee
Signed-off-by: Kevin Rocard <krocard@google.com>
6 years agoVTS: fix VtsHalKeymasterV3_0Target issue
Iris Chang [Tue, 10 Oct 2017 16:22:34 +0000 (16:22 +0000)]
VTS: fix VtsHalKeymasterV3_0Target issue
am: 926a8e7440

Change-Id: I762f63b3f239169d93b33f7164d4eb3f9bd8f3d9

6 years agoVTS: fix VtsHalKeymasterV3_0Target issue
Iris Chang [Thu, 14 Sep 2017 07:23:18 +0000 (15:23 +0800)]
VTS: fix VtsHalKeymasterV3_0Target issue

Failed cases:
AttestationTest.RsaAttestation
AttestationTest.EcAttestation

Analysis:
The verify_attestation_record() in Keymaster_hidl_hal_test.cpp calls
parse_attestation_record() to set the value of att_challenge. It fails
to compare att_challenge with challenge by memcmp.
Because setToExternal() method uses buffer pointer to local variable
(record), not use memcpy to copy into itself buffer in
parse_attestation_record(). When it leaves the parse_attestation_record(),
we will get the att_challenge which is null buffer to compare with challenge
incorrectly.

Fix: use memcpy to copy the buffer.

Bug: 65039571
Test: build passed. VtsHalKeymasterV3_0Target -> PASSED: 106, FAILED: 0.
Change-Id: I700a9242cc9a5f4cb196b62860823601e4088531
Merged-In: I700a9242cc9a5f4cb196b62860823601e4088531
(cherry picked from 54ca32a130a0450b444848046199d62876b55301)

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 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 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 agoAudio VTS: remove expectation of policy configuration unicity
Kevin Rocard [Fri, 8 Sep 2017 22:12:14 +0000 (22:12 +0000)]
Audio VTS: remove expectation of policy configuration unicity
am: 76efe01bbd

Change-Id: I23948718eaed0a687cc7dbd6b4940fc8514fbe1b

6 years agoAudio VTS: remove expectation of policy configuration unicity
Kevin Rocard [Fri, 8 Sep 2017 18:29:32 +0000 (11:29 -0700)]
Audio VTS: remove expectation of policy configuration unicity

The XML audio policy configuration is expected to be located in /vendor.
Additionally for AOSP builds (no vendor) and as a fallback in case of
missing vendor configuration, the system supports loading the XML from
/system.
This fallback config is not guaranty to be compatible with the phone HAL,
but probably does for a wide range of android phones
(at least for basic use cases).

In each of those cases, there is only one configuration file, either in
/vendor or /system.
As a result, the audio policy configuration VTS test was asserting that
only one audio_policy_configuration.xml could be found.
If multiple were found, one was useless and the platform integrator
should remove it.

But VTS are now run on phone with a /system partition from an AOSP
build and a /vendor partition from a device build.
In such scenario two audio policy config are now present on the device,
thus breaking the VTS expectation.

This is not a problem for the framework because it does not look for a /system
config if one was found in /vendor.

In the future, the /system XML might be removed as boards
specific config should be found in /vendor only as per treble
requirements.

Test: Compile
Bug: 65482614
Change-Id: I7cd7eb7284c0aa52782aa591411f8a9bf2a87a76
Signed-off-by: Kevin Rocard <krocard@google.com>
6 years agoMerge "bug fix: handle multiple port settings change events signalled at once" into...
Ram Mohan M [Thu, 7 Sep 2017 23:40:43 +0000 (23:40 +0000)]
Merge "bug fix: handle multiple port settings change events signalled at once" into oc-dev
am: f8677dcfdb

Change-Id: If65c243e72e890660fc4481b375b23ba109ae577

6 years agoMerge "bug fix: handle multiple port settings change events signalled at once" into...
TreeHugger Robot [Thu, 7 Sep 2017 23:30:17 +0000 (23:30 +0000)]
Merge "bug fix: handle multiple port settings change events signalled at once" into oc-dev

6 years agoMerge "graphics: ignore/reduce spurious vsync in VTS" into oc-dev
Chia-I Wu [Thu, 7 Sep 2017 17:36:15 +0000 (17:36 +0000)]
Merge "graphics: ignore/reduce spurious vsync in VTS" into oc-dev
am: ef3389add5  -s ours

Change-Id: I9cee260396d3f24e86443afe36780894397638e1

6 years agoMerge "graphics: ignore/reduce spurious vsync in VTS" into oc-dev
Chia-I Wu [Thu, 7 Sep 2017 17:18:25 +0000 (17:18 +0000)]
Merge "graphics: ignore/reduce spurious vsync in VTS" into oc-dev

6 years agoAllow partially dynamic profile in the audio policy config
Kevin Rocard [Wed, 6 Sep 2017 23:35:24 +0000 (23:35 +0000)]
Allow partially dynamic profile in the audio policy config
am: 3092b31b14

Change-Id: Ibafc8f571bdc32353bd974a7fce6e3fd84238711

6 years agoAllow partially dynamic profile in the audio policy config
Kevin Rocard [Thu, 31 Aug 2017 23:45:08 +0000 (16:45 -0700)]
Allow partially dynamic profile in the audio policy config

The audio policy configuration deserializer supports fully dynamic
profiles by not setting a profile in a port.
It also supports partially dynamic profile by
not setting @format, @samplingRates or @channelMasks

Nevertheless this last feature of partial dynamic profile
was mistakenly not allowed by the configuration xsd.

Bug: 63541622
Change-Id: I78f77a77a3c62bba28a7da5a0e3c8c608b5a01d9
Merged-In: I78f77a77a3c62bba28a7da5a0e3c8c608b5a01d9
Signed-off-by: Kevin Rocard <krocard@google.com>
(cherry picked from commit 43d25f2f26eee5655214cf5426a629916aece061)

6 years agoMerge "bug fix: restore support for broken flag" into oc-dev
Ram Mohan M [Sun, 3 Sep 2017 12:55:06 +0000 (12:55 +0000)]
Merge "bug fix: restore support for broken flag" into oc-dev
am: 40aad4e988

Change-Id: Ied23d04f5c38f81ad732424d3577edf4d3cc085d

6 years agoMerge "bug fix: restore support for broken flag" into oc-dev
TreeHugger Robot [Sun, 3 Sep 2017 12:42:39 +0000 (12:42 +0000)]
Merge "bug fix: restore support for broken flag" into oc-dev

6 years agobug fix: restore support for broken flag
Ram Mohan M [Wed, 30 Aug 2017 06:26:33 +0000 (11:56 +0530)]
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

6 years agoMerge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev
Ram Mohan M [Sat, 2 Sep 2017 03:44:45 +0000 (03:44 +0000)]
Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev
am: febad058bd

Change-Id: I09128311ab40316b17d99daeae375ff7535d0e63

6 years agoMerge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev
TreeHugger Robot [Sat, 2 Sep 2017 03:40:15 +0000 (03:40 +0000)]
Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev

6 years agographics: ignore/reduce spurious vsync in VTS
Chia-I Wu [Mon, 24 Jul 2017 18:11:01 +0000 (11:11 -0700)]
graphics: ignore/reduce spurious vsync in VTS

Bug: 62802718
Test: VTS
Change-Id: I1ed73e0b2f14414f540865a96e6bd5ac1599fbf3
(cherry picked from commit e0ff3e4a2679cb505e801db5df4d1546f6caa41a)

6 years agobug fix: handle multiple port settings change events signalled at once
Ram Mohan M [Tue, 29 Aug 2017 08:38:47 +0000 (14:08 +0530)]
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

6 years agobug fix: disable timestamp deviation test for audio decoder
Ram Mohan M [Fri, 1 Sep 2017 08:28:40 +0000 (13:58 +0530)]
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

6 years agoDon't send more than 2K to addRngEntropy
Shawn Willden [Fri, 11 Aug 2017 20:33:44 +0000 (14:33 -0600)]
Don't send more than 2K to addRngEntropy

Bug: 63745893
Test: VtsHalKeymasterV3_0TargetTest
Change-Id: I136920d3b62e026b22fbe06594bc40ccddc57dc3
(cherry picked from commit 3e1267edf0621185043482a805fcec0fa2a63719)

6 years agoMerge "Audio VTS: Look for Audio policy config in all supported folders" into oc-dev
Kevin Rocard [Tue, 29 Aug 2017 00:34:29 +0000 (00:34 +0000)]
Merge "Audio VTS: Look for Audio policy config in all supported folders" into oc-dev
am: 4f3c28ad7a

Change-Id: I43a48595607a1c38475aec81232008ac76cdd5b1

6 years agoMerge "Audio VTS: Look for Audio policy config in all supported folders" into oc-dev
TreeHugger Robot [Tue, 29 Aug 2017 00:19:03 +0000 (00:19 +0000)]
Merge "Audio VTS: Look for Audio policy config in all supported folders" into oc-dev

6 years agoMerge "Sensor VTS testBatchingOperation flakiness fix" into oc-dev
Peng Xu [Fri, 25 Aug 2017 22:21:41 +0000 (22:21 +0000)]
Merge "Sensor VTS testBatchingOperation flakiness fix" into oc-dev
am: 751953859c

Change-Id: I8800d11d6b137488674f3159941135bce374d3a6

6 years agoMerge "Sensor VTS testBatchingOperation flakiness fix" into oc-dev
TreeHugger Robot [Fri, 25 Aug 2017 22:12:24 +0000 (22:12 +0000)]
Merge "Sensor VTS testBatchingOperation flakiness fix" into oc-dev

6 years agoMerge "bug fix: output colorformat configuration fixed" into oc-dev
Ram Mohan M [Fri, 25 Aug 2017 05:16:14 +0000 (05:16 +0000)]
Merge "bug fix: output colorformat configuration fixed" into oc-dev
am: 1e15657ca1

Change-Id: I1bf3c443058ab57baba7c51e3f9f0ae662e61882

6 years agoMerge "bug fix: output colorformat configuration fixed" into oc-dev
TreeHugger Robot [Fri, 25 Aug 2017 05:09:25 +0000 (05:09 +0000)]
Merge "bug fix: output colorformat configuration fixed" into oc-dev

6 years agoAudio VTS: Look for Audio policy config in all supported folders
Kevin Rocard [Tue, 22 Aug 2017 01:32:49 +0000 (18:32 -0700)]
Audio VTS: Look for Audio policy config in all supported folders

audio_policy_configuration.xml can be loaded from the following places:
 - /system/etc/
 - /vendor/etc
 - /odm/etc
Nevertheless the config validation test was expecting it to be in
/vendor/etc exclusively.

This patch changes the test logic to:
 - look for the config files in all 3 folders
 - make sure the config is unique

Test: run the following script to check for regressions and test that
      invalid config make the test fail.
  (
  set -xe
  runVTS() { vts-tradefed run commandAndExit vts \
        --skip-all-system-status-check --primary-abi-only \
        --skip-preconditions --module VtsHalAudioV2_0Target \
        -t CheckConfig.audioPolicyConfigurationValidation; }

  echo "# Test valid config"
  runVTS

  echo "# Test multiple invalid match"
  adb shell touch /system/etc/audio_policy_configuration.xml
  ! runVTS
  adb shell rm /system/etc/audio_policy_configuration.xml

  echo "# Test multiple valid match"
  adb shell cp /{vendor,system}/etc/audio_policy_configuration.xml
  ! runVTS
  adb shell rm /system/etc/audio_policy_configuration.xml

  echo "# Test invalid config"
  adb shell sed -i /defaultOutputDevice/p /vendor/etc/audio_policy_configuration.xml
  ! runVTS
  adb shell sed -i '"/defaultOutputDevice/{p;N;d}"' /vendor/etc/audio_policy_configuration.xml

  echo "# Test that the test did not break the config"
  runVTS
  )

Bug: 64881365
Change-Id: I9db5e6f727d19fd654a3cc543a2aaab196682001
Signed-off-by: Kevin Rocard <krocard@google.com>
6 years agoMerge "bug fix: close file pointer" into oc-dev
Ram Mohan M [Sat, 19 Aug 2017 01:45:20 +0000 (01:45 +0000)]
Merge "bug fix: close file pointer" into oc-dev
am: 8735fb45a5

Change-Id: I7ca0f4992804da30eaac7ca24b2dbe9004a5ac55

6 years agoMerge "bug fix: configure input port buffer size" into oc-dev
Ram Mohan M [Sat, 19 Aug 2017 01:44:50 +0000 (01:44 +0000)]
Merge "bug fix: configure input port buffer size" into oc-dev
am: bc769a7e5a

Change-Id: I3a9b89f4c64d1129ea59a0144937832d93ed81fd

6 years agoSensor VTS testBatchingOperation flakiness fix
Peng Xu [Sat, 19 Aug 2017 00:50:57 +0000 (17:50 -0700)]
Sensor VTS testBatchingOperation flakiness fix

Bug: 64732324
Bug: 63529148
Test: testBatchingOperation passes consistently
Change-Id: Id4e8075e249f3658f5683d53d6dc8403ee32bc4a

6 years agoMerge "bug fix: close file pointer" into oc-dev
TreeHugger Robot [Sat, 19 Aug 2017 00:30:10 +0000 (00:30 +0000)]
Merge "bug fix: close file pointer" into oc-dev

6 years agoMerge "bug fix: configure input port buffer size" into oc-dev
TreeHugger Robot [Sat, 19 Aug 2017 00:30:05 +0000 (00:30 +0000)]
Merge "bug fix: configure input port buffer size" into oc-dev

6 years agobug fix: output colorformat configuration fixed
Ram Mohan M [Tue, 4 Jul 2017 13:51:19 +0000 (19:21 +0530)]
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

6 years agoAllow clearkey tests to run if no vendor modules
Jeff Tinker [Fri, 18 Aug 2017 04:07:04 +0000 (04:07 +0000)]
Allow clearkey tests to run if no vendor modules
am: 5db2e67293

Change-Id: I594b8b5e9857d4a6c19d0847acf7c13949e13614

6 years agoAllow clearkey tests to run if no vendor modules
Jeff Tinker [Thu, 17 Aug 2017 17:09:32 +0000 (10:09 -0700)]
Allow clearkey tests to run if no vendor modules

DRM cannot skip all tests when vendor modules are
not present. Currently even the clearkey tests are
skipping when there is no vendor lib. This can be
replicated by just pushing the DRM VTS test binary
build from master (or DR or MR) and executing the
binary without VTS. All test cases skip (including
clearkey).

This change prevents exiting when no vendor
modules are found.

Test: Push the test binary and confirm that the
clearkey tests run.

bug:64459011
Change-Id: Ic5f554d9b223d6ca4bf26f92f072254cc18f4620

6 years agobug fix: close file pointer
Ram Mohan M [Wed, 9 Aug 2017 09:22:20 +0000 (14:52 +0530)]
bug fix: close file pointer

Bug: 63796949

Merged-In: Ifd8a27d851cbc659ea8108e0879747cd9169680a
Change-Id: Ifd8a27d851cbc659ea8108e0879747cd9169680a

6 years agobug fix: configure input port buffer size
Ram Mohan M [Wed, 9 Aug 2017 06:41:23 +0000 (12:11 +0530)]
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

6 years agoMerge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dr1-dev
TreeHugger Robot [Tue, 15 Aug 2017 20:44:58 +0000 (20:44 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dr1-dev

6 years agoMerge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev
Emilian Peev [Tue, 15 Aug 2017 20:27:50 +0000 (20:27 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev
am: e2dae8c258

Change-Id: I396a157f3522843c3d36870719cb3e5176754dcc

6 years agoMerge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev
TreeHugger Robot [Tue, 15 Aug 2017 20:17:11 +0000 (20:17 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev

6 years agoCamera: Switch 'cancelPictureFail' to 'cancelPictureNOP'
Emilian Peev [Fri, 11 Aug 2017 09:33:22 +0000 (10:33 +0100)]
Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO
Merged-In: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1

6 years agoMerge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev
Tomasz Wasilczyk [Fri, 11 Aug 2017 22:16:16 +0000 (22:16 +0000)]
Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev
am: 52af675d14

Change-Id: Ic5080cb6b22cd3984d0eb39d87cabf7c8a30178d

6 years agoMerge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev
Tomasz Wasilczyk [Fri, 11 Aug 2017 22:11:31 +0000 (22:11 +0000)]
Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev

6 years agoUpdate broadcastradio HAL 1.0 VTS tests.
Tomasz Wasilczyk [Tue, 8 Aug 2017 01:00:28 +0000 (18:00 -0700)]
Update broadcastradio HAL 1.0 VTS tests.

This is a cherry-pick of the following VTS changes from O MR1 to O:
 - ag/2462390: Fix Broadcast radio 1.0 VTS to not fail on
   other radio types than AM/FM.
 - ag/2477726: Make tuned field description more precise and
   fix VTS tests to obey it.
 - ag/2549733: Implement out-of-band metadata images (a test for in-band).

Additionally, it fixes config callback handling in OpenTunerTwice test.

Bug: b/64324703
Test: VTS
Change-Id: I590b9b4d927fb0cd41f92e919343b5b94ad0e246
Merged-In: I590b9b4d927fb0cd41f92e919343b5b94ad0e246

6 years agoCamera: Switch 'cancelPictureFail' to 'cancelPictureNOP'
Emilian Peev [Fri, 11 Aug 2017 09:33:22 +0000 (10:33 +0100)]
Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Merged-In: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO

Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1

6 years agoMerge "Bluetooth: Check the number of file descriptors" into oc-dr1-dev
TreeHugger Robot [Thu, 10 Aug 2017 03:05:38 +0000 (03:05 +0000)]
Merge "Bluetooth: Check the number of file descriptors" into oc-dr1-dev

6 years agoMerge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev
TreeHugger Robot [Thu, 10 Aug 2017 00:35:17 +0000 (00:35 +0000)]
Merge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev

6 years agoNFC: Add OWNERS file for default/
Myles Watson [Wed, 9 Aug 2017 18:27:40 +0000 (11:27 -0700)]
NFC: Add OWNERS file for default/

Test: build
Change-Id: I6e854923fdd66991bcc7ed3446106d148abea982
(cherry picked from commit 31439b1d24d49fcb431dfa48bdaa0b1d57deded4)

6 years agoBluetooth: Add OWNERS file for default/
Myles Watson [Wed, 9 Aug 2017 18:27:15 +0000 (11:27 -0700)]
Bluetooth: Add OWNERS file for default/

Test: build
Change-Id: Ib4cbd5502ac0791d4bc700f28de68d0548256de6
(cherry picked from commit 7c127dc4cbf90713bcdbce89fa166228c912847d)

6 years agoBluetooth: Check the number of file descriptors
Myles Watson [Tue, 8 Aug 2017 23:00:30 +0000 (16:00 -0700)]
Bluetooth: Check the number of file descriptors

Opening the serial port(s) returns -1 when it fails, check
that the number of file descriptors is greater than 0 and
less than the size of the array.

Test: VtsHalBluetoothV1_0TargetTest, Bluetooth start/stop
Bug: 64420262
Change-Id: Icbffa5bd8c02030e389bac03aa5e1e78cc3c6057
(cherry picked from commit 89316af23eb7aa300015a3d80bc97891cfe6031e)

6 years agoMerge "Skip direct report test if sensor is not available" into oc-dev
Peng Xu [Wed, 9 Aug 2017 17:04:32 +0000 (17:04 +0000)]
Merge "Skip direct report test if sensor is not available" into oc-dev
am: 63524f9fcc

Change-Id: Iffe683356ed925142ef877ecb5289463d8d4ac1e

6 years agoMerge "Skip direct report test if sensor is not available" into oc-dev
TreeHugger Robot [Wed, 9 Aug 2017 16:48:07 +0000 (16:48 +0000)]
Merge "Skip direct report test if sensor is not available" into oc-dev

6 years agowifi(vts): Remove a tight loop of toggling wifi state
Roshan Pius [Tue, 8 Aug 2017 15:43:50 +0000 (08:43 -0700)]
wifi(vts): Remove a tight loop of toggling wifi state

The tight loop of HAL start/stop exposes a race condition within the HAL.
Adding a fix for preventing this race would need a fix which would be
pretty risky at this point of the release. The issue itself is unlikely
to happen in real use cases because wifi toggles (user initiated or test
scripts initiated) via framework goes through a series of processing
before it invokes the HAL start/stop.

Bug: 64195190
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 VtsHalWifiV1_0Target -l
INFO`

Change-Id: I4e4d65f8b6e2e423a3a5f26e5a97a78b7e99c3e8

6 years agoMerge "Fix typo in in offload control VTS test" into oc-dr1-dev
TreeHugger Robot [Wed, 9 Aug 2017 01:59:19 +0000 (01:59 +0000)]
Merge "Fix typo in in offload control VTS test" into oc-dr1-dev

6 years agoFix typo in in offload control VTS test
Erik Kline [Tue, 8 Aug 2017 23:54:14 +0000 (16:54 -0700)]
Fix typo in in offload control VTS test

Test: built, flashed, run
Bug: 38220415
Bug: 64479437
Change-Id: I3aa4afa3b52297dd92e37b4f39a5bd25b129ba04

6 years agoMerge "Add the error code "OPERATION_NOT_ALLOWED" and log the error." into oc-dr1-dev
TreeHugger Robot [Tue, 8 Aug 2017 20:41:27 +0000 (20:41 +0000)]
Merge "Add the error code "OPERATION_NOT_ALLOWED" and log the error." into oc-dr1-dev

6 years agoAdd the error code "OPERATION_NOT_ALLOWED" and log the error.
yinxu [Mon, 7 Aug 2017 17:29:10 +0000 (10:29 -0700)]
Add the error code "OPERATION_NOT_ALLOWED" and log the error.

Test: Telephony sanity tests, run vts -m VtsHalRadioV1_1Target
Bug: 64426938
Change-Id: Id4f06842186b10a52640372c23bc2b368cc1e9ab

6 years agowifi(vts): Ignore IWifi.stop() returning ERROR_NOT_AVAILABLE
Roshan Pius [Thu, 3 Aug 2017 22:05:05 +0000 (15:05 -0700)]
wifi(vts): Ignore IWifi.stop() returning ERROR_NOT_AVAILABLE

When IWifi.stop() is invoked back to back (happens in the ConfigureChip
vts test), the HAL would return ERROR_NOT_AVAILABLE if the previous stop
is still being processed. This is not an error that needs to fail the test,
but a legitimate status for stop. We have a retry mechanism to handle
this in both the VTS test and framework for the case where IWifi.start()
is invoked while the previous stop is being processed.

While there, corrected a few log messages emitted by the HAL to debug
such startup/stop issues better.

Bug: 63971806
Test: `vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module VtsHalWifiV1_0Target -l
INFO`

Change-Id: I5e3470ac97541a6ea10aceec9b737e5d03ed5206

6 years agoMerge "Refactor and expand tetheroffload Control and Config VTS" into oc-dr1-dev
TreeHugger Robot [Tue, 8 Aug 2017 03:57:48 +0000 (03:57 +0000)]
Merge "Refactor and expand tetheroffload Control and Config VTS" into oc-dr1-dev

6 years agoMerge "Tetheroffload Control and Config VTS" into oc-dr1-dev
TreeHugger Robot [Tue, 8 Aug 2017 03:57:43 +0000 (03:57 +0000)]
Merge "Tetheroffload Control and Config VTS" into oc-dr1-dev

6 years agobug fix: handle OMX_EventBufferFlag events
Ram Mohan M [Tue, 8 Aug 2017 00:47:23 +0000 (00:47 +0000)]
bug fix: handle OMX_EventBufferFlag events
am: c8df8940e1

Change-Id: Ifa9493d869e42a76100011d5af20c5062bc44891

6 years agobug fix: handle OMX_EventBufferFlag events
Ram Mohan M [Thu, 22 Jun 2017 08:23:12 +0000 (13:53 +0530)]
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

6 years agoRefactor and expand tetheroffload Control and Config VTS
Erik Kline [Mon, 7 Aug 2017 18:32:45 +0000 (03:32 +0900)]
Refactor and expand tetheroffload Control and Config VTS

Test: as follows
    - built
    - flashed
    - booted
    - adb sync && adb shell ...$test
Bug: 38220415
Change-Id: Iab715c45ad0457f3eea43ee408fa947ffc01c4aa

6 years agoTetheroffload Control and Config VTS
Tyler Wear [Wed, 19 Jul 2017 20:48:57 +0000 (13:48 -0700)]
Tetheroffload Control and Config VTS

Basic VTS for Tetheroffload Config and Control  HALs.

Bug: 38220415
Change-Id: I6d0867e019aca58d8a6bd1404b8b85ed98274f11
CRs-fixed: 2083182

6 years agoSkip direct report test if sensor is not available
Peng Xu [Thu, 3 Aug 2017 21:53:35 +0000 (14:53 -0700)]
Skip direct report test if sensor is not available

Skip SensorsHidlTest.*AshmemDirectReportOperation* tests if sensor
is not available on device.

Bug: 64230704
Test: compiles and test pass for a pixel device with mag sensor
      masked in hal.

Change-Id: I9bfbea6301891eaa2b272bd3d15b0e237799b5d6

6 years agoReduce max keymaster message size to 2K
Shawn Willden [Thu, 3 Aug 2017 20:10:50 +0000 (14:10 -0600)]
Reduce max keymaster message size to 2K

Bug: 63745895
Test: VtsHalKeymasterV3_0TargetTest
Change-Id: I9b8c2e551f04bd2124462095f232bd08ff1f63c6

6 years agonfc: NCI 2.0: Add CORE_INIT_CMD for NCI 2.0
Ruchi Kandoi [Thu, 3 Aug 2017 19:04:38 +0000 (19:04 +0000)]
nfc: NCI 2.0: Add CORE_INIT_CMD for NCI 2.0
am: 029633dcd0

Change-Id: I7377ba90f9fe42f3d2880c5c9e6f494203050213

6 years agoMerge "graphics: ignore/reduce spurious vsync in VTS" into oc-dr1-dev
TreeHugger Robot [Thu, 3 Aug 2017 17:35:26 +0000 (17:35 +0000)]
Merge "graphics: ignore/reduce spurious vsync in VTS" into oc-dr1-dev

6 years agonfc: NCI 2.0: Add CORE_INIT_CMD for NCI 2.0
Ruchi Kandoi [Wed, 2 Aug 2017 00:30:17 +0000 (17:30 -0700)]
nfc: NCI 2.0: Add CORE_INIT_CMD for NCI 2.0

Bug: 64245250
Test: run vts test
Change-Id: I4ca434fb1fb44a6278942f020ddd02b162d7f605
(cherry picked from commit c901276e6063e739b742435a161e51978c5f4ec5)

6 years agoMerge "Camera: reset callback after test done" into oc-dev
Yin-Chia Yeh [Wed, 2 Aug 2017 01:19:33 +0000 (01:19 +0000)]
Merge "Camera: reset callback after test done" into oc-dev
am: 72ff9a1e38  -s ours

Change-Id: I4cf59eaf2e9583af574d50f9ddb830eb34e3356d

6 years agoMerge "Camera: reset callback after test done" into oc-dev
TreeHugger Robot [Tue, 1 Aug 2017 21:40:35 +0000 (21:40 +0000)]
Merge "Camera: reset callback after test done" into oc-dev

6 years agoresolve merge conflicts of 345a396c to oc-dr1-dev
sqian [Tue, 1 Aug 2017 19:21:55 +0000 (12:21 -0700)]
resolve merge conflicts of 345a396c to oc-dr1-dev

Test: I solemnly swear I tested this conflict resolution.
Change-Id: I5d816e17b2c8b113d07bf5e827d8c9378af761d9
Merged-In: I5d816e17b2c8b113d07bf5e827d8c9378af761d9

6 years agoMerge "Add EMPTY_RECORD for getDeviceIdentity" into oc-dev
Shuo Qian [Tue, 1 Aug 2017 17:55:32 +0000 (17:55 +0000)]
Merge "Add EMPTY_RECORD for getDeviceIdentity" into oc-dev

6 years agoAdd EMPTY_RECORD for getDeviceIdentity
sqian [Fri, 28 Jul 2017 18:28:27 +0000 (11:28 -0700)]
Add EMPTY_RECORD for getDeviceIdentity

Remove same unit test case with arguments "","0" for:
- sendEnvelope
- sendTerminalResponseToSim
- sendEnvelopeWithStatus

Test: verified by vendor, treehuger
Bug: 62926561
Change-Id: I5f535bdfc5821275a7ea2571d411374e0d7a8822
Merged-In: I5f535bdfc5821275a7ea2571d411374e0d7a8822

6 years agoMerge "Add corresponding error code to fix vts test" into oc-dev
Shuo Qian [Tue, 1 Aug 2017 00:25:20 +0000 (00:25 +0000)]
Merge "Add corresponding error code to fix vts test" into oc-dev
am: 417af3a797  -s ours

Change-Id: I98205a431af725626f8f692d9b69d4c4c1f6fe5b

6 years agoMerge "Add corresponding error code to fix vts test" into oc-dev
Shuo Qian [Tue, 1 Aug 2017 00:19:48 +0000 (00:19 +0000)]
Merge "Add corresponding error code to fix vts test" into oc-dev

6 years agoMerge "Add corresponding error code to fix vts test" into oc-dr1-dev
Shuo Qian [Tue, 1 Aug 2017 00:19:33 +0000 (00:19 +0000)]
Merge "Add corresponding error code to fix vts test" into oc-dr1-dev

6 years agoMerge "Audio policy config xsd: add missing gains and address in ports" into oc-dev
Kevin Rocard [Tue, 1 Aug 2017 00:05:16 +0000 (00:05 +0000)]
Merge "Audio policy config xsd: add missing gains and address in ports" into oc-dev
am: 90e68d31cf

Change-Id: I0aa31b2996c8aae97d0b7ba16d9943f47d70a1fe

6 years agoMerge "Audio policy config xsd: add missing gains and address in ports" into oc-dev
TreeHugger Robot [Mon, 31 Jul 2017 23:56:02 +0000 (23:56 +0000)]
Merge "Audio policy config xsd: add missing gains and address in ports" into oc-dev

6 years agoMerge "Wifi Offload HAL: Improve VTS tests for scanConfig" into oc-dr1-dev
TreeHugger Robot [Mon, 31 Jul 2017 19:52:44 +0000 (19:52 +0000)]
Merge "Wifi Offload HAL: Improve VTS tests for scanConfig" into oc-dr1-dev

6 years agoMerge "NO PARTIAL RERUN Camera: Update stream consumer usage flag" into oc-dev
Emilian Peev [Mon, 31 Jul 2017 16:28:15 +0000 (16:28 +0000)]
Merge "NO PARTIAL RERUN Camera: Update stream consumer usage flag" into oc-dev
am: 456dc6cc86  -s ours

Change-Id: I10d00cc7a42127f5235a97dc72f76e608dfb5fea

6 years agoMerge "NO PARTIAL RERUN Camera: Update stream consumer usage flag" into oc-dev
TreeHugger Robot [Mon, 31 Jul 2017 16:22:23 +0000 (16:22 +0000)]
Merge "NO PARTIAL RERUN Camera: Update stream consumer usage flag" into oc-dev

6 years agoMerge "Camera: Update stream consumer usage flag" into oc-dr1-dev
TreeHugger Robot [Mon, 31 Jul 2017 16:21:44 +0000 (16:21 +0000)]
Merge "Camera: Update stream consumer usage flag" into oc-dr1-dev

6 years agoCamera: fix various VTS issues
Yin-Chia Yeh [Mon, 31 Jul 2017 08:55:46 +0000 (08:55 +0000)]
Camera: fix various VTS issues
am: a0dd95d45d  -s ours

Change-Id: Ib438269494c5c25f1759f7ec016cbca97bec9072

6 years agoNO PARTIAL RERUN Camera: Update stream consumer usage flag
Emilian Peev [Wed, 26 Jul 2017 08:46:38 +0000 (09:46 +0100)]
NO PARTIAL RERUN Camera: Update stream consumer usage flag

It is possible that the current default consumer usage
flag may not be supported by some provider implementations.
Use either HW composer or some other flag that is appropriate
for the specific use case.

Merged-In: I04f89bf67166805191d6d40e5bd93c15ebc97ea6
Bug: 63913159
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO

Change-Id: I04f89bf67166805191d6d40e5bd93c15ebc97ea6

6 years agoRadioV1_1 VTS Tests: startKeepalive, stopKeepalive
Nathan Harold [Wed, 19 Jul 2017 23:48:44 +0000 (16:48 -0700)]
RadioV1_1 VTS Tests: startKeepalive, stopKeepalive

VTS tests for the initial implementaion of NATT
Keepalive support in the V1_1 HAL.

Bug:63530561
Test: this
Change-Id: Iafabd4b0f8a8f4da78f65bf4061d47f4bfd69a7a

6 years agoCamera: fix various VTS issues
Yin-Chia Yeh [Wed, 26 Jul 2017 23:53:20 +0000 (16:53 -0700)]
Camera: fix various VTS issues

Also fix wrong return values for processCaptureRequest in default
wrapper.

Test: running camera VTS
Bug: 64041692
Change-Id: I397390af7c85a776713f6287bef1c4d11c721c9a
Merged-In: I397390af7c85a776713f6287bef1c4d11c721c9a

6 years agoAdd corresponding error code to fix vts test
sqian [Mon, 24 Jul 2017 20:51:24 +0000 (13:51 -0700)]
Add corresponding error code to fix vts test

Test: run vts
Bug: 63152815
Bug: 62989786
Bug: 64041522
Change-Id: Ifbd17f85b39221de31ec7884b30dc176e1cbd3ab
Merged-In: Ifbd17f85b39221de31ec7884b30dc176e1cbd3ab

6 years agoMerge "Add vts test for setCarrierInfoForImsiEncryption" into oc-dr1-dev
Shuo Qian [Fri, 28 Jul 2017 19:15:45 +0000 (19:15 +0000)]
Merge "Add vts test for setCarrierInfoForImsiEncryption" into oc-dr1-dev