diff options
author | Philipp Maier <pmaier@sysmocom.de> | 2023-08-08 16:12:27 +0200 |
---|---|---|
committer | dexter <pmaier@sysmocom.de> | 2023-08-17 14:12:14 +0000 |
commit | a6708cf6132b199c79150efa400b19c942665f2f (patch) | |
tree | 3ead006e5b26b760e9602f3397de72a0c81c131c | |
parent | 7e9acc3c4ef70c9a72978748d3d569250ced894b (diff) |
PCUIF: rename tlli to msg_id
To confirm downlink IMMEDIATE ASSIGNMENT messages, we use the TLLI as an
identifier and the related record member is also called "tlli".
Unfortunately this is misleading since the message identifier does not
necessarly have to be a TLLI. It is just an implementation detail that
osmo-pcu uses the TLLI as a message identifier.
To make that clear, lets rename the tlli member (and variable and
parameter names where it is passed on) to "msg_id".
(Since this change only renames variables and struct members it will not
break compatibility with other programs that use the PCUIF)
Related: OS#5927
Depends: osmo-pcu.git I4a25039dfe329e68879bc68936e49c4b190625e6
Change-Id: I1db29d5b1920e351c452b798c3260654c2cbe0cb
-rw-r--r-- | bts/BTS_Tests.ttcn | 2 | ||||
-rw-r--r-- | library/PCUIF_CodecPort.ttcn | 4 | ||||
-rw-r--r-- | library/PCUIF_Types.ttcn | 14 | ||||
-rw-r--r-- | pcu/PCUIF_Components.ttcn | 10 |
4 files changed, 15 insertions, 15 deletions
diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn index 1ab0f8a6..e8ae1c0b 100644 --- a/bts/BTS_Tests.ttcn +++ b/bts/BTS_Tests.ttcn @@ -5820,7 +5820,7 @@ testcase TC_pcu_data_req_pch() runs on test_CT { f_pcu_data_req(0, 0, 0, 0, 0, PCU_IF_SAPI_PCH, '313233'O & c_PCU_DATA); } else { var PCUIF_pch_dt pch_dt; - pch_dt.tlli := '01020304'O; + pch_dt.msg_id := '01020304'O; pch_dt.imsi := "00101000000000123"; pch_dt.data := c_PCU_DATA; f_pcu_data_req(0, 0, 0, 0, 0, PCU_IF_SAPI_PCH_DT, enc_PCUIF_pch_dt(pch_dt)); diff --git a/library/PCUIF_CodecPort.ttcn b/library/PCUIF_CodecPort.ttcn index 8824a113..87a5b157 100644 --- a/library/PCUIF_CodecPort.ttcn +++ b/library/PCUIF_CodecPort.ttcn @@ -140,7 +140,7 @@ function f_pcuif_listen(PCUIF_CODEC_PT pt, charstring sock) return integer { } function f_PCUIF_tx_imm_ass_pch(PCUIF_CODEC_PT pt, integer conn_id, octetstring imm_ass, hexstring imsi, - uint8_t bts_nr := 0, boolean wait_for_cnf := true, OCT4 tlli := '01020304'O) return uint32_t { + uint8_t bts_nr := 0, boolean wait_for_cnf := true, OCT4 msg_id := '01020304'O) return uint32_t { var PCUIF_send_data sd; timer T := 3.0; @@ -156,7 +156,7 @@ function f_PCUIF_tx_imm_ass_pch(PCUIF_CODEC_PT pt, integer conn_id, octetstring ts_PCUIF_DATA_REQ(bts_nr, 0, 0, 0, 0, PCU_IF_SAPI_PCH, prefix & imm_ass))); } else { var PCUIF_pch_dt pch_dt; - pch_dt.tlli := tlli; + pch_dt.msg_id := msg_id; pch_dt.imsi := hex2str(imsi); pch_dt.data := imm_ass; pt.send(t_SD_PCUIF(conn_id, ts_PCUIF_DATA_REQ(bts_nr, 0, 0, 0, 0, PCU_IF_SAPI_PCH_DT, enc_PCUIF_pch_dt(pch_dt)))); diff --git a/library/PCUIF_Types.ttcn b/library/PCUIF_Types.ttcn index 94056b91..2e3667a9 100644 --- a/library/PCUIF_Types.ttcn +++ b/library/PCUIF_Types.ttcn @@ -106,7 +106,7 @@ type record PCUIF_data { type record PCUIF_data_cnf_dt { PCUIF_Sapi sapi, - OCT4 tlli, + OCT4 msg_id, uint32_t fn, uint16_t arfcn, uint8_t trx_nr, @@ -315,11 +315,11 @@ type record PCUIF_container { /* Record to send a (confirmed) IMMEDIATE ASSIGNMENT message via PCH. The record is sent by the PCU to the BTS as a * data request (data_req) under SAPI PCU_IF_SAPI_PCH_DT. */ type record PCUIF_pch_dt { - OCT4 tlli, + OCT4 msg_id, charstring imsi length(17), octetstring data length(23) } with { - variant (tlli) "BYTEORDER(last)" + variant (msg_id) "BYTEORDER(last)" variant (imsi) "FIELDLENGTH(17)" variant (data) "FIELDLENGTH(23)" }; @@ -698,14 +698,14 @@ template (value) PCUIF_Message ts_PCUIF_DATA_CNF_DT(template (value) uint8_t bts template (value) uint32_t fn, template (value) uint16_t arfcn, template (value) PCUIF_Sapi sapi, - template (value) OCT4 tlli) := { + template (value) OCT4 msg_id) := { msg_type := PCU_IF_MSG_DATA_CNF_DT, bts_nr := bts_nr, spare := '0000'O, u := { data_cnf_dt := { sapi := sapi, - tlli := tlli, + msg_id := msg_id, fn := fn, arfcn := arfcn, trx_nr := trx_nr, @@ -722,14 +722,14 @@ template PCUIF_Message tr_PCUIF_DATA_CNF_DT(template uint8_t bts_nr := ?, template uint8_t trx_nr := ?, template uint8_t ts_nr := ?, template PCUIF_Sapi sapi := ?, - template OCT4 tlli := ?) := { + template OCT4 msg_id := ?) := { msg_type := PCU_IF_MSG_DATA_CNF_DT, bts_nr := bts_nr, spare := ?, u := { data_cnf_dt := { sapi := sapi, - tlli := tlli, + msg_id := msg_id, fn := ?, arfcn := ?, trx_nr := trx_nr, diff --git a/pcu/PCUIF_Components.ttcn b/pcu/PCUIF_Components.ttcn index 8a1207ab..a6c5bae0 100644 --- a/pcu/PCUIF_Components.ttcn +++ b/pcu/PCUIF_Components.ttcn @@ -140,7 +140,7 @@ type record BTS_PTCCH_Block { type record BTS_CCCH_Block { uint8_t bts_nr, PCUIF_data raw, - OCT4 tlli optional, + OCT4 msg_id optional, charstring imsi optional, GsmRrMessage rr_msg }; @@ -161,11 +161,11 @@ template BTS_PTCCH_Block tr_PCUIF_DATA_PTCCH(template uint8_t bts_nr, template BTS_CCCH_Block tr_PCUIF_DATA_RR(template uint8_t bts_nr, template PCUIF_data raw, template GsmRrMessage rr_msg := ?, - template OCT4 tlli := *, + template OCT4 msg_id := *, template charstring imsi := *) := { bts_nr := bts_nr, raw := raw, - tlli := tlli, + msg_id := msg_id, imsi := imsi, rr_msg := rr_msg }; @@ -550,11 +550,11 @@ runs on RAW_PCU_BTS_CT { if (pcu_msg_rr.raw.sapi == PCU_IF_SAPI_PCH_DT) { pch_dt := dec_PCUIF_pch_dt(pcu_msg_rr.raw.data); - pcu_msg_rr.tlli := pch_dt.tlli; + pcu_msg_rr.msg_id := pch_dt.msg_id; pcu_msg_rr.imsi := pch_dt.imsi; pcu_msg_rr.rr_msg := dec_GsmRrMessage(pch_dt.data); } else { - pcu_msg_rr.tlli := omit; + pcu_msg_rr.msg_id := omit; pcu_msg_rr.imsi := omit; pcu_msg_rr.rr_msg := dec_GsmRrMessage(data); } |