dexter has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/29703 )

Change subject: abis_rsl: show full request reference in log
......................................................................

abis_rsl: show full request reference in log

When a CHAN RQD is received via RSL, we show ra and other parameters.
Lets also show T1, T3 and T2.

Change-Id: I78499b49ae176b736e384e193fadc0bdd669dffa
---
M src/osmo-bsc/abis_rsl.c
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  msuraev: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/src/osmo-bsc/abis_rsl.c b/src/osmo-bsc/abis_rsl.c
index 51824f5..c4e5267 100644
--- a/src/osmo-bsc/abis_rsl.c
+++ b/src/osmo-bsc/abis_rsl.c
@@ -1914,8 +1914,10 @@

        /* Determine channel request cause code */
        rqd->reason = get_reason_by_chreq(rqd->ref.ra, bts->network->neci);
-       LOG_BTS(bts, DRSL, LOGL_INFO, "CHAN RQD: reason: %s (ra=0x%02x, 
neci=0x%02x, chreq_reason=0x%02x)\n",
-               get_value_string(gsm_chreq_descs, rqd->reason), rqd->ref.ra, 
bts->network->neci, rqd->reason);
+       LOG_BTS(bts, DRSL, LOGL_INFO, "CHAN RQD: reason: %s (ra=0x%02x, t1=%d, 
t3=%d, t2=%d, neci=0x%02x, chreq_reason=0x%02x)\n",
+               get_value_string(gsm_chreq_descs, rqd->reason), rqd->ref.ra,
+               rqd->ref.t1, rqd->ref.t3_high << 3 | rqd->ref.t3_low, 
rqd->ref.t2,
+               bts->network->neci, rqd->reason);

        rate_ctr_inc(rate_ctr_group_get_ctr(bts->bts_ctrs, 
BTS_CTR_CHREQ_TOTAL));
        switch (rqd->reason) {

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I78499b49ae176b736e384e193fadc0bdd669dffa
Gerrit-Change-Number: 29703
Gerrit-PatchSet: 11
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: lynxis lazus <[email protected]>
Gerrit-Reviewer: msuraev <[email protected]>
Gerrit-CC: fixeria <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to