Change in osmo-ttcn3-hacks[master]: bts: Disable PCU related tests if PCU socket not configured

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

Pau Espin Pedrol gerrit-no-reply at lists.osmocom.org
Tue Nov 20 16:31:14 UTC 2018


Pau Espin Pedrol has submitted this change and it was merged. ( https://gerrit.osmocom.org/11841 )

Change subject: bts: Disable PCU related tests if PCU socket not configured
......................................................................

bts: Disable PCU related tests if PCU socket not configured

With some real HW setups, there's no PCU (osmo-pcu) available locally,
for instance when using a sysmobts, or when using a nanoBTS. There's no
need to waste time and generate extra output by running this tests in
this case.

Furthermore, some tests seem to crash sometimes in that setup, probably
due to using invalid fd (-1):
MTC at 801a0da9866a: Setting RSL_SYSTEM_INFO_4 (4): '31061C62F224002A4740E50400'O
 /osmo-ttcn3-hacks/bts/BTS_Tests: Segmentation fault occurred
/usr/lib/titan/libttcn3-parallel-dynamic.so(_Z14signal_handleri+0xa3)[0x7f0c33b48073]
/lib/x86_64-linux-gnu/libc.so.6(+0x33060)[0x7f0c321aa060]
/osmo-ttcn3-hacks/bts/UD_PT.so(_ZN12UD__PortType15UD__PT_PROVIDER13outgoing_sendERKN9UD__Types14UD__send__dataE+0xf0)[0x7f0c349e42f8]
/osmo-ttcn3-hacks/bts/PCUIF_CodecPort.so(_ZN16PCUIF__CodecPort16PCUIF__CODEC__PT4sendERKNS_17PCUIF__send__dataERK9COMPONENT+0x19e)[0x7f0c37e1731a]
/osmo-ttcn3-hacks/bts/PCUIF_CodecPort.so(_ZN16PCUIF__CodecPort16PCUIF__CODEC__PT4sendERKNS_26PCUIF__send__data_templateE+0x5f)[0x7f0c37e174f7]
/osmo-ttcn3-hacks/bts/BTS_Tests.so(_ZN10BTS__Tests20f__TC__pcu__act__reqERK7INTEGERS2_S2_RK7BOOLEAN+0x411)[0x7f0c3eec3210]
/osmo-ttcn3-hacks/bts/BTS_Tests.so(_ZN10BTS__Tests28testcase_TC__pcu__deact__reqEbd+0x15d)[0x7f0c3eec4f27]
/osmo-ttcn3-hacks/bts/BTS_Tests.so(+0xfb65d)[0x7f0c3eeef65d]
/usr/lib/titan/libttcn3-parallel-dynamic.so(_ZN11Module_List15execute_controlEPKc+0x1c)[0x7f0c33af3fbc]

Change-Id: I773c7ec52dd8532bf160e92ffefc8d936ca55de2
---
M bts/BTS_Tests.ttcn
1 file changed, 21 insertions(+), 17 deletions(-)

Approvals:
  Jenkins Builder: Verified
  Neels Hofmeyr: Looks good to me, approved



diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index cde9562..753d89b 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -4186,23 +4186,27 @@
 	execute( TC_ipa_crcx_mdcx_mdcx_dlcx_not_active() );
 	execute( TC_ipa_crcx_sdcch_not_active() );
 
-	execute( TC_pcu_act_req() );
-	execute( TC_pcu_act_req_wrong_ts() );
-	execute( TC_pcu_act_req_wrong_bts() );
-	execute( TC_pcu_act_req_wrong_trx() );
-	execute( TC_pcu_deact_req() );
-	execute( TC_pcu_deact_req_wrong_ts() );
-	execute( TC_pcu_ver_si13() );
-	execute( TC_pcu_data_req_wrong_bts() );
-	execute( TC_pcu_data_req_wrong_trx() );
-	execute( TC_pcu_data_req_wrong_ts() );
-	execute( TC_pcu_data_req_ts_inactive() );
-	execute( TC_pcu_data_req_pdtch() );
-	execute( TC_pcu_data_req_ptcch() );
-	execute( TC_pcu_data_req_agch() );
-	execute( TC_pcu_data_req_imm_ass_pch() );
-	execute( TC_pcu_rach_content() );
-	execute( TC_pcu_paging_from_rsl() );
+	if (mp_pcu_socket != "") {
+		execute( TC_pcu_act_req() );
+		execute( TC_pcu_act_req_wrong_ts() );
+		execute( TC_pcu_act_req_wrong_bts() );
+		execute( TC_pcu_act_req_wrong_trx() );
+		execute( TC_pcu_deact_req() );
+		execute( TC_pcu_deact_req_wrong_ts() );
+		execute( TC_pcu_ver_si13() );
+		execute( TC_pcu_data_req_wrong_bts() );
+		execute( TC_pcu_data_req_wrong_trx() );
+		execute( TC_pcu_data_req_wrong_ts() );
+		execute( TC_pcu_data_req_ts_inactive() );
+		execute( TC_pcu_data_req_pdtch() );
+		execute( TC_pcu_data_req_ptcch() );
+		execute( TC_pcu_data_req_agch() );
+		execute( TC_pcu_data_req_imm_ass_pch() );
+		execute( TC_pcu_rach_content() );
+		execute( TC_pcu_paging_from_rsl() );
+	} else {
+		log("PCU socket path not available, skipping PCU tests");
+	}
 
 	execute( TC_dyn_osmo_pdch_act_deact() );
 	execute( TC_dyn_osmo_pdch_unsol_deact() );

-- 
To view, visit https://gerrit.osmocom.org/11841
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I773c7ec52dd8532bf160e92ffefc8d936ca55de2
Gerrit-Change-Number: 11841
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181120/fbdce318/attachment.htm>


More information about the gerrit-log mailing list