diff options
author | Ederson de Souza <ederson.desouza@intel.com> | 2019-10-30 15:24:40 -0700 |
---|---|---|
committer | GStreamer Merge Bot <gitlab-merge-bot@gstreamer-foundation.org> | 2019-11-19 13:35:00 +0000 |
commit | b09bc36160a1d0c01f6eb3cbc77247333d0daf3e (patch) | |
tree | b01cc8aeb3eef2ccd4b7011c01e1622ae35a629c /ext/avtp/gstavtpcvfdepay.c | |
parent | 31861b095ac54356cef26d0c644619c7aac7c054 (diff) | |
download | gstreamer-plugins-bad-b09bc36160a1d0c01f6eb3cbc77247333d0daf3e.tar.gz |
avtpcvfdepay: Tone down some log messages
Most of avtpcvfdepay messages are currently logged as warnings, which can
make some scenarios - such as receiving two AVTP streams on the same
pipeline - too verbose.
This patch tones those message down to INFO or DEBUG level - more in
sync with avtpaafdepay logging.
Diffstat (limited to 'ext/avtp/gstavtpcvfdepay.c')
-rw-r--r-- | ext/avtp/gstavtpcvfdepay.c | 34 |
1 files changed, 17 insertions, 17 deletions
diff --git a/ext/avtp/gstavtpcvfdepay.c b/ext/avtp/gstavtpcvfdepay.c index 4ca7572f4..03369caa3 100644 --- a/ext/avtp/gstavtpcvfdepay.c +++ b/ext/avtp/gstavtpcvfdepay.c @@ -267,7 +267,7 @@ gst_avtp_cvf_depay_validate_avtpdu (GstAvtpCvfDepay * avtpcvfdepay, gint r; if (G_UNLIKELY (map->size < AVTP_CVF_H264_HEADER_SIZE)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Incomplete AVTP header, expected it to have size of %zd, got %zd", AVTP_CVF_H264_HEADER_SIZE, map->size); goto end; @@ -277,8 +277,8 @@ gst_avtp_cvf_depay_validate_avtpdu (GstAvtpCvfDepay * avtpcvfdepay, r = avtp_pdu_get ((struct avtp_common_pdu *) pdu, AVTP_FIELD_SUBTYPE, &val32); g_assert (r == 0); - if (G_UNLIKELY (val32 != AVTP_SUBTYPE_CVF)) { - GST_WARNING_OBJECT (avtpcvfdepay, + if (val32 != AVTP_SUBTYPE_CVF) { + GST_DEBUG_OBJECT (avtpcvfdepay, "Unexpected AVTP header subtype %d, expected %d", val32, AVTP_SUBTYPE_CVF); goto end; @@ -287,7 +287,7 @@ gst_avtp_cvf_depay_validate_avtpdu (GstAvtpCvfDepay * avtpcvfdepay, r = avtp_pdu_get ((struct avtp_common_pdu *) pdu, AVTP_FIELD_VERSION, &val32); g_assert (r == 0); if (G_UNLIKELY (val32 != 0)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Unexpected AVTP header version %d, expected %d", val32, 0); goto end; } @@ -295,15 +295,15 @@ gst_avtp_cvf_depay_validate_avtpdu (GstAvtpCvfDepay * avtpcvfdepay, r = avtp_cvf_pdu_get (pdu, AVTP_CVF_FIELD_SV, &val); g_assert (r == 0); if (G_UNLIKELY (val != 1)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Unexpected AVTP header stream valid %ld, expected %d", val, 1); goto end; } r = avtp_cvf_pdu_get (pdu, AVTP_CVF_FIELD_STREAM_ID, &val); g_assert (r == 0); - if (G_UNLIKELY (val != avtpbasedepayload->streamid)) { - GST_WARNING_OBJECT (avtpcvfdepay, + if (val != avtpbasedepayload->streamid) { + GST_DEBUG_OBJECT (avtpcvfdepay, "Unexpected AVTP header stream id 0x%lx, expected 0x%lx", val, avtpbasedepayload->streamid); goto end; @@ -312,7 +312,7 @@ gst_avtp_cvf_depay_validate_avtpdu (GstAvtpCvfDepay * avtpcvfdepay, r = avtp_cvf_pdu_get (pdu, AVTP_CVF_FIELD_FORMAT, &val); g_assert (r == 0); if (G_UNLIKELY (val != AVTP_CVF_FORMAT_RFC)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Unexpected AVTP header format %ld, expected %d", val, AVTP_CVF_FORMAT_RFC); goto end; @@ -321,7 +321,7 @@ gst_avtp_cvf_depay_validate_avtpdu (GstAvtpCvfDepay * avtpcvfdepay, r = avtp_cvf_pdu_get (pdu, AVTP_CVF_FIELD_FORMAT_SUBTYPE, &val); g_assert (r == 0); if (G_UNLIKELY (val != AVTP_CVF_FORMAT_SUBTYPE_H264)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Unsupported AVTP header format subtype %ld", val); goto end; } @@ -329,7 +329,7 @@ gst_avtp_cvf_depay_validate_avtpdu (GstAvtpCvfDepay * avtpcvfdepay, r = avtp_cvf_pdu_get (pdu, AVTP_CVF_FIELD_STREAM_DATA_LEN, &val); g_assert (r == 0); if (G_UNLIKELY (map->size < sizeof (*pdu) + val)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "AVTP packet size %ld too small, expected at least %lu", map->size - AVTP_CVF_H264_HEADER_SIZE, sizeof (*pdu) + val); goto end; @@ -471,7 +471,7 @@ gst_avtp_cvf_depay_process_last_fragment (GstAvtpCvfDepay * avtpcvfdepay, GstFlowReturn ret = GST_FLOW_OK; if (G_UNLIKELY (avtpcvfdepay->fragments == NULL)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Received final fragment, but no start fragment received. Dropping it."); goto end; } @@ -564,7 +564,7 @@ gst_avtp_cvf_depay_handle_fu_a (GstAvtpCvfDepay * avtpcvfdepay, if (start) { if (G_UNLIKELY (avtpcvfdepay->fragments != NULL)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Received starting fragment, but previous one is not complete. Dropping old fragment"); gst_avtp_cvf_depay_push_and_discard (avtpcvfdepay); } @@ -576,7 +576,7 @@ gst_avtp_cvf_depay_handle_fu_a (GstAvtpCvfDepay * avtpcvfdepay, if (!start && !end) { if (G_UNLIKELY (avtpcvfdepay->fragments == NULL)) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Received intermediate fragment, but no start fragment received. Dropping it."); gst_avtp_cvf_depay_push_and_discard (avtpcvfdepay); goto end; @@ -608,7 +608,7 @@ gst_avtp_cvf_depay_handle_single_nal (GstAvtpCvfDepay * avtpcvfdepay, GST_DEBUG_OBJECT (avtpcvfdepay, "Handling single NAL unit"); if (avtpcvfdepay->fragments != NULL) { - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "Received single NAL unit, but previous fragment is incomplete. Dropping fragment."); gst_avtp_cvf_depay_push_and_discard (avtpcvfdepay); } @@ -647,7 +647,7 @@ gst_avtp_cvf_depay_chain (GstPad * pad, GstObject * parent, GstBuffer * buffer) gst_buffer_map (buffer, &map, GST_MAP_READ); if (!gst_avtp_cvf_depay_validate_avtpdu (avtpcvfdepay, &map)) { - GST_WARNING_OBJECT (avtpcvfdepay, "Invalid AVTPDU buffer, dropping it"); + GST_DEBUG_OBJECT (avtpcvfdepay, "Invalid AVTPDU buffer, dropping it"); goto end; } @@ -656,14 +656,14 @@ gst_avtp_cvf_depay_chain (GstPad * pad, GstObject * parent, GstBuffer * buffer) case STAP_B_TYPE: case MTAP16_TYPE: case MTAP24_TYPE: - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "AVTP aggregation packets not supported, dropping it"); break; case FU_A_TYPE: ret = gst_avtp_cvf_depay_handle_fu_a (avtpcvfdepay, buffer, &map); break; case FU_B_TYPE: - GST_WARNING_OBJECT (avtpcvfdepay, + GST_DEBUG_OBJECT (avtpcvfdepay, "AVTP fragmentation FU-B packets not supported, dropping it"); break; default: |