Skip to content

RDKEMW-21923 : [LLAMA /Cello]Observed the message "Sorry, fast forward isn't available for this programme followed by techanical fault error" while performing fast scrubbing by long-pressing the Right and Left navigation keys. #216

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
rekhap2kandhavelan wants to merge 3 commits into
base: support/0.2.0_8.6
Choose a base branch
from
Open

RDKEMW-21923 : [LLAMA /Cello]Observed the message "Sorry, fast forward isn't available for this programme followed by techanical fault error" while performing fast scrubbing by long-pressing the Right and Left navigation keys. #216

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2864e81
Handling pause pause wedge case
rkandh015 Jul 28, 2026
151da1f
Handle wedge pause-pause conditions
rkandh015 Jul 31, 2026
b001776
The fix adds a second check for PLAYING→PAUSED timeout
rkandh015 Aug 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 36 additions & 2 deletions InterfacePlayerRDK.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -505,6 +505,25 @@ void InterfacePlayerRDK::ConfigurePipeline(int format, int audioFormat, int subF
gst_context_unref(context);
}

// non-blocking state query — if pipeline is wedged in PAUSED->PAUSED,
// force NULL reset BEFORE issuing any new state transition.
// This prevents the double-wedge where ConfigurePipeline is called from a
// recovery re-seek while the pipeline is still physically stuck in PAUSED.
{
GstState cur = GST_STATE_NULL, pend = GST_STATE_NULL;
gst_element_get_state(interfacePlayerPriv->gstPrivateContext->pipeline,
&cur, &pend, 0 /* non-blocking */);
if (cur == GST_STATE_PAUSED && pend == GST_STATE_PAUSED)
{
MW_LOG_WARN("ConfigurePipeline: pipeline wedged (PAUSED->PAUSED), "
"forcing NULL reset before state transition");
SetStateWithWarnings(interfacePlayerPriv->gstPrivateContext->pipeline, GST_STATE_NULL);
// Wait up to 200ms for NULL to complete before proceeding
gst_element_get_state(interfacePlayerPriv->gstPrivateContext->pipeline,
&cur, &pend, 200 * GST_MSECOND);
}
}

if (interfacePlayerPriv->gstPrivateContext->pauseOnStartPlayback && GST_NORMAL_PLAY_RATE == interfacePlayerPriv->gstPrivateContext->rate)
{
MW_LOG_INFO("Setting state to GST_STATE_PAUSED - pause on playback enabled");
Expand Down Expand Up @@ -3453,11 +3472,11 @@ static GstState validateStateWithMsTimeout( InterfacePlayerRDK *pInterfacePlayer
float timeout = 100.0;
InterfacePlayerPriv* privatePlayer = pInterfacePlayerRDK->GetPrivatePlayer();
gint gstGetStateCnt = GST_ELEMENT_GET_STATE_RETRY_CNT_MAX;

GstStateChangeReturn ret = GST_STATE_CHANGE_FAILURE;
do
{
if ((GST_STATE_CHANGE_SUCCESS
== gst_element_get_state(privatePlayer->gstPrivateContext->pipeline, &gst_current, &gst_pending, timeout * GST_MSECOND))
== ( ret = gst_element_get_state(privatePlayer->gstPrivateContext->pipeline, &gst_current, &gst_pending, timeout * GST_MSECOND)) )
&& (gst_current == stateToValidate))
{
GST_WARNING(
Expand All @@ -3471,6 +3490,21 @@ static GstState validateStateWithMsTimeout( InterfacePlayerRDK *pInterfacePlayer

MW_LOG_ERR("validateStateWithMsTimeout - PIPELINE gst_element_get_state - FAILURE : State = %d, Pending = %d",
gst_current, gst_pending);

if (ret == GST_STATE_CHANGE_ASYNC && gst_current == GST_STATE_PAUSED && gst_pending == GST_STATE_PAUSED)
{
MW_LOG_WARN("validateStateWithMsTimeout: PAUSED->PAUSED wedged detected, returning error to let AAMP recover");
return GST_STATE_VOID_PENDING;
}

if (ret == GST_STATE_CHANGE_ASYNC && gst_current == GST_STATE_PLAYING && gst_pending == GST_STATE_PAUSED)
{
MW_LOG_WARN("validateStateWithMsTimeout: PLAYING->PAUSED transition timed out "
"(State=%d, Pending=%d), returning error to let AAMP recover",
gst_current, gst_pending);
return GST_STATE_VOID_PENDING;
}

return gst_current;
}

Expand Down