<p>Pau Espin Pedrol <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/10743">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">osmo-trx: Add osmo_signal to stop whole transceiver chain correctly on error<br><br>Transceiver::stop() can only be called from either CTRL iface thread or<br>from main thread (running osmocom loop). That's because stop attempts to<br>cancel and then join all the other threads, which would then lock if<br>attempting to stop from some of them.<br>As a result, the best option is to indicate to the user of the<br>transceiver option (osmo-trx.cpp) to stop it in a correct fashion by<br>destroying the object from the main thread.<br><br>Change-Id: Iac1d2dbe2328e735db2d4b933cb67b1af1babca1<br>---<br>M CommonLibs/Makefile.am<br>A CommonLibs/osmo_signal.h<br>M Transceiver52M/Transceiver.cpp<br>M Transceiver52M/Transceiver.h<br>M Transceiver52M/osmo-trx.cpp<br>5 files changed, 81 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/43/10743/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/10743">change 10743</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/10743"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-trx </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Iac1d2dbe2328e735db2d4b933cb67b1af1babca1 </div>
<div style="display:none"> Gerrit-Change-Number: 10743 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@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: Pau Espin Pedrol <pespin@sysmocom.de> </div>