summaryrefslogtreecommitdiffstats
path: root/media
diff options
context:
space:
mode:
authorSungtak Lee <taklee@google.com>2018-09-14 14:57:02 -0700
committerSungtak Lee <taklee@google.com>2018-09-18 17:45:36 -0700
commit5c8e455d732c9952a7e58bff56df35ce7c31c3bc (patch)
treeb992c79b9f0283365034357124b85fd82ca5e3e5 /media
parent51a04b2fe26a5b3941af56cf836def3c2f37477f (diff)
downloadandroid_hardware_interfaces-5c8e455d732c9952a7e58bff56df35ce7c31c3bc.tar.gz
android_hardware_interfaces-5c8e455d732c9952a7e58bff56df35ce7c31c3bc.tar.bz2
android_hardware_interfaces-5c8e455d732c9952a7e58bff56df35ce7c31c3bc.zip
Change buffer invalidataion FMQ to unsync
Buffer Invalidation FMQ is designed as a shared FMQ between clients in order to minimize memory consumption. In that case, regarding FMQ is unsync FMQ in order to not to block writer in any situation. Bug: 112203066 Change-Id: I58989ccd332ba790193007f43dd66ecbd617f484
Diffstat (limited to 'media')
-rw-r--r--media/bufferpool/2.0/IAccessor.hal2
-rw-r--r--media/bufferpool/2.0/types.hal2
2 files changed, 2 insertions, 2 deletions
diff --git a/media/bufferpool/2.0/IAccessor.hal b/media/bufferpool/2.0/IAccessor.hal
index bd7094594..ab7c02d45 100644
--- a/media/bufferpool/2.0/IAccessor.hal
+++ b/media/bufferpool/2.0/IAccessor.hal
@@ -68,5 +68,5 @@ interface IAccessor {
generates (ResultStatus status, IConnection connection,
int64_t connectionId,
fmq_sync<BufferStatusMessage> toFmqDesc,
- fmq_sync<BufferInvalidationMessage> fromFmqDesc);
+ fmq_unsync<BufferInvalidationMessage> fromFmqDesc);
};
diff --git a/media/bufferpool/2.0/types.hal b/media/bufferpool/2.0/types.hal
index 7ce53b18b..456e4aabd 100644
--- a/media/bufferpool/2.0/types.hal
+++ b/media/bufferpool/2.0/types.hal
@@ -100,7 +100,7 @@ struct BufferStatusMessage {
struct BufferInvalidationMessage {
/**
* Buffers from fromBufferId to toBufferId must be invalidated.
- * Both of fromBufferId and toBufferId are inclusive.
+ * fromBufferId is inclusive, but toBufferId is not inclusive.
* If fromBufferId > toBufferID, wrap happens. In that case
* the wrap is based on UINT32_MAX.
*/