pespin submitted this change.

View Change



2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified
l1gprs: Log slotmask when logging TBF

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

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 change 34142. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Ib96b01c591dbcc6aa9e0527e0be499ef8aee2863
Gerrit-Change-Number: 34142
Gerrit-PatchSet: 8
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged