<p>laforge has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-remsim/+/26473">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Add separate log category for RSPRO protocol<br><br>Change-Id: I2e158f5181a24ae2256aae12d26865c4907eabad<br>---<br>M src/debug.c<br>M src/debug.h<br>M src/rspro_client_fsm.c<br>3 files changed, 10 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-remsim refs/changes/73/26473/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/debug.c b/src/debug.c</span><br><span>index b98ca08..a538697 100644</span><br><span>--- a/src/debug.c</span><br><span>+++ b/src/debug.c</span><br><span>@@ -36,6 +36,11 @@</span><br><span>          .loglevel = LOGL_INFO,</span><br><span>               .enabled = 1,</span><br><span>        },</span><br><span style="color: hsl(120, 100%, 40%);">+    [DRSPRO] = {</span><br><span style="color: hsl(120, 100%, 40%);">+          .name = "DRSPRO",</span><br><span style="color: hsl(120, 100%, 40%);">+           .loglevel = LOGL_INFO,</span><br><span style="color: hsl(120, 100%, 40%);">+                .enabled = 1,</span><br><span style="color: hsl(120, 100%, 40%);">+ },</span><br><span> };</span><br><span> </span><br><span> const struct log_info log_info = {</span><br><span>diff --git a/src/debug.h b/src/debug.h</span><br><span>index 2a3e5a3..51331f4 100644</span><br><span>--- a/src/debug.h</span><br><span>+++ b/src/debug.h</span><br><span>@@ -4,6 +4,7 @@</span><br><span> enum {</span><br><span>      DMAIN,</span><br><span>       DST2,</span><br><span style="color: hsl(120, 100%, 40%);">+ DRSPRO,</span><br><span> };</span><br><span> </span><br><span> extern const struct log_info log_info;</span><br><span>diff --git a/src/rspro_client_fsm.c b/src/rspro_client_fsm.c</span><br><span>index 5718015..bfd7ebd 100644</span><br><span>--- a/src/rspro_client_fsm.c</span><br><span>+++ b/src/rspro_client_fsm.c</span><br><span>@@ -68,8 +68,8 @@</span><br><span> {</span><br><span>    struct msgb *msg = rspro_enc_msg(rspro);</span><br><span>     if (!msg) {</span><br><span style="color: hsl(0, 100%, 40%);">-             LOGP(DMAIN, LOGL_ERROR, "Error encoding RSPRO: %s\n", rspro_msgt_name(rspro));</span><br><span style="color: hsl(0, 100%, 40%);">-                osmo_log_backtrace(DMAIN, LOGL_ERROR);</span><br><span style="color: hsl(120, 100%, 40%);">+                LOGP(DRSPRO, LOGL_ERROR, "Error encoding RSPRO: %s\n", rspro_msgt_name(rspro));</span><br><span style="color: hsl(120, 100%, 40%);">+             osmo_log_backtrace(DRSPRO, LOGL_ERROR);</span><br><span>              ASN_STRUCT_FREE(asn_DEF_RsproPDU, rspro);</span><br><span>            return -1;</span><br><span>   }</span><br><span>@@ -87,7 +87,7 @@</span><br><span> {</span><br><span>   if (!rspro) {</span><br><span>                LOGPFSML(srvc->fi, LOGL_ERROR, "Attempt to transmit NULL\n");</span><br><span style="color: hsl(0, 100%, 40%);">-              osmo_log_backtrace(DMAIN, LOGL_ERROR);</span><br><span style="color: hsl(120, 100%, 40%);">+                osmo_log_backtrace(DRSPRO, LOGL_ERROR);</span><br><span>              return -EINVAL;</span><br><span>      }</span><br><span>    if (osmo_fsm_inst_dispatch(srvc->fi, SRVC_E_RSPRO_TX, rspro) < 0) {</span><br><span>@@ -432,7 +432,7 @@</span><br><span>      .allstate_event_mask = S(SRVC_E_ESTABLISH) | S(SRVC_E_DISCONNECT),</span><br><span>   .allstate_action = srvc_allstate_action,</span><br><span>     .timer_cb = server_conn_fsm_timer_cb,</span><br><span style="color: hsl(0, 100%, 40%);">-   .log_subsys = DMAIN,</span><br><span style="color: hsl(120, 100%, 40%);">+  .log_subsys = DRSPRO,</span><br><span>        .event_names = server_conn_fsm_event_names,</span><br><span> };</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-remsim/+/26473">change 26473</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-remsim/+/26473"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-remsim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I2e158f5181a24ae2256aae12d26865c4907eabad </div>
<div style="display:none"> Gerrit-Change-Number: 26473 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>