Commit graph

73 commits

Author SHA1 Message Date
Jiyong Park
e786494ff3 Remove TODO comment for configstore threadpool size
No problem with current configuration has been reported.

Bug: 34857894
Test: not required
Change-Id: Ie23b234eccb2707323032e37b27b387eb96918e5
2017-07-14 10:53:26 +09:00
Jae Shin
6e3ec61aab Add structural test VtsHalConfigstoreV1_1Target
Add vts test module for configstore@1.1

Test: make vts -j40 && run commandAndExit vts -m
VtsHalConfigstoreV1_1Target
Bug: 63071317

Change-Id: I405f4409bc2ac4acdf07728b950ba9b653eb3328
2017-07-03 13:47:11 +09:00
Jae Shin
97478a5ca0 Assert getService does not return nullptr
Make sure getService return value is not null before running
testcase

Test: make vts -j40 && vts-tradefed run commandAndExit vts -m
VtsHalConfigstoreV1_0Target
Bug:62931371

Merged-In: I432e07c0c61a308b814d88cab9027a95ff24d8b5
(cherry picked from commit 60ff24c62e)

Change-Id: Id9bb2a3fb2c4caf02c7e03c1529cefe2570c5453
2017-06-30 13:11:23 +09:00
Steven Moreland
f04fdeb83e Update internal makefiles.
These makefiles are not in dev topic branches
or AOSP currently so they were not updated. This
update adds hidl-generated-module defaults.

Test: pass
Change-Id: Ie52c54877d795bf5358aa600e6d1a13a4f166d25
2017-06-05 17:47:36 -07:00
Jaesoo Lee
eb72e4ff97 Merge "configstore-utils: add feature to cache service handles" am: 03b02609ff am: fbb8bb0671 am: c028065f5e
am: 1c231dfdad

Change-Id: I86407072bf4bbef94dc625dede248a2e9b29719f
2017-05-25 01:26:33 +00:00
Jaesoo Lee
1c231dfdad Merge "configstore-utils: add feature to cache service handles" am: 03b02609ff am: fbb8bb0671
am: c028065f5e

Change-Id: I30de198a55d1a283db949498e887a06f9565bbe2
2017-05-25 01:24:03 +00:00
Jaesoo Lee
056d77d523 configstore-utils: add feature to cache service handles
Previously, the service handle for the configstore needs to be retrieved
for every calls. This change reduces the time for retrieving multiple
configstore items from a same client by caching the service handle per a
configstore interface class.

Bug: 38411900
Test: built and tested on sailfish.
Change-Id: Iaff81e9f2b026aee0bd3865f5d5ed3fb7b6056e0
2017-05-24 10:04:32 +09:00
Steven Moreland
aeef430de6 Merge "Update makefiles." am: 9119d4ccd1 am: 242b2288d7 am: c6da54569f
am: 67b9ae7d9c

Change-Id: I8462abc85f4f87840c4cee13361290e42746ce82
2017-05-20 00:22:03 +00:00
Steven Moreland
67b9ae7d9c Merge "Update makefiles." am: 9119d4ccd1 am: 242b2288d7
am: c6da54569f

Change-Id: Ib7f7a9cdb64985d545443edac5bb8e9c169fc592
2017-05-20 00:19:40 +00:00
Steven Moreland
06c85d986a Update makefiles.
Adds default configuration to all hals.

Test: pass
Change-Id: Icdf6b56df82c4b7b7de815e7febea9a07ab96856
2017-05-18 16:14:16 -07:00
Jaesoo Lee
812e85e126 configstore: configstore HAL is up-revisioned to 1.1
This change provides a reference implementation of the configstore HAL
v1.1.

Bug: 37727469
Test: Built sailfish-userdebug and configstore-1.1 works
Change-Id: I75e7fd1da8e90ae48d779a3ba28957c5a93a5529
2017-05-17 15:30:17 +09:00
Jaesoo Lee
4c15eccb92 configstore: include types.h instead of ISurfaceFlingerConfigs.h am: 92c77bd6e7 am: 4b1848247f
am: d21a105380

Change-Id: I1a674bd0827b0dccdb01fded168cf3122de81afa
2017-05-12 05:16:36 +00:00
Jaesoo Lee
92c77bd6e7 configstore: include types.h instead of ISurfaceFlingerConfigs.h
Utils.h does not need to include ISurfaceFlingerConfigs.h.

Bug: 37727469
Test: successfully built sailfish-userdebug
Change-Id: I6650020bfc3b265d871ca3244b8900c336f98419
2017-05-11 09:37:03 +09:00
Steven Moreland
8e89a43590 Update makefiles for c++-sources and c++-headers. am: cb360c60d8 am: 4bd4e7fb3b
am: 127b116fe5

Change-Id: I3a2aa76acef0f0d1455bfad775e87e8089d357e8
2017-05-10 02:58:00 +00:00
Steven Moreland
cb360c60d8 Update makefiles for c++-sources and c++-headers.
Test: pass
Bug: 38174080
Change-Id: I683455736eed35e6ced24e2763c2bdb5fb6c9601
2017-05-09 14:34:19 -07:00
Steven Moreland
f90189a79f Merge "Remove redundant dependencies" am: 62769b7196 am: 170e2e85a2
am: 5fba1db1d2

Change-Id: I661fff75d140b4359af055a0f08f08ef33c889c3
2017-05-01 23:37:04 +00:00
Treehugger Robot
62769b7196 Merge "Remove redundant dependencies" 2017-05-01 23:19:25 +00:00
Steven Moreland
ee1d248ae2 Remove redundant dependencies
android.hidl.base@1.0 and android.hidlmanager@1.0 are built into libhidltransport.

Test: links
Bug: 33276472
Change-Id: I424371932af129bba9d58dcf8652c64297f35f05
2017-05-01 09:47:58 -07:00
Steven Moreland
c2af01d66b Merge "configstore-utils: vendor_available" am: c3e640c7f0 am: 759e1d28bd
am: 51ba6a1e43

Change-Id: I9fc5652422f983e37c195abec84ba594993d1407
2017-04-28 03:18:22 +00:00
Steven Moreland
65af0612ce configstore-utils: vendor_available
Test: with BOARD_VNDK_VERSION := current
Change-Id: I4fe86224d18bc7e7579c13f9273bc4efa25df1ea
2017-04-27 18:34:24 -07: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
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
Andreas Huber
6733baff6c 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:16:27 -07:00
Iliyan Malchev
1f8c6639e6 Merge "configstore: add startGraphicsAllocatorService" into oc-dev 2017-04-13 00:42:13 +00:00
Chia-I Wu
1224640571 configstore: add startGraphicsAllocatorService
Add ISurfaceFlingerConfigs::startGraphicsAllocatorService to control
whether SurfaceFlinger should start IAllocator service.

Bug: 36462585
Test: boots
Change-Id: I421413673246cc7c552bc9eeeb7a9ce2783faf9e
2017-04-11 20:57:26 -07:00
Steven Moreland
0dd5ba23fb Update makefiles.
Test: pass
Change-Id: I5cf42c8ff1e11207394e4852ba0bd03e4e3b7d2e
2017-04-11 16:41:07 -07:00
Steven Moreland
b2799c370f Update makefiles.
Test: pass
Change-Id: I5cf42c8ff1e11207394e4852ba0bd03e4e3b7d2e
2017-04-11 09:57:40 -07:00
Jaesoo Lee
a8959849ba print log message for values retrieved from configstore
This CL adds a code for printing log messages in clients-side utility
library functions (getXXX) for configstore so that the developer can
figure out which configuration values are actually retrieved from the configstore.

Bug: 36275627
Test: Built and check the log message appears

Change-Id: I7d5d80ca7a0317816c71016e013f9e73ad23ee08
Merged-In: I2c0895f8afbbb2947b62164acaf62a491c451dc0
(cherry picked from commit 0dc72ecbb7)
2017-04-06 11:10:03 +09:00
Pavel Maltsev
1c5704d08c Handle hidl transaction errors in ConfigStore
Test: surface flinger not failing, android now bootsup

Bug: b/36445794
Change-Id: I64cc404bec71f5e4eea2e0034f07b86fb60a3e32
Merged-In: I22fa7aab9fa92bc04333aaa0eef45891ebeba8e7
(cherry picked from commit 076b792ade608a7fdf4e600a42550ad17296a39e)
2017-04-06 11:10:03 +09:00
Jaesoo Lee
0481c3f717 removed log print statements for configstore from SurfaceFlingerConfigs.cpp
Log messages for the retrieved values from configstore will be
automatically printed at the client side (by configstore-utils).

Bug: 36275627
Test: Built and check the log messages
Change-Id: Id5033a74ef58778e93e39be6ac5b9e10ee28c5cf
2017-04-05 11:37:30 +09:00
Jaesoo Lee
0dc72ecbb7 print log message for values retrieved from configstore
This CL adds a code for printing log messages in clients-side utility
library functions (getXXX) for configstore so that the developer can
figure out which configuration values are actually retrieved from the configstore.

Bug: 36275627
Test: Built and check the log message appears

Change-Id: I2c0895f8afbbb2947b62164acaf62a491c451dc0
2017-04-05 11:37:30 +09:00
Keun-young Park
a232a45b59 add components necessary for bootanim to animation class
- necessary for running shutdown animation:
 android.hardware.graphics.composer
 android.hardware.graphics.allocator
 android.hardware.configstore

bug: 36526187
Test: many reboots
Change-Id: Iba3eefddd2bd33c28152c17081a4ad13619d330d
2017-03-31 16:56:08 -07:00
Andreas Huber
40d3a9bd8c Convert all comments into "doxygen-ready" comments.
Bug: 36453077
Test: mma
Change-Id: I0b1f77dfae5d2258969e33d85ecf45401ffbdfaa
2017-03-29 08:53:58 -07:00
Steven Moreland
53b21566bc Update makefiles. am: e90c31f125 am: f11981de68
am: 103bada0e1

