<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 4:</p><p style="white-space: pre-wrap; word-wrap: break-word;">was there any feedback yet to my question raised on January 7:<br>"Doesn't every RAN connection have a fsm instance?  why not simply log through that FSM and use that FSMs log level / subsystem?"</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Sorry, missed that comment.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The useful part is to use the ran_conn->fi's id as context info.</p><p style="white-space: pre-wrap; word-wrap: break-word;">FI's log level: the FI's log level is more about the level on which to see the FSM transitions. If I LOG_RAN_CONN(LOGL_ERROR), I want to log it as error, not as debug, just because the FI is on debug.</p><p style="white-space: pre-wrap; word-wrap: break-word;">FSM's subsys: a ran_conn has various stuff happening: BSSMAP, RANAP, CC, SMS, USSD, MM, RR, ... and so far a lot of logging uses the conn's (so far vsub) context as logging context info. If we used the FSM's logging subsys for all of those, then we would substantially flatten the use of logging categories.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In the end all that we need from the FI is its logging context.<br>But I realize that I can indeed use LOGPFSMSL() with identical features, in next patch set.</p><p><a href="https://gerrit.osmocom.org/12470">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12470">change 12470</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/12470"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I0a7809840428b1e028df6eb683bc5ffcc8df474a </div>
<div style="display:none"> Gerrit-Change-Number: 12470 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@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: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 11 Jan 2019 17:48:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>