<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 16:</p><p style="white-space: pre-wrap; word-wrap: break-word;">(6 comments)</p><p style="white-space: pre-wrap; word-wrap: break-word;">The patch is quite large. I think it will need some cleanup here and there. I have tested this with TTCN3 and my test network. It seems to work fine. I saw TC_two_crcx_and_rtp_osmux_bidir_ipv6_ipv4 and TC_two_crcx_mdcx_and_rtp_osmux_ipv6_ipv4 failing though, but that may also be a local issue.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It probably may make sense to add a section about how to configure threads to the manual.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Manual is missing since there is apparently still a lot to discuss...</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25432">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25432/16/include/osmocom/mgcp/mgcp_threads.h">File include/osmocom/mgcp/mgcp_threads.h:</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-mgw/+/25432/16/include/osmocom/mgcp/mgcp_threads.h@62">Patch Set #16, Line 62:</a> <code style="font-family:monospace,monospace">};</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">maybe struct mgcp_request_data should go into mgcp_protocol.h. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I've put it here since it is now part of the threading feature, and only used as part of the messages passed to the threads after partially parsing the request.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25432/16/src/libosmo-mgcp/mgcp_protocol.c">File src/libosmo-mgcp/mgcp_protocol.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-mgw/+/25432/16/src/libosmo-mgcp/mgcp_protocol.c@475">Patch Set #16, Line 475:</a> <code style="font-family:monospace,monospace">                   #if 0</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Is there any problem with that part? It should not interfere much much with the threads since it exi […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is related to the discovery that the endp find part did not work as expected, because the rq.endp = mgcp_endp_by_name_trunk(&rc, pdata.epname, rq.trunk); call returns empty usable eps if the req is wildcaded - even in the case of a dlcx * or something like that, so basically in cases where there should not be a EP nor should one be required.<br>Addiotnally the error codes returned so far differ depending on the actual mgcp command, so this was reworked and basically keeps the current error codes (and therefore behavior related to the ttcn3 tests) while making sure it now works as expected.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25432">change 25432</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-mgw/+/25432"/><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-Change-Id: I31be8253600c8af0a43c967d0d128f5ba7b16260 </div>
<div style="display:none"> Gerrit-Change-Number: 25432 </div>
<div style="display:none"> Gerrit-PatchSet: 24 </div>
<div style="display:none"> Gerrit-Owner: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 13 Sep 2021 17:49:20 +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: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>