<p><a href="https://gerrit.osmocom.org/10686">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/10686/1/src/osmo_gsm_tester/ms_driver.py">File src/osmo_gsm_tester/ms_driver.py:</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/10686/1/src/osmo_gsm_tester/ms_driver.py@105">Patch Set #1, Line 105:</a> <code style="font-family:monospace,monospace">        self._test_case.run_test(self._loop, self._test_duration)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Not sure right now how the SimpleLoop integrates with osmo-gsm-tester event_loop.py:EventLoop. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ok, looking at the code it seems you implement your own select here. That's not going to work nice with osmo-gsm-tester, since EventLoop is used to manage/handle events for other object types, like osmo-* slave processes being launched and see if they exited prematurely, dbus messages with dbus, smpp socket fd handling, etc.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So you should move your code to use EventLoop. It seems rgistering an FD to poll is not there supported in EventLoop rigth waya, but it should be easy to add by adding a new Glib.Source subclass in event_loop.py as explained in [1] then add APIs "(un)register_poll_fd" to EventLoop which uses those.</p><p style="white-space: pre-wrap; word-wrap: break-word;">[1] https://blogs.gnome.org/desrt/2012/05/09/glib-mainloop-sources-in-python-e-g-for-irclib/</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also related:<br>https://osmocom.org/issues/2512<br>https://osmocom.org/issues/3234</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10686">change 10686</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/10686"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-tester </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I586b6d532c6e6395b4e6f2cf2128372237d05a7d </div>
<div style="display:none"> Gerrit-Change-Number: 10686 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Holger Freyther <holger@freyther.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 29 Aug 2018 11:36:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>