<p>neels <strong>uploaded patch set #5</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24461">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">update the lchan name to always reflect VAMOS shadowness<br><br>Change gsm_lchan_name_compute() to a function that in-place updates the<br>lchan->name. That allows calling it numerous times with the talloc<br>handled internally. Rename it to lchan_update_name().<br><br>Add 'shadow' to lchan_update_name() and lchan_fsm_update_id() for VAMOS<br>shadow lchans, and also print the lchan index that it is a shadow for,<br>instead of the index in the lchan array.<br><br>When set_pchan_is() updates the VAMOSness of the lchans, call<br>lchan_fsm_update_id(). From lchan_fsm_update_id() also call<br>lchan_update_name().<br><br>This is a bit convoluted for legacy reasons. There are utility programs<br>and C tests using bts_trx.c but not lchan_fsm.c. lchan_update_name()<br>lives in gsm_data.c for that reason. This patch calls<br>lchan_update_name() from lchan_fsm_update_id() and not vice versa to<br>avoid having to add stubbed lchan_fsm_update_id() functions to all<br>utility programs and C tests.<br><br>We can't easily unify the lchan->name and lchan->fi->id without lots of<br>refactoring rippling through all those little utility programs and C<br>tests.<br><br>Change-Id: I7c2bae3b895a91f1b99b4147ecc0e3009cb7439a<br>---<br>M include/osmocom/bsc/gsm_data.h<br>M include/osmocom/bsc/lchan_fsm.h<br>M src/osmo-bsc/assignment_fsm.c<br>M src/osmo-bsc/bts_trx.c<br>M src/osmo-bsc/gsm_data.c<br>M src/osmo-bsc/lchan_fsm.c<br>M src/osmo-bsc/timeslot_fsm.c<br>7 files changed, 25 insertions(+), 12 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/61/24461/5</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24461">change 24461</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/+/24461"/><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: I7c2bae3b895a91f1b99b4147ecc0e3009cb7439a </div>
<div style="display:none"> Gerrit-Change-Number: 24461 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>