Attention is currently required from: Hoernchen, laforge, pespin, fixeria.
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 (#5).
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,875 insertions(+), 32 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/21/32021/5
--
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: 5
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: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: Hoernchen, laforge, pespin, fixeria.
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 (#4).
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,860 insertions(+), 32 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/21/32021/4
--
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: 4
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: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: Hoernchen, laforge, fixeria.
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 (#3).
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,856 insertions(+), 32 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/21/32021/3
--
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: 3
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: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: Hoernchen, laforge, fixeria.
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 (#2).
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,856 insertions(+), 32 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/21/32021/2
--
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: 2
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: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: Hoernchen, fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/32022 )
Change subject: layer23: modem: Test GMM layer through VTY
......................................................................
Patch Set 1:
(4 comments)
File src/host/layer23/src/modem/gmm.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/32022/comment/3079976d_7a585119
PS1, Line 92: rc = 1;
> Just do `return 1` straight away, `rc` is not really needed?
It's still handy to check returned value while debugging. I see no real problem with this.
File src/host/layer23/src/modem/llc.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/32022/comment/2f199d27_3409bbe9
PS1, Line 62: rc = osmo_gprs_gmm_prim_llc_lower_up(llc_prim);
> Same here, you're assigning `rc` but then unconditionally overwriting it?
Good for debugging, so you can print rc here.
File src/host/layer23/src/modem/vty.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/32022/comment/10eb59d5_42a4360e
PS1, Line 159: LLC_CMDG_DESC
> copy-paste, you should add `GMM_CMDG_DESC`.
Ack
https://gerrit.osmocom.org/c/osmocom-bb/+/32022/comment/04d91535_effa0781
PS1, Line 161: "Hard-coded GMM Attach Request (SAPI=GMM, TLLI=0xe1c5d364)\n")
> Something is wrong here, what part of the command this docstring corresponds to?
Ack
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/32022
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Iba0663075468670a29aceafe5196cae3cab050eb
Gerrit-Change-Number: 32022
Gerrit-PatchSet: 1
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: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 23 Mar 2023 10:37:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment