From 50b5cdfecc84c79b92e60e42cab0070fa1a982ec Mon Sep 17 00:00:00 2001 From: Chih-Yu Huang Date: Wed, 22 Jul 2020 12:37:00 +0900 Subject: plugin_store: Switch to use C2VdaPooledBlockPool The V4L2 stateful API requires the caller pass the same buffers to the output queue slot. Originally we use C2PooledBlockPool for byte buffer mode, but C2PooledBlockPool cannot get buffer identifier. This CL switches to use C2VdaPooledBlockPool, which guarantees to return a fixed set of buffers and could query the buffer index. Bug: 161770200 Test: pass e2e test Change-Id: Ief2bdd9ceb6655cd47627ccf6c239a1e6d73482e --- components/V4L2DecodeComponent.cpp | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) (limited to 'components/V4L2DecodeComponent.cpp') diff --git a/components/V4L2DecodeComponent.cpp b/components/V4L2DecodeComponent.cpp index 8a86466..0f59d79 100644 --- a/components/V4L2DecodeComponent.cpp +++ b/components/V4L2DecodeComponent.cpp @@ -275,12 +275,7 @@ void V4L2DecodeComponent::getVideoFramePool(std::unique_ptr* poo return; } - // TODO(b/160307705): Consider to remove the dependency of C2VdaBqBlockPool. - if (blockPool->getAllocatorId() == C2PlatformAllocatorStore::BUFFERQUEUE) { - reinterpret_cast(blockPool.get())->requestNewBufferSet(numBuffers); - } - - *pool = VideoFramePool::Create(std::move(blockPool), size, pixelFormat, mIsSecure, + *pool = VideoFramePool::Create(std::move(blockPool), numBuffers, size, pixelFormat, mIsSecure, mDecoderTaskRunner); } -- cgit v1.2.3