<p><a href="https://gerrit.osmocom.org/13173">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/13173/1//COMMIT_MSG">Commit Message:</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/13173/1//COMMIT_MSG@25">Patch Set #1, Line 25:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">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<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">this is not always true.  Do you have a specific reference for this?  I know that e.g. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I realize the reasoning is a bit flawed here.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If the MSC Pages again, it would Page, wait for timeout, and after that Page again.<br>So the decision to Page again would be part of the Paging implementation.<br>If at some point the decision is that Paging expired, the pending transactions should be cleaned up.</p><p style="white-space: pre-wrap; word-wrap: break-word;">What this code is currently doing is Page, then the Paging code decides that it failed.<br>The SMS transaction gets told that reaching the subscriber has failed.<br>And then what happens is that the SMS queue starts over, by circumstance picks the very same SMS again,<br>and then asks the Paging code to start Paging from scratch right away as a consequence of that.<br>So this is not really about the Paging code, it is the SMS queue code trying to deliver the same failed SMS over and over and over, from a single periodical SMS queue trigger. It is re-triggering Paging from the wrong angle.<br>So if we also had a Paging retry in osmo-msc, then the SMS queue would restart this series of Paging attempts again and again == it spams Pagings.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I should clarify the commit log...</p></li></ul></li></ul><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: comment </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: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 11 Mar 2019 05:03:15 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>