Attention is currently required from: jolly, fixeria, pespin. dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/32694 )
Change subject: gprs_rlcmac: also use PCU_IF_SAPI_PCH_DT for paging MAC blocks ...................................................................... Patch Set 3: (2 comments) File src/gprs_rlcmac.c: https://gerrit.osmocom.org/c/osmo-pcu/+/32694/comment/e87f1a09_fc3b2929 PS1, Line 49: mi->tmsi > gprs_rlcmac_paging_request() is called from gprs_bssgp_pcu. […] I have now analyzed this further. The PCUIF really behaves a bit confusing here. It is true that osmo-bts confirms IMMEDIATE ASSIGNMENT and PAGING COMMANDs but osmo-pcu will silently drop the confirmations for PAGING COMMAND messages. I have already fixed this in osmo-bts and we also do not have to worry about the TMSI, we can just pass GSM_RESERVED_TMSI and everything will be fine. File src/gprs_rlcmac.c: https://gerrit.osmocom.org/c/osmo-pcu/+/32694/comment/f05f4833_6b7d80d3 PS2, Line 33: The caller must use a TMSI/TLLI > Is there anything in the specs. […] I just deducted that from the code, we always set mi.type to GSM_MT_TYPE_TMSI. I just wanted to make sure that we won't overlook anything if we should change this. But since we do not need the TMSI/TLLI for the confirmation anymore (see below) this line is no longer needed. -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/32694 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: I99cfe373fa157cfb32b74c113ad9935347653a71 Gerrit-Change-Number: 32694 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: jolly <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: jolly <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Comment-Date: Tue, 16 May 2023 14:48:45 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <[email protected]> Comment-In-Reply-To: dexter <[email protected]> Gerrit-MessageType: comment
