commit 4ee43026073b6f983be4a508fdc7b3600150e7fe
author wentao.ma <wentao.ma@amlogic.com> Wed Dec 14 13:22:57 2022 +0800
committer wentao ma <wentao.ma@amlogic.com> Wed Dec 14 22:13:56 2022 -0800
tree a2d6f5d7067d2dfb5c215ff60e572d6df1175b14
parent 292380e063c418a5d4e4e7551e6a8a3700a9333c

libdvr: video is freeze when fast backward to beginning [1/1]

PD#SWPL-103956

Problem:
Video freezes when fast backward to beginning.

Solution:
An fffb_replay is introduced to help finish the last FB play at beginning
position of a recording. In addition to function correctness, another
benefit is that after play, demux buffer is cleared due to stopVideoDecoding
invocation in the process. The following resume operation will not be
affected by the invalid cache length.

Verify:
Locally tested OK in AndroidT environment.

Signed-off-by: Wentao.MA <wentao.ma@amlogic.com>
Change-Id: I48a423a677563e4f285157635118e5300e775261

src/dvr_playback.c

diff --git a/src/dvr_playback.c b/src/dvr_playback.c
index e03dc5f..ede6eb9 100644
--- a/src/dvr_playback.c
+++ b/src/dvr_playback.c
@@ -1505,6 +1505,8 @@
       //DVR_PB_INFO("write  write_success:%d input_buffer.buf_size:%d", write_success, input_buffer.buf_size);
     } else {
       pthread_mutex_unlock(&player->segment_lock);
+      DVR_PB_DEBUG("write time out write_success:%d buf_size:%d systime:%u",
+          write_success, input_buffer.buf_size, _dvr_time_getClock());
       write_success = 0;
       if (CONTROL_SPEED_ENABLE == 1) {
 check1:
@@ -3227,7 +3229,7 @@
   *id = player->cur_segment_id;
 
   DVR_PB_INFO("***get playback slider position within segment. segment_id [%lld],"
-      " segment_slider_pos[%7d ms] = segment_read_pos[%7lld ms] - tsplayer_cache_len[%5lld ms],"
+      " segment_slider_pos[%7d ms] = segment_read_pos[%7lld ms] - tsplayer_cache_len[%5ld ms],"
       " last id [%lld] pos [%lld]",
       player->cur_segment_id,cur_time,cur,cache,player->last_send_time_id,pos);
 
@@ -3582,6 +3584,16 @@
     //for fb cmd, we need open pre segment.if reach first one segment, send begin event
     int ret = _change_to_next_segment((DVR_PlaybackHandle_t)player);
     if (ret != DVR_SUCCESS && IS_FB(player->speed)) {
+
+      // An fffb_replay is required here to help finish the last FB play
+      // at beginning position of a recording. In addition to function
+      // correctness, another benefit is that after play, demux buffer
+      // is cleared due to stopVideoDecoding invocation in the process.
+      // The following resume operation will not be affected by the invalid
+      // cache length.
+      player->next_fffb_time =_dvr_time_getClock() + FFFB_SLEEP_TIME;
+      _dvr_playback_fffb_replay(handle);
+
       dvr_mutex_unlock(&player->lock);
       DVR_PB_DEBUG("unlock");
       dvr_playback_pause(handle, DVR_FALSE);