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

[S] Change in libosmocore[master]: Add environment variable to set io_uring size
by laforge 18 Sep '25

18 Sep '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/40782?usp=email ) Change subject: Add environment variable to set io_uring size ...................................................................... Add environment variable to set io_uring size The term "LIBOSMO_IO_URING_INITIAL_SIZE" is related to the following patch, which will increment the size of the io_uring automatically if the initial size is too small. Related: OS#6705 Change-Id: I55289d9282e13aa1bf82f3931c85c196752f1484 --- M src/core/osmo_io_uring.c 1 file changed, 23 insertions(+), 2 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve lynxis lazus: Looks good to me, but someone else must approve laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/core/osmo_io_uring.c b/src/core/osmo_io_uring.c index 1eaa5e6..48f6f0d 100644 --- a/src/core/osmo_io_uring.c +++ b/src/core/osmo_io_uring.c @@ -52,15 +52,21 @@ #include "osmo_io_internal.h" -#define IOFD_URING_ENTRIES 4096 +#define IOFD_URING_INITIAL_SIZE 4096 +/* 32768 refers to the IORING_MAX_ENTRIES of the kernel (io_uring/io_uring.h). */ +#define IOFD_URING_MAXIMUM_SIZE 32768 #define OSMO_IO_URING_BATCH "LIBOSMO_IO_URING_BATCH" +#define OSMO_IO_URING_INITIAL_SIZE "LIBOSMO_IO_URING_INITIAL_SIZE" + #define OSMO_IO_URING_READ_SQE "LIBOSMO_IO_URING_READ_SQE" bool g_io_uring_batch = false; bool g_io_uring_submit_needed = false; +static unsigned int g_io_uring_size = IOFD_URING_INITIAL_SIZE; + static int g_io_uring_read_sqes = 1; struct osmo_io_uring { @@ -103,7 +109,22 @@ if ((env = getenv(OSMO_IO_URING_BATCH))) g_io_uring_batch = true; - rc = io_uring_queue_init(IOFD_URING_ENTRIES, &g_ring.ring, 0); + if ((env = getenv(OSMO_IO_URING_INITIAL_SIZE))) { + int env_value; + rc = osmo_str_to_int(&env_value, env, 10, 1, IOFD_URING_MAXIMUM_SIZE); + if (rc < 0) { + fprintf(stderr, "Error: Initial io_uring size out of range (1..%d).\n", + IOFD_URING_MAXIMUM_SIZE); + exit(1); + } + if ((env_value & (env_value - 1))) { + fprintf(stderr, "Error: Initial io_uring size must be a positive power of two.\n"); + exit(1); + } + g_io_uring_size = env_value; + } + + rc = io_uring_queue_init(g_io_uring_size, &g_ring.ring, 0); if (rc < 0) osmo_panic("failure during io_uring_queue_init(): %s\n", strerror(-rc)); -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/40782?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I55289d9282e13aa1bf82f3931c85c196752f1484 Gerrit-Change-Number: 40782 Gerrit-PatchSet: 8 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in libosmocore[master]: Send multiple read/recvfrom/recvmsg SQEs in advance
by laforge 18 Sep '25

18 Sep '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/40494?usp=email ) Change subject: Send multiple read/recvfrom/recvmsg SQEs in advance ...................................................................... Send multiple read/recvfrom/recvmsg SQEs in advance Multiple read or recvfrom operations can be submitted via SQEs when using io_uring. This allows reading multiple packet / more data between calls of osmo_select_main() the main loop. Having more than one SQE submitted is optional and the number can be controlled via environment variable. Related: OS#6705 Change-Id: Id50a8900fa2fe6de553e5025feae7e1e8d501e30 --- M include/osmocom/core/osmo_io.h M src/core/libosmocore.map M src/core/osmo_io.c M src/core/osmo_io_internal.h M src/core/osmo_io_uring.c 5 files changed, 103 insertions(+), 14 deletions(-) Approvals: pespin: Looks good to me, approved laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/include/osmocom/core/osmo_io.h b/include/osmocom/core/osmo_io.h index 0b3dc83..0733240 100644 --- a/include/osmocom/core/osmo_io.h +++ b/include/osmocom/core/osmo_io.h @@ -213,6 +213,7 @@ enum osmo_io_fd_mode mode, const struct osmo_io_ops *ioops, void *data); int osmo_iofd_set_cmsg_size(struct osmo_io_fd *iofd, size_t cmsg_size); int osmo_iofd_set_io_buffers(struct osmo_io_fd *iofd, enum osmo_io_op op, uint8_t buffers); +int osmo_iofd_set_sqes(struct osmo_io_fd *iofd, enum osmo_io_op op, uint8_t sqes); int osmo_iofd_register(struct osmo_io_fd *iofd, int fd); int osmo_iofd_unregister(struct osmo_io_fd *iofd); unsigned int osmo_iofd_txqueue_len(struct osmo_io_fd *iofd); diff --git a/src/core/libosmocore.map b/src/core/libosmocore.map index 239597c..198b31e 100644 --- a/src/core/libosmocore.map +++ b/src/core/libosmocore.map @@ -276,6 +276,7 @@ osmo_iofd_set_alloc_info; osmo_iofd_set_cmsg_size; osmo_iofd_set_io_buffers; +osmo_iofd_set_sqes; osmo_iofd_set_data; osmo_iofd_set_ioops; osmo_iofd_set_priv_nr; diff --git a/src/core/osmo_io.c b/src/core/osmo_io.c index 62d2362..2f3af06 100644 --- a/src/core/osmo_io.c +++ b/src/core/osmo_io.c @@ -816,6 +816,14 @@ iofd->io_read_buffers = 1; iofd->io_write_buffers = 1; + if (osmo_iofd_ops.setup) { + int rc = osmo_iofd_ops.setup(iofd); + if (rc < 0) { + osmo_iofd_free(iofd); + return NULL; + } + } + return iofd; } @@ -868,6 +876,35 @@ return 0; } +/*! Set the number of SQEs that are submitted to an io_unring before completion is received. + * + * If the io_using backend is selected, this API function can be used to tell the osmo_io process how many SQE are + * scheduled in advance. + * The feature is currently supports scheduling read SQEs only. + * + * \param[in] iofd the iofd file descriptor + * \param[in] op the osmo_io_op (read) to set the number of IO buffers for + * \param[in] number of scheduled SQEs + * \returns zero on success, a negative value on error + */ +int osmo_iofd_set_sqes(struct osmo_io_fd *iofd, enum osmo_io_op op, uint8_t sqes) +{ + if (iofd->mode != OSMO_IO_FD_MODE_READ_WRITE) + return -EINVAL; + + if (g_io_backend != OSMO_IO_BACKEND_IO_URING) + return -EINVAL; + + if (op != OSMO_IO_OP_READ) + return -EINVAL; + + if (sqes < 1 || sqes > IOFD_MSGHDR_MAX_READ_SQES) + return -EINVAL; + + iofd->u.uring.num_read_sqes = sqes; + return 0; +} + /*! Register the osmo_io_fd for active I/O. * * Calling this function will register a previously initialized osmo_io_fd for performing I/O. diff --git a/src/core/osmo_io_internal.h b/src/core/osmo_io_internal.h index d28e3d7..f983b77 100644 --- a/src/core/osmo_io_internal.h +++ b/src/core/osmo_io_internal.h @@ -25,6 +25,7 @@ #endif struct iofd_backend_ops { + int (*setup)(struct osmo_io_fd *iofd); int (*register_fd)(struct osmo_io_fd *iofd); int (*unregister_fd)(struct osmo_io_fd *iofd); int (*close)(struct osmo_io_fd *iofd); @@ -49,6 +50,8 @@ #define IOFD_FLAG_ISSET(iofd, flag) ((iofd)->flags & (flag)) +#define IOFD_MSGHDR_MAX_READ_SQES 32 + struct osmo_io_fd { /*! linked list for internal management */ struct llist_head list; @@ -107,7 +110,12 @@ struct { bool read_enabled; bool write_enabled; - void *read_msghdr; + /*! requested number of simultaniously submitted read SQEs */ + uint8_t num_read_sqes; + /*! array of simultaneously submitted read SQEs */ + void *read_msghdr[IOFD_MSGHDR_MAX_READ_SQES]; + /*! current number of simultaneously submitted read SQEs */ + uint8_t reads_submitted; void *write_msghdr; /* TODO: index into array of registered fd's? */ /* osmo_fd for non-blocking connect handling */ diff --git a/src/core/osmo_io_uring.c b/src/core/osmo_io_uring.c index 018bdab..1eaa5e6 100644 --- a/src/core/osmo_io_uring.c +++ b/src/core/osmo_io_uring.c @@ -56,9 +56,13 @@ #define OSMO_IO_URING_BATCH "LIBOSMO_IO_URING_BATCH" +#define OSMO_IO_URING_READ_SQE "LIBOSMO_IO_URING_READ_SQE" + bool g_io_uring_batch = false; bool g_io_uring_submit_needed = false; +static int g_io_uring_read_sqes = 1; + struct osmo_io_uring { struct osmo_fd event_ofd; struct io_uring ring; @@ -103,6 +107,15 @@ if (rc < 0) osmo_panic("failure during io_uring_queue_init(): %s\n", strerror(-rc)); + if ((env = getenv(OSMO_IO_URING_READ_SQE))) { + g_io_uring_read_sqes = atoi(env); + if (g_io_uring_read_sqes < 1 || g_io_uring_read_sqes > IOFD_MSGHDR_MAX_READ_SQES) { + fprintf(stderr, "Invalid osmo_uring read SQEs requested: \"%s\"\n Allowed range: 1..%d\n", + env, IOFD_MSGHDR_MAX_READ_SQES); + exit(1); + } + } + rc = eventfd(0, 0); if (rc < 0) { io_uring_queue_exit(&g_ring.ring); @@ -134,7 +147,7 @@ g_io_uring_submit_needed = true; } -static void iofd_uring_submit_recv(struct osmo_io_fd *iofd, enum iofd_msg_action action) +static void iofd_uring_submit_recv_sqe(struct osmo_io_fd *iofd, enum iofd_msg_action action) { struct msgb *msg; struct iofd_msghdr *msghdr; @@ -196,15 +209,37 @@ iofd_io_uring_submit(); - /* NOTE: This only works if we have one read per fd */ - iofd->u.uring.read_msghdr = msghdr; + iofd->u.uring.read_msghdr[iofd->u.uring.reads_submitted] = msghdr; + iofd->u.uring.reads_submitted++; +} + +static void iofd_uring_submit_recv(struct osmo_io_fd *iofd, enum iofd_msg_action action) +{ + /* Submit more read SQEs in advance, if requested. */ + while (iofd->u.uring.reads_submitted < iofd->u.uring.num_read_sqes) + iofd_uring_submit_recv_sqe(iofd, action); } /*! completion call-back for READ/RECVFROM */ static void iofd_uring_handle_recv(struct iofd_msghdr *msghdr, int rc) { struct osmo_io_fd *iofd = msghdr->iofd; - uint8_t idx; + uint8_t idx, i; + + /* Find which read_msghdr is completed and remove from list. */ + for (idx = 0; idx < iofd->u.uring.reads_submitted; idx++) { + if (iofd->u.uring.read_msghdr[idx] == msghdr) + break; + } + if (idx == iofd->u.uring.reads_submitted) { + LOGP(DLIO, LOGL_FATAL, "Read SQE completion, but msghdr not found, please fix!\n"); + return; + } + /* Remove entry at idx. */ + iofd->u.uring.reads_submitted--; + for (i = idx; i < iofd->u.uring.reads_submitted; i++) + iofd->u.uring.read_msghdr[i] = iofd->u.uring.read_msghdr[i + 1]; + iofd->u.uring.read_msghdr[i] = NULL; for (idx = 0; idx < msghdr->io_len; idx++) { struct msgb *msg = msghdr->msg[idx]; @@ -238,9 +273,6 @@ if (iofd->u.uring.read_enabled && !IOFD_FLAG_ISSET(iofd, IOFD_FLAG_CLOSED)) iofd_uring_submit_recv(iofd, msghdr->action); - else - iofd->u.uring.read_msghdr = NULL; - iofd_msghdr_free(msghdr); } @@ -300,7 +332,7 @@ IOFD_FLAG_UNSET(iofd, IOFD_FLAG_IN_CALLBACK); - if (IOFD_FLAG_ISSET(iofd, IOFD_FLAG_TO_FREE) && !iofd->u.uring.read_msghdr && !iofd->u.uring.write_msghdr) + if (IOFD_FLAG_ISSET(iofd, IOFD_FLAG_TO_FREE) && !iofd->u.uring.reads_submitted && !iofd->u.uring.write_msghdr) talloc_free(iofd); } @@ -410,11 +442,18 @@ if (iofd->u.uring.read_enabled && !IOFD_FLAG_ISSET(iofd, IOFD_FLAG_CLOSED)) iofd_uring_read_enable(iofd); - if (IOFD_FLAG_ISSET(iofd, IOFD_FLAG_TO_FREE) && !iofd->u.uring.read_msghdr && !iofd->u.uring.write_msghdr) + if (IOFD_FLAG_ISSET(iofd, IOFD_FLAG_TO_FREE) && !iofd->u.uring.reads_submitted && !iofd->u.uring.write_msghdr) talloc_free(iofd); return 0; } +static int iofd_uring_setup(struct osmo_io_fd *iofd) +{ + iofd->u.uring.num_read_sqes = g_io_uring_read_sqes; + + return 0; +} + static int iofd_uring_register(struct osmo_io_fd *iofd) { if (iofd->mode != OSMO_IO_FD_MODE_RECVMSG_SENDMSG) @@ -440,18 +479,20 @@ { struct io_uring_sqe *sqe; struct iofd_msghdr *msghdr; + uint8_t idx; - if (iofd->u.uring.read_msghdr) { - msghdr = iofd->u.uring.read_msghdr; + for (idx = 0; idx < iofd->u.uring.reads_submitted; idx++) { + msghdr = iofd->u.uring.read_msghdr[idx]; + iofd->u.uring.read_msghdr[idx] = NULL; sqe = io_uring_get_sqe(&g_ring.ring); OSMO_ASSERT(sqe != NULL); io_uring_sqe_set_data(sqe, NULL); LOGPIO(iofd, LOGL_DEBUG, "Cancelling read\n"); - iofd->u.uring.read_msghdr = NULL; talloc_steal(OTC_GLOBAL, msghdr); msghdr->iofd = NULL; io_uring_prep_cancel(sqe, msghdr, 0); } + iofd->u.uring.reads_submitted = 0; if (iofd->u.uring.write_msghdr) { msghdr = iofd->u.uring.write_msghdr; @@ -533,7 +574,7 @@ { iofd->u.uring.read_enabled = true; - if (iofd->u.uring.read_msghdr) + if (iofd->u.uring.reads_submitted) return; /* This function is called again, once the socket is connected. */ @@ -580,6 +621,7 @@ } const struct iofd_backend_ops iofd_uring_ops = { + .setup = iofd_uring_setup, .register_fd = iofd_uring_register, .unregister_fd = iofd_uring_unregister, .close = iofd_uring_close, -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/40494?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Id50a8900fa2fe6de553e5025feae7e1e8d501e30 Gerrit-Change-Number: 40494 Gerrit-PatchSet: 16 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in libosmocore[master]: Put all io_uring related read and write states into sub structures
by laforge 18 Sep '25

18 Sep '25

Attention is currently required from: jolly. laforge has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/libosmocore/+/40856?usp=email ) Change subject: Put all io_uring related read and write states into sub structures ...................................................................... Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/40856?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I3cf4ab6d9aebf5532ad174c90c7b0e9633491c88 Gerrit-Change-Number: 40856 Gerrit-PatchSet: 5 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 18 Sep 2025 09:41:53 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in libosmocore[master]: Remove old empty io_uring
by laforge 18 Sep '25

18 Sep '25

Attention is currently required from: jolly. laforge has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/libosmocore/+/40855?usp=email ) Change subject: Remove old empty io_uring ...................................................................... Patch Set 6: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/40855?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Id2d2a0400ad442198c684ea0ead4eaeaead4c53d Gerrit-Change-Number: 40855 Gerrit-PatchSet: 6 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 18 Sep 2025 09:41:36 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in libosmocore[master]: Automatically increase io_uring, if too small.
by laforge 18 Sep '25

18 Sep '25

Attention is currently required from: jolly, pespin. laforge has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/libosmocore/+/40725?usp=email ) Change subject: Automatically increase io_uring, if too small. ...................................................................... Patch Set 14: Code-Review+2 (1 comment) Patchset: PS2: > I think having two rings and gradually migrating over is clean, and we can test the scenario by runn […] Done -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/40725?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Id9230146acc8d54bfd44834e783c31b37bd64bca Gerrit-Change-Number: 40725 Gerrit-PatchSet: 14 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 18 Sep 2025 09:41:05 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in libosmocore[master]: Add environment variable to set io_uring size
by laforge 18 Sep '25

18 Sep '25

Attention is currently required from: jolly. laforge has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/libosmocore/+/40782?usp=email ) Change subject: Add environment variable to set io_uring size ...................................................................... Patch Set 7: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/40782?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I55289d9282e13aa1bf82f3931c85c196752f1484 Gerrit-Change-Number: 40782 Gerrit-PatchSet: 7 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 18 Sep 2025 09:40:09 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-msc[master]: tests/{ctrl,vty}_test_runner.py: dump stdout/stderr
by fixeria 18 Sep '25

18 Sep '25

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/41127?usp=email ) Change subject: tests/{ctrl,vty}_test_runner.py: dump stdout/stderr ...................................................................... tests/{ctrl,vty}_test_runner.py: dump stdout/stderr From time to time, we're seeing the following error in Jenkins: ---------------------------------------------------------------------- Traceback (most recent call last): File ".../tests/vty_test_runner.py", line 70, in tearDown raise Exception("Process returned %d" % rc) Exception: Process returned -9 ---------------------------------------------------------------------- Let's dump stdout/stderr of the process to get more information. Change-Id: Ie6a2f5b1bc56d35513643b52923403798e5a0a5a Related: OS#5665 --- M tests/ctrl_test_runner.py M tests/vty_test_runner.py 2 files changed, 37 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/tests/ctrl_test_runner.py b/tests/ctrl_test_runner.py index 16d1e7c..127d747 100755 --- a/tests/ctrl_test_runner.py +++ b/tests/ctrl_test_runner.py @@ -26,6 +26,8 @@ import socket import sys import struct +import subprocess +import tempfile import osmopy.obscvty as obscvty import osmopy.osmoutil as osmoutil @@ -50,8 +52,12 @@ cfi = config_index + 1 osmo_ctrl_cmd[cfi] = os.path.join(confpath, osmo_ctrl_cmd[cfi]) + self.stdout = tempfile.TemporaryFile() + self.stderr = tempfile.TemporaryFile() try: - self.proc = osmoutil.popen_devnull(osmo_ctrl_cmd) + # self.proc = osmoutil.popen_devnull(osmo_ctrl_cmd) + print("Launching: PWD=%s %s" % (os.getcwd(), ' '.join([repr(c) for c in osmo_ctrl_cmd]))) + self.proc = subprocess.Popen(osmo_ctrl_cmd, stdout=self.stdout, stderr=self.stderr) except OSError: print("Current directory: %s" % os.getcwd(), file=sys.stderr) print("Consider setting -b", file=sys.stderr) @@ -62,11 +68,23 @@ self.connect("127.0.0.1", appport) self.next_id = 1000 + def dump_file(self, file, name): + file.seek(0) + data = file.read() + print('=' * 80) + print(name) + print('=' * 80) + print(data.decode()) + def tearDown(self): self.disconnect() rc = osmoutil.end_proc(self.proc) if rc is not None and rc != 0: + self.dump_file(self.stdout, 'stdout') + self.dump_file(self.stderr, 'stderr') raise Exception("Process returned %d" % rc) + self.stdout.close() + self.stderr.close() def disconnect(self): if not (self.sock is None): diff --git a/tests/vty_test_runner.py b/tests/vty_test_runner.py index 2dfa155..571080e 100755 --- a/tests/vty_test_runner.py +++ b/tests/vty_test_runner.py @@ -20,6 +20,7 @@ import unittest import socket import subprocess +import tempfile import osmopy.obscvty as obscvty import osmopy.osmoutil as osmoutil @@ -51,8 +52,12 @@ cfi = config_index + 1 osmo_vty_cmd[cfi] = os.path.join(confpath, osmo_vty_cmd[cfi]) + self.stdout = tempfile.TemporaryFile() + self.stderr = tempfile.TemporaryFile() try: - self.proc = osmoutil.popen_devnull(osmo_vty_cmd) + # self.proc = osmoutil.popen_devnull(osmo_vty_cmd) + print("Launching: PWD=%s %s" % (os.getcwd(), ' '.join([repr(c) for c in osmo_vty_cmd]))) + self.proc = subprocess.Popen(osmo_vty_cmd, stdout=self.stdout, stderr=self.stderr) except OSError: print("Current directory: %s" % os.getcwd(), file=sys.stderr) print("Consider setting -b", file=sys.stderr) @@ -61,13 +66,25 @@ appport = self.vty_app()[0] self.vty = obscvty.VTYInteract(appstring, "127.0.0.1", appport) + def dump_file(self, file, name): + file.seek(0) + data = file.read() + print('=' * 80) + print(name) + print('=' * 80) + print(data.decode()) + def tearDown(self): if self.vty: self.vty._close_socket() self.vty = None rc = osmoutil.end_proc(self.proc) if rc is not None and rc != 0: + self.dump_file(self.stdout, 'stdout') + self.dump_file(self.stderr, 'stderr') raise Exception("Process returned %d" % rc) + self.stdout.close() + self.stderr.close() class TestVTYMSC(TestVTYBase): -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/41127?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: Ie6a2f5b1bc56d35513643b52923403798e5a0a5a Gerrit-Change-Number: 41127 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-msc[master]: tests/{ctrl,vty}_test_runner.py: dump stdout/stderr
by osmith 18 Sep '25

18 Sep '25

Attention is currently required from: fixeria. osmith has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-msc/+/41127?usp=email ) Change subject: tests/{ctrl,vty}_test_runner.py: dump stdout/stderr ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/41127?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Ie6a2f5b1bc56d35513643b52923403798e5a0a5a Gerrit-Change-Number: 41127 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Thu, 18 Sep 2025 06:10:10 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[L] Change in osmo-ttcn3-hacks[master]: Move RAN_Configurations module parameter definition to *_Tests.default
by pespin 17 Sep '25

17 Sep '25

Attention is currently required from: pespin. Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41129?usp=email to look at the new patch set (#3). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: Move RAN_Configurations module parameter definition to *_Tests.default ...................................................................... Move RAN_Configurations module parameter definition to *_Tests.default This way we can easily tweak values, which usually involve other programs' config like osmo-stp, without needing to recompile ttcn-3. Furthermore, we avoid ending up in the situation where the .ttcn file contains invalid data which is alwayred overridden in .default, like it happened in MSC_Tests.ttcn. Change-Id: Id7e77a2ecf68520a853dc35fe0013a0611e744a1 --- M bsc/BSC_Tests.default M bsc/BSC_Tests.ttcn M bsc/BSC_Tests_SCCPlite.cfg M hnbgw/HNBGW_Tests.default M hnbgw/HNBGW_Tests.ttcn M msc/MSC_Tests.ttcn M sgsn/SGSN_Tests.cfg M sgsn/SGSN_Tests.default M sgsn/SGSN_Tests.ttcn 9 files changed, 146 insertions(+), 170 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/29/41129/3 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41129?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Id7e77a2ecf68520a853dc35fe0013a0611e744a1 Gerrit-Change-Number: 41129 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Attention: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-ttcn3-hacks[master]: smlc: Remove unneeded RAN_Adapter.ttcnpp from gen_links.sh
by pespin 17 Sep '25

17 Sep '25

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41130?usp=email to look at the new patch set (#2). Change subject: smlc: Remove unneeded RAN_Adapter.ttcnpp from gen_links.sh ...................................................................... smlc: Remove unneeded RAN_Adapter.ttcnpp from gen_links.sh Change-Id: I332c09c65a9dba40bca2851eb34cd3edf8da8727 --- M smlc/gen_links.sh 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/30/41130/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41130?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I332c09c65a9dba40bca2851eb34cd3edf8da8727 Gerrit-Change-Number: 41130 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 0

2025

2024

2023

2022

gerrit-log September 2025