<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 6:</p><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 6:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I hope I have found a better way now to transfer the sysinfo to the PCU. Actually there is a SAPI 4 in the PCUIF protocol through which we already send SI13. This can be extended to transfer SI1 and SI3 as well.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Nice! Would be funny it we discovered this after merging your initial approach ;) If you remove the first (redundant?) octet in DATA.ind messages, there would be no need to bump PCUIF version at all.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">There would be still the problem that we would have to have a way to tell that the SI should be deleted.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Basically I understand you suggestion as follows:</p><ul><li>Remove osmo-si type at the beginning and use RSL IEI type.</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">01 06 [SI 13 octets] ==> New SI, store in struct, set si13_is_set to true<br>01 ==> set si13_is_set to false.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe also:</p><p style="white-space: pre-wrap; word-wrap: break-word;">(nothing) ==> set si13_is_set to false.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Sure, we can have that, but I do not see a big advantage over the now existing approach.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22369">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/osmo-ttcn3-hacks/+/22369/6/pcu/PCU_Tests.ttcn">File pcu/PCU_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/osmo-ttcn3-hacks/+/22369/6/pcu/PCU_Tests.ttcn@3589">Patch Set #6, Line 3589:</a> <code style="font-family:monospace,monospace">'01'O</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">What is the purpose of this octet? I guess to indicate the type of SI? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess this is the IEI that is used on RSL level. All SI are prefixed with this pseudo header.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22369">change 22369</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/+/22369"/><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: I9054ab0e969c0fbfdc671c92d44cc61360959adc </div>
<div style="display:none"> Gerrit-Change-Number: 22369 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 01 Feb 2021 15:32:22 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>