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);
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34142
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Ib96b01c591dbcc6aa9e0527e0be499ef8aee2863
Gerrit-Change-Number: 34142
Gerrit-PatchSet: 8
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
