Attention is currently required from: fixeria, pespin.
lynxis lazus has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/40961?usp=email )
Change subject: Split Iu RANAP RAB PS act/deact over different layers ......................................................................
Patch Set 4:
(1 comment)
File src/sgsn/gprs_gmm.c:
https://gerrit.osmocom.org/c/osmo-sgsn/+/40961/comment/48f1dbbd_50e45702?usp... : PS4, Line 1016: if (ctx->ran_type == MM_CTX_T_UTRAN_Iu && Please don't check here for UTRAN_Iu. The check for Iu would be much better if you do it on line 1012, because Service Request is only valid for Iu and I think it is already checked earlier.
*BUT* The real reason why I don't want to have here, because gmm_auth() will be removed anyways by the vlr patch series and I want to keep the changes of your commit minimal.