This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Neels Hofmeyr gerrit-no-reply at lists.osmocom.orgReview at https://gerrit.osmocom.org/1777 fix missing ~ in bit logic for lchan->si.valid in rsl_rx_sacch_inf_mod() Upon rx of a SACCH INFO Modify for an lchan that has no L3 INFO IE, clear only the si.valid bit in question, instead of clearing all but the one in question. BTW, It first looked like it could be, but is *not* the cause for OS#1945 (SI5ter never sent to MS). Change-Id: I0df20b6643b0bfd219ce1df594075838d8406719 --- M src/common/rsl.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/77/1777/1 diff --git a/src/common/rsl.c b/src/common/rsl.c index 5c4e4ff..7ab43cd 100644 --- a/src/common/rsl.c +++ b/src/common/rsl.c @@ -1345,7 +1345,7 @@ gsm_lchan_name(lchan), get_value_string(osmo_sitype_strs, osmo_si)); } else { - lchan->si.valid &= (1 << osmo_si); + lchan->si.valid &= ~(1 << osmo_si); LOGP(DRSL, LOGL_INFO, "%s Rx RSL Disabling SACCH FILLING (SI%s)\n", gsm_lchan_name(lchan), get_value_string(osmo_sitype_strs, osmo_si)); -- To view, visit https://gerrit.osmocom.org/1777 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0df20b6643b0bfd219ce1df594075838d8406719 Gerrit-PatchSet: 1 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>