Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/33766 )
Change subject: paging: also accept IMSI strings shorter than 3 ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1: Why should we ever receive short IMSIs with strlen()<3? can you share an expected case? I don't see any.
I agree that if no IMSI is provided we should use SAPI PCU_IF_SAPI_AGCH. That is something that should be corrected in osmo-pcu. However, I guess if we do that, we still require a confirmation from osmo-bts? (I can take care of that if you want.)
Yes it's probably good in general to receive .cnf from AGCH too, since this allows also estimating time(rs) better in osmo-pcu for UL TBF assignment.