<p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224">View Change</a></p><p>8 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224/1/src/bts.cpp">File src/bts.cpp:</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/+/24224/1/src/bts.cpp@372">Patch Set #1, Line 372:</a> <code style="font-family:monospace,monospace">  /* We known the target MS for the paging req, send the req only on PDCH</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">IMO, it would be a lot more readable if we had to separate functions for both cases:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  - when GprsMs is not NULL, and we know where to page;<br>  - when GprsMs is NULL, and we simply page everywhere.</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224/1/src/bts.cpp@376">Patch Set #1, Line 376:</a> <code style="font-family:monospace,monospace">8</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">ARRAY_SIZE(ms->ul_tbf->pdch)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224/1/src/bts.cpp@387">Patch Set #1, Line 387:</a> <code style="font-family:monospace,monospace">8</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">ARRAY_SIZE(ms->dl_tbf->pdch)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224/1/src/bts.cpp@447">Patch Set #1, Line 447:</a> <code style="font-family:monospace,monospace">                               LOGPDCH(&bts->trx[trx].pdch[ts], DRLCMAC, LOGL_INFO, "Paging on PACCH\n");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This looks more like a separate change to me, why doing it here?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224/1/src/gprs_bssgp_pcu.c">File src/gprs_bssgp_pcu.c:</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/+/24224/1/src/gprs_bssgp_pcu.c@183">Patch Set #1, Line 183:</a> <code style="font-family:monospace,monospace">BSSGP_CAUSE_COND_IE_ERR</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">BSSGP_CAUSE_MISSING_MAND_IE</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224/1/src/gprs_bssgp_pcu.c@190">Patch Set #1, Line 190:</a> <code style="font-family:monospace,monospace">BSSGP_CAUSE_COND_IE_ERR</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">BSSGP_CAUSE_INV_MAND_INF</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224/1/src/gprs_bssgp_pcu.c@211">Patch Set #1, Line 211:</a> <code style="font-family:monospace,monospace">  if (TLVP_PRESENT(tp, BSSGP_IE_TLLI))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Are we safe against an evil SGSN here? ;)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224/1/src/gprs_bssgp_pcu.c@230">Patch Set #1, Line 230:</a> <code style="font-family:monospace,monospace">specific BTS</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">How can you be sure that the MS is listening to a specific BTS? You only get notified when it enters another RAC, so what if the MS has switched to another BTS while you still have a GprsMs for it in the old BTS?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24224">change 24224</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/+/24224"/><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: Iea46d5321a29d800813b1aa2bf4ce175ce45e2cf </div>
<div style="display:none"> Gerrit-Change-Number: 24224 </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: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 18 May 2021 20:00:48 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>