commit dadaeae3639053303725ae6221ba61641d7def2d
author MingShu Pang <quic_mpang@quicinc.com> Thu Jun 06 17:52:54 2024 +0800
committer Huang, Yidong <quic_yidongh@quicinc.com> Wed Jun 19 10:07:05 2024 +0800
tree 4ef42795eb16c64c9a78f8c9a5cf96f5898ac368
parent 4f195983e77b3112fcbc72b60d1dca5f09cf5c93

libaudiohal: Decrease mLastReplyLifeTimeNs to update the reply more often

Currently the expire time is the minimum of the double of buffer size,
because regular transfers should update the reply more often,
so this is a fallback—and 100 ms in case the buffer is very large

AV sync in Nuplayer needs corresponding time even for cached
position, so sometimes timeNs can not be updated because of big
mLastReplyLifeTimeNs, that will cause audio timestamp can not be updated
timely.

Decrease mLastReplyLifeTimeNs and remove the double of buffer size
to update the reply more often.

Bug: 346698514
Test: AV sync test.

Change-Id: I66cfc4ae53cff0864141dae554c903ce10c8b87e
(cherry picked from 8882b4f9df720ee4542f6da5a414ab0da8079d59)

media/libaudiohal/impl/StreamHalAidl.cpp

diff --git a/media/libaudiohal/impl/StreamHalAidl.cpp b/media/libaudiohal/impl/StreamHalAidl.cpp
index 6c0dc76..12a8019 100644
--- a/media/libaudiohal/impl/StreamHalAidl.cpp
+++ b/media/libaudiohal/impl/StreamHalAidl.cpp
@@ -84,8 +84,8 @@
           mStream(stream),
           mVendorExt(vext),
           mLastReplyLifeTimeNs(
-                  std::min(static_cast<size_t>(100),
-                          2 * mContext.getBufferDurationMs(mConfig.sample_rate))
+                  std::min(static_cast<size_t>(20),
+                           mContext.getBufferDurationMs(mConfig.sample_rate))
                   * NANOS_PER_MILLISECOND)
 {
     ALOGD("%p %s::%s", this, getClassName().c_str(), __func__);