Change in libosmo-abis[master]: ortp: resynchronize rtp session on timestamp changes

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

dexter gerrit-no-reply at lists.osmocom.org
Wed May 30 09:15:05 UTC 2018


dexter has uploaded this change for review. ( https://gerrit.osmocom.org/9379


Change subject: ortp: resynchronize rtp session on timestamp changes
......................................................................

ortp: resynchronize rtp session on timestamp changes

When libortp detects a timestamp jump, we resynchronize the session
in order to restore proper rtp frame reception. However, we do not
resynchronize on an SSRC change. An SSRC change usually indicates
the change of an RTP stream source, which is a much more profound
event than a timestamp change, so we should resynchronize the
session on SSRC changes as well.

Also, not resynchronizing the session on an SSRC change causes
problems when the jitter buffer feature is used. In those cases
it takes libortp a long time until it finally detects the timestamp
jump and the session resyhcornization happens late.

Change-Id: I8688a55cd20d0e14493c357db80754851e12f2fb
Related OS#3299
---
M src/trau/osmo_ortp.c
1 file changed, 2 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/79/9379/1

diff --git a/src/trau/osmo_ortp.c b/src/trau/osmo_ortp.c
index 8ea05e4..8eecd9f 100644
--- a/src/trau/osmo_ortp.c
+++ b/src/trau/osmo_ortp.c
@@ -114,7 +114,8 @@
 	uint32_t ssrc = rtp_session_get_recv_ssrc(rs);
 
 	LOGP(DLMIB, LOGL_INFO,
-	     "osmo-ortp(%d): ssrc_changed to 0x%08x\n", port, ssrc);
+	     "osmo-ortp(%d): ssrc_changed to 0x%08x, resyncing\n", port, ssrc);
+	rtp_session_resync(rs);
 }
 
 static void ortp_sig_cb_pt(RtpSession *rs, void *data)

-- 
To view, visit https://gerrit.osmocom.org/9379
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8688a55cd20d0e14493c357db80754851e12f2fb
Gerrit-Change-Number: 9379
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180530/99fb8b79/attachment.htm>


More information about the gerrit-log mailing list