Commit Graph

3886 Commits

Author SHA1 Message Date
Shawn Willden
fa67c8fc1d Fix Keymaster HAL crash.
Test: VTS test validates this fix
Change-Id: I5996c2166cf4141c97424df64a380f45b001d52e
2017-04-24 08:11:07 -06:00
Eric Laurent
30f78a13b0 Merge "Audio primary hal: fix member function used before init" into oc-dev 2017-04-19 17:43:49 +00:00
Kevin Rocard
34b44cd7db Audio primary hal: fix member function used before init
prepare for write was refactor by Ib4170eb6a9f88f9352d0912083b43d600771bb8e
to use temporary variable in a rai pattern.
Nevertheless the member variable was still used too early in
prepareForRead resulting in a crash for capture.

Test: assistant, camcorder
Bug: 37492059

Change-Id: I08140834959c440798cd40700dd089adca2f3e40
Signed-off-by: Kevin Rocard <krocard@google.com>
2017-04-19 09:38:42 -07:00
TreeHugger Robot
c10abf09f7 Merge "Fix test case for setSimCardPower radio api." into oc-dev 2017-04-19 03:04:03 +00:00
TreeHugger Robot
d7e88525a7 Merge changes from topic 'vts-audio-fix' into oc-dev
* changes:
  Audio HAL: Destroy EventFlag on failed prepareTo{write,read}
  Audio HAL: Check for buffer size overflow
  Audio HAL: Detect buffer memory allocation failure
  Audio HAL: Detect openDevice failure
  Audio HAL VTS: Fix documentation
2017-04-19 00:44:34 +00:00
TreeHugger Robot
2e25e562fa Merge "Use IBinder to identify callbacks in VHAL" into oc-dev 2017-04-18 23:23:10 +00:00
Pavel Maltsev
41a000da58 Use IBinder to identify callbacks in VHAL
Right now there's no way to distinguish multiple clients, thus we cannot
associate IVehicleCallback from subscribe and unsubscribe method. As a
temporary solution we will use pointer to IBinder from underlying proxy
object until HILD will have some API to do that.

Test: verified unit tests run, checked that subsequent 'subscribe' calls
do not create new client objects in VHAL

Bug: b/37254245
Change-Id: I029564b7b03e4821287c2110d0929a5c4cbce0ce
2017-04-18 14:20:11 -07:00
TreeHugger Robot
41c639f7bc Merge "audio hal: explicitly import headers" into oc-dev 2017-04-18 13:07:01 +00:00
TreeHugger Robot
c45ef5f15a Merge "Write VTS target test for Configstore HAL" into oc-dev 2017-04-18 05:49:42 +00:00
PO HUNG CHEN
fdd891393c Merge "Add IScheduleTest for VTS libhwbinder scheduler test" into oc-dev 2017-04-18 02:20:32 +00:00
TreeHugger Robot
206e38a4e8 Merge "Remove dependency of libhardware_legacy from HAL services" into oc-dev 2017-04-18 00:26:33 +00:00
Sanket Padawe
f9723eef1f Fix test case for setSimCardPower radio api.
Bug: 35800570
Test: vts
Change-Id: I93e78c46971158cd14666931b527d8e268e0a39e
2017-04-17 17:07:32 -07:00
Steven Moreland
a8a9cc6606 audio hal: explicitly import headers
Test: works with BOARD_VNDK_VERSION := current
Bug: 33241851
Change-Id: I34983a5527a16e8e9f5a6a87edac0b741aeb67d6
2017-04-17 15:56:52 -07:00
TreeHugger Robot
69c42429d1 Merge "Adding a custom property to the default VHAL impl" into oc-dev 2017-04-17 20:57:38 +00:00
Pavel Maltsev
794fc4f9ff Adding a custom property to the default VHAL impl
This property is supposed to be used mostly from e2e tests (a simple
test-case is provided in separate CL)

Test: make -j && runtest -x packages/services/Car/tests/vehiclehal_test/

Bug: b/36510399
Change-Id: I09b24f22ab328eee1ef6add60901ed03bf046874
2017-04-17 10:39:27 -07:00
Howard Chen
812f5d3101 Add IScheduleTest for VTS libhwbinder scheduler test
Add IScheduleTest.hal:
    send(cfg, callerSta)
cfg: 1 for verbose output
caller_status:  (caller_rt_priority <<16 )| caller_cpu

with the return value defined as:
((1 if no priority inheritance)<<16) | (1 if no cpu sync)
The implementation is in ScheduleTest.cpp

Bug:36705188
Test: sailfish/prebuilt kernel/oc-dev with the libhwbinder_latency
  vts test case
Change-Id: Ie2a837c8d0d4fa95c6fd0ebd50e76412cb808df8
2017-04-17 14:34:34 +08:00
Jae Shin
cc8b5c1c48 Write VTS target test for Configstore HAL
Bug: 35915192
Test: make vts -j40 && vts-tradefed run commandAndExit vts
--module VtsHalConfigstoreV1_0Target

