<p>neels <strong>uploaded patch set #10</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24369">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">allow explixit TSC Set and TSC on chan activ / modif / assignment<br><br>Activating / modifying to a VAMOS mode will require picking specific TSC<br>Set / TSC. It is a bad idea to pick the TSC in each message encoding<br>function, rather make this choice centrally.<br><br>So far we pick the training sequence code to use based on the timeslot<br>configuration, and this TSC is determined only upon encoding the RSL<br>messages.<br><br>Instead, pick the TSC to use upon the initial lchan activation /<br>modification request; store this in the request structs and pass through<br>the activation / modification code paths.<br><br>For VAMOS modes, we also need to pick a TSC Set. Do so also upon activ /<br>modif request. Note that the TSC Set is not yet applied in this patch,<br>it will be applied in upcoming VAMOS patches.<br><br>The activ / modif request may pass -1 for tsc_set and/or tsc to indicate<br>no specific choice of TSC Set and TSC, resulting in the same behavior as<br>before this patch.<br><br>For example, lchan->activate.info.tsc* may be passed as -1. The exact<br>choice for tsc_set and tsc is then stored in lchan->activate.tsc*, i.e.<br>one level up (the .info sub-struct is considered as immutable input<br>args). The lchan->activate.tsc* are the values actually encoded in RSL<br>messages. After the ACK, the lchan->activate.tsc* is stored in<br>lchan->tsc* to indicate the TSC actually in use. Same for modif.<br><br>Note that in 3GPP TS 45.002, the TSC Set are numbered 1 to 4, while the<br>TSC are numbered 0 to 7. On the wire, though, TSC Set is sent as 0 to 3<br>value. This is a weird discrepancy, odd choice made by the spec authors.<br>For conformance with the spec wording, I decided to pass the TSC Set<br>number as a 1-4 ranged value, and only convert it to the 0-3 on-the-wire<br>format upon message encoding. So log messages and VTY output will<br>indicate the first TSC Set as "1", but the first TSC as "0", as defined<br>in 3GPP TS 45.002, even if that is somewhat weird.<br><br>Related: SYS#5315 OS#4940<br>Change-Id: Ic665125255d7354f5499d10dda1dd866ab243d24<br>---<br>M include/osmocom/bsc/assignment_fsm.h<br>M include/osmocom/bsc/gsm_data.h<br>M src/osmo-bsc/abis_rsl.c<br>M src/osmo-bsc/assignment_fsm.c<br>M src/osmo-bsc/bsc_vty.c<br>M src/osmo-bsc/gsm_04_08_rr.c<br>M src/osmo-bsc/lchan_fsm.c<br>M src/osmo-bsc/lcs_loc_req.c<br>8 files changed, 81 insertions(+), 14 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/69/24369/10</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24369">change 24369</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/+/24369"/><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: Ic665125255d7354f5499d10dda1dd866ab243d24 </div>
<div style="display:none"> Gerrit-Change-Number: 24369 </div>
<div style="display:none"> Gerrit-PatchSet: 10 </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-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>