Attention is currently required from: laforge, fixeria, pespin.
neels has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-hnbgw/+/33169 )
Change subject: fix umts_cell_id_name(): show CID
......................................................................
Patch Set 4:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/33169/comment/faaf6407_eeeac9b9
PS3, Line 12: Use OTC_SELECT
Let me add that I don't know how it happened, but i overlooked some responses when
writing the above.
I appreciate the benchmark:
the static __thread buffer implementation 1294870
times per second on my laptop, while the OTC_SELECT variant runs "only" 808880
times per second.
It would also be interesting to compare whether there is a difference between the two
variants when the logging is turned off: is there an impact by unused static buffers?
I still don't get in general, why we should use a
more expensive OTC_SELECT approach over a static buffer approach *without a clear
requirement or reason*. It's not that it makes the code any easier to read or write.
But clearly, it does, and clearly there are reasons for it, and requirements that
OTC_SELECT meets better than static buffers do.
Using OTC_SELECT helps to completely rule out several entire families of problems in code
maintenance, <cue the previous long comment>
--
To view, visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/33169
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Id903b8579ded8b908e644808e440d373bcca8da4
Gerrit-Change-Number: 33169
Gerrit-PatchSet: 4
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 15 Jun 2023 02:51:29 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de>
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment