fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/24022 )
Change subject: trx_toolkit/trx_sniff.py: enrich decoding error messages
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
> ping
This patchset is related to the VAMOS feature (https://osmocom.org/issues/4941), for which we exceeded the time budget and which was unfortunately put on hold by the customer. I will be happy to resume working on TRXDv2 support in trx_toolkit once the ticket is active again.
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/24022
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I625f5ae0ed9373cbece5f2201759ed0b51f0db68
Gerrit-Change-Number: 24022
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Thu, 28 Apr 2022 15:03:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/24021 )
Change subject: trx_toolkit/data_if.py: enrich encoding/decoding error messages
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
> ping
This patchset is related to the VAMOS feature (https://osmocom.org/issues/4941), for which we exceeded the time budget and which was unfortunately put on hold by the customer. I will be happy to resume working on TRXDv2 support in trx_toolkit once the ticket is active again.
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/24021
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Iec80890e52f7aa24c145356b5f2e1f66a47fbd4c
Gerrit-Change-Number: 24021
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Thu, 28 Apr 2022 15:02:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27974 )
Change subject: paging: Recalculate work timer if waiting for retrans
......................................................................
paging: Recalculate work timer if waiting for retrans
If the queue is only holding requests in retransmition state, when we
add a new one, we have to re-calculate the work timer to a lower value
instead of letting it wait for the first retransmit to be ready.
Change-Id: Ibd4f8921c92f7481f0b9943041c141640ab812c8
---
M src/osmo-bsc/paging.c
1 file changed, 22 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/74/27974/1
diff --git a/src/osmo-bsc/paging.c b/src/osmo-bsc/paging.c
index fcb038b..1430c22 100644
--- a/src/osmo-bsc/paging.c
+++ b/src/osmo-bsc/paging.c
@@ -432,7 +432,28 @@
t3113_timeout_s = calculate_timer_3113(req, reqs_before_same_pgroup);
osmo_timer_schedule(&req->T3113, t3113_timeout_s, 0);
- paging_schedule_if_needed(bts_entry);
+
+ /* Trigger scheduler if needed: */
+ if (!osmo_timer_pending(&bts_entry->work_timer)) {
+ paging_handle_pending_requests(bts_entry);
+ } else if (last_initial_req == NULL) {
+ /* Worker timer is armed -> there was already one req before
+ * last_initial_req is NULL -> There were no initial requests in
+ * the list, aka the timer is waiting for retransmition,
+ * which is a longer period.
+ * Let's recaculate the time to adapt it to initial_period: */
+ struct timespec now, elapsed, tdiff;
+ osmo_clock_gettime(CLOCK_MONOTONIC, &now);
+ /* This is what used to be the first req (retrans state) in the queue: */
+ req = llist_entry(req->entry.next, struct gsm_paging_request, entry);
+ timespecsub(&now, &req->last_attempt_ts, &elapsed);
+ if (timespeccmp(&elapsed, &initial_period, <)) {
+ timespecsub(&initial_period, &elapsed, &tdiff);
+ } else {
+ tdiff = (struct timespec){.tv_sec = 0, .tv_nsec = 0 };
+ }
+ osmo_timer_schedule(&bts_entry->work_timer, tdiff.tv_sec, tdiff.tv_nsec / 1000);
+ } /* else: worker is already ongoing submitting initial requests, nothing do be done */
return 0;
}
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/27974
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ibd4f8921c92f7481f0b9943041c141640ab812c8
Gerrit-Change-Number: 27974
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newchange
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27973 )
Change subject: paging: Early stop work_timer when paging queue becomes empty
......................................................................
paging: Early stop work_timer when paging queue becomes empty
There's no need to keep the timer running, since anyway upon next
trigger it will simply early exit in paging_handle_pending_requests()
becuase there's no more work to do.
Change-Id: I096ab7231f52c741c5fded37acd5b309e1de06e3
---
M src/osmo-bsc/paging.c
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/73/27973/1
diff --git a/src/osmo-bsc/paging.c b/src/osmo-bsc/paging.c
index f2bcf96..fcb038b 100644
--- a/src/osmo-bsc/paging.c
+++ b/src/osmo-bsc/paging.c
@@ -84,6 +84,8 @@
llist_del(&to_be_deleted->entry);
bsc_subscr_put(to_be_deleted->bsub, BSUB_USE_PAGING_REQUEST);
talloc_free(to_be_deleted);
+ if (llist_empty(&paging_bts->pending_requests))
+ osmo_timer_del(&paging_bts->work_timer);
}
static void page_ms(struct gsm_paging_request *request)
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/27973
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I096ab7231f52c741c5fded37acd5b309e1de06e3
Gerrit-Change-Number: 27973
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newchange