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

Change in osmo-sgsn[master]: Standarize lle and llme state enum & value_string

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/30856 ) Change subject: Standarize lle and llme state enum & value_string ...................................................................... Standarize lle and llme state enum & value_string Change-Id: Iaf102ce5ca60854fe7eb1af17c73a80e7c76181b --- M include/osmocom/sgsn/gprs_llc.h M src/sgsn/gprs_llc.c M src/sgsn/gprs_llc_vty.c 3 files changed, 13 insertions(+), 12 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/56/30856/1 diff --git a/include/osmocom/sgsn/gprs_llc.h b/include/osmocom/sgsn/gprs_llc.h index 7ba5510..32b16b0 100644 --- a/include/osmocom/sgsn/gprs_llc.h +++ b/include/osmocom/sgsn/gprs_llc.h @@ -90,6 +90,7 @@ GPRS_LLES_LOCAL_REL = 6, /* Local Release */ GPRS_LLES_TIMER_REC = 7, /* Timer Recovery */ }; +extern const struct value_string gprs_llc_lle_state_names[]; enum gprs_llc_llme_state { GPRS_LLMS_UNASSIGNED = 1, /* No TLLI yet */ diff --git a/src/sgsn/gprs_llc.c b/src/sgsn/gprs_llc.c index fda0319..84fad9e 100644 --- a/src/sgsn/gprs_llc.c +++ b/src/sgsn/gprs_llc.c @@ -50,6 +50,17 @@ { 0, NULL } }; +const struct value_string gprs_llc_lle_state_names[] = { + { GPRS_LLES_UNASSIGNED, "TLLI Unassigned" }, + { GPRS_LLES_ASSIGNED_ADM, "TLLI Assigned" }, + { GPRS_LLES_LOCAL_EST, "Local Establishment" }, + { GPRS_LLES_REMOTE_EST, "Remote Establishment" }, + { GPRS_LLES_ABM, "Asynchronous Balanced Mode" }, + { GPRS_LLES_LOCAL_REL, "Local Release" }, + { GPRS_LLES_TIMER_REC, "Timer Recovery" }, + { 0, NULL } +}; + static struct gprs_llc_llme *llme_alloc(uint32_t tlli); static int gprs_llc_tx_xid(const struct gprs_llc_lle *lle, struct msgb *msg, int command); diff --git a/src/sgsn/gprs_llc_vty.c b/src/sgsn/gprs_llc_vty.c index 124daa3..4572f95 100644 --- a/src/sgsn/gprs_llc_vty.c +++ b/src/sgsn/gprs_llc_vty.c @@ -39,22 +39,11 @@ #include <osmocom/vty/vty.h> #include <osmocom/vty/command.h> -struct value_string gprs_llc_state_strs[] = { - { GPRS_LLES_UNASSIGNED, "TLLI Unassigned" }, - { GPRS_LLES_ASSIGNED_ADM, "TLLI Assigned" }, - { GPRS_LLES_LOCAL_EST, "Local Establishment" }, - { GPRS_LLES_REMOTE_EST, "Remote Establishment" }, - { GPRS_LLES_ABM, "Asynchronous Balanced Mode" }, - { GPRS_LLES_LOCAL_REL, "Local Release" }, - { GPRS_LLES_TIMER_REC, "Timer Recovery" }, - { 0, NULL } -}; - static void vty_dump_lle(struct vty *vty, struct gprs_llc_lle *lle) { struct gprs_llc_params *par = &lle->params; vty_out(vty, " SAPI %2u State %s VUsend=%u, VUrecv=%u", lle->sapi, - get_value_string(gprs_llc_state_strs, lle->state), + get_value_string(gprs_llc_lle_state_names, lle->state), lle->vu_send, lle->vu_recv); vty_out(vty, " Vsent=%u Vack=%u Vrecv=%u, RetransCtr=%u%s", lle->v_sent, lle->v_ack, lle->v_recv, -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/30856 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: Iaf102ce5ca60854fe7eb1af17c73a80e7c76181b Gerrit-Change-Number: 30856 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 5 months

1
0
0 0

Change in osmo-bsc[master]: pcu_sock: fill in TRX parameters properly

by dexter

Attention is currently required from: dexter. Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-bsc/+/30850 to look at the new patch set (#2). Change subject: pcu_sock: fill in TRX parameters properly ...................................................................... pcu_sock: fill in TRX parameters properly The pcu_sock interface has eveolved. There is a trx info indication that transfers trx (and hopping parameter) to the PCU. Fill in those parameters and send them to the PCU. Change-Id: Ie5c1abcf7b25fe00b6c228c49648e018f94190f0 Related: OS#5198 --- M src/osmo-bsc/pcu_sock.c 1 file changed, 55 insertions(+), 25 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/50/30850/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/30850 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: Ie5c1abcf7b25fe00b6c228c49648e018f94190f0 Gerrit-Change-Number: 30850 Gerrit-PatchSet: 2 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-MessageType: newpatchset

2 years, 5 months

1
0
0 0

Change in osmo-gsm-tester[master]: fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs

by fixeria

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/30853 ) Change subject: fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs ...................................................................... fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs Change-Id: I44797a047639ce816adba626be72fda07f7eb1f8 Related: osmocom-bb.git I8e0f826c9b2a886f94624176e34e7d197e93d25f --- M contrib/jenkins-build-osmocom-bb.sh 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: pespin: Looks good to me, approved; Verified diff --git a/contrib/jenkins-build-osmocom-bb.sh b/contrib/jenkins-build-osmocom-bb.sh index aa2e778..0c09aee 100755 --- a/contrib/jenkins-build-osmocom-bb.sh +++ b/contrib/jenkins-build-osmocom-bb.sh @@ -8,6 +8,7 @@ FW_RPM_URL="http://download.opensuse.org/repositories/home:/mnhauke:/osmocom:/nightly/o…" build_repo libosmocore --disable-doxygen +build_repo libosmo-gprs build_repo_dir osmocom-bb src/host/virt_phy ${SANITIZE_FLAGS} build_repo_dir osmocom-bb src/host/osmocon ${SANITIZE_FLAGS} build_repo_dir osmocom-bb src/host/layer23 ${SANITIZE_FLAGS} -- To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/30853 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gsm-tester Gerrit-Branch: master Gerrit-Change-Id: I44797a047639ce816adba626be72fda07f7eb1f8 Gerrit-Change-Number: 30853 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 5 months

1
0
0 0

Change in osmo-gsm-tester[master]: fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs

by pespin

Attention is currently required from: fixeria. pespin has removed a vote from this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/30853 ) Change subject: fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs ...................................................................... Removed Verified-1 by Jenkins Builder (1000002) -- To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/30853 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gsm-tester Gerrit-Branch: master Gerrit-Change-Id: I44797a047639ce816adba626be72fda07f7eb1f8 Gerrit-Change-Number: 30853 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: deleteVote

2 years, 5 months

1
0
0 0

Change in osmo-gsm-tester[master]: fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs

by pespin

Attention is currently required from: fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/30853 ) Change subject: fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs ...................................................................... Patch Set 1: Verified+1 Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/30853 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gsm-tester Gerrit-Branch: master Gerrit-Change-Id: I44797a047639ce816adba626be72fda07f7eb1f8 Gerrit-Change-Number: 30853 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Tue, 03 Jan 2023 22:30:51 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 5 months

1
0
0 0

Change in osmo-gsm-tester[master]: fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/30853 ) Change subject: fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs ...................................................................... fix contrib/jenkins-build-osmocom-bb.sh: build libosmo-gprs Change-Id: I44797a047639ce816adba626be72fda07f7eb1f8 Related: osmocom-bb.git I8e0f826c9b2a886f94624176e34e7d197e93d25f --- M contrib/jenkins-build-osmocom-bb.sh 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester refs/changes/53/30853/1 diff --git a/contrib/jenkins-build-osmocom-bb.sh b/contrib/jenkins-build-osmocom-bb.sh index aa2e778..0c09aee 100755 --- a/contrib/jenkins-build-osmocom-bb.sh +++ b/contrib/jenkins-build-osmocom-bb.sh @@ -8,6 +8,7 @@ FW_RPM_URL="http://download.opensuse.org/repositories/home:/mnhauke:/osmocom:/nightly/o…" build_repo libosmocore --disable-doxygen +build_repo libosmo-gprs build_repo_dir osmocom-bb src/host/virt_phy ${SANITIZE_FLAGS} build_repo_dir osmocom-bb src/host/osmocon ${SANITIZE_FLAGS} build_repo_dir osmocom-bb src/host/layer23 ${SANITIZE_FLAGS} -- To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/30853 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gsm-tester Gerrit-Branch: master Gerrit-Change-Id: I44797a047639ce816adba626be72fda07f7eb1f8 Gerrit-Change-Number: 30853 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 5 months

