Attention is currently required from: dexter.
View Change
1 comment:
Patchset:
Patch Set #6:
IMO, it does not worth to introduce such a layering violation (i.e. parsing GSM RR messages in the PCUIF protocol logic). If osmo-pcu is fine with dropping confirmations for paging, I think it's fine to leave osmo-bts as it is. If some day osmo-pcu needs the confirmations for paging messages, it would simply stop discarding and will make use of them.
To view, visit change 32734. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I8b8264d28b1b1deb08774cdba58dd4c6dafe115d
Gerrit-Change-Number: 32734
Gerrit-PatchSet: 6
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-Attention: dexter <pmaier@sysmocom.de>
Gerrit-Comment-Date: Mon, 05 Jun 2023 12:56:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment