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/.
laforge gerrit-no-reply at lists.osmocom.orglaforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gbproxy/+/25577 ) Change subject: gbproxy: Avoid sending STATUS on unexpected BLOCK_ACK ...................................................................... Patch Set 1: (2 comments) https://gerrit.osmocom.org/c/osmo-gbproxy/+/25577/1//COMMIT_MSG Commit Message: https://gerrit.osmocom.org/c/osmo-gbproxy/+/25577/1//COMMIT_MSG@12 PS1, Line 12: > why should it not "goto err_no_bvc" instead? […] I think unsolicited ACKs are typically ignored by many protocols. After all, they are notifying us about somethign we are not (or no longer) interested in. Not sure what the BSSGP spec says about it, but I could imagine that unsoliciated ACKs are to be ignored. https://gerrit.osmocom.org/c/osmo-gbproxy/+/25577/1/src/gb_proxy.c File src/gb_proxy.c: https://gerrit.osmocom.org/c/osmo-gbproxy/+/25577/1/src/gb_proxy.c@1378 PS1, Line 1378: break; > should it log an error here and/or increase the rate counter? it is indeed a bit inconsistent that an UNBLOCK_ACK or RESET_ACK are now handled significantly different than a BLOCK_ACK... -- To view, visit https://gerrit.osmocom.org/c/osmo-gbproxy/+/25577 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gbproxy Gerrit-Branch: master Gerrit-Change-Id: Ic9f34a27412d6e15ca1198ee140f66a076b5c6b6 Gerrit-Change-Number: 25577 Gerrit-PatchSet: 1 Gerrit-Owner: daniel <dwillmann at sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge at osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu> Gerrit-CC: osmith <osmith at sysmocom.de> Gerrit-Comment-Date: Fri, 24 Sep 2021 19:51:42 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: osmith <osmith at sysmocom.de> Gerrit-MessageType: comment -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210924/59941f0a/attachment.htm>