<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123">View Change</a></p><p>9 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/doc/manuals/chapters/mgwpool.adoc">File doc/manuals/chapters/mgwpool.adoc:</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-bsc/+/25123/10/doc/manuals/chapters/mgwpool.adoc@7">Patch Set #10, Line 7:</a> <code style="font-family:monospace,monospace">machnies. Until osmo-mgw includes multithreading support, it may also be used ot</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">to</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/doc/manuals/chapters/mgwpool.adoc@8">Patch Set #10, Line 8:</a> <code style="font-family:monospace,monospace">scale-out to multiple cores on a single host."</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Remove character: "</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/doc/manuals/chapters/mgwpool.adoc@11">Patch Set #10, Line 11:</a> <code style="font-family:monospace,monospace">the pool will automatically assign the call to the MGW with the lowest load.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">hint/idea: It may make sense to defie some "maximum call load" config or similar to vty, so one can calculate % of load when taking the decision. That's in case you have different hosts (different HW) serving calls running osmo-mgw.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/doc/manuals/chapters/mgwpool.adoc@41">Patch Set #10, Line 41:</a> <code style="font-family:monospace,monospace">local host ip-addresses or different ports. When OsmoMGW is installed from</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">IP addresses</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/doc/manuals/chapters/mgwpool.adoc@64">Patch Set #10, Line 64:</a> <code style="font-family:monospace,monospace">each MGW must be different. Otherwise it won't be possible to distinguish the</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">should be</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/doc/manuals/chapters/mgwpool.adoc@72">Patch Set #10, Line 72:</a> <code style="font-family:monospace,monospace">calls.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do we have a vty command to avoid establishing new calls on a mgw node? (EDIT) Self-answer: yes. it would be nice to put a link to an anchor to the section below explaining it.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/doc/manuals/chapters/mgwpool.adoc@76">Patch Set #10, Line 76:</a> <code style="font-family:monospace,monospace">The VTY implenets a 'show mgw-pool' command that lists the currently configured</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">implements</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/include/osmocom/bsc/gsm_data.h">File include/osmocom/bsc/gsm_data.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-bsc/+/25123/10/include/osmocom/bsc/gsm_data.h@1363">Patch Set #10, Line 1363:</a> <code style="font-family:monospace,monospace">              /* Single MGCP client configuration under msc node (also required for</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I would avoid maintaining different structs from previous versions. Let's always use a pool and put there only 1 mgw if needed.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123/10/src/osmo-bsc/osmo_bsc_main.c">File src/osmo-bsc/osmo_bsc_main.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-bsc/+/25123/10/src/osmo-bsc/osmo_bsc_main.c@890">Patch Set #10, Line 890:</a> <code style="font-family:monospace,monospace">        mgcp_client_single = mgcp_client_init(bsc_gsmnet, bsc_gsmnet->mgw.conf);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">AS I said, let's rather fill the pool always from the VTY?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe here, if the pool is empty, create one and put it into the pool.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25123">change 25123</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-bsc/+/25123"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I8f33ab2cea04b545c403a6fe479aa963a0fc0d0d </div>
<div style="display:none"> Gerrit-Change-Number: 25123 </div>
<div style="display:none"> Gerrit-PatchSet: 10 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 23 Aug 2021 14:24:39 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>