No it also have a call to rtp_session_resync(rs); - look at the diff. And it does not fix 1803 alone, only in combination with other fixes.

On 10/12/2016 01:37 PM, Neels Hofmeyr wrote:

In below patch (already merged) there's apparently only a LOGP tweak. Looks like something is missing there and Harald didn't spot it? It can't possibly fix OS#1803, can it?

~Neels

On Wed, Oct 12, 2016 at 09:26:01AM +0000, Harald Welte wrote:

...

Harald Welte has submitted this change and it was merged.

Change subject: Resync RTP session in case of timestamp jump ......................................................................

Resync RTP session in case of timestamp jump

This will prevent the timestamp jump to occur between each call for DTX if something goes wrong. It's handy as oRTP does not manage well timestamp jump if it is not resynced.

Fixes: OS#1803 Change-Id: Iae5da832218dbd1ede7a9451102074f2a5eb66df

---

M src/trau/osmo_ortp.c 1 file changed, 2 insertions(+), 1 deletion(-)

Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified

diff --git a/src/trau/osmo_ortp.c b/src/trau/osmo_ortp.c index 6ce70d2..e3c6234 100644 --- a/src/trau/osmo_ortp.c +++ b/src/trau/osmo_ortp.c @@ -139,7 +139,8 @@ uint32_t ts = rtp_session_get_current_recv_ts(rs);

LOGP(DLMIB, LOGL_NOTICE,

•     "osmo-ortp(%d): timestamp_jump, new TS %d\n", port, ts);
  

•     "osmo-ortp(%d): timestamp_jump, new TS %d, resyncing\n", port, ts);
  

• rtp_session_resync(rs);

}

-- To view, visit https://gerrit.osmocom.org/1028 To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged Gerrit-Change-Id: Iae5da832218dbd1ede7a9451102074f2a5eb66df Gerrit-PatchSet: 4 Gerrit-Project: libosmo-abis Gerrit-Branch: master Gerrit-Owner: Max msuraev@sysmocom.de Gerrit-Reviewer: Harald Welte laforge@gnumonks.org Gerrit-Reviewer: Jenkins Builder