Change-Id: Ibe6eeeacab5660a8cf70d59dd9a4864d6f70ab80
2017-03-27 17:51:28 +00:00
Steven Moreland
e90c31f125 Update makefiles.
Test: pass
Change-Id: I925baccbe51a32b7b7f3f66cf2918d017ee6b039
2017-03-24 21:50:45 -07:00
TreeHugger Robot
5306597978 Merge "Handle hidl transaction errors in ConfigStore" 2017-03-24 05:50:23 +00:00
Pavel Maltsev
a3e24b557c Handle hidl transaction errors in ConfigStore
Test: surface flinger not failing, android now bootsup

Bug: b/36445794
Change-Id: I22fa7aab9fa92bc04333aaa0eef45891ebeba8e7
2017-03-23 10:24:14 -07:00
Jaesoo Lee
00e7509c03 removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)
HIDL_FETCH_... is required only for passthrough mode, but Configstore
does not provide passthrough mode.

Bug: 35325577
Test: build and check configstore works

Change-Id: I35e916ad4a6c321303a3888dabb4884b17146e16
Merged-In: Iac3ed57568f18c3a8a7f7423a361b0fe45aa54ab
2017-03-23 10:13:15 +09:00
Jaesoo Lee
f54419bb54 removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)
HIDL_FETCH_... is required only for passthrough mode, but Configstore
does not provide passthrough mode.

Bug: 35325577
Test: build and check configstore works
Change-Id: Iac3ed57568f18c3a8a7f7423a361b0fe45aa54ab
2017-03-23 10:09:02 +09:00
TreeHugger Robot
565a418fba Merge "Create max acquired buffer for framebuffer for SF" 2017-03-22 18:51:31 +00:00
Jaesoo Lee
a43c1d3160 Merge "disabled passthrough mode for Configstore" am: 7f8b76a322 am: 37b5d80abf
am: 7af7140958

Change-Id: I98bb865ed1b5a28a4e57a4da849ef2db685f032c
2017-03-22 01:07:52 +00:00
Fabien Sanglard
3d9f5ad284 Create max acquired buffer for framebuffer for SF
Change-Id: I502cfea9e03943c2f2bb353e71cfd860b2797fd5
2017-03-21 15:53:52 -07:00
Steven Thomas
cb4172a98c Revert "Revert "Add useVrFlinger() method to SurfaceFlinger ConfigStore""
This reverts commit 2877c167ff.

Add useVrFlinger() method to SurfaceFlinger ConfigStore, which is
used to ensure we only start vr flinger on Daydream ready devices.

Bug: 35885165

Test: Checked for "SurfaceFlinger UseVrFlinger" in the logs,
confirmed it only shows up for Daydream ready devices.

Change-Id: I839b650b63cd008de03feb4c771b330374bd0682
2017-03-21 19:56:57 +00:00
Jaesoo Lee
2d66f6183a disabled passthrough mode for Configstore
This change removes target "android.hardware.configstore@1.0-impl" to
disable passthrough mode.

Bug: 35325577
Test: android.hardware.configstore@1.0-impl.so not exist in outputs and
    configstore works in binderized mode

Change-Id: Ieedb6fea179d50ac35fc32b0e21752b2334558ae
2017-03-21 14:48:39 +09:00
Jin Qian
eb6540a751 Merge "Revert "Add useVrFlinger() method to SurfaceFlinger ConfigStore"" 2017-03-21 00:19:33 +00:00
Jin Qian
2877c167ff Revert "Add useVrFlinger() method to SurfaceFlinger ConfigStore"
This reverts commit d78f6b34b6.

broke multiple builds, e.g.
https://android-build.googleplex.com/builds/submitted/3833064/test_suites_x86_64_fastbuild3d_linux/latest/logs

Change-Id: Id0dea5b63fad8896a186486d137d8505aad12c26
2017-03-21 00:09:09 +00:00
Steven Thomas
84e9b09f5d Merge "Add useVrFlinger() method to SurfaceFlinger ConfigStore" 2017-03-20 23:43:53 +00:00
Steven Thomas
d78f6b34b6 Add useVrFlinger() method to SurfaceFlinger ConfigStore
Add useVrFlinger() method to SurfaceFlinger ConfigStore, which is
used to ensure we only start vr flinger on Daydream ready devices.

Bug: 35885165

Test: Checked for "SurfaceFlinger UseVrFlinger" in the logs,
confirmed it only shows up for Daydream ready devices.

Change-Id: I3c39b1f094e28f06b510171fc0968cf71f5d5550
2017-03-20 14:15:26 -07:00
Steven Moreland
3d0244bd9d Merge "configstore: respect service registration status." am: e8e87df57f am: 9761a112e1
am: 5f44edfb21

Change-Id: I9db78e30ec60088e09384a8134304f0f73bd391e
2017-03-20 17:27:59 +00:00
Steven Moreland
f88a9fe446 configstore: respect service registration status.
Bug: 36424561
Test: pass
Change-Id: I546af49d589e455b6dd97ca45d2330aa82059a4a
2017-03-19 14:39:36 -07:00