December 2024 - gerrit-log - lists.osmocom.org

[S] Change in pysim[master]: ara_m: put hex string values in quotes

by laforge

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/39056?usp=email ) Change subject: ara_m: put hex string values in quotes ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: > Its due to an aesthetic conflict with the patch after this one. […] I am not convinced, sorry. I'd rather keep things in-line with how we do it elsewhere. I think it's fine to quote only the empty string case. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39056?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ia365e83638ff4f1b1a02e1671852cf2006f80c7f Gerrit-Change-Number: 39056 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 10 Dec 2024 23:15:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>

6 months, 2 weeks

1
0
0 0

[L] Change in libosmo-abis[master]: Drop ipa_proxy

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/39082?usp=email ) Change subject: Drop ipa_proxy ...................................................................... Drop ipa_proxy This module is not used anywhere, not I'm aware it has been used in recent years. Drop it, since IPA generic functionalities don't belong in libosmo-abis. If somebody ever needs such a feature, it can be ported to libosmo-netif. Change-Id: I7fbc3ad7277369d21cf31159407ef6672ebdf64f --- M TODO-RELEASE M include/Makefile.am D include/osmocom/abis/ipa_proxy.h M src/Makefile.am D src/ipa_proxy.c M tests/Makefile.am D tests/ipa_proxy_test.c 7 files changed, 2 insertions(+), 723 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/TODO-RELEASE b/TODO-RELEASE index 4cf9cea..e540591 100644 --- a/TODO-RELEASE +++ b/TODO-RELEASE @@ -13,3 +13,4 @@ libosmocore >1.10.0 bigger iofd tx_queue.max_length default value libosmo-abis API break Remove ipaccess_fd_cb() libosmo-netif >1.5.1 osmo_ipa_ka_fsm_inst APIs +libosmo-abis Remove API ipa_proxy_vty_init() \ No newline at end of file diff --git a/include/Makefile.am b/include/Makefile.am index 2d22a55..d555e62 100644 --- a/include/Makefile.am +++ b/include/Makefile.am @@ -1,7 +1,7 @@ noinst_HEADERS=mISDNif.h internal.h nobase_include_HEADERS = osmocom/abis/ipa.h osmocom/abis/trau_frame.h \ - osmocom/abis/ipa_proxy.h osmocom/abis/ipaccess.h osmocom/abis/abis.h \ + osmocom/abis/ipaccess.h osmocom/abis/abis.h \ osmocom/abis/subchan_demux.h osmocom/abis/e1_input.h \ osmocom/abis/lapd.h osmocom/abis/lapd_pcap.h osmocom/abis/unixsocket_proto.h \ osmocom/trau/csd_ra2.h \ diff --git a/include/osmocom/abis/ipa_proxy.h b/include/osmocom/abis/ipa_proxy.h deleted file mode 100644 index 9b4efa9..0000000 --- a/include/osmocom/abis/ipa_proxy.h +++ /dev/null @@ -1,6 +0,0 @@ -#ifndef _IPA_PROXY_H_ -#define _IPA_PROXY_H_ - -void ipa_proxy_vty_init(void); - -#endif diff --git a/src/Makefile.am b/src/Makefile.am index 724dbd5..650fc22 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -31,7 +31,6 @@ libosmoabis_la_SOURCES = init.c \ e1_input.c \ e1_input_vty.c \ - ipa_proxy.c \ subchan_demux.c \ trau_frame.c \ input/e1d.c \ diff --git a/src/ipa_proxy.c b/src/ipa_proxy.c deleted file mode 100644 index 6a211ee..0000000 --- a/src/ipa_proxy.c +++ /dev/null @@ -1,656 +0,0 @@ -#include <stdio.h> -#include <stdbool.h> -#include <unistd.h> -#include <stdlib.h> -#include <errno.h> -#include <string.h> -#include <time.h> -#include <sys/fcntl.h> -#include <sys/socket.h> -#include <sys/ioctl.h> -#include <arpa/inet.h> - -#include <osmocom/core/select.h> -#include <osmocom/core/bitvec.h> -#include <osmocom/core/msgb.h> -#include <osmocom/core/logging.h> -#include <osmocom/core/talloc.h> -#include <osmocom/core/socket.h> -#include <osmocom/vty/vty.h> -#include <osmocom/vty/command.h> -#include <osmocom/gsm/tlv.h> -#include <osmocom/gsm/protocol/ipaccess.h> -#include <osmocom/abis/e1_input.h> -#include <osmocom/abis/ipa.h> - -static void *tall_ipa_proxy_ctx; - -extern void *libosmo_abis_ctx; - -/* - * data structures used by the IPA VTY commands - */ -static LLIST_HEAD(ipa_instance_list); - -enum ipa_proxy_instance_net_type { - IPA_INSTANCE_T_NONE, - IPA_INSTANCE_T_BIND, - IPA_INSTANCE_T_CONNECT, - IPA_INSTANCE_T_MAX -}; - -struct ipa_proxy_instance_net { - char *addr; - uint16_t port; - enum ipa_proxy_instance_net_type type; -}; - -struct ipa_proxy_instance { - struct llist_head head; -#define IPA_INSTANCE_NAME 16 - char name[IPA_INSTANCE_NAME]; - struct ipa_proxy_instance_net net; - int refcnt; -}; - -static LLIST_HEAD(ipa_proxy_route_list); - -/* Several routes pointing to the same instances share this. */ -struct ipa_proxy_route_shared { - int refcnt; - - /* this file descriptor is used to accept() new connections. */ - struct osmo_fd bfd; - - struct { - struct ipa_proxy_instance *inst; - struct bitvec streamid_map; - uint8_t streamid_map_data[(0xff+1)/8]; - uint8_t streamid[0xff + 1]; - } src; - struct { - struct ipa_proxy_instance *inst; - struct bitvec streamid_map; - uint8_t streamid_map_data[(0xff+1)/8]; - uint8_t streamid[0xff + 1]; - } dst; - - struct llist_head conn_list; -}; - -/* One route is composed of two instances. */ -struct ipa_proxy_route { - struct llist_head head; - - struct { - uint8_t streamid; - } src; - struct { - uint8_t streamid; - } dst; - - struct ipa_proxy_route_shared *shared; -}; - -enum ipa_conn_state { - IPA_CONN_S_NONE, - IPA_CONN_S_CONNECTING, - IPA_CONN_S_CONNECTED, - IPA_CONN_S_MAX -}; - -/* One route may forward more than one connection. */ -struct ipa_proxy_conn { - struct llist_head head; - - struct ipa_server_conn *src; - struct ipa_client_conn *dst; - struct ipa_proxy_route *route; -}; - -/* - * socket callbacks used by IPA VTY commands - */ -static int ipa_sock_dst_cb(struct ipa_client_conn *link, struct msgb *msg) -{ - struct ipaccess_head *hh; - struct ipa_proxy_conn *conn = link->data; - - LOGP(DLINP, LOGL_NOTICE, "received message from client side\n"); - - hh = (struct ipaccess_head *)msg->data; - /* check if we have a route for this message. */ - if (bitvec_get_bit_pos( - &conn->route->shared->dst.streamid_map, - hh->proto) != ONE) { - LOGP(DLINP, LOGL_NOTICE, "we don't have a " - "route for streamid 0x%x\n", hh->proto); - msgb_free(msg); - return 0; - } - /* mangle message, if required. */ - hh->proto = conn->route->shared->src.streamid[hh->proto]; - - ipa_server_conn_send(conn->src, msg); - return 0; -} - -static int ipa_sock_src_cb(struct ipa_server_conn *peer, struct msgb *msg) -{ - struct ipaccess_head *hh; - struct ipa_proxy_conn *conn = peer->data; - - LOGP(DLINP, LOGL_NOTICE, "received message from server side\n"); - - hh = (struct ipaccess_head *)msg->data; - /* check if we have a route for this message. */ - if (bitvec_get_bit_pos(&conn->route->shared->src.streamid_map, - hh->proto) != ONE) { - LOGP(DLINP, LOGL_NOTICE, "we don't have a " - "route for streamid 0x%x\n", hh->proto); - msgb_free(msg); - return 0; - } - /* mangle message, if required. */ - hh->proto = conn->route->shared->dst.streamid[hh->proto]; - - ipa_client_conn_send(conn->dst, msg); - return 0; -} - -static int -ipa_sock_src_accept_cb(struct ipa_server_link *link, int fd) -{ - struct ipa_proxy_route *route = link->data; - struct ipa_proxy_conn *conn; - - conn = talloc_zero(tall_ipa_proxy_ctx, struct ipa_proxy_conn); - if (conn == NULL) { - LOGP(DLINP, LOGL_ERROR, "cannot allocate memory for " - "origin IPA\n"); - close(fd); - return -ENOMEM; - } - conn->route = route; - - conn->src = ipa_server_conn_create(tall_ipa_proxy_ctx, link, fd, - ipa_sock_src_cb, NULL, conn); - if (conn->src == NULL) { - LOGP(DLINP, LOGL_ERROR, "could not create server peer: %s\n", - strerror(errno)); - return -ENOMEM; - } - - LOGP(DLINP, LOGL_NOTICE, "now trying to connect to destination\n"); - - conn->dst = ipa_client_conn_create2(NULL, NULL, 0, - NULL, 0, - route->shared->dst.inst->net.addr, - route->shared->dst.inst->net.port, - NULL, - ipa_sock_dst_cb, - NULL, - conn); - if (conn->dst == NULL) { - LOGP(DLINP, LOGL_ERROR, "could not create client: %s\n", - strerror(errno)); - return -ENOMEM; - } - if (ipa_client_conn_open(conn->dst) < 0) { - LOGP(DLINP, LOGL_ERROR, "could not start client: %s\n", - strerror(errno)); - return -ENOMEM; - } - llist_add(&conn->head, &route->shared->conn_list); - return 0; -} - -/* - * VTY commands for IPA - */ -static int __ipa_instance_add(struct vty *vty, int argc, const char *argv[]) -{ - struct ipa_proxy_instance *ipi; - enum ipa_proxy_instance_net_type type; - struct in_addr addr; - uint16_t port; - - if (argc < 4) - return CMD_ERR_INCOMPLETE; - - llist_for_each_entry(ipi, &ipa_instance_list, head) { - if (strncmp(ipi->name, argv[0], IPA_INSTANCE_NAME) != 0) - continue; - - vty_out(vty, "%% instance `%s' already exists%s", - ipi->name, VTY_NEWLINE); - return CMD_WARNING; - } - if (strncmp(argv[1], "bind", IPA_INSTANCE_NAME) == 0) - type = IPA_INSTANCE_T_BIND; - else if (strncmp(argv[1], "connect", IPA_INSTANCE_NAME) == 0) - type = IPA_INSTANCE_T_CONNECT; - else - return CMD_ERR_INCOMPLETE; - - if (inet_aton(argv[2], &addr) < 0) { - vty_out(vty, "%% invalid address %s%s", argv[1], VTY_NEWLINE); - return CMD_WARNING; - } - port = atoi(argv[3]); - - ipi = talloc_zero(tall_ipa_proxy_ctx, struct ipa_proxy_instance); - if (ipi == NULL) { - vty_out(vty, "%% can't allocate memory for new instance%s", - VTY_NEWLINE); - return CMD_WARNING; - } - osmo_strlcpy(ipi->name, argv[0], sizeof(ipi->name)); - ipi->net.type = type; - ipi->net.addr = talloc_strdup(tall_ipa_proxy_ctx, argv[2]); - ipi->net.port = port; - llist_add_tail(&ipi->head, &ipa_instance_list); - - return CMD_SUCCESS; -} - -DEFUN(ipa_instance_add, ipa_instance_add_cmd, - "ipa instance NAME (bind|connect) IP tcp port PORT", - "Bind or connect instance to address and port") -{ - return __ipa_instance_add(vty, argc, argv); -} - -DEFUN(ipa_instance_del, ipa_instance_del_cmd, - "no ipa instance NAME", - "Delete instance to address and port") -{ - struct ipa_proxy_instance *ipi; - - if (argc < 1) - return CMD_ERR_INCOMPLETE; - - llist_for_each_entry(ipi, &ipa_instance_list, head) { - if (strncmp(ipi->name, argv[0], IPA_INSTANCE_NAME) != 0) - continue; - - if (ipi->refcnt > 0) { - vty_out(vty, "%% instance `%s' is in use%s", - ipi->name, VTY_NEWLINE); - return CMD_WARNING; - } - llist_del(&ipi->head); - talloc_free(ipi); - return CMD_SUCCESS; - } - vty_out(vty, "%% instance `%s' does not exist%s", - ipi->name, VTY_NEWLINE); - - return CMD_WARNING; -} - -DEFUN(ipa_instance_show, ipa_instance_show_cmd, - "ipa instance show", "Show existing ipaccess proxy instances") -{ - struct ipa_proxy_instance *this; - - llist_for_each_entry(this, &ipa_instance_list, head) { - vty_out(vty, "instance %s %s %s tcp port %u%s", - this->name, this->net.addr, - this->net.type == IPA_INSTANCE_T_BIND ? - "bind" : "connect", - this->net.port, VTY_NEWLINE); - } - return CMD_SUCCESS; -} - -static int __ipa_route_add(struct vty *vty, int argc, const char *argv[]) -{ - struct ipa_proxy_instance *ipi = vty->index; - struct ipa_proxy_instance *src = NULL, *dst = NULL; - uint32_t src_streamid, dst_streamid; - struct ipa_proxy_route *route, *matching_route = NULL; - struct ipa_proxy_route_shared *shared = NULL; - int ret; - - if (argc < 4) - return CMD_ERR_INCOMPLETE; - - llist_for_each_entry(ipi, &ipa_instance_list, head) { - if (strncmp(ipi->name, argv[0], IPA_INSTANCE_NAME) == 0) { - src = ipi; - continue; - } - if (strncmp(ipi->name, argv[2], IPA_INSTANCE_NAME) == 0) { - dst = ipi; - continue; - } - } - if (src == NULL) { - vty_out(vty, "%% instance `%s' does not exists%s", - argv[0], VTY_NEWLINE); - return CMD_WARNING; - } - if (dst == NULL) { - vty_out(vty, "%% instance `%s' does not exists%s", - argv[2], VTY_NEWLINE); - return CMD_WARNING; - } - if (src->net.type != IPA_INSTANCE_T_BIND) { - vty_out(vty, "%% instance `%s' is not of bind type%s", - argv[0], VTY_NEWLINE); - return CMD_WARNING; - } - if (dst->net.type != IPA_INSTANCE_T_CONNECT) { - vty_out(vty, "%% instance `%s' is not of connect type%s", - argv[2], VTY_NEWLINE); - return CMD_WARNING; - } - src_streamid = strtoul(argv[1], NULL, 16); - if (src_streamid > 0xff) { - vty_out(vty, "%% source streamid must be " - ">= 0x00 and <= 0xff%s", VTY_NEWLINE); - return CMD_WARNING; - } - dst_streamid = strtoul(argv[3], NULL, 16); - if (dst_streamid > 0xff) { - vty_out(vty, "%% destination streamid must be " - ">= 0x00 and <= 0xff%s", VTY_NEWLINE); - return CMD_WARNING; - } - llist_for_each_entry(route, &ipa_proxy_route_list, head) { - if (route->shared->src.inst == src && - route->shared->dst.inst == dst) { - if (route->src.streamid == src_streamid && - route->dst.streamid == dst_streamid) { - vty_out(vty, "%% this route already exists%s", - VTY_NEWLINE); - return CMD_WARNING; - } - matching_route = route; - break; - } - } - /* new route for this configuration. */ - route = talloc_zero(tall_ipa_proxy_ctx, struct ipa_proxy_route); - if (route == NULL) { - vty_out(vty, "%% can't allocate memory for new route%s", - VTY_NEWLINE); - return CMD_WARNING; - } - route->src.streamid = src_streamid; - route->dst.streamid = dst_streamid; - - if (matching_route != NULL) { - /* there's already a master route for these configuration. */ - if (matching_route->shared->src.inst != src) { - vty_out(vty, "%% route does not contain " - "source instance `%s'%s", - argv[0], VTY_NEWLINE); - return CMD_WARNING; - } - if (matching_route->shared->dst.inst != dst) { - vty_out(vty, "%% route does not contain " - "destination instance `%s'%s", - argv[2], VTY_NEWLINE); - return CMD_WARNING; - } - /* use already existing shared routing information. */ - shared = matching_route->shared; - } else { - struct ipa_server_link *link; - - /* this is a brand new route, allocate shared routing info. */ - shared = talloc_zero(tall_ipa_proxy_ctx, struct ipa_proxy_route_shared); - if (shared == NULL) { - vty_out(vty, "%% can't allocate memory for " - "new route shared%s", VTY_NEWLINE); - return CMD_WARNING; - } - shared->src.streamid_map.data_len = - sizeof(shared->src.streamid_map_data); - shared->src.streamid_map.data = - shared->src.streamid_map_data; - shared->dst.streamid_map.data_len = - sizeof(shared->dst.streamid_map_data); - shared->dst.streamid_map.data = - shared->dst.streamid_map_data; - - link = ipa_server_link_create(tall_ipa_proxy_ctx, NULL, - "0.0.0.0", - src->net.port, - ipa_sock_src_accept_cb, route); - if (link == NULL) { - vty_out(vty, "%% can't bind instance `%s' to port%s", - src->name, VTY_NEWLINE); - return CMD_WARNING; - } - if (ipa_server_link_open(link) < 0) { - vty_out(vty, "%% can't bind instance `%s' to port%s", - src->name, VTY_NEWLINE); - return CMD_WARNING; - } - INIT_LLIST_HEAD(&shared->conn_list); - } - route->shared = shared; - src->refcnt++; - route->shared->src.inst = src; - dst->refcnt++; - route->shared->dst.inst = dst; - shared->src.streamid[src_streamid] = dst_streamid; - shared->dst.streamid[dst_streamid] = src_streamid; - ret = bitvec_set_bit_pos(&shared->src.streamid_map, src_streamid, ONE); - if (ret < 0) { - vty_out(vty, "%% bad bitmask (?)%s", VTY_NEWLINE); - return CMD_WARNING; - } - ret = bitvec_set_bit_pos(&shared->dst.streamid_map, dst_streamid, ONE); - if (ret < 0) { - vty_out(vty, "%% bad bitmask (?)%s", VTY_NEWLINE); - return CMD_WARNING; - } - shared->refcnt++; - - llist_add_tail(&route->head, &ipa_proxy_route_list); - return CMD_SUCCESS; -} - -DEFUN(ipa_route_add, ipa_route_add_cmd, - "ipa route instance NAME streamid HEXNUM " - "instance NAME streamid HEXNUM", "Add IPA route") -{ - return __ipa_route_add(vty, argc, argv); -} - -DEFUN(ipa_route_del, ipa_route_del_cmd, - "no ipa route instance NAME streamid HEXNUM " - "instance NAME streamid HEXNUM", "Delete IPA route") -{ - struct ipa_proxy_instance *ipi = vty->index; - struct ipa_proxy_instance *src = NULL, *dst = NULL; - uint32_t src_streamid, dst_streamid; - struct ipa_proxy_route *route, *matching_route = NULL; - struct ipa_proxy_conn *conn, *tmp; - - if (argc < 4) - return CMD_ERR_INCOMPLETE; - - llist_for_each_entry(ipi, &ipa_instance_list, head) { - if (strncmp(ipi->name, argv[0], IPA_INSTANCE_NAME) == 0) { - src = ipi; - continue; - } - if (strncmp(ipi->name, argv[2], IPA_INSTANCE_NAME) == 0) { - dst = ipi; - continue; - } - } - if (src == NULL) { - vty_out(vty, "%% instance `%s' does not exists%s", - argv[0], VTY_NEWLINE); - return CMD_WARNING; - } - if (dst == NULL) { - vty_out(vty, "%% instance `%s' does not exists%s", - argv[2], VTY_NEWLINE); - return CMD_WARNING; - } - if (src->net.type != IPA_INSTANCE_T_BIND) { - vty_out(vty, "%% instance `%s' is not of bind type%s", - argv[0], VTY_NEWLINE); - return CMD_WARNING; - } - if (dst->net.type != IPA_INSTANCE_T_CONNECT) { - vty_out(vty, "%% instance `%s' is not of connect type%s", - argv[2], VTY_NEWLINE); - return CMD_WARNING; - } - src_streamid = strtoul(argv[1], NULL, 16); - if (src_streamid > 0xff) { - vty_out(vty, "%% source streamid must be " - ">= 0x00 and <= 0xff%s", VTY_NEWLINE); - return CMD_WARNING; - } - dst_streamid = strtoul(argv[3], NULL, 16); - if (dst_streamid > 0xff) { - vty_out(vty, "%% destination streamid must be " - ">= 0x00 and <= 0xff%s", VTY_NEWLINE); - return CMD_WARNING; - } - llist_for_each_entry(route, &ipa_proxy_route_list, head) { - if (route->shared->src.inst == src && - route->shared->dst.inst == dst && - route->src.streamid == src_streamid && - route->dst.streamid == dst_streamid) { - matching_route = route; - break; - } - } - if (matching_route == NULL) { - vty_out(vty, "%% no route with that configuration%s", - VTY_NEWLINE); - return CMD_WARNING; - } - /* delete this route from list. */ - llist_del(&matching_route->head); - - if (--matching_route->shared->refcnt == 0) { - /* nobody else using this route, release all resources. */ - llist_for_each_entry_safe(conn, tmp, - &matching_route->shared->conn_list, head) { - ipa_server_conn_destroy(conn->src); - llist_del(&conn->head); - talloc_free(conn); - } - osmo_fd_unregister(&route->shared->bfd); - close(route->shared->bfd.fd); - route->shared->bfd.fd = -1; - - talloc_free(route->shared); - } else { - /* otherwise, revert the mapping that this route applies. */ - bitvec_set_bit_pos(&matching_route->shared->src.streamid_map, - src_streamid, ZERO); - bitvec_set_bit_pos(&matching_route->shared->dst.streamid_map, - dst_streamid, ZERO); - matching_route->shared->src.streamid[src_streamid] = 0x00; - matching_route->shared->dst.streamid[dst_streamid] = 0x00; - } - matching_route->shared->src.inst->refcnt--; - matching_route->shared->dst.inst->refcnt--; - talloc_free(matching_route); - - return CMD_SUCCESS; -} - -DEFUN(ipa_route_show, ipa_route_show_cmd, - "ipa route show", "Show existing ipaccess proxy routes") -{ - struct ipa_proxy_route *this; - - llist_for_each_entry(this, &ipa_proxy_route_list, head) { - vty_out(vty, "route instance %s streamid 0x%.2x " - "instance %s streamid 0x%.2x%s", - this->shared->src.inst->name, this->src.streamid, - this->shared->dst.inst->name, this->dst.streamid, - VTY_NEWLINE); - } - return CMD_SUCCESS; -} - -/* - * Config for ipaccess-proxy - */ -DEFUN(ipa_cfg, ipa_cfg_cmd, "ipa", "Configure the ipaccess proxy") -{ - vty->index = NULL; - vty->node = L_IPA_NODE; - return CMD_SUCCESS; -} - -/* all these below look like enable commands, but without the ipa prefix. */ -DEFUN(ipa_route_cfg_add, ipa_route_cfg_add_cmd, - "route instance NAME streamid HEXNUM " - "instance NAME streamid HEXNUM", "Add IPA route") -{ - return __ipa_route_add(vty, argc, argv); -} - -DEFUN(ipa_instance_cfg_add, ipa_instance_cfg_add_cmd, - "instance NAME (bind|connect) IP tcp port PORT", - "Bind or connect instance to address and port") -{ - return __ipa_instance_add(vty, argc, argv); -} - -struct cmd_node ipa_node = { - L_IPA_NODE, - "%s(config-ipa)# ", - 1, -}; - -static int ipa_cfg_write(struct vty *vty) -{ - bool heading = false; - struct ipa_proxy_instance *inst; - struct ipa_proxy_route *route; - - llist_for_each_entry(inst, &ipa_instance_list, head) { - if (!heading) { - vty_out(vty, "ipa%s", VTY_NEWLINE); - heading = true; - } - vty_out(vty, " instance %s %s %s tcp port %u%s", - inst->name, - inst->net.type == IPA_INSTANCE_T_BIND ? - "bind" : "connect", - inst->net.addr, - inst->net.port, VTY_NEWLINE); - } - llist_for_each_entry(route, &ipa_proxy_route_list, head) { - vty_out(vty, " route instance %s streamid 0x%.2x " - "instance %s streamid 0x%.2x%s", - route->shared->src.inst->name, route->src.streamid, - route->shared->dst.inst->name, route->dst.streamid, - VTY_NEWLINE); - } - return CMD_SUCCESS; -} - -void ipa_proxy_vty_init(void) -{ - tall_ipa_proxy_ctx = - talloc_named_const(libosmo_abis_ctx, 1, "ipa_proxy"); - - install_lib_element(ENABLE_NODE, &ipa_instance_add_cmd); - install_lib_element(ENABLE_NODE, &ipa_instance_del_cmd); - install_lib_element(ENABLE_NODE, &ipa_instance_show_cmd); - install_lib_element(ENABLE_NODE, &ipa_route_add_cmd); - install_lib_element(ENABLE_NODE, &ipa_route_del_cmd); - install_lib_element(ENABLE_NODE, &ipa_route_show_cmd); - - install_lib_element(CONFIG_NODE, &ipa_cfg_cmd); - install_node(&ipa_node, ipa_cfg_write); - install_lib_element(L_IPA_NODE, &ipa_instance_cfg_add_cmd); - install_lib_element(L_IPA_NODE, &ipa_route_cfg_add_cmd); -} diff --git a/tests/Makefile.am b/tests/Makefile.am index 4614b58..04ce529 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -4,7 +4,6 @@ check_PROGRAMS = e1inp_ipa_bsc_test \ e1inp_ipa_bts_test \ - ipa_proxy_test \ subchan_demux/subchan_demux_test \ ipa_recv/ipa_recv_test \ raa_prime/test_dec \ @@ -37,9 +36,6 @@ e1inp_ipa_bts_test_SOURCES = e1inp_ipa_bts_test.c e1inp_ipa_bts_test_LDADD = $(ABIS_LA_LIBS) -ipa_proxy_test_SOURCES = ipa_proxy_test.c -ipa_proxy_test_LDADD = $(ABIS_LA_LIBS) - subchan_demux_subchan_demux_test_SOURCES = subchan_demux/subchan_demux_test.c subchan_demux_subchan_demux_test_LDADD = $(ABIS_LA_LIBS) diff --git a/tests/ipa_proxy_test.c b/tests/ipa_proxy_test.c deleted file mode 100644 index 6c486bd..0000000 --- a/tests/ipa_proxy_test.c +++ /dev/null @@ -1,55 +0,0 @@ -#include <stdio.h> -#include <osmocom/core/talloc.h> -#include <osmocom/abis/abis.h> -#include <osmocom/abis/e1_input.h> -#include <osmocom/abis/ipa_proxy.h> -#include <osmocom/core/logging.h> -#include <osmocom/core/application.h> -#include <osmocom/vty/vty.h> -#include <osmocom/vty/command.h> -#include <osmocom/vty/telnet_interface.h> - -static void *tall_test; - -#define DIPA_PROXY_TEST 0 - -struct log_info_cat ipa_proxy_test_cat[] = { - [DIPA_PROXY_TEST] = { - .name = "DLINP_IPA_PROXY_TEST", - .description = "IPA proxy test", - .color = "\033[1;35m", - .enabled = 1, .loglevel = LOGL_NOTICE, - }, -}; - -const struct log_info ipa_proxy_test_log_info = { - .filter_fn = NULL, - .cat = ipa_proxy_test_cat, - .num_cat = ARRAY_SIZE(ipa_proxy_test_cat), -}; - -static struct vty_app_info vty_info = { - .name = "ipa-proxy-test", - .version = "1.0", -}; - -#define IPA_PROXY_TEST_TELNET_PORT 4260 - -int main(void) -{ - tall_test = talloc_named_const(NULL, 1, "ipa proxy test"); - libosmo_abis_init(tall_test); - msgb_talloc_ctx_init(tall_test, 0); - osmo_init_logging2(tall_test, &ipa_proxy_test_log_info); - - vty_init(&vty_info); - ipa_proxy_vty_init(); - - telnet_init_default(tall_test, NULL, IPA_PROXY_TEST_TELNET_PORT); - - LOGP(DIPA_PROXY_TEST, LOGL_NOTICE, "entering main loop\n"); - - while (1) { - osmo_select_main(0); - } -} -- To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/39082?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-abis Gerrit-Branch: master Gerrit-Change-Id: I7fbc3ad7277369d21cf31159407ef6672ebdf64f Gerrit-Change-Number: 39082 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

6 months, 2 weeks

1
0
0 0

[L] Change in libosmo-abis[master]: Drop ipa_proxy

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-abis/+/39082?usp=email ) Change subject: Drop ipa_proxy ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/39082?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmo-abis Gerrit-Branch: master Gerrit-Change-Id: I7fbc3ad7277369d21cf31159407ef6672ebdf64f Gerrit-Change-Number: 39082 Gerrit-PatchSet: 2 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: Tue, 10 Dec 2024 23:14:25 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

6 months, 2 weeks

1
0
0 0

[XS] Change in osmo-uecups[master]: jenkins.sh: Skip building unneeded libosmo-abis

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-uecups/+/39093?usp=email ) Change subject: jenkins.sh: Skip building unneeded libosmo-abis ...................................................................... jenkins.sh: Skip building unneeded libosmo-abis Change-Id: I375cbf95c6bbe3af275aea00e4e13f26784c0d3d --- M contrib/jenkins.sh 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index c5259e5..7510990 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -18,9 +18,8 @@ verify_value_string_arrays_are_terminated.py $(find . -name "*.[hc]") -osmo-build-dep.sh libosmocore "" ac_cv_path_DOXYGEN=false -osmo-build-dep.sh libosmo-netif "" ac_cv_path_DOXYGEN=false -osmo-build-dep.sh libosmo-abis "" ac_cv_path_DOXYGEN=false +osmo-build-dep.sh libosmocore "" --disable-doxygen +osmo-build-dep.sh libosmo-netif "" --disable-doxygen export PKG_CONFIG_PATH="$inst/lib/pkgconfig:$PKG_CONFIG_PATH" export LD_LIBRARY_PATH="$inst/lib" -- To view, visit https://gerrit.osmocom.org/c/osmo-uecups/+/39093?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-uecups Gerrit-Branch: master Gerrit-Change-Id: I375cbf95c6bbe3af275aea00e4e13f26784c0d3d Gerrit-Change-Number: 39093 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

6 months, 2 weeks

1
0
0 0

[XS] Change in osmo-uecups[master]: jenkins.sh: Skip building unneeded libosmo-abis

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-uecups/+/39093?usp=email ) Change subject: jenkins.sh: Skip building unneeded libosmo-abis ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-uecups/+/39093?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-uecups Gerrit-Branch: master Gerrit-Change-Id: I375cbf95c6bbe3af275aea00e4e13f26784c0d3d Gerrit-Change-Number: 39093 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: Tue, 10 Dec 2024 23:14:10 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

6 months, 2 weeks

1
0
0 0

[XS] Change in osmo-sgsn[master]: jenkins.sh: No need to build libosmo-sigtran with doxygen

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/39096?usp=email ) Change subject: jenkins.sh: No need to build libosmo-sigtran with doxygen ...................................................................... jenkins.sh: No need to build libosmo-sigtran with doxygen Change-Id: Iab3d1b2e1591e66702a01def6c322f4a11970ead --- M contrib/jenkins.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index ced8367..d3c7616 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -38,7 +38,7 @@ osmo-build-dep.sh osmo-hlr if [ "x$IU" = "x--enable-iu" ]; then - osmo-build-dep.sh libosmo-sigtran + osmo-build-dep.sh libosmo-sigtran "" --disable-doxygen osmo-build-dep.sh libasn1c #osmo-build-dep.sh asn1c aper-prefix # only needed for make regen in osmo-iuh osmo-build-dep.sh osmo-iuh -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/39096?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: Iab3d1b2e1591e66702a01def6c322f4a11970ead Gerrit-Change-Number: 39096 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

6 months, 2 weeks

1
0
0 0

[XS] Change in osmo-sgsn[master]: jenkins.sh: No need to build libosmo-sigtran with doxygen

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/39096?usp=email ) Change subject: jenkins.sh: No need to build libosmo-sigtran with doxygen ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/39096?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: Iab3d1b2e1591e66702a01def6c322f4a11970ead Gerrit-Change-Number: 39096 Gerrit-PatchSet: 2 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: Tue, 10 Dec 2024 23:13:58 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

6 months, 2 weeks

1
0
0 0

[XS] Change in osmo-bsc[master]: jenkins.sh: No need to build libosmo-sigtran with doxygen

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/39098?usp=email ) Change subject: jenkins.sh: No need to build libosmo-sigtran with doxygen ...................................................................... jenkins.sh: No need to build libosmo-sigtran with doxygen Change-Id: I450b5a9c875a34ae220a44f8a620b973538dc1c9 --- M contrib/jenkins.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index 9f9d2f7..6713376 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -56,7 +56,7 @@ osmo-build-dep.sh libosmo-netif "" --disable-doxygen osmo-build-dep.sh libosmo-abis -osmo-build-dep.sh libosmo-sigtran +osmo-build-dep.sh libosmo-sigtran "" --disable-doxygen osmo-build-dep.sh osmo-mgw # Additional configure options and depends -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/39098?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I450b5a9c875a34ae220a44f8a620b973538dc1c9 Gerrit-Change-Number: 39098 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

