[MERGED] openbsc[master]: DTX: add data necessary for scheduling

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/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Sun Jun 12 15:39:47 UTC 2016


Harald Welte has submitted this change and it was merged.

Change subject: DTX: add data necessary for scheduling
......................................................................


DTX: add data necessary for scheduling

DTXd: to schedule SID repetition we have to know when previous SID was
sent (fn) and if it was UPDATE or FIRST SID (is_update).
DTXu: to properly set Marker bit in outgoing RTP we have to know the
beginning of talkspurt. For codecs without explicit ONSET event we can
do it by setting the flag (ul_sid) upon receiving SID and unsetting it
on speech frames.

Change-Id: I79cbec3b6c6fed5de385f8e202ceaf0b13234778
Related: OS#22, OS#1701
---
M openbsc/include/openbsc/gsm_data_shared.h
1 file changed, 5 insertions(+), 0 deletions(-)

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



diff --git a/openbsc/include/openbsc/gsm_data_shared.h b/openbsc/include/openbsc/gsm_data_shared.h
index 60a689c..a8bcd5d 100644
--- a/openbsc/include/openbsc/gsm_data_shared.h
+++ b/openbsc/include/openbsc/gsm_data_shared.h
@@ -289,7 +289,12 @@
 		struct {
 			uint8_t buf[16];
 			uint8_t len;
+			uint32_t fn;
+			bool is_update;
 		} last_sid;
+		/* set for each SID frame to detect talkspurt for codecs without
+		   explicit ONSET event */
+		bool ul_sid;
 		uint8_t last_cmr;
 		uint32_t last_fn;
 		/* indicates if DTXd was active during DL measurement period */

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I79cbec3b6c6fed5de385f8e202ceaf0b13234778
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list