OSDN Git Service
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
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)
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
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
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
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>
Iris Chang [Tue, 10 Oct 2017 16:22:34 +0000 (16:22 +0000)]
VTS: fix VtsHalKeymasterV3_0Target issue
am:
926a8e7440
Change-Id: I762f63b3f239169d93b33f7164d4eb3f9bd8f3d9
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)
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
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>
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
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
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
TreeHugger Robot [Tue, 12 Sep 2017 06:46:22 +0000 (06:46 +0000)]
Merge "Reduce max keymaster message size to 2K" into oc-dev
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
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>
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
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
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
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
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
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)
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
TreeHugger Robot [Sun, 3 Sep 2017 12:42:39 +0000 (12:42 +0000)]
Merge "bug fix: restore support for broken flag" into oc-dev
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
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
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
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)
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
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
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)
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
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
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
TreeHugger Robot [Fri, 25 Aug 2017 22:12:24 +0000 (22:12 +0000)]
Merge "Sensor VTS testBatchingOperation flakiness fix" 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
TreeHugger Robot [Fri, 25 Aug 2017 05:09:25 +0000 (05:09 +0000)]
Merge "bug fix: output colorformat configuration fixed" into oc-dev
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>
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
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
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
TreeHugger Robot [Sat, 19 Aug 2017 00:30:10 +0000 (00:30 +0000)]
Merge "bug fix: close file pointer" 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
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
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
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
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
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
TreeHugger Robot [Tue, 15 Aug 2017 20:44:58 +0000 (20:44 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dr1-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
TreeHugger Robot [Tue, 15 Aug 2017 20:17:11 +0000 (20:17 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev
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
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
Tomasz Wasilczyk [Fri, 11 Aug 2017 22:11:31 +0000 (22:11 +0000)]
Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev
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
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
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
TreeHugger Robot [Thu, 10 Aug 2017 00:35:17 +0000 (00:35 +0000)]
Merge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev
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)
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)
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)
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
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
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
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
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
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
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
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
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
TreeHugger Robot [Tue, 8 Aug 2017 03:57:43 +0000 (03:57 +0000)]
Merge "Tetheroffload Control and Config VTS" into oc-dr1-dev
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
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
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
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
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
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
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
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
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)
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
TreeHugger Robot [Tue, 1 Aug 2017 21:40:35 +0000 (21:40 +0000)]
Merge "Camera: reset callback after test done" into oc-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
Shuo Qian [Tue, 1 Aug 2017 17:55:32 +0000 (17:55 +0000)]
Merge "Add EMPTY_RECORD for getDeviceIdentity" into oc-dev
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
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
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
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
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
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
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
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
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
TreeHugger Robot [Mon, 31 Jul 2017 16:21:44 +0000 (16:21 +0000)]
Merge "Camera: Update stream consumer usage flag" into oc-dr1-dev
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
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
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
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
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
Shuo Qian [Fri, 28 Jul 2017 19:15:45 +0000 (19:15 +0000)]
Merge "Add vts test for setCarrierInfoForImsiEncryption" into oc-dr1-dev