<p>pespin <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-trx/+/19089">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, but someone else must approve
  Hoernchen: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">doc/manuals: Update thread documentation after dropping CTRL sock threads<br><br>Per-ARFCN CTRL threads managing CTRL socket loops were dropped a while<br>ago, but it was forgotten to udpate the documentation.<br><br>Change-Id: I34d117325e60b04b075c205d21bb0b827a5e8c52<br>---<br>M doc/manuals/chapters/code-architecture.adoc<br>1 file changed, 5 insertions(+), 10 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/doc/manuals/chapters/code-architecture.adoc b/doc/manuals/chapters/code-architecture.adoc</span><br><span>index 18d0e3a..f1feb2c 100644</span><br><span>--- a/doc/manuals/chapters/code-architecture.adoc</span><br><span>+++ b/doc/manuals/chapters/code-architecture.adoc</span><br><span>@@ -39,8 +39,7 @@</span><br><span> radioiface [label="RadioInterface"];</span><br><span> radiodev [label="RadioDevice"];</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-trans:nw->trans:ne [label="Trx.ControlServiceLoop_0"];</span><br><span style="color: hsl(0, 100%, 40%);">-trans:nw->trans:ne [label="Trx.ControlServiceLoop_1"];</span><br><span style="color: hsl(120, 100%, 40%);">+trans:nw->trans:ne [label="Main"];</span><br><span> trans:w->radioiface:w [label="Trx.TxPriorityQueueServiceLoop_0"];</span><br><span> trans:w->radioiface:w [label="Trx.TxPriorityQueueServiceLoop_1"];</span><br><span> radioiface:e->trans:e [label="Trx.RxServiceLoop_0"];</span><br><span>@@ -92,14 +91,6 @@</span><br><span> </span><br><span> Dynamic amount of Threads (1 per RF logical channel on the `Transceiver` instance):</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-* `ControlServiceLoop`: Handles commands from the Per-ARFCN Control Interface</span><br><span style="color: hsl(0, 100%, 40%);">-  socket (<<trx_if_control>>). Each thread is responsible for managing one</span><br><span style="color: hsl(0, 100%, 40%);">-  socket related to one ARFCN or which is the same, to one RF logical channel.</span><br><span style="color: hsl(0, 100%, 40%);">-  These are the only threads expected to use the private `start()` and `stop()`</span><br><span style="color: hsl(0, 100%, 40%);">-  methods of the `Transceiver()` class, since those methods don't stop any of</span><br><span style="color: hsl(0, 100%, 40%);">-  the `ControlServiceLoop` threads as they must keep running to handle new</span><br><span style="color: hsl(0, 100%, 40%);">-  commands (for instance, to re-start processing samples with the _POWERON_</span><br><span style="color: hsl(0, 100%, 40%);">-  command).</span><br><span> * `RxServiceLoop`: Each thread of this type pulls bursts from the</span><br><span>   `RadioInterface` FIFO for one specific logical RF channel and handles it</span><br><span>   according to the slot and burst correlation type, finally sending proper data</span><br><span>@@ -108,6 +99,10 @@</span><br><span>   Manager UDP socket (<<trx_if>>), and fills the `RadioInterface` with it</span><br><span>   setting clock related information.</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+All the Per-ARFCN Control Interface socket (<<trx_if_control>>) commands are</span><br><span style="color: hsl(120, 100%, 40%);">+handled by the event loop runnnig on the main thread. This is the only thread</span><br><span style="color: hsl(120, 100%, 40%);">+expected to use the private `start()` and `stop()` methods.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> [[code_component_radioiface]]</span><br><span> === RadioInterface</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-trx/+/19089">change 19089</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-trx/+/19089"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-trx </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I34d117325e60b04b075c205d21bb0b827a5e8c52 </div>
<div style="display:none"> Gerrit-Change-Number: 19089 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@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: merged </div>