<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15171">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15171/2//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15171/2//COMMIT_MSG@10">Patch Set #2, Line 10:</a> <code style="font-family:monospace,monospace">which sends L3 messages for Iu and Gb at the same time.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Not at the same time. It sends L3 messages over the requested L2 layer (controlled by gb_index).</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15171/2/sgsn/SGSN_Tests.ttcn">File sgsn/SGSN_Tests.ttcn:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15171/2/sgsn/SGSN_Tests.ttcn@525">Patch Set #2, Line 525:</a> <code style="font-family:monospace,monospace">function f_send_l3(template (value) PDU_L3_MS_SGSN l3_mo, integer gb_index := 0, boolean initial := false) runs on BSSGP_ConnHdlr {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">May have more sense to have an array of initial states (boolean) in BSSGP_ConnHdlr with size equal to values available in gb_index (btw, can we call it ran_index?). This way initial state is maintained inside this function and tests don't need to care about whether they are sending initial message or not (and they can still play with the value by modifying the array directly if needed).</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15171/2/sgsn/SGSN_Tests.ttcn@644">Patch Set #2, Line 644:</a> <code style="font-family:monospace,monospace">                         var IntegrityProtectionAlgorithm uia_chosen := 0; /*standard_UMTS_integrity_algorithm_UIA1;*/</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">why is this commented?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15171/2/sgsn/SGSN_Tests.ttcn@646">Patch Set #2, Line 646:</a> <code style="font-family:monospace,monospace">                           //BSSAP.receive(tr_RANAP_CommonId(imsi_hex2oct(g_pars.imsi)));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">why is this commented out? because sgsn should send it but it doesn't?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15171">change 15171</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15171"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: If47ad2be459ca7b87d9071d9ff020a51821e4433 </div>
<div style="display:none"> Gerrit-Change-Number: 15171 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 03 Sep 2019 18:35:28 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>