15 Feb
2023
15 Feb
'23
5:59 p.m.
Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/31339 )
Change subject: ho: remove timeout for HO_ST_WAIT_LCHAN_ACTIVE ......................................................................
Patch Set 1:
(1 comment)
File src/osmo-bsc/handover_fsm.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/31339/comment/e4dc77ca_c742dde6 PS1, Line 196: [HO_ST_WAIT_LCHAN_ACTIVE] = { .T = 23042 }, maybe leave the line with: [HO_ST_WAIT_LCHAN_ACTIVE] = { /* here some doc reference to some other timer */ }
So explain why the T23042 is not needed there (because another timer takes care of tearing down everything).
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/31339
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ibf740aaa9bddc2de85cf8087ad90bab47aac12c2
Gerrit-Change-Number: 31339
Gerrit-PatchSet: 1
Gerrit-Owner: osmith osmith@sysmocom.de
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin pespin@sysmocom.de
Gerrit-Attention: osmith osmith@sysmocom.de
Gerrit-Comment-Date: Wed, 15 Feb 2023 16:59:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment