Attention is currently required from: pespin, fixeria.
dexter has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/30873 )

Change subject: pcu_sock: guard against too many TRX
......................................................................


Patch Set 3:

(1 comment)

File src/osmo-bsc/pcu_sock.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/30873/comment/33e42c76_273cf208
PS2, Line 233:                  break;
> What I mentioned in the osmo-bts.git patch. […]
Technically its better to break as if we would continue we would run all the 
remaining TRX through the check above. In any case, the savings are minute so 
that it basically makes no difference at all.



--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/30873
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I351080a112f3d3fdf833ee7fa0d77c4cd1d13e42
Gerrit-Change-Number: 30873
Gerrit-PatchSet: 3
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Tue, 10 Jan 2023 11:41:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <[email protected]>
Gerrit-MessageType: comment

Reply via email to