pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/35785?usp=email )

Change subject: modem: Improve logging when forwarding data to GRR ......................................................................

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(-)

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

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