OpenBSC

openbsc@lists.osmocom.org

4315 discussions

18 Apr '16

Hi. How to properly use range_enc_arfcns() and range_enc_range*() from arfcn_range_decode.h? There're not many comments around the code. From function signature it seems like range_enc_arfcns() will put results into w parameter but in system_information.c after that there's also call to one of range_enc_range*(). I've tried to feed output to wireshark but it decode some unexpected stuff. So, if I have a sequence of ints which I'd like to run through range512 for example - ho do I get the output? -- Max Suraev <msuraev(a)sysmocom.de> http://www.sysmocom.de/ ======================================================================= * sysmocom - systems for mobile communications GmbH * Alt-Moabit 93 * 10559 Berlin, Germany * Sitz / Registered office: Berlin, HRB 134158 B * Geschaeftsfuehrer / Managing Directors: Holger Freyther, Harald Welte

2 3

[PATCH] Refactor range encoding
by msuraev＠sysmocom.de 18 Apr '16

18 Apr '16

From: Max <msuraev(a)sysmocom.de> Convert functions always returning the same constant into void type. Remove useless check of return value. --- openbsc/include/openbsc/arfcn_range_encode.h | 10 +++++----- openbsc/src/libbsc/arfcn_range_encode.c | 19 +++++++------------ openbsc/src/libbsc/system_information.c | 22 ++++++++++------------ openbsc/tests/gsm0408/gsm0408_test.c | 24 +++++++----------------- 4 files changed, 29 insertions(+), 46 deletions(-) diff --git a/openbsc/include/openbsc/arfcn_range_encode.h b/openbsc/include/openbsc/arfcn_range_encode.h index bd85d6a..7992c70 100644 --- a/openbsc/include/openbsc/arfcn_range_encode.h +++ b/openbsc/include/openbsc/arfcn_range_encode.h @@ -14,13 +14,13 @@ enum { #define RANGE_ENC_MAX_ARFCNS 29 int range_enc_determine_range(const int *arfcns, int size, int *f0_out); -int range_enc_arfcns(const int rng, const int *arfcns, int sze, int *out, int idx); +void range_enc_arfcns(const int rng, const int *arfcns, int sze, int *out, int idx); int range_enc_find_index(const int rng, const int *arfcns, int size); int range_enc_filter_arfcns(int *arfcns, const int sze, const int f0, int *f0_included); -int range_enc_range128(uint8_t *chan_list, int f0, int *w); -int range_enc_range256(uint8_t *chan_list, int f0, int *w); -int range_enc_range512(uint8_t *chan_list, int f0, int *w); -int range_enc_range1024(uint8_t *chan_list, int f0, int f0_incl, int *w); +void range_enc_range128(uint8_t *chan_list, int f0, int *w); +void range_enc_range256(uint8_t *chan_list, int f0, int *w); +void range_enc_range512(uint8_t *chan_list, int f0, int *w); +void range_enc_range1024(uint8_t *chan_list, int f0, int f0_incl, int *w); #endif diff --git a/openbsc/src/libbsc/arfcn_range_encode.c b/openbsc/src/libbsc/arfcn_range_encode.c index e67bf0a..31fb8b8 100644 --- a/openbsc/src/libbsc/arfcn_range_encode.c +++ b/openbsc/src/libbsc/arfcn_range_encode.c @@ -79,7 +79,7 @@ int range_enc_find_index(const int range, const int *freqs, const int size) * \param size The size of the list of ARFCNs * \param out Place to store the W(i) output. */ -int range_enc_arfcns(const int range, +void range_enc_arfcns(const int range, const int *arfcns, int size, int *out, const int index) { @@ -100,11 +100,11 @@ int range_enc_arfcns(const int range, /* Test the two recursion anchors and stop processing */ if (size == 0) - return 0; + return; if (size == 1) { out[index] = 1 + arfcns[0]; - return 0; + return; } /* Now do the processing */ @@ -131,7 +131,6 @@ int range_enc_arfcns(const int range, out, index + greatest_power_of_2_lesser_or_equal_to(index + 1)); range_enc_arfcns((range -1 ) / 2, arfcns_right, r_size, out, index + (2 * greatest_power_of_2_lesser_or_equal_to(index + 1))); - return 0; } /* @@ -265,39 +264,35 @@ static void write_all_wn(uint8_t *chan_list, int bit_offs, } } -int range_enc_range128(uint8_t *chan_list, int f0, int *w) +void range_enc_range128(uint8_t *chan_list, int f0, int *w) { chan_list[0] = 0x8C; write_orig_arfcn(chan_list, f0); write_all_wn(&chan_list[2], 1, w, 28, 7); - return 0; } -int range_enc_range256(uint8_t *chan_list, int f0, int *w) +void range_enc_range256(uint8_t *chan_list, int f0, int *w) { chan_list[0] = 0x8A; write_orig_arfcn(chan_list, f0); write_all_wn(&chan_list[2], 1, w, 21, 8); - return 0; } -int range_enc_range512(uint8_t *chan_list, int f0, int *w) +void range_enc_range512(uint8_t *chan_list, int f0, int *w) { chan_list[0] = 0x88; write_orig_arfcn(chan_list, f0); write_all_wn(&chan_list[2], 1, w, 17, 9); - return 0; } -int range_enc_range1024(uint8_t *chan_list, int f0, int f0_included, int *w) +void range_enc_range1024(uint8_t *chan_list, int f0, int f0_included, int *w) { chan_list[0] = 0x80 | (f0_included << 2); write_all_wn(&chan_list[0], 6, w, 16, 10); - return 0; } int range_enc_filter_arfcns(int *arfcns, diff --git a/openbsc/src/libbsc/system_information.c b/openbsc/src/libbsc/system_information.c index 43a492a..da36652 100644 --- a/openbsc/src/libbsc/system_information.c +++ b/openbsc/src/libbsc/system_information.c @@ -173,7 +173,7 @@ static inline int enc_freq_lst_range(uint8_t *chan_list, int w[RANGE_ENC_MAX_ARFCNS]; int f0_included = 0; int arfcns_used = 0; - int i, rc, range, f0; + int i, range, f0; /* * Select ARFCNs according to the rules in bitvec2freq_list @@ -202,24 +202,22 @@ static inline int enc_freq_lst_range(uint8_t *chan_list, f0, &f0_included); memset(w, 0, sizeof(w)); - rc = range_enc_arfcns(range, arfcns, arfcns_used, w, 0); - if (rc != 0) - return -3; + range_enc_arfcns(range, arfcns, arfcns_used, w, 0); /* Select the range and the amount of bits needed */ switch (range) { case ARFCN_RANGE_128: - return range_enc_range128(chan_list, f0, w); - break; + range_enc_range128(chan_list, f0, w); + return 0; case ARFCN_RANGE_256: - return range_enc_range256(chan_list, f0, w); - break; + range_enc_range256(chan_list, f0, w); + return 0; case ARFCN_RANGE_512: - return range_enc_range512(chan_list, f0, w); - break; + range_enc_range512(chan_list, f0, w); + return 0; case ARFCN_RANGE_1024: - return range_enc_range1024(chan_list, f0, f0_included, w); - break; + range_enc_range1024(chan_list, f0, f0_included, w); + return 0; default: return -4; }; diff --git a/openbsc/tests/gsm0408/gsm0408_test.c b/openbsc/tests/gsm0408/gsm0408_test.c index d6abce6..f0992f0 100644 --- a/openbsc/tests/gsm0408/gsm0408_test.c +++ b/openbsc/tests/gsm0408/gsm0408_test.c @@ -230,11 +230,7 @@ static int test_single_range_encoding(int range, const int *orig_arfcns, f0, &f0_included); memset(w, 0, sizeof(w)); - rc = range_enc_arfcns(range, arfcns, arfcns_used, w, 0); - if (rc != 0) { - printf("Cannot compute range W(k), rc = %d\n", rc); - return 1; - } + range_enc_arfcns(range, arfcns, arfcns_used, w, 0); if (!silent) fprintf(stderr, "range=%d, arfcns_used=%d, f0=%d, f0_included=%d\n", @@ -243,24 +239,20 @@ static int test_single_range_encoding(int range, const int *orig_arfcns, /* Select the range and the amount of bits needed */ switch (range) { case ARFCN_RANGE_128: - rc = range_enc_range128(chan_list, f0, w); + range_enc_range128(chan_list, f0, w); break; case ARFCN_RANGE_256: - rc = range_enc_range256(chan_list, f0, w); + range_enc_range256(chan_list, f0, w); break; case ARFCN_RANGE_512: - rc = range_enc_range512(chan_list, f0, w); + range_enc_range512(chan_list, f0, w); break; case ARFCN_RANGE_1024: - rc = range_enc_range1024(chan_list, f0, f0_included, w); + range_enc_range1024(chan_list, f0, f0_included, w); break; default: return 1; }; - if (rc != 0) { - printf("Cannot encode range, rc = %d\n", rc); - return 1; - } if (!silent) printf("chan_list = %s\n", @@ -471,8 +463,7 @@ static void test_print_encoding() break; } - rc = range_enc_range512(chan_list, (1 << 9) | 0x96, w); - VERIFY(rc, ==, 0); + range_enc_range512(chan_list, (1 << 9) | 0x96, w); printf("Range512: %s\n", osmo_hexdump(chan_list, ARRAY_SIZE(chan_list))); } @@ -496,8 +487,7 @@ static void test_si_range_helpers() printf("Element is: %d => freqs[i] = %d\n", i, i >= 0 ? freqs3[i] : -1); VERIFY(i, ==, 0); - i = range_enc_arfcns(1023, freqs1, ARRAY_SIZE(freqs1), ws, 0); - VERIFY(i, ==, 0); + range_enc_arfcns(1023, freqs1, ARRAY_SIZE(freqs1), ws, 0); for (i = 0; i < sizeof(freqs1)/sizeof(freqs1[0]); ++i) { printf("w[%d]=%d\n", i, ws[i]); -- 2.8.1

1 0

[PATCH 1/3] Add SI2quater support to SI3
by msuraev＠sysmocom.de 16 Apr '16

16 Apr '16

From: Max <msuraev(a)sysmocom.de> Advertise SI2 quater presence and location (if available) using SI3 according to 3GPP TS 44.018 § 10.5.2.34 --- openbsc/include/openbsc/rest_octets.h | 3 ++- openbsc/src/libbsc/bsc_init.c | 1 + openbsc/src/libbsc/rest_octets.c | 7 +++++++ openbsc/src/libbsc/system_information.c | 8 +++++++- 4 files changed, 17 insertions(+), 2 deletions(-) diff --git a/openbsc/include/openbsc/rest_octets.h b/openbsc/include/openbsc/rest_octets.h index 963b010..9560b14 100644 --- a/openbsc/include/openbsc/rest_octets.h +++ b/openbsc/include/openbsc/rest_octets.h @@ -43,7 +43,8 @@ struct gsm48_si_ro_info { present:1; } scheduling; struct gsm48_si3_gprs_ind gprs_ind; - + /* SI 3 specific */ + uint8_t si2quater_indicator; /* SI 4 specific */ struct gsm48_lsa_params lsa_params; uint16_t cell_id; diff --git a/openbsc/src/libbsc/bsc_init.c b/openbsc/src/libbsc/bsc_init.c index 713109c..fd8dd66 100644 --- a/openbsc/src/libbsc/bsc_init.c +++ b/openbsc/src/libbsc/bsc_init.c @@ -140,6 +140,7 @@ int gsm_bts_trx_set_system_infos(struct gsm_bts_trx *trx) gen_si[n_si++] = SYSINFO_TYPE_2; gen_si[n_si++] = SYSINFO_TYPE_2bis; gen_si[n_si++] = SYSINFO_TYPE_2ter; + gen_si[n_si++] = SYSINFO_TYPE_2quater; gen_si[n_si++] = SYSINFO_TYPE_3; gen_si[n_si++] = SYSINFO_TYPE_4; diff --git a/openbsc/src/libbsc/rest_octets.c b/openbsc/src/libbsc/rest_octets.c index 4545794..fa35f21 100644 --- a/openbsc/src/libbsc/rest_octets.c +++ b/openbsc/src/libbsc/rest_octets.c @@ -129,6 +129,13 @@ int rest_octets_si3(uint8_t *data, const struct gsm48_si_ro_info *si3) /* GPRS Indicator */ append_gprs_ind(&bv, &si3->gprs_ind); + bitvec_set_bit(&bv, H); /* 3G Early Classmark Sending Restriction controlled by early_cm_ctrl above */ + + if (si3->si2quater_indicator) { + bitvec_set_bit(&bv, H); /* indicator struct present */ + bitvec_set_uint(&bv, 0, 1); /* message is sent on BCCH Norm */ + } + bitvec_spare_padding(&bv, (bv.data_len*8)-1); return bv.data_len; } diff --git a/openbsc/src/libbsc/system_information.c b/openbsc/src/libbsc/system_information.c index 1ee9d41..62ee306 100644 --- a/openbsc/src/libbsc/system_information.c +++ b/openbsc/src/libbsc/system_information.c @@ -510,6 +510,7 @@ static struct gsm48_si_ro_info si_info = { .ra_colour = 0, .present = 1, }, + .si2quater_indicator = 0, .lsa_params = { .present = 0, }, @@ -545,7 +546,12 @@ static int generate_si3(uint8_t *output, struct gsm_bts *bts) } else { si_info.si2ter_indicator = 0; } - + if ((bts->si_valid & (1 << SYSINFO_TYPE_2quater))) { + LOGP(DRR, LOGL_INFO, "SI 2quater is included.\n"); + si_info.si2quater_indicator = 1; + } else { + si_info.si2quater_indicator = 0; + } /* SI3 Rest Octets (10.5.2.34), containing CBQ, CELL_RESELECT_OFFSET, TEMPORARY_OFFSET, PENALTY_TIME Power Offset, 2ter Indicator, Early Classmark Sending, -- 2.8.1

2 3

Re: Lack of maintenance for osmo-bts-trx
by Tom Tsou 16 Apr '16

16 Apr '16

Hi Harald, On Mon, Apr 4, 2016 at 8:10 AM, Harald Welte <laforge(a)gnumonks.org> wrote: > Ever since the merge in August 2015, as far as I recall, I never saw a > single message from any osmo-bts-trx user whether > a) the L1SAP merge back then works for them > b) it introduced any regressions I suspect that there were issues since the beginning of the merge. The issue that I saw with master was reported by Sipos Csaba back in Nov 2015. http://lists.osmocom.org/pipermail/openbsc/2015-November/000856.html Unlike Alexander, I was never able to setup a working version with older commits including the one specified in this post. http://lists.osmocom.org/pipermail/openbsc/2015-September/000398.html I guess there is something wrong with the OML setup since trx->mo.nm_state is reporting NM_OPSTATE_NULL during the post-RACH channel allocation, but I am not familiar enough with those procedures to debug more. > While some people think I sometimes write good code, I fail to believe > that all those intrusive changes didn't break anything for OsmoTRX > users. Still, no feedback and no fall-out was reported. I suspect the case was that very few users, if any, had working setups to break or to report regressions. > Like any larger project, particularly one with different hardware > support, we need somebody with essential interest in that hardware to > maintain the respective back-end. Think of device driver maintainers in > the Linux kernel as an example. Agreed. > We need a sub-maintainer for osmo-bts-trx, one who > 1) consistently follows the developments in master and checks for > regressiosn > 2) fixes osmo-bts-trx specific bugs like the fact that GPRS measurment > values are not passed to the PCU, breaking rate/link adaption for > this platform > 3) tests interoperability with OsmoTRX and other OpenBTS transceivers > out three > 4) ensures that important osmo-bts-trx related patches/branches end up > in master, ensuring that master is what people use. Telling users to > use a branch different than master is *wrong* for anything but the > most bleeding edge / experiemental code which is to be merged ASAP. i I can provide a supporting role in the above tasks on behalf of Ettus but we do need a working starting point - that makes number 4 the most critical item. Unfortunately, I am not able to resolve the patch differences (I painfully tried) to make master the main user branch. I agree that directing users to the Fairwaves branch is the wrong approach, but right now it is the only branch for osmo-bts-trx that works. -TT

4 4

[PATCH] Add function to get osmo_earfcn bit size
by msuraev＠sysmocom.de 16 Apr '16

16 Apr '16

From: Max <msuraev(a)sysmocom.de> This function returns the number of bits necessary to pack osmo_earfcn into Repeated E-UTRAN Neighbour Cells IE (3GPP TS 44.018 Table 10.5.2.33b.1). --- include/osmocom/gsm/sysinfo.h | 1 + src/gsm/libosmogsm.map | 1 + src/gsm/sysinfo.c | 24 ++++++++++++++++++++++++ 3 files changed, 26 insertions(+) diff --git a/include/osmocom/gsm/sysinfo.h b/include/osmocom/gsm/sysinfo.h index cb7e10d..c7972f4 100644 --- a/include/osmocom/gsm/sysinfo.h +++ b/include/osmocom/gsm/sysinfo.h @@ -66,6 +66,7 @@ typedef uint8_t sysinfo_buf_t[GSM_MACBLOCK_LEN]; extern const struct value_string osmo_sitype_strs[_MAX_SYSINFO_TYPE]; int osmo_earfcn_add(struct osmo_earfcn_si2q *e, uint16_t arfcn, uint8_t meas_bw); int osmo_earfcn_del(struct osmo_earfcn_si2q *e, uint16_t arfcn); +size_t osmo_earfcn_bit_size(const struct osmo_earfcn_si2q *e); void osmo_earfcn_init(struct osmo_earfcn_si2q *e); uint8_t osmo_sitype2rsl(enum osmo_sysinfo_type si_type); enum osmo_sysinfo_type osmo_rsl2sitype(uint8_t rsl_si); diff --git a/src/gsm/libosmogsm.map b/src/gsm/libosmogsm.map index a3e4e14..6886a6c 100644 --- a/src/gsm/libosmogsm.map +++ b/src/gsm/libosmogsm.map @@ -31,6 +31,7 @@ dbm2rxlev; osmo_earfcn_add; osmo_earfcn_del; +osmo_earfcn_bit_size; osmo_earfcn_init; gprs_cipher_gen_input_i; diff --git a/src/gsm/sysinfo.c b/src/gsm/sysinfo.c index 42e717f..3ec5444 100644 --- a/src/gsm/sysinfo.c +++ b/src/gsm/sysinfo.c @@ -144,6 +144,30 @@ int osmo_earfcn_add(struct osmo_earfcn_si2q *e, uint16_t arfcn, uint8_t meas_bw) return -ENOMEM; } +/*! \brief Return number of bits necessary to represent earfcn struct as + * Repeated E-UTRAN Neighbour Cells IE from 3GPP TS 44.018 Table 10.5.2.33b.1 + * \param[in,out] e earfcn struct + * \returns number of bits + */ +size_t osmo_earfcn_bit_size(const struct osmo_earfcn_si2q *e) +{ + /* 1 stop bit + 5 bits for THRESH_E-UTRAN_high */ + size_t i, bits = 6; + for (i = 0; i < e->length; i++) { + if (e->arfcn[i] != OSMO_EARFCN_INVALID) { + bits += 17; + if (OSMO_EARFCN_MEAS_INVALID == e->meas_bw[i]) + bits++; + else + bits += 4; + } + } + bits += (e->prio_valid) ? 4 : 1; + bits += (e->thresh_lo_valid) ? 6 : 1; + bits += (e->qrxlm_valid) ? 6 : 1; + return bits; +} + /*! \brief Delete arfcn (and corresponding measurement bandwith) from earfcn * struct * \param[in,out] e earfcn struct -- 2.8.1

2 1

[PATCH] logging.c: assert category_token allocation (CID #57871)
by Neels Hofmeyr 16 Apr '16

16 Apr '16

--- src/logging.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/logging.c b/src/logging.c index 35746e9..b96e64a 100644 --- a/src/logging.c +++ b/src/logging.c @@ -190,6 +190,7 @@ void log_parse_category_mask(struct log_target* target, const char *_mask) target->categories[i].enabled = 0; category_token = strtok(mask, ":"); + OSMO_ASSERT(category_token); do { for (i = 0; i < osmo_log_info->num_cat; ++i) { size_t length, cat_length; -- 2.1.4

2 1

[PATCH] sua_test_common.c: use global RANAP SSN definition
by Neels Hofmeyr 16 Apr '16

16 Apr '16

The test 'hardcoded' a local define for RANAP SSN as 142, instead use the proper OSMO_SCCP_SSN_RANAP define from sccp_sap.h --- tests/sigtran/sua_test_common.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/sigtran/sua_test_common.c b/tests/sigtran/sua_test_common.c index aed4899..66fade0 100644 --- a/tests/sigtran/sua_test_common.c +++ b/tests/sigtran/sua_test_common.c @@ -44,8 +44,6 @@ static void sccp_make_addr_pc_ssn(struct osmo_sccp_addr *addr, uint32_t pc, uint addr->pc = pc; } -#define SSN_RANAP 142 - struct osmo_prim_hdr *make_conn_req(uint32_t conn_id) { struct msgb *msg = msgb_alloc(1024, "conn_req"); @@ -56,8 +54,8 @@ struct osmo_prim_hdr *make_conn_req(uint32_t conn_id) OSMO_SCU_PRIM_N_CONNECT, PRIM_OP_REQUEST, msg); /* Set SSN for calling and called addr */ - sccp_make_addr_pc_ssn(&prim->u.connect.called_addr, 2, SSN_RANAP); - sccp_make_addr_pc_ssn(&prim->u.connect.calling_addr, 1, SSN_RANAP); + sccp_make_addr_pc_ssn(&prim->u.connect.called_addr, 2, OSMO_SCCP_SSN_RANAP); + sccp_make_addr_pc_ssn(&prim->u.connect.calling_addr, 1, OSMO_SCCP_SSN_RANAP); prim->u.connect.sccp_class = 2; prim->u.connect.conn_id = conn_id; -- 2.1.4

2 1

[PATCH] Fix RANAP SSN: it's 142, not 143
by Neels Hofmeyr 16 Apr '16

16 Apr '16

143 is actually the SSN for RNSAP. Wireshark displayed a RNSAP message type and malformed packet warning until I fixed this to 142. Now I get the proper RANAP and id-Paging reported. There has been a reallocation for RANAP and RNSAP SSNs, though the old SSN for RANAP is apparently 32 (seen in a pcap from a real 3G network). When I send 32 instead of 142, wireshark also decodes the message as valid RANAP. --- include/osmocom/sigtran/sccp_sap.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/osmocom/sigtran/sccp_sap.h b/include/osmocom/sigtran/sccp_sap.h index 5696b47..15aa840 100644 --- a/include/osmocom/sigtran/sccp_sap.h +++ b/include/osmocom/sigtran/sccp_sap.h @@ -106,7 +106,7 @@ enum osmo_sccp_ssn { OSMO_SCCP_SSN_RES_INTL = 0x0c, OSMO_SCCP_SSN_BISDN = 0x0d, OSMO_SCCP_SSN_TC_TEST = 0x0e, - OSMO_SCCP_SSN_RANAP = 143, + OSMO_SCCP_SSN_RANAP = 142, }; struct osmo_sccp_gt { -- 2.1.4

2 1

[PATCH] compiler warning: unconstify arg passed to OCTET_STRING_noalloc()
by Neels Hofmeyr 15 Apr '16

15 Apr '16

This is after the patch to libasn1c, removing str arg constness. --- src/ranap_msg_factory.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ranap_msg_factory.c b/src/ranap_msg_factory.c index bdae92e..461df1a 100644 --- a/src/ranap_msg_factory.c +++ b/src/ranap_msg_factory.c @@ -181,7 +181,7 @@ struct msgb *ranap_new_msg_dt(uint8_t sapi, const uint8_t *nas, unsigned int nas ies.sapi = RANAP_SAPI_sapi_0; /* Avoid copying + later freeing of OCTET STRING */ - OCTET_STRING_noalloc(&ies.nas_pdu, nas, nas_len); + OCTET_STRING_noalloc(&ies.nas_pdu, (uint8_t*)nas, nas_len); /* ies -> dt */ rc = ranap_encode_directtransferies(&dt, &ies); -- 2.1.4

3 4

Lack of maintenance for osmo-bts-trx
by Harald Welte 15 Apr '16

15 Apr '16

Dear all, I am growing incredibly frustrated at the fact that osmo-bts-trx is effectively unmaintained, despite the fact that there are plenty of users as well as even commercial users / companies (particularly Fairwaves). I'm not pointing Fairwaves out because I am associated with sysmocom. I'm pointing them out as for me as Osmocom project founder and co-maintainer, I sincerely think something is wrong here. And particuarly commercial vendors of BTSs using the Osmocom stack should at the very absolute minimum ensure proper maintenance of their own hardware support in the respective Osmocom projects. They should equally consider contributing to maintenance and developemnt of the 'core' code of those projects, too, in order to fairly share the burden of that effort. I explicitly asked for proper maintenance of osmo-bts-trx first in August 2014 in this post: http://lists.osmocom.org/pipermail/openbsc/2014-August/007657.html Back in 2014 and the first half of 2015 I spent a lot of time in forward-porting and cleaning up the the (fairwaves-commissioned?) l1sap changes, as well as the associated osmo-bts-trx port. Back then, there were several patches in related (several) branches that I could not merge. See my mail (and follow-up mails) from http://lists.osmocom.org/pipermail/openbsc/2015-September/000379.html Ever since the merge in August 2015, as far as I recall, I never saw a single message from any osmo-bts-trx user whether a) the L1SAP merge back then works for them b) it introduced any regressions The same happened after the phy_link/phy_interface merge. While some people think I sometimes write good code, I fail to believe that all those intrusive changes didn't break anything for OsmoTRX users. Still, no feedback and no fall-out was reported. Like any larger project, particularly one with different hardware support, we need somebody with essential interest in that hardware to maintain the respective back-end. Think of device driver maintainers in the Linux kernel as an example. For osmo-bts-sysmo, it is clear that sysmocom has such a vital interest, and that it is the primary hardware on whcih Holger and I are working on. For osmo-bts-octphy, Octasic funds related maintenance work to be done at sysmocom. We need a sub-maintainer for osmo-bts-trx, one who 1) consistently follows the developments in master and checks for regressiosn 2) fixes osmo-bts-trx specific bugs like the fact that GPRS measurment values are not passed to the PCU, breaking rate/link adaption for this platform 3) tests interoperability with OsmoTRX and other OpenBTS transceivers out three 4) ensures that important osmo-bts-trx related patches/branches end up in master, ensuring that master is what people use. Telling users to use a branch different than master is *wrong* for anything but the most bleeding edge / experiemental code which is to be merged ASAP. I thus strongly suggest that the users and proponents of osmo-bts-trx get together and see how they can ensure the proper maintenance of this port. Thanks for your consideration. Regards, Harald -- - Harald Welte <laforge(a)gnumonks.org> http://laforge.gnumonks.org/ ============================================================================ "Privacy in residential applications is a desirable marketing option." (ETSI EN 300 175-7 Ch. A6)

4 4

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

OpenBSC