Change in osmo-sgsn[master]: mm_iu: Expect E_PMM_PS_ATTACH when in ST_PMM_IDLE

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Mon Mar 29 18:13:41 UTC 2021


pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/23500 )

Change subject: mm_iu: Expect E_PMM_PS_ATTACH when in ST_PMM_IDLE
......................................................................

mm_iu: Expect E_PMM_PS_ATTACH when in ST_PMM_IDLE

It can happen that the MS tries to attach while SGSN's MM Iu state is
ST_PMM_IDLE (eg because UE was hard rebooted). Since Attach is a
specific case of getting a Connection Established, also allow it as a
trigger to transit to state ST_PMM_CONNECTED.

Related: SYS#5389
Change-Id: Ia74a062ddc3052faad569f1428f0ddd02e5b188d
---
M src/sgsn/gprs_mm_state_iu_fsm.c
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, but someone else must approve
  fixeria: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/sgsn/gprs_mm_state_iu_fsm.c b/src/sgsn/gprs_mm_state_iu_fsm.c
index e571026..69b4ef0 100644
--- a/src/sgsn/gprs_mm_state_iu_fsm.c
+++ b/src/sgsn/gprs_mm_state_iu_fsm.c
@@ -81,6 +81,7 @@
 static void st_pmm_idle(struct osmo_fsm_inst *fi, uint32_t event, void *data)
 {
 	switch(event) {
+	case E_PMM_PS_ATTACH:
 	case E_PMM_PS_CONN_ESTABLISH:
 		mm_state_iu_fsm_state_chg(fi, ST_PMM_CONNECTED);
 		break;
@@ -117,7 +118,10 @@
 		.action = st_pmm_connected,
 	},
 	[ST_PMM_IDLE] = {
-		.in_event_mask = X(E_PMM_IMPLICIT_DETACH) | X(E_PMM_PS_CONN_ESTABLISH),
+		.in_event_mask =
+			X(E_PMM_IMPLICIT_DETACH) |
+			X(E_PMM_PS_CONN_ESTABLISH) |
+			X(E_PMM_PS_ATTACH),
 		.out_state_mask = X(ST_PMM_DETACHED) | X(ST_PMM_CONNECTED),
 		.name = "Idle",
 		.onenter = st_pmm_idle_on_enter,

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/23500
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ia74a062ddc3052faad569f1428f0ddd02e5b188d
Gerrit-Change-Number: 23500
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann at sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210329/1aa485d3/attachment.htm>


More information about the gerrit-log mailing list