Attention is currently required from: fixeria.
1 comment:
Patchset:
IMO, it does not worth to introduce such a layering violation (i.e. […]
In osmo-bsc we also distinguish between PAGING COMMAND and IMMEDIATE ASSIGNMENT. and it is done in the same way: https://gitea.osmocom.org/cellular-infrastructure/osmo-bsc/src/branch/master/src/osmo-bsc/pcu_sock.c#L557
(If you think looking directly in the MAC block is a problem then we might consider adding a message type field to struct gsm_pcu_if_pch_dt)
The reason why I think that this must be fixed is the following: When we confirm PAGING COMMAND messages as well, then this has the appearance that it would be mandatory to confirm PAGING COMMAND messages. This has caused confusion in the past and we should make sure that we won't run into this again. Also since osmo-bsc does not confirm PAGING COMMAND messages we would end up having two different behaviors.
To view, visit change 32734. To unsubscribe, or for help writing mail filters, visit settings.