<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 2: Code-Review-2</p><p style="white-space: pre-wrap; word-wrap: break-word;">It breaks the uhd ipc test backend because that does not have a vty nor does it need one, and the change was squashed at the time because that was what people wanted and it was reviewed at the time, so please do not just undo changes that were part of larger patchsets that led to known working releases.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I must admit that it's not obvious. I found the change [1] now, and IMHO it would have been better to merge that patch separately. Or at least add a comment to the code explaining why we should not assert() there, unlike all other invocations of osmo_cpu_sched_vty_apply_localthread().</p><p style="white-space: pre-wrap; word-wrap: break-word;">[1] https://gerrit.osmocom.org/c/osmo-trx/+/19649</p><p><a href="https://gerrit.osmocom.org/c/osmo-trx/+/20874">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-trx/+/20874">change 20874</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-trx/+/20874"/><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-Change-Id: I424233074cad42ae1520c92d6e627eb983303478 </div>
<div style="display:none"> Gerrit-Change-Number: 20874 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 24 Oct 2020 14:02:15 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>