<p style="white-space: pre-wrap; word-wrap: break-word;">I am sort-of undecided if this coulnter should be in the function that transmits an immediate assignment.   In the end, it is about successful processing of a channel request.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, in reality, and particularly in the current code those two are the same.  However, I'm not sure if e.g. it would make sense at some point to re-transmit immediate assignments?  Or what about the immediate assignment without paging that the PCU does for tbf establishment in some states?  It maybe doesn't go thorugh this code path, but I'm trying to express why it feels a bit odd to me.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not convinced it's much better inside the lchan_fsm where rsl_tx_imm_assignment() is called.  So maybe just leave it as-is but maybe put a comment on top explaining why we - at least so far - count here?</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18719">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18719">change 18719</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/+/18719"/><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: I5c51956569223e802f4789dc5ed1605d18f80aea </div>
<div style="display:none"> Gerrit-Change-Number: 18719 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: daniel <dwillmann@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-Comment-Date: Tue, 09 Jun 2020 11:05:55 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>