pespin submitted this change.

View Change


Approvals: osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified fixeria: Looks good to me, approved
layer23: modem: sm: Properly inform SM layer of ownership transfer

Change-Id: I95ed63fe6442369ab3afb75cbd9908b87addbb3e
---
M src/host/layer23/src/modem/sm.c
1 file changed, 10 insertions(+), 0 deletions(-)

diff --git a/src/host/layer23/src/modem/sm.c b/src/host/layer23/src/modem/sm.c
index 2a404d4..56b223f 100644
--- a/src/host/layer23/src/modem/sm.c
+++ b/src/host/layer23/src/modem/sm.c
@@ -162,6 +162,7 @@
case OSMO_PRIM(OSMO_GPRS_SNDCP_SNSM_ACTIVATE, PRIM_OP_INDICATION):
LOGP(DSM, LOGL_INFO, "%s(): Rx %s\n", __func__, pdu_name);
rc = osmo_gprs_sndcp_prim_dispatch_snsm(sndcp_prim);
+ rc = 1; /* Tell SM that we take ownership of the prim. */
break;
default:
LOGP(DSM, LOGL_ERROR, "%s(): Unexpected Rx %s\n", __func__, pdu_name);

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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I95ed63fe6442369ab3afb75cbd9908b87addbb3e
Gerrit-Change-Number: 32588
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged