pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-netif/+/30754 )
Change subject: amr: constify input buffer in osmo_amr_is_oa()
......................................................................
amr: constify input buffer in osmo_amr_is_oa()
Change-Id: I72ef6fc700f628ea158ea888a5f1e9dc8776ab7e
---
M include/osmocom/netif/amr.h
M src/amr.c
2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-netif refs/changes/54/30754/1
diff --git a/include/osmocom/netif/amr.h b/include/osmocom/netif/amr.h
index 47f7f3d..e8f08d4 100644
--- a/include/osmocom/netif/amr.h
+++ b/include/osmocom/netif/amr.h
@@ -159,7 +159,7 @@
size_t osmo_amr_bytes(uint8_t amr_cmr);
size_t osmo_amr_bits(uint8_t amr_ft);
-bool osmo_amr_is_oa(uint8_t *payload, unsigned int payload_len);
+bool osmo_amr_is_oa(const uint8_t *payload, unsigned int payload_len);
int osmo_amr_oa_to_bwe(uint8_t *payload, unsigned int payload_len);
int osmo_amr_bwe_to_oa(uint8_t *payload, unsigned int payload_len,
unsigned int payload_maxlen);
diff --git a/src/amr.c b/src/amr.c
index 66f41c6..b648d54 100644
--- a/src/amr.c
+++ b/src/amr.c
@@ -113,7 +113,7 @@
* \param[inout] payload user provided memory containing the AMR payload.
* \param[in] payload_len overall length of the AMR payload.
* \returns true when the payload is octet aligned. */
-bool osmo_amr_is_oa(uint8_t *payload, unsigned int payload_len)
+bool osmo_amr_is_oa(const uint8_t *payload, unsigned int payload_len)
{
/* NOTE: The distinction between octet-aligned and bandwith-efficient
* mode normally relys on out of band methods that explicitly select
--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/30754
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I72ef6fc700f628ea158ea888a5f1e9dc8776ab7e
Gerrit-Change-Number: 30754
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newchange
Attention is currently required from: osmith.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/30751 )
Change subject: jobs/master-builds: use osmo-iuh/master for asn1c
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/30751
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ic97a23a175b301e6f42f7ca67deb84dcc3ee3f90
Gerrit-Change-Number: 30751
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 16:09:29 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
daniel has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/30689 )
Change subject: shutdown_fsm: Add power_ramp_force() to jump straight to the tgt power
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
I reran the tests and the only ones that failed on my machine (that don't fail in jenkins) are
TC_speech_osmux_tch{h,f}
I believe this is unrelated to this patch and some sort of configuration issue on my part:
20221220150640643 DL1P NOTICE (bts=0,trx=0,ts=1,ss=0) SACCH no SI available (lchan.c:266)
20221220150640643 DRSL INFO (bts=0,trx=0,ts=1,ss=0) Rx RSL IPAC_CRCX (rsl.c:3819)
20221220150640643 DRSL ERROR (bts=0,trx=0,ts=1,ss=0) Rx RSL IPAC XXcx with Osmux CIDgoes against configured Osmux policy 'off' (rsl.c:2823)
20221220150640643 DRSL NOTICE (bts=0,trx=0,ts=1,ss=0) RSL Tx IPAC_CRCX_NACK (rsl.c:2651)
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/30689
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ida1d7bbf094958b9142af306dbf84206729a609e
Gerrit-Change-Number: 30689
Gerrit-PatchSet: 2
Gerrit-Owner: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 16:09:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: dexter.
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/30726 )
Change subject: rsl: allow configuration of HR GSM RTP format via CHANNEL ACTIVATE
......................................................................
Patch Set 2:
(1 comment)
File src/common/rsl.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-2181):
https://gerrit.osmocom.org/c/osmo-bts/+/30726/comment/6b62f5a5_4ad76eaa
PS2, Line 1633: } else if (TLVP_VAL(tp, RSL_IE_OSMO_HR_GSM_RTP_FMT)[0] == 0x01) {
please, no space before tabs
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/30726
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Id53ebebdf987fc59cf4e38bcddb75663fd7bb8fc
Gerrit-Change-Number: 30726
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 15:28:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: dexter.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-bts/+/30725
to look at the new patch set (#2).
Change subject: l1sap: Accept RFC5993 and TS 101.318 HR GSM payload
......................................................................
l1sap: Accept RFC5993 and TS 101.318 HR GSM payload
Unfotunately there are two different RTP formats for HR GSM specified
and it is unclear which should be used with GSM networks. Also esch BTS
model may have a preference for either one or the other format.
Lets set internal flags to determine the preference for each BTS model
and then lets use this information to convert incoming RTP packets into
the prefered format. Doing so we will make sure that always both formats
are accepted.
Change-Id: I17f0b546042fa333780fd2f5c315898ab0df574c
Related: OS#5688
---
M include/osmo-bts/bts.h
M src/common/l1sap.c
M src/osmo-bts-lc15/main.c
M src/osmo-bts-oc2g/main.c
M src/osmo-bts-sysmo/main.c
M src/osmo-bts-trx/main.c
M src/osmo-bts-virtual/main.c
7 files changed, 71 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/25/30725/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/30725
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I17f0b546042fa333780fd2f5c315898ab0df574c
Gerrit-Change-Number: 30725
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: Hoernchen.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/30607 )
Change subject: properly update osmocom-bb submodule, for real this time..
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> I suggest additionally removing the 'branch' from .gitmodules to avoid confusion.
(accidentally marked as resolved, undone)
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/30607
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Icb3ba575cf5235db7e6c03df3d7d45133d535c50
Gerrit-Change-Number: 30607
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 15:13:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment
fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30485 )
Change subject: trxcon: TRXCON_EV_DCH_REL_REQ does not switch to TRXCON_ST_RESET
......................................................................
trxcon: TRXCON_EV_DCH_REL_REQ does not switch to TRXCON_ST_RESET
It's not really clear to me how the L1 is supposed to handle the
L1CTL_DM_REL_REQ message, which maps to the TRXCON_EV_DCH_REL_REQ
in our case. Lookig at the layer23 code I see that it's sent along
with the L1CTL_RESET_REQ (maps to the TRXCON_EV_RESET_SCHED_REQ).
The layer1 firmware does reset some parameters on receipt of the
L1CTL_DM_REL_REQ and some more parameters on the L1CTL_RESET_REQ.
It's clear though that we should not switch to the TRXCON_ST_RESET
on receipt of the L1CTL_DM_REL_REQ. The layer23 application
(e.g. mobile) may send L1CTL_DM_EST_REQ right after that, and we
won't be able to transition from the TRXCON_ST_RESET directly to
the TRXCON_ST_DEDICATED. Such transition is neither implemented
nor permitted by 3GPP TS 44.004, Figure 5.1.
Ideally, according to 3GPP TS 44.004, Figure 5.1, we should have
an additional transient state 'TUNING DCH' in the trxcon_fsm and
switch there on reciept of the L1CTL_DM_REL_REQ. But currently
it's not implemented and adding it would require some effort.
As a temporary solution, let's do not change the current state and
stay in the TRXCON_ST_DEDICATED. This workaround is needed to make
mobile terminated calls work in the mobile app.
Change-Id: I5bbe6ca4cc6299f9faf343822c992a6872a45081
Related: OS#5599
---
M src/host/trxcon/src/trxcon_fsm.c
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
fixeria: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
msuraev: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/src/host/trxcon/src/trxcon_fsm.c b/src/host/trxcon/src/trxcon_fsm.c
index 11b60ac..e3224a9 100644
--- a/src/host/trxcon/src/trxcon_fsm.c
+++ b/src/host/trxcon/src/trxcon_fsm.c
@@ -428,7 +428,7 @@
break;
case TRXCON_EV_DCH_REL_REQ:
l1sched_reset(trxcon->sched, false);
- osmo_fsm_inst_state_chg(fi, TRXCON_ST_RESET, 0, 0);
+ /* TODO: switch to (not implemented) TRXCON_ST_DCH_TUNING? */
break;
case TRXCON_EV_SET_TCH_MODE_REQ:
{
@@ -551,7 +551,7 @@
break;
case TRXCON_EV_DCH_REL_REQ:
l1sched_reset(trxcon->sched, false);
- osmo_fsm_inst_state_chg(fi, TRXCON_ST_RESET, 0, 0);
+ /* TODO: switch to (not implemented) TRXCON_ST_DCH_TUNING? */
break;
default:
OSMO_ASSERT(0);
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30485
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I5bbe6ca4cc6299f9faf343822c992a6872a45081
Gerrit-Change-Number: 30485
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30613 )
Change subject: layer23: fix rx_l1_sim_conf(): msg->l2h is NULL, use msg->l1h
......................................................................
layer23: fix rx_l1_sim_conf(): msg->l2h is NULL, use msg->l1h
The mobile app crashes when using a Calypso phone and specifically
when using its built-in SIM reader. The problem is that msg->l2h
is NULL in rx_l1_sim_conf(), so msg->l1h must be used instead.
Assert failed msgb->l2h /usr/local/include/osmocom/core/msgb.h:162
Change-Id: I7c68a3ad393be5fd0413e00e119a06db59672357
---
M src/host/layer23/src/common/l1ctl.c
1 file changed, 1 insertion(+), 4 deletions(-)
Approvals:
pespin: Looks good to me, but someone else must approve
osmith: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/host/layer23/src/common/l1ctl.c b/src/host/layer23/src/common/l1ctl.c
index 7482788..94079c7 100644
--- a/src/host/layer23/src/common/l1ctl.c
+++ b/src/host/layer23/src/common/l1ctl.c
@@ -736,10 +736,7 @@
/* just forward the SIM response to the SIM handler */
static int rx_l1_sim_conf(struct osmocom_ms *ms, struct msgb *msg)
{
- uint16_t len = msgb_l2len(msg);
- uint8_t *data = msg->data;
-
- LOGP(DL1C, LOGL_INFO, "SIM %s\n", osmo_hexdump(data, len));
+ LOGP(DL1C, LOGL_INFO, "SIM %s\n", msgb_hexdump_l1(msg));
sim_apdu_resp(ms, msg);
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30613
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I7c68a3ad393be5fd0413e00e119a06db59672357
Gerrit-Change-Number: 30613
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30614 )
Change subject: mobile: rework writing BA to file, move to a function
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30614
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Id8bc216c146127d9c9995379c9e56450d328f46d
Gerrit-Change-Number: 30614
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 14:56:12 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737 )
Change subject: library/L1CTL_Types: eliminate warning about missing 'h0h1' field
......................................................................
Patch Set 1:
(1 comment)
File library/L1CTL_Types.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737/comment/4395235e_aaf4…
PS1, Line 653: h0h1 := -,
> As you can see, this template is marked 'private'. […]
Ah I see it's left this way because it is then initialized in child templates, thanks for the explanation!
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737
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: I73c86eea4708823e9c004a5b830793ce705dcb75
Gerrit-Change-Number: 30737
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 14:52:18 +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
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737 )
Change subject: library/L1CTL_Types: eliminate warning about missing 'h0h1' field
......................................................................
Patch Set 1:
(1 comment)
File library/L1CTL_Types.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737/comment/c8ec2d87_272f…
PS1, Line 653: h0h1 := -,
> And when you encode the template then it gives no problem later? Just willing to understand/learn wh […]
As you can see, this template is marked 'private'. It's a common base for both ts_L1CTL_DM_EST_REQ_H0 and ts_L1CTL_DM_EST_REQ_H1, which are built on top of it (note the 'modifies' keyword below). The point is to have common fields initialized in the base template and initialize the 'h0h1' field in the child templates. TITAN warned us that 'h0h1' is missing in the template definition, but this is intentional, so in this patch we explicitly tell it that this field should remain uninitialized. The '-' in TTCN-3 is a bit like the undefined in JavaScript.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737
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: I73c86eea4708823e9c004a5b830793ce705dcb75
Gerrit-Change-Number: 30737
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 14:47:39 +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
Attention is currently required from: fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737 )
Change subject: library/L1CTL_Types: eliminate warning about missing 'h0h1' field
......................................................................
Patch Set 1:
(1 comment)
File library/L1CTL_Types.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737/comment/81949745_8817…
PS1, Line 653: h0h1 := -,
> Explicitly tells the compiler that this field is intentionally unitialized here.
And when you encode the template then it gives no problem later? Just willing to understand/learn what is this used for.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737
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: I73c86eea4708823e9c004a5b830793ce705dcb75
Gerrit-Change-Number: 30737
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 14:22:29 +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: pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737 )
Change subject: library/L1CTL_Types: eliminate warning about missing 'h0h1' field
......................................................................
Patch Set 1:
(1 comment)
File library/L1CTL_Types.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737/comment/6d20d848_f540…
PS1, Line 653: h0h1 := -,
> what does this "-" do here?
Explicitly tells the compiler that this field is intentionally unitialized here.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737
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: I73c86eea4708823e9c004a5b830793ce705dcb75
Gerrit-Change-Number: 30737
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 13:30:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: Hoernchen, fixeria, pespin.
Hello Jenkins Builder, laforge, fixeria, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/30416
to look at the new patch set (#26).
Change subject: ms-trx support
......................................................................
ms-trx support
This is basically a trxcon that includes a transceiver, and can just
be used with existing and future apps supporting the trxcon interface,
i.e. mobile or ccch_scan.
Supports bladerf and uhd.
Currently using hardcoded sched/prios aimed at a setup with working,
reliable usb and reserved cores, for example a raspi 4 (ONLY 4, not 3,
not 2, not any other version)
Additionally builds test tools used for development: osmo-trx-syncthing*
see https://osmocom.org/projects/baseband/wiki/MS-side_GPRS for the
project description and details
Change-Id: I36c65a8c725c4da76dc70006cd96b0a2b6878e84
---
M .gitignore
M .gitmodules
M Makefile.am
M Transceiver52M/Makefile.am
A Transceiver52M/ms/bladerf_specific.h
A Transceiver52M/ms/itrq.h
A Transceiver52M/ms/l1ctl_server.c
A Transceiver52M/ms/l1ctl_server_cb.cpp
A Transceiver52M/ms/logging.c
A Transceiver52M/ms/ms.cpp
A Transceiver52M/ms/ms.h
A Transceiver52M/ms/ms_rx_burst.h
A Transceiver52M/ms/ms_rx_burst_test.cpp
A Transceiver52M/ms/ms_rx_lower.cpp
A Transceiver52M/ms/ms_upper.cpp
A Transceiver52M/ms/ms_upper.h
A Transceiver52M/ms/sch.c
A Transceiver52M/ms/sch.h
A Transceiver52M/ms/uhd_specific.h
M configure.ac
M contrib/jenkins.sh
M osmocom-bb
22 files changed, 3,473 insertions(+), 9 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/16/30416/26
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/30416
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I36c65a8c725c4da76dc70006cd96b0a2b6878e84
Gerrit-Change-Number: 30416
Gerrit-PatchSet: 26
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/30724 )
Change subject: gsm_08_58: add IEI to signal HR GSM RTP format via RSL
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
Patchset:
PS1:
BLocking until detailed explanation is provided on why is this needed.
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/30724
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I843c8a0cbe5898311f3af3294cec7c8bc96c4eb8
Gerrit-Change-Number: 30724
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 13:05:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/30728 )
Change subject: rsl.adoc: add info about RSL_IE_OSMO_HR_GSM_RTP_FMT
......................................................................
Patch Set 2: Code-Review-1
(2 comments)
File doc/manuals/abis/rsl.adoc:
https://gerrit.osmocom.org/c/osmo-bts/+/30728/comment/cc59fb63_c826c405
PS2, Line 1096: On the receiving side (Downlink) osmoBTS will always accept both formats. The format
OsmoBTS
https://gerrit.osmocom.org/c/osmo-bts/+/30728/comment/9c4a0498_d53c51f7
PS2, Line 1106: (Bits 7 - 1 are reserved for future use and must be set to zero.)
so how does the BSC know which format does the BTS support? It just selects one without knowing which ones are supported?
Why does OsmoBSC need to know which format is going to be used?
Can this simply be configured in OsmoBSC or inferred from bts type in osmo-bsc.cfg?
I'm not sure having an extension IE is the way to go here. Let's try to understand better the motivations and requirements first.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/30728
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I1d40f688ad1729efffee846e7f9efd06cf64ce28
Gerrit-Change-Number: 30728
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 13:05:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/30716 )
Change subject: trau_sync: add Ericsson RBS GPRS TRAU synchronization pattern (16kbps)
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/libosmo-abis/+/30716/comment/07477726_dcddf33c
PS1, Line 10: introduces a propritary synchronization pattern. There are slightly different
> Done
I still see "propritary" written there?
--
To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/30716
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I8398f8bbfc51530e37c4328f25155b774394e779
Gerrit-Change-Number: 30716
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 12:58:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737 )
Change subject: library/L1CTL_Types: eliminate warning about missing 'h0h1' field
......................................................................
Patch Set 1:
(1 comment)
File library/L1CTL_Types.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737/comment/b5989ce3_19e6…
PS1, Line 653: h0h1 := -,
what does this "-" do here?
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30737
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: I73c86eea4708823e9c004a5b830793ce705dcb75
Gerrit-Change-Number: 30737
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Dec 2022 12:53:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/30721 )
Change subject: stream: Avoid useless polling if tx_queue becomes empty
......................................................................
stream: Avoid useless polling if tx_queue becomes empty
Before this patch, the WRITE poll flag was being left ON and waited to
be polled again by the kernel in order to disable it.
Let's spate that extra polling cycle which only creates more polling
triggers, context switches, etc.
Change-Id: I1dd2145249a7322ad95e49be588fd472f00734e1
---
M src/stream.c
1 file changed, 10 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
osmith: Looks good to me, but someone else must approve
laforge: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
diff --git a/src/stream.c b/src/stream.c
index 1ca4e9a..503fe3c 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -387,7 +387,12 @@
} else if (ret < msgb_length(msg)) {
LOGP(DLINP, LOGL_ERROR, "short send: %d < exp %u\n", ret, msgb_length(msg));
}
+
msgb_free(msg);
+
+ if (llist_empty(&cli->tx_queue))
+ osmo_fd_write_disable(&cli->ofd);
+
return 0;
}
@@ -1351,8 +1356,11 @@
msgb_free(msg);
- if (llist_empty(&conn->tx_queue) && (conn->flags & OSMO_STREAM_SRV_F_FLUSH_DESTROY))
- osmo_stream_srv_destroy(conn);
+ if (llist_empty(&conn->tx_queue)) {
+ osmo_fd_write_disable(&conn->ofd);
+ if (conn->flags & OSMO_STREAM_SRV_F_FLUSH_DESTROY)
+ osmo_stream_srv_destroy(conn);
+ }
}
static int osmo_stream_srv_cb(struct osmo_fd *ofd, unsigned int what)
--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/30721
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I1dd2145249a7322ad95e49be588fd472f00734e1
Gerrit-Change-Number: 30721
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/30734 )
Change subject: llc: add enum osmo_gprs_llc_{location,prim_sap}
......................................................................
llc: add enum osmo_gprs_llc_{location,prim_sap}
Change-Id: I5606c0b41196f25adbc00cfdce9927daaed9a901
Related: OS#5502
---
M include/osmocom/gprs/llc/llc.h
M include/osmocom/gprs/llc/llc_prim.h
M src/llc/Makefile.am
A src/llc/llc_prim.c
4 files changed, 56 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
msuraev: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
diff --git a/include/osmocom/gprs/llc/llc.h b/include/osmocom/gprs/llc/llc.h
index ff12884..0f0ab83 100644
--- a/include/osmocom/gprs/llc/llc.h
+++ b/include/osmocom/gprs/llc/llc.h
@@ -7,6 +7,13 @@
#include <osmocom/core/msgb.h>
+/* Section 7.1.2 LLC layer service primitives */
+enum osmo_gprs_llc_location {
+ OSMO_GPRS_LLC_LOCATION_UNSET,
+ OSMO_GPRS_LLC_LOCATION_MS,
+ OSMO_GPRS_LLC_LOCATION_SGSN,
+};
+
/* Section 6.2.3 Service Access Point Identifier (SAPI) */
enum osmo_gprs_llc_sapi {
OSMO_GPRS_LLC_SAPI_GMM = 1,
diff --git a/include/osmocom/gprs/llc/llc_prim.h b/include/osmocom/gprs/llc/llc_prim.h
index f58c5fa..dc5f097 100644
--- a/include/osmocom/gprs/llc/llc_prim.h
+++ b/include/osmocom/gprs/llc/llc_prim.h
@@ -6,6 +6,21 @@
#include <stddef.h>
#include <osmocom/core/prim.h>
+#include <osmocom/core/utils.h>
+
+/* Section 7.1.0 */
+enum osmo_gprs_llc_prim_sap {
+ OSMO_GPRS_LLC_SAP_LLGM,
+ OSMO_GPRS_LLC_SAP_LL,
+ OSMO_GPRS_LLC_SAP_GRR,
+ OSMO_GPRS_LLC_SAP_BSSGP,
+};
+
+extern const struct value_string osmo_gprs_llc_prim_sap_names[];
+static inline const char *osmo_gprs_llc_prim_sap_name(enum osmo_gprs_llc_prim_sap val)
+{
+ return get_value_string(osmo_gprs_llc_prim_sap_names, val);
+}
/* TS 04.64 Section 7.1.2 Table 7: LLC layer primitives (GMM/SNDCP/SMS/TOM) */
/* TS 04.65 Section 5.1.2 Table 2: Service primitives used by SNDCP */
diff --git a/src/llc/Makefile.am b/src/llc/Makefile.am
index c25841e..317b855 100644
--- a/src/llc/Makefile.am
+++ b/src/llc/Makefile.am
@@ -26,6 +26,7 @@
libosmo_gprs_llc_la_SOURCES = \
crc24.c \
llc_pdu.c \
+ llc_prim.c \
llc_xid.c \
misc.c \
$(NULL)
diff --git a/src/llc/llc_prim.c b/src/llc/llc_prim.c
new file mode 100644
index 0000000..030494a
--- /dev/null
+++ b/src/llc/llc_prim.c
@@ -0,0 +1,33 @@
+/* GPRS LLC protocol primitive implementation as per 3GPP TS 44.064 */
+/*
+ * (C) 2022 by sysmocom - s.f.m.c. GmbH <info(a)sysmocom.de>
+ *
+ * All Rights Reserved
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as published by
+ * the Free Software Foundation; either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+#include <osmocom/core/utils.h>
+
+#include <osmocom/gprs/llc/llc.h>
+#include <osmocom/gprs/llc/llc_prim.h>
+
+const struct value_string osmo_gprs_llc_prim_sap_names[] = {
+ { OSMO_GPRS_LLC_SAP_LLGM, "LLGM" },
+ { OSMO_GPRS_LLC_SAP_LL, "LL" },
+ { OSMO_GPRS_LLC_SAP_GRR, "GRR" },
+ { OSMO_GPRS_LLC_SAP_BSSGP, "BSSGP" },
+ { 0, NULL }
+};
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/30734
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I5606c0b41196f25adbc00cfdce9927daaed9a901
Gerrit-Change-Number: 30734
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged