Change in libosmocore[master]: gprs_ns2_vc_fsm: reset the alive elapsed timeout everytime

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Fri Feb 12 09:37:51 UTC 2021


laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/22870 )

Change subject: gprs_ns2_vc_fsm: reset the alive elapsed timeout everytime
......................................................................

gprs_ns2_vc_fsm: reset the alive elapsed timeout everytime

The alive elapsed timeout was only set once on the start of the
test procedure but not every time an ALIVE PDU was sent.

Fixes: OS#4997
Change-Id: I029696dfff21919f97ac4c33cdd82162b5ab1555
---
M src/gb/gprs_ns2_vc_fsm.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved
  daniel: Looks good to me, but someone else must approve



diff --git a/src/gb/gprs_ns2_vc_fsm.c b/src/gb/gprs_ns2_vc_fsm.c
index 88ce8ae..31cfb51 100644
--- a/src/gb/gprs_ns2_vc_fsm.c
+++ b/src/gb/gprs_ns2_vc_fsm.c
@@ -205,6 +205,7 @@
 	case NS_TOUT_TNS_TEST:
 		priv->alive.mode = NS_TOUT_TNS_ALIVE;
 		priv->alive.N = 0;
+		osmo_clock_gettime(CLOCK_MONOTONIC, &priv->alive.timer_started);
 		ns2_tx_alive(priv->nsvc);
 		osmo_timer_schedule(&priv->alive.timer, nsi->timeout[NS_TOUT_TNS_ALIVE], 0);
 		break;

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I029696dfff21919f97ac4c33cdd82162b5ab1555
Gerrit-Change-Number: 22870
Gerrit-PatchSet: 6
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210212/8981e127/attachment.htm>


More information about the gerrit-log mailing list