pespin submitted this change.

View Change


Approvals: Jenkins Builder: Verified jolly: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
modem: Improve logging when forwarding data to GRR

Change-Id: Idf80c0fa1c8293afbe354442a72f5b1c5a84f9e1
---
M src/host/layer23/src/modem/llc.c
1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/host/layer23/src/modem/llc.c b/src/host/layer23/src/modem/llc.c
index 142f3cf..fe920ee 100644
--- a/src/host/layer23/src/modem/llc.c
+++ b/src/host/layer23/src/modem/llc.c
@@ -142,7 +142,12 @@

switch (llc_prim->oph.sap) {
case OSMO_GPRS_LLC_SAP_GRR:
- LOGP(DLLC, LOGL_DEBUG, "%s(): Rx %s ll=[%s]\n", __func__, pdu_name,
+ LOGP(DLLC, LOGL_DEBUG, "%s(): Rx %s TLLI=0x%08x SAPI=%s RadioPrio=%u ll=[%s]\n",
+ __func__, pdu_name, llc_prim->grr.tlli,
+ /* data_req.sapi in same union pos: */
+ osmo_gprs_llc_sapi_name(llc_prim->grr.unitdata_req.sapi),
+ /* data_req.radio_prio in same union pos: */
+ llc_prim->grr.unitdata_req.radio_prio,
osmo_hexdump(llc_prim->grr.ll_pdu, llc_prim->grr.ll_pdu_len));
/* Forward it to lower layers, pass ownership over to RLCMAC: */
/* Optimization: LLC-GRR-UNITDATA-IND is 1-to-1 ABI compatible with

To view, visit change 35785. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Idf80c0fa1c8293afbe354442a72f5b1c5a84f9e1
Gerrit-Change-Number: 35785
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: jolly <andreas@eversberg.eu>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged