<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe it makes more sense to move some of those out. I think<br>"_utils" still is a misnomer for the central Abis RR<br>implementation, could become abis_rr.c, but we don't really need to<br>rename the file.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I agree, and I wouldn't mind the rename either.  However, it's not "Abis RR",<br>but "gsm0408_rr".  Afte rall, RR is a Layer3 sub-layer spoken over Abis and Um.</p><p style="white-space: pre-wrap; word-wrap: break-word;">See also my recently-proposed patch to move the CC related 04.08 messages to<br>a separate file in osmo-msc.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Then there is bsc_api.c, that appears to have been a kitchen sink<br>for everything and anything. We removed struct bsc_api already. Do<br>we agree on moving bsc_api.c's RR functions? </p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">yes, I would be more than happy with that.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">If gsm_04_08_utils.c<br>is the main "RR" file, then maybe handle_rr_ho_*() and<br>gsm_04_08_rcvmsg should move from bsc_api.c to gsm_04_08_utils.c<br>instead of abis_rsl.c?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">ACK</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(The only functions left in bsc_api.c are gsm0808_page() and<br>gsm0808_cipher_mode(), maybe they can move to the callers, paging.c<br>and osmo_bsc_bssap.c, to completely drop bsc_api.c)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">great!</p><p><a href="https://gerrit.osmocom.org/9669">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9669">change 9669</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/9669"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ie8ee334145bf7bc3a601d395ea7ab9b2009b61c7 </div>
<div style="display:none"> Gerrit-Change-Number: 9669 </div>
<div style="display:none"> Gerrit-PatchSet: 12 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 24 Jul 2018 05:13:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>