<p><a href="https://gerrit.osmocom.org/13027">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13027/1/pcu/PCU_Tests_RAW_SNS.ttcn">File pcu/PCU_Tests_RAW_SNS.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/13027/1/pcu/PCU_Tests_RAW_SNS.ttcn@99">Patch Set #1, Line 99:</a> <code style="font-family:monospace,monospace">       NSCP[idx].send(t_NS_Send(g_ns_conn_id[idx], ts_SNS_CHG_WEIGHT(g_nsconfig[idx].nsei, 25, v4)));</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Is g_nsconfig[idx].nsei correct? Shouldn't it be g_nsconfig[idx_chg]. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">g_nsconfig[idx_chg] should be correct.  However, the NSEI is actually identical across all conigurations, at least in sane/valid scenarios where *one* BSS is adressed.  WE just historically didn't have multiple NS-VCs so we had one NSEI per NS-VC.</p><p style="white-space: pre-wrap; word-wrap: break-word;">"idx" is the index of the NS-VC through which the SNS message is to be sent, which can be any of the NS-VCs there are between the BSS and SGSN.  "idx_chg" is the index of the [same or other] NS-VC that we want to change.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13027">change 13027</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/13027"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia63865139f63969ccde010b01ddb8fb073209f3a </div>
<div style="display:none"> Gerrit-Change-Number: 13027 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 26 Feb 2019 10:23:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>