March 2026 - gerrit-log - lists.osmocom.org

[M] Change in osmo-trx[master]: transceiver: add optional Laurent burst LUT modulation (int16)
by Timur Davydov 03 Mar '26

03 Mar '26

Timur Davydov has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-trx/+/42248?usp=email ) Change subject: transceiver: add optional Laurent burst LUT modulation (int16) ...................................................................... transceiver: add optional Laurent burst LUT modulation (int16) Introduce an optional Burst LUT optimization for Laurent-based burst modulation, enabled via --with-burst-lut. - Add precomputed Laurent LUT tables (float and int16 I/Q variants) - Generate LUT at initialization (prepareBurstLUT) - Use LUT-based modulation for sps=4 when enabled - Keep original modulateBurstLaurent() as fallback - Add configure.ac option and USE_BURST_LUT define This reduces runtime computation by replacing per-burst Laurent processing with memcpy-based LUT lookups. Change-Id: I1715c2d33dc55fe1c7be5c6e7259d378ea5e80b2 --- M Transceiver52M/sigProcLib.cpp M configure.ac 2 files changed, 180 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/48/42248/1 diff --git a/Transceiver52M/sigProcLib.cpp b/Transceiver52M/sigProcLib.cpp index 5fac365..bcea848 100644 --- a/Transceiver52M/sigProcLib.cpp +++ b/Transceiver52M/sigProcLib.cpp @@ -51,6 +51,29 @@ /** Lookup tables for trigonometric approximation */ static float sincTable[TABLESIZE+1]; // add 1 element for wrap around +/** Lookup tables for burst processing */ +#ifdef USE_BURST_LUT +#define BURST_SCALING 32700 /* Scaling factor */ + +static bool burst_lut_prepared = false; + +/** Complex Laurent LUT table (float complex samples). + * Layout: prologue, per-4bit-combination filters and epilogue. Each + * entry contains complex<float> samples used for fast burst modulation. + */ +static complex lut_laurent4_prologue[4 * 4]; /* 4 prologue filters, each with 4 taps */ +static complex lut_laurent4[16 * 4 * 4]; /* 16 possible 4-bit combinations, each with 4 filters of 4 taps */ +static complex lut_laurent4_epilogue[4 * 4]; /* 4 epilogue filters, each with 4 taps */ + +/** Integer LUT table (interleaved I/Q int16 samples). + * These tables contain int16_t I/Q pairs (interleaved) generated from + * the complex LUT and scaled by `BURST_SCALING` for integer output. + */ +static int16_t i16_lut_laurent4_prologue[4 * 4 * 2]; /* 4 prologue filters, each with 4 taps */ +static int16_t i16_lut_laurent4[16 * 4 * 4 * 2]; /* 16 possible 4-bit combinations */ +static int16_t i16_lut_laurent4_epilogue[4 * 4 * 2]; /* 4 epilogue filters, each with 4 taps */ +#endif /* USE_BURST_LUT */ + /** Constants */ static const float M_PI_F = (float)M_PI; @@ -669,6 +692,146 @@ return c0_shaped; } +#ifdef USE_BURST_LUT +/** + * @brief Modulate a burst using the precomputed Laurent approximation LUT. + * @param bits The input bit vector to be modulated (bit values 0/1). + * @param bitlen The length of the input bit vector. + * @param outbuf Output buffer to store interleaved int16 I/Q samples. + * The buffer is filled with int16 pairs (I0,Q0,I1,Q1,...). + * @return The number of complex samples (I/Q pairs) written to `outbuf`. + */ +unsigned modulateBits(uint8_t *bits, unsigned bitlen, int16_t *outbuf) +{ + unsigned off = 0; + unsigned lut_idx = (bits[0] << 1) | (bits[1]); + + memcpy(outbuf + off, i16_lut_laurent4_prologue, sizeof(int16_t) * 12 * 2); + off += 12 * 2; + for (unsigned i = 2; i < bitlen; ++i) { + lut_idx = ((lut_idx << 1) | bits[i]) & 0xf; + memcpy(outbuf + off, &i16_lut_laurent4[2 * (64 * (3 - (i & 0x3)) + 4 * lut_idx)], sizeof(int16_t) * 4 * 2); + off += 4 * 2; + } + for (unsigned i = bitlen; i < bitlen + 1; ++i) { + lut_idx = ((lut_idx << 1) | 0) & 0xf; + memcpy(outbuf + off, &i16_lut_laurent4[2 * (64 * (3 - (i & 0x3)) + 4 * lut_idx)], sizeof(int16_t) * 4 * 2); + off += 4 * 2; + } + memcpy(outbuf + off, i16_lut_laurent4_epilogue, sizeof(int16_t) * 12 * 2); + off += 12 * 2; + + return off >> 1; +} + +/** + * @brief Modulate a burst using the precomputed Laurent approximation LUT. + * @param bits The input bit vector to be modulated. + * @return A newly allocated `signalVector` containing complex<float> + * samples (I/Q) ready for transmission. The vector length is 625 + * samples. The caller is responsible for deleting the returned + * `signalVector`. + */ +static signalVector *modulateBurstLaurentLutInt(const BitVector& bits) +{ + signalVector *burst = new signalVector(625, 16); + signalVector::iterator it = burst->begin(); + + memcpy((void *)it, lut_laurent4_prologue, sizeof(complex) * 12); + it += 12; + + unsigned lut_idx = (bits[0] << 1) | (bits[1]); + for (unsigned i = 2; i < bits.size(); i++) { + lut_idx = ((lut_idx << 1) | bits[i]) & 0xf; + memcpy((void *)it, &lut_laurent4[64 * (3 - (i % 4)) + 4 * lut_idx], sizeof(complex) * 4); + it += 4; + } + for (unsigned i = bits.size(); i < bits.size() + 1; ++i) { + lut_idx = ((lut_idx << 1) | 0) & 0xf; + memcpy((void *)it, &lut_laurent4[64 * (3 - (i % 4)) + 4 * lut_idx], sizeof(complex) * 4); + it += 4; + } + + memcpy((void *)it, lut_laurent4_epilogue, sizeof(complex) * 12); + it += 12; + + return burst; +} + +/** + * @brief Modulate a burst using the precomputed Laurent LUT if prepared, + * otherwise fall back to the original Laurent modulation routine. + * @param bits The input bit vector to be modulated. + * @note If `burst_lut_prepared` is true, `modulateBurstLaurentLutInt` + * is used which returns a `signalVector` of length 625. Otherwise, + * `modulateBurstLaurent` is called instead. + * @return A newly allocated `signalVector` containing complex<float> + * samples (I/Q) ready for transmission. The caller owns the + * returned vector. + */ +static signalVector *modulateBurstLaurentLut(const BitVector &bits) +{ + return (burst_lut_prepared) ? modulateBurstLaurentLutInt(bits) : modulateBurstLaurent(bits); +} + +/** + * @brief Prepare the burst modulation LUT by generating the Laurent approximation for all possible + * bit combinations and storing them in the LUT tables. + * @return 0 on success, -1 on failure. + */ +static int prepareBurstLUT() +{ + complex modtbl[256 * 4 * 4]; + const unsigned B = 8; + for (int b = 0; b < (1 << B); ++b) { + char v[NORMAL_BURST_NBITS + 1]; /* extra byte for null terminator */ + memset(v, '0', NORMAL_BURST_NBITS); + v[NORMAL_BURST_NBITS] = 0; + + for (int c = 0; c < B; ++c) { + if ((b >> (B - c - 1)) & 1) { + v[8 + c] = '1'; + } + } + + BitVector bv(v); + signalVector* sv = modulateBurstLaurent(bv); + + for (unsigned t = 0; t < 16; ++t) { + lut_laurent4_prologue[t] = sv->operator[](t); + i16_lut_laurent4_prologue[2 * t + 0] = lut_laurent4_prologue[t].r * BURST_SCALING; + i16_lut_laurent4_prologue[2 * t + 1] = lut_laurent4_prologue[t].i * BURST_SCALING; + } + + for (unsigned t = 600; t < 616; ++t) { + lut_laurent4_epilogue[t - 600] = sv->operator[](t); + i16_lut_laurent4_epilogue[2 * (t - 600) + 0] = lut_laurent4_epilogue[t - 600].r * BURST_SCALING; + i16_lut_laurent4_epilogue[2 * (t - 600) + 1] = lut_laurent4_epilogue[t - 600].i * BURST_SCALING; + } + + int j_off = (8 + B - 3) * 4; + for (int k = 0; k < 16; ++k) { + modtbl[b * 16 + k] = sv->operator[](j_off + k); + } + } + + // Compose modulation LUT table + for (int q = 0; q < 4; ++q) { // quadrant + for (int i = 0; i < 16; ++i) { + for (int k = 0; k < 4; ++k) { + lut_laurent4[(q * 16 + i) * 4 + k ] = modtbl[(i << q) * 16 + (12 - q * 4) + k]; + + i16_lut_laurent4[2 * ((q * 16 + i) * 4 + k) + 0] = lut_laurent4[(q * 16 + i) * 4 + k].r * BURST_SCALING; + i16_lut_laurent4[2 * ((q * 16 + i) * 4 + k) + 1] = lut_laurent4[(q * 16 + i) * 4 + k].i * BURST_SCALING; + } + } + } + + burst_lut_prepared = true; + return 0; +} +#endif /* USE_BURST_LUT */ + static signalVector *rotateEdgeBurst(const signalVector &symbols, int sps) { signalVector *burst; @@ -973,7 +1136,11 @@ if (emptyPulse) return rotateBurst(wBurst, guardPeriodLength, sps); else if (sps == 4) +#ifdef USE_BURST_LUT + return modulateBurstLaurentLut(wBurst); +#else return modulateBurstLaurent(wBurst); +#endif else return modulateBurstBasic(wBurst, guardPeriodLength, sps); } @@ -2141,6 +2308,10 @@ generateSincTable(); initGMSKRotationTables(); +#ifdef USE_BURST_LUT + prepareBurstLUT(); +#endif + GSMPulse1 = generateGSMPulse(1); GSMPulse4 = generateGSMPulse(4); diff --git a/configure.ac b/configure.ac index c81b089..910f18b 100644 --- a/configure.ac +++ b/configure.ac @@ -185,6 +185,15 @@ [enable x86 SSE support (default)]) ]) +AC_ARG_WITH(burst-lut, [ + AC_HELP_STRING([--with-burst-lut], + [enable Burst LUT optimization]) +]) + +AS_IF([test "x$with_burst_lut" = "xyes"], [ + AC_DEFINE(USE_BURST_LUT, 1, Define to 1 for using Burst LUT) +]) + AS_IF([test "x$with_neon" = "xyes"], [ AC_DEFINE(HAVE_NEON, 1, Support ARM NEON) ]) -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42248?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I1715c2d33dc55fe1c7be5c6e7259d378ea5e80b2 Gerrit-Change-Number: 42248 Gerrit-PatchSet: 1 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com>

1 0

[XL] Change in osmo-trx[master]: feat(usdr): add USDR backend support (osmo-trx-usdr)
by pespin 03 Mar '26

03 Mar '26

Attention is currently required from: Hoernchen, Timur Davydov, fixeria, laforge, osmith. pespin has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email ) Change subject: feat(usdr): add USDR backend support (osmo-trx-usdr) ...................................................................... Patch Set 7: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I5d1d25921514954c4929ae6e7352168b3ceb05df Gerrit-Change-Number: 42198 Gerrit-PatchSet: 7 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: Hoernchen <ewild(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Comment-Date: Mon, 02 Mar 2026 21:31:25 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XL] Change in osmo-trx[master]: feat(usdr): add USDR backend support (osmo-trx-usdr)
by pespin 03 Mar '26

03 Mar '26

Attention is currently required from: Hoernchen, Timur Davydov, fixeria, laforge, osmith. pespin has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email ) Change subject: feat(usdr): add USDR backend support (osmo-trx-usdr) ...................................................................... Patch Set 7: (3 comments) File Transceiver52M/device/usdr/USDRDevice.cpp: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/4ca2944c_fada73b1?usp… : PS4, Line 12: This program is free software: you can redistribute it and/or modify > Added the `*` prefix. Done https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/0e7fd224_a4abd789?usp… : PS4, Line 84: if (res) { > Removed the exceptions, switched to return code checking for consistency with other devices. Done File doc/manuals/chapters/trx-backends.adoc: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/baa2b6c7_00590375?usp… : PS3, Line 86: where multiple small SDRs are combined to present one or more TRX instances > You’re right, that line was incorrect. […] Done -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I5d1d25921514954c4929ae6e7352168b3ceb05df Gerrit-Change-Number: 42198 Gerrit-PatchSet: 7 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: Hoernchen <ewild(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Comment-Date: Mon, 02 Mar 2026 21:30:54 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: Timur Davydov <dtv.comp(a)gmail.com>

1 0

[S] Change in osmo-bsc[master]: nokia_site: Add new BTS types to the list
by Csaba Sipos 03 Mar '26

03 Mar '26

Csaba Sipos has posted comments on this change by Csaba Sipos. ( https://gerrit.osmocom.org/c/osmo-bsc/+/42246?usp=email ) Change subject: nokia_site: Add new BTS types to the list ...................................................................... Set Ready For Review -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/42246?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: Ia0dbbf148394d8205dc9219b41cfba3cf62bdeaa Gerrit-Change-Number: 42246 Gerrit-PatchSet: 1 Gerrit-Owner: Csaba Sipos <metro4(a)freemail.hu> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Mon, 02 Mar 2026 19:21:55 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No

1 0

[XL] Change in osmo-trx[master]: feat(usdr): add USDR backend support (osmo-trx-usdr)
by Timur Davydov 03 Mar '26

03 Mar '26

Attention is currently required from: Hoernchen, fixeria, laforge, osmith, pespin. Timur Davydov has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email ) Change subject: feat(usdr): add USDR backend support (osmo-trx-usdr) ...................................................................... Patch Set 7: (1 comment) File doc/manuals/chapters/trx-backends.adoc: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/050777ca_bb63c923?usp… : PS3, Line 86: where multiple small SDRs are combined to present one or more TRX instances > So in case it's not clear from previous comments, this documentation line is incorrect: […] You’re right, that line was incorrect. I’ve already updated the documentation to remove that sentence. -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I5d1d25921514954c4929ae6e7352168b3ceb05df Gerrit-Change-Number: 42198 Gerrit-PatchSet: 7 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: Hoernchen <ewild(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 02 Mar 2026 15:55:54 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: Timur Davydov <dtv.comp(a)gmail.com>

1 0

[M] Change in osmo-trx[master]: build: install common/GSM/transceiver libraries for emscripten and ad...
by pespin 03 Mar '26

03 Mar '26

Attention is currently required from: Hoernchen, Timur Davydov, fixeria, osmith. pespin has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42243?usp=email ) Change subject: build: install common/GSM/transceiver libraries for emscripten and add pkg-config files ...................................................................... Patch Set 2: (1 comment) Patchset: PS1: > For Emscripten I’m actually building them as static libraries. […] If we are linking dynamically against the stuff in CommonLibs/ and GSM/ then I'd say it's unintended. If that's the case, please submit a patch to always statically link those files into osmo-trx-*. I see no need for the .pc files since those object files are not expected to be used by other programs, they are internal to osmo-trx. -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42243?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I2495c5b5c79f2fe74bef83902058e1d2207c1f3c Gerrit-Change-Number: 42243 Gerrit-PatchSet: 2 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Comment-Date: Mon, 02 Mar 2026 15:13:32 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: Timur Davydov <dtv.comp(a)gmail.com>

1 0

[XL] Change in osmo-trx[master]: feat(usdr): add USDR backend support (osmo-trx-usdr)
by pespin 03 Mar '26

03 Mar '26

Attention is currently required from: Hoernchen, Timur Davydov, fixeria, laforge, osmith. pespin has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email ) Change subject: feat(usdr): add USDR backend support (osmo-trx-usdr) ...................................................................... Patch Set 7: (1 comment) File doc/manuals/chapters/trx-backends.adoc: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/f56387e0_33a4d303?usp… : PS3, Line 86: where multiple small SDRs are combined to present one or more TRX instances > An osmo-trx can serve multiple osmo-bts-trx transceivers (TRX) (see "chan" VTY ndoe and or "multi-ar […] So in case it's not clear from previous comments, this documentation line is incorrect: "where multiple small SDRs are combined to present one or more TRX instances". -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I5d1d25921514954c4929ae6e7352168b3ceb05df Gerrit-Change-Number: 42198 Gerrit-PatchSet: 7 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: Hoernchen <ewild(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Comment-Date: Mon, 02 Mar 2026 15:10:43 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: Timur Davydov <dtv.comp(a)gmail.com>

1 0

[XL] Change in osmo-trx[master]: feat(usdr): add USDR backend support (osmo-trx-usdr)
by pespin 03 Mar '26

03 Mar '26

Attention is currently required from: Hoernchen, Timur Davydov, fixeria, laforge, osmith. pespin has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email ) Change subject: feat(usdr): add USDR backend support (osmo-trx-usdr) ...................................................................... Patch Set 7: (5 comments) File Transceiver52M/device/usdr/USDRDevice.h: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/b35a4c7c_3fb1ea8b?usp… : PS4, Line 12: This program is distributed in the hope that it will be useful, > Fixed indentation. Done File Transceiver52M/device/usdr/USDRDevice.cpp: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/a4dd6af4_b9045e5d?usp… : PS4, Line 475: std::string USDRDevice::getTxAntenna(size_t chan ) > Now removed :) Done File debian/osmo-trx-usdr.install: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/1f38ce73_bf0c0523?usp… : PS4, Line 3: /usr/bin/osmo-trx-usdr > Removed the leading slashes to make it consistent with the other `*.install` files. Done File doc/manuals/chapters/trx-backends.adoc: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/70aa0278_2ca91958?usp… : PS3, Line 86: where multiple small SDRs are combined to present one or more TRX instances > Correct. So far, I've only tested one `osmo-trx-usdr` instance with one `osmo-bts-trx`. […] An osmo-trx can serve multiple osmo-bts-trx transceivers (TRX) (see "chan" VTY ndoe and or "multi-arfcn enable") in the same BTS. You cannot serve the same BTS from multiple osmo-trx processes nor service multiple BTS from the same osmot-rx process. File doc/manuals/chapters/trx-devices.adoc: https://gerrit.osmocom.org/c/osmo-trx/+/42198/comment/efce6c50_9f4d7ab6?usp… : PS3, Line 102: scale the number of TRX instances. > I’ve adjusted the documentation in this file. Done -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I5d1d25921514954c4929ae6e7352168b3ceb05df Gerrit-Change-Number: 42198 Gerrit-PatchSet: 7 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: Hoernchen <ewild(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Comment-Date: Mon, 02 Mar 2026 15:09:51 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: osmith <osmith(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: Timur Davydov <dtv.comp(a)gmail.com>

1 0

[S] Change in osmo-trx[master]: debian/*.install: make paths consistent
by pespin 03 Mar '26

03 Mar '26

Attention is currently required from: Timur Davydov, fixeria, osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-trx/+/42245?usp=email ) Change subject: debian/*.install: make paths consistent ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42245?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I4b9de40e1fc6d3935ae8ff82804d94cb863663e9 Gerrit-Change-Number: 42245 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Comment-Date: Mon, 02 Mar 2026 15:05:48 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-trx[master]: fix(threads): centralize portable strerror handling
by pespin 03 Mar '26

03 Mar '26

pespin has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42199?usp=email ) Change subject: fix(threads): centralize portable strerror handling ...................................................................... Patch Set 6: (3 comments) File CommonLibs/Threads.cpp: https://gerrit.osmocom.org/c/osmo-trx/+/42199/comment/dc1d0743_499dc82b?usp… : PS1, Line 58: char *err_str = NULL; > Fixed Done https://gerrit.osmocom.org/c/osmo-trx/+/42199/comment/7fdcb4b4_9960e6b4?usp… : PS1, Line 59: #if defined(__GLIBC__) && defined(_GNU_SOURCE) > I’ve moved this into Utils as a reusable strerror_buf() helper and added a C++ wrapper strerror_str( […] Done File CommonLibs/Utils.cpp: https://gerrit.osmocom.org/c/osmo-trx/+/42199/comment/809bb3a2_cbb4dd14?usp… : PS2, Line 36: > Replaced all `strerror_r()` with `strerror_buf()` Done -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42199?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I642aff8a9f98823e117c4debd19384ddf5975039 Gerrit-Change-Number: 42199 Gerrit-PatchSet: 6 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com> 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-Comment-Date: Mon, 02 Mar 2026 15:05:13 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: Timur Davydov <dtv.comp(a)gmail.com>

1 0

2026

2025

2024

2023

2022

gerrit-log March 2026