h264decoder: Add some debug messages

This commit is contained in:
Seungha Yang 2020-04-20 16:00:30 +09:00
parent e96435dfea
commit 6a7ea29084
2 changed files with 40 additions and 4 deletions

View file

@ -1130,6 +1130,9 @@ gst_h264_decoder_do_output_picture (GstH264Decoder * self,
return; return;
} }
GST_LOG_OBJECT (self, "Outputting picture %p (frame_num %d, poc %d)",
picture, picture->frame_num, picture->pic_order_cnt);
if (picture->pic_order_cnt < priv->last_output_poc) { if (picture->pic_order_cnt < priv->last_output_poc) {
GST_WARNING_OBJECT (self, GST_WARNING_OBJECT (self,
"Outputting out of order %d -> %d, likely a broken stream", "Outputting out of order %d -> %d, likely a broken stream",
@ -1157,8 +1160,14 @@ gst_h264_decoder_finish_current_picture (GstH264Decoder * self)
klass = GST_H264_DECODER_GET_CLASS (self); klass = GST_H264_DECODER_GET_CLASS (self);
if (klass->end_picture) { if (klass->end_picture) {
if (!klass->end_picture (self, priv->current_picture)) if (!klass->end_picture (self, priv->current_picture)) {
GST_WARNING_OBJECT (self,
"end picture failed, marking picture %p non-existing "
"(frame_num %d, poc %d)", priv->current_picture,
priv->current_picture->frame_num,
priv->current_picture->pic_order_cnt);
priv->current_picture->nonexisting = TRUE; priv->current_picture->nonexisting = TRUE;
}
} }
/* finish picture takes ownership of the picture */ /* finish picture takes ownership of the picture */
@ -1371,6 +1380,10 @@ gst_h264_decoder_sliding_window_picture_marking (GstH264Decoder * self)
return FALSE; return FALSE;
} }
GST_TRACE_OBJECT (self,
"Unmark reference flag of picture %p (frame_num %d, poc %d)",
to_unmark, to_unmark->frame_num, to_unmark->pic_order_cnt);
to_unmark->ref = FALSE; to_unmark->ref = FALSE;
gst_h264_picture_unref (to_unmark); gst_h264_picture_unref (to_unmark);
} }
@ -1409,6 +1422,13 @@ gst_h264_decoder_reference_picture_marking (GstH264Decoder * self,
* from DPB and how to mark/unmark existing reference pictures, do so. * from DPB and how to mark/unmark existing reference pictures, do so.
* Otherwise, fall back to default sliding window process */ * Otherwise, fall back to default sliding window process */
if (picture->dec_ref_pic_marking.adaptive_ref_pic_marking_mode_flag) { if (picture->dec_ref_pic_marking.adaptive_ref_pic_marking_mode_flag) {
if (picture->nonexisting) {
GST_WARNING_OBJECT (self,
"Invalid memory management operation for non-existing picture "
"%p (frame_num %d, poc %d", picture, picture->frame_num,
picture->pic_order_cnt);
}
return gst_h264_decoder_handle_memory_management_opt (self, picture); return gst_h264_decoder_handle_memory_management_opt (self, picture);
} }
@ -1470,7 +1490,8 @@ gst_h264_decoder_finish_picture (GstH264Decoder * self,
GstH264Picture *tmp = (GstH264Picture *) iter->data; GstH264Picture *tmp = (GstH264Picture *) iter->data;
GST_TRACE_OBJECT (self, GST_TRACE_OBJECT (self,
"\t%dth picture %p (poc %d)", i, tmp, tmp->pic_order_cnt); "\t%dth picture %p (frame_num %d, poc %d)", i, tmp,
tmp->frame_num, tmp->pic_order_cnt);
i++; i++;
} }
#endif #endif
@ -1486,7 +1507,8 @@ gst_h264_decoder_finish_picture (GstH264Decoder * self,
GstH264Picture *tmp = (GstH264Picture *) iter->data; GstH264Picture *tmp = (GstH264Picture *) iter->data;
GST_TRACE_OBJECT (self, GST_TRACE_OBJECT (self,
"\t%dth picture %p (poc %d)", i, tmp, tmp->pic_order_cnt); "\t%dth picture %p (frame_num %d, poc %d)", i, tmp,
tmp->frame_num, tmp->pic_order_cnt);
i++; i++;
} }
#endif #endif
@ -1766,12 +1788,16 @@ gst_h264_decoder_decode_slice (GstH264Decoder * self)
GstH264DecoderPrivate *priv = self->priv; GstH264DecoderPrivate *priv = self->priv;
GstH264Slice *slice = &priv->current_slice; GstH264Slice *slice = &priv->current_slice;
GstH264Picture *picture = priv->current_picture; GstH264Picture *picture = priv->current_picture;
gboolean ret;
if (!picture) { if (!picture) {
GST_ERROR_OBJECT (self, "No current picture"); GST_ERROR_OBJECT (self, "No current picture");
return FALSE; return FALSE;
} }
GST_LOG_OBJECT (self, "Decode picture %p (frame_num %d, poc %d)",
picture, picture->frame_num, picture->pic_order_cnt);
if (slice->header.field_pic_flag == 0) if (slice->header.field_pic_flag == 0)
priv->max_pic_num = priv->max_frame_num; priv->max_pic_num = priv->max_frame_num;
else else
@ -1779,5 +1805,12 @@ gst_h264_decoder_decode_slice (GstH264Decoder * self)
g_assert (klass->decode_slice); g_assert (klass->decode_slice);
return klass->decode_slice (self, picture, slice); ret = klass->decode_slice (self, picture, slice);
if (!ret) {
GST_WARNING_OBJECT (self,
"Subclass didn't want to decode picture %p (frame_num %d, poc %d)",
picture, picture->frame_num, picture->pic_order_cnt);
}
return ret;
} }

View file

@ -252,6 +252,9 @@ gst_h264_dpb_delete_by_poc (GstH264Dpb * dpb, gint poc)
g_array_index (dpb->pic_list, GstH264Picture *, i); g_array_index (dpb->pic_list, GstH264Picture *, i);
if (picture->pic_order_cnt == poc) { if (picture->pic_order_cnt == poc) {
GST_TRACE ("remove picture %p for poc %d (frame num %d) from dpb",
picture, poc, picture->frame_num);
g_array_remove_index (dpb->pic_list, i); g_array_remove_index (dpb->pic_list, i);
return; return;
} }