Change-Id: I85692a0cf140b62adaa6b5384fea6d3b4090ba98
2017-04-17 13:10:36 +09:00
TreeHugger Robot
ba2420a87f Merge "Remove dependency of libhardware_legacy" into oc-dev 2017-04-17 03:05:10 +00:00
TreeHugger Robot
5bd796b649 Merge "dumpstate hal: use vendor toybox" into oc-dev 2017-04-16 14:00:49 +00:00
TreeHugger Robot
c68698337c Merge "VtsHalDrmV1_0Target test failure" into oc-dev 2017-04-15 07:53:27 +00:00
Hung-ying Tyan
593eeb7a4b Remove dependency of libhardware_legacy
It is not used by bluetooth and nfc services.
Bug: 33241851
Bug: 37305530
Test: built and tested on sailfish

Change-Id: I599073296baa5eee72a1fce8caa4eb8352f66218
Merged-In: I599073296baa5eee72a1fce8caa4eb8352f66218
2017-04-15 13:43:36 +08:00
Iliyan Malchev
554df3738d gnss@1.0-service: drop inet group, add gps group
Both system and gps groups might be relevant to the implementation of a
GNSS HAL.

b/36651420 [QCOM Treble] marlin/sailfish location daemon uses Binder
b/36085168 STOPSHIP - HAL (GNSS) cannot have internet access
b/35757613 STOPSHIP:  O GNSS TTFF - Regressed to Cold Start (XTRA
	   broken) (revert SELinux Change)

Test: marlin

Change-Id: I8aa5b18ef094b8540e7838174154f060a0c3fa9a
Merged-In: I8aa5b18ef094b8540e7838174154f060a0c3fa9a
Signed-off-by: Iliyan Malchev <malchev@google.com>
2017-04-15 05:12:55 +00:00
Hung-ying Tyan
4f1bbf7074 Remove dependency of libhardware_legacy from HAL services
It is not used by these services.

Bug: 33241851
Bug: 37305530
Test: built and tested on sailfish
Change-Id: I1f3f51981d7dfcf27e18d9495c747b7662867d31
2017-04-15 10:17:16 +08:00
Sandeep Patil
fd472b82f5 dumpstate hal: use vendor toybox
Bug: 36463595
Test: Dumpstate works

Change-Id: I9f79924fe6c2c7624f97ce24dcf95f61ee165146
Signed-off-by: Sandeep Patil <sspatil@google.com>
2017-04-14 19:14:42 -07:00
TreeHugger Robot
939c906a4f Merge "Use fmq for camera capture result." into oc-dev 2017-04-14 23:42:54 +00:00
Jeff Tinker
fdde895c34 Merge "Skip tests when the drm scheme is not installed" into oc-dev 2017-04-14 23:41:49 +00:00
Jeff Tinker
779678f559 VtsHalDrmV1_0Target test failure
The DrmHalVendorPluginTest.RestoreKeysClosedSession test wasn't
taking into account the policy of the asset, so both streaming
and offline assets were included in the test which is incorrect.

Test: drm vts test passes

bug:37358379
Change-Id: I41116e77a1ae306bb1a484e3714360c8553590f0
2017-04-14 16:13:19 -07:00
TreeHugger Robot
04b086dd08 Merge "setActiveGroup null, 0 length, unwritable dir should fail" into oc-dev 2017-04-14 23:10:32 +00:00
TreeHugger Robot
007d23e7cb Merge "Add testcases for restoreKeys API." into oc-dev 2017-04-14 21:38:30 +00:00
Yifan Hong
993e3d0286 Use fmq for camera capture result.
Test: camera works.

Bug: 35788245 [Treble] Pass camera metadata using shared memory

Change-Id: Ia8ba8ac4c8f7d1a89035f549c0281a30190554bb
2017-04-14 14:28:48 -07:00
Kevin Chyn
d7e9b0fdc7 setActiveGroup null, 0 length, unwritable dir should fail
with this change, VTS test needs to run in /data/system/
due to SELinux policy + Linux directory permissions

Test: run VTS tests and make sure they all pass
Fixes 37153318

Change-Id: Ib1f19ce7fc0d08ae2941aec5c08c5857c7a9f2fd
2017-04-14 14:10:19 -07:00
Jeff Tinker
be310b2b77 Skip tests when the drm scheme is not installed
Not all drm hal implmentations will be present on every device,
but the vendor modules will be. This change allows tests to be skipped
when a drm hal implementation isn't present.

Test: drm vts tests pass with widevine on a gms
device (tests executed) and tests pass on an aosp
device (tests skipped).

