Change in osmo-msc[master]: Add SGs Interface

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
Wed Jan 23 00:25:19 UTC 2019


Neels Hofmeyr has posted comments on this change. ( https://gerrit.osmocom.org/11642 )

Change subject: Add SGs Interface
......................................................................


Patch Set 33:

(1 comment)

https://gerrit.osmocom.org/#/c/11642/33/src/libmsc/sgs_iface.c
File src/libmsc/sgs_iface.c:

https://gerrit.osmocom.org/#/c/11642/33/src/libmsc/sgs_iface.c@112
PS33, Line 112: 	ran_conn_put(conn, RAN_CONN_USE_SGs_SERVICE);
I need to understand the intention, so I can move this to the inter-MSC role stuff without breaking it:

I thought the above ran_conn_get(USE_SGs) would remain in place until the SGs conn gets released (if there is such a thing).
Why is this put() down again immediately?
Or, from another angle, why do we then need to get() it in the first place? (does it get deallocated by those events otherwise?)
And then, in the end of this function, the conn that we return, is it only used by compl_l3 (??) and gets deallocated immediately?



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

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I73359925fc1ca72b33a1466e6ac41307f2f0b11d
Gerrit-Change-Number: 11642
Gerrit-PatchSet: 33
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: dexter <pmaier at sysmocom.de>
Gerrit-CC: Stefan Sperling <stsp at stsp.name>
Gerrit-Comment-Date: Wed, 23 Jan 2019 00:25:19 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190123/8326acc5/attachment.htm>


More information about the gerrit-log mailing list