pespin submitted this change.

View Change

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
osmux: Log refcounting of osmux_handle_list

Change-Id: Ia873e3021078976748762baaf406865149b2a090
---
M src/libosmo-mgcp/mgcp_osmux.c
1 file changed, 6 insertions(+), 3 deletions(-)

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 change 29310. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: Ia873e3021078976748762baaf406865149b2a090
Gerrit-Change-Number: 29310
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged