<p>Neels Hofmeyr <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/13114">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">log: don't spam with "can not patch PT" messages<br><br>currently, when looking at an osmo-mgw log output with a phone call working<br>completely fine, you see these messages on LOGL_ERROR for each RTP packet:<br><br>20190304221420619 DRTP ERROR endpoint:0x0 can not patch PT because no suitable egress codec was found. (mgcp_network.c:761)<br>20190304221420639 DRTP ERROR endpoint:0x1 can not patch PT because no suitable egress codec was found. (mgcp_network.c:761)<br>20190304221420639 DRTP ERROR endpoint:0x0 can not patch PT because no suitable egress codec was found. (mgcp_network.c:761)<br>20190304221420659 DRTP ERROR endpoint:0x1 can not patch PT because no suitable egress codec was found. (mgcp_network.c:761)<br>20190304221420659 DRTP ERROR endpoint:0x0 can not patch PT because no suitable egress codec was found. (mgcp_network.c:761)<br><br>Put these messages on DEBUG level instead. They currently do not convey useful<br>information.<br><br>Possibly our MGCP clients could use some changes in behavior regarding payload<br>types, but since that topic is quite unresolved, let's not spam the ERROR log<br>level with this.<br><br>Change-Id: I4afc41fd472ec8ba887b9263fbac62de50d7cef9<br>---<br>M src/libosmo-mgcp/mgcp_network.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libosmo-mgcp/mgcp_network.c b/src/libosmo-mgcp/mgcp_network.c</span><br><span>index 33738bc..30ed7fd 100644</span><br><span>--- a/src/libosmo-mgcp/mgcp_network.c</span><br><span>+++ b/src/libosmo-mgcp/mgcp_network.c</span><br><span>@@ -756,7 +756,7 @@</span><br><span>   if (is_rtp) {</span><br><span>                rc = mgcp_patch_pt(conn_src, conn_dst, buf, len);</span><br><span>            if (rc < 0) {</span><br><span style="color: hsl(0, 100%, 40%);">-                        LOGP(DRTP, LOGL_ERROR,</span><br><span style="color: hsl(120, 100%, 40%);">+                        LOGP(DRTP, LOGL_DEBUG,</span><br><span>                            "endpoint:0x%x can not patch PT because no suitable egress codec was found.\n",</span><br><span>                            ENDPOINT_NUMBER(endp));</span><br><span>                 }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/13114">change 13114</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/13114"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I4afc41fd472ec8ba887b9263fbac62de50d7cef9 </div>
<div style="display:none"> Gerrit-Change-Number: 13114 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Max <msuraev@sysmocom.de> </div>