<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think this really backs up the fact that it's needed to release DL TBF upon a Packet Resource Request resceived... it's rather simply fixing the way it was being released.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Do you understand it differently?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Sorry, I don't have a strong opinion here. It looks like we should dig deeper into the history and the related specs. It's interesting that the commit message I found yesterday is basically an implicit revert of his other commit:</p><p style="white-space: pre-wrap; word-wrap: break-word;">commit ae0a799f4472a75912b444cc2f09361aaea27963<br>Author: Jacob Erlbeck <jerlbeck@sysmocom.de><br>Date:   Tue Sep 1 11:28:25 2015 +0200</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">    bts: Release DL TBF instead of killing in rcv_resource_request<br>    <br>    Currently an existing DL TBF is freed immediately, when a resource<br>    request is received. This makes sense since the MS might have dropped<br>    it when switching to the PDCH signaled via the AGCH for the SBA. But<br>    if the TBF still is assumed to exist on the MS side, there might be<br>    TFI collisions if the old TBF object is not kept to block its TFI<br>    for some time.<br>    <br>    This commit changes rcv_resource_request to call release() instead of<br>    tbf_free() on the DL TBF object (if it exists).<br>    <br>    Sponsored-by: On-Waves ehf</pre><p style="white-space: pre-wrap; word-wrap: break-word;">This piece of code has been changed so many times, see also:</p><p style="white-space: pre-wrap; word-wrap: break-word;">e91bd3babd5c04a154f296607b401a5050dcba31<br>cf6ae9d12f9bf406a8f164f8109410d01328d913</p><p style="white-space: pre-wrap; word-wrap: break-word;">and this one seems to be the origin:</p><p style="white-space: pre-wrap; word-wrap: break-word;">commit 99a107dbeef03b2e80aff82023cdfc5c10109b5b<br>Author: Andreas Eversberg <jolly@eversberg.eu><br>Date:   Thu Sep 27 09:21:52 2012 +0200</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">    Free existing UL/DL TBF, if RACH has been received from MS<br>    <br>    In this case the mobile has lost existing flows, so it make sense to free<br>    them. The TFI(s) can be re-used immidiately, because they are not<br>    associated by MS anymore.</pre><p style="white-space: pre-wrap; word-wrap: break-word;">So it looks like it was done on purpose, and then changed +100500 times.</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/19194">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/19194">change 19194</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-pcu/+/19194"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcu </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iab4afb66f0d671f7ad54909d2685a1613e12ab4d </div>
<div style="display:none"> Gerrit-Change-Number: 19194 </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: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 09 Jul 2020 07:46:25 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>