Change in libosmocore[master]: fsm: support graceful osmo_fsm_inst_term() cascades

Harald Welte gerrit-no-reply at
Wed Mar 27 07:36:47 UTC 2019

Harald Welte has posted comments on this change. ( )

Change subject: fsm: support graceful osmo_fsm_inst_term() cascades

Patch Set 4:

Commit Message:
PS4, Line 33: Note: at least osmo-msc's msc_vlr_tests' expected output needs to be adjusted
            : after merging this, because of logging changes for FSM deallocations
this is *very* bad.  Is there any way we can avoid this?
PS4, Line 36: This patch's mechanism is not thread safe. It might be made thread safe
            : if the new global variable osmo_fsm_inst_terminating_root were handled
            : per-thread instead of once globally.
please mark the related global variables as __thread right away. It doesn't cost us anything, AFAIK.
File include/osmocom/core/fsm.h:
PS4, Line 118: bool terminating;
as this breaks ABI, please add TODO-RELEASE entry.
File src/fsm.c:
PS4, Line 397: static int osmo_fsm_inst_terminating = 0;
             : static struct osmo_fsm_inst *osmo_fsm_inst_terminating_root = NULL;
mark as __thread, possibly even put both in a struct and then have a "static __thread struct" for tidyness?

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8eda67540a1cd444491beb7856b9fcd0a3143b18
Gerrit-Change-Number: 13392
Gerrit-PatchSet: 4
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at>
Gerrit-CC: Harald Welte <laforge at>
Gerrit-Comment-Date: Wed, 27 Mar 2019 07:36:47 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list