Attention is currently required from: Hoernchen, pespin.
Hello Jenkins Builder, Hoernchen, laforge, fixeria,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/libosmo-gprs/+/32021
to look at the new patch set (#7).
Change subject: gmm: Initial implementation of GPRS Attach
......................................................................
gmm: Initial implementation of GPRS Attach
This patch contains further work on several areas for the GMM layer,
like fixes and improvements in existing primitives, initial FSM
implementation, initial Tx and Rx of some GMM messages, etc.
Related: OS#5501
Change-Id: I212053b3a3f27ef7d63503c3d5ef08453b2d2056
---
M include/osmocom/gprs/gmm/Makefile.am
M include/osmocom/gprs/gmm/gmm.h
A include/osmocom/gprs/gmm/gmm_ms_fsm.h
A include/osmocom/gprs/gmm/gmm_pdu.h
M include/osmocom/gprs/gmm/gmm_prim.h
M include/osmocom/gprs/gmm/gmm_private.h
M include/osmocom/gprs/rlcmac/rlcmac_prim.h
M libosmo-gprs-gmm.pc.in
M src/gmm/Makefile.am
M src/gmm/gmm.c
A src/gmm/gmm_ms_fsm.c
A src/gmm/gmm_pdu.c
M src/gmm/gmm_prim.c
M src/gmm/misc.c
M tests/gmm/Makefile.am
M tests/gmm/gmm_prim_test.c
M tests/gmm/gmm_prim_test.err
M tests/gmm/gmm_prim_test.ok
18 files changed, 1,879 insertions(+), 32 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/21/32021/7
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32021
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I212053b3a3f27ef7d63503c3d5ef08453b2d2056
Gerrit-Change-Number: 32021
Gerrit-PatchSet: 7
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: dexter.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/32029 )
Change subject: bts: bts: Explain Ericsson's interface switch (IS)
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS1:
> asciidoc: ERROR: bts.adoc: line 700: [tabledef-default] total width less than 100%: 45%,25%,25% […]
this still does seem to fail even in the updated patch:
asciidoc: ERROR: bts.adoc: line 700: [tabledef-default] total width less than 100%: 45%,25%,25%
asciidoc: ERROR: bts.adoc: line 716: [tabledef-default] total width less than 100%: 45%,25%,25%
asciidoc: ERROR: bts.adoc: line 751: [tabledef-default] total width less than 100%: 45%,25%,25%
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/32029
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ib6ebd7fdfe9063c0d8cacf53ffd27f6099d9038a
Gerrit-Change-Number: 32029
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 24 Mar 2023 11:08:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment
dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-pcu/+/32047 )
Change subject: doc: overview: put BTS and PCU on the same rank
......................................................................
doc: overview: put BTS and PCU on the same rank
The manual so far describes how OsmoPCU is run in co-location with the
BTS but Figure 1 shows the PCU next to the BSC. Lets put BTS and PCU on
the same rank so that they appear next to each other. This reflects the
reality better.
Change-Id: Ifa170f3956b4629765a88ed62e877a204541d9ea
---
M doc/manuals/chapters/overview.adoc
1 file changed, 15 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/47/32047/1
diff --git a/doc/manuals/chapters/overview.adoc b/doc/manuals/chapters/overview.adoc
index 6ae6f7e..5b831d3 100644
--- a/doc/manuals/chapters/overview.adoc
+++ b/doc/manuals/chapters/overview.adoc
@@ -27,6 +27,7 @@
PCU->SGSN [label="Gb"]
SGSN->GGSN [label="GTP"]
PCU [color=red]
+ { rank=same BTS PCU }
}
----
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/32047
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ifa170f3956b4629765a88ed62e877a204541d9ea
Gerrit-Change-Number: 32047
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newchange
Attention is currently required from: Hoernchen, pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32021 )
Change subject: gmm: Initial implementation of GPRS Attach
......................................................................
Patch Set 6:
(1 comment)
File libosmo-gprs-gmm.pc.in:
https://gerrit.osmocom.org/c/libosmo-gprs/+/32021/comment/d4ab2a02_1f304602
PS1, Line 9: libosmo-gprs-llc
> AFAIU that's done automatically through ${shlibs:Depends}.
Ah, indeed. Too much magic happening automatically there...
What I am still worried about is that you haven't updated the following files:
* `src/gmm/Makefile.am` - add `$(top_builddir)/src/csn1/libosmo-gprs-llc.la` to `libosmo_gprs_gmm_la_LIBADD`.
* `libosmo-gprs-gmm.pc.in` - add `libosmo-gprs-llc` to `Requires`.
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32021
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I212053b3a3f27ef7d63503c3d5ef08453b2d2056
Gerrit-Change-Number: 32021
Gerrit-PatchSet: 6
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 24 Mar 2023 09:56:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment