<p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/23526">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/23526/1/tests/tbf/TbfTest.err">File tests/tbf/TbfTest.err:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/23526/1/tests/tbf/TbfTest.err@a418">Patch Set #1, Line 418:</a> <code style="font-family:monospace,monospace">r</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">are those kind of changes of 'prip' expected? I don't immediately see how?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">That's due to the "if (poll-scheduled()) return false) being removed. That's removed because a TBF should actually be fine scheduling a control block even if a POLL is scheduled, since multiple POLL schedules should be possible concurrently. This check was there to prevent that happening since the whole implementation was not supporting that kind of thing. As a result, need_control_ts() may return now true in that situation, where it used to return false, hence increasing the priority of this tbf.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/23526">change 23526</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/+/23526"/><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: I875f51cade95faeb2d79dcebfead4c83e23a731b </div>
<div style="display:none"> Gerrit-Change-Number: 23526 </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-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Mar 2021 15:05:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>