Attention is currently required from: laforge, fixeria.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bsc/+/27885
)
Change subject: paging: Submit up to 20 paging requests in a single work iteration
......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS3:
aside from fixeria's comments: I'd appreciate
some kind of illustration why 20 was chosen. […]
First of all, this is not alone the
ultimae fix/improvmement for this code, see it as a quick improvement fixed the problem
spotted in SYS#5922 where basically we saw that osmo-bsc schedules paging at a rate of 2
sec (due to scheduling 1 pag_req every 0.5 secs).
Since in there we saw figures of ~210 req/m = ~3.5 req/s, that meant we need a rate >=4
req/s.
Since I also saw 20 to be the amount of slots provided by the credit timer, I also though
sending 20 at once was acceptable. I didn't do a big analysis and I could go for a
smaller number, like 10. But as I said in the end I expect this to be a quick &
temporary improvement step.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bsc/+/27885
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I609fa67834b426456f48f6fb2acb601c5905f178
Gerrit-Change-Number: 27885
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 25 Apr 2022 10:08:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment