Attention is currently required from: jolly, dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32930 )
Change subject: ASCI: Add 3GPP TS 44.068 and 44.069 protocol definitions
......................................................................
Patch Set 1: Code-Review-1
(3 comments)
File include/osmocom/gsm/protocol/gsm_44_068.h:
https://gerrit.osmocom.org/c/libosmocore/+/32930/comment/390285fe_aa935f5f
PS1, Line 19: enum gcc_iei {
missing osmo_ prefix in all definitions, in order to keep everything in the same namespace. (yes, I know old code in libosmocore doesn't do it).
https://gerrit.osmocom.org/c/libosmocore/+/32930/comment/fad2f37d_739a3e61
PS1, Line 123:
> we usually add a an inline function that returns the name using get_value_string for convenience. […]
Ack
File src/gsm/gsm44068.c:
https://gerrit.osmocom.org/c/libosmocore/+/32930/comment/bc86e110_f0b2b020
PS1, Line 22: const struct value_string gcc_msg_type_names[] = {
osmo_* and OSMO_* prefix missing everywhere.
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/32930
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I3554cea47e714c8fca18c3e9c0e6e80695915a90
Gerrit-Change-Number: 32930
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 23 May 2023 15:08:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: dexter.
Hello Jenkins Builder, dexter,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/libosmo-gprs/+/32925
to look at the new patch set (#2).
Change subject: gmm: Implement periodic RAU procedure
......................................................................
gmm: Implement periodic RAU procedure
Related: OS#5501
Change-Id: Ie7cba8a2fe3b97bebc558c957cc1bcbe3cc81dbc
---
M include/osmocom/gprs/gmm/gmm_ms_fsm.h
M include/osmocom/gprs/gmm/gmm_pdu.h
M include/osmocom/gprs/gmm/gmm_private.h
M src/gmm/gmm.c
M src/gmm/gmm_ms_fsm.c
M src/gmm/gmm_pdu.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
10 files changed, 571 insertions(+), 15 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/25/32925/2
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32925
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Ie7cba8a2fe3b97bebc558c957cc1bcbe3cc81dbc
Gerrit-Change-Number: 32925
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: jolly.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32930 )
Change subject: ASCI: Add 3GPP TS 44.068 and 44.069 protocol definitions
......................................................................
Patch Set 1:
(1 comment)
File src/gsm/libosmogsm.map:
https://gerrit.osmocom.org/c/libosmocore/+/32930/comment/ab644932_c2c18b69
PS1, Line 474: gcc_msg_type_names;
Ideally, all new symbols should be prefixed with `osmo_`. The `gsm/protocol` directory is a bit special and this rule is not really followed, but I would still suggest adding `gsm44068_` prefix or something like that to avoid naming conflicts.
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/32930
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I3554cea47e714c8fca18c3e9c0e6e80695915a90
Gerrit-Change-Number: 32930
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Tue, 23 May 2023 14:54:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Hoernchen.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/32945 )
Change subject: ms: block burst q to upper layer
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmo-trx/+/32945/comment/554f4d87_9b6a297f
PS1, Line 8:
this might benefit from some detailed description. It is difficult to understand this without expert knowledge.
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/32945
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Id0aad606a296b2885617013ce6637204357b13d7
Gerrit-Change-Number: 32945
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-CC: dexter <pmaier(a)sysmocom.de>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 23 May 2023 14:50:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32925 )
Change subject: gmm: Implement periodic RAU procedure
......................................................................
Patch Set 1:
(1 comment)
File src/gmm/gmm.c:
https://gerrit.osmocom.org/c/libosmo-gprs/+/32925/comment/569b327b_9129b0b7
PS1, Line 1011: return -EINVAL; /* TODO: what to do on error? */
> Maybe add a comment about what happens at the moment?
That's exactly what the comment is about, I didn't check yet, need sto be checked further in the future 😊
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32925
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Ie7cba8a2fe3b97bebc558c957cc1bcbe3cc81dbc
Gerrit-Change-Number: 32925
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 23 May 2023 14:49:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: pespin.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32925 )
Change subject: gmm: Implement periodic RAU procedure
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File src/gmm/gmm.c:
https://gerrit.osmocom.org/c/libosmo-gprs/+/32925/comment/ccd6c2a4_628eb0b6
PS1, Line 1011: return -EINVAL; /* TODO: what to do on error? */
Maybe add a comment about what happens at the moment?
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32925
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Ie7cba8a2fe3b97bebc558c957cc1bcbe3cc81dbc
Gerrit-Change-Number: 32925
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 23 May 2023 14:42:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment