Change in osmo-trx[master]: Transceiver: Lower some log levels which have an associated counter

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Fri Jul 17 16:34:59 UTC 2020


pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-trx/+/19307 )


Change subject: Transceiver: Lower some log levels which have an associated counter
......................................................................

Transceiver: Lower some log levels which have an associated counter

They are left as INFO instead of DEBUG since they show possible
timing/performance issues in the setup.

Change-Id: I8aab10054ac89c29b871259fdbe59636723ddfb6
---
M Transceiver52M/Transceiver.cpp
1 file changed, 5 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/07/19307/1

diff --git a/Transceiver52M/Transceiver.cpp b/Transceiver52M/Transceiver.cpp
index 3be7746..55d0acd 100644
--- a/Transceiver52M/Transceiver.cpp
+++ b/Transceiver52M/Transceiver.cpp
@@ -445,7 +445,7 @@
     mtx->lock();
 
     while ((burst = mTxPriorityQueues[i].getStaleBurst(nowTime))) {
-      LOGCHAN(i, DTRXDDL, NOTICE) << "dumping STALE burst in TRX->SDR interface ("
+      LOGCHAN(i, DTRXDDL, INFO) << "dumping STALE burst in TRX->SDR interface ("
                   << burst->getTime() <<" vs " << nowTime << "), retrans=" << state->mRetrans;
       state->ctrs.tx_stale_bursts++;
       ratectr_changed = true;
@@ -469,7 +469,7 @@
       modFN = nowTime.FN() % state->fillerModulus[TN];
       bursts[i] = state->fillerTable[modFN][TN];
       if (i == 0 && state->mFiller == FILLER_ZERO) {
-        LOGCHAN(i, DTRXDDL, NOTICE) << "No Tx burst available for " << nowTime
+        LOGCHAN(i, DTRXDDL, INFO) << "No Tx burst available for " << nowTime
                                     << ", retrans=" << state->mRetrans;
         state->ctrs.tx_unavailable_bursts++;
         ratectr_changed = true;
@@ -1073,12 +1073,12 @@
     if (delta == 1) {
         /* usual expected scenario, continue code flow */
     } else if (delta == 0) {
-      LOGCHAN(chan, DTRXDDL, NOTICE) << "Rx TRXD msg with repeated FN " << currTime;
+      LOGCHAN(chan, DTRXDDL, INFO) << "Rx TRXD msg with repeated FN " << currTime;
       state->ctrs.tx_trxd_fn_repeated++;
       dispatch_trx_rate_ctr_change(state, chan);
       return true;
     } else if (delta < 0) {
-      LOGCHAN(chan, DTRXDDL, NOTICE) << "Rx TRXD msg with previous FN " << currTime
+      LOGCHAN(chan, DTRXDDL, INFO) << "Rx TRXD msg with previous FN " << currTime
                                      << " vs last " << state->last_dl_time_rcv[tn];
        state->ctrs.tx_trxd_fn_outoforder++;
        dispatch_trx_rate_ctr_change(state, chan);
@@ -1088,7 +1088,7 @@
          * on consecutive FNs in TRX0 since it must transmit continuously in all
          * setups. Also, osmo-trx supports optionally filling empty bursts on
          * its own. In that case bts-trx is not obliged to submit all bursts. */
-      LOGCHAN(chan, DTRXDDL, NOTICE) << "Rx TRXD msg with future FN " << currTime
+      LOGCHAN(chan, DTRXDDL, INFO) << "Rx TRXD msg with future FN " << currTime
                                      << " vs last " << state->last_dl_time_rcv[tn]
                                      << ", " << delta - 1 << " FN lost";
       state->ctrs.tx_trxd_fn_skipped += delta - 1;

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/19307
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I8aab10054ac89c29b871259fdbe59636723ddfb6
Gerrit-Change-Number: 19307
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200717/3c0653bd/attachment.htm>


More information about the gerrit-log mailing list