6 months, 2 weeks

1
0
0 0

[XS] Change in osmo-bsc[master]: jenkins.sh: No need to build libosmo-sigtran with doxygen

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-bsc/+/39098?usp=email ) Change subject: jenkins.sh: No need to build libosmo-sigtran with doxygen ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/39098?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I450b5a9c875a34ae220a44f8a620b973538dc1c9 Gerrit-Change-Number: 39098 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: Tue, 10 Dec 2024 23:13:49 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

6 months, 2 weeks

1
0
0 0

[XS] Change in osmo-msc[master]: jenkins.sh: No need to build libosmo-sigtran with doxygen

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/39099?usp=email ) Change subject: jenkins.sh: No need to build libosmo-sigtran with doxygen ...................................................................... jenkins.sh: No need to build libosmo-sigtran with doxygen Change-Id: I5fdb9b139c1c233a0499ba1fbcd9046491714dd4 --- M contrib/jenkins.sh 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index 26212ed..b202765 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -33,7 +33,7 @@ mkdir "$deps" || true -osmo-build-dep.sh libosmocore "" ac_cv_path_DOXYGEN=false +osmo-build-dep.sh libosmocore "" --disable-doxygen verify_value_string_arrays_are_terminated.py $(find . -name "*.[hc]") @@ -43,7 +43,7 @@ osmo-build-dep.sh libosmo-netif "" --disable-doxygen osmo-build-dep.sh libosmo-abis -osmo-build-dep.sh libosmo-sigtran +osmo-build-dep.sh libosmo-sigtran "" --disable-doxygen osmo-build-dep.sh libsmpp34 osmo-build-dep.sh osmo-mgw osmo-build-dep.sh osmo-hlr -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/39099?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I5fdb9b139c1c233a0499ba1fbcd9046491714dd4 Gerrit-Change-Number: 39099 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

6 months, 2 weeks

1
0
0 0

2025

2024

2023

2022

gerrit-log December 2024