Attention is currently required from: osmith.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmo-gprs/+/33811 )
Change subject: sm: Handle GMMSM-MODIFY.ind primitive
......................................................................
Patch Set 6:
(2 comments)
File include/osmocom/gprs/sm/sm_private.h:
https://gerrit.osmocom.org/c/libosmo-gprs/+/33811/comment/5f3b91e3_a2d2bab8
PS6, Line 159: struct osmo_gprs_sm_prim *gprs_sm_prim_alloc_smreg_pdp_deact_ind(void);
should everything about deact_ind go into a separate
patch? or how does it relate to GMMSM-MODIFY. […]
It's just part of the handling
of GMMSM-MODIFY.ind, see the rest of the patch.
In summary, GMMSM-MODIFY.ind is trigged by rx GMM RAU Accept, which contains a PDP Context
Status IE, which may trigger in the SM layer the deactivation of PDP context, which are
then to be announced to the upper layers (through the SMREG interface).
File src/sm/sm_prim.c:
https://gerrit.osmocom.org/c/libosmo-gprs/+/33811/comment/c79ed4a4_bd54e9dd
PS6, Line 603: */
did you leave this and the TODO below intentionally?
Yes, this is not really needed for now since it's for acknowledged mode, which
afaik we are not even implementing on the network side.
That's really another feature which may be implemented in the future. I'm just
leaving the TODO there since I was looking at related specs.
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-gprs/+/33811
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Ic765b7a565cac4abcf34d8c6868e103971d17822
Gerrit-Change-Number: 33811
Gerrit-PatchSet: 6
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 20 Jul 2023 11:10:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: comment