1
0
0 0

Change in osmocom-bb[master]: fixup: gsmmap: move this utility to 'layer23/src/misc/'

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30852 ) Change subject: fixup: gsmmap: move this utility to 'layer23/src/misc/' ...................................................................... fixup: gsmmap: move this utility to 'layer23/src/misc/' Remove non-existing anymore Makefile target 'gsmmap'. Change-Id: Ia38094cf66ae292b8595b25d87161298fa4312a7 --- M src/Makefile 1 file changed, 1 insertion(+), 16 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/52/30852/1 diff --git a/src/Makefile b/src/Makefile index d92acbc..0835a85 100644 --- a/src/Makefile +++ b/src/Makefile @@ -13,7 +13,7 @@ all: libosmocore-target nofirmware firmware mtk-firmware -nofirmware: layer23 osmocon trxcon gsmmap gprsdecode virtphy +nofirmware: layer23 osmocon trxcon gprsdecode virtphy libosmocore-target: shared/libosmocore/build-target/src/.libs/libosmocore.a @@ -71,19 +71,6 @@ make -C host/trxcon -.PHONY: gsmmap -gsmmap: host/gsmmap/gsmmap - -host/gsmmap/configure: host/gsmmap/configure.ac - cd host/gsmmap && autoreconf -i - -host/gsmmap/Makefile: host/gsmmap/configure - cd host/gsmmap && ./configure $(HOST_CONFARGS) - -host/gsmmap/gsmmap: host/gsmmap/Makefile - make -C host/gsmmap - - .PHONY: layer23 layer23: host/layer23/layer23 @@ -121,7 +108,6 @@ make -C shared/libosmocore/build-target $@ make -C host/layer23 $@ make -C host/osmocon $@ - make -C host/gsmmap $@ make -C host/gprsdecode $@ make -C host/virt_phy $@ make -C host/trxcon $@ @@ -132,7 +118,6 @@ rm -rf shared/libosmocore/build-target make -C host/layer23 $@ make -C host/osmocon $@ - make -C host/gsmmap $@ make -C host/gprsdecode $@ make -C host/virt_phy $@ make -C host/trxcon $@ -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30852 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: Ia38094cf66ae292b8595b25d87161298fa4312a7 Gerrit-Change-Number: 30852 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 5 months

1
0
0 0

Change in osmocom-bb[master]: gsmmap: move this utility to 'layer23/src/misc/'

