commit 1e9513e6f670202c675be0e0b6c88610ea4b5163
author Sungtak Lee <taklee@google.com> Tue Dec 12 07:14:40 2023 +0000
committer Sungtak Lee <taklee@google.com> Fri Jan 05 22:40:19 2024 +0000
tree a843f4111f98fc4f8415a708480a121b1a7c18de
parent 977176c221a533a1e786b9aca189d6825765374d

media.c2 aidl: minor fixes

C2 AIDL fixes.
- Allow setting a new output surface as the same surface
- Remove verbose logs from sync fence
- return C2_BLOCKING instead of C2_TIMED_OUT from C2IgbaBlockPool

Bug: 254050314
Change-Id: I4e15853e7fed0dc803db1204c541e7936624cbd3

media/codec2/hal/client/GraphicsTracker.cpp

diff --git a/media/codec2/hal/client/GraphicsTracker.cpp b/media/codec2/hal/client/GraphicsTracker.cpp
index ca2f6e6..01b0678 100644
--- a/media/codec2/hal/client/GraphicsTracker.cpp
+++ b/media/codec2/hal/client/GraphicsTracker.cpp
@@ -254,8 +254,7 @@
         ret = igbp->getUniqueId(&bqId);
     }
     if (ret != ::android::OK ||
-            prevCache->mGeneration == generation ||
-            (bqId != 0 && prevCache->mBqId == bqId)) {
+            prevCache->mGeneration == generation) {
         ALOGE("new surface configure fail due to wrong or same bqId or same generation:"
               "igbp(%d:%llu -> %llu), gen(%lu -> %lu)", (bool)igbp,
               (unsigned long long)prevCache->mBqId, (unsigned long long)bqId,