[PATCH] osmo-pcu[master]: Revert "Use Timing Advance Index in UL assignments"

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

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Tue Feb 27 21:50:53 UTC 2018


Review at  https://gerrit.osmocom.org/6976

Revert "Use Timing Advance Index in UL assignments"

This reverts commit 6298fbb7b2f3639fde994633e33ba54a64a6ef9b,
I8b17be78a46c0bc17516b7c90f35aa4768010ae4.

Commit I52ec9b07413daabba8cd5f1fba5c7b3af6a33389 /
896574e92bea09ed8d39688b6fdf504e84521746 was found (empirically) to be a
regression, rendering GPRS service fatally unreliable.

This reverted commit seems to follow after the regression and is reverted along
with it.

Related: OS#3013
Change-Id: I5e0fd8c9c3b89e519e7382e3d0bb24e0aeddeff6
---
M src/encoding.cpp
1 file changed, 2 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/76/6976/1

diff --git a/src/encoding.cpp b/src/encoding.cpp
index 39cbe24..e96894f 100644
--- a/src/encoding.cpp
+++ b/src/encoding.cpp
@@ -255,9 +255,7 @@
 		rc = write_alpha_gamma(dest, alpha, gamma);
 		check(rc);
 
-		rc = write_ta_index(dest, ta_idx);
-		check(rc);
-
+		set_0(dest); /* No TIMING_ADVANCE_INDEX */
 		set_0(dest); /* No TBF_STARTING_TIME */
 	} else {
 		set_0(dest); /* Single Block Allocation */
@@ -319,9 +317,7 @@
 		rc = write_alpha_gamma(dest, alpha, gamma);                 /* ALPHA and GAMMA */
 		check(rc);
 
-		rc = write_ta_index(dest, ta_idx);
-		check(rc);
-
+		set_0(dest);                                                /* No TIMING_ADVANCE_INDEX */
 		set_0(dest);                                                /* No TBF_STARTING_TIME */
 		bitvec_set_bit(dest, L);                                    /* No Additions for Rel-7 */
 	} else {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5e0fd8c9c3b89e519e7382e3d0bb24e0aeddeff6
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>



More information about the gerrit-log mailing list