<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24600">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24600/1/doc/manuals/chapters/control.adoc">File doc/manuals/chapters/control.adoc:</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-bsc/+/24600/1/doc/manuals/chapters/control.adoc@41">Patch Set #1, Line 41:</a> <code style="font-family:monospace,monospace">|handover_ho_active|RW|No|"0","1","default"|Enable/disable handover.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I did not notice that. This probably means that I have dig a bit deeper. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is now also fixed. It was not easy as well since the control interface seems to have a lot of quirks (and maybe bugs). I do not see through entirely, but by design it seems not to be possible to install a command twice on different nodes. Also there seems to be no way to find on which nodes we are. The node variable is a void pointer to the struct we are working on. (e.g. bts) but we have no way to check if we are really on the BTS node. We just have to trust that cmd->node contains what we expect.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24600/1/src/osmo-bsc/handover_ctrl.c">File src/osmo-bsc/handover_ctrl.c:</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-bsc/+/24600/1/src/osmo-bsc/handover_ctrl.c@80">Patch Set #1, Line 80:</a> <code style="font-family:monospace,monospace">CTRL_CMD_DEFINE(NAME, "handover_"#NAME); \</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The vty command name is in VTY_CMD; […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I have thought through both. Option 1 is a bit problematic as we have VTY commands that have spaces (which become to dots) and - characters in them. We can not use - in the C code. Also I think it would be a mess to replace all the struct members.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Adding another argument to the macro is the better solution I think. Ironically it turned out (I did not know that before) that the control interface commands must have spaces where later the dots are expected. So we can just use VTY_CMD_PREFIX and VTY_CMD directly and we get exactly what we need. Unfortunately there are min-free-slots tch/f and min-free-slots tch/h. The / is a forbidden character in the control interface. So we still need the additional argument to define a string without /. At least I managed to make the new parameter (CTRL_CMD) optional.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24600">change 24600</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-bsc/+/24600"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I12f143906818fd6b16e8783157cbb1eb51e49ffc </div>
<div style="display:none"> Gerrit-Change-Number: 24600 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: neels <nhofmeyr@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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 16 Jun 2021 12:55:38 +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: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>