by fixeria

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30829 ) Change subject: gsmmap: move this utility to 'layer23/src/misc/' ...................................................................... gsmmap: move this utility to 'layer23/src/misc/' What I find weird is that the gsmmap is using files from layer23, and vice-versa cell_log from layer23 is using files from this utility. And somehow they are separate sub-projects. I see no reason why gsmmap must be in its own sub-project. Change-Id: I2bc9c8897f3c7ccf207be0146f7b55fc733a6abb --- M contrib/jenkins.sh D src/host/gsmmap/.gitignore D src/host/gsmmap/Makefile.am D src/host/gsmmap/configure.ac D src/host/gsmmap/geo.c D src/host/gsmmap/geo.h D src/host/gsmmap/git-version-gen M src/host/layer23/.gitignore M src/host/layer23/include/osmocom/bb/misc/Makefile.am R src/host/layer23/include/osmocom/bb/misc/geo.h R src/host/layer23/include/osmocom/bb/misc/locate.h R src/host/layer23/include/osmocom/bb/misc/log.h M src/host/layer23/src/misc/Makefile.am M src/host/layer23/src/misc/cell_log.c M src/host/layer23/src/misc/geo.c R src/host/layer23/src/misc/gsmmap.c R src/host/layer23/src/misc/locate.c R src/host/layer23/src/misc/log.c 18 files changed, 29 insertions(+), 301 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index c65387e..5cc3a67 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -47,7 +47,7 @@ # TODO: make sure 'distcheck' passes also for these # TODO: make sure '--enable-werror' passes also for these -for dir in gsmmap osmocon; do +for dir in osmocon; do cd $base/src/host/$dir autoreconf -fi ./configure @@ -65,7 +65,7 @@ fi # Test 'maintainer-clean' -for dir in gprsdecode layer23 gsmmap osmocon trxcon virt_phy; do +for dir in gprsdecode layer23 osmocon trxcon virt_phy; do cd "$base/src/host/$dir" make maintainer-clean done diff --git a/src/host/gsmmap/.gitignore b/src/host/gsmmap/.gitignore deleted file mode 100644 index 661fd13..0000000 --- a/src/host/gsmmap/.gitignore +++ /dev/null @@ -1,35 +0,0 @@ -# autoreconf by-products -*.in - -aclocal.m4 -autom4te.cache/ -configure -depcomp -install-sh -missing - -# configure by-products -.deps/ -Makefile - -config.status -version.h - -# build by-products -*.o - -gsmmap - -# various -.version -.tarball-version - -# IDA file -*.id* -*.nam -*.til - -# Other test files -*.dump -*.bin -*.log diff --git a/src/host/gsmmap/Makefile.am b/src/host/gsmmap/Makefile.am deleted file mode 100644 index 29be15c..0000000 --- a/src/host/gsmmap/Makefile.am +++ /dev/null @@ -1,17 +0,0 @@ -AUTOMAKE_OPTIONS = foreign dist-bzip2 1.6 - -# versioning magic -BUILT_SOURCES = $(top_srcdir)/.version -$(top_srcdir)/.version: - echo $(VERSION) > $@-t && mv $@-t $@ -dist-hook: - echo $(VERSION) > $(distdir)/.tarball-version - -INCLUDES = $(all_includes) -I../layer23/include -DHOST_BUILD -AM_CFLAGS=-Wall $(LIBOSMOCORE_CFLAGS) $(LIBOSMOGSM_CFLAGS) - -sbin_PROGRAMS = gsmmap - -gsmmap_SOURCES = gsmmap.c geo.c locate.c log.c ../layer23/src/common/sysinfo.c ../layer23/src/common/networks.c ../layer23/src/common/logging.c -gsmmap_LDADD = $(LIBOSMOGSM_LIBS) $(LIBOSMOCORE_LIBS) -lm - diff --git a/src/host/gsmmap/configure.ac b/src/host/gsmmap/configure.ac deleted file mode 100644 index 3a42d4c..0000000 --- a/src/host/gsmmap/configure.ac +++ /dev/null @@ -1,26 +0,0 @@ -dnl Process this file with autoconf to produce a configure script -AC_INIT([gsmmap], - m4_esyscmd([./git-version-gen .tarball-version]), - [baseband-devel(a)lists.osmocom.org]) - -AM_INIT_AUTOMAKE([dist-bzip2]) - -dnl kernel style compile messages -m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])]) - -dnl checks for programs -AC_PROG_MAKE_SET -AC_PROG_CC -AC_PROG_INSTALL - -dnl checks for libraries -PKG_CHECK_MODULES(LIBOSMOCORE, libosmocore) -PKG_CHECK_MODULES(LIBOSMOGSM, libosmogsm) - -dnl checks for header files -AC_HEADER_STDC - -dnl Checks for typedefs, structures and compiler characteristics - -AC_OUTPUT( - Makefile) diff --git a/src/host/gsmmap/geo.c b/src/host/gsmmap/geo.c deleted file mode 100644 index 65633d2..0000000 --- a/src/host/gsmmap/geo.c +++ /dev/null @@ -1,47 +0,0 @@ -#include <math.h> -#include "geo.h" - -void geo2space(double *x, double *y, double *z, double lon, double lat) -{ - *z = sin(lat / 180.0 * PI) * POLE_RADIUS; - *x = sin(lon / 180.0 * PI) * cos(lat / 180.0 * PI) * EQUATOR_RADIUS; - *y = -cos(lon / 180.0 * PI) * cos(lat / 180.0 * PI) * EQUATOR_RADIUS; -} - -void space2geo(double *lon, double *lat, double x, double y, double z) -{ - double r; - - /* bring geoid to 1m radius */ - z = z / POLE_RADIUS; - x = x / EQUATOR_RADIUS; - y = y / EQUATOR_RADIUS; - - /* normalize */ - r = sqrt(x * x + y * y + z * z); - z = z / r; - x = x / r; - y = y / r; - - *lat = asin(z) / PI * 180; - *lon = atan2(x, -y) / PI * 180; -} - -double distinspace(double x1, double y1, double z1, double x2, double y2, - double z2) -{ - double x = x1 - x2; - double y = y1 - y2; - double z = z1 - z2; - - return sqrt(x * x + y * y + z * z); -} - -double distonplane(double x1, double y1, double x2, double y2) -{ - double x = x1 - x2; - double y = y1 - y2; - - return sqrt(x * x + y * y); -} - diff --git a/src/host/gsmmap/geo.h b/src/host/gsmmap/geo.h deleted file mode 100644 index 25e26cb..0000000 --- a/src/host/gsmmap/geo.h +++ /dev/null @@ -1,12 +0,0 @@ -/* WGS 84 */ -#define EQUATOR_RADIUS 6378137.0 -#define POLE_RADIUS 6356752.314 - -#define PI 3.1415926536 - -void geo2space(double *x, double *y, double *z, double lat, double lon); -void space2geo(double *lat, double *lon, double x, double y, double z); -double distinspace(double x1, double y1, double z1, double x2, double y2, - double z2); -double distonplane(double x1, double y1, double x2, double y2); - diff --git a/src/host/gsmmap/git-version-gen b/src/host/gsmmap/git-version-gen deleted file mode 100755 index 652fac6..0000000 --- a/src/host/gsmmap/git-version-gen +++ /dev/null @@ -1,151 +0,0 @@ -#!/bin/sh -# Print a version string. -scriptversion=2010-01-28.01 - -# Copyright (C) 2007-2010 Free Software Foundation, Inc. -# -# This program is free software: you can redistribute it and/or modify -# it under the terms of the GNU General Public License as published by -# the Free Software Foundation; either version 3 of the License, or -# (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with this program. If not, see <http://www.gnu.org/licenses/>. - -# This script is derived from GIT-VERSION-GEN from GIT: http://git.or.cz/. -# It may be run two ways: -# - from a git repository in which the "git describe" command below -# produces useful output (thus requiring at least one signed tag) -# - from a non-git-repo directory containing a .tarball-version file, which -# presumes this script is invoked like "./git-version-gen .tarball-version". - -# In order to use intra-version strings in your project, you will need two -# separate generated version string files: -# -# .tarball-version - present only in a distribution tarball, and not in -# a checked-out repository. Created with contents that were learned at -# the last time autoconf was run, and used by git-version-gen. Must not -# be present in either $(srcdir) or $(builddir) for git-version-gen to -# give accurate answers during normal development with a checked out tree, -# but must be present in a tarball when there is no version control system. -# Therefore, it cannot be used in any dependencies. GNUmakefile has -# hooks to force a reconfigure at distribution time to get the value -# correct, without penalizing normal development with extra reconfigures. -# -# .version - present in a checked-out repository and in a distribution -# tarball. Usable in dependencies, particularly for files that don't -# want to depend on config.h but do want to track version changes. -# Delete this file prior to any autoconf run where you want to rebuild -# files to pick up a version string change; and leave it stale to -# minimize rebuild time after unrelated changes to configure sources. -# -# It is probably wise to add these two files to .gitignore, so that you -# don't accidentally commit either generated file. -# -# Use the following line in your configure.ac, so that $(VERSION) will -# automatically be up-to-date each time configure is run (and note that -# since configure.ac no longer includes a version string, Makefile rules -# should not depend on configure.ac for version updates). -# -# AC_INIT([GNU project], -# m4_esyscmd([build-aux/git-version-gen .tarball-version]), -# [bug-project@example]) -# -# Then use the following lines in your Makefile.am, so that .version -# will be present for dependencies, and so that .tarball-version will -# exist in distribution tarballs. -# -# BUILT_SOURCES = $(top_srcdir)/.version -# $(top_srcdir)/.version: -# echo $(VERSION) > $@-t && mv $@-t $@ -# dist-hook: -# echo $(VERSION) > $(distdir)/.tarball-version - -case $# in - 1) ;; - *) echo 1>&2 "Usage: $0 \$srcdir/.tarball-version"; exit 1;; -esac - -tarball_version_file=$1 -nl=' -' - -# First see if there is a tarball-only version file. -# then try "git describe", then default. -if test -f $tarball_version_file -then - v=`cat $tarball_version_file` || exit 1 - case $v in - *$nl*) v= ;; # reject multi-line output - [0-9]*) ;; - *) v= ;; - esac - test -z "$v" \ - && echo "$0: WARNING: $tarball_version_file seems to be damaged" 1>&2 -fi - -if test -n "$v" -then - : # use $v -elif - v=`git describe --abbrev=4 --match='osmocon_v*' HEAD 2>/dev/null \ - || git describe --abbrev=4 HEAD 2>/dev/null` \ - && case $v in - osmocon_[0-9]*) ;; - osmocon_v[0-9]*) ;; - *) (exit 1) ;; - esac -then - # Is this a new git that lists number of commits since the last - # tag or the previous older version that did not? - # Newer: v6.10-77-g0f8faeb - # Older: v6.10-g0f8faeb - case $v in - *-*-*) : git describe is okay three part flavor ;; - *-*) - : git describe is older two part flavor - # Recreate the number of commits and rewrite such that the - # result is the same as if we were using the newer version - # of git describe. - vtag=`echo "$v" | sed 's/-.*//'` - numcommits=`git rev-list "$vtag"..HEAD | wc -l` - v=`echo "$v" | sed "s/$.*$-$.*$/\1-$numcommits-\2/"`; - ;; - esac - - # Change the first '-' to a '.', so version-comparing tools work properly. - # Remove the "g" in git describe's output string, to save a byte. - v=`echo "$v" | sed 's/-/./;s/$.*$-g/\1-/;s/^osmocon_//'`; -else - v="UNKNOWN" -fi - -v=`echo "$v" |sed 's/^v//'` - -# Don't declare a version "dirty" merely because a time stamp has changed. -git status > /dev/null 2>&1 - -dirty=`sh -c 'git diff-index --name-only HEAD' 2>/dev/null` || dirty= -case "$dirty" in - '') ;; - *) # Append the suffix only if there isn't one already. - case $v in - *-dirty) ;; - *) v="$v-dirty" ;; - esac ;; -esac - -# Omit the trailing newline, so that m4_esyscmd can use the result directly. -echo "$v" | tr -d '\012' - -# Local variables: -# eval: (add-hook 'write-file-hooks 'time-stamp) -# time-stamp-start: "scriptversion=" -# time-stamp-format: "%:y-%02m-%02d.%02H" -# time-stamp-end: "$" -# End: diff --git a/src/host/layer23/.gitignore b/src/host/layer23/.gitignore index 301e2c0..333c28c 100644 --- a/src/host/layer23/.gitignore +++ b/src/host/layer23/.gitignore @@ -33,5 +33,6 @@ src/misc/cbch_sniff src/misc/ccch_scan src/misc/layer23 +src/misc/gsmmap src/mobile/mobile src/modem/modem diff --git a/src/host/layer23/include/osmocom/bb/misc/Makefile.am b/src/host/layer23/include/osmocom/bb/misc/Makefile.am index 5976090..14d3af9 100644 --- a/src/host/layer23/include/osmocom/bb/misc/Makefile.am +++ b/src/host/layer23/include/osmocom/bb/misc/Makefile.am @@ -1 +1,8 @@ -noinst_HEADERS = layer3.h rslms.h cell_log.h +noinst_HEADERS = \ + cell_log.h \ + geo.h \ + layer3.h \ + locate.h \ + log.h \ + rslms.h \ + $(NULL) diff --git a/src/host/layer23/src/misc/geo.h b/src/host/layer23/include/osmocom/bb/misc/geo.h similarity index 100% rename from src/host/layer23/src/misc/geo.h rename to src/host/layer23/include/osmocom/bb/misc/geo.h diff --git a/src/host/gsmmap/locate.h b/src/host/layer23/include/osmocom/bb/misc/locate.h similarity index 100% rename from src/host/gsmmap/locate.h rename to src/host/layer23/include/osmocom/bb/misc/locate.h diff --git a/src/host/gsmmap/log.h b/src/host/layer23/include/osmocom/bb/misc/log.h similarity index 100% rename from src/host/gsmmap/log.h rename to src/host/layer23/include/osmocom/bb/misc/log.h diff --git a/src/host/layer23/src/misc/Makefile.am b/src/host/layer23/src/misc/Makefile.am index 230c04f..869fe7b 100644 --- a/src/host/layer23/src/misc/Makefile.am +++ b/src/host/layer23/src/misc/Makefile.am @@ -26,11 +26,11 @@ echo_test \ cell_log \ cbch_sniff \ + gsmmap \ $(NULL) noinst_HEADERS = \ bcch_scan.h \ - geo.h \ $(NULL) bcch_scan_SOURCES = \ @@ -62,3 +62,11 @@ $(top_srcdir)/src/common/main.c \ app_cbch_sniff.c \ $(NULL) + +gsmmap_LDADD = $(LDADD) -lm +gsmmap_SOURCES = \ + gsmmap.c \ + geo.c \ + locate.c \ + log.c \ + $(NULL) diff --git a/src/host/layer23/src/misc/cell_log.c b/src/host/layer23/src/misc/cell_log.c index f01b87e..dc98383 100644 --- a/src/host/layer23/src/misc/cell_log.c +++ b/src/host/layer23/src/misc/cell_log.c @@ -39,7 +39,7 @@ #include <osmocom/bb/common/networks.h> #include <osmocom/bb/common/gps.h> #include <osmocom/bb/misc/cell_log.h> -#include "../../../gsmmap/geo.h" +#include <osmocom/bb/misc/geo.h> #define READ_WAIT 2, 0 #define RACH_WAIT 0, 900000 diff --git a/src/host/layer23/src/misc/geo.c b/src/host/layer23/src/misc/geo.c index 65633d2..e02a239 100644 --- a/src/host/layer23/src/misc/geo.c +++ b/src/host/layer23/src/misc/geo.c @@ -1,5 +1,6 @@ #include <math.h> -#include "geo.h" + +#include <osmocom/bb/misc/geo.h> void geo2space(double *x, double *y, double *z, double lon, double lat) { diff --git a/src/host/gsmmap/gsmmap.c b/src/host/layer23/src/misc/gsmmap.c similarity index 99% rename from src/host/gsmmap/gsmmap.c rename to src/host/layer23/src/misc/gsmmap.c index 33d0638..d98f230 100644 --- a/src/host/gsmmap/gsmmap.c +++ b/src/host/layer23/src/misc/gsmmap.c @@ -30,9 +30,9 @@ #include <osmocom/bb/common/networks.h> #include <osmocom/bb/common/logging.h> -#include "log.h" -#include "geo.h" -#include "locate.h" +#include <osmocom/bb/misc/log.h> +#include <osmocom/bb/misc/geo.h> +#include <osmocom/bb/misc/locate.h> /* * structure of power and cell infos diff --git a/src/host/gsmmap/locate.c b/src/host/layer23/src/misc/locate.c similarity index 98% rename from src/host/gsmmap/locate.c rename to src/host/layer23/src/misc/locate.c index ed0ac93..eb37285 100644 --- a/src/host/gsmmap/locate.c +++ b/src/host/layer23/src/misc/locate.c @@ -5,8 +5,8 @@ #include <errno.h> #include <math.h> -#include "geo.h" -#include "locate.h" +#include <osmocom/bb/misc/geo.h> +#include <osmocom/bb/misc/locate.h> #define CIRCLE_PROBE 30.0 #define FINETUNE_RADIUS 5.0 diff --git a/src/host/gsmmap/log.c b/src/host/layer23/src/misc/log.c similarity index 99% rename from src/host/gsmmap/log.c rename to src/host/layer23/src/misc/log.c index 97bef49..bd8de7f 100644 --- a/src/host/gsmmap/log.c +++ b/src/host/layer23/src/misc/log.c @@ -20,8 +20,7 @@ #include <stdlib.h> #include <osmocom/bb/common/osmocom_data.h> - -#include "log.h" +#include <osmocom/bb/misc/log.h> extern struct power power; extern struct sysinfo sysinfo; -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30829 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I2bc9c8897f3c7ccf207be0146f7b55fc733a6abb Gerrit-Change-Number: 30829 Gerrit-PatchSet: 4 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 5 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: bsc: TC_assignment_emerg_setup_allow(_imei): Test full voice establis...

