<p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/20217">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-sgsn/+/20217/1/src/sgsn/gprs_gmm.c">File src/sgsn/gprs_gmm.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-sgsn/+/20217/1/src/sgsn/gprs_gmm.c@305">Patch Set #1, Line 305:</a> <code style="font-family:monospace,monospace">0x44</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why 0x44?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">the quick answer, I noted in wireshark two radio priorities here:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">    Radio Priority 2 - Radio priority for TOM8<br>        .100 .... = Radio Priority (TOM8): priority level 4 (lowest) (4)<br>    Radio Priority - Radio priority for SMS<br>        .... .100 = Radio Priority (PDP or SMS): priority level 4 (lowest) (4)</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br>Radio Priority 2 used to be .000 (above is with this patch) - Without following the code too much, but seeing that<br> struct gsm48_attach_ack {...} <br>has only one field for radio_prio, I guessed that this uint8_t <br>corresponds to both and that the intention is that 'TOM8' should also be priority 4, best to set it seeing as how the MTK does not like this IE being 000</pre></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/20217">change 20217</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-sgsn/+/20217"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I810cd541eb5764ee3f2c238bcd3a10836228d0b5 </div>
<div style="display:none"> Gerrit-Change-Number: 20217 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 20 Sep 2020 23:08:48 +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: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>