Change-Id: I0b87f501a45103960193279eb5b4b2cd1c113acd
2017-04-14 13:28:32 -07:00
TreeHugger Robot
e1df499267 Merge "soundtrigger: Use "default" instance name for HAL service" into oc-dev 2017-04-14 18:24:16 +00:00
TreeHugger Robot
580125c253 Merge "Route DRM HAL libbinder traffic to /dev/vndbinder" into oc-dev 2017-04-14 18:20:02 +00:00
Iliyan Malchev
2dd67a43b6 Route GNSS HAL libbinder traffic to /dev/vndbinder
Bug: 36565056
Test: no more gnss crash at boot

Change-Id: Ib3935beeb521dc655c5d7e1b6d7680a190b3fd4e
Signed-off-by: Iliyan Malchev <malchev@google.com>
2017-04-14 16:01:30 +00:00
Mikhail Naganov
3acaa668c1 soundtrigger: Use "default" instance name for HAL service
"default" implies using "primary" legacy device module.
This doesn't change anything for binderized mode, because
the server was only registering itself as "sound_trigger.primary",
so it was the only possibility anyways.

Usage of "stub" (debugging) module is still possible
if audioflinger and corresponding services are built with
USE_LEGACY_LOCAL_AUDIO_HAL=true (added makefile check to ensure
that).

Test: "Ok Google" from screen off state on sailfish
Bug: 36570720
Change-Id: I88f1ebd4c1efd0f91e37458746c336a4beef33ae
2017-04-14 15:52:11 +00:00
TreeHugger Robot
be31e7c571 Merge "Added missing RenderScript HIDL API call to VTS test." into oc-dev 2017-04-14 15:45:43 +00:00
Iliyan Malchev
136b5e46ec Route DRM HAL libbinder traffic to /dev/vndbinder
b/37309712
Test: marlin

Change-Id: I72c2304842bd21b5393c17be5beeeb90a74d5345
Signed-off-by: Iliyan Malchev <malchev@google.com>
2017-04-14 15:25:16 +00:00
Shawn Willden
dc9965665e Require attestation app ID.
Bug: 37318025
Test: Manually tested
Change-Id: Iaa992c8d22e0c88c2a2570355199befa484adc19
2017-04-14 11:38:17 +00:00
Shawn Willden
4b93bb21dc Fix handling of KM3 attestations.
Test: Manually tested
Bug: 37318025
Change-Id: I91c0c0fe6d36fb4a02d6b58a3b0836a6f8996012
2017-04-14 11:37:42 +00:00
Ryan Campbell
6799c3a5c5 Add testcases for restoreKeys API.
Add a positive and two negative test cases for the restoreKeys
function in DRM vendor plugin.

Test: vts-tradefed run commandAndExit vts -m VtsHalDrmV1_0Target
--primary-abi-only --skip-all-system-status-check
Bug: 34178477
Change-Id: Ia7f323dbd92e7281efa96b9b7d07b85123e819f4
2017-04-14 00:18:16 -07:00
TreeHugger Robot
5df7894213 Merge "audio.common@2.0-util: vendor_available + in Soong" into oc-dev 2017-04-14 04:00:09 +00:00
TreeHugger Robot
411268f5ae Merge "Move bluetooth test to vendor." into oc-dev 2017-04-14 03:59:46 +00:00
TreeHugger Robot
8afd952d48 Merge "Changed the naming scheme for Treble java libraries (both static and dynamic)" into oc-dev 2017-04-14 02:13:28 +00:00
Yifan Hong
84deaeb4f9 Merge "Use fmq for camera capture request." into oc-dev 2017-04-14 02:10:16 +00:00
TreeHugger Robot
a1a42dffd0 Merge "fingerprint: clean up includes" into oc-dev 2017-04-14 01:14:22 +00:00
Michael Butler
3f986a00cd Added missing RenderScript HIDL API call to VTS test.
ScriptGroupSetInput was never called, so it was added to the
ScriptGroupTest. After this CL gets merged, all RenderScript HIDL API
calls will be tested.

Bug: 35915961

Test: mm on oc-dev, runs and passes on oc-dev
Change-Id: I9153895ac55be757eef59e81837a9518fa9e85ab
2017-04-13 17:47:58 -07:00
TreeHugger Robot
e3c0b02d33 Merge changes I52c156ed,I9fb9a924 into oc-dev
* changes:
  Add VTS tests for Wfd
  Add VTS tests for Bonjour and Upnp service
2017-04-14 00:13:17 +00:00
Steven Moreland
750ede4cf9 fingerprint: clean up includes
Bug: 33241851
Test: links
Test: works with BOARD_VNDK_VERSION := current
Change-Id: I7cfec2b8459964edf9a180d01cfdd90dae376fb9
2017-04-13 16:13:47 -07:00
Andreas Huber
6fa374ad30 Changed the naming scheme for Treble java libraries (both static and dynamic)
android.hardware.foo@1.0-java => android.hardware.foo-V1.0-java

Bug: 37207894
Test: make
Change-Id: I177f4cfd42f635aaf096107af8309d1a82ed41c0
Merged-In: I8b91881dfaf507c0d852d56ebbb53ff0987e5a34
2017-04-13 15:23:46 -07:00