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/.
pespin gerrit-no-reply at lists.osmocom.orgpespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/17352 ) Change subject: trxcon/scheduler: substitute lost TDMA frames on Downlink ...................................................................... Patch Set 4: Code-Review-1 (2 comments) https://gerrit.osmocom.org/c/osmocom-bb/+/17352/4/src/host/trxcon/sched_trx.c File src/host/trxcon/sched_trx.c: https://gerrit.osmocom.org/c/osmocom-bb/+/17352/4/src/host/trxcon/sched_trx.c@622 PS4, Line 622: if (lchan->tdma.num_proc == 0) On num_proc overflow you are gonna hit this again. https://gerrit.osmocom.org/c/osmocom-bb/+/17352/4/src/host/trxcon/sched_trx.c@663 PS4, Line 663: lchan->tdma.num_proc++; you probably don't need num_proc at all but simply a boolean to know if at least one was processed. -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/17352 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I3551d79796a3730565c2c70577e9d134e636f275 Gerrit-Change-Number: 17352 Gerrit-PatchSet: 4 Gerrit-Owner: fixeria <axilirator at gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge at osmocom.org> Gerrit-Reviewer: pespin <pespin at sysmocom.de> Gerrit-Comment-Date: Mon, 09 Mar 2020 09:26:10 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200309/27802b43/attachment.htm>