<p><a href="https://gerrit.osmocom.org/c/libosmo-abis/+/24514">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/libosmo-abis/+/24514/1/src/input/ipa.c">File src/input/ipa.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/libosmo-abis/+/24514/1/src/input/ipa.c@233">Patch Set #1, Line 233:</a> <code style="font-family:monospace,monospace"></code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">He's just adding a new line here because he's adding new blocks to the function, so that different s […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-abis/+/24514/1/src/input/ipa.c@247">Patch Set #1, Line 247:</a> <code style="font-family:monospace,monospace">          ret = osmo_sock_wait(link->ofd->fd, timeout_msec);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">So this is the use case you were saying it's fine blocking? Inside a library function? Doesn't look  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">You have to check I205f68a3a7f35fee4c38a7cfba2b014237df2727 (osmo-bts). It depends how the library is used. timeout_msec is optional. If it is 0, then there is no blocking at all. If the user wants to block in order to make sure that the TCP connection towards the bsc is working before moving on, then timeout_msec can be set to a reasonable value.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In osmo-bts we have this case where the abis connection is made before the main loop is entered and there it makes sense to try all possible BSCs until one is found that is up.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-abis/+/24514">change 24514</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/libosmo-abis/+/24514"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-abis </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I7bc4345876e3c20266985270d0ddfa770437a053 </div>
<div style="display:none"> Gerrit-Change-Number: 24514 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 17 Jun 2021 10:42:23 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>