Change in osmo-mgw[master]: osmux: Log refcounting of osmux_handle_list

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/29310 ) Change subject: osmux: Log refcounting of osmux_handle_list ...................................................................... osmux: Log refcounting of osmux_handle_list Change-Id: Ia873e3021078976748762baaf406865149b2a090 --- M src/libosmo-mgcp/mgcp_osmux.c 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve osmith: Looks good to me, but someone else must approve pespin: Looks good to me, approved diff --git a/src/libosmo-mgcp/mgcp_osmux.c b/src/libosmo-mgcp/mgcp_osmux.c index 17acc9a..b635aef 100644 --- a/src/libosmo-mgcp/mgcp_osmux.c +++ b/src/libosmo-mgcp/mgcp_osmux.c @@ -78,10 +78,10 @@ llist_for_each_entry(h, &osmux_handle_list, head) { if (osmo_sockaddr_cmp(&h->rem_addr, rem_addr) == 0) { - LOGP(DOSMUX, LOGL_DEBUG, - "Using existing OSMUX handle for rem_addr=%s\n", - osmo_sockaddr_to_str(rem_addr)); h->refcnt++; + LOGP(DOSMUX, LOGL_DEBUG, + "Using existing OSMUX handle for addr=%s (rfcnt=%u)\n", + osmo_sockaddr_to_str(rem_addr), h->refcnt); return h; } } @@ -96,6 +96,9 @@ llist_for_each_entry(h, &osmux_handle_list, head) { if (h->in == in) { + LOGP(DOSMUX, LOGL_DEBUG, + "Putting existing OSMUX handle for addr=%s (rfcnt=%u)\n", + osmo_sockaddr_to_str(&h->rem_addr), h->refcnt); if (--h->refcnt == 0) { LOGP(DOSMUX, LOGL_INFO, "Releasing unused osmux handle for %s\n", -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/29310 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Ia873e3021078976748762baaf406865149b2a090 Gerrit-Change-Number: 29310 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged