Change in libosmocore[master]: log: fsm: fix logging of negative T upon timeout

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/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Thu Feb 21 01:37:00 UTC 2019


Neels Hofmeyr has uploaded this change for review. ( https://gerrit.osmocom.org/12988


Change subject: log: fsm: fix logging of negative T upon timeout
......................................................................

log: fsm: fix logging of negative T upon timeout

fi->T values are int, i.e. can be negative. Do not log them as unsigned, i.e.
print "Timeout of T-1" instead of "Timeout of T4294967295".

Negative T timers are useful to indicate freely invented timer numbers, to
distinguish them from proper 3GPP defined T numbers. So far I was using numbers
like T993210 or T9999 for invented T, but T-1, T-2 etc. is a better solution.

Change-Id: I3a59457623da9309fbbda235fe18fadd1636bff6
---
M src/fsm.c
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/88/12988/1

diff --git a/src/fsm.c b/src/fsm.c
index eb457a1..900ada6 100644
--- a/src/fsm.c
+++ b/src/fsm.c
@@ -205,9 +205,9 @@
 {
 	struct osmo_fsm_inst *fi = data;
 	struct osmo_fsm *fsm = fi->fsm;
-	uint32_t T = fi->T;
+	int32_t T = fi->T;
 
-	LOGPFSM(fi, "Timeout of T%u\n", fi->T);
+	LOGPFSM(fi, "Timeout of T%d\n", fi->T);
 
 	if (fsm->timer_cb) {
 		int rc = fsm->timer_cb(fi);

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3a59457623da9309fbbda235fe18fadd1636bff6
Gerrit-Change-Number: 12988
Gerrit-PatchSet: 1
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190221/5dead164/attachment.htm>


More information about the gerrit-log mailing list