OSDN Git Service
TreeHugger Robot [Thu, 24 Aug 2017 12:22:34 +0000 (12:22 +0000)]
Merge "Modify StateTransitions test" into oc-mr1-dev
TreeHugger Robot [Mon, 21 Aug 2017 18:34:31 +0000 (18:34 +0000)]
Merge "Camera test statically links to dependencies." into oc-mr1-dev
Ram Mohan M [Sat, 19 Aug 2017 01:48:16 +0000 (01:48 +0000)]
Merge "bug fix: close file pointer" into oc-dev am:
8735fb45a5
am:
51f04884ba -s ours
Change-Id: Iaa1a97e0771874b65adf0eab0a48de7eed78be6d
Ram Mohan M [Sat, 19 Aug 2017 01:47:54 +0000 (01:47 +0000)]
Merge "bug fix: configure input port buffer size" into oc-dev am:
bc769a7e5a
am:
2eedb46745 -s ours
Change-Id: Ic49186056e830ac1edbe299fc35ced4105f2690c
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
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
Nick Desaulniers [Fri, 18 Aug 2017 17:00:44 +0000 (10:00 -0700)]
biometrics: fingerprint: add locking to default impl
There exists the following race condition:
a). thread A receives setNotify and sets the callback to some object
b). thread B of the wrapped implementation calls
BiometricsFingerprint::notify which it was given a handle to. Thread B
executes past the nullptr check:
c). thread A receives setNotify and sets the callback to some other
object (or nullptr)
d). thread B resumes in notify with unknown state
Add mutex to protect access to mClientCallback.
Change-Id: I9163204ff5802e9246056caeb2a7857e6138531c
Fixes:
64802340
Test: VtsHalBiometricsFingerprintV2_1IfaceFuzzer
TreeHugger Robot [Fri, 18 Aug 2017 16:54:14 +0000 (16:54 +0000)]
Merge "Increase thread pool for VHAL 2.1" into oc-mr1-dev
Pavel Maltsev [Wed, 16 Aug 2017 23:43:44 +0000 (16:43 -0700)]
Increase thread pool for VHAL 2.1
Fixes:
36510399
Test: runtest -x
packages/services/Car/tests/vehiclehal_test/src/com/android/car/vehiclehal/test/E2ePerformanceTest.java
Change-Id: Id5638f63c6ff23bb78675fa07a2f3922bb7e3d73
Jeff Tinker [Fri, 18 Aug 2017 04:13:04 +0000 (04:13 +0000)]
Allow clearkey tests to run if no vendor modules am:
5db2e67293
am:
66812dc364
Change-Id: I6369dd486c35c842c1d0f1f151b3b468ef94ab4c
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
TreeHugger Robot [Fri, 18 Aug 2017 03:07:50 +0000 (03:07 +0000)]
Merge "Load libRS_internal.so from the rs namespace" into oc-mr1-dev
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
Miao Wang [Wed, 16 Aug 2017 23:47:06 +0000 (23:47 +0000)]
Merge "Remove the data type from the OperationType enum." into oc-mr1-dev
Edwin Wong [Wed, 16 Aug 2017 22:22:38 +0000 (22:22 +0000)]
Merge "Add test for additional ClearKey UUID." into oc-mr1-dev
Tri Vo [Wed, 16 Aug 2017 17:33:24 +0000 (17:33 +0000)]
Merge "Audio test statically links to dependencies." into oc-mr1-dev
Jiyong Park [Wed, 16 Aug 2017 14:30:42 +0000 (23:30 +0900)]
Load libRS_internal.so from the rs namespace
Bug:
64747884
Test: VtsHalRenderscriptV1_0TargetTest successful on the device built
with BOARD_VNDK_VERSION=current and [system] namespace config is applied
to /data/nativetest[64]/* processes.
Change-Id: I9e967c80ac2dba718cd47e1f378bcbf18abe9ad2
TreeHugger Robot [Wed, 16 Aug 2017 04:25:11 +0000 (04:25 +0000)]
Merge "bug fix: close file pointer" into oc-mr1-dev
TreeHugger Robot [Wed, 16 Aug 2017 04:23:43 +0000 (04:23 +0000)]
Merge "bug fix: configure input port buffer size" into oc-mr1-dev
TreeHugger Robot [Tue, 15 Aug 2017 22:53:06 +0000 (22:53 +0000)]
Merge "[AWARE] Fix maxNdpSessions documentation" into oc-mr1-dev
Roshan Pius [Tue, 15 Aug 2017 22:14:00 +0000 (22:14 +0000)]
Merge "wifi(implementation): Reduce logging severity" into oc-mr1-dev
Emilian Peev [Tue, 15 Aug 2017 21:11:13 +0000 (21:11 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dr1-dev
am:
92d3bc1e1a -s ours
Change-Id: I1a648056655a978d867b50a53d929c646fc6d035
Emilian Peev [Tue, 15 Aug 2017 21:10:24 +0000 (21:10 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev am:
e2dae8c258
am:
5d18c87e88 -s ours
Change-Id: Ib03c1512099ec88e1c65672dadbe29ef3b9d9086
TreeHugger Robot [Tue, 15 Aug 2017 20:51:33 +0000 (20:51 +0000)]
Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-mr1-dev
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
Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
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
Tri Vo [Fri, 4 Aug 2017 22:27:06 +0000 (15:27 -0700)]
Audio test statically links to dependencies.
This test now statically links to libs not guaranteed to be on the
device.
Bug:
64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions
--module VtsHalAudioEffectV2_0Target
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions
--module VtsHalAudioV2_0Target
Change-Id: I331d7e4abdf7e520df6d508a8059b6358ad4398f
Roshan Pius [Tue, 15 Aug 2017 15:42:50 +0000 (08:42 -0700)]
wifi(implementation): Reduce logging severity
IE elements should not be present in the batched scan results, but it
being there doesn't cause any problems. The IE elements will be ignored
when we send it over the HIDL interface, so reduce the log level to
prevent log spamming.
Bug:
64448186
Test: None
Change-Id: Ia0d0312a113c907bb0992bb8f41e734fe9d05114
Ram Mohan M [Wed, 28 Jun 2017 05:29:59 +0000 (10:59 +0530)]
Modify StateTransitions test
During loaded <-> idle state transition, check for events
is done after allocating all the buffers on a port. This is now
done after allocation of every single buffer. This is done to
improve scope of testing
Bug:
63796949
Change-Id: Icf3405ac6716f9533d31189f6c4a4c4ac908da5f
Ram Mohan M [Wed, 9 Aug 2017 09:22:20 +0000 (14:52 +0530)]
bug fix: close file pointer
Bug:
63796949
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
Change-Id: Ib760c4f55b094260a0abd120f852dcf1899df4e0
TreeHugger Robot [Mon, 14 Aug 2017 22:16:17 +0000 (22:16 +0000)]
Merge "Tetheroffload tests static link to HAL def libs." into oc-mr1-dev
Etan Cohen [Mon, 14 Aug 2017 19:10:42 +0000 (12:10 -0700)]
[AWARE] Fix maxNdpSessions documentation
Clarify meaning of maxNdpSessions capability. Total, not per interface.
Bug:
64685830
Test: builds
Change-Id: I24fc0a97172a4ca161f432b6c5c1dc55a44dfb12
Miao Wang [Thu, 3 Aug 2017 03:21:06 +0000 (20:21 -0700)]
Remove the data type from the OperationType enum.
- Now the driver needs to report a list of tuple
{OperationType, OperandType} for capabilities.
- Any time Operation information is passed across HIDL,
it should be passed as the tuple {OperationType, OperandType}
Bug:
63905942
Test: mm
Change-Id: I909b5acf4936f65c242ee0925d3a1ac665f46131
Tomasz Wasilczyk [Sat, 12 Aug 2017 00:15:55 +0000 (00:15 +0000)]
Merge "Widen dummy band range to cover all regions." into oc-mr1-dev
Tomasz Wasilczyk [Fri, 11 Aug 2017 22:19:44 +0000 (22:19 +0000)]
Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev am:
52af675d14
am:
fc73b0fcbd -s ours
Change-Id: I4f9fcf22f98f9c40290d321fabe05a082524da19
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
Tri Vo [Fri, 11 Aug 2017 21:52:56 +0000 (14:52 -0700)]
Tetheroffload tests static link to HAL def libs.
Bug:
64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalTetherOffloadControlV1_0Target
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalTetherOffloadConfigV1_0Target
Change-Id: Ib771c07646f3e219c7b9ba4be5515552d7fd6348
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
TreeHugger Robot [Fri, 11 Aug 2017 18:56:16 +0000 (18:56 +0000)]
Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-mr1-dev
TreeHugger Robot [Fri, 11 Aug 2017 16:40:06 +0000 (16:40 +0000)]
Merge "configstore: sandbox with seccomp filter" into oc-mr1-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.
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
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
TreeHugger Robot [Fri, 11 Aug 2017 01:47:49 +0000 (01:47 +0000)]
Merge "Mark diagnostic values as either being numerically equivalent to OBD2 values, or providing the equivalent OBD2 value" into oc-mr1-dev
Tri Vo [Fri, 11 Aug 2017 01:07:22 +0000 (01:07 +0000)]
Merge "Keymaster test statically links to dependencies." into oc-mr1-dev
Tomasz Wasilczyk [Thu, 10 Aug 2017 19:32:45 +0000 (12:32 -0700)]
Widen dummy band range to cover all regions.
Bug: b/
64115131
Test: VTS, instrumentation
Change-Id: I34e9aac440a003226200243c835e6116ba7eae90
Tri Vo [Thu, 10 Aug 2017 20:43:07 +0000 (20:43 +0000)]
Merge "Neural networks test static links to HAL def libs." into oc-mr1-dev
Tri Vo [Fri, 4 Aug 2017 20:01:48 +0000 (13:01 -0700)]
Keymaster test statically links to dependencies.
This test now statically links to libs not guaranteed to be on the
device.
Bug:
64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalKeymasterV3_0Target
Change-Id: I6a7b8c116153f18f61a71e5b5bef98343a4de43b
Tri Vo [Thu, 10 Aug 2017 17:45:07 +0000 (17:45 +0000)]
Merge "VTS tests now static link to HAL def libs. #2" into oc-mr1-dev
Tri Vo [Thu, 3 Aug 2017 23:37:47 +0000 (16:37 -0700)]
VTS tests now static link to HAL def libs. #2
And use VtsHalTargetTestDefaults.
Bug:
64040096
Test: crop out all non-affected tests from vts-hal-hidl.xml and run
vts-tradefed run commandAndExit vts-hal-hidl --skip-all-system-status-check
--skip-preconditions
2 failures, same as without this change:
GatekeeperHidlTest.DeleteAllUsersTest
GatekeeperHidlTest.DeleteUserTest
Change-Id: I8f6995e9536a9aefe283ee3effec9f5a7f03b620
Jeff Vander Stoep [Fri, 7 Jul 2017 05:29:12 +0000 (22:29 -0700)]
configstore: sandbox with seccomp filter
Configstore HAL is accessible to third party apps and thus requires
a tight sandbox that reflects the limited system access this HAL
needs.
We use two primary mechanisms to sandbox configstore, selinux and
seccomp, with the goal of restricting its access to userspace and
the kernel. The addition of a seccomp filter is primarily aimed
at reducing the kernel's attack surface that is reachable by
configstore HAL.
Seccomp filters are architecture dependent, so filters need to be
added for each architecture. This change adds a seccomp filter for
arm64 and issues a non-fatal runtime warning for other architectures
which still require a seccomp filter.
Bug:
36453956
Test: boot Marlin and Angler. Verify that configstore is not aborting
due to seccomp violations.
Test: "cat proc/<configstore pid>/status | grep seccomp " returns:
seccomp: 2
Which indicates that configstore is using seccomp-bpf.
Change-Id: Iab014ff357b7329085a5e18a92f51838d2c72371
(cherry picked from commit
ed95043d6433c5900984f24dbb101bba78109775)
TreeHugger Robot [Thu, 10 Aug 2017 03:42:37 +0000 (03:42 +0000)]
Merge "Graphics tests statically link to HAL definition libs." into oc-mr1-dev
Myles Watson [Thu, 10 Aug 2017 03:10:57 +0000 (03:10 +0000)]
Merge "Bluetooth: Check the number of file descriptors" into oc-dr1-dev
am:
269d9ba09a
Change-Id: I1260d97229e5330481178c8f802e2c7f6b651582
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 02:30:29 +0000 (02:30 +0000)]
Merge "Renderscript test static link to HAL def libs." into oc-mr1-dev
Myles Watson [Thu, 10 Aug 2017 00:39:03 +0000 (00:39 +0000)]
Merge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev
am:
5397fa8175
Change-Id: I1587c5451950a89c6aac991a0d8b7dacea16fe21
Myles Watson [Thu, 10 Aug 2017 00:38:46 +0000 (00:38 +0000)]
NFC: Add OWNERS file for default/
am:
a4d1a74700
Change-Id: I43572572901298d1e915c9753798361bdd5d38c2
TreeHugger Robot [Thu, 10 Aug 2017 00:35:17 +0000 (00:35 +0000)]
Merge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev
Tri Vo [Wed, 9 Aug 2017 23:41:15 +0000 (23:41 +0000)]
Merge "Media omx tests static link to dependencies." into oc-mr1-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)
TreeHugger Robot [Wed, 9 Aug 2017 21:23:20 +0000 (21:23 +0000)]
Merge changes I51b1e596,I3aece23a into oc-mr1-dev
* changes:
bug fix: output colorformat configuration fixed
bug fix: android list management in dequeueMsg()
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:15:44 +0000 (17:15 +0000)]
Merge "Skip direct report test if sensor is not available" into oc-dev am:
63524f9fcc
am:
7bea962c9c
Change-Id: I99d4eb05d1348e24910c5ebe2e87b6b334138b31
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 [Wed, 9 Aug 2017 05:09:17 +0000 (05:09 +0000)]
wifi(vts): Remove a tight loop of toggling wifi state
am:
0d1bb97447
Change-Id: Icec63b9ffd5d12003da44bdde73edde04e55f11a
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
Erik Kline [Wed, 9 Aug 2017 02:05:34 +0000 (02:05 +0000)]
Merge "Fix typo in in offload control VTS test" into oc-dr1-dev
am:
5d77efddbd
Change-Id: I2be944da8800040540bf98aeb197a933d1da0e42
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
Edwin Wong [Tue, 8 Aug 2017 22:31:30 +0000 (15:31 -0700)]
Add test for additional ClearKey UUID.
Test for both ClearKey plugin UUIDs in DrmHalClearkeyFactoryTest.
Test: VTS - VtsHalDrmV1_0TargetTest
bug:
64386897
Change-Id: Id7f140518910a87efdf9eaa19ce16113e206264f
yinxu [Tue, 8 Aug 2017 20:51:47 +0000 (20:51 +0000)]
Merge "Add the error code "OPERATION_NOT_ALLOWED" and log the error." into oc-dr1-dev
am:
3a57cca59c
Change-Id: I6a1aeea07e64b4748151f62ae4827132eba32781
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
TreeHugger Robot [Tue, 8 Aug 2017 19:26:42 +0000 (19:26 +0000)]
Merge "Add OWNERS for broadcast radio HAL 1.1." into oc-mr1-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 [Tue, 8 Aug 2017 17:10:25 +0000 (17:10 +0000)]
wifi(vts): Ignore IWifi.stop() returning ERROR_NOT_AVAILABLE
am:
8c216524a3
Change-Id: I1bf9be99d4e673209edb893040483bf772ee3ca0
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 06:16:47 +0000 (06:16 +0000)]
Merge "Update IRadioResponse.hal to add new error codes." into oc-mr1-dev
Erik Kline [Tue, 8 Aug 2017 04:04:48 +0000 (04:04 +0000)]
Merge "Refactor and expand tetheroffload Control and Config VTS" into oc-dr1-dev
am:
6d5cf73b75
Change-Id: I004e81a764313fdaf23f9580b144e1ee2ba79b27
Tyler Wear [Tue, 8 Aug 2017 04:04:34 +0000 (04:04 +0000)]
Merge "Tetheroffload Control and Config VTS" into oc-dr1-dev
am:
a11e731b6e
Change-Id: I24712d426e1227b1f96cfa35f9f7b320e27fc901
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
Tri Vo [Fri, 4 Aug 2017 21:42:31 +0000 (14:42 -0700)]
Media omx tests static link to dependencies.
These tests now statically link to libs not guaranteed to be on the
device, which include HAL definition libs.
Also, remove global include paths.
Bug:
64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalMediaOmxV1_0Host
Change-Id: Iea1dc549704f61b1416357d72dc1bf26864978d4
TreeHugger Robot [Tue, 8 Aug 2017 01:15:41 +0000 (01:15 +0000)]
Merge "thermal: IThermal V1.1, add methods to send thermal throttling events" into oc-mr1-dev
Ram Mohan M [Tue, 8 Aug 2017 00:49:23 +0000 (00:49 +0000)]
bug fix: handle OMX_EventBufferFlag events am:
c8df8940e1
am:
2921996e7a -s ours
Change-Id: I660fb164aa2f1ed1420e1cb5f81df462200e49bd
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
Tomasz Wasilczyk [Mon, 7 Aug 2017 19:00:09 +0000 (12:00 -0700)]
Add OWNERS for broadcast radio HAL 1.1.
Test: not necessary
Change-Id: I73881db46382276f56963c9d3a40cb0bdc1a2070
Tomasz Wasilczyk [Tue, 8 Aug 2017 00:35:09 +0000 (00:35 +0000)]
Merge "Change VENDOR program types to range." into oc-mr1-dev
TreeHugger Robot [Mon, 7 Aug 2017 23:51:34 +0000 (23:51 +0000)]
Merge "Remove unused variables." into oc-mr1-dev
Tri Vo [Fri, 4 Aug 2017 23:38:23 +0000 (16:38 -0700)]
Camera test statically links to dependencies.
This test now statically links to HAL def libs which
are not guaranteed to be on the device.
Bug:
64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalCameraProviderV2_4Target
Change-Id: Ib33fdfd45b10840cce363fcda291a62e56b1d4c8
Todd Poynor [Wed, 24 May 2017 03:28:15 +0000 (20:28 -0700)]
thermal: IThermal V1.1, add methods to send thermal throttling events
Add IThermalCallback and IThermal::registerThermalCallback() method.
Frameworks code calls this method to register a callback used by the
IThermal HAL implementation to send thermal events to the framework
ThermalService.
Bug:
30982366
Test: VtsHalThermalV1_1Target on marlin
manual test on marlin using marlin 1.1 HAL with modified
thermal-engine.conf and temporary debug code for notification
Change-Id: Ib49ad93a9495e3af515fced4e46f20186661fe07
(cherry picked from commit
cf964d79c1d41bba999e58cb476af099191a6634)