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


Change subject: meas: Improve logging formatting
......................................................................

meas: Improve logging formatting

Change-Id: I15d66da3f41a9f14a542b96436d65a0581bbb45a
---
M src/gprs_rlcmac_meas.cpp
1 file changed, 15 insertions(+), 7 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/89/32389/1

diff --git a/src/gprs_rlcmac_meas.cpp b/src/gprs_rlcmac_meas.cpp
index d14f405..a3d891c 100644
--- a/src/gprs_rlcmac_meas.cpp
+++ b/src/gprs_rlcmac_meas.cpp
@@ -124,8 +124,8 @@
        if (!sum)
                return -EINVAL;

-       LOGP(DRLCMACMEAS, LOGL_DEBUG, "DL Loss of TLLI 0x%08x: Received: %4d  "
-               "Lost: %4d  Sum: %4d\n", tbf->tlli(), received, lost, sum);
+       LOGP(DRLCMACMEAS, LOGL_DEBUG, "%s DL Loss: Received: %4d Lost: %4d  
Sum: %4d\n",
+            tbf_name(dl_tbf_as_tbf_const(tbf)), received, lost, sum);
 
        tbf->m_bw.dl_loss_received += received;
        tbf->m_bw.dl_loss_lost += lost;
@@ -154,9 +154,8 @@
        if (!sum)
                return -EINVAL;

-       LOGP(DRLCMACMEAS, LOGL_DEBUG, "DL packet loss of IMSI=%s / TLLI=0x%08x: 
"
-               "%d%%\n", tbf->imsi(), tbf->tlli(),
-               tbf->m_bw.dl_loss_lost * 100 / sum);
+       LOGP(DRLCMACMEAS, LOGL_DEBUG, "%s DL packet loss: %d%%\n",
+            tbf_name(dl_tbf_as_tbf_const(tbf)), tbf->m_bw.dl_loss_lost * 100 / 
sum);

        return 0;
 }
@@ -179,8 +178,8 @@
                return 0;

        tbf->m_bw.dl_throughput = (tbf->m_bw.dl_bw_octets << 10) / 
((elapsed.tv_sec << 10) + (elapsed.tv_nsec >> 20));
-       LOGP(DRLCMACMEAS, LOGL_INFO, "DL Bandwitdh of IMSI=%s / TLLI=0x%08x: "
-               "%d KBits/s\n", tbf->imsi(), tbf->tlli(), 
tbf->m_bw.dl_throughput);
+       LOGP(DRLCMACMEAS, LOGL_INFO, "%s DL Bandwitdh: %d KBits/s\n",
+            tbf_name(dl_tbf_as_tbf_const(tbf)), tbf->m_bw.dl_throughput);

        /* reset bandwidth values timestamp */
        memcpy(bw_tv, &now_tv, sizeof(*bw_tv));

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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I15d66da3f41a9f14a542b96436d65a0581bbb45a
Gerrit-Change-Number: 32389
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-MessageType: newchange

Reply via email to