pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34142 )

(

2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: l1gprs: Log slotmask when logging TBF ......................................................................

l1gprs: Log slotmask when logging TBF

Change-Id: Ib96b01c591dbcc6aa9e0527e0be499ef8aee2863 --- M src/shared/l1gprs.c 1 file changed, 13 insertions(+), 4 deletions(-)

Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified

diff --git a/src/shared/l1gprs.c b/src/shared/l1gprs.c index e4c1fc7..61b6eea 100644 --- a/src/shared/l1gprs.c +++ b/src/shared/l1gprs.c @@ -48,9 +48,9 @@ #define LOG_TBF_CFG_REQ_ARGS(req) \ (req)->tbf_ref, (req)->slotmask, ntohl((req)->start_fn)

-#define LOG_TBF_FMT "%cL-TBF#%03d" +#define LOG_TBF_FMT "%cL-TBF#%03d(slotmask=0x%02x)" #define LOG_TBF_ARGS(tbf) \ - (tbf)->uplink ? 'U' : 'D', tbf->tbf_ref + (tbf)->uplink ? 'U' : 'D', (tbf)->tbf_ref, (tbf)->slotmask

#define TDMA_FN_INVALID 0xffffffff

@@ -548,7 +548,7 @@ tbf = l1gprs_find_tbf(gprs, true, req->tbf_ref); if (tbf == NULL) { LOGP_GPRS(gprs, LOGL_ERROR, "%s(): " LOG_TBF_FMT " not found\n", - __func__, 'U', req->tbf_ref); + __func__, 'U', req->tbf_ref, req->slotmask); return -ENOENT; } l1gprs_unregister_tbf(gprs, tbf); @@ -609,7 +609,7 @@ tbf = l1gprs_find_tbf(gprs, false, req->tbf_ref); if (tbf == NULL) { LOGP_GPRS(gprs, LOGL_ERROR, "%s(): " LOG_TBF_FMT " not found\n", - __func__, 'D', req->tbf_ref); + __func__, 'D', req->tbf_ref, req->slotmask); return -ENOENT; } l1gprs_unregister_tbf(gprs, tbf);