<p>Neels Hofmeyr <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/13173">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">sms queue: avoid repeated Paging for a failed SMS<br><br>So far, sms_pending_failed() starts a new sms_queue_trigger() run. The<br>intention behind that might have been to fill up the queue when sending SMS has<br>failed, but the practical effect is actually bad:<br><br>As current ttcn3-msc-test runs show, a failed MT SMS gets triggered multiple<br>times in short succession, i.e. osmo-msc repeatedly sends Paging Requests for<br>the same subscriber.<br><br>This special case happens actually only when there are few SMS still in the DB<br>to be delivered. In the TTCN3 test, there is exactly one MT SMS for one<br>subscriber, and retriggering the queue brings up the same SMS every time.<br><br>See f_tc_lu_and_mt_sms_paging_and_nothing() and f_tc_sgsap_mt_sms_and_nothing()<br>which say:<br>"/* Expect the MSC to page exactly 10 times before giving up */"<br><br>This is bad because an MSC should send a Paging Request exactly once. Retrying<br>failed Paging is clearly the task of the BSC, not the MSC. The remaining code<br>around Paging correctly follows this paradigm, but this retrigger doesn't.<br><br>Do not immediately trigger the SMS queue on a failed MT SMS. Instead, leave it<br>up to the periodical SMS queue trigger to decide.<br><br>This patch will cause the MT SMS tests in ttcn3-msc-tests to fail, because the<br>test expectations are bogus. The patch fixing the test run is listed 'Related'<br>below.<br><br>Related: I7dce12942a65eaaf97f78ca69401c7f93faacb9e (osmo-ttcn3-hacks)<br>Change-Id: I24bf9f1c1167efe1080ae4cf47ed2ef0bd981e49<br>---<br>M src/libmsc/sms_queue.c<br>1 file changed, 0 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/73/13173/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/13173">change 13173</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/13173"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I24bf9f1c1167efe1080ae4cf47ed2ef0bd981e49 </div>
<div style="display:none"> Gerrit-Change-Number: 13173 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>