<p style="white-space: pre-wrap; word-wrap: break-word;">Hi Philipp,</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 1:</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 1: Code-Review-2</p><p style="white-space: pre-wrap; word-wrap: break-word;">We cannot just bump the protocol version here, as we also need to test the 'latest' osmo-pcu. See my comments to the related changes for osmo-pcu.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I have seen we have a module parameter for the version number and we always support the current version number and the latest. If we can not bumb the version number here, where else can we bump it and don't we have to bump it if we modify indications?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">bumping the version is not enough, you also need to change the protocol definition in a way that it's possible to encode/decode messages for the old and the new versions. It would be a lot easier to introduce a new PCUIF message for sending System Information, instead of (ab)using INFO.ind for that. This way you can keep using the current protocol version, older IUT would just ignore/reject these new messages.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22369">View Change</a></p><ul style="list-style: none; padding: 0;"></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: 1 </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-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 25 Jan 2021 15:32:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>