Change in osmo-ttcn3-hacks[master]: Remove CSFB references from RR Chan Release CellSelectionIndicator IE

pespin gerrit-no-reply at lists.osmocom.org
Thu Apr 15 12:15:32 UTC 2021


pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/23764 )


Change subject: Remove CSFB references from RR Chan Release CellSelectionIndicator IE
......................................................................

Remove CSFB references from RR Chan Release CellSelectionIndicator IE

The IE contains a cell list for the MS to register after the channel
is released. The IE is used in CSFB, but not only in that case: it's
also used in SRVCC.
Hence let's remove the CSFB references since the scope is more wide.

Related: SYS#5337
Change-Id: Ia1eeda98fc21aa92bb2e41b5e4761c5cf6516a7e
---
M bsc/BSC_Tests.ttcn
M library/L3_Templates.ttcn
2 files changed, 9 insertions(+), 9 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/64/23764/1

diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn
index e4e2d1a..6170e48 100644
--- a/bsc/BSC_Tests.ttcn
+++ b/bsc/BSC_Tests.ttcn
@@ -1748,7 +1748,7 @@
 			   boolean expect_rll_rel_req := true,
 			   boolean handle_rll_rel := true,
 			   boolean is_csfb := false,
-			   template CellSelIndValue csfb_expect_cells := omit,
+			   template CellSelIndValue expect_cells := omit,
 			   template RR_Cause expect_rr_cause := ?
 			   ) runs on test_CT {
 
@@ -1767,7 +1767,7 @@
 		got_deact_sacch := true;
 		repeat;
 	}
-	[is_csfb] IPA_RSL[bts_nr].receive(tr_ASP_RSL_UD(tr_RSL_DATA_REQ(rsl_chan_nr, ?, decmatch tr_RRM_RR_RELEASE_CSFB))) -> value ud {
+	[is_csfb] IPA_RSL[bts_nr].receive(tr_ASP_RSL_UD(tr_RSL_DATA_REQ(rsl_chan_nr, ?, decmatch tr_RRM_RR_RELEASE_CellSelectInd))) -> value ud {
 		got_rr_chan_rel := true;
 
 		if (f_rsl_find_ie(ud.rsl, RSL_IE_L3_INFO, l3_ie) == false) {
@@ -1776,16 +1776,16 @@
 		}
 		l3 := dec_PDU_ML3_NW_MS(l3_ie.l3_info.payload);
 
-		if (not istemplatekind(csfb_expect_cells, "omit")) {
+		if (not istemplatekind(expect_cells, "omit")) {
 			var CellSelIndValue cells := dec_CellSelIndValue(
 				l3.msgs.rrm.channelRelease.cellSelectionIndicator.cellSelectionIndicatorValue);
 
-			log("GOT RR CHANNEL RELEASE CSFB CELLS: ", cells);
-			if (match(cells, csfb_expect_cells)) {
+			log("GOT RR CHANNEL RELEASE WITH CELLS: ", cells);
+			if (match(cells, expect_cells)) {
 				setverdict(pass);
 			} else {
-				log("EXPECTED CSFB CELLS: ", csfb_expect_cells);
-				setverdict(fail, "Received CSFB cells list on RR Channel Release does not match expectations");
+				log("EXPECTED CELLS: ", expect_cells);
+				setverdict(fail, "Received cells list on RR Channel Release does not match expectations");
 			}
 		}
 
@@ -2323,7 +2323,7 @@
 		}
 
 		f_expect_chan_rel(0, dt.rsl_chan_nr, expect_rll_rel_req := false, is_csfb := true,
-				  csfb_expect_cells := expect_cells);
+				  expect_cells := expect_cells);
 	}
 
 	for (var integer i := 1; i < total_earfcns; i := i + 1) {
diff --git a/library/L3_Templates.ttcn b/library/L3_Templates.ttcn
index 5bd87eb..5d688de 100644
--- a/library/L3_Templates.ttcn
+++ b/library/L3_Templates.ttcn
@@ -910,7 +910,7 @@
 	}
 }
 
-template PDU_ML3_NW_MS tr_RRM_RR_RELEASE_CSFB(template OCT1 cause := ?) modifies tr_RRM_RR_RELEASE := {
+template PDU_ML3_NW_MS tr_RRM_RR_RELEASE_CellSelectInd(template OCT1 cause := ?) modifies tr_RRM_RR_RELEASE := {
 	msgs := {
 		rrm := {
 			channelRelease := {

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/23764
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ia1eeda98fc21aa92bb2e41b5e4761c5cf6516a7e
Gerrit-Change-Number: 23764
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210415/c36fac80/attachment.htm>


More information about the gerrit-log mailing list