laforge submitted this change.

View Change


Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
mobile: log state/event details on error

Change-Id: I24a4635e9a9a0984c0e1036e8b6ea19b08a06223
---
M src/host/layer23/src/mobile/gsm48_mm.c
1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/host/layer23/src/mobile/gsm48_mm.c b/src/host/layer23/src/mobile/gsm48_mm.c
index fb7dcf8..265b720 100644
--- a/src/host/layer23/src/mobile/gsm48_mm.c
+++ b/src/host/layer23/src/mobile/gsm48_mm.c
@@ -4319,7 +4319,8 @@
&& ((1 << mm->substate) & eventstatelist[i].substates))
break;
if (i == EVENTSLLEN) {
- LOGP(DMM, LOGL_NOTICE, "Message unhandled at this state.\n");
+ LOGP(DMM, LOGL_NOTICE, "Message %s unhandled in state %s.\n",
+ get_mmevent_name(msg_type), gsm48_mm_state_names[mm->state]);
return 0;
}


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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I24a4635e9a9a0984c0e1036e8b6ea19b08a06223
Gerrit-Change-Number: 31492
Gerrit-PatchSet: 3
Gerrit-Owner: msuraev <msuraev@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged