January 2025 - gerrit-log - lists.osmocom.org

[M] Change in osmo-mgw[master]: mgw: Rename and move code freeing endp connection

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39183?usp=email ) Change subject: mgw: Rename and move code freeing endp connection ...................................................................... mgw: Rename and move code freeing endp connection All the code to free conns in an endpoint is really entanged, which also makes it inefficient in several places because the list of conns is iterated twice, or even ~N*2. * Move code freeing the conn object to its own mgcp_conn_free() function. * Rename functions acting on endp to be mgcp_endp_* prefix, and move them to mgcp_endp.{c,h}. * Remove old mgcp_conn_free(endp, id) since it's not really needed anymore. Change-Id: I9a87a07699dd8aa7856d5036e9b95a4ddf699a15 --- M include/osmocom/mgcp/mgcp_conn.h M include/osmocom/mgcp/mgcp_endp.h M src/libosmo-mgcp/mgcp_conn.c M src/libosmo-mgcp/mgcp_endp.c M src/libosmo-mgcp/mgcp_protocol.c M tests/mgcp/mgcp_test.c 6 files changed, 44 insertions(+), 53 deletions(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve daniel: Looks good to me, but someone else must approve diff --git a/include/osmocom/mgcp/mgcp_conn.h b/include/osmocom/mgcp/mgcp_conn.h index 4f7b705..6219579 100644 --- a/include/osmocom/mgcp/mgcp_conn.h +++ b/include/osmocom/mgcp/mgcp_conn.h @@ -239,12 +239,10 @@ struct mgcp_conn *mgcp_conn_alloc(void *ctx, struct mgcp_endpoint *endp, enum mgcp_conn_type type, char *name); +void mgcp_conn_free(struct mgcp_conn *conn); struct mgcp_conn *mgcp_conn_get(struct mgcp_endpoint *endp, const char *id); struct mgcp_conn_rtp *mgcp_conn_get_rtp(struct mgcp_endpoint *endp, const char *id); -void mgcp_conn_free(struct mgcp_endpoint *endp, const char *id); -void mgcp_conn_free_oldest(struct mgcp_endpoint *endp); -void mgcp_conn_free_all(struct mgcp_endpoint *endp); char *mgcp_conn_dump(struct mgcp_conn *conn); struct mgcp_conn *mgcp_find_dst_conn(struct mgcp_conn *conn); struct mgcp_conn *mgcp_conn_get_oldest(struct mgcp_endpoint *endp); diff --git a/include/osmocom/mgcp/mgcp_endp.h b/include/osmocom/mgcp/mgcp_endp.h index 7bf034b..8e5b4cc 100644 --- a/include/osmocom/mgcp/mgcp_endp.h +++ b/include/osmocom/mgcp/mgcp_endp.h @@ -141,6 +141,8 @@ bool mgcp_endp_avail(struct mgcp_endpoint *endp); void mgcp_endp_add_conn(struct mgcp_endpoint *endp, struct mgcp_conn *conn); void mgcp_endp_remove_conn(struct mgcp_endpoint *endp, struct mgcp_conn *conn); +void mgcp_endp_free_conn_oldest(struct mgcp_endpoint *endp); +void mgcp_endp_free_conn_all(struct mgcp_endpoint *endp); void mgcp_endp_strip_name(char *epname_stripped, const char *epname, const struct mgcp_trunk *trunk); struct mgcp_endpoint *mgcp_endp_find_specific(const char *epname, diff --git a/src/libosmo-mgcp/mgcp_conn.c b/src/libosmo-mgcp/mgcp_conn.c index 5eb4897..9c60dfa 100644 --- a/src/libosmo-mgcp/mgcp_conn.c +++ b/src/libosmo-mgcp/mgcp_conn.c @@ -146,7 +146,7 @@ { struct mgcp_conn *conn = data; LOGPCONN(conn, DLMGCP, LOGL_ERROR, "connection timed out!\n"); - mgcp_conn_free(conn->endp, conn->id); + mgcp_conn_free(conn); } void mgcp_conn_watchdog_kick(struct mgcp_conn *conn) @@ -286,20 +286,17 @@ rate_ctr_inc(rate_ctr_group_get_ctr(all_stats, RTP_NUM_CONNECTIONS)); } -/*! free a connection by its ID. - * \param[in] endp associated endpoint - * \param[in] id identification number of the connection */ -void mgcp_conn_free(struct mgcp_endpoint *endp, const char *id) +/*! free a connection + * \param[in] conn the conn to free. May be NULL. +*/ +void mgcp_conn_free(struct mgcp_conn *conn) { - struct mgcp_conn *conn; - - conn = mgcp_conn_get(endp, id); if (!conn) return; switch (conn->type) { case MGCP_CONN_TYPE_RTP: - aggregate_rtp_conn_stats(endp, &conn->u.rtp); + aggregate_rtp_conn_stats(conn->endp, &conn->u.rtp); mgcp_rtp_conn_cleanup(&conn->u.rtp); break; default: @@ -310,45 +307,11 @@ } osmo_timer_del(&conn->watchdog); - mgcp_endp_remove_conn(endp, conn); + mgcp_endp_remove_conn(conn->endp, conn); /* WARN: endp may have be freed after call to mgcp_endp_remove_conn */ talloc_free(conn); } -/*! free oldest connection in the list. - * \param[in] endp associated endpoint */ -void mgcp_conn_free_oldest(struct mgcp_endpoint *endp) -{ - struct mgcp_conn *conn; - - if (llist_empty(&endp->conns)) - return; - - conn = llist_last_entry(&endp->conns, struct mgcp_conn, entry); - if (!conn) - return; - - mgcp_conn_free(endp, conn->id); -} - -/*! free all connections at once. - * \param[in] endp associated endpoint */ -#if defined(__has_attribute) -#if __has_attribute(no_sanitize) -__attribute__((no_sanitize("undefined"))) /* ubsan detects a misaligned load */ -#endif -#endif -void mgcp_conn_free_all(struct mgcp_endpoint *endp) -{ - struct mgcp_conn *conn; - - /* Drop all items in the list, might be consecutive! */ - while ((conn = llist_first_entry_or_null(&endp->conns, struct mgcp_conn, entry))) - mgcp_conn_free(endp, conn->id); - - return; -} - /*! dump basic connection information to human readable string. * \param[in] conn to dump * \returns human readable string */ diff --git a/src/libosmo-mgcp/mgcp_endp.c b/src/libosmo-mgcp/mgcp_endp.c index 4c870ec..6973b43 100644 --- a/src/libosmo-mgcp/mgcp_endp.c +++ b/src/libosmo-mgcp/mgcp_endp.c @@ -655,6 +655,35 @@ mgcp_endp_release(endp); } +/*! free oldest connection in the list. + * \param[in] endp associated endpoint */ +void mgcp_endp_free_conn_oldest(struct mgcp_endpoint *endp) +{ + struct mgcp_conn *conn; + + if (llist_empty(&endp->conns)) + return; + + conn = llist_last_entry(&endp->conns, struct mgcp_conn, entry); + mgcp_conn_free(conn); +} + +/*! free all connections at once. + * \param[in] endp associated endpoint */ +#if defined(__has_attribute) +#if __has_attribute(no_sanitize) +__attribute__((no_sanitize("undefined"))) /* ubsan detects a misaligned load */ +#endif +#endif +void mgcp_endp_free_conn_all(struct mgcp_endpoint *endp) +{ + struct mgcp_conn *conn; + + /* Drop all items in the list, might be consecutive! */ + while ((conn = llist_first_entry_or_null(&endp->conns, struct mgcp_conn, entry))) + mgcp_conn_free(conn); +} + /*! release endpoint, all open connections are closed. * \param[in] endp endpoint to release */ void mgcp_endp_release(struct mgcp_endpoint *endp) @@ -665,7 +694,7 @@ * all connections have been removed already. In case * that there are still connections open (e.g. when * RSIP is executed), free them all at once. */ - mgcp_conn_free_all(endp); + mgcp_endp_free_conn_all(endp); /* We must only decrement the stat item when the endpoint as actually * claimed. An endpoint is claimed when a call-id is set */ diff --git a/src/libosmo-mgcp/mgcp_protocol.c b/src/libosmo-mgcp/mgcp_protocol.c index 47fa30b..dbe6760 100644 --- a/src/libosmo-mgcp/mgcp_protocol.c +++ b/src/libosmo-mgcp/mgcp_protocol.c @@ -975,7 +975,7 @@ /* There is no more room for a connection, make some * room by blindly tossing the oldest of the two two * connections */ - mgcp_conn_free_oldest(endp); + mgcp_endp_free_conn_oldest(endp); } else { /* There is no more room for a connection, leave * everything as it is and return with an error */ @@ -1020,7 +1020,6 @@ "CRCX: unable to allocate RTP connection\n"); rate_ctr_inc(rate_ctr_group_get_ctr(rate_ctrs, MGCP_CRCX_FAIL_ALLOC_CONN)); goto error2; - } conn = mgcp_conn_get_rtp(endp, _conn->id); @@ -1487,7 +1486,7 @@ /* delete connection */ LOGPCONN(conn->conn, DLMGCP, LOGL_DEBUG, "DLCX: deleting conn:%s\n", mgcp_conn_dump(conn->conn)); - mgcp_conn_free(endp, conn_id); + mgcp_conn_free(conn->conn); LOGPENDP(endp, DLMGCP, LOGL_NOTICE, "DLCX: connection successfully deleted\n"); diff --git a/tests/mgcp/mgcp_test.c b/tests/mgcp/mgcp_test.c index de359d5..44927e1 100644 --- a/tests/mgcp/mgcp_test.c +++ b/tests/mgcp/mgcp_test.c @@ -1230,7 +1230,7 @@ pl_test_dat[i].expected); } - mgcp_conn_free_all(&endp); + mgcp_endp_free_conn_all(&endp); } talloc_free(trunk); @@ -1513,7 +1513,7 @@ } force_monotonic_time_us = -1; - mgcp_conn_free_all(&endp); + mgcp_endp_free_conn_all(&endp); talloc_free(trunk); talloc_free(cfg); } -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39183?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I9a87a07699dd8aa7856d5036e9b95a4ddf699a15 Gerrit-Change-Number: 39183 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

5 months, 2 weeks

1
0
0 0

[M] Change in osmo-mgw[master]: mgw: Rename and move several get_conn funcs acting on endp object

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39184?usp=email ) Change subject: mgw: Rename and move several get_conn funcs acting on endp object ...................................................................... mgw: Rename and move several get_conn funcs acting on endp object These functions act on the endp object, hence prefix them with mgcp_endp_* and move them to mgcp_endp.{c,h}. Change-Id: Ifff01331db68998e9e23f99d8836d96022d550c2 --- M include/osmocom/mgcp/mgcp_conn.h M include/osmocom/mgcp/mgcp_endp.h M src/libosmo-mgcp/mgcp_conn.c M src/libosmo-mgcp/mgcp_e1.c M src/libosmo-mgcp/mgcp_endp.c M src/libosmo-mgcp/mgcp_msg.c M src/libosmo-mgcp/mgcp_protocol.c M src/libosmo-mgcp/mgcp_vty.c M tests/mgcp/mgcp_test.c 9 files changed, 87 insertions(+), 86 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved daniel: Looks good to me, but someone else must approve diff --git a/include/osmocom/mgcp/mgcp_conn.h b/include/osmocom/mgcp/mgcp_conn.h index 6219579..de9bae6 100644 --- a/include/osmocom/mgcp/mgcp_conn.h +++ b/include/osmocom/mgcp/mgcp_conn.h @@ -240,10 +240,6 @@ struct mgcp_conn *mgcp_conn_alloc(void *ctx, struct mgcp_endpoint *endp, enum mgcp_conn_type type, char *name); void mgcp_conn_free(struct mgcp_conn *conn); -struct mgcp_conn *mgcp_conn_get(struct mgcp_endpoint *endp, const char *id); -struct mgcp_conn_rtp *mgcp_conn_get_rtp(struct mgcp_endpoint *endp, - const char *id); char *mgcp_conn_dump(struct mgcp_conn *conn); struct mgcp_conn *mgcp_find_dst_conn(struct mgcp_conn *conn); -struct mgcp_conn *mgcp_conn_get_oldest(struct mgcp_endpoint *endp); void mgcp_conn_watchdog_kick(struct mgcp_conn *conn); diff --git a/include/osmocom/mgcp/mgcp_endp.h b/include/osmocom/mgcp/mgcp_endp.h index 8e5b4cc..4f41818 100644 --- a/include/osmocom/mgcp/mgcp_endp.h +++ b/include/osmocom/mgcp/mgcp_endp.h @@ -148,3 +148,8 @@ struct mgcp_endpoint *mgcp_endp_find_specific(const char *epname, const struct mgcp_trunk *trunk); void mgcp_endp_release(struct mgcp_endpoint *endp); + +struct mgcp_conn *mgcp_endp_get_conn(struct mgcp_endpoint *endp, const char *id); +struct mgcp_conn *mgcp_endp_get_conn_oldest(struct mgcp_endpoint *endp); +struct mgcp_conn_rtp *mgcp_endp_get_conn_rtp(struct mgcp_endpoint *endp, + const char *id); diff --git a/src/libosmo-mgcp/mgcp_conn.c b/src/libosmo-mgcp/mgcp_conn.c index 9c60dfa..9d03550 100644 --- a/src/libosmo-mgcp/mgcp_conn.c +++ b/src/libosmo-mgcp/mgcp_conn.c @@ -74,7 +74,7 @@ /* ensure that the generated conn_id is unique * for this endpoint */ - if (!mgcp_conn_get_rtp(endp, id_hex)) { + if (!mgcp_endp_get_conn_rtp(endp, id_hex)) { osmo_strlcpy(id, id_hex, MGCP_CONN_ID_MAXLEN); return 0; } @@ -212,58 +212,6 @@ return conn; } -/*! find a connection by its ID. - * \param[in] endp associated endpoint - * \param[in] id identification number of the connection - * \returns pointer to allocated connection, NULL if not found */ -struct mgcp_conn *mgcp_conn_get(struct mgcp_endpoint *endp, const char *id) -{ - struct mgcp_conn *conn; - const char *id_upper; - const char *conn_id; - - if (!id || !*id) - return NULL; - - /* Ignore leading zeros in needle */ - while (*id == '0') - id++; - - /* Use uppercase to compare identifiers, to avoid mismatches: RFC3435 2.1.3.2 "Names of - * Connections" defines the id as a hex string, so clients may return lower case hex even though - * we sent upper case hex in the CRCX response. */ - id_upper = osmo_str_toupper(id); - - llist_for_each_entry(conn, &endp->conns, entry) { - /* Ignore leading zeros in haystack */ - for (conn_id=conn->id; *conn_id == '0'; conn_id++); - - if (strcmp(conn_id, id_upper) == 0) - return conn; - } - - return NULL; -} - -/*! find an RTP connection by its ID. - * \param[in] endp associated endpoint - * \param[in] id identification number of the connection - * \returns pointer to allocated connection, NULL if not found */ -struct mgcp_conn_rtp *mgcp_conn_get_rtp(struct mgcp_endpoint *endp, - const char *id) -{ - struct mgcp_conn *conn; - - conn = mgcp_conn_get(endp, id); - if (!conn) - return NULL; - - if (conn->type == MGCP_CONN_TYPE_RTP) - return &conn->u.rtp; - - return NULL; -} - static void aggregate_rtp_conn_stats(struct mgcp_endpoint *endp, struct mgcp_conn_rtp *conn_rtp) { struct rate_ctr_group *all_stats = endp->trunk->ratectr.all_rtp_conn_stats; @@ -377,16 +325,6 @@ return NULL; } -/*! get oldest connection in the list. - * \param[in] endp associated endpoint */ -struct mgcp_conn *mgcp_conn_get_oldest(struct mgcp_endpoint *endp) -{ - if (llist_empty(&endp->conns)) - return NULL; - - return llist_last_entry(&endp->conns, struct mgcp_conn, entry); -} - const struct value_string mgcp_conn_rtp_type_names[] = { { MGCP_RTP_DEFAULT, "rtp" }, { MGCP_RTP_OSMUX, "osmux" }, diff --git a/src/libosmo-mgcp/mgcp_e1.c b/src/libosmo-mgcp/mgcp_e1.c index 1428824..fac1ff6 100644 --- a/src/libosmo-mgcp/mgcp_e1.c +++ b/src/libosmo-mgcp/mgcp_e1.c @@ -656,7 +656,7 @@ /* In order to determine the codec, find the oldest connection on * the endpoint and use its codec information. Normally on an E1 * endpoint no more than one connection should exist. */ - conn = mgcp_conn_get_oldest(endp); + conn = mgcp_endp_get_conn_oldest(endp); OSMO_ASSERT(conn); codec = conn->u.rtp.end.codec; OSMO_ASSERT(codec); diff --git a/src/libosmo-mgcp/mgcp_endp.c b/src/libosmo-mgcp/mgcp_endp.c index 6973b43..b2ece20 100644 --- a/src/libosmo-mgcp/mgcp_endp.c +++ b/src/libosmo-mgcp/mgcp_endp.c @@ -684,6 +684,68 @@ mgcp_conn_free(conn); } +/*! find a connection by its ID. + * \param[in] endp associated endpoint + * \param[in] id identification number of the connection + * \returns pointer to allocated connection, NULL if not found */ +struct mgcp_conn *mgcp_endp_get_conn(struct mgcp_endpoint *endp, const char *id) +{ + struct mgcp_conn *conn; + const char *id_upper; + const char *conn_id; + + if (!id || !*id) + return NULL; + + /* Ignore leading zeros in needle */ + while (*id == '0') + id++; + + /* Use uppercase to compare identifiers, to avoid mismatches: RFC3435 2.1.3.2 "Names of + * Connections" defines the id as a hex string, so clients may return lower case hex even though + * we sent upper case hex in the CRCX response. */ + id_upper = osmo_str_toupper(id); + + llist_for_each_entry(conn, &endp->conns, entry) { + /* Ignore leading zeros in haystack */ + for (conn_id = conn->id; *conn_id == '0'; conn_id++); + + if (strcmp(conn_id, id_upper) == 0) + return conn; + } + + return NULL; +} + +/*! get oldest connection in the list. + * \param[in] endp associated endpoint */ +struct mgcp_conn *mgcp_endp_get_conn_oldest(struct mgcp_endpoint *endp) +{ + if (llist_empty(&endp->conns)) + return NULL; + + return llist_last_entry(&endp->conns, struct mgcp_conn, entry); +} + +/*! find an RTP connection by its ID. + * \param[in] endp associated endpoint + * \param[in] id identification number of the connection + * \returns pointer to allocated connection, NULL if not found */ +struct mgcp_conn_rtp *mgcp_endp_get_conn_rtp(struct mgcp_endpoint *endp, + const char *id) +{ + struct mgcp_conn *conn; + + conn = mgcp_endp_get_conn(endp, id); + if (!conn) + return NULL; + + if (conn->type == MGCP_CONN_TYPE_RTP) + return &conn->u.rtp; + + return NULL; +} + /*! release endpoint, all open connections are closed. * \param[in] endp endpoint to release */ void mgcp_endp_release(struct mgcp_endpoint *endp) diff --git a/src/libosmo-mgcp/mgcp_msg.c b/src/libosmo-mgcp/mgcp_msg.c index a46ffc2..5900a65 100644 --- a/src/libosmo-mgcp/mgcp_msg.c +++ b/src/libosmo-mgcp/mgcp_msg.c @@ -294,7 +294,7 @@ } /* Check if connection exists */ - if (mgcp_conn_get(endp, conn_id)) + if (mgcp_endp_get_conn(endp, conn_id)) return 0; LOGPENDP(endp, DLMGCP, LOGL_ERROR, diff --git a/src/libosmo-mgcp/mgcp_protocol.c b/src/libosmo-mgcp/mgcp_protocol.c index dbe6760..d166a4e 100644 --- a/src/libosmo-mgcp/mgcp_protocol.c +++ b/src/libosmo-mgcp/mgcp_protocol.c @@ -1022,7 +1022,7 @@ goto error2; } - conn = mgcp_conn_get_rtp(endp, _conn->id); + conn = mgcp_endp_get_conn_rtp(endp, _conn->id); OSMO_ASSERT(conn); if (mgcp_parse_conn_mode(mode, endp, conn->conn) != 0) { @@ -1237,7 +1237,7 @@ return create_err_response(endp, endp, 515, "MDCX", pdata->trans); } - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); if (!conn) { rate_ctr_inc(rate_ctr_group_get_ctr(rate_ctrs, MGCP_MDCX_FAIL_CONN_NOT_FOUND)); return create_err_response(endp, endp, 400, "MDCX", pdata->trans); @@ -1475,7 +1475,7 @@ } /* Find the connection */ - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); if (!conn) { rate_ctr_inc(rate_ctr_group_get_ctr(rate_ctrs, MGCP_DLCX_FAIL_INVALID_CONNID)); goto error3; diff --git a/src/libosmo-mgcp/mgcp_vty.c b/src/libosmo-mgcp/mgcp_vty.c index b82f3df..042841c 100644 --- a/src/libosmo-mgcp/mgcp_vty.c +++ b/src/libosmo-mgcp/mgcp_vty.c @@ -1418,7 +1418,7 @@ endp = trunk->endpoints[endp_no]; conn_id = argv[2]; - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); if (!conn) { vty_out(vty, "Conn ID %s is invalid.%s", conn_id, VTY_NEWLINE); diff --git a/tests/mgcp/mgcp_test.c b/tests/mgcp/mgcp_test.c index 44927e1..c257d6a 100644 --- a/tests/mgcp/mgcp_test.c +++ b/tests/mgcp/mgcp_test.c @@ -965,7 +965,7 @@ endp = mgcp_endp_by_name(NULL, last_endpoint, cfg); OSMO_ASSERT(endp); - conn = mgcp_conn_get_rtp(endp, "1"); + conn = mgcp_endp_get_conn_rtp(endp, "1"); if (conn) { OSMO_ASSERT(conn); @@ -1209,7 +1209,7 @@ mgcp_conn_alloc(NULL, &endp, MGCP_CONN_TYPE_RTP, "test-connection"); OSMO_ASSERT(_conn); - conn = mgcp_conn_get_rtp(&endp, _conn->id); + conn = mgcp_endp_get_conn_rtp(&endp, _conn->id); OSMO_ASSERT(conn); state = &conn->state; packets_rx = rate_ctr_group_get_ctr(conn->ctrg, RTP_PACKETS_RX_CTR); @@ -1461,7 +1461,7 @@ _conn = mgcp_conn_alloc(NULL, &endp, MGCP_CONN_TYPE_RTP, "test-connection"); OSMO_ASSERT(_conn); - conn = mgcp_conn_get_rtp(&endp, _conn->id); + conn = mgcp_endp_get_conn_rtp(&endp, _conn->id); OSMO_ASSERT(conn); rtp = &conn->end; @@ -1548,7 +1548,7 @@ OSMO_ASSERT(strcmp(last_endpoint,"rtpbridge/1@mgw") == 0); endp = mgcp_endp_by_name(NULL, last_endpoint, cfg); OSMO_ASSERT(endp); - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); OSMO_ASSERT(conn); OSMO_ASSERT(conn->end.codec->payload_type == 18); @@ -1564,7 +1564,7 @@ OSMO_ASSERT(strcmp(last_endpoint,"rtpbridge/2@mgw") == 0); endp = mgcp_endp_by_name(NULL, last_endpoint, cfg); OSMO_ASSERT(endp); - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); OSMO_ASSERT(conn); OSMO_ASSERT(conn->end.codec->payload_type == 18); @@ -1585,7 +1585,7 @@ OSMO_ASSERT(strcmp(last_endpoint,"rtpbridge/3@mgw") == 0); endp = mgcp_endp_by_name(NULL, last_endpoint, cfg); OSMO_ASSERT(endp); - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); OSMO_ASSERT(conn); OSMO_ASSERT(conn->end.codec->payload_type == 0); @@ -1601,7 +1601,7 @@ OSMO_ASSERT(strcmp(last_endpoint,"rtpbridge/4@mgw") == 0); endp = mgcp_endp_by_name(NULL, last_endpoint, cfg); OSMO_ASSERT(endp); - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); OSMO_ASSERT(conn); OSMO_ASSERT(conn->end.codec->payload_type == 18); @@ -1617,7 +1617,7 @@ OSMO_ASSERT(strcmp(last_endpoint,"rtpbridge/5@mgw") == 0); endp = mgcp_endp_by_name(NULL, last_endpoint, cfg); OSMO_ASSERT(endp); - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); OSMO_ASSERT(conn); OSMO_ASSERT(conn->end.codec->payload_type == 0); @@ -1629,7 +1629,7 @@ OSMO_ASSERT(strcmp(last_endpoint,"rtpbridge/5@mgw") == 0); endp = mgcp_endp_by_name(NULL, last_endpoint, cfg); OSMO_ASSERT(endp); - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); OSMO_ASSERT(conn); OSMO_ASSERT(conn->end.codec->payload_type == 3); OSMO_ASSERT(osmo_sockaddr_port(&conn->end.addr.u.sa) == 16434); @@ -1646,7 +1646,7 @@ talloc_free(endp->last_response); talloc_free(endp->last_trans); endp->last_response = endp->last_trans = NULL; - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); OSMO_ASSERT(!conn); last_endpoint[0] = '\0'; @@ -1660,7 +1660,7 @@ OSMO_ASSERT(strcmp(last_endpoint,"rtpbridge/5@mgw") == 0); endp = mgcp_endp_by_name(NULL, last_endpoint, cfg); OSMO_ASSERT(endp); - conn = mgcp_conn_get_rtp(endp, conn_id); + conn = mgcp_endp_get_conn_rtp(endp, conn_id); OSMO_ASSERT(conn); OSMO_ASSERT(conn->end.codec->payload_type == 0); @@ -1689,7 +1689,7 @@ _conn = mgcp_conn_alloc(NULL, endp, MGCP_CONN_TYPE_RTP, "test-connection"); OSMO_ASSERT(_conn); - conn = mgcp_conn_get_rtp(endp, _conn->id); + conn = mgcp_endp_get_conn_rtp(endp, _conn->id); OSMO_ASSERT(conn); OSMO_ASSERT(conn->state.stats.initialized == 0); @@ -2332,7 +2332,7 @@ for (i = 0; i < ARRAY_SIZE(conn_id_request); i++) { const char *needle = conn_id_request[i]; printf("needle='%s' ", needle); - conn_match = mgcp_conn_get(&endp, needle); + conn_match = mgcp_endp_get_conn(&endp, needle); OSMO_ASSERT(conn_match); printf("found '%s'\n", conn_match->id); OSMO_ASSERT(conn_match == conn); -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39184?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Ifff01331db68998e9e23f99d8836d96022d550c2 Gerrit-Change-Number: 39184 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

5 months, 2 weeks

1
0
0 0

[XS] Change in osmo-mgw[master]: mgw: Avoid 2nd lookup of conn in endp during CRCX

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39208?usp=email ) Change subject: mgw: Avoid 2nd lookup of conn in endp during CRCX ...................................................................... mgw: Avoid 2nd lookup of conn in endp during CRCX There's no need to lookup (iterate over conns) for rtp_conn in endp, we have it accessible in constant time. While at it, simplify some accesses to the pointer after it. Change-Id: I8316e76a1789d4a8eaa1eb89bdaf86e25dc0a1e8 --- M src/libosmo-mgcp/mgcp_protocol.c 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Jenkins Builder: Verified daniel: Looks good to me, but someone else must approve pespin: Looks good to me, approved laforge: Looks good to me, but someone else must approve diff --git a/src/libosmo-mgcp/mgcp_protocol.c b/src/libosmo-mgcp/mgcp_protocol.c index 4e63e4d..6d315ca 100644 --- a/src/libosmo-mgcp/mgcp_protocol.c +++ b/src/libosmo-mgcp/mgcp_protocol.c @@ -1022,10 +1022,10 @@ goto error2; } - conn = mgcp_endp_get_conn_rtp(endp, _conn->id); + conn = mgcp_conn_get_conn_rtp(_conn); OSMO_ASSERT(conn); - if (mgcp_parse_conn_mode(mode, endp, conn->conn) != 0) { + if (mgcp_parse_conn_mode(mode, endp, _conn) != 0) { error_code = 517; rate_ctr_inc(rate_ctr_group_get_ctr(rate_ctrs, MGCP_CRCX_FAIL_INVALID_MODE)); goto error2; @@ -1108,12 +1108,12 @@ } } - LOGPCONN(conn->conn, DLMGCP, LOGL_DEBUG, + LOGPCONN(_conn, DLMGCP, LOGL_DEBUG, "CRCX: Creating connection: port: %u\n", conn->end.local_port); /* Send dummy packet, see also comments in mgcp_keepalive_timer_cb() */ OSMO_ASSERT(trunk->keepalive_interval >= MGCP_KEEPALIVE_ONCE); - if (conn->conn->mode & MGCP_CONN_RECV_ONLY && + if (_conn->mode & MGCP_CONN_RECV_ONLY && trunk->keepalive_interval != MGCP_KEEPALIVE_NEVER) send_dummy(endp, conn); -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39208?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I8316e76a1789d4a8eaa1eb89bdaf86e25dc0a1e8 Gerrit-Change-Number: 39208 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

5 months, 2 weeks

1
0
0 0

[M] Change in osmo-mgw[master]: mgw: Clean up access to conn_rtp from conn

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39186?usp=email ) Change subject: mgw: Clean up access to conn_rtp from conn ...................................................................... mgw: Clean up access to conn_rtp from conn Add a new mgcp_conn_get_conn_rtp() and use it everywhere instead of accessing deep structure fields. Change-Id: Iee2c19598e9570ea3b1ceba3cdfd2a5f5be2c954 --- M include/osmocom/mgcp/mgcp_conn.h M src/libosmo-mgcp/mgcp_conn.c M src/libosmo-mgcp/mgcp_e1.c M src/libosmo-mgcp/mgcp_endp.c M src/libosmo-mgcp/mgcp_msg.c M src/libosmo-mgcp/mgcp_osmux.c M src/libosmo-mgcp/mgcp_protocol.c M src/libosmo-mgcp/mgcp_stat.c M src/libosmo-mgcp/mgcp_vty.c 9 files changed, 39 insertions(+), 27 deletions(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified daniel: Looks good to me, but someone else must approve laforge: Looks good to me, but someone else must approve diff --git a/include/osmocom/mgcp/mgcp_conn.h b/include/osmocom/mgcp/mgcp_conn.h index de9bae6..106544a 100644 --- a/include/osmocom/mgcp/mgcp_conn.h +++ b/include/osmocom/mgcp/mgcp_conn.h @@ -28,6 +28,7 @@ #include <osmocom/mgcp/osmux.h> #include <osmocom/core/linuxlist.h> #include <osmocom/core/rate_ctr.h> +#include <osmocom/core/utils.h> #include <osmocom/gsm/iuup.h> #include <inttypes.h> @@ -237,6 +238,12 @@ return conn->type == MGCP_RTP_IUUP; } +static inline struct mgcp_conn_rtp *mgcp_conn_get_conn_rtp(struct mgcp_conn *conn) +{ + OSMO_ASSERT(conn->type == MGCP_CONN_TYPE_RTP); + return &conn->u.rtp; +} + struct mgcp_conn *mgcp_conn_alloc(void *ctx, struct mgcp_endpoint *endp, enum mgcp_conn_type type, char *name); void mgcp_conn_free(struct mgcp_conn *conn); diff --git a/src/libosmo-mgcp/mgcp_conn.c b/src/libosmo-mgcp/mgcp_conn.c index 9d03550..403cbda 100644 --- a/src/libosmo-mgcp/mgcp_conn.c +++ b/src/libosmo-mgcp/mgcp_conn.c @@ -239,13 +239,16 @@ */ void mgcp_conn_free(struct mgcp_conn *conn) { + struct mgcp_conn_rtp *conn_rtp; + if (!conn) return; switch (conn->type) { case MGCP_CONN_TYPE_RTP: - aggregate_rtp_conn_stats(conn->endp, &conn->u.rtp); - mgcp_rtp_conn_cleanup(&conn->u.rtp); + conn_rtp = mgcp_conn_get_conn_rtp(conn); + aggregate_rtp_conn_stats(conn->endp, conn_rtp); + mgcp_rtp_conn_cleanup(conn_rtp); break; default: /* NOTE: This should never be called with an @@ -268,24 +271,26 @@ static char str[sizeof(conn->name)+sizeof(conn->id)+256]; char ipbuf[INET6_ADDRSTRLEN]; struct osmo_strbuf sb = { .buf = str, .len = sizeof(str) }; + struct mgcp_conn_rtp *conn_rtp; if (!conn) return "NULL"; switch (conn->type) { case MGCP_CONN_TYPE_RTP: + conn_rtp = mgcp_conn_get_conn_rtp(conn); OSMO_STRBUF_PRINTF(sb, "(%s/%s C:%s r=%s:%u<->l=%s:%u", conn->name, mgcp_conn_rtp_type_name(conn->type), conn->id, - osmo_sockaddr_ntop(&conn->u.rtp.end.addr.u.sa, ipbuf) ? : "NULL", - osmo_sockaddr_port(&conn->u.rtp.end.addr.u.sa), - conn->u.rtp.end.local_addr ? : "NULL", - conn->u.rtp.end.local_port); + osmo_sockaddr_ntop(&conn_rtp->end.addr.u.sa, ipbuf) ? : "NULL", + osmo_sockaddr_port(&conn_rtp->end.addr.u.sa), + conn_rtp->end.local_addr ? : "NULL", + conn_rtp->end.local_port); - switch (conn->u.rtp.type) { + switch (conn_rtp->type) { case MGCP_RTP_OSMUX: - OSMO_STRBUF_PRINTF(sb, " CID=%u", conn->u.rtp.osmux.local_cid); + OSMO_STRBUF_PRINTF(sb, " CID=%u", conn_rtp->osmux.local_cid); break; default: break; diff --git a/src/libosmo-mgcp/mgcp_e1.c b/src/libosmo-mgcp/mgcp_e1.c index fac1ff6..428d652 100644 --- a/src/libosmo-mgcp/mgcp_e1.c +++ b/src/libosmo-mgcp/mgcp_e1.c @@ -650,6 +650,7 @@ void mgcp_e1_endp_update(struct mgcp_endpoint *endp) { struct mgcp_conn *conn; + struct mgcp_conn_rtp *conn_rtp; struct mgcp_rtp_codec *codec; enum osmo_tray_sync_pat_id sync_pat_id; @@ -658,7 +659,8 @@ * endpoint no more than one connection should exist. */ conn = mgcp_endp_get_conn_oldest(endp); OSMO_ASSERT(conn); - codec = conn->u.rtp.end.codec; + conn_rtp = mgcp_conn_get_conn_rtp(conn); + codec = conn_rtp->end.codec; OSMO_ASSERT(codec); /* Update codec information */ diff --git a/src/libosmo-mgcp/mgcp_endp.c b/src/libosmo-mgcp/mgcp_endp.c index b2ece20..c91a00d 100644 --- a/src/libosmo-mgcp/mgcp_endp.c +++ b/src/libosmo-mgcp/mgcp_endp.c @@ -741,7 +741,7 @@ return NULL; if (conn->type == MGCP_CONN_TYPE_RTP) - return &conn->u.rtp; + return mgcp_conn_get_conn_rtp(conn); return NULL; } diff --git a/src/libosmo-mgcp/mgcp_msg.c b/src/libosmo-mgcp/mgcp_msg.c index 5900a65..2360a0b 100644 --- a/src/libosmo-mgcp/mgcp_msg.c +++ b/src/libosmo-mgcp/mgcp_msg.c @@ -110,20 +110,16 @@ ret = -1; } - /* Special handling for RTP connections */ - if (conn->type == MGCP_CONN_TYPE_RTP) { - conn->u.rtp.end.output_enabled = !!(conn->mode & MGCP_CONN_SEND_ONLY); - } - LOGPENDP(endp, DLMGCP, LOGL_DEBUG, "conn:%s\n", mgcp_conn_dump(conn)); - LOGPCONN(conn, DLMGCP, LOGL_DEBUG, "connection mode '%s' %d\n", mode, conn->mode); - /* Special handling für RTP connections */ + /* Special handling for RTP connections */ if (conn->type == MGCP_CONN_TYPE_RTP) { + struct mgcp_conn_rtp *conn_rtp = mgcp_conn_get_conn_rtp(conn); + conn_rtp->end.output_enabled = !!(conn->mode & MGCP_CONN_SEND_ONLY); LOGPCONN(conn, DLMGCP, LOGL_DEBUG, "output_enabled %u\n", - conn->u.rtp.end.output_enabled); + conn_rtp->end.output_enabled); } /* The VTY might change the connection mode at any time, so we have diff --git a/src/libosmo-mgcp/mgcp_osmux.c b/src/libosmo-mgcp/mgcp_osmux.c index f8383d6..0c285dd 100644 --- a/src/libosmo-mgcp/mgcp_osmux.c +++ b/src/libosmo-mgcp/mgcp_osmux.c @@ -265,7 +265,7 @@ if (conn->type != MGCP_CONN_TYPE_RTP) continue; - conn_rtp = &conn->u.rtp; + conn_rtp = mgcp_conn_get_conn_rtp(conn); if (!mgcp_conn_rtp_is_osmux(conn_rtp)) continue; diff --git a/src/libosmo-mgcp/mgcp_protocol.c b/src/libosmo-mgcp/mgcp_protocol.c index d166a4e..4e63e4d 100644 --- a/src/libosmo-mgcp/mgcp_protocol.c +++ b/src/libosmo-mgcp/mgcp_protocol.c @@ -172,7 +172,7 @@ /* Find the "sister" connection */ llist_for_each_entry(_conn, &endp->conns, entry) { if (_conn->id != conn->conn->id) { - conn_src = &_conn->u.rtp; + conn_src = mgcp_conn_get_conn_rtp(_conn); break; } } @@ -819,7 +819,7 @@ /* Try to find an destination RTP connection that we can include in the codec decision. */ conn_dst = mgcp_find_dst_conn(conn->conn); if (conn_dst && conn_dst->type == MGCP_CONN_TYPE_RTP) - conn_dst_rtp = &conn_dst->u.rtp; + conn_dst_rtp = mgcp_conn_get_conn_rtp(conn_dst); else conn_dst_rtp = NULL; @@ -1609,7 +1609,7 @@ llist_for_each_entry(conn, &endp->conns, entry) { if (conn->type == MGCP_CONN_TYPE_RTP && conn->mode == MGCP_CONN_RECV_ONLY) - send_dummy(endp, &conn->u.rtp); + send_dummy(endp, mgcp_conn_get_conn_rtp(conn)); } } diff --git a/src/libosmo-mgcp/mgcp_stat.c b/src/libosmo-mgcp/mgcp_stat.c index c66ce6b..081a381 100644 --- a/src/libosmo-mgcp/mgcp_stat.c +++ b/src/libosmo-mgcp/mgcp_stat.c @@ -141,7 +141,7 @@ * keep this option open: */ switch (conn->type) { case MGCP_CONN_TYPE_RTP: - mgcp_format_stats_rtp(str, str_len, &conn->u.rtp); + mgcp_format_stats_rtp(str, str_len, mgcp_conn_get_conn_rtp(conn)); break; default: break; diff --git a/src/libosmo-mgcp/mgcp_vty.c b/src/libosmo-mgcp/mgcp_vty.c index 042841c..8e0360f 100644 --- a/src/libosmo-mgcp/mgcp_vty.c +++ b/src/libosmo-mgcp/mgcp_vty.c @@ -254,7 +254,8 @@ * connection types (E1) as soon as * the implementation is available */ if (conn->type == MGCP_CONN_TYPE_RTP) { - dump_rtp_end(vty, &conn->u.rtp); + struct mgcp_conn_rtp *conn_rtp = mgcp_conn_get_conn_rtp(conn); + dump_rtp_end(vty, conn_rtp); } } } @@ -1357,10 +1358,11 @@ endp = trunk->endpoints[endp_no]; int loop = atoi(argv[2]); llist_for_each_entry(conn, &endp->conns, entry) { - if (conn->type == MGCP_CONN_TYPE_RTP) + if (conn->type == MGCP_CONN_TYPE_RTP) { /* Handle it like a MDCX, switch on SSRC patching if enabled */ - mgcp_rtp_end_config(endp, 1, &conn->u.rtp.end); - else { + struct mgcp_conn_rtp *conn_rtp = mgcp_conn_get_conn_rtp(conn); + mgcp_rtp_end_config(endp, 1, &conn_rtp->end); + } else { /* FIXME: Introduce support for other connection (E1) * types when implementation is available */ vty_out(vty, "%%Can't enable SSRC patching," -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39186?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Iee2c19598e9570ea3b1ceba3cdfd2a5f5be2c954 Gerrit-Change-Number: 39186 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

5 months, 2 weeks

1
0
0 0

[XS] Change in osmo-mgw[master]: mgw: Avoid 2nd lookup of conn in endp during CRCX

by pespin

pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39208?usp=email ) Change subject: mgw: Avoid 2nd lookup of conn in endp during CRCX ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39208?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I8316e76a1789d4a8eaa1eb89bdaf86e25dc0a1e8 Gerrit-Change-Number: 39208 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Sat, 04 Jan 2025 23:30:26 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

5 months, 2 weeks

1
0
0 0

[M] Change in osmo-mgw[master]: mgw: Clean up access to conn_rtp from conn

by pespin

pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39186?usp=email ) Change subject: mgw: Clean up access to conn_rtp from conn ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39186?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Iee2c19598e9570ea3b1ceba3cdfd2a5f5be2c954 Gerrit-Change-Number: 39186 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Sat, 04 Jan 2025 23:30:20 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

5 months, 2 weeks

1
0
0 0

[S] Change in osmo-mgw[master]: mgw: mgcp_network.c: Simplify use of conn_rtp ptr

by pespin

pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39185?usp=email ) Change subject: mgw: mgcp_network.c: Simplify use of conn_rtp ptr ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39185?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I5bab15fc793434173660769a8e60dae4ae4aa4c6 Gerrit-Change-Number: 39185 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Sat, 04 Jan 2025 23:30:16 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

5 months, 2 weeks

1
0
0 0

[M] Change in osmo-mgw[master]: mgw: Rename and move several get_conn funcs acting on endp object

by pespin

Attention is currently required from: daniel. pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39184?usp=email ) Change subject: mgw: Rename and move several get_conn funcs acting on endp object ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39184?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Ifff01331db68998e9e23f99d8836d96022d550c2 Gerrit-Change-Number: 39184 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Comment-Date: Sat, 04 Jan 2025 23:30:11 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

5 months, 2 weeks

1
0
0 0

[M] Change in osmo-mgw[master]: mgw: Rename and move code freeing endp connection

by pespin

Attention is currently required from: daniel, fixeria. pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39183?usp=email ) Change subject: mgw: Rename and move code freeing endp connection ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39183?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I9a87a07699dd8aa7856d5036e9b95a4ddf699a15 Gerrit-Change-Number: 39183 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Comment-Date: Sat, 04 Jan 2025 23:30:07 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

5 months, 2 weeks

1
0
0 0

[S] Change in osmo-mgw[master]: mgw: Introduce mgcp_rtp_end_init()

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39214?usp=email ) Change subject: mgw: Introduce mgcp_rtp_end_init() ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39214?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I29f57dfde60e3134f1db422dc63e7c58d3db7d54 Gerrit-Change-Number: 39214 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Sat, 04 Jan 2025 19:10:11 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

5 months, 2 weeks

1
0
0 0

2025

2024

2023

2022

gerrit-log January 2025