pespin has uploaded this change for review. (
https://gerrit.osmocom.org/c/libosmo-sigtran/+/42108?usp=email )
Change subject: ss7_as: Optimize ASP Tx selection in Override traffic mode
......................................................................
ss7_as: Optimize ASP Tx selection in Override traffic mode
Cache the last selected ASP, and expect it to still be the active ASP
most of the time until it changes.
Change-Id: I3d480d23591f4bd216293be60b22389b182fd8f3
---
M src/ss7_as.c
1 file changed, 10 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran
refs/changes/08/42108/1
diff --git a/src/ss7_as.c b/src/ss7_as.c
index f5f7c82..aa53e9d 100644
--- a/src/ss7_as.c
+++ b/src/ss7_as.c
@@ -402,8 +402,16 @@
{
struct ss7_as_asp_assoc *assoc;
- llist_for_each_entry(assoc, &as->asp_list, as_entry) {
- if (osmo_ss7_asp_active(assoc->asp))
+ /* Hot path: Override traffic mode has only max 1 active ASP at a time.
+ * Unless there's a change in state, the last ASP used to transmit is
most
+ * probably the active one: */
+ if (as->last_asp_idx_sent &&
osmo_ss7_asp_active(as->last_asp_idx_sent->asp))
+ return as->last_asp_idx_sent->asp;
+
+ /* Slow path: Activate ASP changed, look it up: */
+ for (unsigned int i = 0; i < as->num_asps; i++) {
+ assoc = ss7_as_asp_assoc_llist_round_robin(as,
&as->last_asp_idx_sent);
+ if (assoc && osmo_ss7_asp_active(assoc->asp))
return assoc->asp;
}
return NULL;
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/42108?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I3d480d23591f4bd216293be60b22389b182fd8f3
Gerrit-Change-Number: 42108
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
