Change in osmo-ttcn3-hacks[master]: Revert "BSSAP_LE_Emulation: configure sending vs receiving RESET"

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 gerrit-no-reply at lists.osmocom.org
Thu Oct 15 01:58:02 UTC 2020


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


Change subject: Revert "BSSAP_LE_Emulation: configure sending vs receiving RESET"
......................................................................

Revert "BSSAP_LE_Emulation: configure sending vs receiving RESET"

This should not have been merged. It is an intermediate attempt to make the BSC
send the BSSMAP-LE RESET, which doesn't work because the BSC is not restarted
across test runs. I dropped the patch from my branch, while it remained on
gerrit accidentally; then someone else merged it without noticing that it was
no longer part of my lcs branch.

This reverts commit b2b3704d2a0289332798ba4aeff5b8e29d297ee5.

Change-Id: If191cf0ee5c239066fa41621e812efbdcca2a2b8
---
M library/BSSAP_LE_Emulation.ttcn
1 file changed, 2 insertions(+), 6 deletions(-)



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

diff --git a/library/BSSAP_LE_Emulation.ttcn b/library/BSSAP_LE_Emulation.ttcn
index 7818be1..d95506c 100644
--- a/library/BSSAP_LE_Emulation.ttcn
+++ b/library/BSSAP_LE_Emulation.ttcn
@@ -341,8 +341,7 @@
 	boolean role_ms,
 	/* needed for performing BSSMAP RESET */
 	SCCP_PAR_Address sccp_addr_local optional,
-	SCCP_PAR_Address sccp_addr_peer optional,
-	boolean send_reset
+	SCCP_PAR_Address sccp_addr_peer optional
 }
 
 private altstep as_reset_ack() runs on BSSAP_LE_Emulation_CT {
@@ -505,8 +504,7 @@
 	f_conn_table_init();
 	f_expect_table_init();
 
-	if (g_ran_ops.send_reset
-	    and isvalue(ops.sccp_addr_peer) and isvalue(ops.sccp_addr_local)) {
+	if (isvalue(ops.sccp_addr_peer) and isvalue(ops.sccp_addr_local)) {
 		f_sleep(1.0);	/* HACK to wait for M3UA/ASP to be ACTIVE */
 		f_bssap_le_reset(ops.sccp_addr_peer, ops.sccp_addr_local);
 	}
@@ -522,8 +520,6 @@
 		var integer targetPointCode;
 
 		alt {
-		[not g_ran_ops.send_reset] as_reset_ack();
-
 		[] as_main_bssap_le();
 
 		[g_ran_ops.role_ms] CLIENT.receive(PDU_DTAP_MO:?) -> value dtap_mo sender vc_conn {

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20646
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: If191cf0ee5c239066fa41621e812efbdcca2a2b8
Gerrit-Change-Number: 20646
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201015/8f74ab78/attachment.htm>


More information about the gerrit-log mailing list