by pespin

Attention is currently required from: fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30847 ) Change subject: bsc: TC_assignment_emerg_setup_allow(_imei): Test full voice establishment ...................................................................... Patch Set 1: (1 comment) Commit Message: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30847/comment/ff5dc612_aff0… PS1, Line 9: data plane setup > What do you mean by "data plane"? voice, I think it's clear in a call the data plane is the voice/RTP and in packet switch the data plane is the IP data sent over the tunnel. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30847 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I9f39f861398669a8eb1da242595de584725e5b83 Gerrit-Change-Number: 30847 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Tue, 03 Jan 2023 17:30:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: comment

2 years, 5 months

1
0
0 0

Change in osmo-bsc[master]: bsc_compl_l3(): Update documentation regarding no bsc_subscr in conn

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/30851 ) Change subject: bsc_compl_l3(): Update documentation regarding no bsc_subscr in conn ...................................................................... bsc_compl_l3(): Update documentation regarding no bsc_subscr in conn TTCN3 have been improved in osmo-ttcn3-hacks.git Change-Id I6e99ac39f32c9a981420b73f8d7d1568d2fa1c54 to use valid l3 data. Related: SYS#6280 Change-Id: I4918f1741d465abf8b06a9c65199a44b09778299 --- M src/osmo-bsc/gsm_08_08.c 1 file changed, 6 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/51/30851/1 diff --git a/src/osmo-bsc/gsm_08_08.c b/src/osmo-bsc/gsm_08_08.c index 3e0683c..a2bab4b 100644 --- a/src/osmo-bsc/gsm_08_08.c +++ b/src/osmo-bsc/gsm_08_08.c @@ -389,11 +389,10 @@ if (osmo_mobile_identity_decode_from_l3(&mi, msg, false)) { LOG_COMPL_L3(pdisc, mtype, LOGL_ERROR, "Cannot extract Mobile Identity: %s\n", msgb_hexdump_c(OTC_SELECT, msg)); - /* Likely this is an invalid Complete Layer 3 message that deserves to be rejected. However, the current - * state of our ttcn3 tests does send invalid Layer 3 Info in some tests and expects osmo-bsc to not - * care about that. So, changing the behavior to rejecting on missing MI causes test failure and, if at - * all, should happen in a separate patch. - * See e.g. BSC_Tests.TC_chan_rel_rll_rel_ind: "dt := * f_est_dchan('23'O, 23, '00010203040506'O);" + /* Likely this is an invalid Complete Layer 3 message that deserves to be rejected. However, the BSC is + * not expected to look at this layer so it's duty of the MSC to reject it. + * Hence, keep on going with the conn without an assigned bsc_subscr, forwarding the L3 to the MSC and + * letting it take decision on the matter. */ } else { bsub = bsc_subscr_find_or_create_by_mi(bsc_gsmnet->bsc_subscribers, &mi, __func__); @@ -440,6 +439,8 @@ paged_from_msc = NULL; paging_reasons = BSC_PAGING_NONE; if (pdisc == GSM48_PDISC_RR && mtype == GSM48_MT_RR_PAG_RESP) { + /* It only makes sense to attempt to find a pending paging request if the subscriber from the + * Paging Response can be identified (bsub != NULL). */ if (conn->bsub) paging_request_stop(&paged_from_msc, &paging_reasons, bts, conn->bsub); if (!paged_from_msc) { -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/30851 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I4918f1741d465abf8b06a9c65199a44b09778299 Gerrit-Change-Number: 30851 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 5 months

1
0
0 0

2025

2024

2023

2022

gerrit-log January 2023