<p style="white-space: pre-wrap; word-wrap: break-word;">It seems to me that this case was implemented looking at the current behaviour of OsmoPCU, which may be incorrect (remember the situation with PTCCH/U, when everybody wrongly expected data blocks on it). Could you please add the spec. references where the process of Downlink TBF establishment is described? At least where they state that Immediate Assignment (assigning a Downlink TBF) shall be sent on PCH (and not on AGCH). The code itself looks good to me.</p><p>Patch set 6:<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-ttcn3-hacks/+/15681">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/15681/6/pcu/PCU_Tests_RAW.ttcn">File pcu/PCU_Tests_RAW.ttcn:</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/15681/6/pcu/PCU_Tests_RAW.ttcn@625">Patch Set #6, Line 625:</a> <code style="font-family:monospace,monospace">data</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">LOL, I just checked the source code of OsmoPCU, and, as it turns out, replying the whole payload from DATA.req is only needed to parse TLLI from it. Who ever thought it's a good idea...</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15681">change 15681</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-ttcn3-hacks/+/15681"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I3be5123ff5294e5851652ec14d54589442082b28 </div>
<div style="display:none"> Gerrit-Change-Number: 15681 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </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: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 09 Oct 2019 15:23:26 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>