commit c7af1da30e5b73e9fc0c8933a47d88012dfaec83
author Yichi Chen <yichichen@google.com> Tue Dec 01 14:39:36 2020 +0800
committer Wonsik Kim <wonsik@google.com> Wed Dec 16 07:47:04 2020 +0000
tree 62d0e2366488c71d10ec59105940319046187250
parent 731c31ec84dc287aa383dbfaee27769a8cb1516a

ACodec: Reduce the TWOmxNode wrapper in GraphicBufferSource

The wrapper of TWOmxNode introduced the extra binder calls to its
creator and got the chance to send back the same request to the original
requester. To avoid the redundant binder transactions, the patch simply
remove the wrapper by using the internal HAL interface.

Bug: 173502220
Bug: 173661647
Test: Trace recording without redundant binder calls in emptyBuffer
Merged-In: I77ac10d12a15c4b07a37ac7e644400dba85a03a9
Change-Id: I77ac10d12a15c4b07a37ac7e644400dba85a03a9

media/libstagefright/ACodec.cpp

diff --git a/media/libstagefright/ACodec.cpp b/media/libstagefright/ACodec.cpp
index bc84d78..edbd99e 100644
--- a/media/libstagefright/ACodec.cpp
+++ b/media/libstagefright/ACodec.cpp
@@ -7035,10 +7035,9 @@
         return err;
     }
 
-    using hardware::media::omx::V1_0::utils::TWOmxNode;
     err = statusFromBinderStatus(
             mCodec->mGraphicBufferSource->configure(
-                    new TWOmxNode(mCodec->mOMXNode),
+                    mCodec->mOMXNode->getHalInterface<IOmxNode>(),
                     static_cast<hardware::graphics::common::V1_0::Dataspace>(dataSpace)));
     if (err != OK) {
         ALOGE("[%s] Unable to configure for node (err %d)",