falconia has submitted this change. ( https://gerrit.osmocom.org/c/osmo-e1d/+/37319?usp=email )
Change subject: e1_ts_stop: clear tx_started flag
......................................................................
e1_ts_stop: clear tx_started flag
ts->raw.tx_started flag gets set in the channelized mux process
when bytes begin arriving from the raw TS client application.
However, there is no reset of this flag anywhere, hence if a user
opens some TS, then kills that client application, then opens it
again (with the same or different client app), there is a flood
of error messages about TS read underflow. Solution: clear the
tx_started flag when the TS is stopped.
Change-Id: I28ff68208f1d3d20dc57c5306fec74bd2f884cb1
---
M src/ctl.c
1 file changed, 18 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
tnt: Looks good to me, approved
diff --git a/src/ctl.c b/src/ctl.c
index e63589b..a9818b3 100644
--- a/src/ctl.c
+++ b/src/ctl.c
@@ -119,6 +119,7 @@
ts->raw.rx_buf = NULL;
ts->raw.rx_buf_size = 0;
ts->raw.rx_buf_used = 0;
+ ts->raw.tx_started = false;
}
static void
--
To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/37319?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1d
Gerrit-Branch: master
Gerrit-Change-Id: I28ff68208f1d3d20dc57c5306fec74bd2f884cb1
Gerrit-Change-Number: 37319
Gerrit-PatchSet: 1
Gerrit-Owner: falconia <falcon(a)freecalypso.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: falconia <falcon(a)freecalypso.org>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: tnt <tnt(a)246tNt.com>
Gerrit-MessageType: merged
falconia has submitted this change. ( https://gerrit.osmocom.org/c/osmo-e1d/+/37318?usp=email )
Change subject: configure.ac: lower libusb-1.0 requirement to 1.0.20
......................................................................
configure.ac: lower libusb-1.0 requirement to 1.0.20
Current configure.ac requires libusb version to be >= 1.0.21.
Slackware 14.2 has libusb version 1.0.20. Lowering the
requirement in configure.ac produces a successful build,
i.e., the actual code does not use any libusb APIs that are
new with version 1.0.21.
Change-Id: I15586757e2985184359a4488c79f2c587affeb56
---
M configure.ac
1 file changed, 16 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/configure.ac b/configure.ac
index 8bec040..cea8eb4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -38,7 +38,7 @@
PKG_CHECK_MODULES(LIBOSMOCORE, libosmocore >= 1.9.0)
PKG_CHECK_MODULES(LIBOSMOVTY, libosmovty >= 1.9.0)
PKG_CHECK_MODULES(LIBOSMOUSB, libosmousb >= 1.9.0)
-PKG_CHECK_MODULES(LIBUSB, libusb-1.0 >= 1.0.21)
+PKG_CHECK_MODULES(LIBUSB, libusb-1.0 >= 1.0.20)
AC_CONFIG_MACRO_DIR([m4])
--
To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/37318?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1d
Gerrit-Branch: master
Gerrit-Change-Id: I15586757e2985184359a4488c79f2c587affeb56
Gerrit-Change-Number: 37318
Gerrit-PatchSet: 1
Gerrit-Owner: falconia <falcon(a)freecalypso.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: falconia <falcon(a)freecalypso.org>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: tnt <tnt(a)246tNt.com>
Gerrit-MessageType: merged
osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/37329?usp=email )
Change subject: ttcn3-ggsn-test/jenkins: tweak loop logic
......................................................................
ttcn3-ggsn-test/jenkins: tweak loop logic
Iterate over TEST_CONFIGS instead of TEST_CONFIGS_ALL and then filtering
with test_config_enabled.
Change-Id: Ie7701abe6ab7cff139766eca9240e0f26af32231
---
M ttcn3-ggsn-test/jenkins.sh
1 file changed, 13 insertions(+), 5 deletions(-)
Approvals:
pespin: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/ttcn3-ggsn-test/jenkins.sh b/ttcn3-ggsn-test/jenkins.sh
index 7c9c7ac..bd0fae1 100755
--- a/ttcn3-ggsn-test/jenkins.sh
+++ b/ttcn3-ggsn-test/jenkins.sh
@@ -109,11 +109,7 @@
CONFIGS_DIR="osmo-ggsn"
fi
-for i in $TEST_CONFIGS_ALL; do
- if ! test_config_enabled "$i"; then
- continue
- fi
-
+for i in $TEST_CONFIGS; do
# OS#6106: Currently it is not possible to configure multiple APNs with
# gtpu-mode kernel-gtp in OsmoGGSN, because it cannot share the GTP-U
# bind socket between tunnels.
--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/37329?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: Ie7701abe6ab7cff139766eca9240e0f26af32231
Gerrit-Change-Number: 37329
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/docker-playground/+/37329?usp=email )
Change subject: ttcn3-ggsn-test/jenkins: tweak loop logic
......................................................................
ttcn3-ggsn-test/jenkins: tweak loop logic
Iterate over TEST_CONFIGS instead of TEST_CONFIGS_ALL and then filtering
with test_config_enabled.
Change-Id: Ie7701abe6ab7cff139766eca9240e0f26af32231
---
M ttcn3-ggsn-test/jenkins.sh
1 file changed, 13 insertions(+), 5 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/29/37329/1
diff --git a/ttcn3-ggsn-test/jenkins.sh b/ttcn3-ggsn-test/jenkins.sh
index 7c9c7ac..bd0fae1 100755
--- a/ttcn3-ggsn-test/jenkins.sh
+++ b/ttcn3-ggsn-test/jenkins.sh
@@ -109,11 +109,7 @@
CONFIGS_DIR="osmo-ggsn"
fi
-for i in $TEST_CONFIGS_ALL; do
- if ! test_config_enabled "$i"; then
- continue
- fi
-
+for i in $TEST_CONFIGS; do
# OS#6106: Currently it is not possible to configure multiple APNs with
# gtpu-mode kernel-gtp in OsmoGGSN, because it cannot share the GTP-U
# bind socket between tunnels.
--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/37329?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: Ie7701abe6ab7cff139766eca9240e0f26af32231
Gerrit-Change-Number: 37329
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newchange