OSDN Git Service
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 [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 [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 [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 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>
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
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 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>
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:18:25 +0000 (17:18 +0000)]
Merge "graphics: ignore/reduce spurious vsync in VTS" into oc-dev
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)
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
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)
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
TreeHugger Robot [Fri, 25 Aug 2017 22:12:24 +0000 (22:12 +0000)]
Merge "Sensor VTS testBatchingOperation flakiness fix" 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
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>
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 [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:17:11 +0000 (20:17 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" 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
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 [Wed, 9 Aug 2017 16:48:07 +0000 (16:48 +0000)]
Merge "Skip direct report test if sensor is not available" into oc-dev
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
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 [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)
TreeHugger Robot [Tue, 1 Aug 2017 21:40:35 +0000 (21:40 +0000)]
Merge "Camera: reset callback after test done" 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
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:19:48 +0000 (00:19 +0000)]
Merge "Add corresponding error code to fix vts test" 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
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
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
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
Kevin Rocard [Thu, 27 Jul 2017 18:41:53 +0000 (11:41 -0700)]
Audio policy config xsd: add missing gains and address in ports
No nexus/pixel device uses port gains and their configuration were taken
as reference for the xsd creation.
Gains in mixPort and devicePort are supported by the code and use by
oem. As a result the xsd should allow them.
For validation of this path, the xsd was run against the example xml in
the audiopolicy source. Several other misalignment were found. They will
be fix in an other patch.
The address is also an optional field that was forgotten for the same
reason.
Bug: b/
63827061
Test: -noout xmllint --schema hardware/interfaces/audio/2.0/config/audio_policy_configuration.xsd frameworks/av/services/audiopolicy/config/audio_policy_configuration.xml
Test: the above command fails for some other xml node unrelated to this bug
Test: this is tracked by b/
38184704
Change-Id: I8dae15eb85a6a6d43c87aa747daf92a88d3fdcc0
Signed-off-by: Kevin Rocard <krocard@google.com>
sqian [Mon, 24 Jul 2017 21:05:29 +0000 (14:05 -0700)]
Add corresponding error code to fix vts test
Test: run vts
Bug:
63152815
Bug:
62989786
Bug:
64041522
Change-Id: Ifbd17f85b39221de31ec7884b30dc176e1cbd3ab
Merged-In: Ifbd17f85b39221de31ec7884b30dc176e1cbd3ab
Eino-Ville Talvala [Tue, 25 Jul 2017 20:58:44 +0000 (20:58 +0000)]
Merge "Camera: Don't enforce result metadata queue support" into oc-dev
Emilian Peev [Mon, 24 Jul 2017 15:29:54 +0000 (16:29 +0100)]
Camera: Don't enforce result metadata queue support
Some provider implementations could still be using direct
transfer of the result metadata through hw binder.
Merged-In: Ib2adaff005af77385767a32c1ed069371a6e9b7a
Bug:
63970723
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalCameraProviderV2_4Target -l INFO
Change-Id: Ib2adaff005af77385767a32c1ed069371a6e9b7a
Roshan Pius [Mon, 24 Jul 2017 21:13:48 +0000 (14:13 -0700)]
supplicant(vts): Remove start/stop framework
This is already done by the test harness on the host side. So, no need
to invoke this from within the gtest.
Bug:
63914399
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
VtsHalWifiSupplicantV1_0Target -l INFO`
Change-Id: I43edc03492bf5e03f9a4236265c43baa5efcd61d
(cherry picked from commit
145cce98b56674404a6148835f008df44086db2e)
Merged-Id: I43edc03492bf5e03f9a4236265c43baa5efcd61d
Shuo Qian [Tue, 25 Jul 2017 00:14:21 +0000 (00:14 +0000)]
Merge "Add Sim absent assertion in the 1.0 setup" into oc-dev
TreeHugger Robot [Sat, 22 Jul 2017 15:51:40 +0000 (15:51 +0000)]
Merge "Fixed the HCI ACL Header definitions for PB and BC Flags" into oc-dev
Stanley Tng [Wed, 19 Jul 2017 17:55:01 +0000 (10:55 -0700)]
Fixed the HCI ACL Header definitions for PB and BC Flags
The byte offsets for Packet Boundary (PB) and Broadcast (BC) flags
in the HCI ACL Header is swapped. This will fixed it so that it matches
the Bluetooth Specifications.
Bug:
63241720
Test: Run manual VTS tests for Bluetooth
Change-Id: Ifc1f4d4f054a1dab13458c563e47bfb884f39d92
(cherry picked from commit
7915b71fb9fdc8a378b2aae9b618d67bf64d4971)
Legler Wu [Fri, 30 Jun 2017 03:00:07 +0000 (11:00 +0800)]
Update VTS tests so that Mediatek pass more VTS for O.
Bug: b/
62926561
Test: vts tests
Change-Id: Ic927f9efd437c6411fb14feac8c681334e4b5e60
Merged-in: Ic927f9efd437c6411fb14feac8c681334e4b5e60
sqian [Fri, 21 Jul 2017 18:49:03 +0000 (11:49 -0700)]
Add Sim absent assertion in the 1.0 setup
Currently the VTS tests for radio are for Sim Absent only. Adding this
assertion in the set up before running VTS tests clarifies the current
tests running is for sim absent only.
Test: run vts
Bug:
63898623
Change-Id: Ie09fb8bcedfd0b35b14f0a70952f44b6f6b082aa
Shuo Qian [Fri, 21 Jul 2017 18:58:12 +0000 (18:58 +0000)]
Merge "Update IRadioResponse.hal documentation" into oc-dev
Shuo Qian [Thu, 20 Jul 2017 23:57:25 +0000 (23:57 +0000)]
Merge "Fix some VTS for 2017 devices" into oc-dev
sqian [Wed, 19 Jul 2017 18:54:03 +0000 (11:54 -0700)]
Fix some VTS for 2017 devices
- sendDeviceState
- setIndicationFilter
Mentioned in b/
63148938
- getNeighboringCids
Add Request_not_support for Apis mentioned in Bugs:
-
62989786
-
62986286
-
62987946
Test: run vts
Bug:
63130111
Bug:
63148938
Bug:
62989786
Bug:
62986286
Bug:
62987946
Change-Id: Ib51f67d403bb27f409018a98c07d6aa6aa6d19f8
Merged-In: Ib51f67d403bb27f409018a98c07d6aa6aa6d19f8
(cherry picked from commit
fe0699535e8d17b7716a10e64c2528abc9ffaecb)
Yin-Chia Yeh [Tue, 11 Jul 2017 20:44:35 +0000 (13:44 -0700)]
Camera: reset callback after test done
Test: VTS test pass
Bug:
63570734
Change-Id: Ia7b1e8e9baae2885d9c43821fe2933df883f7892
Merged-In: Ia7b1e8e9baae2885d9c43821fe2933df883f7892
Emilian Peev [Wed, 21 Jun 2017 13:52:54 +0000 (14:52 +0100)]
Camera: Try to test additional types of camera providers NO PARTIAL RERUN
Check whether different types of camera providers got
registered and try to test them if present.
Merged-In: I5c0c678e74d56cd8307d2f4e63b7c6dfa95d210d
Bug:
62835396
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalCameraProviderV2_4Target -l INFO
Change-Id: I5c0c678e74d56cd8307d2f4e63b7c6dfa95d210d
Connor O'Brien [Thu, 29 Jun 2017 02:06:23 +0000 (02:06 +0000)]
Merge "Fix consumerir test bugs" into oc-dev
TreeHugger Robot [Thu, 29 Jun 2017 01:46:13 +0000 (01:46 +0000)]
Merge "Camera: Add support for testing partial results" into oc-dev
Connor O'Brien [Wed, 28 Jun 2017 23:46:43 +0000 (16:46 -0700)]
Fix consumerir test bugs
The tests currently fail for some correct implementations; revise them
so this no longer occurs.
Bug:
63081607
Test: built & ran revised test
Change-Id: I21b72fd864de57272bd6eb42dca238f661137b93
Signed-off-by: Connor O'Brien <connoro@google.com>
Yahan Zhou [Wed, 28 Jun 2017 17:27:20 +0000 (17:27 +0000)]
Merge "Fix ICryptoFactory service" into oc-dev
Emilian Peev [Tue, 23 May 2017 15:37:11 +0000 (16:37 +0100)]
Camera: Add support for testing partial results
Currently the result handling is very basic. Partial
results and inflight queues are not supported. More
elaborate test cases like burst or streaming will
require such functionality to be present.
Merged-In: I3769d4ccc0047dc4eda173ea3bca538b026be8d2
Bug:
62550376
Bug:
37533777
Test: adb shell
/data/nativetest/VtsHalCameraProviderV2_4TargetTest/VtsHalCameraProviderV2_4TargetTest
Change-Id: I3769d4ccc0047dc4eda173ea3bca538b026be8d2
bohu [Fri, 23 Jun 2017 19:31:42 +0000 (12:31 -0700)]
vts: fix TryDifferentGovernors in Power hal test
The test should return when it cannot open cpu governor files
BUG:
62138092
Test: build sdk_gphone_x86-userdebug, launch emulator,
run test: run vts -m VtsHalPowerV1_0Target
The test should all pass.
Change-Id: I3b47261e9525b6a04e995dac1ad63d9cc088b004
TreeHugger Robot [Fri, 23 Jun 2017 17:51:15 +0000 (17:51 +0000)]
Merge "Camera: fix recursive lock in CameraDevice 1.0" into oc-dev
TreeHugger Robot [Fri, 23 Jun 2017 15:32:14 +0000 (15:32 +0000)]
Merge "Test import of a EC P-521 key." into oc-dev
TreeHugger Robot [Fri, 23 Jun 2017 04:57:02 +0000 (04:57 +0000)]
Merge "[WIFI] Remove dependency on vendor-specific library" into oc-dev
Yahan Zhou [Fri, 23 Jun 2017 00:04:41 +0000 (17:04 -0700)]
Camera: fix recursive lock in CameraDevice 1.0
Several functions call close() while they are holding a lock, which
results in recursive locking. This CL implements close_locked() to avoid
such behavior.
Bug:
62919192
Test: run vts -m VtsHalCameraProviderV2_4Target, and camera should still
work after it
Change-Id: Ib38e1de19ed3c927bfb645c0c777c04f157f2b88
Shawn Willden [Fri, 16 Jun 2017 14:18:06 +0000 (08:18 -0600)]
Test import of a EC P-521 key.
This failure is also diagnosed by CTS, but it should be validated in VTS
as well.
Merged-In: Ia7654ff8813942fbca9dfa838337e9de5839a9e2
Bug:
33945114
Test: adb shell data/nativetests64/VtsHalKeymasterV3_0TargetTest/VtsHalKeymasterV3_0TargetTest
Change-Id: Ia7654ff8813942fbca9dfa838337e9de5839a9e2
Yahan Zhou [Wed, 21 Jun 2017 21:24:09 +0000 (14:24 -0700)]
Fix ICryptoFactory service
BUG:
62642564
BUG:
62138343
Test: vts -m VtsHalDrmV1_0Target
Change-Id: Ic697a32b111804fc3dc78f1b6e6b77538d7eeea9
(cherry picked from commit
85a5d8855b07aec3727532983de38c324960d6d1)
Fabien Sanglard [Tue, 20 Jun 2017 02:25:31 +0000 (19:25 -0700)]
Skip Validate
b/
37474580
b/
62806392
Test: marlin, ryu
Change-Id: I9d4ece40a8a093d768bcb4edd332cf0188735e66
sqian [Wed, 21 Jun 2017 21:51:25 +0000 (14:51 -0700)]
Update IRadioResponse.hal documentation
Update the error codes
A copy of https://partner-android-review.googlesource.com/#/c/836552/
+ some missed updated error codes
+ request_not_supported
Test: Manual
Bug:
35671595
Change-Id: I2880b442b5fd56149abc83949c12e2eba5ee67df
Merged-In: I2880b442b5fd56149abc83949c12e2eba5ee67df
TreeHugger Robot [Wed, 21 Jun 2017 00:45:16 +0000 (00:45 +0000)]
Merge "Check return from DeleteKey correctly." into oc-dev
TreeHugger Robot [Tue, 20 Jun 2017 10:59:01 +0000 (10:59 +0000)]
Merge "Clear notification count to handle different flows" into oc-dev
Wyatt Riley [Mon, 19 Jun 2017 21:42:34 +0000 (14:42 -0700)]
Clear notification count to handle different flows
Notifiation count now cleared in Setup (and
discarded messages warned about in Teardown.)
Fixes:
62645164
Test: passes on device
Change-Id: I7e88da7251464f281d7c98140d7a9b5a73d79542
(cherry picked from commit
6ec696b7fed0c597707762a6d5f5ecd709408aae)
Peng Xu [Mon, 19 Jun 2017 18:43:24 +0000 (11:43 -0700)]
Zero uninitialized field in conversion and fix vts bug
Zero uninitialized sensor event field to zero to avoid undefined
value being passed to framework.
Correct sensor report type check bug in vts.
Bug:
37793558
Bug:
37710268
Test: sensor vts passes
Change-Id: I5042975fea8fe28ae130367ba083a4e4f4e5657e
Ruchi Kandoi [Mon, 19 Jun 2017 16:33:33 +0000 (16:33 +0000)]
Merge "Nfc: Vts: modify WriteInvalidAndThenValidCommand" into oc-dev
Ruchi Kandoi [Thu, 15 Jun 2017 20:22:51 +0000 (13:22 -0700)]
Nfc: Vts: modify WriteInvalidAndThenValidCommand
Invalid Data packets are ignored by the NFC controller in case there
isn't an active RF Interface. The test expects response for the invalid
data packet without any active RF Interface. This patch modifies the
test to send an invalid command instead which will guarantee a response.
Test: run vts test
Bug:
62400568
Change-Id: I7db1d5209b6ef2240ac248f3aba0e27e0a285018
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>
(cherry picked from commit
c86df4dcb74224ea7befc2cb610e43d0df6814ae)
TreeHugger Robot [Fri, 16 Jun 2017 19:22:04 +0000 (19:22 +0000)]
Merge "Camera: fix camera_metadata address expectation" into oc-dev
Ruchi Kandoi [Thu, 15 Jun 2017 22:59:23 +0000 (22:59 +0000)]
Merge "Support NCI 2.0 for NFC VTS" into oc-dev
Yin-Chia Yeh [Thu, 15 Jun 2017 00:29:47 +0000 (17:29 -0700)]
Camera: fix camera_metadata address expectation
On 32bits architecture, binder driver might allocate
non-8 bytes aligned buffers.
Test: compile (problem not reproducible on 64bit SOCs)
Bug:
62637461
Change-Id: I8ef2b3ecba783a8d189ebe18733082d32cff4899
TreeHugger Robot [Wed, 14 Jun 2017 02:22:13 +0000 (02:22 +0000)]
Merge "Purge unsupported RSA key sizes from VTS tests" into oc-dev
TreeHugger Robot [Wed, 14 Jun 2017 01:26:29 +0000 (01:26 +0000)]
Merge "Fixed flaky RenderScript VTS Test" into oc-dev
Janis Danisevskis [Tue, 6 Jun 2017 17:11:07 +0000 (10:11 -0700)]
Purge unsupported RSA key sizes from VTS tests
Support for 256 bit RSA keys are not mandated by keymaster specs and
must not be used in the VTS tests.
Bug:
62581389
Change-Id: If315088db2752ac2efe31fdb95db7ca13c3ce225
Michael Butler [Tue, 23 May 2017 00:01:24 +0000 (17:01 -0700)]
Fixed flaky RenderScript VTS Test
Added missing synchronization before context destroy
Bug:
62535730
Bug:
38385848
Test: make vts, does not fail with multiple tests
Change-Id: Ib0d038969e9a1d6f5ea555ccb60fed55b69e05dc
(cherry picked from commit
076f69aa2bc2301e3c93150503d469c02ed2458c)
Ruchi Kandoi [Thu, 8 Jun 2017 17:27:08 +0000 (10:27 -0700)]
Support NCI 2.0 for NFC VTS
Bug:
62322170
Test: Run vts test
Change-Id: I7ed331dac9d5918675c091620f099b6250e20700
(cherry picked from commit
ff8068d160b642c757f29ee56540dff0af468fe7)
TreeHugger Robot [Mon, 12 Jun 2017 22:50:33 +0000 (22:50 +0000)]
Merge "Bluetooth: Call vendor cleanup() in Close()" into oc-dev
Shuo Qian [Mon, 12 Jun 2017 04:44:15 +0000 (04:44 +0000)]
Merge "Fix vts tests:" into oc-dev
Ram Mohan M [Sat, 10 Jun 2017 02:55:39 +0000 (08:25 +0530)]
Increase Timeout for dequeue message
In vp9 encoder component, the callback responses are being recevied
after timeout. This is seen only in vp9. Even when no buffer processing
is done, Flush test, state transitions test fail due to slow responses.
This behavior is not seen in other components. This requires deeper
understanding of what is main culprit. But for now increase the timeout
Test: VtsHalMediaOmxV1_0Host
Bug:
37501531
Change-Id: I08abfefcf0cb38d312a4a2dc10908e24f88c54fd
TreeHugger Robot [Sat, 10 Jun 2017 01:23:28 +0000 (01:23 +0000)]
Merge "bug fix: fix encoder input port configuration issues" into oc-dev
TreeHugger Robot [Sat, 10 Jun 2017 01:23:28 +0000 (01:23 +0000)]
Merge "bug fix: configure input port format before setInputSurface" into oc-dev
Myles Watson [Thu, 8 Jun 2017 15:38:58 +0000 (08:38 -0700)]
Bluetooth: Call vendor cleanup() in Close()
Bug:
62453652
Test: sailfish and angler start/stop Bluetooth, pair
Change-Id: I8ba80ae1802d1f524acb1d8e26b5bb41fc1612d0
(cherry picked from commit
9eee83093faf0edd023920b16f5251a17ae97f00)
sqian [Fri, 9 Jun 2017 16:24:26 +0000 (09:24 -0700)]
Fix vts tests:
- setCdmaSubscriptionSource
- nvReadItem
- nvWriteItem
- nvWriteCdmaPrl
- getHardwareConfig
- sendEnvelope
- sendTerminalResponseToSim
Apply ag/
2373104/ to prevent merge conflicts
Will update .hal file seperately.
Test: adb shell <vts_binary>
Bug:
35671595
Bug:
35800570
Change-Id: I651af50f122f37721778b1088df40fa6e12ac6e4
Merged-In: I651af50f122f37721778b1088df40fa6e12ac6e4
Ram Mohan M [Thu, 8 Jun 2017 15:31:03 +0000 (21:01 +0530)]
choose input color format between 420p, 420sp
Test: VtsHalMediaOmxV1_0Host
Bug:
37501531
Change-Id: I5fc2f5c69ca545f2ca0b444ce89a8c970f2ff98e
TreeHugger Robot [Fri, 9 Jun 2017 03:12:07 +0000 (03:12 +0000)]
Merge "AudioDec Test: Ignore output port configuration" into oc-dev
TreeHugger Robot [Fri, 9 Jun 2017 03:12:07 +0000 (03:12 +0000)]
Merge "VideoEnc Test: extend lockYCbCr() to hal yuv formats" into oc-dev
TreeHugger Robot [Fri, 9 Jun 2017 03:12:07 +0000 (03:12 +0000)]
Merge "Vts AudioDec Test: Disable EOSTest_M" into oc-dev