Skip to content

LLAMA-18099 [LLAMA G2/G1/Cello][R37.046][R38.023][R40.001] High GST Decode Time leads to OTTPENDING #1416

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 1 commit into
base: support/2.11.1_8.4
Choose a base branch
from
+23 −3
Open

LLAMA-18099 [LLAMA G2/G1/Cello][R37.046][R38.023][R40.001] High GST Decode Time leads to OTTPENDING #1416

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
26 changes: 23 additions & 3 deletions middleware/InterfacePlayerRDK.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -553,6 +553,7 @@ static void GstPlayer_OnFirstVideoFrameCallback(GstElement* object, guint arg0,
{
InterfacePlayerPriv* privatePlayer = pInterfacePlayerRDK->GetPrivatePlayer();
HANDLER_CONTROL_HELPER_CALLBACK_VOID();
MW_LOG_INFO("TRACE_TUNE: GstPlayer_OnFirstVideoFrameCallback invoked");
privatePlayer->gstPrivateContext->firstVideoFrameReceived = true;
pInterfacePlayerRDK->NotifyFirstFrame(eGST_MEDIATYPE_VIDEO);

Expand All @@ -579,6 +580,7 @@ static void GstPlayer_OnAudioFirstFrameAudDecoder(GstElement* object, guint arg0
{
InterfacePlayerPriv* privatePlayer = pInterfacePlayerRDK->GetPrivatePlayer();
HANDLER_CONTROL_HELPER_CALLBACK_VOID();
MW_LOG_INFO("TRACE_TUNE: GstPlayer_OnAudioFirstFrameAudDecoder invoked");
privatePlayer->gstPrivateContext->firstAudioFrameReceived = true;
pInterfacePlayerRDK->NotifyFirstFrame(eGST_MEDIATYPE_AUDIO);
}
Expand Down Expand Up @@ -1176,6 +1178,7 @@ static std::string GetStatus(gpointer pElementOrBin, int& recursionCount, gpoint

case GST_STATE_CHANGE_NO_PREROLL:
returnStringBuilder +="*";
MW_LOG_INFO("TRACE_TUNE: GST_STATE_CHANGE_NO_PREROLL for element %s", name.c_str());
break;

default:
Expand Down Expand Up @@ -3181,7 +3184,10 @@ bool InterfacePlayerRDK::SendHelper(int type, const void *ptr, size_t len, doubl
else
{
GstFlowReturn ret = gst_app_src_push_buffer(GST_APP_SRC(stream->source), buffer);

if (ret == GST_FLOW_OK)
{
MW_LOG_INFO("TRACE_TUNE: Data pushed into pipeline for mediaType[%d] pts %" G_GUINT64_FORMAT " firstBuffer=%d initFragment=%d", mediaType, pts, isFirstBuffer, initFragment);
}
if (ret != GST_FLOW_OK)
{
MW_LOG_ERR("gst_app_src_push_buffer error: %d[%s] mediaType %d", ret, gst_flow_get_name (ret), (int)mediaType);
Expand Down Expand Up @@ -3767,11 +3773,19 @@ void InterfacePlayerRDK::NotifyFirstFrame(int mediaType)
bool notifyFirstBuffer = false;
bool audioOnly = false;
bool requireFirstVideoFrameDisplay = false;
MW_LOG_INFO("TRACE_TUNE: NotifyFirstFrame called mediaType[%d] firstFrameReceived=%d firstVideoFrameReceived=%d firstAudioFrameReceived=%d NumberOfTracks=%d firstFrameRequired=%d",
mediaType,
interfacePlayerPriv->gstPrivateContext->firstFrameReceived,
interfacePlayerPriv->gstPrivateContext->firstVideoFrameReceived,
interfacePlayerPriv->gstPrivateContext->firstAudioFrameReceived,
interfacePlayerPriv->gstPrivateContext->NumberOfTracks,
mFirstFrameRequired);
if (!interfacePlayerPriv->gstPrivateContext->firstFrameReceived && (interfacePlayerPriv->gstPrivateContext->firstVideoFrameReceived
|| (1 == interfacePlayerPriv->gstPrivateContext->NumberOfTracks && (interfacePlayerPriv->gstPrivateContext->firstAudioFrameReceived || interfacePlayerPriv->gstPrivateContext->firstVideoFrameReceived))))
{
interfacePlayerPriv->gstPrivateContext->firstFrameReceived = true;
notifyFirstBuffer = true;
MW_LOG_INFO("TRACE_TUNE: First frame signal state for mediaType[%d]: notifyFirstBuffer=%d", mediaType, notifyFirstBuffer);
PlayerLogManager::setLogLevel(mLOGLEVEL_WARN); //Align with player LogTuneComplete once the first frame starts, required for prod builds
}
if(notifyFirstFrameCallback)
Expand Down Expand Up @@ -4310,6 +4324,10 @@ static gboolean bus_message(GstBus * bus, GstMessage * msg, InterfacePlayerRDK *
}
if(isPlaybinStateChangeEvent && new_state == GST_STATE_PLAYING)
{
MW_LOG_INFO("TRACE_TUNE: GST bus event: playbin state changed to PLAYING, using_westerossink=%d pauseOnStartPlayback=%d",
privatePlayer->gstPrivateContext->using_westerossink,
privatePlayer->gstPrivateContext->pauseOnStartPlayback);

privatePlayer->gstPrivateContext->pauseOnStartPlayback = false;

busEvent.setPlaybackRate = privatePlayer->socInterface->SetPlatformPlaybackRate();
Expand Down Expand Up @@ -4953,8 +4971,10 @@ static GstBusSyncReply bus_sync_handler(GstBus * bus, GstMessage * msg, Interfac

break;
case GST_MESSAGE_ASYNC_DONE:

MW_LOG_INFO("Received GST_MESSAGE_ASYNC_DONE message");
MW_LOG_INFO("TRACE_TUNE: Received GST_MESSAGE_ASYNC_DONE message buffering_in_progress=%d pipelineState=%s",
privatePlayer->gstPrivateContext->buffering_in_progress,
gst_element_state_get_name(privatePlayer->gstPrivateContext->pipelineState));

if (privatePlayer->gstPrivateContext->buffering_in_progress)
{
privatePlayer->gstPrivateContext->bufferingTimeoutTimerId = g_timeout_add_full(BUFFERING_TIMEOUT_PRIORITY, DEFAULT_BUFFERING_TO_MS, buffering_timeout, pInterfacePlayerRDK, NULL);
Expand Down
Loading