Attention is currently required from: laforge.
jolly has uploaded a new patch set (#8) to the change originally created by laforge. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32405 )
Change subject: BSSMAP_Templates: Add VBS/VGCS related templates
......................................................................
BSSMAP_Templates: Add VBS/VGCS related templates
As we want to test VBS/VGCS related aspects of BSC and/or MSC soon,
we will need related templates to construct and match related messages
and IEs.
This requires titan.ProtocolModules.BSSMAP updated for modern-day VGCS/VBS
related message definitions as I submitted in
https://gitlab.eclipse.org/eclipse/titan/titan.ProtocolModules.BSSMAP_v11.2…
Change-Id: I949f731de794b22292b01d0ddf9a75a9e7e7e71d
---
M library/BSSMAP_Templates.ttcn
1 file changed, 726 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/05/32405/8
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32405
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: I949f731de794b22292b01d0ddf9a75a9e7e7e71d
Gerrit-Change-Number: 32405
Gerrit-PatchSet: 8
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: jolly, fixeria, pespin.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647 )
Change subject: GSM_RR_Types: Add support for VBS/VGCS related L3 RR messages
......................................................................
Patch Set 2:
(7 comments)
File library/GSM_RR_Types.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/7559ee54_ec9a…
PS2, Line 590: HEX1
> Below in `UplinkAccessIndTV` you're using `uint4_t`. […]
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/a74d8c8c_cae8…
PS2, Line 597: } with { variant "TAG(cksn, tag = 'B'H)" };
> > ... since I see no "tag" nor "cksn" fields in here? […]
The way I did it, the IEI is tied to the message in whihc the IE occurs, and not to the IE itself. In GSM 04.08 and some other protocols there are instances where the same IE has different IEIs depending on where they appear. In this case, the IEI must be "glued" to the containing message, and not to the IE itself.
In this specific instance, I don't know if the CipheringKeySeqNrTV occurs in other places (with or without different IEI). But I don't see anything wrong with it, given that also in the SPEC, the IEIs are listed in the message, and not in the definition of the IE.
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/d35f1736_7b6d…
PS2, Line 650: } with { variant (len) "LENGTHTO(es,spare,uai_rach,priority)" };
> Please add `variant "PRESENCE(iei = '31'O)"`.
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/c894a58f_6b93…
PS2, Line 656: } with { variant "" };
> Missing `variant "PRESENCE(iei = '32'O)"`.
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/4c6da24f_2540…
PS2, Line 663: } with { variant (len) "LENGTHTO(talker_id)" };
> Missing `variant "PRESENCE(iei = '33'O)"`.
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/5baa25bc_2922…
PS2, Line 670: } with { variant "" };
> Missing `variant "PRESENCE(iei = 8)"`.
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/a2aa0f79_e329…
PS2, Line 690:
> ws
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647
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: I79ca7ee2b94bb370cd7162cfd9db436049998041
Gerrit-Change-Number: 32647
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 02 Jun 2023 08:54:59 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32405 )
Change subject: BSSMAP_Templates: Add VBS/VGCS related templates
......................................................................
Patch Set 6:
(1 comment)
File library/BSSMAP_Templates.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32405/comment/d7da2e04_5a14…
PS6, Line 668: if (istemplatekind(cid, "omit")) {
> I think you also need to handle template kind `*` here, otherwise this function is no different from […]
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32405
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: I949f731de794b22292b01d0ddf9a75a9e7e7e71d
Gerrit-Change-Number: 32405
Gerrit-PatchSet: 6
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 02 Jun 2023 08:54:56 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: laforge.
Hello Jenkins Builder, jolly, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32405
to look at the new patch set (#7).
Change subject: BSSMAP_Templates: Add VBS/VGCS related templates
......................................................................
BSSMAP_Templates: Add VBS/VGCS related templates
As we want to test VBS/VGCS related aspects of BSC and/or MSC soon,
we will need related templates to construct and match related messages
and IEs.
This requires titan.ProtocolModules.BSSMAP updated for modern-day VGCS/VBS
related message definitions as I submitted in
https://gitlab.eclipse.org/eclipse/titan/titan.ProtocolModules.BSSMAP_v11.2…
Change-Id: I949f731de794b22292b01d0ddf9a75a9e7e7e71d
---
M library/BSSMAP_Templates.ttcn
1 file changed, 728 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/05/32405/7
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32405
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: I949f731de794b22292b01d0ddf9a75a9e7e7e71d
Gerrit-Change-Number: 32405
Gerrit-PatchSet: 7
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: jolly, laforge, pespin.
Hello Jenkins Builder, jolly, fixeria, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647
to look at the new patch set (#3).
Change subject: GSM_RR_Types: Add support for VBS/VGCS related L3 RR messages
......................................................................
GSM_RR_Types: Add support for VBS/VGCS related L3 RR messages
This adds support for the VBS/VGCS related L3 RR messages to
GsmRrL3Union / GsmRrL3Message. Only those with proper L3 header
and classic "tabular" syntax are supported so far, no CSN.1 messages
with short L2 header for SACCH.
Change-Id: I79ca7ee2b94bb370cd7162cfd9db436049998041
---
M library/GSM_RR_Types.ttcn
1 file changed, 126 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/47/32647/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647
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: I79ca7ee2b94bb370cd7162cfd9db436049998041
Gerrit-Change-Number: 32647
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset