laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/31466 )
Change subject: docs: Document the file-specific commands for ADF.USIM/EF.EST
......................................................................
docs: Document the file-specific commands for ADF.USIM/EF.EST
Change-Id: Iddba9f25ba957f03ca25628a7742fe40fd79c030
---
M docs/shell.rst
1 file changed, 17 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/docs/shell.rst b/docs/shell.rst
index 801d2f4..c1f37af 100644
--- a/docs/shell.rst
+++ b/docs/shell.rst
@@ -555,6 +555,14 @@
~~~~~~~~~~~~~~~~~~~~~~~~
Read + decode all EF.ARR records in flattened, human-friendly form.
+ADF.USIM/EF.EST: est_service_enable
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Enables a single service in EF.EST. Requires service number as argument.
+
+ADF.USIM/EF.EST: est_service_disable
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Disables a single service in EF.EST. Requires service number as argument.
+
EF.IMSI: update_imsi_plmn
~~~~~~~~~~~~~~~~~~~~~~~~~
Change the PLMN part (MCC+MNC) of the IMSI. Requires a single argument consisting of 5/6 digits of
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/31466
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Iddba9f25ba957f03ca25628a7742fe40fd79c030
Gerrit-Change-Number: 31466
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/31464 )
Change subject: docs: Add reference for various commands
......................................................................
docs: Add reference for various commands
A number of more recently introduced commands were not yet listed in the
manual, let's fix that.
Change-Id: I39150f55eecb5d8ff48292dc5cc0f9e16dd4398c
---
M docs/shell.rst
1 file changed, 124 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/docs/shell.rst b/docs/shell.rst
index 2c8da84..801d2f4 100644
--- a/docs/shell.rst
+++ b/docs/shell.rst
@@ -514,6 +514,8 @@
USIM commands
-------------
+These commands are available only while ADF.USIM (or ADF.ISIM, respectively) is selected.
+
authenticate
~~~~~~~~~~~~
.. argparse::
@@ -539,6 +541,106 @@
:func: ADF_USIM.AddlShellCommands.envelope_sms_parser
+File-specific commands
+----------------------
+
+These commands are valid only if the respective file is currently selected. They perform some
+operation that's specific to this file only.
+
+EF.ARR: read_arr_record
+~~~~~~~~~~~~~~~~~~~~~~~
+Read one EF.ARR record in flattened, human-friendly form.
+
+EF.ARR: read_arr_records
+~~~~~~~~~~~~~~~~~~~~~~~~
+Read + decode all EF.ARR records in flattened, human-friendly form.
+
+EF.IMSI: update_imsi_plmn
+~~~~~~~~~~~~~~~~~~~~~~~~~
+Change the PLMN part (MCC+MNC) of the IMSI. Requires a single argument consisting of 5/6 digits of
+concatenated MCC+MNC.
+
+ADF.USIM/EF.UST: ust_service_activate
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Activates a single service in EF.UST. Requires service number as argument.
+
+ADF.USIM/EF.UST: ust_service_deactivate
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Deactivates a single service in EF.UST. Requires service number as argument.
+
+ADF.USIM/EF.UST: ust_service_check
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Check consistency between services of this file and files present/activated.
+Many services determine if one or multiple files shall be present/activated or if they shall be
+absent/deactivated. This performs a consistency check to ensure that no services are activated
+for files that are not - and vice-versa, no files are activated for services that are not. Error
+messages are printed for every inconsistency found.
+
+ADF.ISIM/EF.IST: ist_service_activate
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Activates a single service in EF.IST. Requires service number as argument.
+
+ADF.ISIM/EF.IST: ist_service_deactivate
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Deactivates a single service in EF.UST. Requires service number as argument.
+
+ADF.ISIM/EF.IST: ist_service_check
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Check consistency between services of this file and files present/activated.
+Many services determine if one or multiple files shall be present/activated or if they shall be
+absent/deactivated. This performs a consistency check to ensure that no services are activated
+for files that are not - and vice-versa, no files are activated for services that are not. Error
+messages are printed for every inconsistency found.
+
+
+UICC Administrative commands
+----------------------------
+
+ETSI TS 102 222 specifies a set of *Administrative Commands*, which can
+be used by the card issuer / operator to modify the file system structure
+(delete files, create files) or even to terminate individual files or the
+entire card.
+
+pySim-shell supports those commands, but **use extreme caution**.
+Unless you know exactly what you're doing, it's very easy to render your
+card unusable. You've been warned!
+
+delete_file
+~~~~~~~~~~~
+.. argparse::
+ :module: pySim.ts_102_222
+ :func: Ts102222Commands.delfile_parser
+
+
+terminate_df
+~~~~~~~~~~~~
+.. argparse::
+ :module: pySim.ts_102_222
+ :func: Ts102222Commands.termdf_parser
+
+terminate_ef
+~~~~~~~~~~~~
+.. argparse::
+ :module: pySim.ts_102_222
+ :func: Ts102222Commands.termdf_parser
+
+terminate_card
+~~~~~~~~~~~~~~
+.. argparse::
+ :module: pySim.ts_102_222
+ :func: Ts102222Commands.tcard_parser
+
+create_ef
+~~~~~~~~~
+.. argparse::
+ :module: pySim.ts_102_222
+ :func: Ts102222Commands.create_parser
+
+create_df
+~~~~~~~~~
+.. argparse::
+ :module: pySim.ts_102_222
+ :func: Ts102222Commands.createdf_parser
ARA-M commands
@@ -622,6 +724,16 @@
intended must be manually inserted again using `aram_store_ref_ar_do`
+GlobalPlatform commands
+-----------------------
+
+pySim-shell has only the mots rudimentary support for GlobalPlatform at this point. Please use dedicated
+projects like GlobalPlatformPro meanwhile.
+
+get_data
+~~~~~~~~
+Performs the GET DATA command as specified by GlobalPlatform.
+
cmd2 settable parameters
------------------------
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/31464
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I39150f55eecb5d8ff48292dc5cc0f9e16dd4398c
Gerrit-Change-Number: 31464
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
Attention is currently required from: iedemam.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/31538 )
Change subject: utils: store more fields from meas-feed in db
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
Patchset:
PS1:
ha, i didn't even remember that this tool existed 😊
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/31538
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I509c939524b11a4ee455bcfc3ebee6c5c35b9fba
Gerrit-Change-Number: 31538
Gerrit-PatchSet: 1
Gerrit-Owner: iedemam <michael(a)kapsulate.com>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: Jenkins Builder
Gerrit-Attention: iedemam <michael(a)kapsulate.com>
Gerrit-Comment-Date: Mon, 27 Feb 2023 13:36:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31529 )
Change subject: Interleaving for CSD
......................................................................
Interleaving for CSD
This patch adds the [de]interleaving for CSD (circuit switched data),
nominally for TCH/F 9.6, but the same is also used for TCH/F 4.8,
TCH/H 4.8, TCH/F 2.4 and TCH/F 14.4.
Related: OS#4396, OS#1572
Change-Id: I6b16c2d0d7febf3883da662b2c7fec543335de12
---
M src/coding/gsm0503_interleaving.c
M src/coding/libosmocoding.map
2 files changed, 68 insertions(+), 0 deletions(-)
Approvals:
fixeria: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/coding/gsm0503_interleaving.c b/src/coding/gsm0503_interleaving.c
index 7809e83..cd2f235 100644
--- a/src/coding/gsm0503_interleaving.c
+++ b/src/coding/gsm0503_interleaving.c
@@ -678,4 +678,56 @@
}
}
+/* 3GPP TS 45.003 Section 3.3.4
+ * The coded bits are reordered and interleaved according to the following rule:
+ * i(B,j) = c(n,k) for k = 0,1,...,455
+ * n = 0,1,...,N,N + 1,...
+ * B = B0 +4n + (k mod 19) + (k div 114)
+ * j = (k mod 19) + 19 (k mod 6)
+ *
+ * The result of the interleaving is a distribution of the reordered 114
+ * bit of a given data block, n = N, over 19 blocks, 6 bits equally
+ * distributed in each block, in a diagonal way over consecutive blocks.
+ *
+ * Or in other words the interleaving is a distribution of the encoded,
+ * reordered 456 bits from four given input data blocks, which taken
+ * together give n = N, over 22 bursts, 6 bits equally distributed in
+ * the first and 22 nd bursts, 12 bits distributed in the second and 21
+ * st bursts, 18 bits distributed in the third and 20 th bursts and 24
+ * bits distributed in the other 16 bursts.
+ *
+ * The block of coded data is interleaved "diagonal", where a new block
+ * of coded data starts with every fourth burst and is distributed over
+ * 22 bursts.
+ *
+ * Also used for TCH/F4.8, TCH/H4.8, TCH/F2.4 and TCH/H2.4 and TCH/F14.4 */
+void gsm0503_tch_f96_interleave(const ubit_t *cB, ubit_t *iB)
+{
+ int j, k, B;
+
+ for (k = 0; k < 456; k++) {
+ /* upper bound for B: 4*n + 18 + 4 = 4*n + 22 */
+ B = /* B0 + 4n + */ (k % 19) + (k / 114);
+ /* upper bound for j: 18 + 19*5 = 113 */
+ j = (k % 19) + 19 * (k % 6);
+ /* upper iB index: 4*n+23*114-1 */
+ iB[B * 114 + j] = cB[k];
+ }
+}
+
+void gsm0503_tch_f96_deinterleave(sbit_t *cB, const sbit_t *iB)
+{
+ int j, k, B;
+
+ for (k = 0; k < 456; k++) {
+ /* upper bound for B: 4*n + 18 + 4 = 4*n + 22 */
+ B = /* B0 + 4n + */ (k % 19) + (k / 114);
+ /* upper bound for j: 18 + 19*5 = 113 */
+ j = (k % 19) + 19 * (k % 6);
+ /* upper iB index: 4*n+23*114-1 */
+ cB[k] = iB[B * 114 + j];
+ }
+}
+
+
/*! @} */
diff --git a/src/coding/libosmocoding.map b/src/coding/libosmocoding.map
index b083564..ff6ea80 100644
--- a/src/coding/libosmocoding.map
+++ b/src/coding/libosmocoding.map
@@ -76,6 +76,8 @@
gsm0503_xcch_interleave;
gsm0503_tch_fr_deinterleave;
gsm0503_tch_fr_interleave;
+gsm0503_tch_f96_deinterleave;
+gsm0503_tch_f96_interleave;
gsm0503_tch_hr_deinterleave;
gsm0503_tch_hr_interleave;
gsm0503_mcs1_ul_deinterleave;
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31529
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I6b16c2d0d7febf3883da662b2c7fec543335de12
Gerrit-Change-Number: 31529
Gerrit-PatchSet: 4
Gerrit-Owner: laforge <laforge(a)osmocom.org>
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-CC: msuraev <msuraev(a)sysmocom.de>
Gerrit-MessageType: merged
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/31538 )
Change subject: utils: store more fields from meas-feed in db
......................................................................
Patch Set 1:
(2 comments)
File src/utils/meas_db.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-4069):
https://gerrit.osmocom.org/c/osmo-bsc/+/31538/comment/442289ff_e5e174f8
PS1, Line 149: 0, &mfm->mr.dl);
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-4069):
https://gerrit.osmocom.org/c/osmo-bsc/+/31538/comment/8898e66a_2fdb5199
PS1, Line 149: 0, &mfm->mr.dl);
please, no space before tabs
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/31538
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I509c939524b11a4ee455bcfc3ebee6c5c35b9fba
Gerrit-Change-Number: 31538
Gerrit-PatchSet: 1
Gerrit-Owner: iedemam <michael(a)kapsulate.com>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Mon, 27 Feb 2023 13:32:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/31535 )
Change subject: jobs/gerrit-pipeline-endianness: ignore submodules
......................................................................
jobs/gerrit-pipeline-endianness: ignore submodules
Let the check only run on the main repository, not on any submodules.
This fixes that it would currently fail in osmo-trx: the osmocom-bb
submodule has a copy of libosmocore where the endianness check is known
to be failing (see https://gerrit.osmocom.org/c/osmocom-bb/+/31403).
Change-Id: I795a64a66b4a2e316a99e6b523cc33a9ed364272
---
M jobs/gerrit-pipeline-endianness.yml
1 file changed, 15 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
msuraev: Looks good to me, but someone else must approve
pespin: Looks good to me, but someone else must approve
osmith: Looks good to me, approved
diff --git a/jobs/gerrit-pipeline-endianness.yml b/jobs/gerrit-pipeline-endianness.yml
index 1b0f423..809cce6 100644
--- a/jobs/gerrit-pipeline-endianness.yml
+++ b/jobs/gerrit-pipeline-endianness.yml
@@ -69,8 +69,7 @@
choosing-strategy: gerrit
wipe-workspace: true
skip-tag: true
- submodule:
- recursive: false
+ disable-submodules: true
builders:
- shell: |
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/31535
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I795a64a66b4a2e316a99e6b523cc33a9ed364272
Gerrit-Change-Number: 31535
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/31446 )
Change subject: rlcmac: Use finished UL TBF to request a new one after Final UL ACK/NACK
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/31446
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I4eb873250f5ee7eafb5b6126382ce088091d0447
Gerrit-Change-Number: 31446
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 27 Feb 2023 12:29:57 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/31445 )
Change subject: rlcmac: ul_tbf: Fail if data block CV=0 retransmitted > 4 times
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/31445
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I1a2e1bca296591c34e25faca3f9048e0274ebbd1
Gerrit-Change-Number: 31445
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 27 Feb 2023 12:29:54 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: arehbein.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/31533 )
Change subject: common: Have PCU socket connection use osmo_wqueue
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
BTW, once merged, remember to submit a similar fix in osmo-bsc.git (it also has a pcu_sock). Maybe also check if osmo-pcu.git has a similar issue.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/31533
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia6e61dda4b3cd4bba76e6acb7771d70335062fe1
Gerrit-Change-Number: 31533
Gerrit-PatchSet: 2
Gerrit-Owner: arehbein <arehbein(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: arehbein <arehbein(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 27 Feb 2023 12:25:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: arehbein.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/31534 )
Change subject: common: Make socket queue max. length configurable
......................................................................
Patch Set 4:
(2 comments)
File src/common/vty.c:
https://gerrit.osmocom.org/c/osmo-bts/+/31534/comment/9b7b72c5_2d108c42
PS4, Line 467: vty_out(vty, " pcu-socket-queue-length %d%s", bts->pcu.sock_qlength_max, VTY_NEWLINE);
wqueue
https://gerrit.osmocom.org/c/osmo-bts/+/31534/comment/1d7f9028_4a20ceb7
PS4, Line 985: "pcu-socket-queue-length <1-" OSMO_STRINGIFY_VAL(BTS_CFG_PCU_SOCK_QLENGTH_MAX_MAX) ">",
why is 128 the maximum? Simply use a big number which can fit in sock_qlength_max, like 2^32-1 or alike.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/31534
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Id6ba6e4eadce9ce82ef2407f4e28346e7fe4abfa
Gerrit-Change-Number: 31534
Gerrit-PatchSet: 4
Gerrit-Owner: arehbein <arehbein(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: arehbein <arehbein(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 27 Feb 2023 12:24:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: arehbein.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/31533 )
Change subject: common: Have PCU socket connection use osmo_wqueue
......................................................................
Patch Set 2: Code-Review-1
(7 comments)
File include/osmo-bts/pcuif_proto.h:
https://gerrit.osmocom.org/c/osmo-bts/+/31533/comment/02ac26ec_654e5dfe
PS2, Line 8: #define PCU_SOCK_QLENGTH_MAX_DEFAULT 10
that's application/peer implementation specific, not really part of the shared protocol and hence shouldn't be placed here.
Bear in mind this file is duplicated in 3 different project, so avoid adding protocol-unneeded stuff in here.
File src/common/pcu_sock.c:
https://gerrit.osmocom.org/c/osmo-bts/+/31533/comment/e0ac2792_f4e2a60b
PS2, Line 988: if (osmo_wqueue_enqueue(&state->upqueue, msg) == -ENOSPC) {
Simply check for any error "< 0"
https://gerrit.osmocom.org/c/osmo-bts/+/31533/comment/9145aecb_4bbcd0c5
PS2, Line 989: LOGP(DLGLOBAL, LOGL_NOTICE, "PCU not reacting (more thatn %d messages waiting). Closing connection\n",
max_length is an int? or an unsigned?
https://gerrit.osmocom.org/c/osmo-bts/+/31533/comment/275826ba_ecc9927d
PS2, Line 992: } else {
Usually we do early return on error, then drop this "else" block.
https://gerrit.osmocom.org/c/osmo-bts/+/31533/comment/c58648fd_4899f0a9
PS2, Line 1092: return -1;
In heere probably -EBADF needs to be returned so that the wqueue potentially calls the write_cb() with a closed/freed socket.
https://gerrit.osmocom.org/c/osmo-bts/+/31533/comment/c46f42b9_b556ea96
PS2, Line 1102: osmo_fd_write_disable(&state->upqueue.bfd);
This is wrong. You should disable the bfd after successful write() only if the queue is empty.
https://gerrit.osmocom.org/c/osmo-bts/+/31533/comment/84533b8f_392f737f
PS2, Line 1181: osmo_wqueue_init(&state->upqueue, PCU_SOCK_QLENGTH_MAX_DEFAULT);
I would split this into 2 commits if possible:
1 commit changing the implementation to wqueue but keeping unlimited wqueue size (is that possible by passin 0 here?)
1 commit afterwards changing the queue size to PCU_SOCK_QLENGTH_MAX_DEFAULT.
btw, PCU_SOCK_WQUEUE_LEN it's more clear imho (had to think hat QLENGTH meant).
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/31533
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia6e61dda4b3cd4bba76e6acb7771d70335062fe1
Gerrit-Change-Number: 31533
Gerrit-PatchSet: 2
Gerrit-Owner: arehbein <arehbein(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: arehbein <arehbein(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 27 Feb 2023 12:22:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
fixeria has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31509 )
Change subject: debian/control: fix typo
......................................................................
debian/control: fix typo
Change-Id: Ia9f99016dcf2a9584b59a253d19f391e8f5e5abb
---
M debian/control
1 file changed, 10 insertions(+), 1 deletion(-)
Approvals:
neels: 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/debian/control b/debian/control
index feb988b..8a1aaa2 100644
--- a/debian/control
+++ b/debian/control
@@ -253,7 +253,7 @@
(at least) other programs that are developed in the sphere of Free Software /
Open Source mobile communication.
.
- This package contains the documentation for the libosmoisdnm library.
+ This package contains the documentation for the libosmoisdn library.
Package: libosmovty9
Section: libs
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31509
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia9f99016dcf2a9584b59a253d19f391e8f5e5abb
Gerrit-Change-Number: 31509
Gerrit-PatchSet: 2
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: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
fixeria has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31508 )
Change subject: debian/control: make libosmocore-doc depend on libosmo{ctrl,gb}-doc
......................................................................
debian/control: make libosmocore-doc depend on libosmo{ctrl,gb}-doc
Change-Id: Icd84afcd035bdca9aabb4ea2b91c1227c4786da7
---
M debian/control
1 file changed, 12 insertions(+), 1 deletion(-)
Approvals:
neels: Looks good to me, but someone else must approve
osmith: Looks good to me, approved
msuraev: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/debian/control b/debian/control
index 9632b7e..feb988b 100644
--- a/debian/control
+++ b/debian/control
@@ -143,7 +143,9 @@
libosmocoding-doc,
libosmogsm-doc,
libosmoisdn-doc,
- libosmovty-doc
+ libosmovty-doc,
+ libosmoctrl-doc,
+ libosmogb-doc
Description: Documentation for the Osmo Core library
This is part of the libosmocore "meta"-library. The libosmocore library
contains various utility functions that were originally developed as part of
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31508
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Icd84afcd035bdca9aabb4ea2b91c1227c4786da7
Gerrit-Change-Number: 31508
Gerrit-PatchSet: 2
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: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged